-
Notifications
You must be signed in to change notification settings - Fork 1.4k
fix: add slash command hints in workflow completion messages #603
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
When artifacts or tasks are complete, the command templates now suggest specific slash commands (/opsx:apply, /opsx:archive) instead of generic guidance, helping users discover the next workflow step. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
📝 WalkthroughWalkthroughThe Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 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 OverviewGreptile SummaryUpdated workflow completion messages in
This improves discoverability by showing users the exact commands to run for the next workflow step. Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant Agent as Claude Agent
participant CLI as OpenSpec CLI
Note over User,CLI: Scenario 1: /opsx:continue completion
User->>Agent: /opsx:continue
Agent->>CLI: openspec status --change "name" --json
CLI-->>Agent: {isComplete: true, artifacts: [...]}
Agent->>Agent: Detects all artifacts complete
Agent->>User: "All artifacts created! You can now implement<br/>this change with `/opsx:apply` or archive<br/>it with `/opsx:archive`."
Note over User,CLI: Scenario 2: /opsx:apply completion
User->>Agent: /opsx:apply
Agent->>CLI: openspec instructions apply --change "name" --json
CLI-->>Agent: {tasks: [...], state: "all_done"}
Agent->>Agent: Implements all tasks
Agent->>User: "All tasks complete! You can archive<br/>this change with `/opsx:archive`."
|
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: 2
🤖 Fix all issues with AI agents
In `@src/core/templates/skill-templates.ts`:
- Around line 1954-1955: Update the final user-facing message in
getApplyChangeSkillTemplate() to match the counterpart by replacing "All tasks
complete! Ready to archive this change." with the consistent hint "All tasks
complete! You can archive this change with `/opsx:archive`." Locate the string
inside the getApplyChangeSkillTemplate function and change only the message text
so formatting and punctuation match the other template.
- Around line 1765-1766: The getContinueChangeSkillTemplate() function uses an
older generic success message; update its suggested message string to match the
other template by including the slash command hints—replace the current text
"All artifacts created! You can now implement this change or archive it." with
"All artifacts created! You can now implement this change with `/opsx:apply` or
archive it with `/opsx:archive`." to keep the UX consistent across templates.
Review CompleteYour review story is ready! Comment !reviewfast on this PR to re-generate the story. |
When artifacts or tasks are complete, the command templates now suggest specific slash commands (/opsx:apply, /opsx:archive) instead of generic guidance, helping users discover the next workflow step.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.