fix: add consistent profile/region prefixes and improved output filtering for multi-account deployments

Author: pgagnidze

.gitignore

@@ -40,3 +40,4 @@ CLAUDE.local.md
 temp/
 tmp/
 IDEAS.local.md
+*.local.md

package-lock.json

@@ -99,60 +99,64 @@ async function main() {
     process.exit(1);
   }
 
-  console.log();
-  const modeMessages = {
-    diff: "Showing differences",
-    changeset: "Creating changesets",
-    execute: "Deploying stacks",
-  };
-  logger.info(
-    modeMessages[args.mode as keyof typeof modeMessages] || "Processing stacks",
-  );
-
   const results = await deployToAllRegions(regions, args, controller.signal);
 
   displayResults(args, results);
 }
 
 function displayHeader(args: ParsedArgs, regions: string[]) {
-  console.log(`
-Multi-Region CDK Deployment
-${Object.entries({
-  Profile: args.profile,
-  Stack: args.stackPattern,
-  Regions: regions.join(" "),
-  Mode: args.mode,
-  Deployment: args.sequential ? "sequential" : "parallel",
-  Dependencies: args.includeDeps ? "included" : "excluded",
-  Timeout: process.env.CDK_TIMEOUT || "not set",
-})
-  .map(([k, v]) => `${logger.dim(k + ":")} ${v}`)
-  .join("\n")}`);
+  logger.header("CDKO: Multi-Region Deployment");
+  console.log();
+
+  logger.subheader("Configuration:");
+  const config = [
+    `  Profile: ${args.profile}`,
+    `  Stack:   ${args.stackPattern}`,
+    `  Regions: ${regions.join(", ")}`,
+    `  Mode:    ${args.mode}`,
+    `  Deployment: ${args.sequential ? "sequential" : "parallel"}`,
+    `  Dependencies: ${args.includeDeps ? "included" : "excluded"}`,
+    `  Timeout: ${process.env.CDK_TIMEOUT || "not set"}`,
+  ];
+  console.log(config.join("\n"));
 }
 
 function displayResults(args: ParsedArgs, results: DeploymentResult[]) {
-  const failedRegions = results.filter((r) => !r.success);
+  const successful = results.filter((r) => r.success);
+  const failed = results.filter((r) => !r.success);
 
-  if (failedRegions.length > 0) {
-    console.log();
-    logger.error(`${failedRegions.length} region(s) failed`);
-    process.exit(1);
+  console.log();
+  logger.phase("Summary", "");
+
+  const status = failed.length > 0 ? "FAILURE" : "SUCCESS";
+
+  console.log(
+    `  Status: ${failed.length > 0 ? logger.dim("❌") : logger.dim("✅")} ${status}`,
+  );
+  console.log(`  Stacks processed: ${results.length}`);
+  console.log(`  Successful: ${successful.length}`);
+
+  if (args.mode === "diff") {
+    console.log(`  With changes: ${successful.length}`);
   }
 
-  console.log();
+  if (failed.length > 0) {
+    console.log(`  Failed: ${failed.length}`);
+  }
 
-  if (args.dryRun) {
-    logger.success("Dry run completed - no actual changes made");
-  } else if (args.mode === "changeset") {
-    logger.success("Changesets created for review in CloudFormation console");
-    logger.info("Execute changesets manually after review");
-  } else if (args.mode === "execute") {
-    const totalDuration = results
+  if (successful.length > 0 && successful.some((r) => r.duration)) {
+    const totalDuration = successful
       .filter((r) => r.duration)
       .reduce((sum, r) => sum + parseFloat(r.duration!), 0);
-    logger.success(`All deployments completed in ${totalDuration.toFixed(1)}s`);
-  } else {
-    logger.success("Differences shown for all regions");
+
+    const minutes = Math.floor(totalDuration / 60);
+    const seconds = Math.floor(totalDuration % 60);
+    const timeStr = minutes > 0 ? `${minutes}m ${seconds}s` : `${seconds}s`;
+    console.log(`  Total time: ${timeStr}`);
+  }
+
+  if (failed.length > 0) {
+    process.exit(1);
   }
 }
 

src/cli/index.ts

@@ -72,11 +72,11 @@ export class AccountManager {
       return patterns;
     }
 
-    logger.info(
-      `Found ${
-        matchedProfiles.length
-      } profile(s) matching pattern: ${matchedProfiles.join(", ")}`,
-    );
+    if (matchedProfiles.length > 1) {
+      logger.info(
+        `Found ${matchedProfiles.length} profile(s) matching pattern: ${matchedProfiles.join(", ")}`,
+      );
+    }
     return matchedProfiles;
   }
 
@@ -86,8 +86,6 @@ export class AccountManager {
     }
 
     try {
-      logger.info(`Discovering account for profile: ${profile}`);
-
       const result =
         await $`aws sts get-caller-identity --profile ${profile} --output json`.quiet();
       const identity = JSON.parse(result.stdout);
@@ -100,7 +98,6 @@ export class AccountManager {
       };
 
       this.accountCache.set(profile, accountInfo);
-      logger.info(`Profile ${profile} → Account ${identity.Account}`);
 
       return accountInfo;
     } catch (error) {
@@ -120,8 +117,6 @@ export class AccountManager {
   }
 
   async getMultiAccountInfo(profiles: string[]): Promise<AccountInfo[]> {
-    logger.info(`Discovering accounts for ${profiles.length} profile(s)...`);
-
     const accountPromises = profiles.map((profile: string) =>
       this.getAccountInfo(profile).catch(
         (error): FailedAccountInfo => ({

src/core/account-manager.ts

@@ -28,14 +28,11 @@ export class CloudAssemblyManager {
 
     try {
       if (existsSync(cloudAssemblyPath)) {
-        logger.info(`Cleaning existing cloud assembly at ${cloudAssemblyPath}`);
         rmSync(cloudAssemblyPath, { recursive: true, force: true });
       }
 
       mkdirSync(cloudAssemblyPath, { recursive: true });
 
-      logger.info("Synthesizing cloud assembly...");
-
       const cdkArgs = ["synth"];
 
       if (process.env.CDK_CLI_NOTICES !== "true") {
@@ -55,7 +52,6 @@ export class CloudAssemblyManager {
       const result = await $({ quiet: true })`cdk ${cdkArgs}`;
 
       if (result.exitCode === 0) {
-        logger.success(`Cloud assembly synthesized to ${cloudAssemblyPath}`);
         this.cloudAssemblyPath = cloudAssemblyPath;
         return cloudAssemblyPath;
       } else {