@sleep2agi/commhub-server 0.9.0-preview.70 → 0.9.0-preview.71

package/src/requirements-start-date.test.ts

@@ -0,0 +1,125 @@
+// 需求的可选开始字段(start,存 start_on):App 甘特图用。
+// 先用「没有 start_on 列」的旧库形状建库、塞行,再走真实启动路径(db.ts + server.ts),看:
+// 列被加上、存量行读出来 start = "" 且别的列原样;POST / PATCH / upsert 都能写;只 PATCH due 不动 start 和描述;
+// 旧客户端(不带 start)的 PATCH 不清掉已有的 start;坏值 400 invalid_start;capabilities 带 start_date。
+import { afterAll, beforeAll, describe, expect, test } from "bun:test";
+import { Database } from "bun:sqlite";
+import { mkdtempSync, rmSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+
+const dir = mkdtempSync(join(tmpdir(), "anet-req-start-"));
+const file = join(dir, "hub.db");
+process.env.COMMHUB_DB = file;
+
+const legacy = new Database(file);
+legacy.exec(`
+  CREATE TABLE IF NOT EXISTS requirements (
+    requirement_id TEXT PRIMARY KEY,
+    network_id     TEXT NOT NULL,
+    title          TEXT NOT NULL,
+    column_name    TEXT NOT NULL DEFAULT 'pool' CHECK(column_name IN ('pool', 'doing', 'done')),
+    priority       TEXT NOT NULL DEFAULT 'normal' CHECK(priority IN ('high', 'normal', 'low', 'lowest')),
+    due_on         TEXT,
+    assignee       TEXT,
+    client_id      TEXT,
+    issues_json    TEXT,
+    created_by     TEXT,
+    created_at     TEXT NOT NULL DEFAULT (datetime('now')),
+    updated_at     TEXT NOT NULL DEFAULT (datetime('now'))
+  );
+`);
+legacy.exec("ALTER TABLE requirements ADD COLUMN description TEXT");
+legacy.run("INSERT INTO requirements (requirement_id, network_id, title, due_on, description) VALUES ('r_old', 'net_legacy', '旧卡', '2026-10-01', '旧描述')");
+expect(legacy.query("SELECT name FROM pragma_table_info('requirements') WHERE name = 'start_on'").all()).toEqual([]);
+legacy.close();
+
+let base = "";
+let token = "";
+let server: { port: number; stop?: (force?: boolean) => void };
+let db: any;
+
+async function api(path: string, init?: RequestInit) {
+  const res = await fetch(`${base}${path}`, { ...init, headers: { Authorization: `Bearer ${token}`, "Content-Type": "application/json" } });
+  return { status: res.status, body: await res.json() as any };
+}
+
+beforeAll(async () => {
+  ({ db } = await import("./db.js"));
+  const { register } = await import("./auth.js");
+  const owner = register(`start_owner_${Date.now()}`, "StartOwner123!", undefined, "seed");
+  token = owner.token!;
+  const mod: any = await import("./server.js");
+  server = mod.bootServer({ port: 0, hostname: "127.0.0.1" });
+  base = `http://127.0.0.1:${server.port}`;
+}, 30_000);
+
+afterAll(() => {
+  try { server?.stop?.(true); } catch {}
+  try { rmSync(dir, { recursive: true, force: true }); } catch {}
+});
+
+describe("optional requirement start date", () => {
+  test("startup adds start_on to an old table; existing rows keep every value and read start = ''", () => {
+    const cols = (db.all("PRAGMA table_info(requirements)") as { name: string }[]).map(c => c.name);
+    expect(cols).toContain("start_on");
+    const row = db.get("SELECT title, due_on, description, start_on FROM requirements WHERE requirement_id = 'r_old'") as any;
+    expect(row).toEqual({ title: "旧卡", due_on: "2026-10-01", description: "旧描述", start_on: null });
+  });
+
+  test("create / PATCH / upsert write start; due-only PATCH and start-less PATCH keep it", async () => {
+    const created = await api("/api/requirements", { method: "POST", body: JSON.stringify({ name: "有开始的", start: "2026-09-20", due: "2026-10-01", description: "别动我" }) });
+    expect(created.status).toBe(201);
+    expect(created.body.requirement.start).toBe("2026-09-20");
+    const id = created.body.requirement.id;
+
+    // 甘特图拖动只发 due:start 和描述原样
+    const dueOnly = await api(`/api/requirements/${id}`, { method: "PATCH", body: JSON.stringify({ due: "2026-10-05" }) });
+    expect(dueOnly.status).toBe(200);
+    expect(dueOnly.body.requirement.due).toBe("2026-10-05");
+    expect(dueOnly.body.requirement.start).toBe("2026-09-20");
+    expect(dueOnly.body.requirement.description).toBe("别动我");
+
+    // 旧客户端不认识 start:它的 PATCH 不带这个字段,不能清掉
+    const old = await api(`/api/requirements/${id}`, { method: "PATCH", body: JSON.stringify({ name: "改名了" }) });
+    expect(old.body.requirement.start).toBe("2026-09-20");
+
+    const instant = await api(`/api/requirements/${id}`, { method: "PATCH", body: JSON.stringify({ start: "2026-09-21T09:30:00+08:00" }) });
+    expect(instant.body.requirement.start).toBe("2026-09-21T01:30:00Z");
+    const cleared = await api(`/api/requirements/${id}`, { method: "PATCH", body: JSON.stringify({ start: "" }) });
+    expect(cleared.status).toBe(200);
+    expect(cleared.body.requirement.start).toBe("");
+    // 只带 start 的 PATCH 不是 empty_patch
+    const onlyStart = await api(`/api/requirements/${id}`, { method: "PATCH", body: JSON.stringify({ start: "2026-09-22" }) });
+    expect(onlyStart.status).toBe(200);
+
+    const up = await api("/api/requirements/upsert", { method: "POST", body: JSON.stringify({ name: "同步来的", external_ref: "github:t/p#9", start: "2026-09-01" }) });
+    expect(up.status).toBe(201);
+    expect(up.body.requirement.start).toBe("2026-09-01");
+    const upAgain = await api("/api/requirements/upsert", { method: "POST", body: JSON.stringify({ external_ref: "github:t/p#9", start: "2026-09-02" }) });
+    expect(upAgain.body.requirement.start).toBe("2026-09-02");
+
+    const noStart = await api("/api/requirements", { method: "POST", body: JSON.stringify({ name: "没开始字段" }) });
+    expect(noStart.body.requirement.start).toBe("");
+  });
+
+  test("bad start values are 400 invalid_start and change nothing", async () => {
+    const created = await api("/api/requirements", { method: "POST", body: JSON.stringify({ name: "坏值", start: "2026-09-20" }) });
+    const id = created.body.requirement.id;
+    for (const bad of ["2026-02-30", "tomorrow", "2026-09-20T10:00:00", 20260920]) {
+      const post = await api("/api/requirements", { method: "POST", body: JSON.stringify({ name: "x", start: bad }) });
+      const patch = await api(`/api/requirements/${id}`, { method: "PATCH", body: JSON.stringify({ start: bad }) });
+      expect(patch.status).toBe(400);
+      expect(patch.body.error).toBe("invalid_start");
+      if (typeof bad === "string") {
+        expect(post.status).toBe(400);
+        expect(post.body.error).toBe("invalid_start");
+      }
+    }
+    const listed = await api("/api/requirements");
+    expect(listed.body.requirements.find((r: any) => r.id === id).start).toBe("2026-09-20");
+    expect(listed.body.capabilities).toContain("start_date");
+    // 旧卡在列表里 start = ""(不是缺字段:新 App 按 capabilities 判断支持与否)
+    expect(listed.body.requirements.every((r: any) => typeof r.start === "string")).toBe(true);
+  });
+});

package/src/requirements.ts

@@ -1,5 +1,6 @@
 // 需求池。长期卡片，存在 Hub 上，手机和电脑读同一份。
 // 不是 tasks：tasks 是正在派给节点的活，状态由节点收尾。
+import { createHash } from "node:crypto";
 import { db } from "./db.js";
 import { normalizeTags, storedTags } from "./requirement-tags.js";
 import { addHumanNetworkScope, canRestWriteNetwork, canRestWriteNetworkAsHuman, resolveRestWriteNetworkId, type RestNetworkScope } from "./network-scope.js";
@@ -52,6 +53,7 @@ type Row = {
   column_name: string;
   priority: string;
   due_on: string | null;
+  start_on: string | null;
   assignee: string | null;
   issues_json: string | null;
   tags_json: string | null;
@@ -285,6 +287,8 @@ function toPublic(row: Row) {
     priority: row.priority,
     assignee: row.assignee || "",
     due: row.due_on || "",
+    // 开始(可空,甘特图用)。形状同 due;没设 = ""。旧 App 不认识这个字段,忽略即可。
+    start: row.start_on || "",
     column: row.column_name,
     createdAt: row.created_at,
     updatedAt: row.updated_at || row.created_at,
@@ -353,7 +357,7 @@ function toPublicFor(ctx: RequirementsRequestContext, row: Row) {
   };
 }
 
-const SELECT = "requirement_id, network_id, title, column_name, priority, due_on, assignee, issues_json, tags_json, created_at, owner_json, participants_json, agent_owner_json, description, checklist_json, project_id, external_ref, external_url, archived, created_by, created_by_json, updated_by_json, updated_at, parent_id, " +
+const SELECT = "requirement_id, network_id, title, column_name, priority, due_on, assignee, issues_json, tags_json, created_at, owner_json, participants_json, agent_owner_json, description, checklist_json, project_id, external_ref, external_url, archived, created_by, created_by_json, updated_by_json, updated_at, parent_id, start_on, " +
   "(SELECT COUNT(*) FROM requirements c WHERE c.parent_id = requirements.requirement_id AND COALESCE(c.archived, 0) = 0) AS children_total, " +
   "(SELECT COUNT(*) FROM requirements c WHERE c.parent_id = requirements.requirement_id AND COALESCE(c.archived, 0) = 0 AND c.column_name = 'done') AS children_done";
 
@@ -427,7 +431,28 @@ function operationOf(req: Request, url: URL): string {
   return "create";
 }
 
-export const REQUIREMENT_CAPABILITIES = ["agent_owner", "description", "checklist", "projects", "due_datetime", "external_ref", "archived", "agent_api", "sub_requirements", "tags", "priority_lowest"] as const;
+/**
+ * GET /api/requirements 的条件请求:ETag = 响应体的哈希,`If-None-Match` 对上就回 304 空体。
+ * app 每 15 s 轮询整张表(生产 500 行 708 KB / gzip 163 KB,中国 → 美国约 0.65 s 纯传输),
+ * 绝大多数轮询里表根本没变。按**响应体**算,所以对这个调用方可见的任何字段变了都会变
+ * (同一张表不同成员看到的不同,ETag 也不同)。不带 If-None-Match 的旧客户端照旧拿 200 全量。
+ */
+export function conditionalJson(req: Request, payload: unknown): Response {
+  const body = JSON.stringify(payload);
+  const etag = `W/"${createHash("sha256").update(body).digest("base64url").slice(0, 27)}"`;
+  // private:按用户可见范围生成,不能被共享缓存复用;no-cache:每次都要回来验证,不会拿旧表当新的。
+  const headers = { ETag: etag, "Cache-Control": "private, no-cache" };
+  if (ifNoneMatchHits(req.headers.get("if-none-match"), etag)) return new Response(null, { status: 304, headers });
+  return new Response(body, { headers: { ...headers, "Content-Type": "application/json;charset=utf-8" } });
+}
+
+function ifNoneMatchHits(header: string | null, etag: string): boolean {
+  if (!header) return false;
+  const bare = (tag: string) => tag.trim().replace(/^W\//, "");
+  return header.split(",").some(tag => tag.trim() === "*" || bare(tag) === bare(etag));
+}
+
+export const REQUIREMENT_CAPABILITIES = ["agent_owner", "description", "checklist", "projects", "due_datetime", "external_ref", "archived", "agent_api", "sub_requirements", "tags", "priority_lowest", "start_date"] as const;
 
 // ── 子需求 ──
 // parent_id:同一网络里的另一张卡;不能成环;最多 5 层(顶层是第 1 层)。删父卡 = 子卡保留、parent_id 置空。
@@ -534,6 +559,8 @@ async function createRequirement(ctx: RequirementsRequestContext, body: Record<s
   if (!PRIORITIES.has(priority)) return jsonError("invalid_priority", 400);
   const due = typeof body.due === "string" ? normalizeDue(body.due) : "";
   if (due === null) return jsonError("invalid_due", 400);
+  const start = typeof body.start === "string" ? normalizeDue(body.start) : "";
+  if (start === null) return jsonError("invalid_start", 400);
   const assignee = typeof body.assignee === "string" ? body.assignee.trim().slice(0, 80) : "";
   const clientId = typeof body.client_id === "string" ? body.client_id.trim() : "";
   if (clientId && !/^[A-Za-z0-9._-]{1,80}$/.test(clientId)) return jsonError("invalid_client_id", 400);
@@ -577,9 +604,9 @@ async function createRequirement(ctx: RequirementsRequestContext, body: Record<s
   try {
     db.run(
       `INSERT INTO requirements
-       (requirement_id, network_id, title, column_name, priority, due_on, assignee, client_id, issues_json, created_by, created_at, updated_at, owner_json, participants_json, agent_owner_json, description, checklist_json, project_id, external_ref, external_url, created_by_json, updated_by_json, archived, parent_id, tags_json)
-       VALUES (?1, ?2, ?3, ?4, ?5, ?6, ?7, ?8, ?9, ?10, ?11, ?11, ?12, ?13, ?14, ?15, ?16, ?17, ?18, ?19, ?20, ?20, 0, ?21, ?22)`,
-      [id, networkId, name, column, priority, due || null, assignee, clientId || null, JSON.stringify(issues), ctx.auth?.userId ?? null, createdAt, people.ownerJson, people.participantsJson, people.agentOwnerJson, description || null, JSON.stringify(checklist), projectId, ref, extUrl, actor, parentId, JSON.stringify(tags)],
+       (requirement_id, network_id, title, column_name, priority, due_on, assignee, client_id, issues_json, created_by, created_at, updated_at, owner_json, participants_json, agent_owner_json, description, checklist_json, project_id, external_ref, external_url, created_by_json, updated_by_json, archived, parent_id, tags_json, start_on)
+       VALUES (?1, ?2, ?3, ?4, ?5, ?6, ?7, ?8, ?9, ?10, ?11, ?11, ?12, ?13, ?14, ?15, ?16, ?17, ?18, ?19, ?20, ?20, 0, ?21, ?22, ?23)`,
+      [id, networkId, name, column, priority, due || null, assignee, clientId || null, JSON.stringify(issues), ctx.auth?.userId ?? null, createdAt, people.ownerJson, people.participantsJson, people.agentOwnerJson, description || null, JSON.stringify(checklist), projectId, ref, extUrl, actor, parentId, JSON.stringify(tags), start || null],
     );
   } catch {
     // 并发的同一个 external_ref / client_id:唯一索引挡住了第二个,回已有的那条。
@@ -597,7 +624,7 @@ async function createRequirement(ctx: RequirementsRequestContext, body: Record<s
 }
 
 // ── 修改(省略的字段保留原值) ──
-const PATCH_FIELDS = ["column", "issues", "tags", "name", "priority", "due", "assignee", "owner", "agent_owner", "participants", "description", "checklist", "project_id", "external_ref", "external_url", "archived", "parent_id"];
+const PATCH_FIELDS = ["column", "issues", "tags", "name", "priority", "due", "assignee", "owner", "agent_owner", "participants", "description", "checklist", "project_id", "external_ref", "external_url", "archived", "parent_id", "start"];
 function patchRequirement(ctx: RequirementsRequestContext, row: Row, body: Record<string, unknown>): Response {
   if (!PATCH_FIELDS.some(k => Object.prototype.hasOwnProperty.call(body, k))) return jsonError("empty_patch", 400);
   const has = (k: string) => Object.prototype.hasOwnProperty.call(body, k);
@@ -628,6 +655,13 @@ function patchRequirement(ctx: RequirementsRequestContext, row: Row, body: Recor
     if (next === null) return jsonError("invalid_due", 400);
     due = next;
   }
+  let start = row.start_on || "";
+  if (has("start")) {
+    if (typeof body.start !== "string") return jsonError("invalid_start", 400);
+    const next = normalizeDue(body.start);
+    if (next === null) return jsonError("invalid_start", 400);
+    start = next;
+  }
   let assignee = row.assignee || "";
   if (has("assignee")) {
     if (typeof body.assignee !== "string") return jsonError("invalid_assignee", 400);
@@ -673,10 +707,10 @@ function patchRequirement(ctx: RequirementsRequestContext, row: Row, body: Recor
   const updatedAt = new Date().toISOString();
   db.run(
     `UPDATE requirements SET column_name = ?1, updated_at = ?2, title = ?7, priority = ?8, due_on = ?9, assignee = ?10, owner_json = ?4, participants_json = ?5, issues_json = ?6,
-       agent_owner_json = ?11, description = ?12, checklist_json = ?13, project_id = ?14, external_ref = ?15, external_url = ?16, archived = ?17, updated_by_json = ?18, parent_id = ?19, tags_json = ?20
+       agent_owner_json = ?11, description = ?12, checklist_json = ?13, project_id = ?14, external_ref = ?15, external_url = ?16, archived = ?17, updated_by_json = ?18, parent_id = ?19, tags_json = ?20, start_on = ?21
      WHERE requirement_id = ?3`,
     [has("column") ? body.column : row.column_name, updatedAt, row.requirement_id, people.ownerJson, people.participantsJson, has("issues") ? JSON.stringify(issues) : row.issues_json, name, priority, due || null, assignee, people.agentOwnerJson,
-      has("description") ? (description || null) : row.description, has("checklist") ? JSON.stringify(checklist) : row.checklist_json, projectId, ref, extUrl, archived, JSON.stringify(actorOf(ctx)), parentId, has("tags") ? JSON.stringify(tags) : row.tags_json],
+      has("description") ? (description || null) : row.description, has("checklist") ? JSON.stringify(checklist) : row.checklist_json, projectId, ref, extUrl, archived, JSON.stringify(actorOf(ctx)), parentId, has("tags") ? JSON.stringify(tags) : row.tags_json, start || null],
   );
   const updated = db.get<Row>(`SELECT ${SELECT} FROM requirements WHERE requirement_id = ?1`, row.requirement_id)!;
   return Response.json(withLegacyOwner(toPublicFor(ctx, updated), people.coerced));
@@ -784,7 +818,7 @@ export async function handleRequirementsRequest(ctx: RequirementsRequestContext)
     if (typeof filtered !== "string") return filtered;
     const rows = db.all<Row>(`${filtered} ORDER BY created_at DESC LIMIT 500`, ...params).map(row => toPublicFor(ctx, row));
     // capabilities:客户端按这个决定显示哪些功能(预计完成能不能带时刻、有没有项目…),不用靠猜字段。
-    return Response.json({ ok: true, requirements: rows, capabilities: REQUIREMENT_CAPABILITIES });
+    return conditionalJson(ctx.req, { ok: true, requirements: rows, capabilities: REQUIREMENT_CAPABILITIES });
   }
 
   if (url.pathname === "/api/requirements" && req.method === "POST") {

package/src/scheduled-tasks-http.test.ts

@@ -1,6 +1,7 @@
-import { afterAll, beforeAll, describe, expect, test } from "bun:test";
+import { afterAll, beforeAll, beforeEach, describe, expect, test } from "bun:test";
 import { existsSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
+import { dirname, join } from "node:path";
+import { fileURLToPath } from "node:url";
 import { tmpdir } from "node:os";
 import { addNetworkMember, createNetworkTokenForNode, register } from "./auth.js";
 import { db } from "./db.js";
@@ -26,6 +27,31 @@ async function api(token: string, path: string, init?: RequestInit) {
   return { status: res.status, body: await res.json() as any };
 }
 
+// #2135 —— 每条测试自己建它要断言的数据:不读前面测试留下的 schedule / run 行,也不靠执行顺序。
+// 单跑任何一条(-t)、从 server/ 或仓根跑,结果都一样。
+async function createSchedule(name: string, extra: Record<string, unknown> = {}) {
+  const res = await api(ownerToken, "/api/scheduled-tasks", {
+    method: "POST",
+    body: JSON.stringify({
+      network_id: networkId,
+      name,
+      target_node_id: nodeId,
+      task: `${name} task`,
+      timezone: "UTC",
+      schedule: { type: "interval", every_seconds: 60 },
+      ...extra,
+    }),
+  });
+  expect(res.status).toBe(201);
+  return res.body.schedule as { schedule_id: string; revision: number; [key: string]: any };
+}
+
+/** Make exactly this schedule due at `at` (default: a minute ago). */
+function makeDue(scheduleId: string, at = new Date(Date.now() - 60_000).toISOString()): string {
+  db.run("UPDATE scheduled_tasks SET next_run_at = ?1 WHERE schedule_id = ?2", [at, scheduleId]);
+  return at;
+}
+
 // 30s 而不是 bun 默认的 5s:这个 hook 跑 register()(KDF)+bootServer(),
 // 本机空载基线 ~620ms,而 2026-08-17 与 2026-08-31 两次 CI 上它被 5s 上限打死
 // (0 pass / 1 fail / Ran 1 test —— 见 #928、#1627)。5247ms 是**被截断的下界**,
@@ -66,8 +92,12 @@ afterAll(() => {
 });
 
 describe("Hub scheduled task API and dispatcher", () => {
-  let scheduleId = "";
-  let revision = 0;
+  // runDueScheduledTasks() sweeps every due schedule in the DB. Park whatever an earlier test
+  // left due (e.g. the race schedule when its workers could not start), so each test's
+  // processed counts only ever see the schedules it made due itself.
+  beforeEach(() => {
+    db.run("UPDATE scheduled_tasks SET next_run_at = ?1 WHERE next_run_at IS NOT NULL", [new Date(Date.now() + 24 * 3600_000).toISOString()]);
+  });
 
   test("schedule math supports timezone-aware daily and weekly forms", () => {
     const daily = nextOccurrence({ type: "daily", time: "09:30" }, "Asia/Shanghai", new Date("2026-08-09T02:00:00Z"));
@@ -90,12 +120,12 @@ describe("Hub scheduled task API and dispatcher", () => {
   });
 
   test("scheduler startup rejects a backend without real transactions", () => {
-    const original = db.dialect;
+    const original = db.atomicTransactions;
     try {
-      (db as any).dialect = "postgres";
+      (db as any).atomicTransactions = false;
       expect(() => assertScheduledTaskBackendSupported()).toThrow("scheduled_tasks_require_transactional_sqlite_backend");
     } finally {
-      (db as any).dialect = original;
+      (db as any).atomicTransactions = original;
     }
     expect(() => assertScheduledTaskBackendSupported()).not.toThrow();
     const misfireColumn = db.all<any>("PRAGMA table_info(scheduled_tasks)").find((column: any) => column.name === "misfire_policy");
@@ -130,11 +160,10 @@ describe("Hub scheduled task API and dispatcher", () => {
     expect(created.body.schedule.misfire_policy).toBe("catch_up_once");
     expect(created.body.schedule.created_by).toBeUndefined();
     expect(created.body.schedule.schedule_json).toBeUndefined();
-    scheduleId = created.body.schedule.schedule_id;
-    revision = created.body.schedule.revision;
   });
 
   test("network scope hides schedules from foreign users and viewers may read", async () => {
+    const scheduleId = (await createSchedule("Scope probe")).schedule_id;
     const visible = await api(viewerToken, `/api/scheduled-tasks?network_id=${encodeURIComponent(networkId)}`);
     expect(visible.status).toBe(200);
     expect(visible.body.schedules.map((x: any) => x.schedule_id)).toContain(scheduleId);
@@ -145,9 +174,9 @@ describe("Hub scheduled task API and dispatcher", () => {
     expect(hidden.status).toBe(403);
   });
 
-  test("due occurrence creates ordinary inbox/task/run atomically and is idempotent", () => {
-    const due = new Date(Date.now() - 120_000).toISOString();
-    db.run("UPDATE scheduled_tasks SET next_run_at = ?1 WHERE schedule_id = ?2", [due, scheduleId]);
+  test("due occurrence creates ordinary inbox/task/run atomically and is idempotent", async () => {
+    const scheduleId = (await createSchedule("Due occurrence probe")).schedule_id;
+    makeDue(scheduleId, new Date(Date.now() - 120_000).toISOString());
     expect(runDueScheduledTasks().processed).toBe(1);
     const run = db.get<any>("SELECT * FROM scheduled_task_runs WHERE schedule_id = ?1 ORDER BY created_at DESC LIMIT 1", scheduleId)!;
     expect(["delivered", "queued"]).toContain(run.status);
@@ -289,11 +318,14 @@ describe("Hub scheduled task API and dispatcher", () => {
 
     const raceDir = mkdtempSync(join(dir, "race-"));
     const gate = join(raceDir, "go");
-    const workerPath = "tests/test601-hub-scheduled-tasks/race-worker.ts";
+    // Resolved from this file, not from the current directory (#2135): `bun test` from server/
+    // used to look for tests/… under server/ and never start the workers.
+    const repoRoot = join(dirname(fileURLToPath(import.meta.url)), "..", "..");
+    const workerPath = join(repoRoot, "tests", "test601-hub-scheduled-tasks", "race-worker.ts");
     const dbPath = activeDbPath;
     const spawnWorker = (id: string) => Bun.spawn(
       ["bun", workerPath, dbPath, raceScheduleId, due, join(raceDir, `ready-${id}`), gate],
-      { cwd: process.cwd(), stdout: "pipe", stderr: "pipe", env: { ...process.env, COMMHUB_DB: dbPath } },
+      { cwd: repoRoot, stdout: "pipe", stderr: "pipe", env: { ...process.env, COMMHUB_DB: dbPath } },
     );
     const waitReady = async (id: string) => {
       const readyPath = join(raceDir, `ready-${id}`);
@@ -326,16 +358,23 @@ describe("Hub scheduled task API and dispatcher", () => {
     expect(db.get<{ count: number }>("SELECT COUNT(*) AS count FROM inbox WHERE meta_json LIKE '%' || ?1 || '%'", raceScheduleId)!.count).toBe(1);
   }, 20_000);
 
-  test("non-overlap skips while prior task is open; rename follows stable node_id", () => {
+  test("non-overlap skips while prior task is open; rename follows stable node_id", async () => {
+    // Its own node: the rename below must not change the node every other test targets.
+    const renameNodeId = `n_sched_rename_${Date.now()}`;
+    db.run("INSERT INTO nodes (node_id, node_name, alias, runtime, network_id) VALUES (?1, 'Rename Probe', 'scheduler-rename-probe', 'codex-sdk', ?2)", [renameNodeId, networkId]);
+    db.run("INSERT INTO sessions (resume_id, alias, status, node_id, network_id) VALUES (?1, 'scheduler-rename-probe', 'idle', ?2, ?3)", [`r_rename_${Date.now()}`, renameNodeId, networkId]);
+    const scheduleId = (await createSchedule("Non-overlap probe", { target_node_id: renameNodeId })).schedule_id;
+    makeDue(scheduleId, new Date(Date.now() - 90_000).toISOString());
+    expect(runDueScheduledTasks().processed).toBe(1);
     const firstTask = db.get<{ task_id: string }>("SELECT task_id FROM scheduled_task_runs WHERE schedule_id = ?1 AND task_id IS NOT NULL LIMIT 1", scheduleId)!.task_id;
-    const due2 = new Date(Date.now() - 60_000).toISOString();
-    db.run("UPDATE scheduled_tasks SET next_run_at = ?1 WHERE schedule_id = ?2", [due2, scheduleId]);
+
+    const due2 = makeDue(scheduleId);
     expect(runDueScheduledTasks().processed).toBe(1);
     expect(db.get<any>("SELECT status FROM scheduled_task_runs WHERE schedule_id = ?1 AND scheduled_for = ?2", scheduleId, due2)!.status).toBe("skipped");
 
     db.run("UPDATE tasks SET status = 'replied', completed_at = datetime('now') WHERE task_id = ?1", [firstTask]);
-    db.run("UPDATE nodes SET alias = 'scheduler-renamed' WHERE node_id = ?1 AND network_id = ?2", [nodeId, networkId]);
-    db.run("UPDATE sessions SET alias = 'scheduler-renamed' WHERE node_id = ?1 AND network_id = ?2", [nodeId, networkId]);
+    db.run("UPDATE nodes SET alias = 'scheduler-renamed' WHERE node_id = ?1 AND network_id = ?2", [renameNodeId, networkId]);
+    db.run("UPDATE sessions SET alias = 'scheduler-renamed' WHERE node_id = ?1 AND network_id = ?2", [renameNodeId, networkId]);
     const due3 = new Date(Date.now() - 30_000).toISOString();
     db.run("UPDATE scheduled_tasks SET next_run_at = ?1 WHERE schedule_id = ?2", [due3, scheduleId]);
     expect(runDueScheduledTasks().processed).toBe(1);
@@ -344,6 +383,7 @@ describe("Hub scheduled task API and dispatcher", () => {
   });
 
   test("full edit validates target and policy, preserves cadence, and recomputes through DST-safe schedule math", async () => {
+    const scheduleId = (await createSchedule("Daily briefing to edit", { priority: "high", timezone: "Asia/Shanghai" })).schedule_id;
     // Pin a recognizable future occurrence so a metadata-only edit can prove
     // that it does not silently reset the interval cadence.
     const pinnedNext = new Date(Date.now() + 45 * 60_000).toISOString();
@@ -413,6 +453,7 @@ describe("Hub scheduled task API and dispatcher", () => {
   });
 
   test("two editors with one revision produce one winner and one refreshable conflict", async () => {
+    const scheduleId = (await createSchedule("Two editors probe")).schedule_id;
     const current = (await api(ownerToken, `/api/scheduled-tasks/${scheduleId}?network_id=${encodeURIComponent(networkId)}`)).body.schedule;
     const [a, b] = await Promise.all([
       api(ownerToken, `/api/scheduled-tasks/${scheduleId}?network_id=${encodeURIComponent(networkId)}`, {
@@ -523,8 +564,18 @@ describe("Hub scheduled task API and dispatcher", () => {
   });
 
   test("optimistic revision, pause/resume, run-now and cancel preserve history", async () => {
+    const scheduleId = (await createSchedule("History probe")).schedule_id;
+    // Its own history before the lifecycle calls: a run, a skipped overlap, and a second run.
+    makeDue(scheduleId, new Date(Date.now() - 180_000).toISOString());
+    expect(runDueScheduledTasks().processed).toBe(1);
+    makeDue(scheduleId, new Date(Date.now() - 120_000).toISOString());
+    expect(runDueScheduledTasks().processed).toBe(1);
+    db.run("UPDATE tasks SET status='replied' WHERE task_id IN (SELECT task_id FROM scheduled_task_runs WHERE schedule_id=?1)", [scheduleId]);
+    makeDue(scheduleId);
+    expect(runDueScheduledTasks().processed).toBe(1);
+    expect(db.get<{ n: number }>("SELECT COUNT(*) AS n FROM scheduled_task_runs WHERE schedule_id = ?1", scheduleId)!.n).toBe(3);
     const latest = await api(ownerToken, `/api/scheduled-tasks/${scheduleId}?network_id=${encodeURIComponent(networkId)}`);
-    revision = latest.body.schedule.revision;
+    const revision = latest.body.schedule.revision;
     const stale = await api(ownerToken, `/api/scheduled-tasks/${scheduleId}?network_id=${encodeURIComponent(networkId)}`, { method: "PATCH", body: JSON.stringify({ revision: revision - 1, status: "paused" }) });
     expect(stale.status).toBe(409);
     const paused = await api(ownerToken, `/api/scheduled-tasks/${scheduleId}?network_id=${encodeURIComponent(networkId)}`, { method: "PATCH", body: JSON.stringify({ revision, status: "paused" }) });

package/src/scheduled-tasks.ts

@@ -57,8 +57,9 @@ const MISFIRE_POLICIES = new Set<MisfirePolicy>(["catch_up_once", "skip"]);
 export const MISFIRE_GRACE_MS = 60_000;
 
 export function assertScheduledTaskBackendSupported(): void {
-  if (db.dialect !== "sqlite") {
-    throw new Error("scheduled_tasks_require_transactional_sqlite_backend");
+  const refusal = db.transactionalFeaturesRefusal;
+  if (refusal !== null) {
+    throw new Error(`scheduled_tasks_require_transactional_sqlite_backend${refusal ? `: ${refusal}` : ""}`);
   }
 }