Skip to content

Code review - 18 Jul 2025 #2

@sanand0

Description

@sanand0

pyproject.toml

  • Create it. Ensure that uv run git+https://... repo will run app.py

README.md

  • Lint with Prettier
  • Reduce verbosity - get usage near top
  • Add screenshot

app.py

  • Lint with Ruff
  • Open http://localhost:8000 via browser automatically where possible
  • Prefer httpx over aiohttp
  • Don't send the data payload every time. The objective is to wokr with local data. Save uploaded files in temporary storage
  • Reduce code size. Drop redundant code (e.g. _generate_summary). Consider creating a config.toml for prompts and other data structures.
  • Consider moving any logic possible to JS, e.g. all LLM calls. We need Python only to access the dataset, run code, etc.

Testing

  • Add automated unit tests for API using pytest

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions