[bitbiter-dev]

Closes #297. Design: note on #296, decision (1A + 2A, 3B, cover SET NULL).

Summary

After a crawl, photos of one storage config sharing Apple's BurstUUID form one Burst; its stored Cover is the earliest member (GLOSSARY.md).

 CrawlHandler.RunAsync
   ActiveStorageConfigs.FindAsync        # shared skip guard (Crawl + BurstDetection)
   pipeline.RunAsync
   ReportDuplicatesAsync
+  queue.EnqueueAsync(BurstDetectionPayload)   # dedup key burst-detection:{id}

 WorkScheduler
-  CrawlSchedules
+  StorageConfigSchedules<CrawlPayload>
+  StorageConfigSchedules<BurstDetectionPayload>   # fallback, crawl interval

BurstDetectionHandler.RunAsync(config)
  skip if config missing/inactive; fail if root missing (nothing marked)
  repair every burst of the config (and memberless ones)
    < 2 active members -> delete burst
    else               -> cover = earliest active member
  for each unchecked Image/LivePhoto, oldest first, in batches of 100
    read original once (EXIF capture time, BurstUUID)  -> BurstCheck row
    if candidate (EXIF time, make, model, thumbnail) and labelled
      join every checked candidate with that label: create / join / merge into oldest burst
      cover = earliest member
    save photo + burst changes together

src/Anichron.Core/
├── Domain/BurstCheck.cs                    # new table, cascades with its asset
├── Domain/Burst.cs                         # PrimaryAssetId nullable, SET NULL
├── Data/Repository/BurstRepository.cs      # every burst query + the checked mark
└── Work/Payloads.cs                        # BurstDetectionPayload, IStorageConfigPayload<TSelf>
src/Anichron.Worker/
├── Bursts/BurstDetection.cs                # handler
├── Bursts/OriginalPhotoReader.cs           # MetadataExtractor: DateTimeOriginal, Apple BurstUUID
├── Work/StorageConfigSchedules.cs          # 3B, replaces CrawlSchedules
└── Work/ActiveStorageConfigs.cs            # shared skip guard, StorageConfigSkipped

• ContentChangedWriter clears the mark (file edited in place is checked again); a moved file keeps mark and burst.
• WorkSchedule<TPayload> is no longer abstract; CrawlSkipped became StorageConfigSkipped (same JSON shape).
• ExifDate.Parse is shared by ExifExtractionMiddleware and the reader.

Beyond the note: a missing root fails the run before any photo is marked, so an unmounted share cannot mark the library as non-candidates. Memberless bursts are loaded and deleted (not one statement); whichever config's run finds one counts it as dissolved.

Evidence

• Gate: MetadataExtractor 2.9.3 on the maintainer's IMG_4386/4387.HEIC yields AppleMakernoteDirectory (no BurstUUID in either), so the label read holds.
• Before: deleting a storage config holding a burst failed on FK_Bursts_MediaAssets_PrimaryAssetId (RESTRICT).
  After: BurstDeleteBehaviourTests (integration): config delete cascades assets and checks, burst stays coverless; deleting the cover nulls it and keeps the other member.
• Handler at the seam (BurstDetectionTests, 38 cases), e.g.

  a(12:02), b(12:01), c(12:03) labelled X  -> {checked 3, created 1}, cover b
  stored {a,b} + {c,d} labelled X, new e   -> {checked 1, merged 1}, all in older burst
  burst with cover soft-deleted            -> cover = earliest active member
  burst left with one active member        -> {dissolved 1}, BurstId cleared
  

• Unit suites pass (pre-push), integration suite passes (58), scripts/mutation-changed.sh 100.00 %. A throwaway run of the handler on real PostgreSQL confirmed the queries translate and dissolve via the tracked SET NULL fix-up.

Merge Danger

Door: one-way

Migration BurstDetection adds BurstChecks and changes Bursts.PrimaryAssetId to nullable SET NULL; the Burst query filter now keeps coverless bursts. Rows written by detection persist.

Blast Radius: Worker

First run per config after deployment reads every photo's original once (cheaper than a crawl, which hashes every file). Crawl results now return StorageConfigSkipped instead of CrawlSkipped (same { reason } shape). No API change.

🤖 Generated with Claude Code