Conversation
The Python CLI's dev mode is deprecated and its in-process auto-reload has been removed, so `lk agent dev` is now the only way to get hot reload. Point the README, the taskfile help output, and the `dev` task at the LiveKit CLI for all three local modes, and move the CLI install instructions into Dev Setup so they come before the first command that needs them. The Dockerfile still starts the deployed agent with `uv run src/agent.py start`, since the container image doesn't include `lk`. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U7yszNU7t5aCHJsKLmU4hv
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U7yszNU7t5aCHJsKLmU4hv
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U7yszNU7t5aCHJsKLmU4hv
Mirrors the Node starter's review round: drop guidance a coding agent can work out on its own, show the direct `uv run src/agent.py start` for production since that's what the Dockerfile runs, and collapse the CLI install block into a three-line list with a link to update instructions. Also a copy pass against the docs style guide: sentence-case headings, no "we", no hyphens standing in for dashes, bulleted rather than numbered non-sequential lists, and less filler throughout. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U7yszNU7t5aCHJsKLmU4hv
The README feature list now says what each piece does for you and links the docs for how: Fish Audio as an expressive, cost-effective default, expressive mode, and adaptive interruption handling, which the starter enables but the list never mentioned. "Using coding agents" points at the bundled AGENTS.md and livekit-agents skill, and the testing section adds Agent Console and `lk agent logs` for debugging. AGENTS.md gains a section on the bundled skill (with a fallback path for tools that do not load skills on their own) and a debugging checklist. Links to the moved testing, observability, and workflows pages now point at their current paths. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U7yszNU7t5aCHJsKLmU4hv
Search returns short excerpts, so reading a result means fetching the full page. Show both commands in the README, and tell coding agents in AGENTS.md to fetch the page before acting on a search result. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U7yszNU7t5aCHJsKLmU4hv
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #103, carrying over the review rounds on the Node starter (livekit-examples/agent-starter-node#65) so the two READMEs stay in step.
What
uv, andrufffor formatting).livekit-agentsskill, with a fallback path for tools that don't load skills on their own, and a debugging checklist.lk docs searchfollowed bylk docs get-page, since search only returns excerpts, and points to the bundledAGENTS.mdand skill.git cloneit was missing.lk agent consoleandlk agent dev, then the directuv run src/agent.py startfor production. That's what the Dockerfile runs, so the old "the container doesn't includelk" caveat goes away.lk agent logs.Notes for reviewers
get-page.lk agentfor console, dev, and start #103's two commits and a no-op merge. The diff againstmainis only these two files, and a squash-merge keepsmainclean.docs.livekit.iolink in both files returns 200 without a redirect.🤖 Generated with Claude Code
https://claude.ai/code/session_01U7yszNU7t5aCHJsKLmU4hv