Skip to content

Add state-complete deck visual review - #9

Merged
theclaymethod merged 1 commit into
mainfrom
codex/artifacture-deck-review
Jul 27, 2026
Merged

Add state-complete deck visual review#9
theclaymethod merged 1 commit into
mainfrom
codex/artifacture-deck-review

Conversation

@theclaymethod

Copy link
Copy Markdown
Owner

What changed

  • add a dedicated P-deck-review verifier rubric and repair operator
  • capture every presentation base, progressive, and visible drill state at the contracted viewport
  • group base/state continuity and adjacent slides into exact two-frame review batches
  • support bounded recapture by stable state ID, failing closed on stale, missing, or unpaired states
  • add an independent deck-review visual-eval family and model-routing guardrails
  • route slide profiles and data-ve-presentation artifacts through the new pass

Why

Slide review previously relied too heavily on an opening frame and generic layout checks. That missed failures inside click-ins and progressive states, weak annotation-to-example mapping, misleading good/bad examples, and repetitive deck structure.

This adds a state-complete, evidence-first review path while preserving the existing verifier report and browser-run contracts.

Impact

  • deck authors receive findings tied to exact rendered state IDs and visible regions
  • repairs can recapture only affected states plus their paired context
  • the route remains fail-closed until a named human reviews a representative production-shaped corpus and a batch-2 model qualifies independently

Validation

  • npm test — 100/100 passing
  • npm run check:manifests
  • npm run ve:check
  • npm run ve:eval — 180 seeded violations, 7 clean fixtures, and 9 design-system cases passing
  • independent specification and repository-standards reviews completed with no remaining findings

@theclaymethod
theclaymethod marked this pull request as ready for review July 27, 2026 14:02
@theclaymethod
theclaymethod merged commit 98c2641 into main Jul 27, 2026
0 of 2 checks passed
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