Skip to content

[openspec]: feat Add verify and archive phase schema support - #2

Merged
stanleykao72 merged 1 commit into
mainfrom
feat/verify-archive-phases
Apr 2, 2026
Merged

stanleykao72 merged 1 commit into
mainfrom
feat/verify-archive-phases

Conversation

@stanleykao72

Copy link
Copy Markdown
Owner

Summary

  • Added VerifyPhaseSchema and ArchivePhaseSchema to SchemaYamlSchema
  • Added verify.pre / verify.post to VALID_HOOK_POINTS and PluginHooksSchema
  • Updated verify-change.ts and archive-change.ts skill templates with schema-aware branching
  • Updated parity test hashes

Test plan

  • vitest: 1463/1466 pass
  • Parity tests: all template hashes updated and passing

🤖 Generated with Claude Code

Enable schema-level verify and archive phase definitions (VerifyPhaseSchema,
ArchivePhaseSchema) so plugins can define custom verification steps and
archive workflows per schema. Also adds verify.pre/verify.post hook points.

Constraint: Must be backward compatible — schemas without verify/archive definitions fallback to existing behavior
Rejected: Only use overlays without framework support | inconsistent with apply phase which has ApplyPhaseSchema
Confidence: high
Scope-risk: moderate
Reversibility: clean
Tested: vitest 1463/1466 pass (1 pre-existing failure in source-specs-normalization)
Not-tested: Runtime schema-aware verify/archive flow (requires plugin with verify/archive definitions)
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@stanleykao72
stanleykao72 merged commit d859518 into main Apr 2, 2026
@stanleykao72
stanleykao72 deleted the feat/verify-archive-phases branch April 2, 2026 07:20
stanleykao72 added a commit that referenced this pull request Apr 13, 2026
Enable schema-level verify and archive phase definitions (VerifyPhaseSchema,
ArchivePhaseSchema) so plugins can define custom verification steps and
archive workflows per schema. Also adds verify.pre/verify.post hook points.

Constraint: Must be backward compatible — schemas without verify/archive definitions fallback to existing behavior
Rejected: Only use overlays without framework support | inconsistent with apply phase which has ApplyPhaseSchema
Confidence: high
Scope-risk: moderate
Reversibility: clean
Tested: vitest 1463/1466 pass (1 pre-existing failure in source-specs-normalization)
Not-tested: Runtime schema-aware verify/archive flow (requires plugin with verify/archive definitions)

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant