Modernize Flutter/FastAPI template tooling#3
Conversation
There was a problem hiding this comment.
Code Review
This pull request modernizes the project's development environment and dependencies. It replaces the custom Dockerfile setup with a standard Dev Container configuration, adopts uv for Python dependency management, updates Android build configurations to Kotlin DSL, and refactors Dart code to use modern syntax and updated packages. The review feedback highlights a safety issue in lib/app/app_view.dart, where r.body['n'] is accessed directly without checking if the API request succeeded or if the response body is null, which could lead to application crashes.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 02e81b813e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Summary
uv.lock, and refresh FastAPI dependencies.--platform=linux/amd64.astral-sh/setup-uvtov8.2.0because it does not publish av8tag.packagedeclarations.Why
This brings the four-year-old template closer to current Flutter/FastAPI development practice while keeping GitHub Codespaces as the baseline devcontainer environment. The devcontainer intentionally avoids Cirrus Labs because that image line is being sunset, and it does not force a Docker platform; Apple Silicon/arm64 users are expected to use a local toolchain instead of the devcontainer.
Commits
f8455c1chore: migrate backend dependencies to uv1c01a13chore: modernize devcontainer setup0faeb92feat: update Flutter template dependenciesc34279echore: modernize Android Gradle configuration8587ba5ci: refresh Flutter and uv workflowsf119df4docs: update setup instructionse918ab2fix: align devcontainer target with Codespaces02e81b8ci: use major GitHub Action versions5b4efb2ci: pin setup-uv action versionfff4f4ffix: guard API counter response12de35achore: remove Android manifest package declarations5f4d03bchore: enable Flutter web in devcontainerCI Fix
The failed job at https://github.com/comfuture/flutter_fastapi/actions/runs/28455329537/job/84328425152 failed while resolving
astral-sh/setup-uv@v8. Theastral-sh/setup-uvrepository hasv8.0.0,v8.1.0, andv8.2.0tags, but nov8tag, so this PR pins that action tov8.2.0.Review Follow-up
api.inc()response handling before readingn.packageattributes so the Gradle namespace remains the Android package source of truth.web-serverlaunch configuration.Validation
flutter analyzeflutter testflutter build apk --debuguv sync --frozenuv run python -c "from server import app; print(app.title)"git diff --checkgmeligio/flutter-android:3.44.4flutter config --enable-web && flutter precache --webworks as theflutteruser