@dotdrelle/wiki-manager 0.15.71 → 0.15.72

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotdrelle/wiki-manager",
-  "version": "0.15.71",
+  "version": "0.15.72",
   "description": "Agentic shell and orchestration cockpit for llm-wiki workspaces.",
   "repository": {
     "type": "git",

package/src/core/agentEvents.js

@@ -342,9 +342,20 @@ function applyEvent(state, event) {
     case 'task.created':
       appendCreatedTask(state, event.payload?.task);
       return;
-    case 'task.assigned':
+    case 'task.assigned': {
+      // The plan step stays the only place the UIs read WHO runs a task. The
+      // workflow projection (workflow.js taskNode) exposes step.executor, and
+      // serve's run-graph inspector renders it as the "Agent" row — without
+      // this mutation that row stayed empty ("—") while the agent was only
+      // visible in the run journal, under Runtime.
+      const assignedStep = planTaskById(state, event.taskId ?? event.payload?.taskId ?? '');
+      const assignedAgent = event.payload?.assignment?.agentInstanceId ?? event.payload?.agentInstanceId ?? null;
+      if (assignedStep && assignedAgent && !assignedStep.executor) {
+        assignedStep.executor = String(assignedAgent);
+      }
       appendLog(state, taskLogLine(state, event, 'assigned'));
       return;
+    }
     case 'task.started':
       // Silent: always follows `task.assigned` (same task, milliseconds apart),
       // which already printed the "started" line.
@@ -1044,7 +1055,7 @@ function formatPlanErrors(errors) {
     : 'unknown';
 }
 
-const LOG_TIME_PREFIX = /^\d{2}:\d{2}:\d{2}\s+/;
+const LOG_TIME_PREFIX = /^\d{2}:\d{2}:\d{2}(?:\s+|\s*·\s*)/;
 const LOG_REPEAT_SUFFIX = / \(×\d+\)$/;
 
 /*

package/src/core/agentEvents.test.js

@@ -750,3 +750,20 @@ test('appendLog collapses repeated dispatch plumbing but never a repeated busine
   assert.equal(failLines.length, 2);
   assert.equal(failLines.some((l) => / \(×\d+\)$/.test(l)), false);
 });
+
+test('task.assigned records the executor on the plan step for the UIs', () => {
+  const session = { workspace: 'w', agentEvents: [] };
+  dispatchAgentEvent(session, createAgentEvent('plan_set', {
+    origin: 'tool',
+    payload: { steps: [{ id: 't1', description: 'Ingest', status: 'pending' }] },
+  }));
+  dispatchAgentEvent(session, createAgentEvent('task.assigned', {
+    origin: 'assignment_manager',
+    runId: 'r1',
+    taskId: 't1',
+    payload: { taskId: 't1', assignment: { agentInstanceId: 'production-main', capability: 'knowledge.update' } },
+  }));
+  const step = (session.agentProjection.plan ?? []).find((candidate) => candidate.id === 't1' || candidate.step === 't1');
+  assert.ok(step, 'plan step exists');
+  assert.equal(step.executor, 'production-main');
+});

package/src/core/buildInfo.json

@@ -1,4 +1,4 @@
 {
-  "version": "0.15.71",
-  "commit": "d5f51af"
+  "version": "0.15.72",
+  "commit": "fccddba"
 }

package/src/core/mcp.js

@@ -1,7 +1,7 @@
 import { existsSync, readFileSync } from 'node:fs';
 import { managerEnvFile, managerMcpEndpointsFile, readEnvFile } from './env.js';
 
-const WIKI_MANAGER_VERSION = '0.15.71';
+const WIKI_MANAGER_VERSION = '0.15.72';
 
 function envValue(key) {
   const filePath = managerEnvFile();

package/src/core/runtimeLog.js

@@ -32,6 +32,29 @@ const ORDERED_FIELDS = [
   'error',
 ];
 
+// The dispatch events whose payload is routing plumbing, not business content:
+// rendered compactly (see formatRuntimeLogPayload). Business events keep the
+// full field=value form.
+const COMPACT_EVENTS = new Set([
+  'agent_status',
+  'agent_execute',
+  'job.accepted',
+  'task.ready',
+  'task.starting',
+  'task.started',
+  'task.completed',
+  'task.failed',
+  'attempt.created',
+  'lock.acquired',
+  'lock.released',
+  'runtime.execute',
+  'runtime.accepted',
+  'runtime.result_returned',
+  'task.result_returned',
+  'runtime.params_refused',
+  'runtime.blind',
+]);
+
 export function normalizeRuntimeLog(input, { session = null } = {}) {
   if (!input || typeof input !== 'object' || Array.isArray(input)) {
     return { message: String(input ?? '') };
@@ -69,6 +92,28 @@ export function formatRuntimeLogPayload(payload = {}, ts = null) {
   }
   const time = timeLabel(ts);
   const event = eventLabel(payload.event);
+  // Dispatch plumbing is rendered as a readable sentence, not a field dump:
+  // `AGENT_STATUS run=… plan=… group=… attempt=… agentType=… workspace=…`
+  // buried the one thing the reader wants — WHO does WHAT on WHICH task, on
+  // WHICH job. The verbosity is kept for business payloads, where fields are
+  // the content.
+  if (COMPACT_EVENTS.has(String(payload.event ?? ''))) {
+    const parts = [time, event];
+    const who = payload.agentInstanceId ?? payload.agentId ?? payload.agentType;
+    if (who) parts.push(shortenUuids(String(who)));
+    const what = [payload.capability, payload.operation].filter(Boolean).join('/');
+    if (what) parts.push(what);
+    const task = shortTaskLabel(payload.taskId);
+    if (task) parts.push(task);
+    if (payload.jobId) parts.push(shortenUuids(String(payload.jobId)));
+    if (payload.status != null) parts.push(String(payload.status));
+    if (payload.error) parts.push(shortenUuids(String(payload.error)).slice(0, 120));
+    if (payload.detail != null && payload.detail !== ''
+      && String(payload.detail).toUpperCase() !== event) {
+      parts.push(shortenUuids(String(payload.detail)));
+    }
+    return parts.join(' · ');
+  }
   const fields = ORDERED_FIELDS
     .map((key) => (key === 'taskId'
       ? formatField(FIELD_ALIASES[key], shortTaskLabel(payload[key]))
@@ -123,7 +168,7 @@ export function shortLogId(value, { maxLength = 40 } = {}) {
 // token — so this one shape separates the two without an event-name list
 // (which is what an earlier enumeration got wrong: it only ever matched the
 // two underscore-form events and missed every dotted one).
-const DISPATCH_PLUMBING_LINE = /^(?:\d{1,2}:\d{2}(?::\d{2})?\s+)?[A-Z][A-Z0-9_]{2,}(?:\s|$)/;
+const DISPATCH_PLUMBING_LINE = /^(?:\d{1,2}:\d{2}(?::\d{2})?\s*(?:·\s*)?)?[A-Z][A-Z0-9_]{2,}(?:\s|$)/;
 
 export function isDispatchPlumbingLine(line) {
   return DISPATCH_PLUMBING_LINE.test(String(line ?? ''));

package/src/core/runtimeLog.test.js

@@ -39,22 +39,27 @@ function payload(event) {
   };
 }
 
-test('formatRuntimeLogPayload formats every dispatcher cycle event with required ids', () => {
+test('formatRuntimeLogPayload renders dispatch plumbing as a compact sentence, business events keep their fields', () => {
+  // Dispatch plumbing: who · what · task · job — no run/plan/group/attempt
+  // soup. Business events (capability.resolving, agent.selected,
+  // task.assigned) keep the full field=value form: those fields ARE the
+  // content.
   for (const event of CYCLE_EVENTS) {
     const line = formatRuntimeLogPayload(payload(event), '2026-07-08T14:42:18.000Z');
-    assert.match(line, /^14:42:18 [A-Z_]+ /);
-    assert.match(line, /run=run-123/);
-    assert.match(line, /plan=4/);
-    assert.match(line, /group=group-build/);
-    assert.match(line, /task=task-build/);
-    assert.match(line, /attempt=attempt-1/);
-    assert.match(line, /agentType=production/);
-    assert.match(line, /agentInstance=production-main/);
-    assert.match(line, /agent=worker-02/);
-    assert.match(line, /job=job-789/);
-    assert.match(line, /workspace=docs/);
-    assert.match(line, /capability=document\.build/);
-    assert.match(line, /operation=build/);
+    if (['capability.resolving', 'agent.selected', 'task.assigned'].includes(event)) {
+      assert.match(line, /run=run-123/);
+      assert.match(line, /workspace=docs/);
+      assert.match(line, /capability=document\.build/);
+      assert.match(line, /operation=build/);
+      continue;
+    }
+    assert.match(line, /^14:42:18 · [A-Z_]+ · /);
+    assert.match(line, /production-main/);
+    assert.match(line, /document\.build\/build/);
+    assert.match(line, /task-build/);
+    assert.match(line, /job-789/);
+    assert.match(line, /cycle detail/);
+    assert.doesNotMatch(line, /run=|plan=|group=|attempt=|agentType=|agent=|workspace=|capability=|operation=/);
   }
 });
 
@@ -111,12 +116,10 @@ test('long UUIDs collapse to a short prefix so log lines stay on one line', () =
     operation: 'build',
   }, '2026-07-08T14:42:18.000Z');
 
-  assert.match(line, /run=7fadad27…/);
-  // The taskId's UUID prefix and hash suffix are dropped entirely: the field
+  assert.match(line, /production-7fadad27…/);
+  // The taskId's UUID prefix and hash suffix are dropped entirely: the line
   // names the work ("taxonomy synthesis"), not an opaque id.
-  assert.match(line, /task="taxonomy synthesis"/);
-  assert.match(line, /attempt=attempt-7fadad27…/);
-  assert.match(line, /agentInstance=production-7fadad27…/);
+  assert.match(line, /taxonomy synthesis/);
   assert.doesNotMatch(line, /7fadad27-0be6-4d08-96e5-664fe7ee841e/);
 });
 

package/src/runtime/runner.test.js

@@ -786,14 +786,14 @@ test('runRuntimeParallelPlan dispatches orchestrated tasks without child LLM loo
   for (const label of ['READY', 'RESOLVING', 'SELECTED', 'ASSIGNED', 'CREATED', 'ACQUIRED', 'AGENT_EXECUTE', 'ACCEPTED', 'AGENT_STATUS', 'RESULT_RETURNED', 'RELEASED', 'COMPLETED']) {
     assert.match(logs, new RegExp(`\\b${label}\\b`));
   }
-  assert.match(logs, /run=run-dispatch/);
+  // Compact dispatch plumbing: the routing fields ride the line as a sentence
+  // (who · what · task · job), no run=/plan=/attempt= soup.
+  assert.match(logs, /AGENT_EXECUTE · production-main · workspace\.diagnose\/doctor · a\b/);
+  assert.match(logs, /ACCEPTED · production-main · workspace\.diagnose\/doctor · a · job-a · queued/);
   assert.match(logs, /task=a/);
-  assert.match(logs, /attempt=a:attempt-1/);
-  assert.match(logs, /agentInstance=production-main/);
-  assert.match(logs, /job=job-a/);
+  // Business events keep the full field=value form.
+  assert.match(logs, /run=run-dispatch/);
   assert.match(logs, /workspace=demo-workspace/);
-  assert.match(logs, /capability=workspace\.diagnose/);
-  assert.match(logs, /operation=doctor/);
 });
 
 test('runRuntimeParallelPlan fails cleanly when scheduler budget is exceeded', async () => {