Skip to content

Fail closed command guard diagnostics and scans - #3611

Merged
Widthdom merged 8 commits into
mainfrom
fix-issue3476-3477-3478
Jun 13, 2026
Merged

Fail closed command guard diagnostics and scans#3611
Widthdom merged 8 commits into
mainfrom
fix-issue3476-3477-3478

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Fail closed in Codex hook adapters when hook payloads are malformed, missing commands, or cannot resolve the git project root.
  • Fail closed in command guard script scanning for missing candidate scripts and over-limit script files, while preserving safe Python module test invocations and scanning Python module script runners.
  • Fail closed for staged secret checks when gitleaks is unavailable, and route git commit aliases/global-option forms through the staged secret check.

Validation

  • python3 -m unittest discover -s .agent_harness/tests
  • dotnet build
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll index . --commits HEAD --json
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Codex adversarial review was run; findings around git alias commit detection and Python module script-runner scanning were fixed in follow-up commits.

Changelog

  • changelog.d/unreleased/3476.fixed.md
  • changelog.d/unreleased/3477.fixed.md
  • changelog.d/unreleased/3478.fixed.md

Fixes #3476
Fixes #3477
Fixes #3478

Refs #3608

@Widthdom
Widthdom marked this pull request as ready for review June 13, 2026 12:23
@Widthdom
Widthdom merged commit a0e6553 into main Jun 13, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3476-3477-3478 branch June 13, 2026 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant