v0.64.0: market top-up + coverage spawn run on every tick including frozen ones, so a grounded fleet can never drain the market to zero (#135 follow-up)

lib/sim/engine.test.ts

@@ -641,6 +641,24 @@ describe('grounded fleet freezes the world clock (#135)', () => {
 
     expect(sim.clockMs).toBeGreaterThan(STARTING_CLOCK_MS); // the repair gate is what keeps time moving
   });
+
+  it('a frozen tick still tops up the market so a grounded fleet never starves (#135 follow-up)', async () => {
+    const inst = await freshDb('frozen-topup.db'); // seeded starter is on the ground → frozen
+    // Drain the open market to zero the way repeated ignores would (removal paths run while frozen).
+    for (const p of await inst.db.select().from(schema.packages)) {
+      if (p.status === 'open') {
+        await inst.db.update(schema.packages).set({ status: 'ignored' }).where(eq(schema.packages.id, p.id));
+      }
+    }
+
+    vi.setSystemTime(T0 + 1_000); // a background tick while the fleet is grounded
+    const sim = await tickOnce(inst);
+
+    expect(sim.clockMs).toBe(STARTING_CLOCK_MS); // freeze preserved…
+    const openRows = (await inst.db.select().from(schema.packages)).filter((p) => p.status === 'open');
+    expect(openRows.length).toBeGreaterThanOrEqual(MARKET_TARGET_OPEN_PACKAGES); // …but the market is topped back up…
+    expect(openRows.some((p) => p.originIcao === 'EDDF')).toBe(true); // …including coverage for the grounded starter's airport
+  });
 });
 
 describe('spawnOpenPackages', () => {

lib/sim/engine.ts

@@ -69,8 +69,8 @@ function withLock<T>(fn: () => Promise<T>): Promise<T> {
 
 // #135: advance decision — pure, so unit tests feed it synthetic fleets. The world clock runs at
 // full scale only while something is happening (at least one plane flying OR in repair); a fully
-// grounded fleet freezes it entirely — no expiry/spawn/repair progress until a dispatch or repair
-// starts again.
+// grounded fleet freezes it entirely — no expiry/delivery/repair progress until a dispatch or
+// repair starts again (the market top-up still runs on frozen ticks, #135 follow-up).
 export function shouldAdvanceTime(planes: Array<{ status: string }>): { advance: boolean; flying: number; repairing: number } {
   let flying = 0;
   let repairing = 0;
@@ -87,8 +87,9 @@ export async function tickOnce(inst: GameDb): Promise<schema.SimStateRow> {
   if (!sim) throw new Error('sim_state row missing');
 
   const nowMs = Date.now();
-  // #135: a fully grounded fleet freezes the world — no advance, no expiry/spawn/repair progress.
-  // lastRealTickMs always tracks real time, so resuming activity never fast-forwards the frozen span.
+  // #135: a fully grounded fleet freezes the world — no advance, no expiry/delivery/repair progress
+  // (the market top-up still runs on frozen ticks — see below). lastRealTickMs always tracks real
+  // time, so resuming activity never fast-forwards the frozen span.
   const planeRows = await inst.db.select().from(schema.planes);
   const { advance } = shouldAdvanceTime(planeRows.map((p) => ({ status: p.status })));
   // #135 (v0.62.0): production ALWAYS runs at the fixed world rate — legacy saves may hold a stale
@@ -105,11 +106,15 @@ export async function tickOnce(inst: GameDb): Promise<schema.SimStateRow> {
     // v0.39.0 (ISS-069): repairs whose gate has passed flip back to 'available' — runs even when
     // nothing landed this tick; BEFORE the spawns so a just-repaired plane gets coverage work too.
     await finishRepairs(inst, clockMs);
-    await spawnOpenPackages(inst, clockMs);
-    // AFTER delivery on purpose: a plane that just landed gets work spawned
-    // at its new location within the same tick.
-    await spawnCoveragePackages(inst, clockMs);
   }
+  // #135 follow-up (v0.64.0): market top-up + coverage spawn run on EVERY tick — frozen ones
+  // included — because while grounded nothing else creates packages but removal paths (ignore)
+  // still run; without this a grounded fleet could drain the market to zero and soft-lock forever.
+  // Both are idempotent no-ops when satisfied, so advancing ticks behave exactly as before.
+  await spawnOpenPackages(inst, clockMs);
+  // AFTER delivery on purpose: a plane that just landed gets work spawned
+  // at its new location within the same tick.
+  await spawnCoveragePackages(inst, clockMs);
 
   // Persist lastRealTickMs even when nothing advanced — it is the tick anchor.
   await inst.db.update(schema.simState).set({ clockMs, lastRealTickMs: nowMs });