agent-log-viewer 1.0.1 → 1.0.2

package/dist/mcp-server.mjs

@@ -1762,6 +1762,17 @@ var VIEWER_SPAWN_CAPABILITY_HEADER = "x-llv-spawn-capability";
 import fs6 from "node:fs";
 import path5 from "node:path";
 import crypto3 from "node:crypto";
+function viewerMcpServerEntry(packageCwd = process.cwd()) {
+  const direct = path5.resolve(packageCwd, "bin", "mcp-server.mjs");
+  const fromStandalone = path5.resolve(packageCwd, "..", "..", "bin", "mcp-server.mjs");
+  const launcher = [direct, fromStandalone].find((candidate) => fs6.existsSync(candidate));
+  if (!launcher)
+    throw new Error(`Viewer MCP launcher could not be resolved from package cwd: ${packageCwd}`);
+  return {
+    command: "bun",
+    args: [launcher]
+  };
+}
 function claudeSpawnPolicyPaths(home, profileId) {
   if (!/^[A-Za-z0-9_-]{1,128}$/.test(profileId))
     throw new Error("Claude spawn policy profile id is invalid");
@@ -1826,19 +1837,20 @@ function claudeProjectMcpServers(cwd) {
     return {};
   return record(readSettings(configPath).mcpServers) ?? {};
 }
-function claudeMcpServers(pathname, cwd, allowlist) {
-  if (!fs6.existsSync(pathname))
-    return {};
-  const state = readSettings(pathname);
+function resolveClaudeMcpServers(home, cwd, mcpStatePath = path5.join(home, ".claude.json")) {
+  const state = fs6.existsSync(mcpStatePath) ? readSettings(mcpStatePath) : {};
   const rootServers = record(state.mcpServers) ?? {};
   const projects = record(state.projects);
   const project = cwd && projects ? record(projects[cwd]) : null;
   const sharedProjectServers = claudeProjectMcpServers(cwd);
   const localProjectServers = record(project?.mcpServers) ?? {};
-  const registered = { ...rootServers, ...sharedProjectServers, ...localProjectServers };
+  return { ...rootServers, ...sharedProjectServers, ...localProjectServers };
+}
+function claudeMcpServers(home, cwd, allowlist, mcpStatePath) {
+  const registered = resolveClaudeMcpServers(home, cwd, mcpStatePath);
   const names = grantedMcpServers(allowlist);
   return Object.fromEntries(names.flatMap((name) => {
-    const definition = record(registered[name]);
+    const definition = record(registered[name]) ?? (name === "viewer" ? { type: "stdio", ...viewerMcpServerEntry() } : null);
     return definition ? [[name, definition]] : [];
   }));
 }
@@ -1931,7 +1943,7 @@ exit 2
     });
   }
   const enforcedSettings = options.allowSubagents ? settings : { ...settings, disableAllHooks: false, allowManagedHooksOnly: false };
-  const mcpServers = claudeMcpServers(options.mcpStatePath ?? path5.join(home, ".claude.json"), options.cwd, options.mcpServers);
+  const mcpServers = claudeMcpServers(home, options.cwd, options.mcpServers, options.mcpStatePath);
   const includedMcpServers = new Set(Object.keys(mcpServers));
   const excludedProjectMcpServers = Object.keys(claudeProjectMcpServers(options.cwd)).filter((name) => !includedMcpServers.has(name));
   const settingsWithoutMcp = Object.fromEntries(Object.entries(enforcedSettings).filter(([key]) => key !== "mcpServers"));
@@ -9171,12 +9183,15 @@ function headlessCodexThreadConfig(configRead, allowSubagents = false, mcpServer
   if (!config || !servers)
     throw new Error("config/read returned no MCP server table");
   const enabled = new Set(grantedMcpServers(mcpServers));
+  const viewerMissing = record2(servers.viewer) === null;
+  const materializedServers = viewerMissing && enabled.has("viewer") ? { ...servers, viewer: viewerMcpServerEntry() } : servers;
   const granted = grantedPlugins(plugins);
   return {
-    mcp_servers: Object.fromEntries(Object.entries(servers).map(([name, server]) => {
+    mcp_servers: Object.fromEntries(Object.entries(materializedServers).map(([name, server]) => {
       const configuredApproval = record2(server)?.default_tools_approval_mode;
       const approval = name === "viewer" ? "approve" : typeof configuredApproval === "string" && MCP_APPROVAL_MODES.has(configuredApproval) ? configuredApproval : null;
       return [name, {
+        ...name === "viewer" && viewerMissing ? server : {},
         enabled: enabled.has(name),
         ...approval ? { default_tools_approval_mode: approval } : {}
       }];
@@ -15841,6 +15856,9 @@ var init_runtimeImageAdmission = __esm(() => {
 });
 
 // src/lib/runtime/flags.ts
+function supervisedRuntimeHostUnavailableReason(subject) {
+  return `${subject} is unavailable; start agent-log-viewer through its CLI and check the CLI log for the host startup failure`;
+}
 function rolledBack(value) {
   if (value === undefined)
     return false;
@@ -24545,6 +24563,9 @@ async function withRuntimeAdmissionRetry(request, options = {}) {
   throw lastError;
 }
 function structuredSpawnFailureReason(error) {
+  if (isRuntimeHostTransportFailure(error)) {
+    return supervisedRuntimeHostUnavailableReason("structured spawn runtime host");
+  }
   const message = error instanceof Error ? error.message : "structured spawn failed";
   return hardenedRedact(message).replace(/\s+/g, " ").trim().slice(0, 240) || "structured spawn failed";
 }
@@ -26075,13 +26096,14 @@ var init_controllerSignal2 = __esm(() => {
 });
 
 // src/lib/roles/defaults.ts
-var REVIEW_FENCES, REVIEW_FRAME_RULES = `Two standing rules. (1) Anchor the frame: when the assignment carries the requester's originating requirement, validate the work against that verbatim requirement, never against the artifact's previous revision — WRONG-PREMISE ("this does not serve the original requirement") is an expected verdict and outranks any finding about internal rigour. (2) Over-engineering pass: on every review, flag machinery heavier than the problem it solves (a library plus wrapper where a native primitive does), name the simpler mechanism, and report what to cut — OVER-BUILT is a first-class verdict, and a round that only removes scope is a successful round.`, ROLE_DEFAULTS;
+var REVIEW_FENCES, REVIEW_FRAME_RULES = `Two standing rules. (1) Anchor the frame: when the assignment carries the requester's originating requirement, validate the work against that verbatim requirement, never against the artifact's previous revision — WRONG-PREMISE ("this does not serve the original requirement") is an expected verdict and outranks any finding about internal rigour. (2) Over-engineering pass: on every review, flag machinery heavier than the problem it solves (a library plus wrapper where a native primitive does), name the simpler mechanism, and report what to cut — OVER-BUILT is a first-class verdict, and a round that only removes scope is a successful round.`, VIEWER_BASE_URL, ROLE_DEFAULTS;
 var init_defaults = __esm(() => {
   init_models();
   REVIEW_FENCES = [
     "Read-only mode: edits, staging, commits, pushes, service restarts, and GitHub comments are prohibited.",
     "Every finding carries file:line evidence. Clean work earns a clear NO FINDINGS verdict."
   ];
+  VIEWER_BASE_URL = `http://127.0.0.1:${process.env.PORT?.trim() || "8898"}`;
   ROLE_DEFAULTS = [
     {
       id: "orchestrator",
@@ -26097,7 +26119,7 @@ var init_defaults = __esm(() => {
         { key: "mergePolicy", label: "Merge policy", description: "Delivery policy for backlog-campaign mode.", kind: "select", options: ["pr", "merge"] },
         { key: "completionPolicy", label: "Completion policy", description: "Terminal policy for backlog-campaign mode.", kind: "select", options: ["pr-opened", "merged", "released"] }
       ],
-      promptScaffold: `You are the Orchestrator. Drive work through the production Viewer API at http://127.0.0.1:8898. Use fresh empty sessions with src lineage; forks are disabled. Keep every worker visible and controllable in the Viewer.
+      promptScaffold: `You are the Orchestrator. Drive work through the production Viewer API at ${VIEWER_BASE_URL}. Use fresh empty sessions with src lineage; forks are disabled. Keep every worker visible and controllable in the Viewer.
 
 Mode: {{mode}}
 Repository: {{repo}}
@@ -26109,7 +26131,7 @@ Completion policy: {{completionPolicy}}
 
 For backlog-campaign mode, inventory dependencies before assignment, use Opus/Sol gates, route backend work to Terra and frontend work to Opus, complete one review round, and require root release checks. Before a Viewer replacement, preserve the external-worker deployment barrier.`,
       safetyFences: [
-        "Viewer control uses http://127.0.0.1:8898 with src lineage.",
+        `Viewer control uses ${VIEWER_BASE_URL} with src lineage.`,
         "Fresh empty sessions only; forks are disabled.",
         "One owner holds a file at a time across active worktrees."
       ],
@@ -26384,7 +26406,7 @@ function validateRoleParams(definition, raw, { requireRequired = true } = {}) {
   return { ok: true, value: values };
 }
 function renderScaffold(template, params) {
-  return template.replace(/\{\{([A-Za-z][A-Za-z0-9]*)\}\}/g, (_match, key2) => String(params[key2] ?? ""));
+  return template.replace(/\{\{([A-Za-z][A-Za-z0-9]*)\}\}/g, (_match, key2) => String(params[key2] ?? "")).replace(/^(Repository|Issue query|Urgent list):\s*\n/gm, "");
 }
 function roleScaffoldBody(definition, params) {
   const frontendGuidance = definition.id === "builder" && params.domain === "frontend" ? `
@@ -36507,17 +36529,25 @@ async function spawnPipelineAgent(input, onReserved) {
   }
   const spec = { ...specBase, launchProfile };
   const client = runtimeHostClient();
+  const unavailable = supervisedRuntimeHostUnavailableReason("pipeline structured runtime host");
   if (!client)
-    throw new Error("pipeline structured runtime host is unavailable");
-  const response = await spawnStructuredConversation({
-    engine: input.role.engine,
-    receipt: begun.receipt,
-    spec,
-    account: account2,
-    prompt: input.prompt,
-    registry,
-    client
-  });
+    throw new Error(unavailable);
+  let response;
+  try {
+    response = await spawnStructuredConversation({
+      engine: input.role.engine,
+      receipt: begun.receipt,
+      spec,
+      account: account2,
+      prompt: input.prompt,
+      registry,
+      client
+    });
+  } catch (error) {
+    if (isRuntimeHostTransportFailure(error))
+      throw new Error(unavailable, { cause: error });
+    throw error;
+  }
   const transcript = response.path ?? null;
   const key2 = transcript ? sessionKeyFromTranscript(input.role.engine, transcript) : null;
   if (transcript && input.parentPath && parent.conversationId) {
@@ -42479,8 +42509,11 @@ var init_seats = __esm(() => {
 });
 
 // src/lib/orchestrator/prompt.ts
-var ORCHESTRATOR_PROMPT_VERSION = 8, ORCHESTRATOR_INITIAL_STATUS_DIRECTIVE = `## Initial visible status
-Your first turn after receiving this mandate must produce a visible assistant status in this conversation. When work remains, inventory the mandate missions and state your plan in that status. When every mission is already complete, reply exactly: "all mandate missions are complete; standing by". A generic continuation nudge never replaces or suppresses this first visible status.`, ORCHESTRATOR_SYSTEM_PROMPT;
+var ORCHESTRATOR_PROMPT_VERSION = 9, ORCHESTRATOR_INITIAL_STATUS_DIRECTIVE = `## Initial visible status
+Your first turn after receiving this mandate must produce a visible assistant status in this conversation. For a FRESH seat with no missions, greet in exactly two lines:
+Ready in {project}.
+Tell me what to ship — I open lanes, spawn implementers and reviewers, and merge on APPROVE. Nothing starts until you ask.
+Use the actual project name in place of {project}. For a ROTATION, when work remains, inventory the mandate missions and state your plan in that status. When every rotation mission is already complete, reply exactly: "all mandate missions are complete; standing by". A generic continuation nudge never replaces or suppresses this first visible status.`, ORCHESTRATOR_SYSTEM_PROMPT;
 var init_prompt = __esm(() => {
   ORCHESTRATOR_SYSTEM_PROMPT = `You are the viewer's built-in Manager (issues #182, #691) — the agent that owns the board and runs the whole conveyor through the viewer's own HTTP API and MCP tools. You never act outside them.
 
@@ -42536,7 +42569,7 @@ YOU decide when to deploy, and you execute it yourself. Your authority is your d
 
 ## Fences
 - Operate exclusively through the viewer API and MCP tools (spawn, flows, pipelines, tasks, files, agent/snapshot, tmux). No direct process or runtime manipulation.
-- The llv-conveyor skill in this checkout is your playbook; follow its spawn-auth notes for agent-initiated calls.
+- If this checkout carries an llv-conveyor skill, it is your playbook; otherwise the conveyor rules above are the playbook.
 - Replacing manual spawns is a non-goal: the user's own agents keep working; you coordinate, you do not take over.
 - Re-derive board state per turn from bounded snapshots rather than accumulating it in context.`;
 });