Skip to content

refactor(validation): separate project plan and evidence authority - #21

Merged
ITSGJY merged 2 commits into
mainfrom
refactor/962-project-validation-plan-evidence
Oct 8, 2026
Merged

ITSGJY merged 2 commits into
mainfrom
refactor/962-project-validation-plan-evidence

Conversation

@ITSGJY

@ITSGJY ITSGJY commented Oct 8, 2026 •

Copy link
Copy Markdown
Owner

Scope

Internal yyjeqhc#962 P0 validation authority refactor:

  • Keep the resolved project recipe step as the single Runner execution plan for project_validate.
  • Separate result/evidence interpretation through ValidationEvidenceProfile, retaining canonical specialist command builders.
  • Verify backend/action/profile identity before starting a project validation Job.
  • Parse structured test counts only through matching evidence profiles; retain established Cargo, Go, and pytest behavior.
  • Add regression coverage for canonical steps, identities, count evidence, incompatible profiles, and Job admission.

Verification

  • cargo test -p webcodex-validation --lib --offline: 115 passed.
  • cargo fmt --all -- --check: passed.
  • git diff --check: passed.
  • Full fork CI 37787599737: SUCCESS on 6f6dd077cda7d0b84e44ab3c77bd7dc5c6e8e7a4.
  • Codex final committed-diff review: PASS.
  • agy final committed-diff review: PASS.

This fork PR validates the upstream yyjeqhc#962 P0 slice. Upstream submission follows review approval.

Copilot AI balanced review requested due to automatic review settings October 8, 2026 13:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ITSGJY
ITSGJY merged commit 5a0bcdf into main Oct 8, 2026
18 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.

2 participants