Skip to content

Fix cross-reference source failure reporting - #76

Open
JordanCoin wants to merge 1 commit into
mainfrom
codex/issue-71-source-errors
Open

Fix cross-reference source failure reporting#76
JordanCoin wants to merge 1 commit into
mainfrom
codex/issue-71-source-errors

Conversation

@JordanCoin

Copy link
Copy Markdown
Owner

Closes #71

Summary

  • distinguish successful checks, failures, and rate-limit skips for each source/entity
  • surface redacted source error types in the CLI and saved JSON reports
  • retain incomplete source status in JSON and avoid logging investigated names or raw errors

Testing

  • python -m pytest tests/test_crossref_status.py tests/test_security_egress_research.py tests/test_security_egress_cli.py tests/test_security_hardening.py -v (65 passed; one existing FastAPI/Starlette deprecation warning)
  • ruff check and ruff format --check
  • git diff --check

Review

Independent review passed after two correction rounds; it specifically verified rate-limit and local ICIJ failures are no longer represented as clean no-match results.

Copilot AI lite review requested due to automatic review settings August 14, 2026 12:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

crossref per-source checkers swallow all exceptions to [] — a broken adapter looks like a clean record

2 participants