Skip to content

refactor(validation): unify read-only execution planning - #7

Closed
ITSGJY wants to merge 1 commit into
mainfrom
refactor/validation-canonical-plan
Closed

ITSGJY wants to merge 1 commit into
mainfrom
refactor/validation-canonical-plan

Conversation

@ITSGJY

@ITSGJY ITSGJY commented Sep 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • make existing Cargo/Go validation adapters own one canonical read-only execution plan
  • derive the legacy synchronous command projection and structured Job argv from the same typed plan
  • remove the duplicate Runtime-side validation argv builder while retaining Runner-side canonical-step validation as an independent trust-boundary check
  • keep cargo_fmt mutation outside read-only planning and preserve existing tool schemas, authorization, Job handoff, validation evidence, and source-fence behavior

Validation

  • cargo check --locked -p webcodex-validation
  • cargo check --locked -p webcodex
  • cargo test --locked -p webcodex-validation --lib — 100 passed
  • structured Cargo/Go plan parity coverage — 6 passed
  • validation handoff coverage — 35 passed
  • Runner canonical validation-step coverage — 10 passed
  • cargo fmt --all -- --check
  • git diff --check
  • deterministic committed-range review: 8 files fully inspected, no warnings or truncation

Validation PR for a bounded preparatory slice of upstream issue yyjeqhc/webcodex#599.

@ITSGJY
ITSGJY force-pushed the refactor/validation-canonical-plan branch 2 times, most recently from 77fe65f to 654bd2d Compare September 25, 2026 16:54
@ITSGJY
ITSGJY force-pushed the refactor/validation-canonical-plan branch from 654bd2d to 65d86ce Compare September 26, 2026 02:43
@ITSGJY

ITSGJY commented Sep 26, 2026

Copy link
Copy Markdown
Owner Author

Validation completed successfully on commit 65d86ce. Upstream review continues in yyjeqhc#684.

@ITSGJY ITSGJY closed this Sep 26, 2026
@ITSGJY
ITSGJY deleted the refactor/validation-canonical-plan branch September 27, 2026 10:06
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