domverse/photoanalyzer
1
0
Fork 0
Code Issues 5 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

US05-04 — Verify, Retry, and Resolve Uncertain Uploads #29

New Issue
Closed
opened 2026-07-13 13:10:48 +02:00 by domverse · 3 comments
domverse commented 2026-07-13 13:10:48 +02:00
Owner
Copy Link

Source specification: delivery_backlog/stories/US05-04-upload-verification.md

US05-04 — Verify, Retry, and Resolve Uncertain Uploads

Epic: E05

As a user, I want uncertain uploads verified before retry so a lost response cannot
create duplicate server assets.

Acceptance criteria

  • Retry policy distinguishes safe failures from accepted-but-unknown outcomes.
  • Verification compares recorded local bytes with authoritative available evidence.
  • Changed-after-upload bytes create a visible stale warning and block unsafe actions.
  • Manual resolution records evidence and audit history; it never silently assumes success.

Automated tests

  • Fault tests cover timeout before acceptance, acceptance then lost response, parser
    uncertainty, safe retry, changed bytes, and repeated verification.

Dependencies

  • US05-03
Source specification: [delivery_backlog/stories/US05-04-upload-verification.md](https://git.domverse-berlin.eu/domverse/photoanalyzer/src/branch/main/delivery_backlog/stories/US05-04-upload-verification.md) # US05-04 — Verify, Retry, and Resolve Uncertain Uploads Epic: [E05](https://git.domverse-berlin.eu/domverse/photoanalyzer/src/branch/main/delivery_backlog/E05-immich-upload.md) As a user, I want uncertain uploads verified before retry so a lost response cannot create duplicate server assets. ## Acceptance criteria - Retry policy distinguishes safe failures from accepted-but-unknown outcomes. - Verification compares recorded local bytes with authoritative available evidence. - Changed-after-upload bytes create a visible stale warning and block unsafe actions. - Manual resolution records evidence and audit history; it never silently assumes success. ## Automated tests - Fault tests cover timeout before acceptance, acceptance then lost response, parser uncertainty, safe retry, changed bytes, and repeated verification. ## Dependencies - US05-03
domverse added this to the E05 — Immich Upload milestone 2026-07-13 13:10:48 +02:00
domverse added the type/featurepriority/criticalarea/backendarea/integration labels 2026-07-13 13:10:48 +02:00
domverse added a new dependency 2026-07-13 13:11:26 +02:00
#28 US05-03 — Parse and Persist Uploader Outcomes
domverse added a new dependency 2026-07-13 13:11:27 +02:00
#30 US05-05 — Operate Uploads in the Browser
domverse added a new dependency 2026-07-13 13:11:28 +02:00
#31 US05-06 — Automate Phase E End-to-End Acceptance
domverse added this to the Photo Analyzer Delivery project 2026-07-13 13:19:46 +02:00
domverse added the status/backlog label 2026-07-13 13:52:52 +02:00
domverse added status/in-progress and removed status/backlog labels 2026-08-16 15:04:57 +02:00
domverse self-assigned this 2026-08-16 15:04:58 +02:00
domverse commented 2026-08-16 15:04:58 +02:00
Author
Owner
Copy Link

Claimed for implementation on branch us/US05-04-verify-retry-and-resolve-uncertain-uploads.

Claimed for implementation on branch `us/US05-04-verify-retry-and-resolve-uncertain-uploads`.
domverse added status/review and removed status/in-progress labels 2026-08-16 15:31:11 +02:00
domverse commented 2026-08-16 15:31:12 +02:00
Author
Owner
Copy Link

Submitted for review: #74

Tests passed:

  • work_item/scripts/python -m unittest discover -s work_item/tests -v
  • work_item/scripts/python -m pytest tests -q
  • work_item/scripts/python -m pytest tests/integration/test_upload_verification.py tests/unit/test_immich_bulk_check.py -q
Submitted for review: https://git.domverse-berlin.eu/domverse/photoanalyzer/pulls/74 Tests passed: - `work_item/scripts/python -m unittest discover -s work_item/tests -v` - `work_item/scripts/python -m pytest tests -q` - `work_item/scripts/python -m pytest tests/integration/test_upload_verification.py tests/unit/test_immich_bulk_check.py -q`
domverse added status/done and removed status/review labels 2026-08-16 15:31:26 +02:00
domverse commented 2026-08-16 15:31:27 +02:00
Author
Owner
Copy Link

Completed and merged via PR #74.

Completed and merged via PR #74.
domverse moved this to Done in Photo Analyzer Delivery on 2026-08-16 15:34:06 +02:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area/backend
Backend services, jobs, or API
 area/database
Schema, persistence, migration, or queries
 area/filesystem
Filesystem identity, mutation, or storage
 area/frontend
Browser UI and client-side application
 area/integration
External tools or provider integrations
 area/testing
Automated test suites, fixtures, and tooling
 priority/critical
Release-blocking safety or correctness
 priority/high
Important phase outcome
 priority/normal
Normal planned work
 status/backlog
Awaiting dependency eligibility or prioritization
 status/blocked
Cannot proceed without an explicit resolution
 status/done
Merged, tested, and completed
 status/in-progress
Currently being implemented
 status/ready
Eligible to be claimed
 status/review
Submitted in a pull request for review
 type/feature
User-facing or domain capability
 type/hardening
Security, resilience, operations, or release hardening
 type/migration
Donor extraction, migration, or legacy consolidation
 type/test
Automated test infrastructure or acceptance coverage
No Label area/backend area/integration priority/critical status/done type/feature
Milestone
No items
No Milestone E05 — Immich Upload
Projects
Clear projects
No project Photo Analyzer Delivery
Assignees
Clear assignees
domverse
No Assignees domverse
1 Participants
Notifications
Due Date
No due date set.
Blocks
#30 US05-05 — Operate Uploads in the Browser
domverse/photoanalyzer
#31 US05-06 — Automate Phase E End-to-End Acceptance
domverse/photoanalyzer
Depends on
#28 US05-03 — Parse and Persist Uploader Outcomes
domverse/photoanalyzer
Reference: domverse/photoanalyzer#29
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?