@pellux/goodvibes-agent 0.1.70 → 0.1.72

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

@@ -1,432 +0,0 @@
-import { resolve } from 'node:path';
-import type { CommandRegistry, CommandContext } from '../command-registry.ts';
-import { AGENT_TEMPLATES } from '@pellux/goodvibes-sdk/platform/tools';
-import { handleRemoteSetupCommand } from './remote-runtime-setup.ts';
-import { handleRemotePoolCommand } from './remote-runtime-pool.ts';
-import { requirePeerClient } from './runtime-services.ts';
-import { requireYesFlag, stripYesFlag } from './confirmation.ts';
-
-type RemoteConnectionLike = { agentId: string };
-type RemoteCancelContext = Pick<CommandContext, 'print'>;
-
-function printRemoteDelegationBoundary(ctx: Pick<CommandContext, 'print'>, requestedAction: string): void {
-  ctx.print([
-    'GoodVibes Agent does not dispatch remote/local coding workers from this surface.',
-    `  requested: ${requestedAction}`,
-    '  policy: keep ordinary work serial in the main assistant conversation',
-    '  build/fix/review: delegate one request to GoodVibes TUI through the public shared-session/build-delegation contract',
-    '  preserve: full original user ask and executionIntent; let TUI own execution and WRFC chain details when explicitly requested',
-  ].join('\n'));
-}
-
-export function handleRemoteCancelCommand(
-  agentId: string | undefined,
-  activeConnections: RemoteConnectionLike[],
-  ctx: RemoteCancelContext,
-): void {
-  if (!agentId) {
-    ctx.print('Usage: /remote cancel <agentId>');
-    return;
-  }
-  const connection = activeConnections.find((entry) => entry.agentId === agentId);
-  if (!connection) {
-    ctx.print(`Unknown remote connection: ${agentId}`);
-    return;
-  }
-  ctx.print([
-    'GoodVibes Agent remote control is read-only.',
-    `  requested: /remote cancel ${agentId}`,
-    '  policy: Agent does not cancel local remote-worker processes from this surface',
-    '  next: inspect with /remote show or delegate explicit build/fix/review work to GoodVibes TUI',
-  ].join('\n'));
-}
-
-export function registerRemoteRuntimeCommands(registry: CommandRegistry): void {
-  registry.register({
-    name: 'remote',
-    aliases: [],
-    description: 'Inspect remote workers, delegated work receipts, and review artifacts',
-    usage: '[list | show [agentId] | supervisor [workerId] | support [workerId] | recover [workerId] | setup [export <path> --yes] | env [export <path> --yes] | tunnel [review|export <path> --yes] | bootstrap [export <path> --yes|inspect <path>] | session <export|inspect|import> <path> [--yes] | pool <list|show> ... | dispatch [template] <description> | dispatch-pool <pool> [template] <description> | contract [workerId] | cancel <agentId> | export <agentId> [path] --yes | artifact list | artifact show <id> | artifact export <id> [path] --yes | review <id> | rerun-local <id> | import <path> --yes]',
-    async handler(args, ctx) {
-      if (args.length === 0) {
-        if (ctx.openRemotePanel) {
-          ctx.openRemotePanel();
-          return;
-        }
-        ctx.print('Remote panel is not available in this runtime.');
-        return;
-      }
-
-      let peerClient;
-      try {
-        peerClient = requirePeerClient(ctx);
-      } catch {
-        ctx.print('Remote runtime services are not available in this runtime.');
-        return;
-      }
-      const peerSnapshot = peerClient.getSnapshot();
-      const remoteRunners = peerClient.runners;
-      const activeConnections = [...peerSnapshot.acp.activeConnections];
-      const subcommand = args[0]?.toLowerCase() ?? 'show';
-
-      if (await handleRemoteSetupCommand(args, ctx, activeConnections, {
-        listContracts: () => remoteRunners.listContracts(),
-        exportSessionBundle: (path) => remoteRunners.exportSessionBundle(path),
-        importSessionBundle: (path) => remoteRunners.importSessionBundle(path),
-      })) {
-        return;
-      }
-
-      if (subcommand === 'list') {
-        const supervisor = peerSnapshot.supervisor;
-        const contracts = peerSnapshot.runners.contracts;
-        const pools = peerSnapshot.runners.pools;
-        const artifacts = peerSnapshot.runners.artifacts;
-        const lines = [
-          'Remote Control Surface',
-          `  active connections: ${activeConnections.length}`,
-          `  worker contracts: ${contracts.length}`,
-          `  worker pools: ${pools.length}`,
-          `  review artifacts: ${artifacts.length}`,
-          `  supervisor sessions: ${supervisor.sessions.length}`,
-          `  degraded sessions: ${supervisor.degradedConnections}`,
-        ];
-        if (activeConnections.length > 0) {
-          lines.push('  connections:');
-          for (const connection of activeConnections.slice(0, 12)) {
-            lines.push(`    ${connection.agentId}  ${connection.transportState}  msgs=${connection.messageCount} errs=${connection.errorCount}  ${connection.label}`);
-          }
-        }
-        if (contracts.length > 0) {
-          lines.push('  contracts:');
-          for (const contract of contracts.slice(0, 12)) {
-            lines.push(`    ${contract.runnerId}  ${contract.template}  ${contract.transport.state}  ${contract.capabilityCeiling.executionProtocol}/${contract.capabilityCeiling.reviewMode}`);
-          }
-        }
-        ctx.print(lines.join('\n'));
-        return;
-      }
-
-      if (subcommand === 'supervisor') {
-        const snapshot = peerSnapshot.supervisor;
-        const runnerId = args[1];
-        const selected = runnerId
-          ? snapshot.sessions.find((entry) => entry.runnerId === runnerId)
-          : snapshot.sessions[0];
-        if (!selected) {
-          ctx.print(runnerId ? `Unknown remote supervisor session: ${runnerId}` : 'No remote supervisor sessions are currently tracked.');
-          return;
-        }
-        ctx.print([
-          `Remote Supervisor ${selected.runnerId}`,
-          `  label: ${selected.label}`,
-          `  transport: ${selected.transportState}`,
-          `  heartbeat: ${selected.heartbeat.status}`,
-          `  heartbeat detail: ${selected.heartbeat.detail}`,
-          `  executionProtocol: ${selected.negotiation.executionProtocol}`,
-          `  reviewMode: ${selected.negotiation.reviewMode}`,
-          `  communicationLane: ${selected.negotiation.communicationLane}`,
-          `  trustClass: ${selected.negotiation.trustClass}`,
-          `  taskId: ${selected.taskId ?? 'n/a'}`,
-          `  messageCount: ${selected.messageCount}`,
-          `  errorCount: ${selected.errorCount}`,
-          ...(selected.lastError ? [`  lastError: ${selected.lastError}`] : []),
-          '  worker support:',
-          ...selected.capabilities.map((capability) => `    ${capability.id}: ${capability.supported ? 'yes' : 'no'} (${capability.detail})`),
-          '  recovery:',
-          ...selected.recovery.map((action) => `    ${action.command} — ${action.reason}`),
-        ].join('\n'));
-        return;
-      }
-
-      if (subcommand === 'support') {
-        const snapshot = peerSnapshot.supervisor;
-        const runnerId = args[1];
-        const selected = runnerId
-          ? snapshot.sessions.find((entry) => entry.runnerId === runnerId)
-          : snapshot.sessions[0];
-        if (!selected) {
-          ctx.print(runnerId ? `Unknown remote worker: ${runnerId}` : 'No remote supervisor sessions are currently tracked.');
-          return;
-        }
-        ctx.print([
-          `Remote Support ${selected.runnerId}`,
-          `  label: ${selected.label}`,
-          `  transport: ${selected.transportState}`,
-          `  executionProtocol: ${selected.negotiation.executionProtocol}`,
-          `  reviewMode: ${selected.negotiation.reviewMode}`,
-          `  communicationLane: ${selected.negotiation.communicationLane}`,
-          `  trustClass: ${selected.negotiation.trustClass}`,
-          '  worker support:',
-          ...selected.capabilities.map((capability) => (
-            `    ${capability.id}: ${capability.supported ? 'supported' : 'missing'} — ${capability.detail}`
-          )),
-        ].join('\n'));
-        return;
-      }
-
-      if (subcommand === 'recover') {
-        const snapshot = peerSnapshot.supervisor;
-        const runnerId = args[1];
-        const selected = runnerId
-          ? snapshot.sessions.find((entry) => entry.runnerId === runnerId)
-          : snapshot.sessions.find((entry) => entry.recovery.length > 0) ?? snapshot.sessions[0];
-        if (!selected) {
-          ctx.print(runnerId ? `Unknown remote worker: ${runnerId}` : 'No remote supervisor sessions are currently tracked.');
-          return;
-        }
-        const nextSteps = selected.recovery.length > 0
-          ? selected.recovery
-          : [{
-              id: 'show',
-              label: 'Review remote runtime',
-              command: `/remote show ${selected.runnerId}`,
-              reason: 'Inspect the current remote session before deciding on recovery.',
-            }];
-        ctx.print([
-          `Remote Recovery ${selected.runnerId}`,
-          `  label: ${selected.label}`,
-          `  transport: ${selected.transportState}`,
-          `  heartbeat: ${selected.heartbeat.status}`,
-          `  detail: ${selected.heartbeat.detail}`,
-          ...(selected.lastError ? [`  lastError: ${selected.lastError}`] : []),
-          ...(selected.taskId ? [`  bound task: ${selected.taskId}`] : []),
-          '  actions:',
-          ...nextSteps.map((action) => `    ${action.command} — ${action.reason}`),
-        ].join('\n'));
-        return;
-      }
-
-      if (handleRemotePoolCommand(args, ctx, remoteRunners)) {
-        return;
-      }
-
-      if (subcommand === 'show') {
-        const agentId = args[1];
-        const connection = agentId
-          ? activeConnections.find((entry) => entry.agentId === agentId)
-          : activeConnections[0];
-        if (!connection) {
-          ctx.print(agentId ? `Unknown remote connection: ${agentId}` : 'No active remote connections.');
-          return;
-        }
-        const contract = remoteRunners.getContract(connection.agentId);
-        ctx.print([
-          `Remote connection ${connection.agentId}`,
-          `  label: ${connection.label}`,
-          `  transport: ${connection.transportState}`,
-          `  completing: ${connection.completing ? 'yes' : 'no'}`,
-          `  connectedAt: ${connection.connectedAt ? new Date(connection.connectedAt).toISOString() : 'n/a'}`,
-          `  messageCount: ${connection.messageCount}`,
-          `  errorCount: ${connection.errorCount}`,
-          `  taskId: ${connection.taskId ?? 'n/a'}`,
-          `  lastError: ${connection.lastError ?? 'n/a'}`,
-          `  contract: ${contract?.id ?? 'n/a'}`,
-          `  pool: ${contract?.poolId ?? 'n/a'}`,
-          `  executionProtocol: ${contract?.capabilityCeiling.executionProtocol ?? 'n/a'}`,
-          `  reviewMode: ${contract?.capabilityCeiling.reviewMode ?? 'n/a'}`,
-          `  communicationLane: ${contract?.capabilityCeiling.communicationLane ?? 'n/a'}`,
-        ].join('\n'));
-        return;
-      }
-
-      if (subcommand === 'dispatch') {
-        let template = 'general';
-        let descriptionArgs = args.slice(1);
-        if (descriptionArgs.length > 0 && descriptionArgs[0] in AGENT_TEMPLATES) {
-          template = descriptionArgs[0]!;
-          descriptionArgs = descriptionArgs.slice(1);
-        }
-        const description = descriptionArgs.join(' ').trim();
-        if (description.length === 0) {
-          ctx.print('Usage: /remote dispatch [template] <description>');
-          return;
-        }
-        printRemoteDelegationBoundary(ctx, `/remote dispatch ${template} ${description}`);
-        return;
-      }
-
-      if (subcommand === 'dispatch-pool') {
-        const poolId = args[1];
-        if (!poolId) {
-          ctx.print('Usage: /remote dispatch-pool <pool> [template] <description>');
-          return;
-        }
-        const pool = remoteRunners.getPool(poolId);
-        if (!pool) {
-          ctx.print(`Unknown remote worker pool: ${poolId}`);
-          return;
-        }
-        let template = pool.preferredTemplate ?? 'general';
-        let descriptionArgs = args.slice(2);
-        if (descriptionArgs.length > 0 && descriptionArgs[0] in AGENT_TEMPLATES) {
-          template = descriptionArgs[0]!;
-          descriptionArgs = descriptionArgs.slice(1);
-        }
-        const description = descriptionArgs.join(' ').trim();
-        if (description.length === 0) {
-          ctx.print('Usage: /remote dispatch-pool <pool> [template] <description>');
-          return;
-        }
-        printRemoteDelegationBoundary(ctx, `/remote dispatch-pool ${pool.id} ${template} ${description}`);
-        return;
-      }
-
-      if (subcommand === 'contract') {
-        const agentId = args[1] ?? activeConnections[0]?.agentId;
-        if (!agentId) {
-          ctx.print('No remote worker contracts are available yet.');
-          return;
-        }
-        const contract = remoteRunners.getContract(agentId);
-        if (!contract) {
-          ctx.print(`Unknown remote worker: ${agentId}`);
-          return;
-        }
-        ctx.print([
-          `Remote worker contract ${contract.id}`,
-          `  workerId: ${contract.runnerId}`,
-          `  label: ${contract.label}`,
-          `  pool: ${contract.poolId ?? '(none)'}`,
-          `  trustClass: ${contract.trustClass}`,
-          `  template: ${contract.template}`,
-          `  transport: ${contract.transport.state}`,
-          `  tools: ${contract.capabilityCeiling.allowedTools.join(', ') || '(none)'}`,
-          `  ceiling: ${contract.capabilityCeiling.capabilityCeilingTools.join(', ') || '(none)'}`,
-          `  protocol: ${contract.capabilityCeiling.executionProtocol}`,
-          `  reviewMode: ${contract.capabilityCeiling.reviewMode}`,
-          `  communicationLane: ${contract.capabilityCeiling.communicationLane}`,
-          `  writeScope: ${contract.capabilityCeiling.writeScope.join(', ') || '(none)'}`,
-        ].join('\n'));
-        return;
-      }
-
-      if (subcommand === 'cancel') {
-        handleRemoteCancelCommand(
-          args[1],
-          activeConnections,
-          ctx,
-        );
-        return;
-      }
-
-      if (subcommand === 'export') {
-        const { rest, yes } = stripYesFlag(args);
-        const agentId = rest[1];
-        if (!agentId) {
-          ctx.print('Usage: /remote export <agentId> [path] --yes');
-          return;
-        }
-        if (!yes) {
-          requireYesFlag(ctx, `export remote review artifact for ${agentId}`, '/remote export <agentId> [path] --yes');
-          return;
-        }
-        const artifact = remoteRunners.captureArtifactForRunner(agentId);
-        if (!artifact) {
-          ctx.print(`Remote artifact export failed for ${agentId}.`);
-          return;
-        }
-        const exported = await remoteRunners.exportArtifact(artifact.id, rest[2]);
-        if (!exported) {
-          ctx.print(`Remote artifact export failed for ${agentId}.`);
-          return;
-        }
-        ctx.print(`Exported remote review artifact ${exported.artifact.id} to ${exported.path}`);
-        return;
-      }
-
-      if (subcommand === 'artifact') {
-        const mode = args[1]?.toLowerCase() ?? 'list';
-        if (mode === 'list') {
-          const artifacts = remoteRunners.listArtifacts();
-          if (artifacts.length === 0) {
-            ctx.print('No remote review artifacts captured yet.');
-            return;
-          }
-          ctx.print([
-            `Remote Review Artifacts (${artifacts.length})`,
-            ...artifacts.slice(0, 12).map((artifact) => (
-              `  ${artifact.id}  ${artifact.runnerId}  ${artifact.task.status}  ${artifact.task.summary}`
-            )),
-          ].join('\n'));
-          return;
-        }
-        if (mode === 'show') {
-          const artifactId = args[2];
-          if (!artifactId) {
-            ctx.print('Usage: /remote artifact show <artifactId>');
-            return;
-          }
-          const summary = remoteRunners.buildReviewSummary(artifactId);
-          ctx.print(summary ?? `Unknown remote artifact: ${artifactId}`);
-          return;
-        }
-        if (mode === 'export') {
-          const { rest, yes } = stripYesFlag(args);
-          const artifactId = rest[2];
-          if (!artifactId) {
-            ctx.print('Usage: /remote artifact export <artifactId> [path] --yes');
-            return;
-          }
-          if (!yes) {
-            requireYesFlag(ctx, `export remote review artifact ${artifactId}`, '/remote artifact export <artifactId> [path] --yes');
-            return;
-          }
-          const exported = await remoteRunners.exportArtifact(artifactId, rest[3]);
-          if (!exported) {
-            ctx.print(`Unknown remote artifact: ${artifactId}`);
-            return;
-          }
-          ctx.print(`Exported remote review artifact ${exported.artifact.id} to ${exported.path}`);
-          return;
-        }
-        ctx.print(`Unknown remote artifact subcommand: ${mode}`);
-        return;
-      }
-
-      if (subcommand === 'review') {
-        const artifactId = args[1];
-        if (!artifactId) {
-          ctx.print('Usage: /remote review <artifactId>');
-          return;
-        }
-        const summary = remoteRunners.buildReviewSummary(artifactId);
-        ctx.print(summary ?? `Unknown remote artifact: ${artifactId}`);
-        return;
-      }
-
-      if (subcommand === 'rerun-local') {
-        const artifactId = args[1];
-        if (!artifactId) {
-          ctx.print('Usage: /remote rerun-local <artifactId>');
-          return;
-        }
-        const artifact = remoteRunners.getArtifact(artifactId);
-        if (!artifact) {
-          ctx.print(`Unknown remote artifact: ${artifactId}`);
-          return;
-        }
-        printRemoteDelegationBoundary(ctx, `/remote rerun-local ${artifactId}`);
-        return;
-      }
-
-      if (subcommand === 'import') {
-        const { rest, yes } = stripYesFlag(args);
-        const path = rest[1];
-        if (!path) {
-          ctx.print('Usage: /remote import <path> --yes');
-          return;
-        }
-        if (!yes) {
-          requireYesFlag(ctx, `import remote review artifact from ${path}`, '/remote import <path> --yes');
-          return;
-        }
-        const artifact = await remoteRunners.importArtifact(path);
-        ctx.print(`Imported remote review artifact ${artifact.id} for worker ${artifact.runnerId}.`);
-        return;
-      }
-
-      ctx.print(`Unknown remote subcommand: ${subcommand}`);
-    },
-  });
-}

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

@@ -1,25 +0,0 @@
-import type { CommandRegistry } from '../command-registry.ts';
-import { handleReplayCommand } from '@pellux/goodvibes-sdk/platform/core';
-import { requireReplayEngine } from './runtime-services.ts';
-import { requireYesFlag, stripYesFlag } from './confirmation.ts';
-
-export function registerReplayRuntimeCommands(registry: CommandRegistry): void {
-  registry.register({
-    name: 'replay',
-    aliases: ['rep'],
-    description: 'Deterministic replay: load, step, seek, diff, and export recorded runs',
-    usage: '[load [runId] | step [n] | seek <rev> | diff | export <path> --yes]',
-    argsHint: '[load|step|seek|diff|export]',
-    handler(args, ctx) {
-      const command = args[0] ?? 'help';
-      const { rest, yes } = stripYesFlag(args);
-      if (command === 'export' && !yes) {
-        requireYesFlag(ctx, `export replay run to ${args[1] ?? '<path>'}`, '/replay export <path> --yes');
-        return;
-      }
-      const replayEngine = requireReplayEngine(ctx);
-      const result = handleReplayCommand({ replayEngine }, command, rest.slice(1));
-      ctx.print(result.output);
-    },
-  });
-}

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

@@ -1,220 +0,0 @@
-import { mkdirSync, readFileSync, writeFileSync } from 'node:fs';
-import { dirname, join, resolve } from 'node:path';
-import type { CommandRegistry } from '../command-registry.ts';
-import type { SelectionAction, SelectionItem } from '../selection-modal.ts';
-import { openCommandPanel, requireServiceRegistry, requireShellPaths } from './runtime-services.ts';
-import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
-import { GOODVIBES_AGENT_SURFACE_ROOT } from '../../config/surface.ts';
-import { requireYesFlag, stripYesFlag } from './confirmation.ts';
-
-export function registerServicesRuntimeCommands(registry: CommandRegistry): void {
-  registry.register({
-    name: 'services',
-    aliases: ['svc'],
-    description: 'Manage API service configurations',
-    usage: '[open|list|inspect <name>|test <name>|resolve <name>|auth <name>|auth-review|doctor|export <path> --yes|import <path> --yes]',
-    async handler(args, ctx) {
-      const parsed = stripYesFlag(args);
-      const commandArgs = [...parsed.rest];
-      const sub = commandArgs[0] ?? 'open';
-      const shellPaths = requireShellPaths(ctx);
-      if (sub === 'open' || sub === 'panel') {
-        openCommandPanel(ctx, 'services');
-        return;
-      }
-      const svcRegistry = requireServiceRegistry(ctx);
-      const all = svcRegistry.getAll();
-      const keys = Object.keys(all);
-      if (sub === 'inspect') {
-        const name = commandArgs[1];
-        if (!name) {
-          ctx.print('Usage: /services inspect <name>');
-          return;
-        }
-        const inspection = await svcRegistry.inspect(name);
-        if (!inspection) {
-          ctx.print(`Unknown service: ${name}`);
-          return;
-        }
-        ctx.print([
-          `Service ${name}`,
-          `  authType: ${inspection.config.authType}`,
-          `  baseUrl: ${inspection.config.baseUrl ?? '(none)'}`,
-          `  primaryCredential: ${inspection.hasPrimaryCredential ? 'present' : 'missing'}`,
-          `  passwordCredential: ${inspection.hasPasswordCredential ? 'present' : 'missing'}`,
-          `  webhookUrl: ${inspection.hasWebhookUrl ? 'present' : 'missing'}`,
-          `  signingSecret: ${inspection.hasSigningSecret ? 'present' : 'missing'}`,
-          `  publicKey: ${inspection.hasPublicKey ? 'present' : 'missing'}`,
-          `  appToken: ${inspection.hasAppToken ? 'present' : 'missing'}`,
-        ].join('\n'));
-        return;
-      }
-      if (sub === 'test') {
-        const name = commandArgs[1];
-        if (!name) {
-          ctx.print('Usage: /services test <name>');
-          return;
-        }
-        const result = await svcRegistry.testConnection(name);
-        ctx.print([
-          `Service test ${name}`,
-          `  ok: ${result.ok ? 'yes' : 'no'}`,
-          `  status: ${result.status ?? 'n/a'}`,
-          `  url: ${result.testedUrl ?? 'n/a'}`,
-          `  error: ${result.error ?? 'none'}`,
-        ].join('\n'));
-        return;
-      }
-      if (sub === 'resolve') {
-        const name = commandArgs[1];
-        if (!name) {
-          ctx.print('Usage: /services resolve <name>');
-          return;
-        }
-        const headers = await svcRegistry.resolveAuth(name);
-        if (!headers) {
-          ctx.print(`Service ${name} has no resolvable auth headers right now.`);
-          return;
-        }
-        ctx.print([
-          `Resolved auth headers for ${name}`,
-          ...Object.keys(headers).map((key) => `  ${key}: <redacted>`),
-        ].join('\n'));
-        return;
-      }
-      if (sub === 'auth') {
-        const name = commandArgs[1];
-        if (!name) {
-          ctx.print('Usage: /services auth <name>');
-          return;
-        }
-        const headers = await svcRegistry.resolveAuth(name);
-        if (!headers) {
-          ctx.print(`Service ${name} has no resolvable auth headers right now.`);
-          return;
-        }
-        ctx.print([
-          `Service auth ${name}`,
-          ...Object.keys(headers).map((key) => `  ${key}: <resolved>`),
-        ].join('\n'));
-        return;
-      }
-      if (sub === 'doctor') {
-        const inspections = await Promise.all(keys.map((name) => svcRegistry.inspect(name)));
-        const issues = inspections
-          .filter((inspection): inspection is NonNullable<typeof inspection> => inspection !== null)
-          .flatMap((inspection) => {
-            const findings: string[] = [];
-            if (!inspection.hasPrimaryCredential) findings.push(`${inspection.config.name}: missing primary credential`);
-            if (inspection.config.authType === 'basic' && !inspection.hasPasswordCredential) findings.push(`${inspection.config.name}: missing password credential`);
-            if (!inspection.config.baseUrl) findings.push(`${inspection.config.name}: no baseUrl configured`);
-            return findings;
-          });
-        ctx.print([
-          'Service Doctor',
-          `  configured: ${keys.length}`,
-          `  issues: ${issues.length}`,
-          ...(issues.length > 0 ? issues.map((issue) => `  ${issue}`) : ['  all configured services passed readiness checks']),
-        ].join('\n'));
-        return;
-      }
-      if (sub === 'auth-review') {
-        const inspections = await Promise.all(keys.map((name) => svcRegistry.inspect(name)));
-        const authCounts = new Map<string, number>();
-        const issues = inspections
-          .filter((inspection): inspection is NonNullable<typeof inspection> => inspection !== null)
-          .flatMap((inspection) => {
-            authCounts.set(inspection.config.authType, (authCounts.get(inspection.config.authType) ?? 0) + 1);
-            const findings: string[] = [];
-            if (!inspection.config.baseUrl) findings.push(`${inspection.config.name}: missing baseUrl`);
-            if ((inspection.config.authType === 'bearer' || inspection.config.authType === 'api-key') && !inspection.hasPrimaryCredential) {
-              findings.push(`${inspection.config.name}: missing primary credential`);
-            }
-            if (inspection.config.authType === 'basic' && !inspection.hasPasswordCredential) {
-              findings.push(`${inspection.config.name}: missing password credential`);
-            }
-            return findings;
-          });
-        ctx.print([
-          'Service Auth Review',
-          `  configured: ${keys.length}`,
-          ...[...authCounts.entries()].map(([authType, count]) => `  ${authType}: ${count}`),
-          ...(issues.length > 0 ? ['', ...issues.map((issue) => `  issue: ${issue}`)] : ['', '  all configured services have a complete auth posture']),
-        ].join('\n'));
-        return;
-      }
-      if (sub === 'export') {
-        const pathArg = commandArgs[1];
-        if (!pathArg) {
-          ctx.print('Usage: /services export <path> --yes');
-          return;
-        }
-        if (!parsed.yes) {
-          requireYesFlag(ctx, `export services config to ${pathArg}`, '/services export <path> --yes');
-          return;
-        }
-        const targetPath = shellPaths.resolveWorkspacePath(pathArg);
-        mkdirSync(dirname(targetPath), { recursive: true });
-        writeFileSync(targetPath, JSON.stringify(all, null, 2) + '\n', 'utf-8');
-        ctx.print(`Exported services config to ${targetPath}`);
-        return;
-      }
-      if (sub === 'import') {
-        const pathArg = commandArgs[1];
-        if (!pathArg) {
-          ctx.print('Usage: /services import <path> --yes');
-          return;
-        }
-        if (!parsed.yes) {
-          requireYesFlag(ctx, `import services config from ${pathArg}`, '/services import <path> --yes');
-          return;
-        }
-        const sourcePath = shellPaths.resolveWorkspacePath(pathArg);
-        try {
-          const parsed = JSON.parse(readFileSync(sourcePath, 'utf-8')) as Record<string, unknown>;
-          const targetPath = shellPaths.resolveProjectPath(GOODVIBES_AGENT_SURFACE_ROOT, 'services.json');
-          mkdirSync(dirname(targetPath), { recursive: true });
-          writeFileSync(targetPath, JSON.stringify(parsed, null, 2) + '\n', 'utf-8');
-          ctx.print(`Imported services config from ${sourcePath}`);
-        } catch (error) {
-          ctx.print(`Failed to import services config: ${summarizeError(error)}`);
-        }
-        return;
-      }
-      if (ctx.openSelection) {
-        const testAction = new Map<string, SelectionAction>([['t', 'select' as const]]);
-        const items: SelectionItem[] = keys.length === 0
-          ? [{ id: '_empty', label: 'No services configured', detail: '.goodvibes/agent/services.json' }]
-          : keys.map((key) => ({ id: key, label: all[key].name ?? key, detail: `${all[key].authType}  ${all[key].baseUrl ?? '(no url)'}`, actions: '[t] test' }));
-        ctx.openSelection('Services', items, { allowSearch: true, customActions: testAction }, (result) => {
-          if (!result || result.item.id === '_empty') return;
-          const svc = all[result.item.id];
-          if (!svc) return;
-          if (result.action === 'select') {
-            void svcRegistry.testConnection(result.item.id).then((testResult) => {
-              ctx.print([
-                `Service test ${result.item.id}`,
-                `  ok: ${testResult.ok ? 'yes' : 'no'}`,
-                `  status: ${testResult.status ?? 'n/a'}`,
-                `  url: ${testResult.testedUrl ?? 'n/a'}`,
-                `  error: ${testResult.error ?? 'none'}`,
-              ].join('\n'));
-            });
-            return;
-          }
-          ctx.print([
-            `Service ${result.item.id}`,
-            `  authType: ${svc.authType}`,
-            `  baseUrl: ${svc.baseUrl ?? '(none)'}`,
-          ].join('\n'));
-        });
-        return;
-      }
-      if (keys.length === 0) {
-        ctx.print('[services] No services configured. Add entries to .goodvibes/agent/services.json');
-        return;
-      }
-      ctx.print(['Services:', '', ...keys.map((key) => `  ${key.padEnd(20)} ${all[key].authType.padEnd(10)} ${all[key].baseUrl ?? '(no url)'}`)].join('\n'));
-    },
-  });
-}