IonWarpbotcommented 2 min agoIonWarp Review — PR #337
3 issues| 2 P1 1 P2
#SevIssueFile
1Bare catch reintroduced — 4th recurrencesync/worker.ts
2Fix shipped as opt-in flag on stored statelib/rollout.ts
3Per-instance patch; sibling sites keep the bugqueue/drain.ts
sync/worker.ts
545556
−++
try { await push(row); } catch (error) { /* dropped */ }try { await push(row); } catch (error) {throw new PushFailed(row.id, error); // learning: swallowed-sync-error
Reintroduced a learning the team fixed three times · Repeat-Failure Learnings
This exact swallowed-sync-error shape was re-patched in #212, #260 and #291. The cached learning says: rethrow named, never drop.
Suggested fixAI fix prompt