@pellux/goodvibes-agent 0.1.117 → 1.0.1

package/src/input/commands/agent-skills-runtime.ts

@@ -8,39 +8,15 @@ import {
   type AgentSkillRecord,
 } from '../../agent/skill-registry.ts';
 import { discoverSkills, type SkillRecord } from '../../agent/skill-discovery.ts';
+import { formatAgentRecordOrigin, formatAgentRecordReviewState } from '../../agent/record-labels.ts';
 import type { CommandContext, CommandRegistry } from '../command-registry.ts';
+import { parseAgentLocalLibraryArgs, type ParsedAgentLocalLibraryArgs } from './agent-local-library-args.ts';
 import { requireShellPaths } from './runtime-services.ts';
 
-interface ParsedSkillArgs {
-  readonly rest: readonly string[];
-  readonly flags: ReadonlyMap<string, string>;
-  readonly yes: boolean;
-}
+const SKILL_VALUE_FLAGS = ['name', 'description', 'procedure', 'tags', 'triggers', 'requires-env', 'requires-command', 'requires-commands', 'skills'] as const;
 
-function parseSkillArgs(args: readonly string[]): ParsedSkillArgs {
-  const flags = new Map<string, string>();
-  const rest: string[] = [];
-  let yes = false;
-  for (let index = 0; index < args.length; index += 1) {
-    const token = args[index] ?? '';
-    if (token === '--yes') {
-      yes = true;
-      continue;
-    }
-    if (token.startsWith('--')) {
-      const key = token.slice(2);
-      const next = args[index + 1];
-      if (next !== undefined && !next.startsWith('--')) {
-        flags.set(key, next);
-        index += 1;
-      } else {
-        flags.set(key, 'true');
-      }
-      continue;
-    }
-    rest.push(token);
-  }
-  return { rest, flags, yes };
+function parseSkillArgs(args: readonly string[]): ParsedAgentLocalLibraryArgs {
+  return parseAgentLocalLibraryArgs(args, { valueFlags: SKILL_VALUE_FLAGS });
 }
 
 function splitList(value: string | undefined): readonly string[] {
@@ -58,23 +34,34 @@ function requiredFlag(flags: ReadonlyMap<string, string>, key: string): string {
   return value;
 }
 
+function formatSkillReceipt(title: string, record: { readonly id: string; readonly name: string }, extra: readonly string[] = []): string {
+  return [
+    `${title} ${record.id}`,
+    `  id ${record.id}`,
+    `  name ${record.name}`,
+    ...extra,
+  ].join('\n');
+}
+
 function summarizeSkill(skill: AgentSkillRecord): string {
   const enabled = skill.enabled ? 'enabled' : 'disabled';
-  const tags = skill.tags.length > 0 ? ` tags=${skill.tags.join(',')}` : '';
+  const tags = skill.tags.length > 0 ? ` tags ${skill.tags.join(', ')}` : '';
   const readiness = evaluateAgentSkillReadiness(skill);
+  const review = formatAgentRecordReviewState(skill.reviewState);
   const ready = readiness.ready ? 'ready' : `needs ${readiness.missing.map(formatAgentSkillRequirement).join(',')}`;
-  return `  ${skill.id}  ${enabled}  ${skill.reviewState}  ${ready}  ${skill.name} - ${skill.description}${tags}`;
+  return `  ${skill.id}  ${enabled}  ${review}  ${ready}  ${skill.name} - ${skill.description}${tags}`;
 }
 
 function summarizeBundle(bundle: AgentSkillBundleRecord, skills: readonly AgentSkillRecord[]): string {
   const enabled = bundle.enabled ? 'enabled' : 'disabled';
   const readiness = evaluateAgentSkillBundleReadiness(bundle, skills);
+  const review = formatAgentRecordReviewState(bundle.reviewState);
   const missing = [
     ...readiness.missingRequirements.map(formatAgentSkillRequirement),
-    ...readiness.missingSkillIds.map((skillId) => `skill:${skillId}`),
+    ...readiness.missingSkillIds.map((skillId) => `skill ${skillId}`),
   ];
   const ready = readiness.ready ? 'ready' : `needs ${missing.join(',')}`;
-  return `  ${bundle.id}  ${enabled}  ${bundle.reviewState}  ${ready}  ${bundle.name} - ${bundle.description}  skills=${bundle.skillIds.join(',')}`;
+  return `  ${bundle.id}  ${enabled}  ${review}  ${ready}  ${bundle.name} - ${bundle.description}  skills ${bundle.skillIds.join(', ')}`;
 }
 
 function renderList(title: string, registry: AgentSkillRegistry, skills: readonly AgentSkillRecord[], emptyMessage?: string): string {
@@ -82,21 +69,21 @@ function renderList(title: string, registry: AgentSkillRegistry, skills: readonl
   if (skills.length === 0) {
     return emptyMessage
       ? `${title}\n  ${emptyMessage}`
-      : `${title}\n  No local Agent skills yet. Create one with /agent-skills create --name <name> --description <summary> --procedure <steps>.`;
+      : `${title}\n  No Agent-local skills yet. Create one with /skills create --name <name> --description <summary> --procedure <steps>.`;
   }
   return [
     `${title} (${skills.length})`,
-    `  store: ${snapshot.path}`,
-    `  enabled: ${snapshot.enabledSkills.length}`,
+    `  store ${snapshot.path}`,
+    `  enabled ${snapshot.enabledSkills.length}`,
     ...skills.map(summarizeSkill),
   ].join('\n');
 }
 
 function summarizeDiscoveredSkill(skill: SkillRecord): string {
   const description = skill.description ? ` - ${skill.description}` : '';
-  const dependencies = skill.dependencies.length > 0 ? ` deps=${skill.dependencies.join(',')}` : '';
-  const includes = skill.includes.length > 0 ? ` includes=${skill.includes.join(',')}` : '';
-  return `  ${skill.name}  ${skill.origin}${description}${dependencies}${includes}\n    path: ${skill.path}`;
+  const dependencies = skill.dependencies.length > 0 ? ` deps ${skill.dependencies.join(', ')}` : '';
+  const includes = skill.includes.length > 0 ? ` includes ${skill.includes.join(', ')}` : '';
+  return `  ${skill.name}  ${skill.origin}${description}${dependencies}${includes}\n    path ${skill.path}`;
 }
 
 function renderDiscoveredSkills(skills: readonly SkillRecord[]): string {
@@ -111,7 +98,7 @@ function renderDiscoveredSkills(skills: readonly SkillRecord[]): string {
     `Discovered Agent skill files (${skills.length})`,
     ...skills.map(summarizeDiscoveredSkill),
     '',
-    'Import one with: /agent-skills import-discovered <name> --yes',
+    'Import one with: /skills import-discovered <name> --yes',
   ].join('\n');
 }
 
@@ -150,23 +137,23 @@ async function importDiscoveredSkill(args: readonly string[], ctx: CommandContex
   const parsed = parseSkillArgs(args);
   const name = parsed.rest.join(' ').trim();
   if (!name) {
-    ctx.print('Usage: /agent-skills import-discovered <name> [--enabled] --yes');
+    ctx.print('Usage: /skills import-discovered <name> [--enabled] --yes');
     return;
   }
   const discovered = findDiscoveredSkill(await discoverSkills(requireShellPaths(ctx)), name);
   if (!discovered) {
-    ctx.print(`Unknown discovered Agent skill: ${name}\nRun /agent-skills discover to inspect available skill files.`);
+    ctx.print(`Unknown discovered Agent skill ${name}\nRun /skills discover to inspect available skill files.`);
     return;
   }
   if (!parsed.yes) {
     ctx.print([
       'Agent skill import preview',
-      `  name: ${discovered.name}`,
-      `  origin: ${discovered.origin}`,
-      `  path: ${discovered.path}`,
-      `  description: ${discovered.description || '(none)'}`,
-      `  procedure characters: ${discovered.body.length}`,
-      '  next: rerun with --yes to import into the Agent-local skill registry',
+      `  name ${discovered.name}`,
+      `  origin ${discovered.origin}`,
+      `  path ${discovered.path}`,
+      `  description ${discovered.description || '(none)'}`,
+      `  procedure characters ${discovered.body.length}`,
+      '  next rerun with --yes to import into the Agent-local skill registry',
     ].join('\n'));
     return;
   }
@@ -182,9 +169,9 @@ async function importDiscoveredSkill(args: readonly string[], ctx: CommandContex
     }),
     enabled: parsed.flags.get('enabled') === 'true',
     source: 'imported',
-    provenance: `discovered:${discovered.origin}:${discovered.path}`,
+    provenance: `Imported file (${discovered.origin}): ${discovered.path}`,
   });
-  ctx.print(`Imported Agent skill ${skill.id}: ${skill.name}`);
+  ctx.print(formatSkillReceipt('Imported Agent skill', skill, [`  enabled ${skill.enabled ? 'yes' : 'no'}`]));
 }
 
 function renderBundleList(title: string, registry: AgentSkillRegistry, bundles: readonly AgentSkillBundleRecord[], emptyMessage?: string): string {
@@ -192,11 +179,11 @@ function renderBundleList(title: string, registry: AgentSkillRegistry, bundles:
   if (bundles.length === 0) {
     return emptyMessage
       ? `${title}\n  ${emptyMessage}`
-      : `${title}\n  No local Agent skill bundles yet. Create one with /agent-skills bundle create --name <name> --description <summary> --skills <id,id>.`;
+      : `${title}\n  No Agent-local skill bundles yet. Create one with /skills bundle create --name <name> --description <summary> --skills <id,id>.`;
   }
   return [
     `${title} (${bundles.length})`,
-    `  store: ${snapshot.path}`,
+    `  store ${snapshot.path}`,
     `  enabled bundles: ${snapshot.enabledBundles.length}`,
     `  active skills: ${snapshot.activeSkills.length}`,
     ...bundles.map((bundle) => summarizeBundle(bundle, snapshot.skills)),
@@ -207,14 +194,13 @@ function renderSkill(skill: AgentSkillRecord): string {
   const readiness = evaluateAgentSkillReadiness(skill);
   return [
     `Skill ${skill.name}`,
-    `  id: ${skill.id}`,
+    `  id ${skill.id}`,
     `  enabled: ${skill.enabled ? 'yes' : 'no'}`,
     `  readiness: ${readiness.ready ? 'ready' : 'needs setup'}`,
     `  requirements: ${skill.requirements.map(formatAgentSkillRequirement).join(', ') || '(none)'}`,
     readiness.missing.length > 0 ? `  missing: ${readiness.missing.map(formatAgentSkillRequirement).join(', ')}` : '',
-    `  review: ${skill.reviewState}`,
-    `  source: ${skill.source}`,
-    `  provenance: ${skill.provenance}`,
+    `  review: ${formatAgentRecordReviewState(skill.reviewState)}`,
+    `  origin: ${formatAgentRecordOrigin(skill.source, skill.provenance)}`,
     `  tags: ${skill.tags.join(', ') || '(none)'}`,
     `  triggers: ${skill.triggers.join(', ') || '(manual)'}`,
     `  created: ${skill.createdAt}`,
@@ -234,17 +220,16 @@ function renderBundle(bundle: AgentSkillBundleRecord, registry: AgentSkillRegist
   const readiness = evaluateAgentSkillBundleReadiness(bundle, registry.snapshot().skills);
   const missing = [
     ...readiness.missingRequirements.map(formatAgentSkillRequirement),
-    ...readiness.missingSkillIds.map((skillId) => `skill:${skillId}`),
+    ...readiness.missingSkillIds.map((skillId) => `skill ${skillId}`),
   ];
   return [
     `Skill Bundle ${bundle.name}`,
-    `  id: ${bundle.id}`,
+    `  id ${bundle.id}`,
     `  enabled: ${bundle.enabled ? 'yes' : 'no'}`,
     `  readiness: ${readiness.ready ? 'ready' : 'needs setup'}`,
     missing.length > 0 ? `  missing: ${missing.join(', ')}` : '',
-    `  review: ${bundle.reviewState}`,
-    `  source: ${bundle.source}`,
-    `  provenance: ${bundle.provenance}`,
+    `  review: ${formatAgentRecordReviewState(bundle.reviewState)}`,
+    `  origin: ${formatAgentRecordOrigin(bundle.source, bundle.provenance)}`,
     `  skills: ${bundle.skillIds.join(', ')}`,
     `  created: ${bundle.createdAt}`,
     `  updated: ${bundle.updatedAt}`,
@@ -257,7 +242,10 @@ function renderBundle(bundle: AgentSkillBundleRecord, registry: AgentSkillRegist
 }
 
 function printError(ctx: CommandContext, error: unknown): void {
-  ctx.print(`Error: ${error instanceof Error ? error.message : String(error)}`);
+  ctx.print([
+    'Error',
+    `  message ${error instanceof Error ? error.message : String(error)}`,
+  ].join('\n'));
 }
 
 function runBundleCommand(args: readonly string[], ctx: CommandContext, skillRegistry: AgentSkillRegistry): void {
@@ -274,7 +262,7 @@ function runBundleCommand(args: readonly string[], ctx: CommandContext, skillReg
   if (sub === 'attention' || sub === 'needs-setup') {
     const snapshot = skillRegistry.snapshot();
     const bundles = snapshot.bundles.filter((bundle) => !evaluateAgentSkillBundleReadiness(bundle, snapshot.skills).ready);
-    ctx.print(renderBundleList('Agent Skill Bundles needing setup', skillRegistry, bundles, 'No local Agent skill bundles need setup.'));
+    ctx.print(renderBundleList('Agent Skill Bundles needing setup', skillRegistry, bundles, 'No Agent-local skill bundles need setup.'));
     return;
   }
   if (sub === 'search') {
@@ -285,11 +273,11 @@ function runBundleCommand(args: readonly string[], ctx: CommandContext, skillReg
   if (sub === 'show') {
     const id = args[1];
     if (!id) {
-      ctx.print('Usage: /agent-skills bundle show <id>');
+      ctx.print('Usage: /skills bundle show <id>');
       return;
     }
     const bundle = skillRegistry.getBundle(id);
-    ctx.print(bundle ? renderBundle(bundle, skillRegistry) : `Unknown Agent skill bundle: ${id}`);
+    ctx.print(bundle ? renderBundle(bundle, skillRegistry) : `Unknown Agent skill bundle ${id}`);
     return;
   }
   if (sub === 'create') {
@@ -300,15 +288,15 @@ function runBundleCommand(args: readonly string[], ctx: CommandContext, skillReg
       skillIds: splitList(requiredFlag(parsed.flags, 'skills')),
       enabled: parsed.flags.get('enabled') === 'true',
       source: 'user',
-      provenance: 'slash-command',
+      provenance: 'Command',
     });
-    ctx.print(`Created Agent skill bundle ${bundle.id}: ${bundle.name}`);
+    ctx.print(formatSkillReceipt('Created Agent skill bundle', bundle));
     return;
   }
   if (sub === 'update') {
     const id = args[1];
     if (!id) {
-      ctx.print('Usage: /agent-skills bundle update <id> [--name ...] [--description ...] [--skills id,id]');
+      ctx.print('Usage: /skills bundle update <id> [--name ...] [--description ...] [--skills id,id]');
       return;
     }
     const parsed = parseSkillArgs(args.slice(2));
@@ -316,46 +304,46 @@ function runBundleCommand(args: readonly string[], ctx: CommandContext, skillReg
       name: parsed.flags.get('name'),
       description: parsed.flags.get('description'),
       skillIds: parsed.flags.has('skills') ? splitList(parsed.flags.get('skills')) : undefined,
-      provenance: 'slash-command',
+      provenance: 'Command',
     });
-    ctx.print(`Updated Agent skill bundle ${updated.id}: ${updated.name}`);
+    ctx.print(formatSkillReceipt('Updated Agent skill bundle', updated));
     return;
   }
   if (sub === 'enable' || sub === 'disable') {
     const id = args[1];
     if (!id) {
-      ctx.print(`Usage: /agent-skills bundle ${sub} <id>`);
+      ctx.print(`Usage: /skills bundle ${sub} <id>`);
       return;
     }
     const bundle = skillRegistry.setBundleEnabled(id, sub === 'enable');
-    ctx.print(`${sub === 'enable' ? 'Enabled' : 'Disabled'} Agent skill bundle ${bundle.id}: ${bundle.name}`);
+    ctx.print(formatSkillReceipt(`${sub === 'enable' ? 'Enabled' : 'Disabled'} Agent skill bundle`, bundle));
     return;
   }
   if (sub === 'review') {
     const id = args[1];
     if (!id) {
-      ctx.print('Usage: /agent-skills bundle review <id>');
+      ctx.print('Usage: /skills bundle review <id>');
       return;
     }
     const bundle = skillRegistry.markBundleReviewed(id);
-    ctx.print(`Reviewed Agent skill bundle ${bundle.id}.`);
+    ctx.print(formatSkillReceipt('Reviewed Agent skill bundle', bundle));
     return;
   }
   if (sub === 'stale') {
     const id = args[1];
     if (!id) {
-      ctx.print('Usage: /agent-skills bundle stale <id> <reason...>');
+      ctx.print('Usage: /skills bundle stale <id> <reason...>');
       return;
     }
     const bundle = skillRegistry.markBundleStale(id, args.slice(2).join(' '));
-    ctx.print(`Marked Agent skill bundle ${bundle.id} stale.`);
+    ctx.print(formatSkillReceipt('Marked Agent skill bundle stale', bundle));
     return;
   }
   if (sub === 'delete' || sub === 'remove') {
     const parsed = parseSkillArgs(args.slice(1));
     const id = parsed.rest[0];
     if (!id) {
-      ctx.print('Usage: /agent-skills bundle delete <id> --yes');
+      ctx.print('Usage: /skills bundle delete <id> --yes');
       return;
     }
     if (!parsed.yes) {
@@ -363,10 +351,10 @@ function runBundleCommand(args: readonly string[], ctx: CommandContext, skillReg
       return;
     }
     const removed = skillRegistry.deleteBundle(id);
-    ctx.print(`Deleted Agent skill bundle ${removed.id}: ${removed.name}`);
+    ctx.print(formatSkillReceipt('Deleted Agent skill bundle', removed));
     return;
   }
-  ctx.print('Usage: /agent-skills bundle [list|enabled|attention|search|show|create|update|enable|disable|review|stale|delete]');
+  ctx.print('Usage: /skills bundle [list|enabled|attention|search|show|create|update|enable|disable|review|stale|delete]');
 }
 
 export async function runAgentSkillsRuntimeCommand(args: readonly string[], ctx: CommandContext): Promise<void> {
@@ -400,7 +388,7 @@ export async function runAgentSkillsRuntimeCommand(args: readonly string[], ctx:
     }
     if (sub === 'attention' || sub === 'needs-setup') {
       const skills = skillRegistry.list().filter((skill) => !evaluateAgentSkillReadiness(skill).ready);
-      ctx.print(renderList('Agent Skills needing setup', skillRegistry, skills, 'No local Agent skills need setup.'));
+      ctx.print(renderList('Agent Skills needing setup', skillRegistry, skills, 'No Agent-local skills need setup.'));
       return;
     }
     if (sub === 'search') {
@@ -411,11 +399,11 @@ export async function runAgentSkillsRuntimeCommand(args: readonly string[], ctx:
     if (sub === 'show') {
       const id = args[1];
       if (!id) {
-        ctx.print('Usage: /agent-skills show <id>');
+        ctx.print('Usage: /skills show <id>');
         return;
       }
       const skill = skillRegistry.get(id);
-      ctx.print(skill ? renderSkill(skill) : `Unknown Agent skill: ${id}`);
+      ctx.print(skill ? renderSkill(skill) : `Unknown Agent skill ${id}`);
       return;
     }
     if (sub === 'create') {
@@ -433,15 +421,15 @@ export async function runAgentSkillsRuntimeCommand(args: readonly string[], ctx:
         }),
         enabled: parsed.flags.get('enabled') === 'true',
         source: 'user',
-        provenance: 'slash-command',
+        provenance: 'Command',
       });
-      ctx.print(`Created Agent skill ${skill.id}: ${skill.name}`);
+      ctx.print(formatSkillReceipt('Created Agent skill', skill));
       return;
     }
     if (sub === 'update') {
       const id = args[1];
       if (!id) {
-        ctx.print('Usage: /agent-skills update <id> [--name ...] [--description ...] [--procedure ...]');
+        ctx.print('Usage: /skills update <id> [--name ...] [--description ...] [--procedure ...]');
         return;
       }
       const parsed = parseSkillArgs(args.slice(2));
@@ -457,46 +445,46 @@ export async function runAgentSkillsRuntimeCommand(args: readonly string[], ctx:
             commands: splitList(parsed.flags.get('requires-command') ?? parsed.flags.get('requires-commands')),
           })
           : undefined,
-        provenance: 'slash-command',
+        provenance: 'Command',
       });
-      ctx.print(`Updated Agent skill ${updated.id}: ${updated.name}`);
+      ctx.print(formatSkillReceipt('Updated Agent skill', updated));
       return;
     }
     if (sub === 'enable' || sub === 'disable') {
       const id = args[1];
       if (!id) {
-        ctx.print(`Usage: /agent-skills ${sub} <id>`);
+        ctx.print(`Usage: /skills ${sub} <id>`);
         return;
       }
       const skill = skillRegistry.setEnabled(id, sub === 'enable');
-      ctx.print(`${sub === 'enable' ? 'Enabled' : 'Disabled'} Agent skill ${skill.id}: ${skill.name}`);
+      ctx.print(formatSkillReceipt(`${sub === 'enable' ? 'Enabled' : 'Disabled'} Agent skill`, skill));
       return;
     }
     if (sub === 'review') {
       const id = args[1];
       if (!id) {
-        ctx.print('Usage: /agent-skills review <id>');
+        ctx.print('Usage: /skills review <id>');
         return;
       }
       const skill = skillRegistry.markReviewed(id);
-      ctx.print(`Reviewed Agent skill ${skill.id}.`);
+      ctx.print(formatSkillReceipt('Reviewed Agent skill', skill));
       return;
     }
     if (sub === 'stale') {
       const id = args[1];
       if (!id) {
-        ctx.print('Usage: /agent-skills stale <id> <reason...>');
+        ctx.print('Usage: /skills stale <id> <reason...>');
         return;
       }
       const skill = skillRegistry.markStale(id, args.slice(2).join(' '));
-      ctx.print(`Marked Agent skill ${skill.id} stale.`);
+      ctx.print(formatSkillReceipt('Marked Agent skill stale', skill));
       return;
     }
     if (sub === 'delete' || sub === 'remove') {
       const parsed = parseSkillArgs(args.slice(1));
       const id = parsed.rest[0];
       if (!id) {
-        ctx.print('Usage: /agent-skills delete <id> --yes');
+        ctx.print('Usage: /skills delete <id> --yes');
         return;
       }
       if (!parsed.yes) {
@@ -504,10 +492,10 @@ export async function runAgentSkillsRuntimeCommand(args: readonly string[], ctx:
         return;
       }
       const removed = skillRegistry.deleteSkill(id);
-      ctx.print(`Deleted Agent skill ${removed.id}: ${removed.name}`);
+      ctx.print(formatSkillReceipt('Deleted Agent skill', removed));
       return;
     }
-    ctx.print('Usage: /agent-skills [list|enabled|attention|discover|import-discovered|search|show|create|update|enable|disable|review|stale|delete|bundle]');
+    ctx.print('Usage: /skills [list|enabled|attention|discover|import-discovered|search|show|create|update|enable|disable|review|stale|delete|bundle]');
   } catch (error) {
     printError(ctx, error);
   }
@@ -515,9 +503,9 @@ export async function runAgentSkillsRuntimeCommand(args: readonly string[], ctx:
 
 export function registerAgentSkillsRuntimeCommands(registry: CommandRegistry): void {
   registry.register({
-    name: 'agent-skills',
-    aliases: ['askills', 'local-skills', 'skills', 'skill'],
-    description: 'Manage local GoodVibes Agent skills',
+    name: 'skills',
+    aliases: ['skill', 'agent-skills', 'askills', 'local-skills'],
+    description: 'Manage Agent-local skills',
     usage: '[list|enabled|attention|discover|import-discovered <name> --yes|search <query>|show <id>|create --name <name> --description <summary> --procedure <steps> [--requires-env A,B] [--requires-command gh,jq]|update <id> [--name ...] [--description ...] [--procedure ...]|enable <id>|disable <id>|review <id>|stale <id> <reason...>|delete <id> --yes|bundle ...]',
     handler: runAgentSkillsRuntimeCommand,
   });

package/src/input/commands/agent-workspace-runtime.ts

@@ -1,18 +1,34 @@
-import type { CommandRegistry } from '../command-registry.ts';
+import type { CommandContext, CommandRegistry } from '../command-registry.ts';
+import { AGENT_WORKSPACE_CATEGORY_IDS } from '../agent-workspace-types.ts';
+
+const AGENT_WORKSPACE_ARGS_HINT = `${AGENT_WORKSPACE_CATEGORY_IDS.join('|')}|connected-host`;
 
 export function registerAgentWorkspaceRuntimeCommands(registry: CommandRegistry): void {
+  function openAgentWorkspace(ctx: CommandContext, categoryId: string | undefined): void {
+    if (!ctx.openAgentWorkspace) {
+      ctx.print('Agent operator workspace is not available in this runtime.');
+      return;
+    }
+    ctx.openAgentWorkspace(categoryId);
+  }
+
   registry.register({
     name: 'agent',
     aliases: ['home', 'operator'],
     description: 'Open the GoodVibes Agent operator workspace',
     usage: '[category]',
-    argsHint: 'home|setup|channels|tools|knowledge|voice-media|profiles|memory|personas|skills|routines|work|automation|delegate',
+    argsHint: AGENT_WORKSPACE_ARGS_HINT,
     handler(args, ctx) {
-      if (!ctx.openAgentWorkspace) {
-        ctx.print('Agent operator workspace is not available in this runtime.');
-        return;
-      }
-      ctx.openAgentWorkspace(args[0]);
+      openAgentWorkspace(ctx, args[0]);
+    },
+  });
+  registry.register({
+    name: 'notes',
+    aliases: ['scratchpad'],
+    description: 'Open Agent-local scratchpad notes in the operator workspace',
+    usage: '',
+    handler(_args, ctx) {
+      openAgentWorkspace(ctx, 'notes');
     },
   });
 }

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

@@ -39,7 +39,7 @@ function countSetupGaps(items: readonly { readonly missingRequirementCount?: num
 }
 
 function formatWorkPlanLine(total: number, counts: StatusCounts): string {
-  if (total === 0) return '  work plan: empty';
+  if (total === 0) return '  work plan empty';
   const active = counts.pending + counts.in_progress + counts.blocked;
   return [
     `  work plan: ${plural(total, 'item')}`,
@@ -52,8 +52,8 @@ function formatWorkPlanLine(total: number, counts: StatusCounts): string {
 }
 
 function formatWarnings(warnings: readonly string[]): readonly string[] {
-  if (warnings.length === 0) return ['  warnings: none'];
-  return ['  warnings:', ...warnings.slice(0, 4).map((warning) => `    - ${warning}`)];
+  if (warnings.length === 0) return ['  warnings none'];
+  return ['  warnings', ...warnings.slice(0, 4).map((warning) => `    - ${warning}`)];
 }
 
 export function formatAgentOperatorBriefing(ctx: CommandContext): string {
@@ -76,63 +76,64 @@ export function formatAgentOperatorBriefing(ctx: CommandContext): string {
 
   const nextActions = [
     snapshot.provider === 'unknown' || snapshot.model === 'unknown'
-      ? 'Choose the assistant model with /model.'
+      ? 'Choose the assistant model from Agent Workspace -> Home -> Choose model.'
       : '',
     snapshot.localMemoryCount === 0
-      ? 'Store durable non-secret facts with /memory add or the Agent workspace memory form.'
+      ? 'Store durable non-secret facts from Agent Workspace -> Memory -> Add memory.'
       : snapshot.localMemoryReviewQueueCount > 0
-        ? `Review ${plural(snapshot.localMemoryReviewQueueCount, 'memory record')} with /memory queue.`
+        ? `Review ${plural(snapshot.localMemoryReviewQueueCount, 'memory record')} from Agent Workspace -> Memory -> Review queue.`
         : '',
     !hasLocalSkillBehavior
-      ? 'Create reusable procedures with /agent-skills create or import reviewed skill files.'
+      ? 'Create reusable procedures from Agent Workspace -> Skills or import reviewed skill files.'
       : '',
     skillSetupGaps > 0
-      ? `Resolve ${plural(skillSetupGaps, 'skill')} with setup gaps from /agent skills.`
+      ? `Resolve ${plural(skillSetupGaps, 'skill')} with setup gaps from Agent Workspace -> Skills.`
       : '',
     skillBundleSetupGaps > 0
-      ? `Resolve ${plural(skillBundleSetupGaps, 'skill bundle')} with setup gaps from /agent skills.`
+      ? `Resolve ${plural(skillBundleSetupGaps, 'skill bundle')} with setup gaps from Agent Workspace -> Skills.`
       : '',
     hasLocalSkillBehavior && skillSetupGaps === 0 && skillBundleSetupGaps === 0 && snapshot.activeSkillCount === 0
-      ? 'Enable reviewed skills or bundles with /agent-skills enabled and /agent-skills bundle enabled.'
+      ? 'Enable reviewed skills or bundles from Agent Workspace -> Skills.'
       : '',
     snapshot.localRoutineCount === 0
-      ? 'Create repeatable workflows with /routines create; promote schedules only with explicit confirmation.'
+      ? 'Create repeatable workflows from Agent Workspace -> Routines; promote schedules only with explicit confirmation.'
       : routineSetupGaps > 0
-        ? `Resolve ${plural(routineSetupGaps, 'routine')} with setup gaps from /agent routines.`
+        ? `Resolve ${plural(routineSetupGaps, 'routine')} with setup gaps from Agent Workspace -> Routines.`
         : snapshot.enabledRoutineCount === 0
-        ? 'Enable reviewed routines with /routines enable.'
+        ? 'Enable reviewed routines from Agent Workspace -> Routines.'
         : '',
     channelSetupGaps > 0
-      ? `Review ${plural(channelSetupGaps, 'enabled channel')} needing setup from /agent channels.`
+      ? `Review ${plural(channelSetupGaps, 'enabled channel')} needing setup from Agent Workspace -> Channels.`
       : '',
     voiceSetupNeedsReview
-      ? 'Review voice setup with /agent voice-media before relying on spoken replies.'
+      ? 'Review voice setup from Agent Workspace -> Voice & Media before relying on spoken replies.'
       : '',
     mediaSetupNeedsReview
-      ? 'Review media provider setup with /agent voice-media before relying on image or media workflows.'
+      ? 'Review media provider setup from Agent Workspace -> Voice & Media before relying on image or media workflows.'
       : '',
-    'Use /agent knowledge for Agent Knowledge status, search, and explicit ingest forms.',
-    'Use /delegate only for explicit build, fix, implementation, or review handoff to GoodVibes TUI.',
+    'Use Agent Workspace -> Knowledge for Agent Knowledge status, search, and explicit ingest forms.',
+    'Use Agent Workspace -> Build Delegation only for explicit build, fix, implementation, or review handoff to GoodVibes TUI.',
   ].filter((line): line is string => line.length > 0);
 
   return [
     'Agent Briefing',
     `  chat route: ${snapshot.provider} / ${snapshot.modelDisplayName}`,
-    `  session: ${snapshot.sessionId}`,
-    `  profile: ${snapshot.activeRuntimeProfile}`,
-    `  policy: ${snapshot.executionPolicy}; WRFC ${snapshot.wrfcPolicy}`,
-    `  knowledge: ${snapshot.knowledgeRoute} (${snapshot.knowledgeIsolation}; no fallback)`,
+    `  session ${snapshot.sessionId}`,
+    `  profile ${snapshot.activeRuntimeProfile}`,
+    `  policy ${snapshot.executionPolicy}; delegated review ${snapshot.delegatedReviewPolicy}`,
+    `  knowledge ${snapshot.knowledgeRoute} (${snapshot.knowledgeIsolation}; no fallback)`,
     '',
     'Readiness',
-    `  setup: ${setupReady}/${snapshot.setupChecklist.length} ready; ${setupRecommended} recommended; ${setupBlocked} blocked`,
+    `  setup ${setupReady}/${snapshot.setupChecklist.length} ready; ${setupRecommended} recommended; ${setupBlocked} blocked`,
     `  local memory: ${plural(snapshot.localMemoryCount, 'record')}; prompt-active ${snapshot.localMemoryPromptActiveCount}; review queue ${snapshot.localMemoryReviewQueueCount}`,
+    `  scratchpad notes: ${plural(snapshot.localNoteCount, 'note')}; review queue ${snapshot.localNoteReviewQueueCount}`,
     `  personas: ${plural(snapshot.localPersonaCount, 'persona')}; active ${snapshot.activePersonaName}`,
     `  skills: ${snapshot.enabledSkillCount}/${snapshot.localSkillCount} enabled; bundles ${snapshot.enabledSkillBundleCount}/${snapshot.localSkillBundleCount}; active ${snapshot.activeSkillCount}; setup gaps ${skillSetupGaps} skill, ${skillBundleSetupGaps} bundle`,
     `  routines: ${snapshot.enabledRoutineCount}/${snapshot.localRoutineCount} enabled; setup gaps ${routineSetupGaps}`,
     `  channels: ${readyChannels}/${snapshot.channels.length} ready; ${enabledChannels} enabled; setup gaps ${channelSetupGaps}`,
     `  voice/media: ${snapshot.voiceProviderCount} voice, ${snapshot.mediaProviderCount} media; browser tools ${snapshot.voiceMediaReadiness.browserToolState}`,
     formatWorkPlanLine(workPlan.total, workPlan.counts),
-    `  schedules: ${enabledJobs}/${jobs.length} visible jobs enabled`,
+    `  schedules ${enabledJobs}/${jobs.length} visible jobs enabled`,
     '',
     'Next Actions',
     ...nextActions.map((line) => `  - ${line}`),