feat: align with spec-kit 1.0 (skills-mode Copilot integration) - #1
Closed
arthurrogado wants to merge 1 commit into
Closed
arthurrogado wants to merge 1 commit into
arthurrogado wants to merge 1 commit into
Conversation
spec-kit 1.0 made the GitHub Copilot integration default to a skills layout (`.github/skills/<cmd>/SKILL.md`) instead of the legacy agents/prompts combo. The wiki extension's `provides.commands` already auto-generates skills correctly, so no schema change is needed — but the extension metadata should reflect the new baseline: - Bump `extension.version` 1.0.0 → 1.0.1 - Tighten `requires.speckit_version` >=0.2.0 → >=1.0.0 (the `/speckit.<dot>` form is still resolved via registered command alias in skills mode, so existing installs keep working) - Document both invocation styles (dot and hyphen) in README so users on skills-mode agents know to type `/speckit-wiki-init` instead of `/speckit.wiki.init`. Hooks (`after_plan`, `after_implement`) keep referencing the canonical dot-form ID because HookExecutor resolves by ID, not by invocation separator. Tested with spec-kit CLI 1.0.1: `specify extension info wiki` reports the new version; `specify extension add wiki --from <archive-url>` installs correctly and auto-registers 5 skills under `.github/skills/speckit-wiki-*/`. Refs: github/spec-kit#3976 (Copilot skills default in 0.16), v1.0 release.
Author
|
Closing this PR after testing spec-kit 1.0 + this extension end-to-end on a fresh project ( Findings:
What would actually help users adopting this extension with spec-kit 1.0:
I tested the dual-invocation case against a fresh spec-kit 1.0.1 install and both forms resolve correctly via the registered command alias. Happy to open a focused PR with just the README invocation table + CHANGELOG note (no version bump, no Thanks for maintaining the wiki extension, it's been a great fit for spec-kit workflows. |
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.
spec-kit 1.0 (and 0.16+) made the GitHub Copilot integration default to a skills layout () instead of the legacy agents/prompts combo. The wiki extension's already auto-generates skills correctly, so no schema change is needed — but the extension metadata should reflect the new baseline.
Changes
Why no
provides.skillsblockI considered adding to mirror the new layout, but verified in the spec-kit 1.0.1 CLI source () that the Copilot integration auto-generates skills from . Adding would be a non-schema field that future CLI versions might reject — see https://github.com/github/spec-kit/blob/main/spec-driven.md for the schema authority.
Tested with
specify extension info wikireports the new versionspecify extension add wiki --from <archive-url>installs correctly and auto-registers 5 skills under.github/skills/speckit-wiki-*/specify integration upgrade copilot --integration-options=\"--skills\"regenerates the skills from the new without changesRefs
Tested locally on github.com/arthurrogado/pontocivil.