fix: correct CLI command from 'sonar ingest monitor' to 'sonar monitor'#9
Conversation
The monitor subcommand lives at the top-level (sonar monitor), not under the ingest namespace (sonar ingest monitor). Fix all user-facing references in diagnostic tips, error messages, and the README.
📝 WalkthroughWalkthroughUpdated CLI command references across documentation and user-facing messaging, replacing deprecated "sonar ingest monitor" commands with the new "sonar monitor" command syntax consistently. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
README.md (1)
36-57: Pre-existing README quality nits (out of scope for this PR but worth tracking)A few pre-existing issues unrelated to this fix:
- Line 36:
evrything→everything- Line 44:
the first time this you run→the first time you run- Line 57: malformed link
<<https://.../>— extra<and/>- Line 73:
actionalable→actionableNone of these were introduced by this PR.
Would you like me to open a separate issue to track these README fixes?
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` around lines 36 - 57, Spelling and formatting issues in README.md: correct "evrything" to "everything" in the line near the "sonar account" example, change "the first time this you run" to "the first time you run" near the "sonar ingest tweets" / "sonar monitor" examples, fix the malformed link by removing the extra angle brackets so the URL is a normal Markdown link under the "Scopes" section, and fix "actionalable"/"appite" to "actionable" where it appears; scan the surrounding text for similar typos and update them so the commands and headings ("sonar account", "sonar ingest tweets", "Scopes") read correctly.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@README.md`:
- Around line 36-57: Spelling and formatting issues in README.md: correct
"evrything" to "everything" in the line near the "sonar account" example, change
"the first time this you run" to "the first time you run" near the "sonar ingest
tweets" / "sonar monitor" examples, fix the malformed link by removing the extra
angle brackets so the URL is a normal Markdown link under the "Scopes" section,
and fix "actionalable"/"appite" to "actionable" where it appears; scan the
surrounding text for similar typos and update them so the commands and headings
("sonar account", "sonar ingest tweets", "Scopes") read correctly.
ℹ️ Review info
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
README.mdsrc/commands/feed.tsxsrc/commands/inbox/index.tsxsrc/commands/ingest/bookmarks.tsxsrc/commands/ingest/tweets.tsx
|
Acknowledged, @coderabbitai — those typos ( |
|
Tip For best results, initiate chat on the files or code changes.
|
2 participants
Summary
The
monitorcommand is a top-level subcommand (sonar monitor), not nested underingest(sonar ingest monitor). This PR corrects all user-facing references to the wrong form.Changes
README.mdsrc/commands/ingest/bookmarks.tsxsrc/commands/ingest/tweets.tsxsrc/commands/feed.tsx--watchvariant)src/commands/inbox/index.tsx10 occurrences corrected across 5 files — all replaced
sonar ingest monitor→sonar monitor(andsonar ingest monitor --watch→sonar monitor --watch).Summary by CodeRabbit