Skip to content

feat(backend): add FastAPI service#48

Open
gac0812 wants to merge 2 commits into
1024XEngineer:mainfrom
gac0812:feat/backend-service
Open

feat(backend): add FastAPI service#48
gac0812 wants to merge 2 commits into
1024XEngineer:mainfrom
gac0812:feat/backend-service

Conversation

@gac0812

@gac0812 gac0812 commented Jul 16, 2026

Copy link
Copy Markdown

Summary

  • add the FastAPI service under backend/
  • include API health testing, environment defaults, and quality-tool configuration
  • add repository ignore rules for Python and Expo generated files

Validation

  • uv run pytest
  • uv run ruff check .
  • uv run mypy

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review Summary

Found one maintainability issue in the test dependency setup. The health test, Ruff, mypy, locked sync, and package build otherwise completed successfully.

View job run

Comment thread backend/pyproject.toml Outdated

[dependency-groups]
dev = [
"httpx",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[P3] Use the supported Starlette test-client dependency

The locked Starlette 1.3.1 prefers httpx2, so the current httpx dependency makes every test run emit StarletteDeprecationWarning; it also fails collection when warnings are treated as errors. Replacing this with httpx2 and refreshing uv.lock removes the warning and keeps the test setup forward-compatible.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in gac0812@ae56f1d. Replaced the deprecated httpx test dependency with httpx2 and refreshed uv.lock. Verified Ruff, formatting, mypy, and uv run pytest -W error (1 passed with no warning).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant