Summary
sync run --dry-run on a legacy .one/sync/data/<plat>.db migration predicts inserted: 30 on rows that a real run would updated. Cosmetic only — no data impact — but misleading.
Repro
- Have a legacy SQLite DB at
.one/sync/data/fathom.db with rows that already exist in the memory store under the same source keys.
one --agent sync run fathom --dry-run
- Output reports
inserted: N where every row would hit the upsert-by-keys updated branch on a real run.
Expected
Dry-run runs the same classify-before-upsert path as a live run (or at least tests mem_records.keys && p_keys against the current set) so the predicted action matches.
Notes
Summary
sync run --dry-runon a legacy.one/sync/data/<plat>.dbmigration predictsinserted: 30on rows that a real run wouldupdated. Cosmetic only — no data impact — but misleading.Repro
.one/sync/data/fathom.dbwith rows that already exist in the memory store under the same source keys.one --agent sync run fathom --dry-runinserted: Nwhere every row would hit the upsert-by-keysupdatedbranch on a real run.Expected
Dry-run runs the same classify-before-upsert path as a live run (or at least tests
mem_records.keys && p_keysagainst the current set) so the predicted action matches.Notes