@dotdrelle/wiki-manager 0.12.0 → 0.12.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotdrelle/wiki-manager",
-  "version": "0.12.0",
+  "version": "0.12.7",
   "description": "Agentic shell and orchestration cockpit for llm-wiki workspaces.",
   "license": "PolyForm-Noncommercial-1.0.0",
   "author": "dotrelle",
@@ -11,7 +11,7 @@
   },
   "scripts": {
     "start": "bun ./bin/wiki-manager.js",
-    "test": "node --test src/agent/graph.test.js src/contracts/schemas.test.js src/core/activity.test.js src/core/agentEvents.test.js src/core/runtimeLog.test.js src/activity/activityAggregator.test.js src/graph/runGraphProjector.test.js src/core/workflow.test.js src/core/planPatch.test.js src/core/agentLoop.test.js src/core/plan.test.js src/core/mcp.test.js src/core/documentIntake.test.js src/core/dockerCompose.test.js src/core/wikiWorkspace.test.js src/core/wikirc.test.js src/core/modelFetch.test.js src/core/startupCheck.test.js src/core/queueStore.test.js src/orchestrator/agentRegistry.test.js src/orchestrator/capabilityRegistry.test.js src/orchestrator/capabilityResolver.test.js src/orchestrator/planValidator.test.js src/orchestrator/planIntegrator.test.js src/orchestrator/scheduler.test.js src/orchestrator/attemptManager.test.js src/orchestrator/resultAggregator.test.js src/orchestrator/approvalPolicy.test.js src/commands/slash.test.js src/shell/repl.test.js src/runtime/store.test.js src/runtime/recoveryManager.test.js src/runtime/server.test.js src/runtime/supervisor.test.js src/runtime/runner.test.js src/runtime/runner.e2e.test.js src/runtime/donna-contract.test.js src/runtime/auth.test.js",
+    "test": "node --test src/agent/graph.test.js src/contracts/schemas.test.js src/core/activity.test.js src/core/buildInfo.test.js src/core/agentEvents.test.js src/core/runtimeLog.test.js src/activity/activityAggregator.test.js src/graph/runGraphProjector.test.js src/core/workflow.test.js src/core/planPatch.test.js src/core/agentLoop.test.js src/core/plan.test.js src/core/mcp.test.js src/core/documentIntake.test.js src/core/dockerCompose.test.js src/core/wikiWorkspace.test.js src/core/wikirc.test.js src/core/modelFetch.test.js src/core/startupCheck.test.js src/core/queueStore.test.js src/orchestrator/agentRegistry.test.js src/orchestrator/capabilityRegistry.test.js src/orchestrator/capabilityResolver.test.js src/orchestrator/planValidator.test.js src/orchestrator/planIntegrator.test.js src/orchestrator/scheduler.test.js src/orchestrator/attemptManager.test.js src/orchestrator/resultAggregator.test.js src/orchestrator/approvalPolicy.test.js src/commands/slash.test.js src/shell/repl.test.js src/runtime/store.test.js src/runtime/controlMessages.test.js src/runtime/recoveryManager.test.js src/runtime/server.test.js src/runtime/supervisor.test.js src/runtime/runner.test.js src/runtime/runner.e2e.test.js src/runtime/donna-contract.test.js src/runtime/auth.test.js",
     "check-versions": "node scripts/check-versions.js",
     "prepack": "node scripts/check-versions.js",
     "prepublishOnly": "node scripts/check-versions.js",

package/src/activity/activityAggregator.js

@@ -10,9 +10,18 @@ export function aggregateActivity(state = {}, events = []) {
   const activities = Array.isArray(state.activities) ? state.activities : [];
   const progress = calculateWeightedProgress(tasks, activities);
   const groups = groupTasks(tasks);
+  const deduped = deduplicateActivities(activities);
+  // Group lines only cover activities attached to plan tasks. An activity
+  // started OUTSIDE the plan (e.g. the agent calls production_start_job
+  // directly after a minimal one-step plan) must still be visible: without
+  // this, the panel showed a done "production_status" step while the actual
+  // ingest ran invisibly at 15%.
   const lines = groups.length > 0
-    ? groups.map((group) => groupLine(group, activities))
-    : deduplicateActivities(activities).map(activityLine);
+    ? [
+      ...groups.map((group) => groupLine(group, activities)),
+      ...unattachedActivities(tasks, deduped).map(activityLine),
+    ]
+    : deduped.map(activityLine);
   return {
     initialSynthesis: initialSynthesisFromState(state, events),
     progress,
@@ -20,6 +29,21 @@ export function aggregateActivity(state = {}, events = []) {
   };
 }
 
+function unattachedActivities(tasks, activities) {
+  const attachedKeys = new Set(tasks
+    .flatMap((task) => [task.activityKey, task.ownerActivityKey, task._activityKey])
+    .filter(Boolean)
+    .map(String));
+  const taskIds = new Set(tasks.map((task) => String(task.id ?? task.step ?? '')).filter(Boolean));
+  return activities.filter((activity) => {
+    const keys = [activity.key, activity.id].filter(Boolean).map(String);
+    if (keys.some((value) => attachedKeys.has(value))) return false;
+    const stepId = String(activity?.progress?.stepId ?? '');
+    if (stepId && taskIds.has(stepId)) return false;
+    return true;
+  });
+}
+
 function groupTasks(tasks) {
   const groups = new Map();
   for (const task of tasks) {
@@ -70,11 +94,18 @@ function groupLine(group, activities) {
     status = `${done}/${total}`;
   }
   const agents = activeAgents > 0 ? ` - ${activeAgents} agent${activeAgents > 1 ? 's' : ''}` : '';
+  // While a task is running, expose ITS live progress as the group percent.
+  // The completion ratio (done/total tasks) said 0% while the label showed
+  // the real activity progress ("— 1 %"), so the TUI badge contradicted the
+  // text. Fall back to the task-completion ratio when nothing is running.
+  const percent = running.length > 0 && activeProgress != null
+    ? Math.round(activeProgress)
+    : (total > 0 ? Math.round((done / total) * 100) : null);
   return {
     id: `group:${group.id}`,
     label: `${icon} ${group.label} - ${status}${agents}`,
     status,
-    progress: { done, total, percent: total > 0 ? Math.round((done / total) * 100) : null },
+    progress: { done, total, percent },
     activeAgents,
   };
 }

package/src/activity/activityAggregator.test.js

@@ -58,3 +58,35 @@ test('aggregateActivity exposes initial synthesis and grouped display lines', ()
   assert.ok(activity.lines.some((line) => /\[\.\.\.\] customer-data\.enrich - 63 %/.test(line.label)));
   assert.ok(activity.lines.some((line) => /\[ \] publish - en attente/.test(line.label)));
 });
+
+test('aggregateActivity keeps activities not attached to any plan task visible', () => {
+  // Regression: a done one-step minimal plan (production_status) masked the
+  // actually-running ingest activity started outside the plan.
+  const state = {
+    plan: [{
+      id: 'status-check',
+      step: 1,
+      description: 'production.production_status',
+      status: 'done',
+      groupId: '1',
+    }],
+    activities: [
+      {
+        key: 'production:prod_192444',
+        id: 'prod_192444',
+        label: 'Ingest b87acaf6-Comite.md',
+        source: 'production',
+        status: 'running',
+        terminal: false,
+        progress: { percent: 15, detail: 'LLM running' },
+      },
+    ],
+  };
+
+  const aggregated = aggregateActivity(state, []);
+  const labels = aggregated.lines.map((line) => line.label).join('\n');
+  assert.match(labels, /\[x\] .* done/, 'the done plan group stays visible');
+  assert.match(labels, /Ingest b87acaf6/, 'the unattached running ingest must appear');
+  const ingestLine = aggregated.lines.find((line) => /Ingest/.test(line.label));
+  assert.equal(ingestLine.status, 'running');
+});