Repository navigation
feat: unify agent skill integration across Claude Code, Codex, and OpenCode (#131) - #231
Merged
Merged
Conversation
…enCode (#131) - Rewrite CLAUDE.md to use @AGENTS.md import (37→9 lines) - Add SpecSync Skill section to AGENTS.md - Refactor installskill.go: two canonical targets (claude-code, agents) with --codex/--opencode aliases and deduplication - Broaden SKILL.md description to cover full lifecycle - Unify doctor diagnostics: detect canonical + legacy paths, validate CLAUDE.md @AGENTS.md imports, report JSON output - Update site/features.json with agent skill diagnostics feature
spec/release-plan requirement 5 (0-indexed, so the sixth) wrapped its text
so the normative keyword was not where the parser looked. Reordered to lead
with the SHALL, which is better spec style regardless: the requirement now
states the rule before the parenthetical about its deferral.
Verified with a negative control — reverting the wording brings
`requirements.5.text: Requirement must contain SHALL or MUST keyword` back,
so the validator is reading the text this change edits and not something
adjacent.
launch-promotion kept its notes at specs/promotion/spec.md, where the
validator correctly refused them: no delta headers, no scenarios, and
"must"/"either-or" rather than normative language. Moved to
promotion-notes.md in the change root, which is what the validator's own
message advises ("or move non-delta notes outside specs/"). That removes
one of the two errors but not the failure — a change with no delta fails
however its notes are filed, and authoring the delta is a judgement about
what the promotion capability should require, not a mechanical fix.
Opens with 27 passed / 15 failed, from 26/16. The remaining 15 are all
"no deltas at all" and need either authored deltas or archiving, which is
a decision rather than a chore.
… unknown group Five cards (group review or none) never reached the page. Merge the cards that said the same thing, add the Review theme, and pin the group list with a test.
Deploying specsync with
|
| Latest commit: |
32f3dce
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://8955c225.specsync.pages.dev |
| Branch Preview URL: | https://feat-131-unify-agent-skill-i.specsync.pages.dev |
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.
--claude-code,--agents);--codexand--opencodeare aliases. Addsinstall-skill --drift,--tempand--rollback.CLAUDE.mdimportsAGENTS.md;doctor installreports canonical and legacy installs, validates the import, and keeps theopenspecdependency check.build.shfails on an unknown group. A test pins the group list.Closes #131