fix(copilot): resolve active spec template#2765
Merged
mnriem merged 1 commit intoJun 1, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Copilot integration’s speckit.specify agent instructions so the spec template is resolved via the preset/template resolution stack (rather than hardcoding the core template path), aligning Copilot behavior with preset overrides and fixing #2761.
Changes:
- Update
templates/commands/specify.mdto instruct the agent to resolve the activespec-template(equivalent tospecify preset resolve spec-template) and use that resolved file. - Add an integration test asserting the generated Copilot specify agent no longer hardcodes
.specify/templates/spec-template.mdand instead references the resolver flow.
Show a summary per file
| File | Description |
|---|---|
| tests/integrations/test_integration_copilot.py | Adds a regression test ensuring the generated speckit.specify agent resolves the active spec template and avoids hardcoded core-template instructions. |
| templates/commands/specify.md | Updates the speckit.specify command template to resolve and load the active spec-template through the preset/template resolution stack. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 2/2 changed files
- Comments generated: 0
Collaborator
|
Thank you! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes #2761.
This updates the Copilot
speckit.specifycommand instructions so generated agents resolve the activespec-templatethrough the Spec Kit preset/template resolution stack instead of hardcoding the core template path. This lets project overrides, presets, and extensions supply the effective spec template for new specs.Testing
uv run --no-sync specify --helpuv run --no-sync pytest -q(3104 passed, 40 skipped)uv run --no-sync pytest tests/integrations/test_integration_copilot.py -q(47 passed)uvx ruff check src/python -m compileall -q src tests/integrations/test_integration_copilot.pygit diff --check -- templates/commands/specify.md tests/integrations/test_integration_copilot.py