Skip to content

ci/security/test: automated audit fixes for CryptoLivePoolPipelines#101

Merged
Pigbibi merged 1 commit into
mainfrom
codex/audit-fix-20260702-1546
Jul 2, 2026
Merged

ci/security/test: automated audit fixes for CryptoLivePoolPipelines#101
Pigbibi merged 1 commit into
mainfrom
codex/audit-fix-20260702-1546

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • redact secret-like findings in the Codex review gate instead of echoing matched values
  • add regression coverage for the redaction behavior
  • add pip check to CI dependency verification

Problems found

  • scripts/gate_codex_app_review.py reported matched secret-like fragments directly in violation messages
  • CI installed dependencies but did not run python -m pip check

Fixes applied

  • normalize secret field names and emit <redacted> in gate findings
  • add tests/test_gate_codex_app_review.py
  • add Verify dependencies step to .github/workflows/ci.yml

Security impact

  • reduces risk of secret-like values being echoed into CI logs or gate output
  • no runtime publish targets, credentials, cloud config, or live-pool contract logic changed

Architecture impact

  • low risk; audit gate output only, plus CI dependency verification

Tests run

  • .venv-audit/bin/python -m pip check
  • .venv-audit/bin/ruff check .
  • .venv-audit/bin/python -m unittest tests.test_gate_codex_app_review -v
  • .venv-audit/bin/python -m unittest discover -s tests -v
  • actionlint
  • git diff --check

Failed or skipped checks with reasons

  • none

Deployment notes

  • no deployment required

Rollback plan

  • revert commit 594818c

Manual follow-up checklist

  • confirm PR checks pass
  • merge after branch protection requirements are satisfied

@Pigbibi Pigbibi added codex AI Codex operations github_actions Pull requests that update GitHub Actions code python Pull requests that update python code human-review-required Requires human review before merge labels Jul 2, 2026
@Pigbibi Pigbibi enabled auto-merge (squash) July 2, 2026 07:50
@Pigbibi Pigbibi merged commit 18054e5 into main Jul 2, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex AI Codex operations github_actions Pull requests that update GitHub Actions code human-review-required Requires human review before merge python Pull requests that update python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant