Skip to content

fix(intake): name the missing recordings in the same refusal - #225

Merged
haribo merged 2 commits into
developfrom
fix/223-name-the-missing-recordings
Sep 9, 2026
Merged

fix(intake): name the missing recordings in the same refusal#225
haribo merged 2 commits into
developfrom
fix/223-name-the-missing-recordings

Conversation

@haribo

@haribo haribo commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • The frugal contract promises one refusal naming every missing address, but checkCaptures built the missing-content list from the captures alone: a client pushing captures and videos got two 409s instead of one (name the missing recordings in the same refusal as the captures #223, reproduced against a local server)
  • Absent recordings now join the same missing-content answer — existence checked only, never their format (ADR 0013)
  • The intake Blobs port gains Exists, already carried by the file store

Test plan

  • TestAMissingRecordingIsNamedInTheFirstRefusal — written first, failed with the exact observed symptom (the refusal named the capture alone), passes now
  • Full gates: gen-check, be-check, be-test, db-test, fe-test-e2e (62)

Test helper extended, no assertion changed: refusingBlobs implements the new Exists with the same refuse-first guarantee.

Closes #223

@haribo
haribo merged commit 9f1e549 into develop Sep 9, 2026
3 checks passed
@haribo
haribo deleted the fix/223-name-the-missing-recordings branch September 9, 2026 09:17
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.

name the missing recordings in the same refusal as the captures

1 participant