Skip to content

fix(required-gate): honor disabled validation inputs#41

Merged
ArchonVII merged 1 commit into
mainfrom
agent/codex/40-required-gate-disable-inputs
Jun 2, 2026
Merged

fix(required-gate): honor disabled validation inputs#41
ArchonVII merged 1 commit into
mainfrom
agent/codex/40-required-gate-disable-inputs

Conversation

@ArchonVII

Copy link
Copy Markdown
Owner

Summary

  • Honor disabled repo-required-gate validation inputs in the final decision job.
  • Require optional dependency review, workflow validation, and policy validation only when both the caller input and classifier output enable them.
  • Keep the PR contract CLI importable under Vitest while preserving the documented node scripts/pr-contract.mjs invocation path.

Verification

  • npm test
command: npm test
location: local
result: pass: 5 test files, 77 tests
timestamp: 2026-06-02T01:37:50Z
  • git diff --check
command: git diff --check
location: local
result: pass: no whitespace errors
timestamp: 2026-06-02T01:38:10Z
  • node scripts/pr-contract.mjs --event-path %TEMP%\github-workflows-pr-contract-event-40.json --json
command: node scripts/pr-contract.mjs --event-path %TEMP%\github-workflows-pr-contract-event-40.json --json
location: local
result: pass: ok true, zero errors, zero warnings
timestamp: 2026-06-02T01:38:21Z

Verification Notes

The suite covers the new decision-job input wiring and confirms scripts/pr-contract.mjs remains importable by Vitest after the CLI compatibility adjustment.

Docs / Changelog

  • README update not needed because the documented repo-required-gate / decision contract is preserved.
  • Changelog fragment not needed because this repository does not define a changelog-fragment requirement for reusable workflow fixes.

Closes #40

Gate the final decision job on both caller inputs and classifier outputs for optional dependency review, workflow validation, and policy validation jobs.

Also keeps the PR contract CLI importable under Vitest by relying on the documented node-based invocation path.

Fixes #40
@ArchonVII
ArchonVII marked this pull request as ready for review June 2, 2026 01:39
@ArchonVII
ArchonVII merged commit baed4d6 into main Jun 2, 2026
1 check passed
@ArchonVII
ArchonVII deleted the agent/codex/40-required-gate-disable-inputs branch June 2, 2026 01:40
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.

repo-required-gate decision ignores disabled gate inputs

1 participant