Replace stale 'supermodel watch' references with 'supermodel'#150
Replace stale 'supermodel watch' references with 'supermodel'#150jonathanpopham merged 1 commit intomainfrom
Conversation
The 'watch' subcommand was removed in #60 — running 'supermodel [path]' is now the daemon. The setup wizard's exit message and several spots in the README still told users to run 'supermodel watch', which fails with 'unknown command'. Reported in #149. - internal/setup/wizard.go:133 — 'just run `supermodel watch`' → '`supermodel`' - internal/setup/wizard.go:194 — 'Run `supermodel watch` to restart' → '`supermodel`' - README.md — 'How it works', the agent compatibility table, and Quick start all updated to use the bare command. Closes #149.
WalkthroughThis PR fixes stale command references across documentation and UI text. After a previous change made Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Review rate limit: 4/5 reviews remaining, refill in 12 minutes. Comment |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
README.md (1)
116-116:⚠️ Potential issue | 🟡 MinorRemove the
watchcommand from the Commands table—it no longer exists.The search across the codebase turned up zero results for any
watchsubcommand registration. This confirms the PR description: thewatchcommand was removed and no longer exists. Since line 116 of README.md still documents it, users would try to runsupermodel watchand get an "unknown command" error, which would be confusing. Delete this line from the documentation.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` at line 116, Remove the stale "watch [path]" entry from the Commands table in README.md (the table row containing `watch [path] | Generate graph files on startup, then keep them updated incrementally`) since the `watch` subcommand no longer exists; update the table so its row is deleted and formatting/column alignment remains correct.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@README.md`:
- Line 116: Remove the stale "watch [path]" entry from the Commands table in
README.md (the table row containing `watch [path] | Generate graph files on
startup, then keep them updated incrementally`) since the `watch` subcommand no
longer exists; update the table so its row is deleted and formatting/column
alignment remains correct.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: c630ff8a-e2ba-4562-9932-6cd53622d20e
📒 Files selected for processing (2)
README.mdinternal/setup/wizard.go
Closes #149.
The `watch` subcommand was removed in #60 ("Make watch the default command"). Running `supermodel [path]` is now the daemon, but the setup wizard's exit message and several spots in the README still told users to run `supermodel watch` — which fails with `Error: unknown command "watch" for "supermodel"`.
Changes
Test plan
Summary by CodeRabbit
supermodelinstead ofsupermodel watchacross all agent-specific setup guides and quickstart examples.