feature/manual-date-override #79

Merged
cameron merged 4 commits from feature/manual-date-override into master 2026-05-07 15:10:38 +00:00
Owner

Add the ability to manually set a date for a file. Added an extended EXIF endpoint that dumps everything we get from exiftool.

Reworking/update of the insight generation/tools.

Add the ability to manually set a date for a file. Added an extended EXIF endpoint that dumps everything we get from exiftool. Reworking/update of the insight generation/tools.
cameron added 1 commit 2026-05-07 15:08:54 +00:00
The DAO swallowed every diesel::update failure as a flat
`anyhow!("Update error")`, then trace_db_call further reduced it to
`DbError { kind: UpdateError }`. Operators saw "update failed for lib
2 Snapchat/foo.mp4: DbError { kind: UpdateError }" with no clue why
(constraint violation? type mismatch? row vanished mid-flight? DB
locked?).

Two changes:
- Preserve the diesel error in the anyhow chain along with the input
  params (lib, rel_path, date_taken, source) so the cause is visible.
- Log the chain at warn-level inside the DAO before the trace wrapper
  collapses it to DbErrorKind::UpdateError, so the warning at the
  call site finally has something diagnosable next to it.
- Treat zero-row updates as a debug-level "row likely retired by the
  missing-file scan" rather than a hard failure — that case is benign
  and shouldn't poison the drain's error tally.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
cameron force-pushed feature/manual-date-override from 2ff06413c6 to 7e1c4ab318 2026-05-07 15:08:54 +00:00 Compare
cameron merged commit 95e21c8128 into master 2026-05-07 15:10:38 +00:00
cameron deleted branch feature/manual-date-override 2026-05-07 15:10:38 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Apps/ImageApi#79