feat(specreview): add gap-analysis to the analysis enum#5
Merged
Conversation
Adds `gap-analysis` as a valid value for the SpecReview `analysis` frontmatter field (schema + authoring skeleton). This is the v0.4.0 feature bump that lets the new quoin `gap-analysis` skill emit a quire-validated SpecReview (one analysis = one doc, like the other review lenses). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
What
Adds
gap-analysisto theSpecReview.analysisenum:spec_artifacts_process/schemas/spec-review-frontmatter.schema.json— append"gap-analysis"toproperties.analysis.enumspec_artifacts_process/skeletons/SpecReview.md— list it in the authoring-comment enumWhy
This is the v0.4.0 feature bump that backs the new quoin
gap-analysisskill (a post-implementation verification gate). The skill emits one quire-validatedSpecReviewper run withanalysis: gap-analysis, matching the one-analysis-per-doc model the other review lenses already use.quire validaterejects the doc today because the enum lacks the value.Coupling
On merge + tag v0.4.0, bump the
default-modules.yamlpin inagent-ix/quointov0.4.0so the gap-analysis evals (EV-030..033) go green in CI. Until then they are RED-in-CI by design (same pattern as the SpecReview archetype rollout / EV-026).Validation
make test+make lintgreen. End-to-end: a sampleanalysis: gap-analysisSpecReview validates (exit 0); a bogus value is correctly rejected against the enum now includinggap-analysis.🤖 Generated with Claude Code