Conversation
localden
commented
Sep 25, 2025
localden
commented
- Better command delineation
- Clarity on AI contribution policy, inspired by @mitchellh in AI tooling must be disclosed for contributions ghostty-org/ghostty#8289
There was a problem hiding this comment.
Pull Request Overview
This PR improves the Spec Kit project by reorganizing command presentation and establishing clearer AI contribution policies. The changes streamline the user experience by better categorizing commands and set clear expectations for AI-assisted contributions.
- Restructured CLI command presentation into core workflow commands and optional enhancement commands
- Added comprehensive AI contribution disclosure requirements with clear examples
- Simplified command descriptions and improved visual organization
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/specify_cli/init.py | Reorganized command presentation into core workflow and optional enhancement sections |
| CONTRIBUTING.md | Added detailed AI contribution policy with disclosure requirements and examples |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| console.print(steps_panel) | ||
|
|
||
| enhancement_lines = [ | ||
| "Optional commands that you can use for your specs (improve quality & confidence)[/bright_black]", |
There was a problem hiding this comment.
Missing opening rich markup tag. The line should start with [bright_black] to match the closing tag at the end.
| "Optional commands that you can use for your specs (improve quality & confidence)[/bright_black]", | |
| "[bright_black]Optional commands that you can use for your specs (improve quality & confidence)[/bright_black]", |
| "Optional commands that you can use for your specs (improve quality & confidence)[/bright_black]", | ||
| "", | ||
| f"○ [cyan]/clarify[/] [bright_black](optional)[/bright_black] - Ask structured questions to de-risk ambiguous areas before planning (run before [cyan]/plan[/] if used)", | ||
| f"○ [cyan]/analyze[/] [bright_black](optional)[/bright_black] - Cross-artifact consistency & alignment report (after [cyan]/tasks[/], before [cyan]/implement[/])" |
There was a problem hiding this comment.
These f-strings don't contain any variable interpolation. Consider using regular strings instead of f-strings for better readability and performance.
| f"○ [cyan]/analyze[/] [bright_black](optional)[/bright_black] - Cross-artifact consistency & alignment report (after [cyan]/tasks[/], before [cyan]/implement[/])" | |
| "○ [cyan]/analyze[/] [bright_black](optional)[/bright_black] - Cross-artifact consistency & alignment report (after [cyan]/tasks[/], before [cyan]/implement[/])" |
Spec Kit Updates
1 participant
