Skip to content

Add authoritative local validation gate#62

Merged
jwalin-shah merged 2 commits into
mainfrom
codex/WP-032-local-validation-gate
May 14, 2026
Merged

Add authoritative local validation gate#62
jwalin-shah merged 2 commits into
mainfrom
codex/WP-032-local-validation-gate

Conversation

@jwalin-shah

Copy link
Copy Markdown
Owner

Summary

  • add a single local validation gate at python3 tools/local_validation.py
  • document the gate in worker-facing validation docs and project context
  • extend packaging/CI contract tests to prove the gate runs executable checks and exits nonzero on failures

Validation

  • python3 -m pytest tests/test_packaging_ci.py -q -> 13 passed
  • python3 -m pytest -q -> 209 passed
  • git diff --check -> passed
  • python3 tools/local_validation.py -> 209 passed, git diff --check passed

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@jwalin-shah

Copy link
Copy Markdown
Owner Author

Portfolio monitor review found a blocking follow-up before merge: Codex review flagged P2 that the canonical local handoff gate checks only unstaged worktree changes with git diff --check, so staged/committed PR whitespace errors can pass. Please include staged/committed changes or compare against the PR base/merge-base before merge.

@jwalin-shah
jwalin-shah merged commit 8e6fa0c into main May 14, 2026
1 check failed
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