Skip to content

feat(preset): add copilot-ask-questions clarify preset#2003

Draft
fdcastel wants to merge 2 commits intogithub:mainfrom
fdcastel:preset/copilot-ask-questions
Draft

feat(preset): add copilot-ask-questions clarify preset#2003
fdcastel wants to merge 2 commits intogithub:mainfrom
fdcastel:preset/copilot-ask-questions

Conversation

@fdcastel
Copy link
Copy Markdown

Summary

Adds a community preset that packages the vscode/askQuestions enhancement for /speckit.clarify instead of changing the core clarify command directly.

This follows the guidance on #1962 to publish the Copilot-specific behavior as a preset for the community to try.

What This Adds

  • A new preset: copilot-ask-questions
  • An override for speckit.clarify that:
    • uses vscode/askQuestions to present all queued clarification questions in a single batched call when available
    • falls back to the existing one-question-at-a-time flow when the tool is not available
  • Preset metadata, README, and license

Why Preset-Based

The previous PR attempted to change the shared core command. This version keeps the core workflow untouched and makes the Copilot-specific interaction model opt-in.

Testing

  • uv run python -m pytest tests/test_presets.py -v --tb=short
    • result: passed locally
  • Full test suite on Windows still has unrelated pre-existing failures in existing bash/git-dependent tests

Related

Packages the vscode/askQuestions enhancement as a community preset,
as recommended by maintainer in github#1962.

When installed, overrides the /speckit.clarify command to batch all
queued questions in a single askQuestions call when the tool is
available, falling back to standard sequential flow otherwise.

Addresses github#1657.
@fdcastel fdcastel requested a review from mnriem as a code owner March 28, 2026 00:46
Copilot AI review requested due to automatic review settings March 28, 2026 00:46
@fdcastel
Copy link
Copy Markdown
Author

@mnriem I don't know what I’m doing here !😄

If you’ve got any wisdom to share, I’m all ears! 🤗

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new opt-in preset intended to override /speckit.clarify with a GitHub Copilot-specific interaction model that batches clarification questions via vscode/askQuestions when available, while keeping the core clarify command unchanged.

Changes:

  • Adds a new preset manifest defining copilot-ask-questions and a command override for speckit.clarify.
  • Introduces a customized speckit.clarify command template that conditionally batches questions using vscode/askQuestions.
  • Adds preset documentation (README) and an MIT license.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
copilot-ask-questions/preset.yml Declares the new preset and command override metadata.
copilot-ask-questions/commands/speckit.clarify.md Implements the batched questioning behavior in the clarify command instructions.
copilot-ask-questions/README.md Documents the preset’s purpose and installation/usage.
copilot-ask-questions/LICENSE Adds MIT license for the preset contents.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +22 to +26
### From a release URL

```bash
# FIXME ?
specify preset add --from https://github.com/github/spec-kit/releases/.../copilot-ask-questions.zip
Copy link

Copilot AI Mar 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The installation snippet still contains a “FIXME” placeholder and a non-existent .../copilot-ask-questions.zip URL. This makes the README unusable as-is. Replace this with a real install path (e.g., specify preset add copilot-ask-questions once it’s in the community catalog, or a concrete GitHub release asset/archive URL for this preset’s repo).

Suggested change
### From a release URL
```bash
# FIXME ?
specify preset add --from https://github.com/github/spec-kit/releases/.../copilot-ask-questions.zip
### From the community catalog
```bash
specify preset add copilot-ask-questions

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was intentional.

I'm looking for guidance on how this will be distributed and what URL should be put here in case this PR lands on main.

- Resolve step 3/4 conflict: qualify 'Do NOT output all at once' to apply
  only when vscode/askQuestions is unavailable; batching is explicitly
  allowed when the tool is available
- Add preset to presets/catalog.community.json (sorted by ID)
- Update README install section with community catalog command and
  concrete release URL (remove FIXME placeholder)
@fdcastel fdcastel marked this pull request as draft March 28, 2026 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: using new VSCode tool askQuestions when available

2 participants