linting; restore deletion

Author: clavery

packages/b2c-cli/src/commands/job/run.ts

@@ -156,6 +156,14 @@ export default class JobRun extends JobCommand<typeof JobRun> {
     return execution;
   }
 
+  private parseBody(body: string): Record<string, unknown> {
+    try {
+      return JSON.parse(body) as Record<string, unknown>;
+    } catch {
+      this.error(t('commands.job.run.invalidBody', 'Invalid JSON body: {{body}}', {body}));
+    }
+  }
+
   private parseParameters(params: string[]): Array<{name: string; value: string}> {
     return params.map((p) => {
       const eqIndex = p.indexOf('=');
@@ -170,12 +178,4 @@ export default class JobRun extends JobCommand<typeof JobRun> {
       };
     });
   }
-
-  private parseBody(body: string): Record<string, unknown> {
-    try {
-      return JSON.parse(body) as Record<string, unknown>;
-    } catch {
-      this.error(t('commands.job.run.invalidBody', 'Invalid JSON body: {{body}}', {body}));
-    }
-  }
 }

packages/b2c-cli/src/utils/slas/client.ts

@@ -109,24 +109,6 @@ export abstract class SlasClientCommand<T extends typeof Command> extends OAuthC
     }),
   };
 
-  /**
-   * Get the SLAS client, ensuring short code is configured.
-   */
-  protected getSlasClient(): SlasClient {
-    const {shortCode} = this.resolvedConfig;
-    if (!shortCode) {
-      this.error(
-        t(
-          'error.shortCodeRequired',
-          'SCAPI short code required. Provide --short-code, set SFCC_SHORTCODE, or configure short-code in dw.json.',
-        ),
-      );
-    }
-
-    const oauthStrategy = this.getOAuthStrategy();
-    return createSlasClient({shortCode}, oauthStrategy);
-  }
-
   /**
    * Ensure tenant exists, creating it if necessary.
    * This is required before creating SLAS clients.
@@ -193,4 +175,22 @@ export abstract class SlasClientCommand<T extends typeof Command> extends OAuthC
       this.log(t('commands.slas.client.create.tenantCreated', 'SLAS tenant created successfully.'));
     }
   }
+
+  /**
+   * Get the SLAS client, ensuring short code is configured.
+   */
+  protected getSlasClient(): SlasClient {
+    const {shortCode} = this.resolvedConfig;
+    if (!shortCode) {
+      this.error(
+        t(
+          'error.shortCodeRequired',
+          'SCAPI short code required. Provide --short-code, set SFCC_SHORTCODE, or configure short-code in dw.json.',
+        ),
+      );
+    }
+
+    const oauthStrategy = this.getOAuthStrategy();
+    return createSlasClient({shortCode}, oauthStrategy);
+  }
 }

packages/b2c-cli/test/functional/e2e_cli_test.sh

@@ -41,8 +41,8 @@ cleanup() {
 
     # Delete sandbox if it was created
     if [ -n "$ODS_ID" ]; then
-        echo "skipping Deleting sandbox: $ODS_ID"
-        #$CLI ods delete "$ODS_ID" --force || true
+        echo "Deleting sandbox: $ODS_ID"
+        $CLI ods delete "$ODS_ID" --force || true
     fi
 
     exit $exit_code

packages/b2c-dx-mcp/src/server.ts

@@ -70,7 +70,6 @@ export class B2CDxMcpServer extends McpServer {
       const startTime = Date.now();
       const result = await handler(args);
       // TODO: Telemetry - Send TOOL_CALLED event with { name, _runtimeMs, isError: result.isError }
-      // @ts-expect-error Ignore unused variable
       const _runtimeMs = Date.now() - startTime;
 
       return result;