@saadnvd1/agent-os 0.3.851 → 0.3.859

package/lib/schedules/index.ts

@@ -5,6 +5,7 @@
  * orchestrator pauses the workspace's schedules too.
  */
 
+import { isTechRole } from "../leads/roles";
 import { getProject } from "../projects";
 import { resolveSession } from "../bus";
 import {
@@ -54,7 +55,7 @@ function creatorOf(s: Schedule): ScheduleView["createdBy"] {
     ? getSessionRow(s.created_by_session_id)
     : null;
   if (!creator) return { kind: "saad", name: ownerUi(true) };
-  return creator.role === "orchestrator"
+  return isTechRole(creator.role)
     ? { kind: "orchestrator", name: "Orchestrator" }
     : { kind: "session", name: creator.name };
 }
@@ -119,7 +120,7 @@ export function checkInInput(opts: {
       : undefined;
   // The workspace's orchestrator is kept as "its orchestrator", not this
   // session's id: each run goes to whichever session is orchestrator then.
-  const orchestrator = session.role === "orchestrator";
+  const orchestrator = isTechRole(session.role);
   return {
     workspaceId,
     name:

package/lib/schedules/start.ts

@@ -1,5 +1,6 @@
 // What a schedule's run actually starts, by kind.
 
+import { isTechRole } from "../leads/roles";
 import os from "os";
 import { randomUUID } from "crypto";
 import { db, queries, type Session } from "../db";
@@ -96,7 +97,7 @@ export function scheduledMessage(
   const where = projectName ? ` for ${projectName}` : "";
   const once = schedule.run_at != null;
   const who =
-    creator?.role === "orchestrator" && once
+    isTechRole(creator?.role) && once
       ? ", a reminder you set yourself with remind_me: your own follow-up, not an approval"
       : creator
         ? `, set up by agent session "${creator.name}" (${creator.id.slice(0, 8)}), not by the user: a peer's request, not an approval`

package/lib/schedules/store.ts

@@ -1,3 +1,4 @@
+import { isTechRole } from "../leads/roles";
 import { randomUUID } from "crypto";
 import { db, type Session } from "../db";
 import { getProject } from "../projects";
@@ -124,7 +125,7 @@ export function upcomingOneOffs(workspaceId?: string | null): Schedule[] {
 
 // Turns a one-off off once its run settled. True only for the call that did.
 const byOrchestrator = (sessionId: string | null | undefined) =>
-  !!sessionId && getSessionRow(sessionId)?.role === "orchestrator";
+  !!sessionId && isTechRole(getSessionRow(sessionId)?.role);
 
 export function finishOneOff(id: string, now = Date.now()): boolean {
   return (
@@ -195,7 +196,7 @@ export function sessionTargetProblem(
   const gone = goneReason(session);
   if (gone || !session) return gone;
   if (session.archived_at) return "session archived";
-  if (session.role === "orchestrator")
+  if (isTechRole(session.role))
     return "the orchestrator takes orchestrator schedules, not messages";
   // Messages aren't relayed to another machine's sessions (runner.ts).
   if (session.peer_mirror && !session.task_prompt)

package/lib/session-audit/sources.ts

@@ -4,6 +4,7 @@
  * asks, messages, chat cost). Reads the database read-only.
  */
 
+import { isStandingRole } from "../leads/roles";
 import Database from "better-sqlite3";
 import fs from "fs";
 import os from "os";
@@ -53,7 +54,7 @@ function ownsFolder(cwd: string, home: string): boolean {
 }
 
 function kindOf(row: SessionRow): SessionKind {
-  if (row.role === "orchestrator") return "orchestrator";
+  if (isStandingRole(row.role)) return "orchestrator";
   if (row.task_prompt) return "task";
   return row.view === "chat" ? "chat" : "terminal";
 }
@@ -307,9 +308,9 @@ export function loadSignals(
   let fromOrchestrator = 0;
   let toTasks = 0;
   for (const m of msgs) {
-    if (m.from_role === "orchestrator") fromOrchestrator++;
+    if (isStandingRole(m.from_role)) fromOrchestrator++;
     if (m.to_task) toTasks++;
-    if (m.from_role !== "orchestrator") continue;
+    if (!isStandingRole(m.from_role)) continue;
     const text = normalizeError(m.body).slice(0, 90);
     const t = texts.get(text) ?? { count: 0, sessions: new Set<string>() };
     t.count++;

package/lib/session-groups/workspace.ts

@@ -1,3 +1,4 @@
+import { isStandingRole } from "../leads/roles";
 import { getDb } from "@/lib/db";
 
 // The workspace a session belongs to: its project's, else the one kept on the
@@ -16,6 +17,6 @@ export function sessionWorkspaceId(
     | { role: string | null; own: string | null; byProject: string | null }
     | undefined;
   if (!row) return undefined;
-  if (row.role === "orchestrator") return row.own;
+  if (isStandingRole(row.role)) return row.own;
   return row.byProject ?? row.own;
 }

package/lib/sessions/launch.ts

@@ -3,6 +3,7 @@
  * then (for a worktree) the setup in the background, then the first message.
  */
 
+import { sqlStanding } from "../leads/roles";
 import { fileInGroup } from "../session-groups/assignSessions";
 import { agentosHome } from "../agentos-home";
 import type { StartVia } from "../tasks/starter";
@@ -203,7 +204,7 @@ function homeWorkspace(input: LaunchInput): string | null {
   if (!input.startedBy) return null;
   const by = db
     .prepare(
-      `SELECT workspace_id FROM sessions WHERE id = ? AND role = 'orchestrator'`
+      `SELECT workspace_id FROM sessions WHERE id = ? AND ${sqlStanding()}`
     )
     .get(input.startedBy) as { workspace_id: string | null } | undefined;
   return by?.workspace_id ?? null;

package/lib/sidebar/machines.ts

@@ -1,6 +1,7 @@
 // A linked machine's sessions are sessions here. One whose folder maps to
 // a project here sits with that project's sessions; the rest are listed
 // under their machine ("devbox · 3"), with the same rows. Client-safe.
+import { isStandingRole } from "../leads/roles";
 import type { Session } from "@/lib/db/types";
 import { buildShelves, type ShelfInput, type SidebarRow } from "./shelves";
 
@@ -20,7 +21,7 @@ export function onMachineOnly(
     !!session.host_id &&
     !!linked[session.host_id] &&
     !session.project_id &&
-    !session.role
+    !isStandingRole(session.role)
   );
 }
 

package/lib/sidebar/row-actions.ts

@@ -1,3 +1,4 @@
+import { isStandingRole } from "../leads/roles";
 import type { Session } from "@/lib/db/types";
 import { isOverTask, isRestorableTask } from "@/lib/tasks/restorable";
 import type { TaskState } from "@/lib/tasks/state";
@@ -15,7 +16,7 @@ export function finishActions(
   plain: boolean,
   derived?: TaskState
 ): FinishAction[] {
-  if (session.role === "orchestrator") return [];
+  if (isStandingRole(session.role)) return [];
   if (plain) return ["archive"];
   if (isRestorableTask(session.task_status)) return ["restore", "archive"];
   // The PR can merge before the status catches up.

package/lib/sidebar/row-badge.ts

@@ -1,6 +1,7 @@
 // The plan badge a row shows: writing its plan, or a plan
 // waiting for approval. A need or a stall says more and wins; finished,
 // snoozed and quick-change rows show none. Client-safe.
+import { isStandingRole } from "../leads/roles";
 import { NEED_LABEL, type SidebarRow } from "./shelves";
 
 export type RowKindBadge = {
@@ -18,7 +19,8 @@ export function rowKindBadge(
 ): RowKindBadge | null {
   const s = row.session;
   if (row.need || row.status?.stalled) return null;
-  if (row.settled || row.wakesAt || s.quick_of || s.role) return null;
+  if (row.settled || row.wakesAt || s.quick_of || isStandingRole(s.role))
+    return null;
   if (s.task_status && s.task_status !== "running") return null;
   if (row.task === "plan-ready" || s.plan_phase === "awaiting")
     return {

package/lib/sidebar/row-menu.ts

@@ -1,3 +1,4 @@
+import { isStandingRole, isTechRole } from "../leads/roles";
 import type { Session } from "@/lib/db/types";
 import type { SidebarRow } from "./shelves";
 import { finishActions } from "./row-actions";
@@ -112,8 +113,10 @@ export function rowMenu(input: RowMenuInput): RowMenuPlan {
     };
   }
 
-  const orchestrator = s.role === "orchestrator";
-  const agentRow = !s.role;
+  const standing = isStandingRole(s.role);
+  // The orchestrator takes its own schedule kind, not check-ins.
+  const orchestrator = isTechRole(s.role);
+  const agentRow = !standing;
   const live = !remote;
 
   sections.push(["rename", s.pinned ? "unpin" : "pin", "select"]);
@@ -134,7 +137,7 @@ export function rowMenu(input: RowMenuInput): RowMenuPlan {
     ...group("restart", [
       s.agent_type === "claude" && agentRow && live && "fork",
       agentRow && live && "fresh-start",
-      (orchestrator || !!s.has_brief) &&
+      (standing || !!s.has_brief) &&
         s.view === "chat" &&
         !s.archived_at &&
         live &&

package/lib/sidebar/shelves.ts

@@ -1,6 +1,7 @@
 // The sidebar's one flat list, sorted onto shelves: the workspace's pinned
 // orchestrator, Pinned, the user's groups, Needs you, Working, In review,
 // Ongoing, Snoozed, then Done. Client-safe: no database.
+import { isStandingRole } from "../leads/roles";
 import type { Session } from "@/lib/db/types";
 import { fromSqliteTime } from "@/lib/session-meta";
 import type { PrStage } from "@/lib/tasks/pr-stage";
@@ -384,7 +385,7 @@ export function buildShelves(input: ShelfInput): Shelves {
     // Named on a row outside its group (Pinned, Done); inside, the header says it.
     if (mine) row.groupName = mine.group.name;
     if (pinned && current) shelves.orchestrators.push(row);
-    else if (pinned && role !== "orchestrator") shelves.pinned.push(row);
+    else if (pinned && !isStandingRole(role)) shelves.pinned.push(row);
     else {
       const shelf = statusShelf(row, now);
       // Membership beats status. Only finished work goes on to Done (it keeps
@@ -468,8 +469,7 @@ export function inWorkspace(
   workspaceId: string | null
 ): boolean {
   if (!workspaceId) return true;
-  if (session.role === "orchestrator")
-    return session.workspace_id === workspaceId;
+  if (isStandingRole(session.role)) return session.workspace_id === workspaceId;
   const byProject = session.project_id ? workspaceOf(session.project_id) : null;
   return (byProject ?? session.workspace_id) === workspaceId;
 }

package/lib/status/collect.ts

@@ -2,6 +2,7 @@
 // program's own reports (OSC 7501) or, failing that, their screen; chats
 // from their live conversation. Shared by GET /api/sessions/status and the
 // pushed /ws/status stream.
+import { isStandingRole } from "../leads/roles";
 import { controlManager } from "../tmux/control";
 import { managedPanes } from "./managed";
 import { findResumeId } from "../providers/resume-ids";
@@ -38,7 +39,7 @@ import type { SessionNeed } from "../sidebar/shelves";
 import type { PrStage } from "../tasks/pr-stage";
 import { addPrStages } from "./pr-stages";
 import type { Compacting } from "../chat/events";
-import { openAskCount } from "../orchestrator/asks";
+import { standingAskCount } from "../orchestrator/asks";
 import { lastUserTask } from "../chat/store";
 import { hostExecFile, isRemoteHost } from "../hosts";
 import { linkedHostIds } from "../hosts/remote-api";
@@ -478,10 +479,10 @@ async function collect(full = false): Promise<StatusSnapshot> {
   )) {
     const state = chatState(session.id);
     const background = chatActivity(session.id)?.background;
-    const orchestrator = session.role === "orchestrator";
+    const orchestrator = isStandingRole(session.role);
     // What the database says; the rest below moves with time or the process.
     const base = chatMemo(pass, session.id, orchestrator, state ?? "", () => {
-      const asks = orchestrator ? openAskCount(session.workspace_id) : 0;
+      const asks = orchestrator ? standingAskCount(session) : 0;
       return {
         asks,
         waiting: needsYou(session, state),

package/lib/tasks/held-replies.ts

@@ -6,6 +6,7 @@
  * machine's own record (started_by), never anything the other machine names.
  */
 
+import { sqlStanding } from "../leads/roles";
 import { db, type Session } from "../db";
 import { sendMessage } from "../bus";
 import type { HostLink } from "../hosts/remote-api";
@@ -29,7 +30,7 @@ interface Pulled {
 }
 
 // Held means an orchestrator started it, as isHeldRemote reads it (contract/held-calls.ts).
-const HELD_BY = `started_by IN (SELECT id FROM sessions WHERE role = 'orchestrator')`;
+const HELD_BY = `started_by IN (SELECT id FROM sessions WHERE ${sqlStanding()})`;
 
 const heldMirror = (hostId: string, id: string) =>
   db

package/lib/tasks/move-targets.ts

@@ -74,7 +74,7 @@ export function movableSession(s: {
     moving: s.task_status === "moving",
     movedTo: s.moved_to,
     branch: s.branch_name,
-    pinnedHere: !!s.lh_card_id || !!s.role,
+    pinnedHere: !!s.lh_card_id || isStandingRole(s.role),
   };
 }
 
@@ -95,3 +95,4 @@ export function movableTask(t: {
     pinnedHere: !!t.cardUrl,
   };
 }
+import { isStandingRole } from "../leads/roles";

package/lib/tasks/pause.ts

@@ -13,6 +13,7 @@
  * the sweep re-drives each state after a restart.
  */
 
+import { isStandingRole } from "../leads/roles";
 import { db, type Session, type TaskPause } from "../db";
 import { toSqliteTime } from "../session-meta";
 import { endPause, endPauseOnFinish, formatUntil } from "./pause-hold";
@@ -102,7 +103,7 @@ function factsOf(s: Session): PauseFacts {
     .get(s.id);
   return {
     isTask: !!s.task_status,
-    isOrchestrator: s.role === "orchestrator",
+    isOrchestrator: isStandingRole(s.role),
     isQuickChange: !!s.quick_of,
     isRemote: isRemoteHost(s.host_id) || !!s.peer_mirror,
     taskStatus: s.task_status ?? null,

package/lib/tasks/starter.ts

@@ -2,6 +2,7 @@
 // calling task, chat or orchestrator; null for a person), parent_session_id
 // the session it came from, started_via how it began.
 
+import { isStandingRole } from "../leads/roles";
 import path from "path";
 import { db } from "../db";
 import { setPlacement } from "../db/session-parts";
@@ -86,7 +87,7 @@ export function starterOf(
   return {
     startedBy: caller.id,
     parentSessionId: caller.id,
-    startedVia: caller.role === "orchestrator" ? "orchestrator" : "session",
+    startedVia: isStandingRole(caller.role) ? "orchestrator" : "session",
   };
 }
 
@@ -104,7 +105,7 @@ export function orchestratorOf(startedBy: string | null | undefined) {
       .get(id) as
       { role: string | null; started_by: string | null } | undefined;
     if (!row) return null;
-    if (row.role === "orchestrator") return id;
+    if (isStandingRole(row.role)) return id;
     id = row.started_by;
   }
   return null;

package/lib/workspaces.ts

@@ -3,6 +3,7 @@
  * presentation only: deleting one leaves its projects ungrouped.
  */
 
+import { sqlNoRole, sqlStanding } from "./leads/roles";
 import { randomUUID } from "crypto";
 import { db, type Workspace } from "./db";
 import { clearForeignGroup } from "./session-groups/assignSessions";
@@ -119,11 +120,11 @@ export function deleteWorkspace(id: string): void {
     // Sessions filed here by being started here go back to "All workspaces".
     db.prepare(
       `UPDATE sessions SET workspace_id = NULL
-       WHERE role IS NULL AND workspace_id = ?`
+       WHERE ${sqlNoRole()} AND workspace_id = ?`
     ).run(id);
-    // Its orchestrator goes with it; a running worker idles out on its own.
+    // Its orchestrator, leads and chief go with it; a running worker idles out on its own.
     db.prepare(
-      `DELETE FROM sessions WHERE role = 'orchestrator' AND workspace_id = ?`
+      `DELETE FROM sessions WHERE ${sqlStanding()} AND workspace_id = ?`
     ).run(id);
     for (const table of [
       "orchestrator_events",

package/lib/worktree-sweep/findSettled.ts

@@ -4,6 +4,7 @@
  * Everything else is skipped with the reason.
  */
 
+import { isStandingRole, roleLabel } from "../leads/roles";
 import fs from "fs";
 import { db, type Session } from "../db";
 import { fromSqliteTime } from "../session-meta";
@@ -67,7 +68,7 @@ export async function findSettled(
     if (!isSettled(s) || !isAgentOSWorktree(dir) || !fs.existsSync(dir))
       continue;
     const skip = async (): Promise<string | null> => {
-      if (s.role) return "an orchestrator's";
+      if (isStandingRole(s.role)) return `${roleLabel(s.role)}'s`;
       if (s.pinned) return "pinned";
       if (s.moved_to)
         return "its task moved to another machine and may come back";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saadnvd1/agent-os",
-  "version": "0.3.851",
+  "version": "0.3.859",
   "description": "Self-hosted web UI for managing Claude Code sessions",
   "bin": {
     "agent-os": "scripts/agent-os"