Repository navigation
Show /sync and the profile name in the README flow - #42
Merged
samyakLambda merged 1 commit intoOct 6, 2026
Merged
Conversation
Review of the refreshed README (AA-460) found /sync missing from the user flow and /profile add missing its name. Both match the released CLI: profile add refuses without a name, and the CLI's own sequence is profile add, sync, run. Add /sync to the command sequence, the quick-start transcript (with the lines a first sync prints) and the command table; name the profile in /profile add. The CI block also gains `rook sync`, because `rook run` refuses an agent that has never been synced; `rook run --test` is the local alternative. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Fixes two points from the review of the refreshed README (AA-460):
/syncwas missing from the user flow. It is now in the command sequence, the quick-start transcript and the command table. The CLI's own order is/explore → /generate → /profile add → /sync → /run. The transcript lines are what a first sync prints:recording 1 agent(s),<agent>: new upstream — recording everything,<agent>: recorded./profile addwas missing its name. The CLI refusesprofile addwithout one ("profile add needs a name"), so the sequence now shows/profile add <name>and the quick start uses/profile add local.The same gap affected In CI:
rook runrefuses an agent that has never been synced ("rook sync — record it, then run"). The CI block now includesrook sync, and one sentence namesrook run --testfor a local-only result. This matches the CI recipe.Checked against the released CLI (v0.1.6). Prettier-clean; the only other diff is table re-padding.
🤖 Generated with Claude Code