@bongos/core 1.19.449 → 1.19.451

package/tests/land_watch.mjs

@@ -28,9 +28,20 @@ async function ta(name, fn) {
 const STALE = 1000;
 
 // ── classifyLand truth table ─────────────────────────────────────────────────────────
-t('shipped → emit (even when stale)', () => {
+t('shipped inside the watch window → emit', () => {
   assert.equal(classifyLand({ status: 'shipped' }, 10, { staleMs: STALE }), 'emit');
-  assert.equal(classifyLand({ status: 'shipped' }, 999999, { staleMs: STALE }), 'emit');
+});
+// task 1003466 — the card-spam bug. 'emit' used to beat 'stale', so a breadcrumb for a
+// task that landed days ago still rendered a "Shipped" banner. With watch() emitting one
+// card per run, a backlog of N stale crumbs cost the builder N prompt-turns of stale news.
+t('shipped but PAST the watch window → drop (a land the builder moved past is not news)', () => {
+  assert.equal(classifyLand({ status: 'shipped' }, 999999, { staleMs: STALE }), 'drop');
+  assert.equal(classifyLand({ status: 'shipped' }, STALE + 1, { staleMs: STALE }), 'drop');
+  assert.equal(classifyLand({ status: 'shipped' }, STALE, { staleMs: STALE }), 'emit', 'exactly at the boundary still emits');
+});
+t('an unknown age → drop (an unparseable breadcrumb must not be immortal)', () => {
+  assert.equal(classifyLand({ status: 'shipped' }, Infinity, { staleMs: STALE }), 'drop');
+  assert.equal(classifyLand({ status: 'confirmed' }, NaN, { staleMs: STALE }), 'drop');
 });
 t('confirmed (still landing) → keep', () => {
   assert.equal(classifyLand({ status: 'confirmed' }, 10, { staleMs: STALE }), 'keep');
@@ -97,7 +108,41 @@ await ta('emits the NEWEST shipped card, keeps confirmed, drops the rest — one
   assert.equal(env.title, 'task_3_card', 'newest shipped wins');
   assert.ok(removed.includes('f3'), 'the emitted crumb is cleared');
   assert.ok(!removed.includes('f2'), 'the still-confirmed crumb is kept');
-  assert.ok(!removed.includes('f1'), 'the 2nd shipped waits for the next turn (one card per run)');
+  assert.ok(removed.includes('f1'), 'the 2nd shipped is drained in the SAME pass, not held for another turn');
+  assert.ok(/task 1\b/.test((env.warnings || []).join(' ')), 'the 2nd shipped is folded into one warning line');
+});
+// task 1003466 — the whole backlog drains in ONE pass. Before this, five landed tasks cost
+// five prompt-turns of cards; now it is one card plus a single "also landed" line.
+await ta('a backlog of shipped crumbs drains in one pass: one card + one also-landed line', async () => {
+  const removed = [];
+  const crumbs = ['1', '2', '3', '4', '5'].map((id, i) => ({
+    taskId: id, at: `2026-07-01T00:00:0${i + 1}.000Z`, _file: `f${id}`, cardInputs: {},
+  }));
+  const env = await watch({
+    nowMs: Date.parse('2026-07-01T00:00:05.500Z'),
+    staleMs: 60_000,
+    getTask: async () => ({ status: 'shipped' }),
+    readCrumbs: () => crumbs,
+    removeCrumb: (f) => removed.push(f),
+  });
+  assert.equal(env.title, 'task_5_card', 'the newest land is the one that gets a card');
+  assert.equal(removed.length, 5, 'every crumb is cleared in this single pass');
+  const warn = (env.warnings || []).join(' ');
+  for (const id of ['1', '2', '3', '4']) assert.ok(warn.includes(`task ${id}`), `task ${id} named in the warning`);
+  assert.ok(!warn.includes('task 5'), 'the carded task is not repeated in the warning');
+  assert.ok(!/#d/.test(warn), 'never a bare #NNN in emitted prose (it autolinks to a GitHub issue)');
+});
+await ta('stale shipped crumbs are dropped silently — no card, no warning', async () => {
+  const removed = [];
+  const env = await watch({
+    nowMs: Date.parse('2026-07-08T00:00:00.000Z'), // a week later
+    staleMs: 30 * 60 * 1000,
+    getTask: async () => ({ status: 'shipped' }),
+    readCrumbs: () => [{ taskId: '77', at: '2026-07-01T00:00:00.000Z', _file: 'f77', cardInputs: {} }],
+    removeCrumb: (f) => removed.push(f),
+  });
+  assert.equal(env, null, 'a week-old land renders nothing');
+  assert.ok(removed.includes('f77'), 'and the crumb is cleared so it never returns');
 });
 await ta('no shipped → no envelope; drops the give-ups', async () => {
   const removed = [];