Skip to content

feat(plan): add machine-checkable success-criteria gate#209

Merged
omartinma merged 5 commits into
mainfrom
feat/machine-checkable-success-criteria
Jul 3, 2026
Merged

feat(plan): add machine-checkable success-criteria gate#209
omartinma merged 5 commits into
mainfrom
feat/machine-checkable-success-criteria

Conversation

@omartinma

@omartinma omartinma commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Description

Adds a machine-checkable success-criteria gate to /plan.

  • New Success Criteria Gate step: each criterion gets a verify: command (or verify: manual <steps>), and vacuous or subjective criteria are rejected and rewritten as something provable.
  • Plan templates (minimal/standard/extensive) carry a fenced success-criteria block — GOAL / SUCCESS CRITERIA / NON-GOALS / VERIFICATION COMMAND — replacing the prose acceptance-criteria section. The block lives once in the shared templates; the per-skill references/ symlinks are untouched.
  • /build parse reference updated to read the new block.

Running the verify commands during a build is out of scope here and tracked in #204.

Closes #203.

Type of Change

  • New feature (feat)
  • Bug fix (fix)
  • Code refactor (refactor)
  • Documentation (docs)
  • CI change (ci)
  • Chore (chore)

@omartinma omartinma requested a review from a team as a code owner July 2, 2026 09:57
@omartinma omartinma requested a review from ryzizub July 2, 2026 10:02
Comment thread skills/shared/references/plan-templates/standard.md

@ryzizub ryzizub left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@omartinma omartinma merged commit 65e6347 into main Jul 3, 2026
6 checks passed
@omartinma omartinma deleted the feat/machine-checkable-success-criteria branch July 3, 2026 10:30
@vgvbot vgvbot mentioned this pull request Jul 3, 2026
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.

feat(plan): add machine-checkable success-criteria gate

2 participants