simplify CLI install command output formatting and show full paths

Author: jumski

pkgs/cli/src/commands/install/copy-migrations.ts

@@ -2,7 +2,7 @@ import fs from 'fs';
 import path from 'path';
 import { createRequire } from 'module';
 import { fileURLToPath } from 'url';
-import { log, confirm, note } from '@clack/prompts';
+import { log, confirm } from '@clack/prompts';
 import chalk from 'chalk';
 
 // Get the directory name in ES modules
@@ -231,38 +231,9 @@ export async function copyMigrations({
     }
   }
 
-  // If no files to copy, show message with details and return false (no changes made)
+  // If no files to copy, show message and return false (no changes made)
   if (filesToCopy.length === 0) {
-    // Show success message
-    log.success('All pgflow migrations are already in place');
-
-    // Show details of already installed migrations
-    if (skippedFiles.length > 0) {
-      const detailedMsg = [
-        'Already installed migrations:',
-        ...skippedFiles.map((file) => {
-          // Find the matching existing file to show how it was installed
-          const matchingFile = existingFiles.find((existingFile) =>
-            existingFile.includes(file)
-          );
-
-          if (matchingFile === file) {
-            // Installed with old direct method
-            return `  ${chalk.dim('•')} ${chalk.bold(file)}`;
-          } else {
-            // Installed with new timestamped method
-            const timestampPart =
-              matchingFile?.substring(0, matchingFile.indexOf(file) - 1) || '';
-            return `  ${chalk.dim('•')} ${chalk.dim(
-              timestampPart + '_'
-            )}${chalk.bold(file)}`;
-          }
-        }),
-      ].join('\n');
-
-      note(detailedMsg, 'Existing pgflow Migrations');
-    }
-
+    log.success(`All ${skippedFiles.length} pgflow migrations are already in place`);
     return false;
   }
 
@@ -276,40 +247,16 @@ export async function copyMigrations({
     file.destination = `${baseTimestamp}_${file.source}`;
   });
 
-  log.info(
-    `Found ${filesToCopy.length} migration${
-      filesToCopy.length !== 1 ? 's' : ''
-    } to install`
-  );
-
-  // Prepare summary message with colored output
-  const summaryParts = [];
-
-  if (filesToCopy.length > 0) {
-    summaryParts.push(
-      `${chalk.green('New migrations to install:')}\n${filesToCopy
-        .map((file) => {
-          // Extract the timestamp part from the new filename
-          const newTimestamp = file.destination.substring(0, 14);
-          // Format: dim timestamp + bright original name
-          return `${chalk.green('+')} ${file.source} → ${chalk.dim(
-            newTimestamp + '_'
-          )}${chalk.bold(file.source)}`;
-        })
-        .join('\n')}`
-    );
-  }
-
-  if (skippedFiles.length > 0) {
-    summaryParts.push(
-      `${chalk.yellow('Already installed:')}\n${skippedFiles
-        .map((file) => `${chalk.yellow('•')} ${file}`)
-        .join('\n')}`
-    );
-  }
+  // Build summary message similar to success output style
+  const summaryMsg = [
+    `New migrations to install:`,
+    ...filesToCopy.map((file) => {
+      const newTimestamp = file.destination.substring(0, 14);
+      return `  ${chalk.dim(newTimestamp + '_')}${chalk.bold(file.source)}`;
+    }),
+  ].join('\n');
 
-  // Show summary and ask for confirmation if not auto-confirming
-  note(summaryParts.join('\n\n'), 'pgflow Migrations');
+  log.info(summaryMsg);
 
   let shouldContinue = autoConfirm;
 

pkgs/cli/src/commands/install/create-edge-function.ts

@@ -1,6 +1,6 @@
 import fs from 'fs';
 import path from 'path';
-import { log, confirm, note } from '@clack/prompts';
+import { log, confirm } from '@clack/prompts';
 import chalk from 'chalk';
 import { getVersion } from '../../utils/get-version.js';
 
@@ -41,40 +41,42 @@ export async function createEdgeFunction({
   const indexPath = path.join(pgflowFunctionDir, 'index.ts');
   const denoJsonPath = path.join(pgflowFunctionDir, 'deno.json');
 
+  // Relative paths for display
+  const relativeFunctionDir = 'supabase/functions/pgflow';
+  const relativeIndexPath = `${relativeFunctionDir}/index.ts`;
+  const relativeDenoJsonPath = `${relativeFunctionDir}/deno.json`;
+
   // Check what needs to be created
-  const filesToCreate: Array<{ path: string; name: string }> = [];
+  const filesToCreate: Array<{ path: string; relativePath: string }> = [];
 
   if (!fs.existsSync(indexPath)) {
-    filesToCreate.push({ path: indexPath, name: 'index.ts' });
+    filesToCreate.push({ path: indexPath, relativePath: relativeIndexPath });
   }
 
   if (!fs.existsSync(denoJsonPath)) {
-    filesToCreate.push({ path: denoJsonPath, name: 'deno.json' });
+    filesToCreate.push({ path: denoJsonPath, relativePath: relativeDenoJsonPath });
   }
 
   // If all files exist, return success
   if (filesToCreate.length === 0) {
-    log.success('ControlPlane edge function files are already in place');
-
     const detailedMsg = [
-      'Existing files:',
-      `  ${chalk.dim('•')} ${chalk.bold('supabase/functions/pgflow/index.ts')}`,
-      `  ${chalk.dim('•')} ${chalk.bold('supabase/functions/pgflow/deno.json')}`,
+      'ControlPlane edge function files are already in place:',
+      `  ${chalk.bold(relativeIndexPath)}`,
+      `  ${chalk.bold(relativeDenoJsonPath)}`,
     ].join('\n');
 
-    note(detailedMsg, 'ControlPlane Edge Function');
+    log.success(detailedMsg);
 
     return false;
   }
 
   // Show what will be created
-  log.info(`Found ${filesToCreate.length} file${filesToCreate.length !== 1 ? 's' : ''} to create`);
-
-  const summaryParts = [`${chalk.green('Files to create:')}\n${filesToCreate
-    .map((file) => `${chalk.green('+')} ${file.name}`)
-    .join('\n')}`];
+  const summaryMsg = [
+    `Files to create:`,
+    ...filesToCreate.map((file) => `  ${chalk.bold(file.relativePath)}`),
+  ].join('\n');
 
-  note(summaryParts.join('\n'), 'ControlPlane Edge Function');
+  log.info(summaryMsg);
 
   // Get confirmation
   let shouldContinue = autoConfirm;
@@ -109,7 +111,7 @@ export async function createEdgeFunction({
   // Show success message
   const detailedSuccessMsg = [
     `Created ${filesToCreate.length} file${filesToCreate.length !== 1 ? 's' : ''}:`,
-    ...filesToCreate.map((file) => `  ${chalk.bold(file.name)}`),
+    ...filesToCreate.map((file) => `  ${chalk.bold(file.relativePath)}`),
   ].join('\n');
 
   log.success(detailedSuccessMsg);

pkgs/cli/src/commands/install/supabase-path-prompt.ts

@@ -28,11 +28,6 @@ export async function supabasePathPrompt(options?: { supabasePath?: string }) {
     }
   }
 
-  // Always prompt for detected paths - don't skip
-  if (detectedPath) {
-    log.info(`Found Supabase project at: ${detectedPath}`);
-  }
-
   const promptMessage = 'Where is your Supabase project located?';
 
   const supabasePath = await text({

pkgs/cli/src/commands/install/update-env-file.ts

@@ -1,6 +1,6 @@
 import fs from 'fs';
 import path from 'path';
-import { log, note, confirm } from '@clack/prompts';
+import { log, confirm } from '@clack/prompts';
 import chalk from 'chalk';
 
 /**
@@ -47,16 +47,12 @@ export async function updateEnvFile({
   // Prepare new content
   let newContent = currentContent;
 
-  // Build diff preview
+  // Check which variables need to be added
   const missingVars: Array<{ key: string; value: string }> = [];
-  const existingVars: Array<string> = [];
 
-  // Check which variables need to be added
   for (const [key, value] of Object.entries(envVars)) {
     if (!newContent.includes(`${key}=`)) {
       missingVars.push({ key, value });
-    } else {
-      existingVars.push(key);
     }
   }
 
@@ -66,33 +62,15 @@ export async function updateEnvFile({
     return false;
   }
 
-  log.info(`Found ${missingVars.length} variable${missingVars.length !== 1 ? 's' : ''} to add`);
-
-  // Build diff preview
-  const diffPreview: Array<string> = [];
-
-  if (isNewFile) {
-    diffPreview.push(`${chalk.green('Creating new .env file with:')}`);
-  } else {
-    diffPreview.push(`${chalk.green('Adding to existing .env file:')}`);
-  }
-
-  // Show variables to be added
-  for (const { key, value } of missingVars) {
-    diffPreview.push(`${chalk.green('+')} ${key}="${value}"`);
-  }
-
-  // Show existing variables if any
-  if (existingVars.length > 0) {
-    diffPreview.push('');
-    diffPreview.push(`${chalk.yellow('Already present:')}`);
-    for (const key of existingVars) {
-      diffPreview.push(`${chalk.yellow('•')} ${key}`);
-    }
-  }
+  // Build summary message
+  const summaryParts = [
+    isNewFile
+      ? `Creating .env file in supabase/functions/:`
+      : `Adding to supabase/functions/.env:`,
+    ...missingVars.map(({ key, value }) => `  ${chalk.bold(`${key}="${value}"`)}`),
+  ];
 
-  // Show the diff preview
-  note(diffPreview.join('\n'), 'Environment Variables');
+  log.info(summaryParts.join('\n'));
 
   // Ask for confirmation if not auto-confirming
   let shouldContinue = autoConfirm;