-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add /opsx:bulk-archive to experimental workflow setup command output #551
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. 📝 WalkthroughWalkthroughAdds a single CLI help entry for the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile Summary
Important Files Changed
Confidence score: 5/5
Sequence DiagramsequenceDiagram
participant User
participant CLI as "OpenSpec CLI"
participant FS as "FileSystem"
participant Templates as "Skill Templates"
User->>CLI: "openspec artifact-experimental-setup"
CLI->>CLI: "Start spinner: Setting up experimental artifact workflow"
CLI->>Templates: "getExploreSkillTemplate()"
Templates-->>CLI: "explore skill template"
CLI->>Templates: "getNewChangeSkillTemplate()"
Templates-->>CLI: "new change skill template"
CLI->>Templates: "getContinueChangeSkillTemplate()"
Templates-->>CLI: "continue change skill template"
CLI->>Templates: "getApplyChangeSkillTemplate()"
Templates-->>CLI: "apply change skill template"
CLI->>Templates: "getFfChangeSkillTemplate()"
Templates-->>CLI: "fast-forward skill template"
CLI->>Templates: "getSyncSpecsSkillTemplate()"
Templates-->>CLI: "sync specs skill template"
CLI->>Templates: "getArchiveChangeSkillTemplate()"
Templates-->>CLI: "archive change skill template"
CLI->>Templates: "getBulkArchiveChangeSkillTemplate()"
Templates-->>CLI: "bulk archive change skill template"
CLI->>Templates: "getVerifyChangeSkillTemplate()"
Templates-->>CLI: "verify change skill template"
loop For each skill template
CLI->>FS: "writeFile(.claude/skills/<skill>/SKILL.md)"
FS-->>CLI: "skill file created"
end
CLI->>Templates: "getOpsxExploreCommandTemplate()"
Templates-->>CLI: "explore command template"
CLI->>Templates: "getOpsxNewCommandTemplate()"
Templates-->>CLI: "new command template"
CLI->>Templates: "getOpsxContinueCommandTemplate()"
Templates-->>CLI: "continue command template"
CLI->>Templates: "getOpsxApplyCommandTemplate()"
Templates-->>CLI: "apply command template"
CLI->>Templates: "getOpsxFfCommandTemplate()"
Templates-->>CLI: "fast-forward command template"
CLI->>Templates: "getOpsxSyncCommandTemplate()"
Templates-->>CLI: "sync command template"
CLI->>Templates: "getOpsxArchiveCommandTemplate()"
Templates-->>CLI: "archive command template"
CLI->>Templates: "getOpsxBulkArchiveCommandTemplate()"
Templates-->>CLI: "bulk archive command template"
CLI->>Templates: "getOpsxVerifyCommandTemplate()"
Templates-->>CLI: "verify command template"
loop For each command template
CLI->>FS: "writeFile(.claude/commands/opsx/<command>.md)"
FS-->>CLI: "command file created"
end
CLI->>FS: "writeFile(openspec/config.yaml)"
FS-->>CLI: "config file created"
CLI->>CLI: "Stop spinner with success message"
CLI->>User: "Display setup completion report with created files and usage instructions"
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 file reviewed, 1 comment
src/commands/artifact-workflow.ts
Outdated
| console.log(' • /opsx:sync - Sync delta specs to main specs'); | ||
| console.log(' • /opsx:verify - Verify implementation matches artifacts'); | ||
| console.log(' • /opsx:archive - Archive a completed change'); | ||
| console.log(' • /opsx:bulk-archive - Archive mulitple completed changes'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
syntax: Typo: 'mulitple' should be 'multiple'
| console.log(' • /opsx:bulk-archive - Archive mulitple completed changes'); | |
| console.log(' • /opsx:bulk-archive - Archive multiple completed changes'); |
Prompt To Fix With AI
This is a comment left during a code review.
Path: src/commands/artifact-workflow.ts
Line: 1028:1028
Comment:
**syntax:** Typo: 'mulitple' should be 'multiple'
```suggestion
console.log(' • /opsx:bulk-archive - Archive multiple completed changes');
```
How can I resolve this? If you propose a fix, please make it concise.
Review CompleteYour review story is ready! Comment !reviewfast on this PR to re-generate the story. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds the missing /opsx:bulk-archive command to the experimental workflow setup output, completing the list of available slash commands that was introduced in version 0.23.0.
Changes:
- Added
/opsx:bulk-archivecommand description to the setup command's slash commands list
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@src/commands/artifact-workflow.ts`:
- Line 1028: Update the user-facing help text string passed to console.log that
currently reads " • /opsx:bulk-archive - Archive mulitple completed changes" to
correct the typo "mulitple" to "multiple"; locate the console.log invocation in
artifact-workflow.ts (the help/usage output section) and replace the misspelled
word in that string literal so the message reads "Archive multiple completed
changes".
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Version 0.23.0 added the experimental /opsx:bulk-archive command but it was omitted from the setup's list of slash commands.
Summary by CodeRabbit
/opsx:bulk-archivecommand in the CLI help output. This update is informational only—no changes to command behavior or execution.✏️ Tip: You can customize this high-level summary in your review settings.