@pellux/goodvibes-agent 0.1.117 → 1.0.1

package/src/agent/persona-registry.ts

@@ -2,6 +2,7 @@ import { existsSync, mkdirSync, readFileSync, renameSync, writeFileSync } from '
 import { dirname } from 'node:path';
 import type { ShellPathService } from '@/runtime/index.ts';
 import { GOODVIBES_AGENT_SURFACE_ROOT } from '../config/surface.ts';
+import { formatAgentRecordReviewState } from './record-labels.ts';
 
 export type AgentPersonaSource = 'user' | 'agent' | 'imported' | 'system';
 export type AgentPersonaReviewState = 'fresh' | 'reviewed' | 'stale';
@@ -106,9 +107,9 @@ function containsSecretLikeText(text: string): boolean {
   return SECRET_PATTERNS.some((pattern) => pattern.test(text));
 }
 
-export function assertNoSecretLikeText(fields: readonly string[]): void {
+export function assertNoSecretLikeText(fields: readonly string[], ownerLabel = 'Agent local records'): void {
   if (fields.some((field) => containsSecretLikeText(field))) {
-    throw new Error('Personas cannot store secret-looking values. Store a secret reference or remove the sensitive text.');
+    throw new Error(`${ownerLabel} cannot store secret-looking values. Store a secret reference or remove the sensitive text.`);
   }
 }
 
@@ -213,9 +214,9 @@ export class AgentPersonaRegistry {
     const description = input.description.trim();
     const body = input.body.trim();
     this.validateRequired(name, description, body);
-    assertNoSecretLikeText([name, description, body, ...(input.tags ?? []), ...(input.triggers ?? [])]);
+    assertNoSecretLikeText([name, description, body, ...(input.tags ?? []), ...(input.triggers ?? [])], 'Personas');
     const duplicate = store.personas.find((persona) => persona.name.toLowerCase() === name.toLowerCase());
-    if (duplicate) throw new Error(`Persona already exists: ${duplicate.id}`);
+    if (duplicate) throw new Error(`Persona already exists ${duplicate.id}`);
     const timestamp = nowIso();
     const persona: AgentPersonaRecord = {
       id: this.nextId(name, store.personas),
@@ -237,14 +238,14 @@ export class AgentPersonaRegistry {
   public update(idOrName: string, input: AgentPersonaUpdateInput): AgentPersonaRecord {
     const store = this.readStore();
     const existing = this.findInStore(store, idOrName);
-    if (!existing) throw new Error(`Unknown persona: ${idOrName}`);
+    if (!existing) throw new Error(`Unknown persona ${idOrName}`);
     const name = input.name === undefined ? existing.name : normalizeName(input.name);
     const description = input.description === undefined ? existing.description : input.description.trim();
     const body = input.body === undefined ? existing.body : input.body.trim();
     this.validateRequired(name, description, body);
-    assertNoSecretLikeText([name, description, body, ...(input.tags ?? []), ...(input.triggers ?? [])]);
+    assertNoSecretLikeText([name, description, body, ...(input.tags ?? []), ...(input.triggers ?? [])], 'Personas');
     const duplicate = store.personas.find((persona) => persona.id !== existing.id && persona.name.toLowerCase() === name.toLowerCase());
-    if (duplicate) throw new Error(`Persona already exists: ${duplicate.id}`);
+    if (duplicate) throw new Error(`Persona already exists ${duplicate.id}`);
     const updated: AgentPersonaRecord = {
       ...existing,
       name,
@@ -268,7 +269,7 @@ export class AgentPersonaRegistry {
   public setActive(idOrName: string): AgentPersonaRecord {
     const store = this.readStore();
     const persona = this.findInStore(store, idOrName);
-    if (!persona) throw new Error(`Unknown persona: ${idOrName}`);
+    if (!persona) throw new Error(`Unknown persona ${idOrName}`);
     this.writeStore({ ...store, activePersonaId: persona.id });
     return persona;
   }
@@ -281,7 +282,7 @@ export class AgentPersonaRegistry {
   public markReviewed(idOrName: string): AgentPersonaRecord {
     const store = this.readStore();
     const existing = this.findInStore(store, idOrName);
-    if (!existing) throw new Error(`Unknown persona: ${idOrName}`);
+    if (!existing) throw new Error(`Unknown persona ${idOrName}`);
     const updated: AgentPersonaRecord = {
       ...existing,
       reviewState: 'reviewed',
@@ -299,7 +300,7 @@ export class AgentPersonaRegistry {
   public markStale(idOrName: string, reason: string): AgentPersonaRecord {
     const store = this.readStore();
     const existing = this.findInStore(store, idOrName);
-    if (!existing) throw new Error(`Unknown persona: ${idOrName}`);
+    if (!existing) throw new Error(`Unknown persona ${idOrName}`);
     const updated: AgentPersonaRecord = {
       ...existing,
       reviewState: 'stale',
@@ -316,7 +317,7 @@ export class AgentPersonaRegistry {
   public deletePersona(idOrName: string): AgentPersonaRecord {
     const store = this.readStore();
     const existing = this.findInStore(store, idOrName);
-    if (!existing) throw new Error(`Unknown persona: ${idOrName}`);
+    if (!existing) throw new Error(`Unknown persona ${idOrName}`);
     this.writeStore({
       ...store,
       activePersonaId: store.activePersonaId === existing.id ? null : store.activePersonaId,
@@ -353,7 +354,7 @@ export class AgentPersonaRegistry {
     try {
       return parseStore(readFileSync(this.storePath, 'utf-8'));
     } catch (error) {
-      throw new Error(`Could not read Agent persona store: ${error instanceof Error ? error.message : String(error)}`);
+      throw new Error(`Could not read Agent persona store ${error instanceof Error ? error.message : String(error)}`);
     }
   }
 
@@ -372,10 +373,10 @@ export function buildActivePersonaPrompt(shellPaths: ShellPathService): string |
     '## Active GoodVibes Agent Persona',
     `Name: ${active.name}`,
     `Description: ${active.description}`,
-    `Review state: ${active.reviewState}`,
+    `Review: ${formatAgentRecordReviewState(active.reviewState)}`,
     '',
     active.body,
     '',
-    'Apply this persona inside the same serial assistant conversation. Do not launch local workers because a persona is active.',
+    'Apply this persona inside the same serial assistant conversation. Do not spawn separate Agent jobs because a persona is active.',
   ].join('\n');
 }

package/src/agent/record-labels.ts

@@ -0,0 +1,107 @@
+const SOURCE_LABELS = {
+  user: 'User',
+  agent: 'Agent',
+  imported: 'Imported',
+  system: 'Built-in',
+  builtin: 'Built-in',
+  local: 'Local',
+  'agent-memory': 'Agent memory',
+} as const satisfies Readonly<Record<string, string>>;
+
+const USER_PROVENANCE_LABELS = new Set<string>([
+  'Workspace',
+  'Workspace capture',
+  'Command',
+  'Manual',
+  'Main conversation',
+]);
+
+const REVIEW_STATE_LABELS = {
+  fresh: 'Needs review',
+  reviewed: 'Reviewed',
+  stale: 'Needs refresh',
+  contradicted: 'Contradicted',
+} as const satisfies Readonly<Record<string, string>>;
+
+type SourceLabelKey = keyof typeof SOURCE_LABELS;
+type ReviewStateLabelKey = keyof typeof REVIEW_STATE_LABELS;
+
+export interface AgentRecordReference {
+  readonly kind: string;
+  readonly ref: string;
+  readonly label?: string;
+}
+
+function titleCaseWords(value: string): string {
+  return value
+    .trim()
+    .replace(/[_-]+/g, ' ')
+    .replace(/\s+/g, ' ')
+    .replace(/\b\w/g, (match) => match.toUpperCase());
+}
+
+export function formatAgentRecordSource(source: string | undefined): string {
+  const normalized = (source ?? '').trim().toLowerCase();
+  if (!normalized) return 'Unknown';
+  if (normalized in SOURCE_LABELS) return SOURCE_LABELS[normalized as SourceLabelKey];
+  return titleCaseWords(source ?? '');
+}
+
+export function formatAgentRecordReviewState(reviewState: string | undefined): string {
+  const normalized = (reviewState ?? '').trim().toLowerCase();
+  if (!normalized) return REVIEW_STATE_LABELS.fresh;
+  if (normalized in REVIEW_STATE_LABELS) return REVIEW_STATE_LABELS[normalized as ReviewStateLabelKey];
+  return titleCaseWords(reviewState ?? '');
+}
+
+export function formatAgentRecordProvenance(provenance: string | undefined): string {
+  const raw = (provenance ?? '').trim();
+  const normalized = raw.toLowerCase();
+  if (!raw) return '';
+  if (normalized === 'agent-workspace') return 'Workspace';
+  if (normalized === 'agent-workspace-learned-behavior') return 'Workspace capture';
+  if (normalized === 'workspace capture') return 'Workspace capture';
+  if (normalized === 'slash-command') return 'Command';
+  if (normalized === 'agent-memory') return 'Agent memory';
+  if (normalized === 'manual') return 'Manual';
+  if (normalized === 'main-conversation') return 'Main conversation';
+  if (normalized === 'plan-command') return 'Planning command';
+  if (normalized === 'agent-media-generation') return 'Media generation';
+  if (normalized === 'agent-channel-send') return 'Channel delivery';
+  if (normalized === 'project-memory') return 'Project memory';
+
+  const discovered = raw.match(/^discovered:([^:]+):(.+)$/i);
+  if (discovered) {
+    return `Imported file (${discovered[1]}): ${discovered[2]}`;
+  }
+
+  const starter = raw.match(/^goodvibes-agent starter:(.+)$/i);
+  if (starter) {
+    return `Starter profile: ${starter[1]}`;
+  }
+
+  return titleCaseWords(raw);
+}
+
+export function formatAgentRecordOrigin(source: string | undefined, provenance: string | undefined): string {
+  const sourceLabel = formatAgentRecordSource(source);
+  const provenanceLabel = formatAgentRecordProvenance(provenance);
+  if (!provenanceLabel || provenanceLabel === sourceLabel) return sourceLabel;
+  if (sourceLabel === 'User' && USER_PROVENANCE_LABELS.has(provenanceLabel)) {
+    return provenanceLabel;
+  }
+  if (sourceLabel === 'Imported' && provenanceLabel.startsWith('Imported file')) {
+    return provenanceLabel;
+  }
+  return `${sourceLabel} (${provenanceLabel})`;
+}
+
+export function formatAgentRecordReference(reference: AgentRecordReference): string {
+  const kind = titleCaseWords(reference.kind);
+  const label = reference.label?.trim();
+  return `${kind}: ${reference.ref}${label ? ` (${label})` : ''}`;
+}
+
+export function formatAgentRecordReferences(references: readonly AgentRecordReference[]): string {
+  return references.map(formatAgentRecordReference).join(', ');
+}

package/src/agent/reminder-schedule-format.ts

@@ -22,20 +22,29 @@ function scheduleValue(preview: ReminderSchedulePreview): string {
   return String(preview.payload.at);
 }
 
+function formatReminderScheduleFailureKind(kind: string): string {
+  if (kind === 'auth_required') return 'authorization required';
+  if (kind === 'connected_host_unavailable') return 'connected host unavailable';
+  if (kind === 'connected_host_route_unavailable') return 'connected host route unavailable';
+  if (kind === 'connected_host_error') return 'connected host error';
+  if (kind === 'version_mismatch') return 'version mismatch';
+  return kind.replace(/[_-]+/g, ' ');
+}
+
 export function formatReminderSchedulePreview(preview: ReminderSchedulePreview): string {
   const delivery = preview.payload.delivery;
   const deliveryTargetCount = delivery?.targets.length ?? 0;
   return [
     'GoodVibes schedule preview for Agent reminder',
-    `  route: ${preview.method} ${preview.route}`,
-    `  name: ${String(preview.payload.name ?? '(service default)')}`,
-    `  reminder: ${preview.message}`,
-    `  schedule: ${preview.payload.kind} ${scheduleValue(preview)}`,
-    `  enabled: ${preview.payload.enabled === false ? 'no' : 'yes'}`,
-    `  delivery: ${delivery?.mode ?? 'none'}${deliveryTargetCount > 0 ? ` (${deliveryTargetCount} target${deliveryTargetCount === 1 ? '' : 's'})` : ''}`,
-    '  target: connected GoodVibes host/main conversation route',
-    '  policy: reminder delivery only; isolated Agent Knowledge only; no default wiki/non-Agent fallback',
-    '  next: rerun with --yes to create this connected reminder schedule',
+    `  route ${preview.method} ${preview.route}`,
+    `  name ${String(preview.payload.name ?? '(service default)')}`,
+    `  reminder ${preview.message}`,
+    `  schedule ${preview.payload.kind} ${scheduleValue(preview)}`,
+    `  enabled ${preview.payload.enabled === false ? 'no' : 'yes'}`,
+    `  delivery ${delivery?.mode ?? 'none'}${deliveryTargetCount > 0 ? ` (${deliveryTargetCount} target${deliveryTargetCount === 1 ? '' : 's'})` : ''}`,
+    '  target connected GoodVibes host/main conversation route',
+    '  policy reminder delivery only; isolated Agent Knowledge only; no default knowledge/non-Agent fallback',
+    '  next rerun with --yes to create this connected reminder schedule',
   ].join('\n');
 }
 
@@ -45,31 +54,31 @@ export function formatReminderScheduleSuccess(result: ReminderScheduleSuccess):
   const status = readString(record, 'status') ?? (record.enabled === false ? 'paused' : 'enabled');
   return [
     'Created GoodVibes schedule for Agent reminder',
-    `  schedule: ${id}`,
-    `  status: ${status}`,
-    `  route: ${result.kind} ${result.route}`,
-    `  reminder: ${result.message}`,
-    '  next: inspect with /schedule list',
+    `  schedule ${id}`,
+    `  status ${status}`,
+    `  route ${result.kind} ${result.route}`,
+    `  reminder ${result.message}`,
+    '  next inspect with /schedule list',
   ].join('\n');
 }
 
 export function formatReminderScheduleFailure(failure: ReminderScheduleFailure): string {
   return [
-    `GoodVibes reminder schedule error: ${failure.kind}`,
+    `GoodVibes reminder schedule error ${formatReminderScheduleFailureKind(failure.kind)}`,
     `  ${failure.error}`,
-    failure.baseUrl ? `  host: ${failure.baseUrl}` : null,
-    `  route: ${REMINDER_SCHEDULE_METHOD} ${failure.route}`,
-    failure.kind === 'version_mismatch' && failure.daemonVersion && failure.expectedSdkVersion
-      ? `  versions: host=${failure.daemonVersion} expected=${failure.expectedSdkVersion}`
+    failure.baseUrl ? `  connected host ${failure.baseUrl}` : null,
+    `  route ${REMINDER_SCHEDULE_METHOD} ${failure.route}`,
+    failure.kind === 'version_mismatch' && failure.connectedHostVersion && failure.expectedSdkVersion
+      ? `  versions connected host ${failure.connectedHostVersion}; expected ${failure.expectedSdkVersion}`
       : null,
     failure.kind === 'auth_required'
-      ? '  next: pair/authenticate with the connected GoodVibes host, then retry with --yes.'
+      ? '  next pair/authenticate with the connected GoodVibes host, then retry with --yes.'
       : null,
-    failure.kind === 'daemon_unavailable'
-      ? '  next: make the connected GoodVibes host available outside Agent, then retry.'
+    failure.kind === 'connected_host_unavailable'
+      ? '  next make the connected GoodVibes host available outside Agent, then retry.'
       : null,
-    failure.kind === 'version_mismatch' || failure.kind === 'daemon_route_unavailable'
-      ? '  next: update the connected GoodVibes host so public schedules.create is available.'
+    failure.kind === 'version_mismatch' || failure.kind === 'connected_host_route_unavailable'
+      ? '  next update the connected GoodVibes host so public schedules.create is available.'
       : null,
   ].filter((line): line is string => Boolean(line)).join('\n');
 }

package/src/agent/reminder-schedule.ts

@@ -4,11 +4,11 @@ import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import { getModelIdFromProviderModel, getProviderIdFromModel } from '../config/provider-model.ts';
 import { SDK_VERSION } from '../version.ts';
 import {
-  resolveAgentDaemonConnection,
+  resolveAgentConnectedHostConnection,
   ROUTINE_SCHEDULE_METHOD,
   ROUTINE_SCHEDULE_ROUTE,
-  type AgentDaemonConfigReader,
-  type AgentDaemonConnection,
+  type AgentConnectedHostConfigReader,
+  type AgentConnectedHostConnection,
   type RoutineScheduleDeliveryKind,
   type RoutineScheduleDeliverySurfaceKind,
   type RoutineScheduleDeliveryTargetSpec,
@@ -58,14 +58,14 @@ export interface ReminderScheduleFailure {
   readonly kind:
     | 'confirmation_required'
     | 'auth_required'
-    | 'daemon_unavailable'
+    | 'connected_host_unavailable'
     | 'version_mismatch'
-    | 'daemon_route_unavailable'
-    | 'daemon_error';
+    | 'connected_host_route_unavailable'
+    | 'connected_host_error';
   readonly error: string;
   readonly route: typeof REMINDER_SCHEDULE_ROUTE;
   readonly baseUrl?: string;
-  readonly daemonVersion?: string;
+  readonly connectedHostVersion?: string;
   readonly expectedSdkVersion?: string;
 }
 
@@ -82,6 +82,7 @@ const DELIVERY_SURFACE_KINDS: readonly RoutineScheduleDeliverySurfaceKind[] = [
   'google-chat',
   'signal',
   'whatsapp',
+  'telephony',
   'imessage',
   'msteams',
   'bluebubbles',
@@ -186,7 +187,7 @@ function deliveryModeFromTargets(targets: readonly RoutineScheduleDeliveryTarget
 function toDeliveryTargetInput(target: RoutineScheduleDeliveryTargetSpec): ScheduleDeliveryTargetInput {
   return {
     kind: target.kind,
-    surfaceKind: target.surfaceKind,
+    surfaceKind: target.surfaceKind as ScheduleDeliveryTargetInput['surfaceKind'],
     address: target.address,
     routeId: target.routeId,
     label: target.label,
@@ -312,7 +313,7 @@ export function parseReminderScheduleArgs(args: readonly string[]): ParsedRemind
       continue;
     }
     if (raw.startsWith('--')) {
-      errors.push(`Unknown option: ${raw}`);
+      errors.push(`Unknown option ${raw}`);
       continue;
     }
     positional.push(raw);
@@ -335,9 +336,9 @@ export function buildReminderSchedulePrompt(message: string): string {
     '',
     'Operator policy:',
     '- Treat this as a reminder delivery, not an autonomous hidden workflow.',
-    '- Use isolated Agent Knowledge routes only when lookup is needed; never use default Knowledge/Wiki or non-Agent knowledge spaces as fallback.',
+    '- Use isolated Agent Knowledge routes only when lookup is needed; never use default knowledge or non-Agent knowledge spaces as fallback.',
     '- Do not perform destructive, costly, externally visible, or secret-handling actions from this reminder without explicit approval.',
-    '- Do not request WRFC from a reminder. If build/fix/review work is needed, ask the user to delegate it explicitly to GoodVibes TUI.',
+    '- Do not request GoodVibes TUI delegation from a reminder. If build/fix/review work is needed, ask the user to delegate it explicitly to GoodVibes TUI.',
     '- Keep the reminder concise and state any next action the user can take.',
   ].join('\n');
 }
@@ -406,7 +407,7 @@ export function buildReminderSchedulePreview(parsed: ParsedReminderScheduleArgs)
   };
 }
 
-async function fetchDaemonStatus(connection: AgentDaemonConnection): Promise<{
+async function fetchConnectedHostStatus(connection: AgentConnectedHostConnection): Promise<{
   readonly ok: boolean;
   readonly status: number;
   readonly body: unknown;
@@ -430,7 +431,7 @@ async function fetchDaemonStatus(connection: AgentDaemonConnection): Promise<{
 
 async function classifyReminderScheduleError(
   error: unknown,
-  connection: AgentDaemonConnection,
+  connection: AgentConnectedHostConnection,
 ): Promise<ReminderScheduleFailure> {
   const message = summarizeError(error);
   const lower = message.toLowerCase();
@@ -438,30 +439,30 @@ async function classifyReminderScheduleError(
     return { ok: false, kind: 'auth_required', error: message, route: REMINDER_SCHEDULE_ROUTE, baseUrl: connection.baseUrl };
   }
   if (lower.includes('404') || lower.includes('not found')) {
-    const daemon = await fetchDaemonStatus(connection);
-    const record = isRecord(daemon.body) ? daemon.body : {};
-    const daemonVersion = readString(record, 'version') ?? 'unknown';
-    if (daemon.ok && daemonVersion !== SDK_VERSION) {
+    const connectedHost = await fetchConnectedHostStatus(connection);
+    const record = isRecord(connectedHost.body) ? connectedHost.body : {};
+    const connectedHostVersion = readString(record, 'version') ?? 'unknown';
+    if (connectedHost.ok && connectedHostVersion !== SDK_VERSION) {
       return {
         ok: false,
         kind: 'version_mismatch',
-        error: `Connected GoodVibes service SDK version ${daemonVersion} does not match Agent SDK pin ${SDK_VERSION}; schedules.create is unavailable.`,
+        error: `Connected GoodVibes host SDK version ${connectedHostVersion} does not match Agent SDK pin ${SDK_VERSION}; schedules.create is unavailable.`,
         route: REMINDER_SCHEDULE_ROUTE,
         baseUrl: connection.baseUrl,
-        daemonVersion,
+        connectedHostVersion,
         expectedSdkVersion: SDK_VERSION,
       };
     }
-    return { ok: false, kind: 'daemon_route_unavailable', error: message, route: REMINDER_SCHEDULE_ROUTE, baseUrl: connection.baseUrl };
+    return { ok: false, kind: 'connected_host_route_unavailable', error: message, route: REMINDER_SCHEDULE_ROUTE, baseUrl: connection.baseUrl };
   }
   if (lower.includes('fetch') || lower.includes('connect') || lower.includes('econnrefused')) {
-    return { ok: false, kind: 'daemon_unavailable', error: message, route: REMINDER_SCHEDULE_ROUTE, baseUrl: connection.baseUrl };
+    return { ok: false, kind: 'connected_host_unavailable', error: message, route: REMINDER_SCHEDULE_ROUTE, baseUrl: connection.baseUrl };
   }
-  return { ok: false, kind: 'daemon_error', error: message, route: REMINDER_SCHEDULE_ROUTE, baseUrl: connection.baseUrl };
+  return { ok: false, kind: 'connected_host_error', error: message, route: REMINDER_SCHEDULE_ROUTE, baseUrl: connection.baseUrl };
 }
 
 export async function createReminderSchedule(
-  connection: AgentDaemonConnection,
+  connection: AgentConnectedHostConnection,
   preview: ReminderSchedulePreview,
 ): Promise<ReminderScheduleResult> {
   if (!connection.token) {
@@ -489,6 +490,6 @@ export async function createReminderSchedule(
   }
 }
 
-export function resolveReminderDaemonConnection(configManager: AgentDaemonConfigReader, homeDirectory: string): AgentDaemonConnection {
-  return resolveAgentDaemonConnection(configManager, homeDirectory);
+export function resolveReminderConnectedHostConnection(configManager: AgentConnectedHostConfigReader, homeDirectory: string): AgentConnectedHostConnection {
+  return resolveAgentConnectedHostConnection(configManager, homeDirectory);
 }

package/src/agent/routine-registry.ts

@@ -3,6 +3,7 @@ import { dirname } from 'node:path';
 import type { ShellPathService } from '@/runtime/index.ts';
 import { GOODVIBES_AGENT_SURFACE_ROOT } from '../config/surface.ts';
 import { assertNoSecretLikeText } from './persona-registry.ts';
+import { formatAgentRecordReviewState } from './record-labels.ts';
 import {
   evaluateAgentSkillReadiness,
   formatAgentSkillRequirement,
@@ -228,9 +229,9 @@ export class AgentRoutineRegistry {
     const steps = input.steps.trim();
     this.validateRequired(name, description, steps);
     const requirements = normalizeAgentSkillRequirements(input.requirements);
-    assertNoSecretLikeText([name, description, steps, ...(input.tags ?? []), ...(input.triggers ?? []), ...requirements.flatMap((requirement) => [requirement.name, requirement.description ?? ''])]);
+    assertNoSecretLikeText([name, description, steps, ...(input.tags ?? []), ...(input.triggers ?? []), ...requirements.flatMap((requirement) => [requirement.name, requirement.description ?? ''])], 'Routines');
     const duplicate = store.routines.find((routine) => routine.name.toLowerCase() === name.toLowerCase());
-    if (duplicate) throw new Error(`Routine already exists: ${duplicate.id}`);
+    if (duplicate) throw new Error(`Routine already exists ${duplicate.id}`);
     const timestamp = nowIso();
     const routine: AgentRoutineRecord = {
       id: this.nextId(name, store.routines),
@@ -255,15 +256,15 @@ export class AgentRoutineRegistry {
   public update(idOrName: string, input: AgentRoutineUpdateInput): AgentRoutineRecord {
     const store = this.readStore();
     const existing = this.findInStore(store, idOrName);
-    if (!existing) throw new Error(`Unknown routine: ${idOrName}`);
+    if (!existing) throw new Error(`Unknown routine ${idOrName}`);
     const name = input.name === undefined ? existing.name : normalizeName(input.name);
     const description = input.description === undefined ? existing.description : input.description.trim();
     const steps = input.steps === undefined ? existing.steps : input.steps.trim();
     this.validateRequired(name, description, steps);
     const requirements = input.requirements === undefined ? existing.requirements : normalizeAgentSkillRequirements(input.requirements);
-    assertNoSecretLikeText([name, description, steps, ...(input.tags ?? []), ...(input.triggers ?? []), ...requirements.flatMap((requirement) => [requirement.name, requirement.description ?? ''])]);
+    assertNoSecretLikeText([name, description, steps, ...(input.tags ?? []), ...(input.triggers ?? []), ...requirements.flatMap((requirement) => [requirement.name, requirement.description ?? ''])], 'Routines');
     const duplicate = store.routines.find((routine) => routine.id !== existing.id && routine.name.toLowerCase() === name.toLowerCase());
-    if (duplicate) throw new Error(`Routine already exists: ${duplicate.id}`);
+    if (duplicate) throw new Error(`Routine already exists ${duplicate.id}`);
     const updated: AgentRoutineRecord = {
       ...existing,
       name,
@@ -288,7 +289,7 @@ export class AgentRoutineRegistry {
   public setEnabled(idOrName: string, enabled: boolean): AgentRoutineRecord {
     const store = this.readStore();
     const existing = this.findInStore(store, idOrName);
-    if (!existing) throw new Error(`Unknown routine: ${idOrName}`);
+    if (!existing) throw new Error(`Unknown routine ${idOrName}`);
     const updated: AgentRoutineRecord = { ...existing, enabled, updatedAt: nowIso() };
     this.writeStore({
       ...store,
@@ -300,7 +301,7 @@ export class AgentRoutineRegistry {
   public markStarted(idOrName: string): AgentRoutineRecord {
     const store = this.readStore();
     const existing = this.findInStore(store, idOrName);
-    if (!existing) throw new Error(`Unknown routine: ${idOrName}`);
+    if (!existing) throw new Error(`Unknown routine ${idOrName}`);
     const timestamp = nowIso();
     const updated: AgentRoutineRecord = {
       ...existing,
@@ -318,7 +319,7 @@ export class AgentRoutineRegistry {
   public markReviewed(idOrName: string): AgentRoutineRecord {
     const store = this.readStore();
     const existing = this.findInStore(store, idOrName);
-    if (!existing) throw new Error(`Unknown routine: ${idOrName}`);
+    if (!existing) throw new Error(`Unknown routine ${idOrName}`);
     const updated: AgentRoutineRecord = {
       ...existing,
       reviewState: 'reviewed',
@@ -336,7 +337,7 @@ export class AgentRoutineRegistry {
   public markStale(idOrName: string, reason: string): AgentRoutineRecord {
     const store = this.readStore();
     const existing = this.findInStore(store, idOrName);
-    if (!existing) throw new Error(`Unknown routine: ${idOrName}`);
+    if (!existing) throw new Error(`Unknown routine ${idOrName}`);
     const updated: AgentRoutineRecord = {
       ...existing,
       reviewState: 'stale',
@@ -353,7 +354,7 @@ export class AgentRoutineRegistry {
   public deleteRoutine(idOrName: string): AgentRoutineRecord {
     const store = this.readStore();
     const existing = this.findInStore(store, idOrName);
-    if (!existing) throw new Error(`Unknown routine: ${idOrName}`);
+    if (!existing) throw new Error(`Unknown routine ${idOrName}`);
     this.writeStore({
       ...store,
       routines: store.routines.filter((routine) => routine.id !== existing.id),
@@ -389,7 +390,7 @@ export class AgentRoutineRegistry {
     try {
       return parseStore(readFileSync(this.storePath, 'utf-8'));
     } catch (error) {
-      throw new Error(`Could not read Agent routine store: ${error instanceof Error ? error.message : String(error)}`);
+      throw new Error(`Could not read Agent routine store ${error instanceof Error ? error.message : String(error)}`);
     }
   }
 
@@ -418,7 +419,7 @@ export function buildEnabledRoutinesPrompt(shellPaths: ShellPathService): string
     ...enabled.slice(0, 8).flatMap((routine) => [
       `### ${routine.name}`,
       `Description: ${routine.description}`,
-      `Review state: ${routine.reviewState}`,
+      `Review: ${formatAgentRecordReviewState(routine.reviewState)}`,
       `Triggers: ${routine.triggers.join(', ') || '(manual)'}`,
       `Readiness: ${evaluateAgentRoutineReadiness(routine).ready ? 'ready' : `missing ${evaluateAgentRoutineReadiness(routine).missing.map(formatAgentSkillRequirement).join(', ')}`}`,
       routine.steps,

package/src/agent/routine-schedule-args.ts

@@ -15,6 +15,7 @@ const DELIVERY_SURFACE_KINDS: readonly RoutineScheduleDeliverySurfaceKind[] = [
   'google-chat',
   'signal',
   'whatsapp',
+  'telephony',
   'imessage',
   'msteams',
   'bluebubbles',
@@ -204,7 +205,7 @@ export function parseRoutineSchedulePromotionArgs(args: readonly string[]): Pars
       continue;
     }
     if (raw.startsWith('--')) {
-      errors.push(`Unknown option: ${raw}`);
+      errors.push(`Unknown option ${raw}`);
       continue;
     }
     if (!routineId) {