refactoring bundle topics and descriptions

Author: clavery

packages/b2c-cli/package.json

@@ -100,40 +100,37 @@
         "description": "WebDAV file operations (ls, get, put, rm, zip, unzip)"
       },
       "mrt": {
-        "description": "Manage Managed Runtime projects and deployments"
+        "description": "Manage Managed Runtime (MRT) projects, environments, bundles, and deployments"
       },
       "mrt:env": {
-        "description": "Manage MRT environments"
+        "description": "Create, update, delete, and configure MRT environments (targets)"
       },
       "mrt:env:var": {
-        "description": "Manage environment variables"
+        "description": "Set and delete environment variables for MRT environments"
       },
       "mrt:env:redirect": {
-        "description": "Manage URL redirects"
+        "description": "Configure URL redirects and clone redirect rules between environments"
       },
       "mrt:org": {
-        "description": "Manage MRT organizations"
+        "description": "List organizations and view B2C Commerce instance connections"
       },
       "mrt:project": {
-        "description": "Manage MRT projects"
+        "description": "Create, update, delete, and configure MRT projects"
       },
       "mrt:project:member": {
-        "description": "Manage project members"
+        "description": "Add, remove, and update project member roles and permissions"
       },
       "mrt:project:notification": {
-        "description": "Manage deployment notifications"
+        "description": "Configure email notifications for deployment events (start, success, failure)"
       },
       "mrt:env:access-control": {
-        "description": "Manage access control headers"
+        "description": "Configure access control headers for environment security"
       },
       "mrt:bundle": {
-        "description": "Manage and deploy bundles"
+        "description": "Push builds, deploy bundles, view deployment history, and download artifacts"
       },
       "mrt:user": {
-        "description": "Manage user profile and settings"
-      },
-      "mrt:b2c": {
-        "description": "Manage B2C Commerce integrations"
+        "description": "View profile, reset API keys, and configure email preferences"
       },
       "ods": {
         "description": "Manage On-Demand Sandboxes"

packages/b2c-cli/src/commands/mrt/bundle/download.ts

@@ -3,13 +3,18 @@
  * SPDX-License-Identifier: Apache-2
  * For full license text, see the license.txt file in the repo root or http://www.apache.org/licenses/LICENSE-2.0
  */
-import {Args} from '@oclif/core';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+import {pipeline} from 'node:stream/promises';
+import {Args, Flags} from '@oclif/core';
 import {MrtCommand} from '@salesforce/b2c-tooling-sdk/cli';
 import {downloadBundle, type DownloadBundleResult} from '@salesforce/b2c-tooling-sdk/operations/mrt';
 import {t} from '../../../i18n/index.js';
 
+type BundleDownloadResult = DownloadBundleResult & {filePath?: string};
+
 /**
- * Get a presigned download URL for a bundle.
+ * Download a bundle artifact from Managed Runtime.
  */
 export default class MrtBundleDownload extends MrtCommand<typeof MrtBundleDownload> {
   static args = {
@@ -19,23 +24,30 @@ export default class MrtBundleDownload extends MrtCommand<typeof MrtBundleDownlo
     }),
   };
 
-  static description = t(
-    'commands.mrt.bundle.download.description',
-    'Get a presigned download URL for a Managed Runtime bundle',
-  );
+  static description = t('commands.mrt.bundle.download.description', 'Download a Managed Runtime bundle artifact');
 
   static enableJsonFlag = true;
 
   static examples = [
     '<%= config.bin %> <%= command.id %> 12345 --project my-storefront',
+    '<%= config.bin %> <%= command.id %> 12345 -p my-storefront -o ./artifacts/my-bundle.tgz',
+    '<%= config.bin %> <%= command.id %> 12345 -p my-storefront --url-only',
     '<%= config.bin %> <%= command.id %> 12345 -p my-storefront --json',
   ];
 
   static flags = {
     ...MrtCommand.baseFlags,
+    output: Flags.string({
+      char: 'o',
+      description: 'Output file path (default: bundle-{bundleId}.tgz)',
+    }),
+    'url-only': Flags.boolean({
+      description: 'Only output the download URL without downloading',
+      default: false,
+    }),
   };
 
-  async run(): Promise<DownloadBundleResult> {
+  async run(): Promise<BundleDownloadResult> {
     this.requireMrtCredentials();
 
     const {bundleId} = this.args;
@@ -47,7 +59,14 @@ export default class MrtBundleDownload extends MrtCommand<typeof MrtBundleDownlo
       );
     }
 
-    this.log(t('commands.mrt.bundle.download.fetching', 'Getting download URL for bundle {{bundleId}}...', {bundleId}));
+    const urlOnly = this.flags['url-only'];
+
+    this.log(
+      t('commands.mrt.bundle.download.fetching', 'Fetching download URL for bundle {{bundleId}} from {{project}}...', {
+        bundleId,
+        project,
+      }),
+    );
 
     try {
       const result = await downloadBundle(
@@ -59,19 +78,66 @@ export default class MrtBundleDownload extends MrtCommand<typeof MrtBundleDownlo
         this.getMrtAuth(),
       );
 
+      // If url-only flag or JSON mode, just return the URL
+      if (urlOnly) {
+        if (!this.jsonEnabled()) {
+          this.log(
+            t('commands.mrt.bundle.download.urlOnly', 'Download URL (valid for 1 hour):\n{{downloadUrl}}', {
+              downloadUrl: result.downloadUrl,
+            }),
+          );
+        }
+        return result;
+      }
+
+      // Download the file
+      const outputPath = this.flags.output ?? `bundle-${bundleId}.tgz`;
+      const absolutePath = path.resolve(outputPath);
+
+      this.log(
+        t('commands.mrt.bundle.download.downloading', 'Downloading bundle {{bundleId}} to {{filePath}}...', {
+          bundleId,
+          filePath: outputPath,
+        }),
+      );
+
+      const response = await fetch(result.downloadUrl);
+      if (!response.ok) {
+        this.error(
+          t('commands.mrt.bundle.download.httpError', 'Failed to download bundle: HTTP {{status}}', {
+            status: response.status,
+          }),
+        );
+      }
+
+      if (!response.body) {
+        this.error(t('commands.mrt.bundle.download.noBody', 'Failed to download bundle: empty response'));
+      }
+
+      // Ensure directory exists
+      const dir = path.dirname(absolutePath);
+      if (dir !== '.') {
+        fs.mkdirSync(dir, {recursive: true});
+      }
+
+      // Stream the response to file
+      const fileStream = fs.createWriteStream(absolutePath);
+      await pipeline(response.body, fileStream);
+
       if (!this.jsonEnabled()) {
         this.log(
-          t('commands.mrt.bundle.download.success', 'Download URL (valid for 1 hour):\n{{downloadUrl}}', {
-            downloadUrl: result.downloadUrl,
+          t('commands.mrt.bundle.download.success', 'Bundle {{bundleId}} downloaded to {{filePath}}', {
+            bundleId,
+            filePath: outputPath,
           }),
         );
       }
 
-      return result;
+      return {...result, filePath: absolutePath};
     } catch (error) {
       if (error instanceof Error) {
         this.error(
-          t('commands.mrt.bundle.download.failed', 'Failed to get download URL: {{message}}', {message: error.message}),
+          t('commands.mrt.bundle.download.failed', 'Failed to download bundle: {{message}}', {message: error.message}),
         );
       }
       throw error;