@pellux/goodvibes-agent 0.1.117 → 1.0.1

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

@@ -9,6 +9,10 @@ import { BUILTIN_SECRET_PROVIDER_SOURCES, describeSecretRef, isSecretRefInput, r
 import { requireBookmarkManager, requireProviderApi, requireSecretsManager } from './runtime-services.ts';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import { requireYesFlag, stripYesFlag } from './confirmation.ts';
+import {
+  formatAgentMediaGenerationResult,
+  generateAgentMedia,
+} from '../../agent/media-generation.ts';
 
 function isGoodVibesSecretRefInput(value: string): boolean {
   const normalized = value.trim();
@@ -23,7 +27,7 @@ function isMalformedGoodVibesSecretRefInput(value: string): boolean {
 function toggleBlocks(typeFilter: string, collapsed: boolean, ctx: CommandContext): void {
   const VALID_TYPES = ['all', 'thinking', 'tool', 'code'] as const;
   if (!VALID_TYPES.includes(typeFilter as typeof VALID_TYPES[number])) {
-    ctx.print(`Unknown type: ${typeFilter}\nValid types: ${VALID_TYPES.join(', ')}`);
+    ctx.print(`Unknown type ${typeFilter}\nValid types ${VALID_TYPES.join(', ')}`);
     return;
   }
   const blockRegistry = ctx.session.conversationManager.getBlockRegistry();
@@ -49,6 +53,60 @@ function toggleBlocks(typeFilter: string, collapsed: boolean, ctx: CommandContex
   ctx.renderRequest();
 }
 
+interface MediaGenerateArgs {
+  readonly prompt: string;
+  readonly providerId?: string;
+  readonly modelId?: string;
+  readonly outputMimeType?: string;
+  readonly yes: boolean;
+  readonly errors: readonly string[];
+}
+
+function readFlagValue(args: readonly string[], index: number, flag: string, errors: string[]): string | null {
+  const value = args[index + 1];
+  if (!value || value.startsWith('--')) {
+    errors.push(`${flag} requires a value.`);
+    return null;
+  }
+  return value;
+}
+
+function parseMediaGenerateArgs(args: readonly string[]): MediaGenerateArgs {
+  const parsed = stripYesFlag([...args]);
+  const errors: string[] = [];
+  const promptParts: string[] = [];
+  let providerId: string | undefined;
+  let modelId: string | undefined;
+  let outputMimeType: string | undefined;
+  for (let index = 0; index < parsed.rest.length; index += 1) {
+    const arg = parsed.rest[index];
+    if (arg === '--provider') {
+      providerId = readFlagValue(parsed.rest, index, arg, errors) ?? providerId;
+      index += 1;
+    } else if (arg === '--model') {
+      modelId = readFlagValue(parsed.rest, index, arg, errors) ?? modelId;
+      index += 1;
+    } else if (arg === '--mime') {
+      outputMimeType = readFlagValue(parsed.rest, index, arg, errors) ?? outputMimeType;
+      index += 1;
+    } else if (arg?.startsWith('--')) {
+      errors.push(`Unknown media generation flag ${arg}`);
+    } else if (arg) {
+      promptParts.push(arg);
+    }
+  }
+  const prompt = promptParts.join(' ').trim();
+  if (!prompt) errors.push('Media generation prompt is required.');
+  return {
+    prompt,
+    ...(providerId ? { providerId } : {}),
+    ...(modelId ? { modelId } : {}),
+    ...(outputMimeType ? { outputMimeType } : {}),
+    yes: parsed.yes,
+    errors,
+  };
+}
+
 export function registerLocalRuntimeCommands(registry: CommandRegistry): void {
   registry.register({ name: 'expand', description: 'Expand blocks by type', usage: '[all|thinking|tool|code]', argsHint: '[all|thinking|tool|code]', handler(args, ctx) { toggleBlocks(args[0] || 'all', false, ctx); } });
   registry.register({ name: 'collapse', description: 'Collapse blocks by type', usage: '[all|thinking|tool|code]', argsHint: '[all|thinking|tool|code]', handler(args, ctx) { toggleBlocks(args[0] || 'all', true, ctx); } });
@@ -130,9 +188,12 @@ export function registerLocalRuntimeCommands(registry: CommandRegistry): void {
         }
         try {
           const resolved = await resolveSecretRef(refText, { resolveLocalSecret: (key) => mgr.get(key) });
-          ctx.print(`[secrets] ${describeSecretRef(refText)}: ${resolved.value ? 'resolved <redacted>' : 'missing'}`);
+          ctx.print([
+            `[secrets] ${describeSecretRef(refText)}`,
+            `  status ${resolved.value ? 'resolved <redacted>' : 'missing'}`,
+          ].join('\n'));
         } catch (error) {
-          ctx.print(`[secrets] ${describeSecretRef(refText)} failed: ${summarizeError(error)}`);
+          ctx.print(`[secrets] ${describeSecretRef(refText)} failed ${summarizeError(error)}`);
         }
         return;
       }
@@ -190,7 +251,10 @@ export function registerLocalRuntimeCommands(registry: CommandRegistry): void {
           scope: flags.has('--user') ? 'user' : flags.has('--project') ? 'project' : undefined,
           medium: flags.has('--plaintext') ? 'plaintext' : flags.has('--secure') ? 'secure' : undefined,
         });
-        ctx.print(`[secrets] Deleted: ${key}`);
+        ctx.print([
+          '[secrets] Deleted',
+          `  key ${key}`,
+        ].join('\n'));
         return;
       }
       ctx.print('[secrets] Usage: /secrets set <KEY> <value> [--user|--project] [--secure|--plaintext] --yes | link <KEY> <secret-ref> [--user|--project] [--secure|--plaintext] --yes | get <KEY> | test <secret-ref> | providers | list | delete <KEY> [--user|--project] [--secure|--plaintext] --yes');
@@ -212,36 +276,42 @@ export function registerLocalRuntimeCommands(registry: CommandRegistry): void {
       const promptText = args.slice(1).join(' ') || `Attached image: ${rawPath.split('/').pop() ?? rawPath}`;
       const projectRoot = ctx.workspace.shellPaths?.workingDirectory ?? ctx.platform.configManager.getWorkingDirectory();
       if (!projectRoot) {
-        ctx.print('Error: working directory is unavailable.');
+        ctx.print([
+          'Error',
+          '  message working directory is unavailable.',
+        ].join('\n'));
         return;
       }
       let resolvedPath: string;
       try {
         resolvedPath = resolveAndValidatePath(rawPath, projectRoot);
       } catch (err) {
-        ctx.print(`Error: ${summarizeError(err)}`);
+        ctx.print([
+          'Error',
+          `  message ${summarizeError(err)}`,
+        ].join('\n'));
         return;
       }
       if (!existsSync(resolvedPath)) {
-        ctx.print(`File not found: ${rawPath}`);
+        ctx.print(`File not found ${rawPath}`);
         return;
       }
       const ext = resolvedPath.slice(resolvedPath.lastIndexOf('.')).toLowerCase();
       const mediaType = ({ '.png': 'image/png', '.jpg': 'image/jpeg', '.jpeg': 'image/jpeg', '.webp': 'image/webp', '.gif': 'image/gif' } as Record<string, string>)[ext];
       if (!mediaType) {
-        ctx.print(`Unsupported image format: ${ext}\nSupported: .png, .jpg, .jpeg, .webp, .gif`);
+        ctx.print(`Unsupported image format ${ext}\nSupported .png, .jpg, .jpeg, .webp, .gif`);
         return;
       }
       const stat = statSync(resolvedPath);
       if (stat.size > 20 * 1024 * 1024) {
-        ctx.print(`Image too large (${(stat.size / 1024 / 1024).toFixed(1)}MB). Maximum: 20MB`);
+        ctx.print(`Image too large (${(stat.size / 1024 / 1024).toFixed(1)}MB). Maximum 20MB`);
         return;
       }
       let data: string;
       try {
         data = (await readFile(resolvedPath)).toString('base64');
       } catch (err) {
-        ctx.print(`Failed to read image: ${summarizeError(err)}`);
+        ctx.print(`Failed to read image ${summarizeError(err)}`);
         return;
       }
       const currentModel = await requireProviderApi(ctx).getCurrentModel();
@@ -253,6 +323,53 @@ export function registerLocalRuntimeCommands(registry: CommandRegistry): void {
     },
   });
 
+  registry.register({
+    name: 'media',
+    description: 'Inspect media providers or generate media through configured providers',
+    usage: 'providers | generate [--provider <id>] [--model <id>] [--mime <mime>] <prompt> --yes',
+    argsHint: '<providers|generate>',
+    async handler(args, ctx) {
+      const subcommand = args[0]?.toLowerCase() || 'providers';
+      const registry = ctx.platform.mediaProviderRegistry;
+      if (!registry) {
+        ctx.print('[media] Media providers are not available in this runtime.');
+        return;
+      }
+      if (subcommand === 'providers' || subcommand === 'status' || subcommand === 'list') {
+        const statuses = await registry.status();
+        ctx.print([
+          '[media] Providers:',
+          ...statuses.map((provider) => `  ${provider.id}: ${provider.state}${provider.configured ? '' : ' (setup needed)'} - ${provider.capabilities.join(', ')}${provider.detail ? ` - ${provider.detail}` : ''}`),
+        ].join('\n'));
+        return;
+      }
+      if (subcommand !== 'generate') {
+        ctx.print('[media] Usage: /media providers | /media generate [--provider <id>] [--model <id>] [--mime <mime>] <prompt> --yes');
+        return;
+      }
+      const artifactStore = ctx.platform.artifactStore;
+      if (!artifactStore) {
+        ctx.print('[media] Artifact storage is not available in this runtime.');
+        return;
+      }
+      const parsed = parseMediaGenerateArgs(args.slice(1));
+      if (parsed.errors.length > 0) {
+        ctx.print(`[media] ${parsed.errors.join('\n[media] ')}`);
+        return;
+      }
+      if (!parsed.yes) {
+        requireYesFlag(ctx, `generate media for "${parsed.prompt}"`, '/media generate [--provider <id>] [--model <id>] [--mime <mime>] <prompt> --yes');
+        return;
+      }
+      try {
+        const result = await generateAgentMedia(registry, artifactStore, parsed);
+        ctx.print(formatAgentMediaGenerationResult(result));
+      } catch (error) {
+        ctx.print(`[media] Generation failed ${summarizeError(error)}`);
+      }
+    },
+  });
+
   registry.register({
     name: 'refresh-models',
     description: 'Refresh model catalog, benchmarks, and token limits',
@@ -267,15 +384,17 @@ export function registerLocalRuntimeCommands(registry: CommandRegistry): void {
         catalogOk = true;
         ctx.print(`Model catalog refreshed: ${catalog.modelCount} models from ${catalog.providerCount} providers`);
       } catch (e) {
-        ctx.print(`Catalog refresh failed: ${summarizeError(e)}`);
+        ctx.print(`Catalog refresh failed ${summarizeError(e)}`);
       }
       ctx.print('Refreshing benchmarks...');
       try {
         const benchmarkCount = await providerApi.refreshBenchmarks();
         benchmarksOk = true;
-        ctx.print(`Benchmarks refreshed${benchmarkCount > 0 ? `: ${benchmarkCount} model records available.` : '.'}`);
+        ctx.print(benchmarkCount > 0
+          ? `Benchmarks refreshed: ${benchmarkCount} model records available.`
+          : 'Benchmarks refreshed.');
       } catch (e) {
-        ctx.print(`Benchmarks refresh failed: ${summarizeError(e)}`);
+        ctx.print(`Benchmarks refresh failed ${summarizeError(e)}`);
       }
       ctx.print('Refreshing token limits...');
       try {
@@ -283,7 +402,7 @@ export function registerLocalRuntimeCommands(registry: CommandRegistry): void {
         limitsOk = true;
         ctx.print(`Token limits refreshed: ${count} models updated.`);
       } catch (e) {
-        ctx.print(`Token limits refresh failed: ${summarizeError(e)}`);
+        ctx.print(`Token limits refresh failed ${summarizeError(e)}`);
       }
       if (!catalogOk || !benchmarksOk || !limitsOk) ctx.print('Some refreshes failed — see messages above.');
     },
@@ -308,14 +427,17 @@ export function registerLocalRuntimeCommands(registry: CommandRegistry): void {
       }
       const favorites = await providerApi.getFavorites();
       if (favorites.pinned.some((entry) => entry.modelId === modelId || entry.registryKey === modelId)) {
-        ctx.print(`Model already pinned: ${modelId}`);
+        ctx.print(`Model already pinned ${modelId}`);
         return;
       }
       try {
         await providerApi.pinModel(modelId);
         ctx.print(`Pinned: ${modelId}`);
       } catch (e) {
-        ctx.print(`Error: ${summarizeError(e)}`);
+        ctx.print([
+          'Error',
+          `  message ${summarizeError(e)}`,
+        ].join('\n'));
       }
     },
   });

package/src/input/commands/local-setup-review.ts

@@ -13,7 +13,7 @@ export interface SetupReviewSnapshot {
   readonly builtinSubscriptionProviderCount: number;
   readonly activeSubscriptionCount: number;
   readonly pendingSubscriptionCount: number;
-  readonly serviceIssues: string[];
+  readonly hostIntegrationIssues: string[];
   readonly skillCount: number;
   readonly pluginCount: number;
   readonly quarantinedPluginCount: number;
@@ -33,11 +33,11 @@ export async function buildSetupReviewSnapshot(ctx: CommandContext): Promise<Set
   const serviceRegistry = requireServiceRegistry(ctx);
   const services = Object.keys(serviceRegistry.getAll()).sort((a, b) => a.localeCompare(b));
   const serviceConfigs = serviceRegistry.getAll();
-  const serviceIssues: string[] = [];
+  const hostIntegrationIssues: string[] = [];
   for (const name of services) {
     const inspection = await serviceRegistry.inspect(name);
     if (!inspection?.hasPrimaryCredential) {
-      serviceIssues.push(`${name}: missing primary credential`);
+      hostIntegrationIssues.push(`${name}: missing primary credential`);
     }
   }
 
@@ -79,13 +79,13 @@ export async function buildSetupReviewSnapshot(ctx: CommandContext): Promise<Set
       message: providerCount > 0 ? `${providerCount} model(s) available` : 'no models available',
     },
     {
-      severity: (services.length === 0 && oauthProviderCount === 0 && builtinSubscriptionProviderCount === 0) ? 'warn' : serviceIssues.length === 0 ? 'pass' : 'warn',
+      severity: (services.length === 0 && oauthProviderCount === 0 && builtinSubscriptionProviderCount === 0) ? 'warn' : hostIntegrationIssues.length === 0 ? 'pass' : 'warn',
       area: 'host',
       message: (services.length === 0 && oauthProviderCount === 0 && builtinSubscriptionProviderCount === 0)
         ? 'no connected host integrations configured'
-        : serviceIssues.length === 0
+        : hostIntegrationIssues.length === 0
           ? `${services.length} host integration(s), ${oauthProviderCount + builtinSubscriptionProviderCount} oauth provider(s), ${activeSubscriptionCount} active subscription override(s)`
-          : `${serviceIssues.length} host integration configuration issue(s)`,
+          : `${hostIntegrationIssues.length} host integration configuration issue(s)`,
     },
     {
       severity: quarantinedPluginCount === 0 ? 'pass' : 'warn',
@@ -111,7 +111,7 @@ export async function buildSetupReviewSnapshot(ctx: CommandContext): Promise<Set
     builtinSubscriptionProviderCount,
     activeSubscriptionCount,
     pendingSubscriptionCount,
-    serviceIssues,
+    hostIntegrationIssues,
     skillCount: skills.length,
     pluginCount: plugins.length,
     quarantinedPluginCount,