docs(cli): sync command reference with implemented CLI surface#901
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughUpdated Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 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 unit tests (beta)
Comment |
prekshivyas
left a comment
There was a problem hiding this comment.
Nice work — this covers the gaps in the command reference well. All commands and flags verified against the codebase.
Two minor suggestions:
-
debug --sandbox NAME:debug.shalso accepts--sandbox NAMEto target a specific sandbox (line 45-47 inscripts/debug.sh). It's not shown in the mainnemoclaw helpoutput, but since this is the full CLI reference, it might be worth documenting here too. -
Ordering: The deprecated
setupsection sits betweenonboardandlist. Consider moving it toward the end (before or afteruninstall) so users see current commands first.
Neither is a blocker — the PR is solid as-is.
|
Addressing the two reviewer suggestions in c695fdc:
@coderabbitai review |
|
✅ Actions performedReview triggered.
|
|
Docs sync is up to date on this branch and the follow-up reviewer suggestions are already addressed in |
2 similar comments
|
Docs sync is up to date on this branch and the follow-up reviewer suggestions are already addressed in |
|
Docs sync is up to date on this branch and the follow-up reviewer suggestions are already addressed in |
|
@WuKongAI-CMU pls resolve merge conflicts |
c695fdc to
f2f26a2
Compare
|
Rebased onto latest main and resolved conflicts in |
Signed-off-by: peteryuqin <peter.yuqin@gmail.com>
f2f26a2 to
6950fd4
Compare
* docs(cli): sync command reference with implemented CLI surface Signed-off-by: peteryuqin <peter.yuqin@gmail.com> * docs(cli): document debug sandbox flag --------- Signed-off-by: peteryuqin <peter.yuqin@gmail.com>
…A#901) * docs(cli): sync command reference with implemented CLI surface Signed-off-by: peteryuqin <peter.yuqin@gmail.com> * docs(cli): document debug sandbox flag --------- Signed-off-by: peteryuqin <peter.yuqin@gmail.com>
…A#901) * docs(cli): sync command reference with implemented CLI surface Signed-off-by: peteryuqin <peter.yuqin@gmail.com> * docs(cli): document debug sandbox flag --------- Signed-off-by: peteryuqin <peter.yuqin@gmail.com>
Summary
/nemoclawslash-command surface, includingonboardandejectsetup,debug, anduninstallTesting
Fixes #758
Summary by CodeRabbit