Skip to content

docs: add link and manual instructions for restaurant finder#1184

Merged
jacobsimionato merged 1 commit into
mainfrom
docs/restaurant-finder-instructions
Apr 15, 2026
Merged

docs: add link and manual instructions for restaurant finder#1184
jacobsimionato merged 1 commit into
mainfrom
docs/restaurant-finder-instructions

Conversation

@zeroasterisk
Copy link
Copy Markdown
Collaborator

Follow-up to PR #1181. Adds a link to the restaurant finder agent source and provides manual running instructions.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the quickstart documentation by adding a section for running the Restaurant Finder agent and client manually. The feedback suggests adding an explicit npm install command to the client setup instructions to ensure all necessary dependencies are installed before starting the development server.

Comment thread docs/quickstart.md
Comment on lines +79 to +80
cd samples/client/lit/shell
npm run dev
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Include an npm install step in the manual instructions to ensure dependencies are available, as npm run does not automatically install them (unlike uv run for the agent). This is especially important if the user skipped the automatic demo command in Step 4.

Suggested change
cd samples/client/lit/shell
npm run dev
cd samples/client/lit/shell
npm install
npm run dev

@jacobsimionato jacobsimionato merged commit 2db8840 into main Apr 15, 2026
11 checks passed
@jacobsimionato jacobsimionato deleted the docs/restaurant-finder-instructions branch April 15, 2026 05:00
@github-project-automation github-project-automation Bot moved this from Todo to Done in A2UI Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants