Skip to content

chore: start explainable fraud holds - #943

Merged
greatest0fallt1me merged 1 commit into
Predictify-org:mainfrom
jenetbamidele2-ai:feature/explainable-fraud-holds
Aug 30, 2026
Merged

chore: start explainable fraud holds#943
greatest0fallt1me merged 1 commit into
Predictify-org:mainfrom
jenetbamidele2-ai:feature/explainable-fraud-holds

Conversation

@jenetbamidele2-ai

Copy link
Copy Markdown
Contributor

Summary:

Imlement explainable fraud holds without exposing sensitive fraud-detection signals.

This change will:

  • Add a safe, deterministic sxplanation model for fraud holds.
  • Keep raw signals such as addresses, funders, transaction hashes, graph edges and pattern keys out of user-facing responses.
  • Preserve admin authorization, scope checks, validation, rate limiting and correlation IDs.
  • Maintain idempotent and concurrency-safe fraud-flag updates without resetting reviewer state.
  • Add focused coverage for successful scans, invalid input, duplicate/retry behaviour, boundary cases, authorization, redaction, and failure recovery.
  • Update the API contract and operational documentation.

Security and failure mode handling:

  • Explanations expose only approves reason codes and user-safe descriptions.
  • Raw detection evidence remains restricted to the internal review boundary.
  • Invalid requests are rejected deterministically.
  • Duplicate and concurrent scans converge through atomic upserts.
  • Detector failure remain diagnosable through correlation IDs and structured legs without logginf sensitive signals.
  • Existing callers remain compatible through additive response changes and preserved repository interfaces.

Validation

  • Focused fraud service, worker, and admin route tests.
  • Authorization and scope regression tests.
  • Redaction and response-contract tests.
  • Retry, duplicate, concurrency and partial-failure scenerios.
  • Repository CI checks.

Closes #933

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@greatest0fallt1me

Copy link
Copy Markdown
Contributor

This PR is linked to issue #933, but it currently contains no changed files, so there is no implementation to review. Please add the substantive code and focused tests required by the issue; I’m leaving the PR open for that update.

@greatest0fallt1me
greatest0fallt1me merged commit ff4731b into Predictify-org:main Aug 30, 2026
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.

[Quality-2][High] Make fraud holds explainable without exposing signals

2 participants