Skip to content

Guard Claude converge skill against unsupported argument hints#3005

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-code-review-comment-another-one
Draft

Guard Claude converge skill against unsupported argument hints#3005
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-code-review-comment-another-one

Conversation

Copilot AI commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

The Claude integration should not suggest that /speckit-converge accepts a feature-name argument. converge relies on existing feature context, so advertising an argument here is misleading.

  • Regression guard

    • Added a focused Claude integration test to assert that converge is not present in ARGUMENT_HINTS.
    • This locks in the current behavior without changing production logic.
  • Scope

    • No runtime behavior changes.
    • No broader integration or template updates beyond this review-comment fix.
def test_converge_has_no_argument_hint(self):
    assert "converge" not in ARGUMENT_HINTS

Copilot AI requested review from Copilot and removed request for Copilot June 16, 2026 15:26
Copilot AI requested review from Copilot and removed request for Copilot June 16, 2026 15:29
Copilot AI changed the title [WIP] Fix code as per review comment Guard Claude converge skill against unsupported argument hints Jun 16, 2026
Copilot AI requested a review from BenBtg June 16, 2026 15:29
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.

2 participants