@sleep2agi/commhub-server 0.9.0-preview.83 → 0.9.0-preview.85

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sleep2agi/commhub-server",
-  "version": "0.9.0-preview.83",
+  "version": "0.9.0-preview.85",
   "description": "CommHub Server — AI Agent communication hub with MCP protocol, multi-network isolation, user auth, and MCP tools (17 collaboration-core + node/provider ops tools; authoritative list: docs-site/docs/api/mcp-tools.md).",
   "type": "module",
   "main": "src/index.ts",

package/src/auth.ts

@@ -1,7 +1,7 @@
 /**
  * V3 Auth module — user registration, login, token management
  */
-import { db, generateId, hashPassword, verifyPassword, hashToken, generateToken, generateUserToken, generateNetworkToken, uuidv4 } from "./db.js";
+import { db, generateId, hashPassword, verifyPassword, hashToken, generateToken, generateUserToken, generateNetworkToken, uuidv4 } from "./db.js"; import { deleteDepartmentsForNetwork } from "./departments.js"; // #419: also ensures network_members.department_id exists (same line on purpose: docs pin auth.ts line numbers)
 import { WEAK_PASSWORDS } from "./password-dict.js";
 import { NETWORK_REST_COLUMNS, NETWORK_REST_SELECT, sqlColumns } from "./rest-projections.js";
 import { deleteAgentGrants, isAgentRestricted } from "./agent-access.js"; import { deleteTaskGrantsForMember, NEW_MEMBER_TASK_ACCESS, type TaskAccessMode } from "./task-access.js"; // 一行两个 import:文档钉着 auth.ts 的行号
@@ -445,7 +445,7 @@ export function deleteNetwork(userId: string, networkId: string): { ok: boolean;
     // single-network user to "ambiguous" or auto-resolve writes INTO the
     // deleted network. Remove memberships with the network.
     db.run("DELETE FROM network_members WHERE network_id = ?1", [networkId]);
-    deleteNetworkAgentGroups(networkId); // #2144
+    deleteNetworkAgentGroups(networkId); deleteDepartmentsForNetwork(networkId); // #2144, #419 (one line: docs pin auth.ts line numbers)
   });
   return { ok: true };
 }
@@ -588,7 +588,8 @@ export function getNetworkMembers(networkId: string) {
             (SELECT COUNT(*) FROM network_member_agent_grants g WHERE g.network_id = nm.network_id AND g.user_id = nm.user_id) AS agent_grant_count,
             CASE WHEN u.role = 'admin' OR nm.role IN ('owner', 'admin') OR nm.task_access = 'all' THEN 'all' ELSE 'scoped' END AS task_access,
             (SELECT COUNT(*) FROM network_member_project_grants pg WHERE pg.network_id = nm.network_id AND pg.user_id = nm.user_id) AS task_project_count,
-            (SELECT COUNT(*) FROM network_member_group_grants gg WHERE gg.network_id = nm.network_id AND gg.user_id = nm.user_id) AS agent_group_count
+            (SELECT COUNT(*) FROM network_member_group_grants gg WHERE gg.network_id = nm.network_id AND gg.user_id = nm.user_id) AS agent_group_count,
+            nm.department_id
      FROM network_members nm JOIN users u ON nm.user_id = u.user_id
      WHERE nm.network_id = ?1 ORDER BY nm.joined_at`,
     networkId);
@@ -836,6 +837,8 @@ const NETWORK_CLEANUP_TABLES = [
   "task_events", "task_terminal_events", "scheduled_task_runs", "external_schedule_edits",
   // 标签颜色:网络本身的元数据(标签本体在 requirements 行里,那张表是阻断项)。
   "network_tags",
+  // 组织架构(board #419):部门是网络本身的元数据。
+  "network_departments",
 ];
 
 function tableExists(name: string): boolean {

package/src/departments-http.test.ts

@@ -0,0 +1,185 @@
+// 组织架构(board #419):GET/POST/PATCH/DELETE /api/networks/:id/departments、PUT …/members/:uid/department。
+// HTTP 集成测试(真实 Bun.serve,私有端口,临时库)。钉住:谁能读 / 谁能写、部门规则(同级不重名、不成环、最多 10 层、
+// 只删空部门、负责人必须是成员)、成员所在部门,以及只增不改(/humans、/members 只多一个 department_id)。
+
+import { describe, expect, test, beforeAll, afterAll } from "bun:test";
+import { mkdtempSync, rmSync } from "fs";
+import { join } from "path";
+import { tmpdir } from "os";
+import { register, createNetworkTokenForNode } from "./auth.js";
+import { db } from "./db.js";
+
+const DIR = mkdtempSync(join(tmpdir(), "anet-departments-"));
+let BASE = "";
+let hub: any = null;
+const PW = "DepartmentsPassw0rd!x";
+let NET = "", OTHER_NET = "";
+const U: Record<string, { token: string; id: string }> = {};
+
+async function send(token: string, method: string, path: string, body?: unknown) {
+  const res = await fetch(`${BASE}${path}`, { method, headers: { ...(token ? { Authorization: `Bearer ${token}` } : {}), "Content-Type": "application/json" }, body: body === undefined ? undefined : JSON.stringify(body) });
+  const text = await res.text();
+  let json: any = null;
+  try { json = JSON.parse(text); } catch {}
+  return { status: res.status, body: json };
+}
+const D = () => `/api/networks/${NET}/departments`;
+const list = async (who = "owner") => (await send(U[who].token, "GET", D())).body;
+const mk = async (name: string, extra: Record<string, unknown> = {}) => {
+  const r = await send(U.owner.token, "POST", D(), { name, ...extra });
+  expect(r.status).toBe(201);
+  return r.body.department as { id: string; name: string; parent_id: string | null };
+};
+
+beforeAll(async () => {
+  process.env.COMMHUB_DB ||= join(DIR, "hub.db");
+  process.env.COMMHUB_UPLOADS_DIR = join(DIR, "uploads");
+  process.env.HOST = "127.0.0.1";
+  const a = register(`dep_owner_${Date.now()}`, PW, undefined, "Owner");
+  expect(a.ok).toBe(true);
+  U.owner = { token: a.token!, id: a.user!.user_id };
+  NET = a.network_id!;
+  db.run("UPDATE users SET role = 'admin' WHERE user_id = ?1", [U.owner.id]);
+  const mod: any = await import("./server.js");
+  hub = mod.bootServer({ port: 0, hostname: "127.0.0.1" });
+  BASE = `http://127.0.0.1:${hub.port}`;
+  const add = async (username: string, role: string) => {
+    expect((await send(U.owner.token, "POST", "/api/admin/users", { username, password: PW, network_id: NET, role })).status).toBe(200);
+    const login = await send("", "POST", "/api/auth/login", { username, password: PW });
+    expect(login.status).toBe(200);
+    return { token: login.body.token as string, id: login.body.user.user_id as string };
+  };
+  const stamp = Date.now();
+  U.admin = await add(`dep_admin_${stamp}`, "admin");
+  U.member = await add(`dep_member_${stamp}`, "member");
+  U.viewer = await add(`dep_viewer_${stamp}`, "viewer");
+  U.node = { token: createNetworkTokenForNode(U.owner.id, NET, "dep-node", "node_dep").token!, id: "node_dep" };
+  const o = register(`dep_outsider_${stamp}`, PW);
+  U.outsider = { token: o.token!, id: o.user!.user_id };
+  OTHER_NET = o.network_id!;
+}, 30_000);
+
+afterAll(() => {
+  try { hub?.stop?.(true); } catch {}
+  try { rmSync(DIR, { recursive: true, force: true }); } catch {}
+});
+
+describe("who can read / write", () => {
+  test("every member, a node token of this network and a hub admin can read; outsiders and other networks' nodes can't", async () => {
+    for (const who of ["owner", "admin", "member", "viewer", "node"]) expect((await send(U[who].token, "GET", D())).status).toBe(200);
+    expect((await send(U.outsider.token, "GET", D())).status).toBe(403);
+    expect((await send("", "GET", D())).status).toBe(401);
+    const otherNode = createNetworkTokenForNode(U.outsider.id, OTHER_NET, "dep-other", "node_dep_other").token!;
+    expect((await send(otherNode, "GET", D())).status).toBe(403);
+  });
+
+  test("only network owner / admin (and hub admin) can write", async () => {
+    expect((await send(U.admin.token, "POST", D(), { name: "管理员建的" })).status).toBe(201);
+    for (const who of ["member", "viewer", "node", "outsider"]) {
+      expect((await send(U[who].token, "POST", D(), { name: `不该建-${who}` })).status).toBe(403);
+    }
+    const anyDept = (await list()).departments[0].id;
+    expect((await send(U.member.token, "PATCH", `${D()}/${anyDept}`, { name: "x" })).status).toBe(403);
+    expect((await send(U.member.token, "DELETE", `${D()}/${anyDept}`)).status).toBe(403);
+    expect((await send(U.member.token, "PUT", `/api/networks/${NET}/members/${U.member.id}/department`, { department_id: anyDept })).status).toBe(403);
+  });
+});
+
+describe("departments", () => {
+  test("create: top level and child, auto id and custom id, sibling names unique, validation", async () => {
+    const top = await mk("军团基建");
+    expect(top.id).toMatch(/^dept_/);
+    expect(top.parent_id).toBeNull();
+    const child = await mk("前端组", { parent_id: top.id, id: "fe-team" });
+    expect(child.id).toBe("fe-team");
+    expect(child.parent_id).toBe(top.id);
+    expect((await send(U.owner.token, "POST", D(), { name: "前端组", parent_id: top.id })).status).toBe(409);
+    expect((await send(U.owner.token, "POST", D(), { name: "前端组" })).status).toBe(201); // same name under another parent is fine
+    expect((await send(U.owner.token, "POST", D(), { name: "另一个", id: "fe-team" })).body.error).toBe("department_id_taken");
+    expect((await send(U.owner.token, "POST", D(), { name: "  " })).body.error).toBe("invalid_department_name");
+    expect((await send(U.owner.token, "POST", D(), { name: "x".repeat(41) })).body.error).toBe("invalid_department_name");
+    expect((await send(U.owner.token, "POST", D(), { name: "坏 id", id: "has space" })).body.error).toBe("invalid_department_id");
+    expect((await send(U.owner.token, "POST", D(), { name: "孤儿", parent_id: "nope" })).body.error).toBe("parent_not_found");
+    expect((await send(U.owner.token, "POST", D(), { name: "外人负责", leader_user_id: U.outsider.id })).body.error).toBe("leader_not_member");
+  });
+
+  test("rename, move (no cycles, depth ≤ 10), set / clear leader, sort", async () => {
+    const a = await mk("移动甲");
+    const b = await mk("移动乙", { parent_id: a.id });
+    const c = await mk("移动丙", { parent_id: b.id });
+    expect((await send(U.owner.token, "PATCH", `${D()}/${a.id}`, { parent_id: c.id })).body.error).toBe("department_cycle");
+    expect((await send(U.owner.token, "PATCH", `${D()}/${a.id}`, { parent_id: a.id })).body.error).toBe("department_cycle");
+    const moved = await send(U.owner.token, "PATCH", `${D()}/${c.id}`, { parent_id: null, name: "移动丙(顶层)" });
+    expect(moved.status).toBe(200);
+    expect(moved.body.department.parent_id).toBeNull();
+    expect(moved.body.department.name).toBe("移动丙(顶层)");
+    const led = await send(U.owner.token, "PATCH", `${D()}/${b.id}`, { leader_user_id: U.member.id, sort: 5 });
+    expect(led.body.department.leader_user_id).toBe(U.member.id);
+    expect(led.body.department.sort).toBe(5);
+    expect((await send(U.owner.token, "PATCH", `${D()}/${b.id}`, { leader_user_id: null })).body.department.leader_user_id).toBeNull();
+    expect((await send(U.owner.token, "PATCH", `${D()}/${b.id}`, {})).body.error).toBe("empty_patch");
+    expect((await send(U.owner.token, "PATCH", `${D()}/missing`, { name: "x" })).status).toBe(404);
+    // depth: a chain of 10 is fine, the 11th level is refused (create and move)
+    let parent: string | null = null;
+    const chain: string[] = [];
+    for (let i = 0; i < 10; i++) { const d = await mk(`层${i}`, parent ? { parent_id: parent } : {}); chain.push(d.id); parent = d.id; }
+    expect((await send(U.owner.token, "POST", D(), { name: "第11层", parent_id: parent })).body.error).toBe("department_too_deep");
+    const two = await mk("两层");
+    await mk("两层的子", { parent_id: two.id });
+    expect((await send(U.owner.token, "PATCH", `${D()}/${two.id}`, { parent_id: chain[8] })).body.error).toBe("department_too_deep");
+  });
+
+  test("members: put into a department, counts, unassign; /members (admin list) carries department_id, /humans unchanged", async () => {
+    const team = await mk("成员测试部");
+    expect((await send(U.owner.token, "PUT", `/api/networks/${NET}/members/${U.member.id}/department`, { department_id: team.id })).body.department_id).toBe(team.id);
+    expect((await send(U.owner.token, "PUT", `/api/networks/${NET}/members/${U.viewer.id}/department`, { department_id: team.id })).status).toBe(200);
+    const l = await list("member");
+    expect(l.departments.find((d: any) => d.id === team.id).member_count).toBe(2);
+    expect(l.members.find((m: any) => m.user_id === U.member.id).department_id).toBe(team.id);
+    expect(l.members.find((m: any) => m.user_id === U.owner.id).department_id).toBeNull();
+    // /humans stays identity-only (its key set is pinned by member-presence / agent-acl tests): who sits where comes
+    // from GET …/departments (members[]), which every member can read.
+    const humans = await send(U.viewer.token, "GET", `/api/networks/${NET}/humans`);
+    expect(humans.body.humans.find((h: any) => h.user_id === U.member.id).department_id).toBeUndefined();
+    const members = await send(U.owner.token, "GET", `/api/networks/${NET}/members`);
+    const row = members.body.members.find((m: any) => m.user_id === U.member.id);
+    expect(row.department_id).toBe(team.id);
+    for (const k of ["user_id", "role", "username", "agent_access", "task_access"]) expect(row).toHaveProperty(k); // old fields unchanged
+    expect((await send(U.owner.token, "PUT", `/api/networks/${NET}/members/${U.outsider.id}/department`, { department_id: team.id })).status).toBe(404);
+    expect((await send(U.owner.token, "PUT", `/api/networks/${NET}/members/${U.member.id}/department`, { department_id: "nope" })).body.error).toBe("department_not_found");
+    expect((await send(U.owner.token, "PUT", `/api/networks/${NET}/members/${U.viewer.id}/department`, { department_id: null })).body.department_id).toBeNull();
+  });
+
+  test("delete only when empty (no sub-departments, no members), with the counts", async () => {
+    const parent = await mk("删除测试部");
+    const kid = await mk("删除测试子部", { parent_id: parent.id });
+    await send(U.owner.token, "PUT", `/api/networks/${NET}/members/${U.admin.id}/department`, { department_id: parent.id });
+    const blocked = await send(U.owner.token, "DELETE", `${D()}/${parent.id}`);
+    expect(blocked.status).toBe(409);
+    expect(blocked.body).toMatchObject({ error: "department_not_empty", children: 1, members: 1 });
+    expect((await send(U.owner.token, "DELETE", `${D()}/${kid.id}`)).status).toBe(200);
+    await send(U.owner.token, "PUT", `/api/networks/${NET}/members/${U.admin.id}/department`, { department_id: null });
+    expect((await send(U.owner.token, "DELETE", `${D()}/${parent.id}`)).status).toBe(200);
+    expect((await list()).departments.some((d: any) => d.id === parent.id)).toBe(false);
+    expect((await send(U.owner.token, "DELETE", `${D()}/${parent.id}`)).status).toBe(404);
+  });
+
+  test("a leader who leaves the network reads as no leader", async () => {
+    const stamp = Date.now();
+    expect((await send(U.owner.token, "POST", "/api/admin/users", { username: `dep_leaver_${stamp}`, password: PW, network_id: NET, role: "member" })).status).toBe(200);
+    const leaver = (await send("", "POST", "/api/auth/login", { username: `dep_leaver_${stamp}`, password: PW })).body.user.user_id as string;
+    const d = await mk("负责人会走", { leader_user_id: leaver });
+    expect(d).toBeTruthy();
+    expect((await send(U.owner.token, "DELETE", `/api/networks/${NET}/members/${leaver}`)).status).toBe(200);
+    expect((await list()).departments.find((x: any) => x.id === d.id).leader_user_id).toBeNull();
+  });
+
+  test("deleting a network removes its departments", async () => {
+    const r = register(`dep_gone_${Date.now()}`, PW);
+    const net = r.network_id!;
+    expect((await send(r.token!, "POST", `/api/networks/${net}/departments`, { name: "会被删" })).status).toBe(201);
+    expect(db.get("SELECT 1 AS x FROM network_departments WHERE network_id = ?1", net)).toBeTruthy();
+    expect((await send(r.token!, "DELETE", `/api/networks/${net}`)).status).toBe(200);
+    expect(db.get("SELECT 1 AS x FROM network_departments WHERE network_id = ?1", net)).toBeFalsy();
+  });
+});

package/src/departments.ts

@@ -0,0 +1,233 @@
+// 人类成员的组织架构(board #419):部门(多层)+ 每个成员归一个部门 + 部门负责人。
+//
+// 只增不改:新表 network_departments,network_members 多一列 department_id(默认 NULL = 未分配)。
+// 旧 app 不读这两样,行为不变。网络本身就是根(parent_id = NULL 的部门挂在网络下面)。
+//
+// 权限:网络里任何成员(含 viewer、受限成员)都能读组织架构 —— 和 /humans 通讯录同一可见范围;
+// 写(建 / 改名 / 移动 / 删除部门、设负责人、调人)只给网络 owner / admin 和 Hub 管理员。节点令牌只能读自己网络的。
+//
+// /humans 通讯录保持只有身份字段(它的键集合被测试钉着);谁在哪个部门从 GET …/departments 的 members[] 读。
+// 规则:同一上级下部门不重名;移动不能挂到自己或自己的下级下面;最多 MAX_DEPTH 层;
+// 只有空部门(没有子部门、没有成员)能删;负责人必须是本网络成员;人离开网络后,作为负责人的引用读出来是 null。
+
+import { db, logAudit } from "./db.js";
+
+try { db.exec("ALTER TABLE network_members ADD COLUMN department_id TEXT"); } catch {}
+db.exec(`
+  CREATE TABLE IF NOT EXISTS network_departments (
+    network_id     TEXT NOT NULL,
+    department_id  TEXT NOT NULL,
+    name           TEXT NOT NULL,
+    parent_id      TEXT,
+    leader_user_id TEXT,
+    sort           INTEGER NOT NULL DEFAULT 0,
+    created_by     TEXT,
+    created_at     TEXT NOT NULL DEFAULT (datetime('now')),
+    updated_at     TEXT NOT NULL DEFAULT (datetime('now')),
+    PRIMARY KEY (network_id, department_id)
+  );
+  CREATE INDEX IF NOT EXISTS idx_network_departments_parent ON network_departments(network_id, parent_id);
+`);
+
+export const MAX_DEPTH = 10;
+export const MAX_DEPARTMENTS = 500;
+const NAME_MAX = 40;
+const ID_RE = /^[A-Za-z0-9_-]{1,64}$/;
+
+type DeptRow = { network_id: string; department_id: string; name: string; parent_id: string | null; leader_user_id: string | null; sort: number; created_at: string; updated_at: string };
+export type Department = { id: string; name: string; parent_id: string | null; leader_user_id: string | null; sort: number; member_count: number; created_at: string; updated_at: string };
+
+type Fail = { ok: false; status: number; error: string };
+type Ok<T> = { ok: true } & T;
+
+const fail = (status: number, error: string): Fail => ({ ok: false, status, error });
+
+function rows(networkId: string): DeptRow[] {
+  return db.all<DeptRow>(
+    "SELECT network_id, department_id, name, parent_id, leader_user_id, sort, created_at, updated_at FROM network_departments WHERE network_id = ?1 ORDER BY sort, created_at, department_id",
+    networkId,
+  );
+}
+function one(networkId: string, id: string): DeptRow | null {
+  return db.get<DeptRow>(
+    "SELECT network_id, department_id, name, parent_id, leader_user_id, sort, created_at, updated_at FROM network_departments WHERE network_id = ?1 AND department_id = ?2",
+    networkId, id,
+  ) ?? null;
+}
+function isMember(networkId: string, userId: string): boolean {
+  return !!db.get<{ user_id: string }>("SELECT user_id FROM network_members WHERE network_id = ?1 AND user_id = ?2", networkId, userId);
+}
+
+/** 读:部门(带直属人数)+ 每个成员在哪个部门。负责人不在本网络了 → null。 */
+export function listDepartments(networkId: string): { departments: Department[]; members: Array<{ user_id: string; department_id: string | null }> } {
+  const members = db.all<{ user_id: string; department_id: string | null }>(
+    "SELECT user_id, department_id FROM network_members WHERE network_id = ?1 ORDER BY joined_at, user_id",
+    networkId,
+  );
+  const memberIds = new Set(members.map((m) => m.user_id));
+  const all = rows(networkId);
+  const ids = new Set(all.map((d) => d.department_id));
+  // 成员行指着一个已经不存在的部门(理论上删除时已清空;防御)→ 当未分配。
+  const placed = members.map((m) => ({ user_id: m.user_id, department_id: m.department_id && ids.has(m.department_id) ? m.department_id : null }));
+  const counts = new Map<string, number>();
+  for (const m of placed) if (m.department_id) counts.set(m.department_id, (counts.get(m.department_id) ?? 0) + 1);
+  return {
+    departments: all.map((d) => ({
+      id: d.department_id,
+      name: d.name,
+      parent_id: d.parent_id,
+      leader_user_id: d.leader_user_id && memberIds.has(d.leader_user_id) ? d.leader_user_id : null,
+      sort: d.sort,
+      member_count: counts.get(d.department_id) ?? 0,
+      created_at: d.created_at,
+      updated_at: d.updated_at,
+    })),
+    members: placed,
+  };
+}
+
+const cleanName = (raw: unknown): string | null => {
+  if (typeof raw !== "string") return null;
+  const name = raw.trim();
+  return name && [...name].length <= NAME_MAX ? name : null;
+};
+
+function depthOf(networkId: string, id: string | null): number {
+  let depth = 0;
+  let cur = id;
+  const seen = new Set<string>();
+  while (cur) {
+    if (seen.has(cur)) return Infinity;
+    seen.add(cur);
+    depth += 1;
+    cur = one(networkId, cur)?.parent_id ?? null;
+  }
+  return depth;
+}
+/** 以 id 为根的子树最深有几层(含自己)。 */
+function subtreeHeight(networkId: string, id: string): number {
+  const children = db.all<{ department_id: string }>("SELECT department_id FROM network_departments WHERE network_id = ?1 AND parent_id = ?2", networkId, id);
+  return 1 + children.reduce((m, c) => Math.max(m, subtreeHeight(networkId, c.department_id)), 0);
+}
+function isDescendant(networkId: string, ancestor: string, id: string | null): boolean {
+  let cur = id;
+  const seen = new Set<string>();
+  while (cur) {
+    if (cur === ancestor) return true;
+    if (seen.has(cur)) return true;
+    seen.add(cur);
+    cur = one(networkId, cur)?.parent_id ?? null;
+  }
+  return false;
+}
+function siblingNameTaken(networkId: string, parentId: string | null, name: string, exceptId?: string): boolean {
+  const sql = parentId === null
+    ? "SELECT department_id FROM network_departments WHERE network_id = ?1 AND parent_id IS NULL AND name = ?2"
+    : "SELECT department_id FROM network_departments WHERE network_id = ?1 AND parent_id = ?3 AND name = ?2";
+  const hits = parentId === null ? db.all<{ department_id: string }>(sql, networkId, name) : db.all<{ department_id: string }>(sql, networkId, name, parentId);
+  return hits.some((h) => h.department_id !== exceptId);
+}
+function publicOne(networkId: string, id: string): Department {
+  return listDepartments(networkId).departments.find((d) => d.id === id)!;
+}
+
+export function createDepartment(networkId: string, actor: string, body: Record<string, unknown>): Ok<{ department: Department }> | Fail {
+  const name = cleanName(body.name);
+  if (!name) return fail(400, "invalid_department_name");
+  let id: string;
+  if (body.id !== undefined && body.id !== null && body.id !== "") {
+    if (typeof body.id !== "string" || !ID_RE.test(body.id)) return fail(400, "invalid_department_id");
+    if (one(networkId, body.id)) return fail(409, "department_id_taken");
+    id = body.id;
+  } else {
+    id = `dept_${crypto.randomUUID().replace(/-/g, "").slice(0, 16)}`;
+  }
+  const parentId = body.parent_id === undefined || body.parent_id === null || body.parent_id === "" ? null : body.parent_id;
+  if (parentId !== null && (typeof parentId !== "string" || !one(networkId, parentId))) return fail(400, "parent_not_found");
+  if (depthOf(networkId, parentId as string | null) + 1 > MAX_DEPTH) return fail(400, "department_too_deep");
+  if (siblingNameTaken(networkId, parentId as string | null, name)) return fail(409, "department_name_taken");
+  const leader = body.leader_user_id === undefined || body.leader_user_id === null || body.leader_user_id === "" ? null : body.leader_user_id;
+  if (leader !== null && (typeof leader !== "string" || !isMember(networkId, leader))) return fail(400, "leader_not_member");
+  if (body.sort !== undefined && !Number.isInteger(body.sort)) return fail(400, "invalid_sort");
+  const count = db.get<{ n: number }>("SELECT COUNT(*) AS n FROM network_departments WHERE network_id = ?1", networkId)?.n ?? 0;
+  if (count >= MAX_DEPARTMENTS) return fail(400, "too_many_departments");
+  const sort = body.sort === undefined
+    ? (db.get<{ n: number }>(
+      parentId === null
+        ? "SELECT COUNT(*) AS n FROM network_departments WHERE network_id = ?1 AND parent_id IS NULL"
+        : "SELECT COUNT(*) AS n FROM network_departments WHERE network_id = ?1 AND parent_id = ?2",
+      ...(parentId === null ? [networkId] : [networkId, parentId]),
+    )?.n ?? 0)
+    : body.sort as number;
+  db.run(
+    "INSERT INTO network_departments (network_id, department_id, name, parent_id, leader_user_id, sort, created_by) VALUES (?1, ?2, ?3, ?4, ?5, ?6, ?7)",
+    [networkId, id, name, parentId, leader, sort, actor],
+  );
+  return { ok: true, department: publicOne(networkId, id) };
+}
+
+export function updateDepartment(networkId: string, id: string, body: Record<string, unknown>): Ok<{ department: Department }> | Fail {
+  const cur = one(networkId, id);
+  if (!cur) return fail(404, "department_not_found");
+  let name = cur.name;
+  let parentId = cur.parent_id;
+  let leader = cur.leader_user_id;
+  let sort = cur.sort;
+  let touched = false;
+  if (body.name !== undefined) {
+    const n = cleanName(body.name);
+    if (!n) return fail(400, "invalid_department_name");
+    name = n; touched = true;
+  }
+  if (body.parent_id !== undefined) {
+    const p = body.parent_id === null || body.parent_id === "" ? null : body.parent_id;
+    if (p !== null && (typeof p !== "string" || !one(networkId, p))) return fail(400, "parent_not_found");
+    if (p !== null && isDescendant(networkId, id, p as string)) return fail(400, "department_cycle");
+    if (depthOf(networkId, p as string | null) + subtreeHeight(networkId, id) > MAX_DEPTH) return fail(400, "department_too_deep");
+    parentId = p as string | null; touched = true;
+  }
+  if (body.leader_user_id !== undefined) {
+    const l = body.leader_user_id === null || body.leader_user_id === "" ? null : body.leader_user_id;
+    if (l !== null && (typeof l !== "string" || !isMember(networkId, l))) return fail(400, "leader_not_member");
+    leader = l as string | null; touched = true;
+  }
+  if (body.sort !== undefined) {
+    if (!Number.isInteger(body.sort)) return fail(400, "invalid_sort");
+    sort = body.sort as number; touched = true;
+  }
+  if (!touched) return fail(400, "empty_patch");
+  if ((name !== cur.name || parentId !== cur.parent_id) && siblingNameTaken(networkId, parentId, name, id)) return fail(409, "department_name_taken");
+  db.run(
+    "UPDATE network_departments SET name = ?3, parent_id = ?4, leader_user_id = ?5, sort = ?6, updated_at = datetime('now') WHERE network_id = ?1 AND department_id = ?2",
+    [networkId, id, name, parentId, leader, sort],
+  );
+  return { ok: true, department: publicOne(networkId, id) };
+}
+
+/** 只删空部门:有子部门 / 有成员 → 409(带数字,界面照着说)。 */
+export function deleteDepartment(networkId: string, id: string): Ok<{ deleted: string }> | (Fail & { children?: number; members?: number }) {
+  if (!one(networkId, id)) return fail(404, "department_not_found");
+  const children = db.get<{ n: number }>("SELECT COUNT(*) AS n FROM network_departments WHERE network_id = ?1 AND parent_id = ?2", networkId, id)?.n ?? 0;
+  const members = db.get<{ n: number }>("SELECT COUNT(*) AS n FROM network_members WHERE network_id = ?1 AND department_id = ?2", networkId, id)?.n ?? 0;
+  if (children || members) return { ...fail(409, "department_not_empty"), children, members };
+  db.run("DELETE FROM network_departments WHERE network_id = ?1 AND department_id = ?2", [networkId, id]);
+  return { ok: true, deleted: id };
+}
+
+/** 把成员放进一个部门(null = 未分配)。 */
+export function setMemberDepartment(networkId: string, userId: string, departmentId: unknown): Ok<{ user_id: string; department_id: string | null }> | Fail {
+  if (!isMember(networkId, userId)) return fail(404, "member_not_found");
+  const dept = departmentId === null || departmentId === "" ? null : departmentId;
+  if (dept !== null && (typeof dept !== "string" || !one(networkId, dept))) return fail(400, "department_not_found");
+  db.run("UPDATE network_members SET department_id = ?3 WHERE network_id = ?1 AND user_id = ?2", [networkId, userId, dept]);
+  return { ok: true, user_id: userId, department_id: dept as string | null };
+}
+
+/** 网络删掉时一起清(和项目授权同一时机调用)。 */
+export function deleteDepartmentsForNetwork(networkId: string): void {
+  db.run("DELETE FROM network_departments WHERE network_id = ?1", [networkId]);
+}
+
+export function auditDepartment(user: { user_id: string; username: string }, action: string, networkId: string, detail: string): void {
+  logAudit(user.user_id, user.username, action, "network", networkId, detail.slice(0, 2000), undefined, networkId);
+}

package/src/node-health-passthrough.test.ts

@@ -0,0 +1,115 @@
+// #448 — report_status.health passthrough (layered node health, report-only).
+//
+// Hub-side contract verified here:
+//   1. A well-formed `health` from the node's own token is kept (memory-only) and readable.
+//   2. Malformed `health` never rejects the report — the node stays online, health reads as null.
+//   3. Another node / a user login cannot speak for this node's health.
+//   4. Stale (> TTL) health reads as null, never as "healthy".
+//
+// 跑法：cd server && COMMHUB_DB=/tmp/node-health.db bun test src/node-health-passthrough.test.ts
+import { afterAll, beforeEach, describe, expect, test } from "bun:test";
+import { Client } from "@modelcontextprotocol/sdk/client/index.js";
+import { InMemoryTransport } from "@modelcontextprotocol/sdk/inMemory.js";
+import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
+import { db } from "./db.js";
+import { registerTools } from "./tools.js";
+import { clearNodeHealthStore, NODE_HEALTH_TTL_MS, readNodeHealth, recordNodeHealth } from "./node-health-store.js";
+
+const NET = "net_health_t";
+const OWNER = "u_health_owner";
+const NODE_A = "health-node-a";
+const NODE_B = "health-node-b";
+const NODE_A_ID = "node_health_a";
+const NODE_B_ID = "node_health_b";
+const T_A = "tok_health_node_a";
+const T_B = "tok_health_node_b";
+
+function cleanup() {
+  for (const t of ["sessions", "nodes", "api_tokens", "network_members", "networks"]) {
+    try { db.run(`DELETE FROM ${t} WHERE network_id = ?1`, [NET]); } catch {}
+  }
+  try { db.run("DELETE FROM users WHERE user_id = ?1", [OWNER]); } catch {}
+  clearNodeHealthStore();
+}
+function seed() {
+  db.run(`INSERT INTO users (user_id, username, password_hash, role, created_at) VALUES (?1, ?2, 'x', 'user', datetime('now'))`, [OWNER, OWNER]);
+  db.run(`INSERT INTO networks (network_id, network_name, owner_id, created_at) VALUES (?1, ?2, ?3, datetime('now'))`, [NET, NET, OWNER]);
+  db.run(`INSERT INTO network_members (user_id, network_id, role, joined_at) VALUES (?1, ?2, 'owner', datetime('now'))`, [OWNER, NET]);
+  for (const [id, alias, tok] of [[NODE_A_ID, NODE_A, T_A], [NODE_B_ID, NODE_B, T_B]]) {
+    db.run(`INSERT INTO nodes (node_id, node_name, alias, network_id, owner_user_id, created_at, updated_at) VALUES (?1, ?2, ?3, ?4, ?5, datetime('now'), datetime('now'))`, [id, alias, alias, NET, OWNER]);
+    db.run(`INSERT INTO api_tokens (token_id, user_id, network_id, scope, name, token_hash, expires_at, revoked_at, bound_node_id) VALUES (?1, ?2, ?3, 'network', ?4, ?5, NULL, NULL, ?6)`, [tok, OWNER, NET, `node:${alias}`, `hash_${tok}`, id]);
+  }
+}
+
+async function connectAs(alias: string, tokenId: string) {
+  const server = new McpServer({ name: "health-test", version: "1" });
+  registerTools(server, undefined, NET, OWNER, alias, true, tokenId);
+  const client = new Client({ name: "health-client", version: "1" });
+  const [ct, st] = InMemoryTransport.createLinkedPair();
+  await server.connect(st);
+  await client.connect(ct);
+  const call = async (args: Record<string, unknown>) => {
+    const r: any = await client.callTool({ name: "report_status", arguments: args });
+    return { isError: r.isError === true, body: JSON.parse(r.content?.[0]?.text ?? "null") };
+  };
+  return { call, close: async () => { await client.close(); await server.close(); } };
+}
+
+const HEALTH = {
+  bridge: "ok",
+  app_server: { ok: false, rtt_ms: null, last_error: "connection refused" },
+  tui: { ok: false, reason: "sleep-placeholder" },
+  model_auth: "revoked",
+};
+
+beforeEach(() => { cleanup(); seed(); });
+afterAll(cleanup);
+
+describe("#448 report_status.health passthrough", () => {
+  test("node's own token: stored and readable as reported", async () => {
+    const a = await connectAs(NODE_A, T_A);
+    try {
+      const r = await a.call({ resume_id: `sdk-${NODE_A_ID}`, alias: NODE_A, status: "idle", node_id: NODE_A_ID, health: HEALTH });
+      expect(r.isError).toBe(false);
+      expect(r.body.ok).not.toBe(false);
+    } finally { await a.close(); }
+    expect(readNodeHealth(NET, NODE_A)?.health).toEqual(HEALTH as any);
+  });
+
+  test("malformed health never rejects the report (node stays online); it reads as not reported", async () => {
+    const a = await connectAs(NODE_A, T_A);
+    try {
+      for (const bad of [42, "ok", { app_server: { ok: "yes" }, model_auth: "weird", tui: [] }, { app_server: { ok: true, rtt_ms: NaN, last_error: 5 } }]) {
+        const r = await a.call({ resume_id: `sdk-${NODE_A_ID}`, alias: NODE_A, status: "idle", node_id: NODE_A_ID, health: bad });
+        expect(r.isError).toBe(false);
+      }
+    } finally { await a.close(); }
+    expect(db.get<{ s: string }>("SELECT status AS s FROM sessions WHERE alias = ?1 AND network_id = ?2", NODE_A, NET)?.s).toBe("idle");
+    // 最后一份里 app_server.ok 合法、其余两格降级成 null —— 留下的是能证实的部分,不是编出来的。
+    expect(readNodeHealth(NET, NODE_A)?.health.app_server).toEqual({ ok: true, rtt_ms: null, last_error: null });
+  });
+
+  test("another node's token cannot speak for this node", async () => {
+    const b = await connectAs(NODE_B, T_B);
+    try {
+      await b.call({ resume_id: `sdk-${NODE_A_ID}`, alias: NODE_A, status: "idle", node_id: NODE_A_ID, health: HEALTH });
+    } finally { await b.close(); }
+    expect(readNodeHealth(NET, NODE_A)).toBeNull();
+  });
+
+  test("stale health reads as null, never as healthy", () => {
+    const t0 = 1_000_000;
+    recordNodeHealth(NET, NODE_A, { bridge: "ok", model_auth: "ok" }, t0);
+    expect(readNodeHealth(NET, NODE_A, t0 + 1_000)?.observed_ms_ago).toBe(1_000);
+    expect(readNodeHealth(NET, NODE_A, t0 + NODE_HEALTH_TTL_MS + 1)).toBeNull();
+  });
+
+  test("a report without health leaves the last one (heartbeats from older code paths do not erase it)", async () => {
+    const a = await connectAs(NODE_A, T_A);
+    try {
+      await a.call({ resume_id: `sdk-${NODE_A_ID}`, alias: NODE_A, status: "idle", node_id: NODE_A_ID, health: HEALTH });
+      await a.call({ resume_id: `sdk-${NODE_A_ID}`, alias: NODE_A, status: "working", node_id: NODE_A_ID });
+    } finally { await a.close(); }
+    expect(readNodeHealth(NET, NODE_A)?.health.model_auth).toBe("revoked");
+  });
+});

package/src/node-health-store.ts

@@ -0,0 +1,57 @@
+// #448 —— agent-node 上报的分层健康(`report_status.health`)的 Hub 侧透传。
+//
+// 「在线」只说明桥进程还活着:现场 ~40 台 codex 共存节点里,有的在 Hub 上 idle / in_flight=0,
+// 实际 app-server 端口没人监听、TUI pane 变成 `sleep infinity`、或登录态 "refresh token revoked"。
+// 节点把这几层分开报上来;这里只负责**收下、按节点存最新一份、在 /api/status 和 status_update 里带出去**。
+//
+// 刻意只放内存,不加列:它是**几秒钟一变的观测值**,不是需要跨重启保留的状态 —— Hub 重启后下一次
+// 心跳(≤3 分钟,翻转时立即)就会补上;持久化一份旧观测值反而会在节点死掉后继续说「app_server ok」。
+// 同理,超过 TTL 的那份按「没报」处理(null),不按「健康」处理。
+
+import { z } from "zod/v4";
+
+export const NODE_HEALTH_TTL_MS = 10 * 60_000;
+
+// 🔴 `.catch(undefined)`:这一格是纯诊断信息,**任何形状都不许拒掉整份 report_status**
+//    (zod 对象里一个已知字段验证失败 = 整份被拒 = 节点在 Hub 上失联,#1225 的形状)。
+//    合法的透传,其余一律当没报。子对象不 strict:新节点多报的键丢掉即可。
+export const nodeHealthSchema = z.object({
+  bridge: z.literal("ok").optional(),
+  app_server: z.object({
+    ok: z.boolean(),
+    rtt_ms: z.number().nullable().catch(null),
+    last_error: z.string().max(200).nullable().catch(null),
+  }).optional().catch(undefined),
+  tui: z.object({
+    ok: z.boolean(),
+    reason: z.string().max(40),
+  }).optional().catch(undefined),
+  model_auth: z.enum(["ok", "revoked", "expired", "unknown"]).optional().catch(undefined),
+}).optional().catch(undefined);
+
+export type NodeHealth = NonNullable<z.infer<typeof nodeHealthSchema>>;
+
+const store = new Map<string, { health: NodeHealth; at: number }>();
+const keyOf = (networkId: string | null | undefined, alias: string) => `${networkId ?? "default"}\0${alias}`;
+
+export function recordNodeHealth(networkId: string | null | undefined, alias: string, health: NodeHealth, now = Date.now()): void {
+  store.set(keyOf(networkId, alias), { health, at: now });
+  // 有界:按 TTL 顺手清掉过期的,别让改过名/删掉的节点在内存里攒着。
+  if (store.size > 256) {
+    for (const [k, v] of store) if (now - v.at > NODE_HEALTH_TTL_MS) store.delete(k);
+  }
+}
+
+/** 最新一份;没有或已过 TTL → null(读的人必须把它当「不知道」,而不是「健康」)。 */
+export function readNodeHealth(
+  networkId: string | null | undefined, alias: string, now = Date.now(),
+): { health: NodeHealth; observed_ms_ago: number } | null {
+  const hit = store.get(keyOf(networkId, alias));
+  if (!hit) return null;
+  const age = now - hit.at;
+  if (age > NODE_HEALTH_TTL_MS) { store.delete(keyOf(networkId, alias)); return null; }
+  return { health: hit.health, observed_ms_ago: Math.max(0, age) };
+}
+
+/** 测试用。 */
+export function clearNodeHealthStore(): void { store.clear(); }

package/src/rest-explicit-columns-http.test.ts

@@ -46,6 +46,8 @@ const GOLDEN_RESPONSE_KEYS = {
     "process_in_flight_count", "network_id", "registered_at", "updated_at",
     "node_id", "session_id", "config_path", "channels", "last_seen_at", "model",
     "external_schedules", "rules_file_capable", "skills_capable", "files_capable", "logs_capable", "runtime", "host", "process_telemetry",
+    // #448 — layered node health (memory-only; null when not reported or stale).
+    "health", "health_observed_ms_ago",
   ],
   task: [
     "task_id", "from_node_id", "from_name", "to_node_id", "to_name", "priority",

package/src/server.ts

@@ -1,6 +1,8 @@
 import { buildServeErrorResponse } from "./serve-error.js";
 import { maybeGzipResponse, trimLightTask } from "./http-gzip";
 import { statusAliasResolverRead } from "./status-alias-resolver.js";
+import { readNodeHealth } from "./node-health-store.js";
+import { auditDepartment, createDepartment, deleteDepartment, listDepartments, setMemberDepartment, updateDepartment } from "./departments.js";
 import { redactMessageRow } from "./redact-tokens.js";
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { parseDbTimestampMs } from "./db-timestamp.js";
@@ -1707,6 +1709,60 @@ return Bun.serve({
       return withCors(req, Response.json({ ok: true, network_id: netId, humans: humans.map((h) => ({ ...h, ...presence.get(h.user_id)! })) }));
     }
 
+    // ── 组织架构(board #419):部门 + 成员所在部门。读:网络任何成员 / 本网络的节点令牌 / Hub 管理员;
+    // 写:网络 owner / admin、Hub 管理员(节点令牌不能写)。规则在 departments.ts。 ──
+    const deptMatch = url.pathname.match(/^\/api\/networks\/([^/]+)\/departments(?:\/([^/]+))?$/);
+    const memberDeptMatch = url.pathname.match(/^\/api\/networks\/([^/]+)\/members\/([^/]+)\/department$/);
+    if (deptMatch || memberDeptMatch) {
+      const token = requestToken(req, { allowQueryToken: false });
+      const resolved = token ? resolveToken(token) : null;
+      if (!resolved) return withCors(req, Response.json({ ok: false, error: "auth required" }, { status: 401 }));
+      const netId = decodeURIComponent((deptMatch ?? memberDeptMatch)![1]);
+      if (resolved.networkId && resolved.networkId !== netId) return withCors(req, Response.json({ ok: false, error: "not a member of this network" }, { status: 403 }));
+      const hubAdmin = isHubAdminCredential(resolved);
+      const role = resolved.networkId ? null : getUserNetworkRole(resolved.user.user_id, netId);
+      if (!hubAdmin && !role && !resolved.networkId) return withCors(req, Response.json({ ok: false, error: "not a member of this network" }, { status: 403 }));
+      if (!db.get("SELECT 1 AS x FROM networks WHERE network_id = ?1", netId)) return withCors(req, Response.json({ ok: false, error: "network_not_found" }, { status: 404 }));
+      if (deptMatch && !deptMatch[2] && req.method === "GET") {
+        return withCors(req, Response.json({ ok: true, network_id: netId, ...listDepartments(netId) }));
+      }
+      const canManage = hubAdmin || (!resolved.networkId && (role === "owner" || role === "admin"));
+      if (!["POST", "PATCH", "PUT", "DELETE"].includes(req.method)) return withCors(req, Response.json({ ok: false, error: "method_not_allowed" }, { status: 405 }));
+      if (!canManage) return withCors(req, Response.json({ ok: false, error: "owner/admin required" }, { status: 403 }));
+      let body: Record<string, unknown> = {};
+      if (req.method !== "DELETE") {
+        try { const b = await req.json(); body = b && typeof b === "object" && !Array.isArray(b) ? b as Record<string, unknown> : {}; } catch { return withCors(req, Response.json({ ok: false, error: "invalid JSON" }, { status: 400 })); }
+      }
+      const reply = (r: { ok: boolean; status?: number } & Record<string, unknown>, okStatus = 200) => {
+        const { status, ...rest } = r;
+        return withCors(req, Response.json(rest, { status: r.ok ? okStatus : status ?? 400 }));
+      };
+      if (memberDeptMatch) {
+        if (req.method !== "PUT") return withCors(req, Response.json({ ok: false, error: "method_not_allowed" }, { status: 405 }));
+        const targetUid = decodeURIComponent(memberDeptMatch[2]);
+        const r = setMemberDepartment(netId, targetUid, body.department_id);
+        if (r.ok) auditDepartment(resolved.user, "member_department_set", netId, `${targetUid} → ${r.department_id ?? "(unassigned)"}`);
+        return reply(r);
+      }
+      const deptId = deptMatch![2] ? decodeURIComponent(deptMatch![2]) : null;
+      if (!deptId && req.method === "POST") {
+        const r = createDepartment(netId, resolved.user.user_id, body);
+        if (r.ok) auditDepartment(resolved.user, "department_created", netId, JSON.stringify({ id: r.department.id, name: r.department.name, parent_id: r.department.parent_id }));
+        return reply(r, 201);
+      }
+      if (deptId && req.method === "PATCH") {
+        const r = updateDepartment(netId, deptId, body);
+        if (r.ok) auditDepartment(resolved.user, "department_updated", netId, JSON.stringify({ id: deptId, ...body }));
+        return reply(r);
+      }
+      if (deptId && req.method === "DELETE") {
+        const r = deleteDepartment(netId, deptId);
+        if (r.ok) auditDepartment(resolved.user, "department_deleted", netId, deptId);
+        return reply(r as { ok: boolean; status?: number } & Record<string, unknown>);
+      }
+      return withCors(req, Response.json({ ok: false, error: "method_not_allowed" }, { status: 405 }));
+    }
+
     // ── V3.13: Network members + invites ──
     const membersMatch = url.pathname.match(/^\/api\/networks\/([^/]+)\/members(?:\/([^/]+))?$/);
     if (membersMatch) {
@@ -2297,6 +2353,12 @@ return Bun.serve({
             uptime_seconds: s.process_uptime_seconds ?? null,
             in_flight_count: s.process_in_flight_count ?? null,
           },
+          // #448 — latest layered health the node reported (memory-only, 10 min TTL);
+          // null = not reported / stale / older agent-node — never "healthy".
+          ...(() => {
+            const h = readNodeHealth(s.network_id, s.alias);
+            return { health: h?.health ?? null, health_observed_ms_ago: h?.observed_ms_ago ?? null };
+          })(),
         };
       });
       const summary = sessions.reduce((acc: any, session: any) => {

package/src/tools.ts

@@ -1,6 +1,7 @@
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { parseDbTimestampMs } from "./db-timestamp.js";
 import { z } from "zod/v4";
+import { nodeHealthSchema, recordNodeHealth } from "./node-health-store.js";
 import { parseAliasFilter } from "./alias-filter.js";
 import { createHash } from "node:crypto";
 import { db, uuidv4, logTaskEvent, chainReplyToParent, hashToken, generateId, generateNetworkToken, syncScheduledRunForTask } from "./db.js";
@@ -827,8 +828,12 @@ export function registerTools(server: McpServer, clientIP?: string, enforceNetwo
       files_capable: z.literal(true).optional(),
       // Node run-log view — same doorbell, op logs_tail.
       logs_capable: z.literal(true).optional(),
+      // #448 — layered node health (bridge / app_server / tui / model_auth).
+      // Report-only, memory-only; any malformed shape degrades to "not reported"
+      // instead of rejecting the whole report (see node-health-store.ts).
+      health: nodeHealthSchema,
     },
-    async ({ resume_id, alias, status, task, output, score, progress, server: srv, hostname: hn, agent: ag, project_dir: pd, version: ver, tmux_name: tmux, node_id, session_id, config_path, channels, model: mdl, node_name: nn, network_id: netId, host, process_telemetry: proc, external_schedules: externalSchedules, config_snapshot: cfgSnap, rules_file_capable: rulesFileCapable, skills_capable: skillsCapable, files_capable: filesCapable, logs_capable: logsCapable }) => {
+    async ({ resume_id, alias, status, task, output, score, progress, server: srv, hostname: hn, agent: ag, project_dir: pd, version: ver, tmux_name: tmux, node_id, session_id, config_path, channels, model: mdl, node_name: nn, network_id: netId, host, process_telemetry: proc, external_schedules: externalSchedules, config_snapshot: cfgSnap, rules_file_capable: rulesFileCapable, skills_capable: skillsCapable, files_capable: filesCapable, logs_capable: logsCapable, health: nodeHealth }) => {
       const effectiveNetId = getNetworkId(netId);
       const sessionNetId = effectiveNetId ?? "default";
       if (!callerTokenIsNetwork || !enforceNetworkId) {
@@ -1051,6 +1056,12 @@ export function registerTools(server: McpServer, clientIP?: string, enforceNetwo
           );
         }
       });
+      // #448 — only the node token bound to this alias may speak for its health
+      // (same rule as the *_capable flags above).
+      const acceptedHealth = nodeHealth && callerTokenIsNetwork && callerAlias && callerAlias === effectiveAlias
+        ? nodeHealth
+        : undefined;
+      if (acceptedHealth) recordNodeHealth(sessionNetId, effectiveAlias, acceptedHealth);
       pushEvent(effectiveAlias, {
         type: "status_update",
         alias: effectiveAlias,
@@ -1059,6 +1070,7 @@ export function registerTools(server: McpServer, clientIP?: string, enforceNetwo
         progress: progress ?? null,
         host: statusHostTelemetry,
         process_telemetry: statusProcessTelemetry,
+        ...(acceptedHealth ? { health: acceptedHealth } : {}),
       }, sessionNetId);
 
       // V2: sync tasks table — report_status(working) → tasks.running