Adding Flutter quickstart.#1935
Conversation
Fixed naming issue to make `flutter run ...` work again to match instructions in readmes. Changed agent default to 3.1-flash-lite which is less throttled during times of high usage.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a new Flutter quickstart guide, updates the default Gemini model in the restaurant finder agent to gemini-3.1-flash-lite, and corrects an import path in the end-to-end tests. The review feedback highlights several issues in the new documentation, such as an unresolved placeholder in a directory path, incorrect step numbering, a copy-paste error referring to the Lit client, an inapplicable troubleshooting section about port 5173, and a typo ('Fluttr').
|
|
||
| ### Other Languages and Frameworks | ||
|
|
||
| While this guide uses the Lit client as an example, A2UI provides samples for other popular frameworks in the `samples/client` directory: |
There was a problem hiding this comment.
This guide is for the Flutter client, but the text mentions "While this guide uses the Lit client as an example". This appears to be a copy-paste error from the Lit quickstart and should be updated to refer to the Flutter client.
While this guide uses the Flutter client as an example, A2UI provides samples for other popular frameworks in the `samples/client` directory:| ### Port Already in Use | ||
|
|
||
| If port 5173 is already in use, the dev server will automatically try the next available port. Check the terminal output for the actual URL. |
There was a problem hiding this comment.
This troubleshooting section mentions port 5173, which is the default port for Vite-based clients (like Lit, React, or Angular). Since this guide is for the Flutter client, which does not use Vite or port 5173, this section is not applicable and should be removed.
| Want to see how it works? Check out: | ||
|
|
||
| - **Agent Code**: `samples/agent/adk/restaurant_finder/` — The Python A2A agent | ||
| - **Client Code**: `samples/client/flutter/` — The Fluttr web client with A2UI renderer |
There was a problem hiding this comment.
There is a typo in this line: "Fluttr" should be "Flutter".
- **Client Code**: `samples/client/flutter/` — The Flutter web client with A2UI renderer
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> Signed-off-by: James Williams <66931+jwill@users.noreply.github.com>
Signed-off-by: James Williams <66931+jwill@users.noreply.github.com>
Removed troubleshooting section about port usage. Signed-off-by: James Williams <66931+jwill@users.noreply.github.com>
Signed-off-by: James Williams <66931+jwill@users.noreply.github.com>
|
Thanks @jwill! I'll take a look at this later today/tomorrow! |
2 participants
Description
Adds flutter version of the quickstart per eng request. Fixed naming issue to make
flutter run ...work again to match instructions in readmes. Changed agent model fallthrough to 3.1-flash-lite which is less throttled during times of high usage.List which issues are fixed by this PR. For larger changes, raising an issue first helps reduce redundant work.
Pre-launch Checklist
One time:
For this PR:
If you need help, consider asking for advice on the discussion board.