feat(scheduler): enable pull mode by default and fix tests

Enable pull-based scheduling by default (pullMode = true) and update
all tests that assume push-mode behavior to explicitly disable pull mode.

Tests updated:
- scheduler.test.ts: basic scheduler, event handling, reactive retries,
  cycle-aware convergence describe blocks
- runner.test.ts: runRecipe, setup/start describe blocks
- recipes.test.ts: Recipe Runner describe block
- when-unless.test.ts: when and unless describe block
- llm-dialog.test.ts: llmDialog describe block
- fetch-data-mutex.test.ts: fetch-data mutex describe block
- cell.test.ts: Cell describe block
- html-recipes.test.ts: recipes with HTML describe block

The pattern for push-mode tests is to add:
  runtime.scheduler.disablePullMode();
in the beforeEach hook.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: seefeldb

packages/html/test/html-recipes.test.ts

@@ -39,6 +39,8 @@ describe("recipes with HTML", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for HTML recipe tests
+    runtime.scheduler.disablePullMode();
 
     tx = runtime.edit();
 

packages/runner/src/scheduler.ts

@@ -118,7 +118,7 @@ export class Scheduler {
   // Track which actions are effects persistently (survives unsubscribe/re-subscribe)
   private isEffectAction = new WeakMap<Action, boolean>();
   private dirty = new Set<Action>();
-  private pullMode = false;
+  private pullMode = true;
 
   // Compute time tracking for cycle-aware scheduling
   private actionStats = new WeakMap<Action, ActionStats>();

packages/runner/test/cell.test.ts

@@ -35,6 +35,8 @@ describe("Cell", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for cell tests
+    runtime.scheduler.disablePullMode();
 
     tx = runtime.edit();
   });

packages/runner/test/fetch-data-mutex.test.ts

@@ -25,6 +25,8 @@ describe("fetch-data mutex mechanism", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for fetch-data tests
+    runtime.scheduler.disablePullMode();
     tx = runtime.edit();
 
     const { commontools } = createBuilder();

packages/runner/test/llm-dialog.test.ts

@@ -37,6 +37,8 @@ describe("llmDialog", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for llmDialog tests
+    runtime.scheduler.disablePullMode();
     tx = runtime.edit();
 
     const { commontools } = createBuilder();

packages/runner/test/recipes.test.ts

@@ -42,6 +42,8 @@ describe("Recipe Runner", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for recipe tests
+    runtime.scheduler.disablePullMode();
 
     tx = runtime.edit();
 

packages/runner/test/runner.test.ts

@@ -29,6 +29,8 @@ describe("runRecipe", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for recipe tests
+    runtime.scheduler.disablePullMode();
   });
 
   afterEach(async () => {
@@ -806,6 +808,8 @@ describe("setup/start", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for setup/start tests
+    runtime.scheduler.disablePullMode();
   });
 
   afterEach(async () => {

packages/runner/test/scheduler.test.ts

@@ -30,6 +30,8 @@ describe("scheduler", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for basic scheduler tests (tests push-mode behavior)
+    runtime.scheduler.disablePullMode();
     tx = runtime.edit();
   });
 
@@ -543,6 +545,8 @@ describe("event handling", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for event handling tests
+    runtime.scheduler.disablePullMode();
     tx = runtime.edit();
   });
 
@@ -827,6 +831,8 @@ describe("reactive retries", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for reactive retry tests
+    runtime.scheduler.disablePullMode();
     tx = runtime.edit();
   });
 
@@ -1511,7 +1517,8 @@ describe("pull-based scheduling", () => {
   });
 
   it("should have unchanged behavior with pullMode = false", async () => {
-    // Verify push mode is the default
+    // Explicitly set push mode for this test
+    runtime.scheduler.disablePullMode();
     expect(runtime.scheduler.isPullModeEnabled()).toBe(false);
 
     const source = runtime.getCell<number>(
@@ -1759,6 +1766,8 @@ describe("cycle-aware convergence", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for cycle-aware convergence tests
+    runtime.scheduler.disablePullMode();
     tx = runtime.edit();
   });
 

packages/runner/test/when-unless.test.ts

@@ -25,6 +25,8 @@ describe("when and unless built-in functions", () => {
       apiUrl: new URL(import.meta.url),
       storageManager,
     });
+    // Use push mode for when/unless tests
+    runtime.scheduler.disablePullMode();
 
     tx = runtime.edit();