-
-
Notifications
You must be signed in to change notification settings - Fork 0
chore: consolidate slash commands from 26 to 14 #310
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
Conversation
## Command Changes ### New Commands - /design - consolidated from design-session + new-feature - /design-ui - reference-driven UI design workflow - /test - unified testing with auto-detection - /spec - contributor-ready implementation guides - /setup - merged from setup-ecosystem + setup-terminal ### Enhanced Commands - /ship - now handles CI-fix and bot-comments autonomously - /start-work - adds session status file for orchestration ### Retired Commands - design-session, new-feature → /design - fix-tests, test-cli, tui-test, run-integration-local → /test - pre-pr, debug-ci-failure, review-bot-comments → /ship - setup-ecosystem, setup-terminal → /setup - next-work, create-worktree → subsumed by plan-work - handoff, retrospective → auto-retrospective (future) - tui-polish → replaced by /design-ui flow ## New Workflow Documentation Added .claude/workflows/ with process docs: - parallel-work.md - orchestration model - autonomous-session.md - worker session flow - human-gates.md - when to escalate - ui-development.md - reference-driven UI design Co-Authored-By: Claude Opus 4.5 <[email protected]>
Summary of ChangesHello @joshsmithxrm, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request represents a significant effort to refine and optimize the development experience by consolidating and enhancing the available slash commands. The primary goal is to simplify interactions, automate routine tasks, and provide a more structured approach to development through new unified commands and detailed workflow documentation. These changes aim to improve efficiency, reduce manual steps, and ensure clearer communication within the development process. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
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.
Code Review
This pull request is an excellent and significant step forward in streamlining the development workflow. The consolidation of 26 slash commands into 14 more powerful and intuitive commands is a great improvement. The new commands like /design, /test, and the enhanced /ship with its autonomous capabilities are particularly impressive. The addition of comprehensive workflow documentation in .claude/workflows/ provides much-needed clarity on the new orchestration model and human-in-the-loop gates. My review found the changes to be well-structured and internally consistent. I have one suggestion regarding the removal of a bash script in the /setup command, which appears to reduce cross-platform support for a developer experience feature.
I am having trouble creating individual review comments. Click here to see my feedback.
.claude/commands/setup-ecosystem.md (160-178)
The bash script for setting up the status line on macOS/Linux has been removed. This drops a useful feature for developers on non-Windows systems who may not use PowerShell as their primary shell. Was this intentional? If cross-platform support is still desired for this feature, it would be beneficial to retain the bash script and the corresponding setup 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.
Pull request overview
This PR consolidates 26 slash commands down to 14 for a streamlined workflow, adds comprehensive workflow documentation in .claude/workflows/, and implements file-based session orchestration for parallel work management (V1).
Key Changes:
- Consolidates testing commands (
fix-tests,test-cli,tui-test,run-integration-local) into unified/testcommand with auto-detection - Merges planning commands (
design-session,new-feature) into/designand adds new/design-uifor reference-driven UI design - Enhances
/shipto autonomously handle CI failures and bot comments (absorbingpre-pr,debug-ci-failure,review-bot-comments) - Introduces session orchestration via status files in
~/.ppds/sessions/with clear human injection points
Reviewed changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
CLAUDE.md |
Updates command reference table to list 14 consolidated commands with references to new workflow documentation |
.claude/workflows/ui-development.md |
Documents reference-driven UI design process for TUI and VS Code with wireframe-first approach |
.claude/workflows/parallel-work.md |
Describes file-based session orchestration pattern enabling multiple parallel Claude sessions |
.claude/workflows/human-gates.md |
Defines when autonomous sessions should escalate to human decision-making (design, security, performance) |
.claude/workflows/autonomous-session.md |
Documents worker session lifecycle from /start-work through implementation to /ship with PR creation |
.claude/commands/test.md |
New unified testing command with auto-detection, consolidating 4 previous testing commands |
.claude/commands/ship.md |
Enhanced to autonomously handle full shipping flow including CI-fix and bot comment resolution |
.claude/commands/setup.md |
Consolidated setup command absorbing setup-ecosystem and setup-terminal |
.claude/commands/start-work.md |
Enhanced to create session status files for orchestrator monitoring |
.claude/commands/spec.md |
New command for generating contributor-ready implementation guides from triaged issues |
.claude/commands/design.md |
New consolidated design command replacing design-session and new-feature |
.claude/commands/design-ui.md |
New command for reference-driven UI design with wireframes |
.claude/commands/tui-test.md |
Removed - functionality absorbed into /test command |
.claude/commands/tui-polish.md |
Removed - replaced by /design-ui flow |
.claude/commands/test-cli.md |
Removed - functionality absorbed into /test command |
.claude/commands/run-integration-local.md |
Removed - functionality absorbed into /test --integration |
.claude/commands/fix-tests.md |
Removed - functionality absorbed into /test iterative fix loop |
.claude/commands/pre-pr.md |
Removed - functionality absorbed into /ship |
.claude/commands/retrospective.md |
Removed - to be replaced by auto-retrospective hook (future) |
.claude/commands/handoff.md |
Removed - subsumed by session status files |
.claude/commands/new-feature.md |
Removed - functionality absorbed into /design |
.claude/commands/design-session.md |
Removed - replaced by /design |
.claude/commands/next-work.md |
Removed - subsumed by /plan-work |
.claude/commands/setup-terminal.md |
Removed - functionality absorbed into /setup --terminal |
.claude/commands/create-worktree.md |
Removed - subsumed by /plan-work |
|
|
||
| 2. **Implement TUI first** | ||
| - Build with proper abstraction | ||
| - Test with `/tui-test` |
Copilot
AI
Jan 9, 2026
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.
The reference to /tui-test should be updated to /test --tui since the /tui-test command has been retired and replaced by the unified /test command.
| - Test with `/tui-test` | |
| - Test with `/test --tui` |
| | Command | Purpose | | ||
| |---------|---------| | ||
| | `/ship` | Commit, push, create PR (runs tests internally) | | ||
| | `/pre-pr` | Full validation (absorbed into /ship) | |
Copilot
AI
Jan 9, 2026
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.
The reference to /pre-pr is outdated since that command has been retired and its functionality absorbed into /ship. Consider removing this table row or updating the description to clarify the historical context.
| | `/pre-pr` | Full validation (absorbed into /ship) | |
| Add the spec as a comment on the issue and add `good first issue` label if appropriate: | ||
|
|
||
| ```bash | ||
| gh issue comment <number> --body "<spec content>" |
Copilot
AI
Jan 9, 2026
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.
The gh issue comment <number> --body "<spec content>" pattern is vulnerable to command injection because <spec content> will include untrusted data such as the GitHub issue title and potentially parts of the body. If this content is interpolated directly into a shell command within double quotes, an attacker can craft an issue title/body containing shell metacharacters or command substitutions (e.g., $(...)) to execute arbitrary commands on the developer’s machine when /spec runs. To fix this, avoid inlining the spec text into a shell command string: either pass it to gh via a safe API (arguments array) or --body-file, or ensure all untrusted content is robustly escaped before being used in the shell.
| gh issue comment <number> --body "<spec content>" | |
| # Save the generated spec to a file, e.g. spec.md, then: | |
| gh issue comment <number> --body-file spec.md |
Summary
.claude/workflows/Command Changes
New Commands
/design/design-ui/test/spec/setupEnhanced Commands
/ship/start-workRetired Commands (14)
design-session,new-feature→/designfix-tests,test-cli,tui-test,run-integration-local→/testpre-pr,debug-ci-failure,review-bot-comments→/shipsetup-ecosystem,setup-terminal→/setupnext-work,create-worktree→ subsumed by/plan-workhandoff,retrospective→ auto-retrospective (future)tui-polish→ replaced by/design-uiflowNew Workflow Documentation
Added
.claude/workflows/with process docs:parallel-work.md- How orchestration worksautonomous-session.md- What worker sessions dohuman-gates.md- When to escalate to humanui-development.md- Reference-driven UI design processHuman Injection Points
Follow-Up Issues
Test plan
/design,/test, etc.)🤖 Generated with Claude Code