Skip to content

[codex] Add storage boundary redaction hardening#76

Merged
Felix-Zuo merged 1 commit into
mainfrom
codex/codeql-redaction-boundary
Jun 25, 2026
Merged

[codex] Add storage boundary redaction hardening#76
Felix-Zuo merged 1 commit into
mainfrom
codex/codeql-redaction-boundary

Conversation

@Felix-Zuo

Copy link
Copy Markdown
Owner

Summary

  • add a defensive storage-boundary redaction pass before state, resume, and JSON artifacts are written
  • preserve explicit sensitive-opt-in behavior while scrubbing unmarked sensitive payloads by default
  • rework sensitive fixture construction to keep CodeQL focused on real storage behavior instead of hard-coded test secrets

Impact

This hardens HulunGuard as a real monitoring product: collected traces can still contain sensitive source material in memory, but unmarked records are scrubbed before persistent local artifacts are created.

Validation

  • python -m ruff check src tests scripts
  • python -m pytest -q
  • python -m hulun_guard threat-model-check --json
  • python -m hulun_guard release-verify --asset-dir .\dist --skip-attestation --json

Closes #75

Comment thread src/hulun_guard/storage.py Fixed
Comment thread src/hulun_guard/storage.py Fixed
Comment thread src/hulun_guard/storage.py Fixed
@Felix-Zuo Felix-Zuo force-pushed the codex/codeql-redaction-boundary branch from e3ee813 to b21b501 Compare June 25, 2026 09:05
Comment thread tests/test_adapter_conformance.py Fixed
Comment thread tests/test_hulun_guard.py Fixed
@Felix-Zuo Felix-Zuo force-pushed the codex/codeql-redaction-boundary branch from b21b501 to d8e6327 Compare June 25, 2026 09:10
@Felix-Zuo Felix-Zuo force-pushed the codex/codeql-redaction-boundary branch from d8e6327 to ddbf4ea Compare June 25, 2026 09:13
@Felix-Zuo Felix-Zuo marked this pull request as ready for review June 25, 2026 09:14
@Felix-Zuo Felix-Zuo merged commit ed53487 into main Jun 25, 2026
7 checks passed
@Felix-Zuo Felix-Zuo deleted the codex/codeql-redaction-boundary branch June 25, 2026 09:14
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.

Eliminate CodeQL clear-text sensitive data alerts

2 participants