feat: add Aider CLI support and update related documentation#1611
feat: add Aider CLI support and update related documentation#1611wolffy-au wants to merge 1 commit intogithub:mainfrom
Conversation
7f6e3a8 to
67726a3
Compare
There was a problem hiding this comment.
Pull request overview
This PR adds support for Aider CLI as a new AI agent in Spec Kit. The integration follows the established multi-agent support pattern by adding Aider configurations across CLI, extensions, release scripts, documentation, and development environment files. However, the PR inadvertently changes the repository references to point to a development fork (wolffy-au/spec-kit-aider) instead of the canonical repository (github/spec-kit), which breaks template downloads for all users.
Changes:
- Adds Aider CLI as a supported AI agent with configuration, commands directory (.aider/commands), and CLI detection
- Updates all agent-related scripts (bash and PowerShell variants) to include Aider in processing logic
- Adds scripts frontmatter to constitution.md template for prerequisite checking
- Updates documentation (README, AGENTS.md) and development environment setup (devcontainer)
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/specify_cli/init.py | Adds Aider to AGENT_CONFIG, updates help text; CRITICAL ISSUE: changes repo to wolffy-au/spec-kit-aider |
| src/specify_cli/extensions.py | Adds Aider to CommandRegistrar AGENT_CONFIGS with trailing comma formatting fixes |
| templates/commands/constitution.md | Adds scripts frontmatter for check-prerequisites execution |
| scripts/bash/update-agent-context.sh | Adds Aider support with .aider/rules/specify-rules.md file path |
| scripts/powershell/update-agent-context.ps1 | Adds Aider support matching bash implementation |
| README.md | Documents Aider CLI in supported agents table and usage examples |
| AGENTS.md | Adds Aider to agent integration table and CLI requirements list |
| .gitignore | Adds .aider patterns and other development artifacts; contains duplicate pattern |
| .python-version | Should not be committed: file is already in .gitignore |
| .github/workflows/scripts/create-release-packages.sh | Adds Aider to agent list and build cases |
| .github/workflows/scripts/create-release-packages.ps1 | Adds Aider to agent list; pre-existing bug: missing shai/agy agents |
| .github/workflows/scripts/create-github-release.sh | Adds Aider release packages with trailing whitespace issue |
| .devcontainer/post-create.sh | Installs aider-chat via pipx |
| .devcontainer/devcontainer.json | Adds lee2py.aider-composer VSCode extension |
Comments suppressed due to low confidence (1)
src/specify_cli/init.py:1361
- The repository owner and name have been changed to "wolffy-au/spec-kit-aider" instead of the canonical "github/spec-kit". This breaks the version command's ability to check for the latest template release version. The repository references should remain pointing to the official repository.
repo_owner = "wolffy-au"
repo_name = "spec-kit-aider"
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| # Define all agents and scripts | ||
| $AllAgents = @('claude', 'gemini', 'copilot', 'cursor-agent', 'qwen', 'opencode', 'windsurf', 'codex', 'kilocode', 'auggie', 'roo', 'codebuddy', 'amp', 'q', 'bob', 'qoder') | ||
| $AllAgents = @('claude', 'gemini', 'copilot', 'cursor-agent', 'qwen', 'opencode', 'windsurf', 'codex', 'kilocode', 'auggie', 'roo', 'codebuddy', 'amp', 'q', 'bob', 'qoder', 'aider') |
There was a problem hiding this comment.
The PowerShell release script is missing 'shai' and 'agy' agents in the AllAgents array at line 363, while the bash version at create-release-packages.sh:233 includes them. This pre-existing discrepancy means PowerShell-based releases won't generate packages for these agents. According to the memory about release packaging, both scripts must maintain identical agent lists for consistency.
| repo_owner = "wolffy-au" | ||
| repo_name = "spec-kit-aider" |
There was a problem hiding this comment.
The repository owner and name have been changed to "wolffy-au/spec-kit-aider" instead of the canonical "github/spec-kit". This breaks template downloads for all users, as this appears to be a development fork. The repository references should remain pointing to the official repository.
This issue also appears on line 1360 of the same file.
| repo_owner = "wolffy-au" | |
| repo_name = "spec-kit-aider" | |
| repo_owner = "github" | |
| repo_name = "spec-kit" |
.python-version
Outdated
| @@ -0,0 +1 @@ | |||
| 3.11.9 | |||
There was a problem hiding this comment.
The .python-version file should not be added to the repository as it's already listed in .gitignore at line 55. This file contains a specific Python version (3.11.9) that may not be relevant for all developers and should remain a local development preference.
| 3.11.9 |
| .genreleases/spec-kit-template-bob-sh-"$VERSION".zip \ | ||
| .genreleases/spec-kit-template-bob-ps-"$VERSION".zip \ | ||
| .genreleases/spec-kit-template-aider-sh-"$VERSION".zip \ | ||
| .genreleases/spec-kit-template-aider-ps-"$VERSION".zip \ |
There was a problem hiding this comment.
Trailing whitespace detected at the end of this line. This is inconsistent with the formatting requirement stated in the constitution.md template at line 81 which says "Avoid trailing whitespace."
| .genreleases/spec-kit-template-aider-ps-"$VERSION".zip \ | |
| .genreleases/spec-kit-template-aider-ps-"$VERSION".zip \ |
|
I've put this PR back into Draft because I still need to finish the Aider integration side of it. |
67726a3 to
caaa0c9
Compare
caaa0c9 to
6f8204b
Compare
The Aider side of this integration is still work in progress (https://github.com/wolffy-au/aider-speckit/tree/feat-spec-kit). Commands currently implemented are: