@pellux/goodvibes-agent 0.1.7 → 0.1.8

package/src/input/commands/tasks-runtime.ts

@@ -1,8 +1,28 @@
 import type { CommandRegistry } from '../command-registry.ts';
 import type { RuntimeTask, TaskLifecycleState } from '@/runtime/index.ts';
 import { reviewWorktreeAttachments } from '@/runtime/index.ts';
-import { requireOperatorClient, requireOpsApi, requirePanelManager, requireShellPaths } from './runtime-services.ts';
-import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
+import { requireOperatorClient, requirePanelManager, requireShellPaths } from './runtime-services.ts';
+
+const BLOCKED_TASK_MUTATIONS: ReadonlySet<string> = new Set([
+  'create',
+  'update',
+  'complete',
+  'fail',
+  'cancel',
+  'pause',
+  'resume',
+  'retry',
+]);
+
+function printTaskMutationBlocked(print: (text: string) => void, subcommand: string): void {
+  print([
+    `Task mutation "${subcommand}" is blocked in GoodVibes Agent.`,
+    '  policy: runtime tasks are read-only from the Agent surface; normal work stays in the main conversation.',
+    '  durable tasks: use /workplan for visible planning and task tracking.',
+    '  build/fix/review: use /delegate <task> to hand explicit implementation work to GoodVibes TUI.',
+    '  result: no local runtime task state was changed.',
+  ].join('\n'));
+}
 
 function sortRuntimeTasks(tasks: RuntimeTask[]): RuntimeTask[] {
   const statusOrder: TaskLifecycleState[] = ['running', 'queued', 'blocked', 'failed', 'completed', 'cancelled'];
@@ -32,8 +52,8 @@ export function registerTasksRuntimeCommands(registry: CommandRegistry): void {
   registry.register({
     name: 'tasks',
     aliases: ['task'],
-    description: 'Inspect and control runtime tasks',
-    usage: '[list [status|kind] | show <taskId> | output <taskId> | create <kind> <owner> <title...> | update <taskId> <title|description|result> <value...> | complete <taskId> [result] | fail <taskId> <error...> | cancel <taskId> [note] | pause <taskId> [note] | resume <taskId> [note] | retry <taskId> [note]]',
+    description: 'Inspect runtime tasks without starting or mutating local background work',
+    usage: '[list [status|kind] | show <taskId> | output <taskId>]',
     handler(args, ctx) {
       if (args.length === 0) {
         if (ctx.showPanel) ctx.showPanel('tasks');
@@ -96,7 +116,7 @@ export function registerTasksRuntimeCommands(registry: CommandRegistry): void {
             return worktrees.total > 0
               ? [
                   `  worktrees: ${worktrees.total} tracked (${worktrees.active} active / ${worktrees.paused} paused / ${worktrees.pendingCleanup} cleanup)`,
-                  `  worktree next: /worktree task ${task.id}`,
+                  '  worktree next: open GoodVibes TUI in the target workspace for recovery.',
                 ]
               : [];
           })(),
@@ -125,106 +145,12 @@ export function registerTasksRuntimeCommands(registry: CommandRegistry): void {
         return;
       }
 
-      if (subcommand === 'create') {
-        const opsApi = requireOpsApi(ctx);
-        const kind = args[1];
-        const owner = args[2];
-        const title = args.slice(3).join(' ').trim();
-        if (!kind || !owner || !title) {
-          ctx.print('Usage: /tasks create <kind> <owner> <title...>');
-          return;
-        }
-        const validKinds = new Set(['exec', 'agent', 'acp', 'scheduler', 'daemon', 'mcp', 'plugin', 'integration']);
-        if (!validKinds.has(kind)) {
-          ctx.print(`Unknown task kind: ${kind}`);
-          return;
-        }
-        const task = opsApi.tasks.create({
-          kind: kind as import('@/runtime/index.ts').TaskKind,
-          owner,
-          title,
-          description: title,
-        });
-        ctx.print(`Created task ${task.id} (${task.kind}) for ${task.owner}.`);
-        return;
-      }
-
-      if (subcommand === 'update') {
-        const opsApi = requireOpsApi(ctx);
-        const taskId = args[1];
-        const field = args[2];
-        const value = args.slice(3).join(' ').trim();
-        if (!taskId || !field || !value) {
-          ctx.print('Usage: /tasks update <taskId> <title|description|result> <value...>');
-          return;
-        }
-        if (field !== 'title' && field !== 'description' && field !== 'result') {
-          ctx.print(`Unsupported task update field: ${field}`);
-          return;
-        }
-        opsApi.tasks.update(taskId, field === 'result' ? { result: value } : { [field]: value });
-        ctx.print(`Updated task ${taskId} field ${field}.`);
-        return;
-      }
-
-      if (subcommand === 'complete') {
-        const opsApi = requireOpsApi(ctx);
-        const taskId = args[1];
-        if (!taskId) {
-          ctx.print('Usage: /tasks complete <taskId> [result]');
-          return;
-        }
-        const result = args.slice(2).join(' ').trim() || undefined;
-        opsApi.tasks.complete(taskId, result);
-        ctx.print(`Completed task ${taskId}.`);
-        return;
-      }
-
-      if (subcommand === 'fail') {
-        const opsApi = requireOpsApi(ctx);
-        const taskId = args[1];
-        const errorText = args.slice(2).join(' ').trim();
-        if (!taskId || !errorText) {
-          ctx.print('Usage: /tasks fail <taskId> <error...>');
-          return;
-        }
-        opsApi.tasks.fail(taskId, { error: errorText });
-        ctx.print(`Failed task ${taskId}.`);
+      if (BLOCKED_TASK_MUTATIONS.has(subcommand)) {
+        printTaskMutationBlocked(ctx.print, subcommand);
         return;
       }
 
-      const taskId = args[1];
-      const note = args.slice(2).join(' ').trim() || undefined;
-      if (!taskId) {
-        ctx.print(`Usage: /tasks ${subcommand} <taskId> [note]`);
-        return;
-      }
-      const opsApi = requireOpsApi(ctx);
-      try {
-        switch (subcommand) {
-          case 'cancel':
-            opsApi.tasks.cancel(taskId, note);
-            ctx.print(`Cancelled task ${taskId}.`);
-            return;
-          case 'pause':
-            opsApi.tasks.pause(taskId, note);
-            ctx.print(`Paused task ${taskId}.`);
-            return;
-          case 'resume':
-            opsApi.tasks.resume(taskId, note);
-            ctx.print(`Resumed task ${taskId}.`);
-            return;
-          case 'retry':
-            opsApi.tasks.retry(taskId, note);
-            ctx.print(`Re-queued task ${taskId}.`);
-            return;
-          default:
-            ctx.print(`Unknown tasks subcommand: ${subcommand}`);
-            return;
-        }
-      } catch (error) {
-        ctx.print(summarizeError(error));
-      }
+      ctx.print(`Unknown tasks subcommand: ${subcommand}`);
     },
   });
 }

package/src/input/commands.ts

@@ -57,6 +57,7 @@ import { registerAgentExternalizedTuiCommands } from './commands/agent-externali
 import { registerDelegationRuntimeCommands } from './commands/delegation-runtime.ts';
 import { registerPersonasRuntimeCommands } from './commands/personas-runtime.ts';
 import { registerAgentSkillsRuntimeCommands } from './commands/agent-skills-runtime.ts';
+import { registerRoutinesRuntimeCommands } from './commands/routines-runtime.ts';
 
 /**
  * registerBuiltinCommands - Register all built-in slash commands into the registry.
@@ -67,6 +68,7 @@ export function registerBuiltinCommands(registry: CommandRegistry): void {
   registerAgentWorkspaceRuntimeCommands(registry);
   registerPersonasRuntimeCommands(registry);
   registerAgentSkillsRuntimeCommands(registry);
+  registerRoutinesRuntimeCommands(registry);
   registerDelegationRuntimeCommands(registry);
   registerConfigCommand(registry);
   registerOperatorRuntimeCommands(registry);

package/src/input/handler-picker-routes.ts

@@ -256,7 +256,7 @@ type LiveTailRouteState = {
     active: boolean;
     scrollUp: () => void;
     scrollDown: () => void;
-    killProcess: () => void;
+    killProcess: () => boolean;
     close: () => void;
   };
   processModal: {
@@ -270,8 +270,7 @@ export function handleLiveTailToken(state: LiveTailRouteState, token: InputToken
   if (!state.liveTailModal.active) return false;
 
   const killAndReturn = (): void => {
-    state.liveTailModal.killProcess();
-    state.handleEscape();
+    if (state.liveTailModal.killProcess()) state.handleEscape();
   };
 
   if (token.type === 'key') {

package/src/panels/provider-health-domains.ts

@@ -200,15 +200,15 @@ export function buildProviderHealthDomainSummaries(
     summary: worktreeSummary.total === 0
       ? 'no persisted worktrees'
       : `${worktreeSummary.total} tracked / ${worktreeIssues} need review`,
-    next: worktreeIssues > 0 ? '/worktree recover <session|task> <id>' : '/worktree review',
+    next: 'externalized to GoodVibes TUI',
     details: [
       worktreeSummary.paused > 0 ? `${worktreeSummary.paused} paused worktree(s)` : '',
       worktreeSummary.pendingCleanup > 0 ? `${worktreeSummary.pendingCleanup} cleanup pending` : '',
       worktreeSummary.discard > 0 ? `${worktreeSummary.discard} marked discard` : '',
     ].filter(Boolean),
     nextSteps: worktreeIssues > 0
-      ? ['/worktree review', '/worktree recover <session|task> <id>']
-      : ['/worktree review'],
+      ? ['Open GoodVibes TUI in the target workspace for recovery', '/delegate <task> for explicit build/fix/review recovery']
+      : ['No Agent worktree action available; use GoodVibes TUI when repository recovery is needed'],
   });
 
   return summaries;

package/src/renderer/agent-workspace.ts

@@ -97,9 +97,10 @@ function snapshotLines(category: AgentWorkspaceCategory, snapshot: AgentWorkspac
   } else if (category.id === 'memory') {
     base.push(
       { text: `Session memories: ${snapshot.sessionMemoryCount}`, fg: PALETTE.info },
+      { text: `Local routines: ${snapshot.localRoutineCount}; enabled: ${snapshot.enabledRoutineCount}`, fg: PALETTE.info },
       { text: `Local skills: ${snapshot.localSkillCount}; enabled: ${snapshot.enabledSkillCount}`, fg: PALETTE.info },
       { text: `Local personas: ${snapshot.localPersonaCount}; active: ${snapshot.activePersonaName}`, fg: PALETTE.info },
-      { text: 'Durable memory, skills, and personas remain Agent-local until shared registry contracts exist.', fg: PALETTE.good },
+      { text: 'Durable memory, routines, skills, and personas remain Agent-local until shared registry contracts exist.', fg: PALETTE.good },
       { text: 'Secrets are rejected/redacted; store secret references instead of secret values.', fg: PALETTE.warn },
     );
   } else if (category.id === 'work') {

package/src/renderer/live-tail-modal.ts

@@ -6,7 +6,7 @@ import type { ProcessEntry } from './process-modal.ts';
 import { getOverlaySurfaceMetrics, getStableOverlayContentRows } from './overlay-viewport.ts';
 
 export interface LiveTailModalDeps {
-  readonly agentManager: Pick<AgentManager, 'cancel' | 'getStatus'>;
+  readonly agentManager: Pick<AgentManager, 'getStatus'>;
   readonly processManager: Pick<ProcessManager, 'stop' | 'getOutput'>;
 }
 
@@ -15,7 +15,7 @@ export interface LiveTailModalDeps {
 /**
  * LiveTailModal — manages state for the live output peek modal.
  *
- * Shows streaming stdout/stderr from a selected background process or agent
+ * Shows streaming stdout/stderr from a selected shell process or agent
  * progress notes. Auto-scrolls to the bottom unless the user scrolled up.
  */
 export class LiveTailModal {
@@ -48,17 +48,17 @@ export class LiveTailModal {
   }
 
   /**
-   * Kill the current process.
-   * Returns true if the process was found and stopped.
+   * Stop the current shell process when it is an exec entry.
+   * Agent entries are read-only in GoodVibes Agent; build execution and
+   * cancellation belong to GoodVibes TUI/shared-session owners.
    */
   killProcess(): boolean {
     if (!this.entry) return false;
 
     if (this.entry.type === 'exec') {
       return this.deps.processManager.stop(this.entry.id);
-    } else {
-      return this.deps.agentManager.cancel(this.entry.id);
     }
+    return false;
   }
 
   /** Retrieve the current output text for the watched process. */
@@ -151,6 +151,6 @@ export function renderLiveTailModal(
     margin: 2,
     targetContentRows,
     sections,
-    hints: ['[Up/Down] Scroll', '[k] Kill', '[Esc] Back'],
+    hints: ['[Up/Down] Scroll', '[k] Stop exec only', '[Esc] Back'],
   }, width);
 }

package/src/renderer/process-indicator.ts

@@ -17,12 +17,12 @@ function truncateToWidth(text: string, maxWidth: number): string {
 }
 
 /**
- * renderProcessIndicator — shows a one-line summary of active background
- * processes below the input area.
+ * renderProcessIndicator — shows a one-line summary of active runtime
+ * activity below the input area.
  *
- * Dimmed when no processes are active, highlighted (cyan) when agents or
- * background exec processes are running. Includes an `Enter to view` hint
- * when active.
+ * Dimmed when no entries are active, highlighted (cyan) when delegated agent
+ * records or shell exec processes are running. Includes an `Enter to view`
+ * hint when active.
  */
 export function renderProcessIndicator(
   width: number,
@@ -62,16 +62,16 @@ export function renderProcessIndicator(
     if (agentCount > 0) parts.push(`${agentCount} agent${agentCount !== 1 ? 's' : ''}`);
     if (toolCount > 0) parts.push(`${toolCount} tool${toolCount !== 1 ? 's' : ''} running`);
     const label = total === 0
-      ? `No background processes  ${GLYPHS.status.pending}  back to input`
+      ? `No runtime activity  ${GLYPHS.status.pending}  back to input`
       : `${parts.join(` ${GLYPHS.navigation.pipeSeparator} `)}  ${GLYPHS.status.pending}  Enter to open  ${GLYPHS.status.pending}  back to input`;
     return renderFocusedStatus(label);
   }
 
   if (total === 0) {
-    return renderPlainStatus('No background processes', { fg: '238', dim: true });
+    return renderPlainStatus('No runtime activity', { fg: '238', dim: true });
   }
 
-  // Build the label: "bg: 2 agents | Turn 3 | write - src/foo.ts"
+  // Build the label: "2 agents | Turn 3 | write - src/foo.ts"
   const parts: string[] = [];
   if (agentCount > 0) {
     parts.push(`${agentCount} agent${agentCount !== 1 ? 's' : ''}`);

package/src/renderer/process-modal.ts

@@ -42,7 +42,7 @@ const WRFC_ROLE_ORDER: Record<string, number> = {
 };
 
 export interface ProcessModalDeps {
-  readonly agentManager: Pick<AgentManager, 'list' | 'getStatus' | 'cancel'>;
+  readonly agentManager: Pick<AgentManager, 'list' | 'getStatus'>;
   readonly processManager: Pick<ProcessManager, 'list' | 'getStatus' | 'stop'>;
   readonly wrfcController: Pick<WrfcController, 'getChain'> & Partial<Pick<WrfcController, 'listChains'>>;
 }
@@ -558,8 +558,9 @@ export class ProcessModal {
   }
 
   /**
-   * Kill the selected process.
-   * Returns true if a process was killed, false otherwise.
+   * Stop the selected shell process.
+   * Agent entries are read-only in GoodVibes Agent; build execution and
+   * cancellation belong to GoodVibes TUI/shared-session owners.
    */
   killSelected(): boolean {
     const entry = this.getSelected();
@@ -567,9 +568,8 @@ export class ProcessModal {
 
     if (entry.type === 'exec') {
       return this.deps.processManager.stop(entry.id);
-    } else {
-      return this.deps.agentManager.cancel(entry.id);
     }
+    return false;
   }
 }
 
@@ -598,12 +598,12 @@ export function renderProcessModal(modal: ProcessModal, width: number, viewportH
 
   if (modal.entries.length === 0) {
     return ModalFactory.createModal({
-      title: 'Background Processes',
+      title: 'Runtime Activity',
       width: boxW,
       margin: boxMargin,
       targetContentRows,
       sections: [
-        { type: 'text', content: 'No background processes running.' },
+        { type: 'text', content: 'No runtime activity.' },
       ],
       hints: ['[Esc] Close'],
     }, width);
@@ -643,7 +643,7 @@ export function renderProcessModal(modal: ProcessModal, width: number, viewportH
   }
 
   return ModalFactory.createModal({
-    title: 'Background Processes',
+    title: 'Runtime Activity',
     width: boxW,
     margin: boxMargin,
     targetContentRows,
@@ -651,6 +651,6 @@ export function renderProcessModal(modal: ProcessModal, width: number, viewportH
     helpers: modal.entries.length > maxVisibleRows
       ? [{ content: `[${window.start + 1}-${window.end} of ${modal.entries.length}]` }]
       : undefined,
-    hints: ['[Up/Down] Navigate', '[Enter] Peek output', '[k] Kill', '[Esc] Close'],
+    hints: ['[Up/Down] Navigate', '[Enter] Details/output', '[k] Stop exec only', '[Esc] Close'],
   }, width);
 }

package/src/runtime/bootstrap.ts

@@ -44,6 +44,7 @@ import { startMcpConfigAutoReload } from '../mcp/runtime-reload.ts';
 import { GOODVIBES_AGENT_SURFACE_ROOT } from '../config/surface.ts';
 import { buildActivePersonaPrompt } from '../agent/persona-registry.ts';
 import { buildEnabledSkillsPrompt } from '../agent/skill-registry.ts';
+import { buildEnabledRoutinesPrompt } from '../agent/routine-registry.ts';
 
 const GOODVIBES_AGENT_OPERATOR_POLICY = [
   '## GoodVibes Agent Operator Policy',
@@ -210,6 +211,7 @@ export async function bootstrapRuntime(
       return joinPromptParts(
         runtime.systemPrompt,
         GOODVIBES_AGENT_OPERATOR_POLICY,
+        buildEnabledRoutinesPrompt(services.shellPaths),
         buildEnabledSkillsPrompt(services.shellPaths),
         buildActivePersonaPrompt(services.shellPaths),
         supplement,

package/src/runtime/services.ts

@@ -88,7 +88,6 @@ import {
 } from '@pellux/goodvibes-sdk/platform/tools';
 import { WorkPlanStore } from '../work-plans/work-plan-store.ts';
 
-const REGULAR_KNOWLEDGE_DB_FILE = 'knowledge-wiki.sqlite';
 const HOME_GRAPH_KNOWLEDGE_DB_FILE = 'knowledge-home-graph.sqlite';
 
 function buildFallbackModelDefinition(provider: string, modelId: string): ModelDefinition {
@@ -170,6 +169,7 @@ export interface RuntimeServices {
   readonly automationManager: AutomationManager;
   readonly gatewayMethods: GatewayMethodCatalog;
   readonly artifactStore: ArtifactStore;
+  /** Compatibility alias that intentionally points at the isolated Agent Knowledge service, not default Knowledge/Wiki. */
   readonly knowledgeService: KnowledgeService;
   readonly agentKnowledgeService: KnowledgeService;
   readonly homeGraphService: HomeGraphService;
@@ -395,10 +395,6 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
       ].join(' '));
     },
   });
-  const knowledgeStore = new KnowledgeStore({
-    configManager,
-    dbFileName: REGULAR_KNOWLEDGE_DB_FILE,
-  });
   const agentKnowledgeStore = new KnowledgeStore({
     configManager,
     dbFileName: GOODVIBES_AGENT_KNOWLEDGE_DB_FILE,
@@ -411,10 +407,6 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     timeoutMs: 20_000,
     maxConcurrent: 1,
   });
-  const knowledgeSemanticService = new KnowledgeSemanticService(knowledgeStore, {
-    llm: knowledgeSemanticLlm,
-    maxLlmSourcesPerReindex: 3,
-  });
   const homeGraphSemanticService = new KnowledgeSemanticService(homeGraphKnowledgeStore, {
     llm: knowledgeSemanticLlm,
     maxLlmSourcesPerReindex: 3,
@@ -424,12 +416,6 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     llm: knowledgeSemanticLlm,
     maxLlmSourcesPerReindex: 3,
   });
-  const knowledgeService = new KnowledgeService(knowledgeStore, artifactStore, undefined, {
-    memoryRegistry,
-    runtimeBus: options.runtimeBus,
-    semanticService: knowledgeSemanticService,
-  });
-  knowledgeService.attachRuntimeBus(options.runtimeBus);
   const agentKnowledgeService = new KnowledgeService(agentKnowledgeStore, artifactStore, undefined, {
     memoryRegistry,
     runtimeBus: options.runtimeBus,
@@ -459,10 +445,6 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     serviceRegistry,
     featureFlags,
   });
-  knowledgeSemanticService.setGapRepairer(createWebKnowledgeGapRepairer({
-    searchService: webSearchService,
-    ingestService: knowledgeService,
-  }));
   agentKnowledgeSemanticService.setGapRepairer(createWebKnowledgeGapRepairer({
     searchService: webSearchService,
     ingestService: agentKnowledgeService,
@@ -597,7 +579,7 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     automationManager,
     gatewayMethods,
     artifactStore,
-    knowledgeService,
+    knowledgeService: agentKnowledgeService,
     agentKnowledgeService,
     homeGraphService,
     projectPlanningService,

package/src/tools/wrfc-agent-guard.ts

@@ -10,6 +10,20 @@ type AgentToolPolicyGuardOptions = {
   readonly getLastUserMessage?: () => string | null;
 };
 
+const READ_ONLY_AGENT_TOOL_MODES = [
+  'status',
+  'list',
+  'templates',
+  'get',
+  'budget',
+  'wrfc-chains',
+  'wrfc-history',
+  'cohort-status',
+  'cohort-report',
+] as const;
+
+const READ_ONLY_AGENT_TOOL_MODE_SET = new Set<string>(READ_ONLY_AGENT_TOOL_MODES);
+
 const LOCAL_AGENT_DENIAL = [
   'GoodVibes Agent does not spawn local Engineer/Reviewer/Tester/Verifier roots or run local WRFC chains.',
   'Keep ordinary assistant work serial in the main conversation.',
@@ -23,6 +37,7 @@ export function installAgentToolPolicyGuard(registry: ToolRegistry, options: Age
 }
 
 export function wrapAgentToolForAgentPolicy(tool: Tool, _options: AgentToolPolicyGuardOptions = {}): void {
+  narrowAgentToolDefinitionForAgentPolicy(tool);
   const originalExecute = tool.execute.bind(tool);
   tool.execute = async (args) => {
     const denial = validateAgentToolInvocationForAgentPolicy(args as AgentToolArgs);
@@ -32,7 +47,7 @@ export function wrapAgentToolForAgentPolicy(tool: Tool, _options: AgentToolPolic
 }
 
 export function validateAgentToolInvocationForAgentPolicy(args: AgentToolArgs): string | null {
-  if (args.mode === 'spawn' || args.mode === 'batch-spawn') return LOCAL_AGENT_DENIAL;
+  if (typeof args.mode === 'string' && !READ_ONLY_AGENT_TOOL_MODE_SET.has(args.mode)) return LOCAL_AGENT_DENIAL;
   return null;
 }
 
@@ -41,6 +56,27 @@ export function normalizeAgentToolInvocationForAgentPolicy(args: AgentToolArgs):
 }
 
 export const AGENT_LOCAL_SPAWN_DENIAL_MESSAGE = LOCAL_AGENT_DENIAL;
+export const AGENT_READ_ONLY_TOOL_MODES = READ_ONLY_AGENT_TOOL_MODES;
+
+function isRecord(value: unknown): value is Record<string, unknown> {
+  return typeof value === 'object' && value !== null && !Array.isArray(value);
+}
+
+function narrowAgentToolDefinitionForAgentPolicy(tool: Tool): void {
+  tool.definition.description = [
+    'Read-only local Agent inspection for GoodVibes Agent.',
+    'This product does not spawn local worker agents or run local WRFC chains.',
+    'For build/fix/review work, delegate to GoodVibes TUI through the explicit build-delegation path instead.',
+  ].join(' ');
+  tool.definition.sideEffects = [];
+
+  const properties = tool.definition.parameters.properties;
+  if (!isRecord(properties)) return;
+  const modeProperty = properties.mode;
+  if (!isRecord(modeProperty)) return;
+  modeProperty.enum = [...READ_ONLY_AGENT_TOOL_MODES];
+  modeProperty.description = 'Read-only Agent inspection mode. Local spawn, batch-spawn, cancel, message, wait, and plan modes are disabled in GoodVibes Agent.';
+}
 
 // Compatibility exports for copied TUI tests/imports during the near-fork phase.
 export const installWrfcAgentToolGuard = installAgentToolPolicyGuard;

package/src/version.ts

@@ -6,7 +6,7 @@ import { join } from 'node:path';
 // The prebuild script updates the fallback value before compilation.
 // Uses import.meta.dir (Bun) to locate package.json relative to this file,
 // which is correct regardless of the process working directory.
-let _version = '0.1.7';
+let _version = '0.1.8';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8'));
   _version = pkg.version ?? _version;

package/.goodvibes/agents/reviewer.md

@@ -1,48 +0,0 @@
----
-name: reviewer
-description: Serial review of GoodVibes Agent changes against product policy, release gates, and TypeScript quality
-tools: [read, find, analyze]
----
-
-You review GoodVibes Agent work. Your job is to identify concrete defects, product-boundary violations, and release risks. You verify claims by inspecting files and cite exact paths.
-
-## Review Priorities
-
-1. Agent policy: main-conversation serial behavior by default; no hidden local agent fanout; no default WRFC.
-2. Product boundary: Agent connects to the external daemon; it does not start, restart, install, or own daemon/listener services.
-3. Delegation boundary: build/fix/review code work is explicitly delegated to GoodVibes TUI through public contracts.
-4. TypeScript quality: Bun-first TypeScript only, no explicit `any`, no authored JavaScript variants.
-5. SDK boundary: public `@pellux/goodvibes-sdk` imports and daemon/operator routes only; no runtime imports from `goodvibes-tui/src/*`.
-6. Packaging: `goodvibes-agent` bin, Agent package identity, Agent docs, and no copied TUI-only package-facing guidance.
-
-## Review Process
-
-1. Read the completion report or changed-file list.
-2. Inspect the files that define the behavior under review.
-3. Check tests and release gates that should cover the change.
-4. Report findings first, ordered by severity.
-5. Keep summaries brief and secondary.
-
-## Output Format
-
-Use this structure:
-
-```text
-Findings
-- severity: file:line - issue and impact
-
-Open Questions
-- question or assumption, if any
-
-Validation Notes
-- tests/gates reviewed or missing
-```
-
-If there are no findings, say so clearly and list residual risk or missing validation.
-
-## What You Do Not Do
-
-- Do not modify code.
-- Do not spawn other agents.
-- Do not broaden scope beyond the reviewed slice.
-- Do not treat copied coding-TUI behavior as acceptable unless it is blocked, externalized, or explicitly delegated.