Skip to content
This repository was archived by the owner on Jul 3, 2026. It is now read-only.

Sanitize API fallback failure comments#63

Merged
Pigbibi merged 1 commit into
mainfrom
codex/sanitize-api-fallback-errors
Jul 3, 2026
Merged

Sanitize API fallback failure comments#63
Pigbibi merged 1 commit into
mainfrom
codex/sanitize-api-fallback-errors

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • summarize provider HTTP errors without raw response bodies
  • redact API-key-like values before writing fallback warnings to issue comments
  • add regression coverage for failed reviewer warning sanitization

Tests

  • python3 -m ruff check scripts/run_monthly_codex_audit.py tests/test_run_monthly_codex_audit.py
  • python3 -m unittest tests.test_run_monthly_codex_audit.RunMonthlyCodexAuditTests.test_run_configured_api_reviews_sanitizes_failed_reviewer_errors tests.test_run_monthly_codex_audit.RunMonthlyCodexAuditTests.test_service_failure_classification_ignores_source_code_secret_words
  • python3 -m unittest discover -s tests
  • git diff --check

Co-Authored-By: Codex <noreply@openai.com>
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

🤖 Codex PR Review

⚠️ Review skipped: The Codex review could not be completed.

Codex service request failed: 401 {"error": "OIDC workflow_ref is not allowed"}

Please ensure a human reviewer checks this PR before merging.

@Pigbibi Pigbibi merged commit 002709d into main Jul 3, 2026
4 checks passed
@Pigbibi Pigbibi deleted the codex/sanitize-api-fallback-errors branch July 3, 2026 08:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant