-
Notifications
You must be signed in to change notification settings - Fork 5k
Spec Kit Improvements #451
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
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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
Copilot reviewed 15 out of 15 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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
Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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
Copilot reviewed 15 out of 15 changed files in this pull request and generated 9 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| "codex": ".codex/", | ||
| "windsurf": ".windsurf/", | ||
| "kilocode": ".kilocode/", | ||
| "auggie": ".auggie/", |
Copilot
AI
Sep 21, 2025
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 Auggie folder is defined here as .auggie/ but elsewhere (packaging and agent context scripts) the path used is .augment/. This mismatch will lead to the security notice not matching the actual generated folder and may cause users to miss ignoring the correct directory. Standardize on one (likely .augment/ given other scripts) across all references.
| "auggie": ".auggie/", | |
| "auggie": ".augment/", |
| mkdir -p "$base_dir/.augment/commands" | ||
| generate_commands auggie md "\$ARGUMENTS" "$base_dir/.augment/commands" "$script" ;; |
Copilot
AI
Sep 21, 2025
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.
Uses .augment path while Python agent_folder_map uses .auggie/. Align this with the chosen canonical folder name to avoid inconsistent directory structures in generated templates.
| mkdir -p "$base_dir/.augment/commands" | |
| generate_commands auggie md "\$ARGUMENTS" "$base_dir/.augment/commands" "$script" ;; | |
| mkdir -p "$base_dir/.auggie/commands" | |
| generate_commands auggie md "\$ARGUMENTS" "$base_dir/.auggie/commands" "$script" ;; |
| $AGENTS_FILE = Join-Path $REPO_ROOT 'AGENTS.md' | ||
| $WINDSURF_FILE = Join-Path $REPO_ROOT '.windsurf/rules/specify-rules.md' | ||
| $KILOCODE_FILE = Join-Path $REPO_ROOT '.kilocode/rules/specify-rules.md' | ||
| $AUGGIE_FILE = Join-Path $REPO_ROOT '.augment/rules/specify-rules.md' |
Copilot
AI
Sep 21, 2025
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.
Auggie path uses .augment while other code (agent_folder_map) uses .auggie/. Unify the directory naming to prevent users from updating one location while the tool expects another.
| $AUGGIE_FILE = Join-Path $REPO_ROOT '.augment/rules/specify-rules.md' | |
| $AUGGIE_FILE = Join-Path $REPO_ROOT '.auggie/rules/specify-rules.md' |
| AGENTS_FILE="$REPO_ROOT/AGENTS.md" | ||
| WINDSURF_FILE="$REPO_ROOT/.windsurf/rules/specify-rules.md" | ||
| KILOCODE_FILE="$REPO_ROOT/.kilocode/rules/specify-rules.md" | ||
| AUGGIE_FILE="$REPO_ROOT/.augment/rules/specify-rules.md" |
Copilot
AI
Sep 21, 2025
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.
Same Auggie directory naming inconsistency (.augment vs .auggie). Standardize the folder name across all scripts and documentation.
| AUGGIE_FILE="$REPO_ROOT/.augment/rules/specify-rules.md" | |
| AUGGIE_FILE="$REPO_ROOT/.auggie/rules/specify-rules.md" |
| specify init my-project --ai qwen | ||
| specify init my-project --ai opencode | ||
| specify init my-project --ai codex | ||
| specify init my-project --ai windsurf |
Copilot
AI
Sep 21, 2025
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.
[nitpick] Kilo Code is supported but missing from the usage examples; add specify init my-project --ai kilocode to keep examples exhaustive and consistent with supported agents.
| specify init my-project --ai windsurf | |
| specify init my-project --ai windsurf | |
| specify init my-project --ai kilocode |
Spec Kit Improvements
README.mdthat documents how to useSPECIFY_FEATUREfor non-git workflows.README.mdto list supported agents.