Skip to content

v2.3.1 — Fix silent data loss in SyncPoller

Choose a tag to compare

View all tags
@pitimon pitimon released this 02 Apr 23:20
· 31 commits to main since this release
v2.3.1
aa94a3c
This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode.

CRITICAL FIX: Silent data loss prevented

2,580 observations were permanently lost on a production client due to a JavaScript 0 || N falsy bug that reported zero syncs as full-batch success, advancing the watermark past unsynced data.

Fixes

  1. 0 || N falsy bug (CRITICAL) — syncBatch() now treats zero-sync on non-empty batch as failure
  2. Unbounded batch queries (HIGH) — Added LIMIT 100 + batch loop to prevent OOM and timeout
  3. All-or-nothing watermark (HIGH) — Watermark now advances only past successfully synced items

Update

claude plugin update memforge-client@pitimon-c-memforge

After update — verify sync works

# Reset watermark to re-sync
rm -f ~/.memforge/.sync-watermark.json

# Restart Claude Code — should see batched sync in logs:
# [SyncPoller] Fresh install — backfilling N existing observations

Closes #51

Assets 2
Loading