Skip to content

fix(processor): redact code execution errors - #94

Open
XnLemon wants to merge 116 commits into
mainfrom
codex/phase2-permission-reason-redaction-sweep
Open

fix(processor): redact code execution errors#94
XnLemon wants to merge 116 commits into
mainfrom
codex/phase2-permission-reason-redaction-sweep

Conversation

@XnLemon

@XnLemon XnLemon commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Objective

Close one Phase2 redaction gap where code execution errors are emitted as model-visible post-processing messages.

Changes

  • Redact ExecuteCode errors before emitting Code execution failed result events.
  • Extend the shared platform redactor with generic whole-line Authorization and Cookie fallbacks.
  • Add platform redactor regression coverage for non-standard Authorization schemes and multi-value Cookie headers.
  • Add code execution regression coverage for Authorization, api_key, URL userinfo, and Cookie secrets in executor errors.

Validation

  • go test ./platform -run Redactor
  • go test ./internal/flow/processor -run CodeExecutionResponseProcessor
  • go vet ./platform ./internal/flow/processor
  • git diff --check
  • open-code-review subagent loop: final review 0 High / 0 Medium / 0 Low

Known Risks / Limitations

  • Scope is intentionally limited to shared redaction patterns and the code-execution error event path.
  • Full go test ./internal/flow/processor still has existing Windows path-separator failures in skill path tests, unrelated to this change.

Follow-up

  • Continue Phase2 closure sweep across remaining model-visible error and governance output surfaces in the next stacked increment.

XnLemon added 30 commits July 8, 2026 11:47
@XnLemon
XnLemon force-pushed the codex/phase2-tool-metadata-redaction branch from 81ad3f1 to e163f51 Compare July 11, 2026 13:38
@XnLemon
XnLemon force-pushed the codex/phase2-permission-reason-redaction-sweep branch from 868f193 to 4cbcd6e Compare July 11, 2026 13:38
@XnLemon
XnLemon changed the base branch from codex/phase2-tool-metadata-redaction to main July 13, 2026 02: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

Development

Successfully merging this pull request may close these issues.

2 participants