Skip to content

docs(openspec): plan #3391 Refuse criteria entries not in the issue body - #4069

Merged
dan-petty merged 2 commits into
mainfrom
fix/3391-refuse-criteria-entries-not-in-the
Nov 7, 2026
Merged

dan-petty merged 2 commits into
mainfrom
fix/3391-refuse-criteria-entries-not-in-the

Conversation

@devops-cli-service

Copy link
Copy Markdown
Contributor

Plans #3391

The stage check compares a plan's criteria list with the acceptance criteria that a person wrote in the issue body. criterion_problems refuses an entry whose text is not one of them, so a plan can no longer copy a criterion out of the refine block. The plan for #3320 in #3333 mapped five refine-block criteria and every gate accepted it. acceptance_criteria stays the only source: it cuts the refine block off, so an issue with no person-written list gets an empty criteria list. The replay's paths gate gives the check the issue body of a plan that has a criteria list. Two tests cover a body with no heading and a body whose only list sits in the refine block. The stage prompt states the refusal.

Change: openspec/changes/3391-refuse-criteria-entries-not-in-the/

@devops-cli-service devops-cli-service Bot added type/docs Documentation additions, guides, manuals, or README updates scope/openspec OpenSpec changes, specs and their checks labels Nov 7, 2026
@dan-petty
dan-petty force-pushed the fix/3391-refuse-criteria-entries-not-in-the branch from 57efbfb to c26d7da Compare November 7, 2026 21:38
@dan-petty
dan-petty marked this pull request as ready for review November 7, 2026 21:45
@dan-petty
dan-petty self-requested a review as a code owner November 7, 2026 21:45
@dan-petty
dan-petty merged commit 60063b1 into main Nov 7, 2026
24 checks passed
@dan-petty
dan-petty deleted the fix/3391-refuse-criteria-entries-not-in-the branch November 7, 2026 21:45
@dan-petty

Copy link
Copy Markdown
Owner

Check results

Head commit: c26d7dacf6fa58e6bcd9c0bcca21ce3bd3f80c0b

Check Conclusion Link
Audit Defect PR Hardening skipped run
Prune PR DevContainer Images success run
devops-cli success run
zizmor success run
trivy-config success run
Tests & Coverage success run
CodeQL success run
Plan Check success run
Dependency Freshness skipped run
Test Job Duration skipped run
Runner Image success run
Tests & Coverage (shard 3 of 4) success run
Rule Pack Tests success run
Tests & Coverage (shard 4 of 4) success run
DevContainer Image success run
Image Pins success run
Tests & Coverage (shard 2 of 4) success run
Docs Site Build success run
Third-Party Image Scan success run
Changed-Test Independence success run
Static Analysis success run
Tests & Coverage (shard 1 of 4) success run
Service Image success run
Analyze (python) success run
Semantic PR title success run
Analyze (actions) success run

Skipped check: Audit Defect PR Hardening
Skipped check: Dependency Freshness
Skipped check: Test Job Duration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope/openspec OpenSpec changes, specs and their checks type/docs Documentation additions, guides, manuals, or README updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant