Skip to content

Fix cross-reference source failure reporting - #76

Merged
JordanCoin merged 1 commit into
mainfrom
codex/issue-71-source-errors
Sep 26, 2026
Merged

JordanCoin merged 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.

@JordanCoin
JordanCoin merged commit 917c298 into main Sep 26, 2026
5 checks passed
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