Update openspec#871
Open
johnml1135 wants to merge 1 commit intomainfrom
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates OpenSpec “primitives” and workflow guidance by refreshing generated metadata and tightening instructions around how the CLI returns artifact context (especially contextFiles), plus revising onboarding and command references.
Changes:
- Bump
generatedByacross OpenSpec skills to1.3.1. - Clarify
contextFilesas “artifact ID -> array of file paths” and adjust task/design reading instructions accordingly. - Update onboarding preflight and command reference docs (introduce
/opsx:propose, add Windows command notes, and add guardrails about not copying<context>/<rules>blocks).
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| openspec/AGENTS.md | Removes the OpenSpec agent instruction file content (file deleted). |
| .github/skills/openspec-verify-change/SKILL.md | Updates verify instructions to treat contextFiles as an artifact-to-paths map. |
| .github/skills/openspec-sync-specs/SKILL.md | Bumps generatedBy metadata version. |
| .github/skills/openspec-onboard/SKILL.md | Updates onboarding flow: CLI install check, command reference tables, Windows notes. |
| .github/skills/openspec-new-change/SKILL.md | Bumps generatedBy metadata version. |
| .github/skills/openspec-ff-change/SKILL.md | Bumps generatedBy metadata version. |
| .github/skills/openspec-explore/SKILL.md | Updates explore guidance to “create a proposal” and formatting tweaks. |
| .github/skills/openspec-continue-change/SKILL.md | Bumps generatedBy metadata version. |
| .github/skills/openspec-bulk-archive-change/SKILL.md | Minor wording/formatting tweaks and removes /opsx:new mention in “no changes” case. |
| .github/skills/openspec-archive-change/SKILL.md | Changes sync behavior guidance to explicitly invoke openspec-sync-specs via tooling instructions. |
| .github/skills/openspec-apply-change/SKILL.md | Clarifies contextFiles shape and instructs reading all paths. |
| .github/prompts/opsx-verify.prompt.md | Mirrors verify guidance updates around contextFiles. |
| .github/prompts/opsx-onboard.prompt.md | Mirrors onboarding updates (CLI install check, tables, Windows notes). |
| .github/prompts/opsx-ff.prompt.md | Adds guardrails about not copying <context>/<rules> blocks into artifacts. |
| .github/prompts/opsx-explore.prompt.md | Mirrors explore guidance updates to “create a proposal”. |
| .github/prompts/opsx-bulk-archive.prompt.md | Minor wording/formatting tweaks and removes /opsx:new mention in “no changes” case. |
| .github/prompts/opsx-archive.prompt.md | Mirrors archive sync behavior guidance update to invoke openspec-sync-specs. |
| .github/prompts/opsx-apply.prompt.md | Mirrors apply guidance updates around contextFiles. |
| ## Preflight | ||
|
|
||
| Before starting, check if OpenSpec is initialized: | ||
| Before starting, check if the OpenSpec CLI is installed: |
| ## Preflight | ||
|
|
||
| Before starting, check if OpenSpec is initialized: | ||
| Before starting, check if the OpenSpec CLI is installed: |
| **Design Adherence**: | ||
| - If design.md exists in contextFiles: | ||
| - If `contextFiles.design` exists: | ||
| - Extract key decisions (look for sections like "Decision:", "Approach:", "Architecture:") |
Comment on lines
+88
to
91
| - If `contextFiles.design` exists: | ||
| - Extract key decisions (look for sections like "Decision:", "Approach:", "Architecture:") | ||
| - Verify implementation follows those decisions | ||
| - If contradiction detected: |
| - If already synced: "Archive now", "Sync anyway", "Cancel" | ||
|
|
||
| If user chooses sync, execute /opsx:sync logic (use the openspec-sync-specs skill). Proceed to archive regardless of choice. | ||
| If user chooses sync, use Task tool (subagent_type: "general-purpose", prompt: "Use Skill tool to invoke openspec-sync-specs for change '<name>'. Delta spec analysis: <include the analyzed delta spec summary>"). Proceed to archive regardless of choice. |
Comment on lines
56
to
61
| - If changes needed: "Sync now (recommended)", "Archive without syncing" | ||
| - If already synced: "Archive now", "Sync anyway", "Cancel" | ||
|
|
||
| If user chooses sync, execute `/opsx:sync` logic. Proceed to archive regardless of choice. | ||
| If user chooses sync, use Task tool (subagent_type: "general-purpose", prompt: "Use Skill tool to invoke openspec-sync-specs for change '<name>'. Delta spec analysis: <include the analyzed delta spec summary>"). Proceed to archive regardless of choice. | ||
|
|
||
| 5. **Perform the archive** |
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.
Update OpenSpec primitives
This change is