@newpeak/barista-cli 0.1.6 → 0.1.7

package/docs/commands/liberica/warehouses/list.md

@@ -0,0 +1,184 @@
+# barista liberica warehouses list
+
+List warehouses with pagination.
+
+## 2.1 Command Metadata
+
+| Field | Value |
+|------|-----|
+| Full Command | `barista liberica warehouses list` |
+| Description | List warehouses with pagination |
+| HTTP Method | GET |
+| Requires Authentication | Yes |
+| Supports Dry-Run | No |
+
+## 2.2 Backend API Reference
+
+### Controller Location
+```
+coffee-liberica-end/
+└── facade/liberica-facade-enterprise/
+    └── src/main/java/com/newpeak/liberica/facade/enterprise/controller/stock/
+        └── EnterpriseWarehouseController.java
+            └── page(@GetResource(path = "/page"))
+                └── public ResponseData<PageResult<MasterWarehouseResponse>> page(
+                        @RequestHeader("X-TENANT-ID") Long tenantId,
+                        @ParameterObject MasterWarehouseRequest masterWarehouseRequest
+                    )
+```
+
+### Request DTO Location
+```
+coffee-liberica-end/
+└── business/liberica-business-master/master-api/
+    └── src/main/java/com/newpeak/liberica/master/api/pojo/request/
+        └── MasterWarehouseRequest.java
+            ├── warehouseCode: String
+            ├── warehouseName: String
+            ├── warehouseClasses: String
+            ├── manager: String
+            ├── warehouseTelephone: String
+            ├── warehouseFax: String
+            ├── warehouseAddress: String
+            ├── enableLocation: String (Y/N)
+            ├── enableNegativeInventory: String (Y/N)
+            ├── statusFlag: Integer
+            └── remark: String
+```
+
+### Response DTO Location
+```
+coffee-liberica-end/
+└── business/liberica-business-master/master-api/
+    └── src/main/java/com/newpeak/liberica/master/api/pojo/response/
+        └── MasterWarehouseResponse.java
+            ├── warehouseId: Long
+            ├── warehouseCode: String
+            ├── warehouseName: String
+            ├── warehouseClasses: String
+            ├── manager: String
+            ├── warehouseTelephone: String
+            ├── warehouseFax: String
+            ├── warehouseAddress: String
+            ├── enableLocation: String (Y/N)
+            ├── enableNegativeInventory: String (Y/N)
+            ├── statusFlag: Integer (1=enable, 2=disable)
+            ├── remark: String
+            └── locations: List<MasterWarehouseLocationResponse>
+```
+
+## 2.3 CLI Parameter Design
+
+### Command Structure
+```
+barista liberica warehouses list [options]
+```
+
+### Global Options
+| Option | Type | Description |
+|------|------|------|
+| `--env` | string | Target environment (dev\|test\|prod-cn\|prod-jp) |
+| `--tenant` | string | Tenant code |
+| `--json` | boolean | JSON output |
+
+### Command Options
+| Option | Short | Type | Required | Default | Description | Maps to Query Param |
+|------|--------|------|------|--------|------|-------------|
+| --page | -p | number | No | 1 | Page number (CLI 1-based) | pageNo (0-based) |
+| --size | -s | number | No | 20 | Page size | pageSize |
+| --status | — | number | No | - | Status filter (1=enable, 2=disable) | statusFlag |
+| --keyword | -k | string | No | - | Keyword search (warehouseName and warehouseCode) | warehouseName / warehouseCode |
+| --classes | — | string | No | - | Warehouse class filter | warehouseClasses |
+| --enable-location | — | boolean | No | - | Filter by location management enabled | enableLocation |
+
+## 2.4 Field Mapping Table
+
+| CLI Parameter | Query Param / DTO Field | Type Conversion | Validation Rule |
+|---------|---------|----------|----------|
+| --page / -p | pageNo | number, CLI page - 1 | >= 0 |
+| --size / -s | pageSize | number | > 0 |
+| --status | statusFlag | number | 1 or 2 |
+| --keyword / -k | warehouseName / warehouseCode | Direct | max=255 |
+| --classes | warehouseClasses | Direct | max=255 |
+| --enable-location | enableLocation | boolean->String (Y/N) | max=1 |
+
+## 2.5 Error Code Reference
+
+### ExceptionEnum Location
+```
+coffee-liberica-end/
+└── business/liberica-business-master/master-api/
+    └── src/main/java/com/newpeak/liberica/master/api/exception/enums/
+        └── MasterWarehouseExceptionEnum.java
+            ├── MASTER_WAREHOUSE_NOT_EXIST("01001370001", "租户仓库不存在")
+            ├── MASTER_WAREHOUSE_CODE_FORMAT_CANNOT_EMPTY("01001370002", "仓库编码规则不能为空")
+            └── MASTER_WAREHOUSE_CODE_FORMAT_NOT_CORRECT("01001370003", "仓库编码规则格式不正确")
+```
+
+### Known Error Codes
+| Error Code | Error Message | Trigger Condition |
+|--------|----------|----------|
+| 01001370001 | Warehouse does not exist | Detail query with non-existent ID |
+
+## 2.6 Permission Check
+
+| Check Item | Location | Description |
+|--------|------|------|
+| Page Query | No specific permission code on `@GetResource(path = "/page")` | Available to authenticated users |
+
+## 2.7 Implementation Notes
+
+1. **Pagination**: CLI uses 1-based page numbers. API uses 0-based. Convert: `pageNo = options.page - 1`.
+2. **Response Fields**: Backend returns `rows`, `pageNo`, `pageSize`, `totalRows`. Do NOT use `records` or `total`.
+3. **Keyword Search**: When `--keyword` is provided, send it as both `warehouseName` and `warehouseCode` query params.
+4. **BigInteger ID**: `warehouseId` in response rows must be kept as `string` in TypeScript.
+5. **Table Output**: Default columns: ID, Code, Name, Classes, Location, Status.
+6. **JSON Output**: When `--json` is used, wrap response with standardized pagination object.
+
+## 2.8 Example Usage
+
+```bash
+# List all warehouses (first page, 20 per page)
+barista liberica warehouses list
+
+# List with pagination
+barista liberica warehouses list --page 2 --size 50
+
+# Filter by status
+barista liberica warehouses list --status 1
+
+# Keyword search
+barista liberica warehouses list --keyword "Main"
+
+# Filter by class
+barista liberica warehouses list --classes "FINISHED_GOOD"
+
+# JSON output
+barista liberica warehouses list --json
+
+# Combine filters
+barista liberica warehouses list \
+  --page 1 \
+  --size 10 \
+  --status 1 \
+  --keyword "WH-001" \
+  --json
+```
+
+## 2.9 API Path
+
+```
+GET /api/enterprise/purchases/warehouse/page
+```
+
+**Query Parameters:**
+```
+?pageNo=0&pageSize=20&statusFlag=1&warehouseCode=Main&warehouseName=Main
+```
+
+**Headers:**
+```
+Authorization: <token>  (No "Bearer " prefix)
+```
+
+**Note**: Do NOT include `X-TENANT-ID` header. Tenant is extracted from JWT token.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@newpeak/barista-cli",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "AI Tools CLI for Liberica and Arabica services",
   "type": "module",
   "bin": "./bin/barista",

package/src/commands/liberica/index.ts

@@ -12,6 +12,7 @@ import { createSuppliersCommand } from './suppliers/index.js';
 import { createMaterialsCommand } from './materials/index.js';
 import { createUomsCommand } from './uoms/index.js';
 import { createCurrencyCommand } from './currency/index.js';
+import { createWarehousesCommand } from './warehouses/index.js';
 
 export function createLibericaCommand(): Command {
   const libericaCommand = new Command('liberica');
@@ -29,6 +30,7 @@ export function createLibericaCommand(): Command {
   libericaCommand.addCommand(createMaterialsCommand());
   libericaCommand.addCommand(createUomsCommand());
   libericaCommand.addCommand(createCurrencyCommand());
+  libericaCommand.addCommand(createWarehousesCommand());
 
   libericaCommand.action(() => {
     console.log(chalk.bold('\n☕ Liberica Commands\n'));
@@ -43,6 +45,8 @@ export function createLibericaCommand(): Command {
     console.log('  Use "barista liberica materials <command>" for material management');
     console.log('  Use "barista liberica uoms <command>" for unit of measure management');
     console.log('  Use "barista liberica currencies <command>" for currency management');
+    console.log('  Use "barista liberica warehouses <command>" for warehouse management');
+    console.log('  Use "barista liberica warehouses locations <command>" for warehouse location management');
     console.log('\n  Run "barista liberica <command> --help" for more details\n');
   });
 

package/src/commands/liberica/materials/create.ts

@@ -29,19 +29,26 @@ export function createMaterialsCreateCommand(): Command {
     .option('--dry-run', 'Preview without making API call')
     .option('--json', 'Output as JSON');
 
-  cmd.action(async () => {
+  cmd.action(async (options) => {
     const context = configManager.getCurrentContext();
-    const opts = cmd.opts();
 
-    let name = opts.name as string | undefined;
-    let materialNo = (opts.no as string | undefined) || (opts['no'] as string | undefined);
-    let materialClass = opts.class as string | undefined;
-    const materialType = opts.type as string | undefined;
-    const materialUnit = opts.unit as string | undefined;
-    const dryRun = opts.dryRun === true;
-    const outputJson = (opts as any).json === true || process.argv.includes('--json');
+    let name = options.name as string | undefined;
+    let materialNo = (options.no as string | undefined) || (options['no'] as string | undefined);
+    let materialClass = options.class as string | undefined;
+    const materialType = options.type as string | undefined;
+    const materialUnit = options.unit as string | undefined;
+    const dryRun = options.dryRun === true;
+    const outputJson = options.json === true;
 
     if (!name) {
+      if (dryRun) {
+        if (outputJson) {
+          console.log(JSON.stringify({ success: false, error: { code: 'DRY_RUN_MISSING_ARGS', message: 'Missing required --name in dry-run mode' } }));
+        } else {
+          console.error(chalk.red('\n✗ --name is required. Cannot preview without material name.\n'));
+        }
+        process.exit(1);
+      }
       const { inputName } = await inquirer.prompt([
         {
           type: 'input',
@@ -54,6 +61,14 @@ export function createMaterialsCreateCommand(): Command {
     }
 
     if (!materialClass) {
+      if (dryRun) {
+        if (outputJson) {
+          console.log(JSON.stringify({ success: false, error: { code: 'DRY_RUN_MISSING_ARGS', message: 'Missing required --class in dry-run mode' } }));
+        } else {
+          console.error(chalk.red('\n✗ --class is required. Cannot preview without material class.\n'));
+        }
+        process.exit(1);
+      }
       const { inputClass } = await inquirer.prompt([
         {
           type: 'input',
@@ -66,6 +81,14 @@ export function createMaterialsCreateCommand(): Command {
     }
 
     if (!materialNo) {
+      if (dryRun) {
+        if (outputJson) {
+          console.log(JSON.stringify({ success: false, error: { code: 'DRY_RUN_MISSING_ARGS', message: 'Missing required --no in dry-run mode' } }));
+        } else {
+          console.error(chalk.red('\n✗ --no is required. Cannot preview without material number.\n'));
+        }
+        process.exit(1);
+      }
       const { inputNo } = await inquirer.prompt([
         {
           type: 'input',

package/src/commands/liberica/orgs/index.ts

@@ -1,6 +1,5 @@
 import { Command } from 'commander';
 import chalk from 'chalk';
-import Table from 'cli-table3';
 import { configManager } from '../../../core/config/manager.js';
 import { apiClient } from '../../../core/api/client.js';
 import { Environment } from '../../../types/index.js';

package/src/commands/liberica/users/create.ts

@@ -105,6 +105,7 @@ export function createUserCreateCommand(): Command {
             await apiClient.updateUserOrgOrApp(environment, tenant, { newOrgId: String(firstOrg.orgId) });
           }
         } catch (err) {
+          // ignore errors during org switch
         }
       }
 

package/src/commands/liberica/warehouses/create.ts

@@ -0,0 +1,204 @@
+import { Command } from 'commander';
+import chalk from 'chalk';
+import inquirer from 'inquirer';
+import { apiClient } from '../../../core/api/client.js';
+import { configManager } from '../../../core/config/manager.js';
+import { CreateWarehouseRequest } from '../../../types/warehouse.js';
+
+function formatTable(data: CreateWarehouseRequest): string {
+  const lines: string[] = [];
+  lines.push(`  ${chalk.gray('Code:')}   ${data.warehouseCode}`);
+  lines.push(`  ${chalk.gray('Name:')}   ${data.warehouseName || '(not set)'}`);
+  if (data.warehouseType) lines.push(`  ${chalk.gray('Type:')}   ${data.warehouseType}`);
+  if (data.warehouseClasses) lines.push(`  ${chalk.gray('Classes:')}   ${data.warehouseClasses}`);
+  if (data.address) lines.push(`  ${chalk.gray('Address:')}   ${data.address}`);
+  if (data.contact) lines.push(`  ${chalk.gray('Contact:')}   ${data.contact}`);
+  if (data.phone) lines.push(`  ${chalk.gray('Phone:')}   ${data.phone}`);
+  if (data.manager) lines.push(`  ${chalk.gray('Manager:')}   ${data.manager}`);
+  if (data.enableLocation !== undefined) {
+    lines.push(`  ${chalk.gray('Enable Location:')}   ${data.enableLocation === 'Y' ? 'Yes' : 'No'}`);
+  }
+  if (data.enableNegativeInventory !== undefined) {
+    lines.push(`  ${chalk.gray('Enable Negative Inventory:')}   ${data.enableNegativeInventory === 'Y' ? 'Yes' : 'No'}`);
+  }
+  if (data.remark) lines.push(`  ${chalk.gray('Remark:')}   ${data.remark}`);
+  return lines.join('\n');
+}
+
+export function createWarehouseCreateCommand(): Command {
+  const cmd = new Command('create');
+  cmd.description('Create a new warehouse (auto-generates code if not provided)');
+
+  cmd
+    .option('-n, --name <name>', 'Warehouse name (required)')
+    .option('-c, --code <code>', 'Warehouse code (auto-generated if not provided)')
+    .option('-t, --type <type>', 'Warehouse type')
+    .option('--classes <classes>', 'Warehouse classes')
+    .option('-a, --address <address>', 'Address')
+    .option('--contact <contact>', 'Contact person')
+    .option('-p, --phone <phone>', 'Contact phone')
+    .option('-m, --manager <manager>', 'Warehouse manager')
+    .option('--enable-location', 'Enable location management')
+    .option('--disable-location', 'Disable location management')
+    .option('--enable-negative-inventory', 'Enable negative inventory')
+    .option('--disable-negative-inventory', 'Disable negative inventory')
+    .option('--remark <remark>', 'Remark')
+    .option('--dry-run', 'Preview without making API call')
+    .option('--json', 'Output as JSON');
+
+  cmd.action(async (options) => {
+    const context = configManager.getCurrentContext();
+
+    let name = options.name as string | undefined;
+    let code = options.code as string | undefined;
+    const type = options.type as string | undefined;
+    const classes = options.classes as string | undefined;
+    const address = options.address as string | undefined;
+    const contact = options.contact as string | undefined;
+    const phone = options.phone as string | undefined;
+    const manager = options.manager as string | undefined;
+    const enableLocationRaw = options.enableLocation as boolean | undefined;
+    const disableLocationRaw = options.disableLocation as boolean | undefined;
+    const enableNegativeInventoryRaw = options.enableNegativeInventory as boolean | undefined;
+    const disableNegativeInventoryRaw = options.disableNegativeInventory as boolean | undefined;
+    const remark = options.remark as string | undefined;
+    const dryRun = options.dryRun === true;
+    const jsonOutput = options.json === true;
+
+    if (!name) {
+      const { inputName } = await inquirer.prompt([
+        {
+          type: 'input',
+          name: 'inputName',
+          message: 'Enter warehouse name:',
+          validate: (input: string) => input.trim().length > 0 || 'Warehouse name is required',
+        },
+      ]);
+      name = inputName;
+    }
+
+    if (!code) {
+      const codeResponse = await apiClient.getCodeByType(context.environment, context.tenant, 'TenantWarehouseCode');
+      if (codeResponse.success && codeResponse.data) {
+        code = codeResponse.data;
+      } else {
+        const errMsg = codeResponse.error?.message || 'Failed to generate warehouse code';
+        if (jsonOutput) {
+          console.log(JSON.stringify({ success: false, error: { code: 'CODE_GENERATION_ERROR', message: errMsg } }));
+        } else {
+          console.error(chalk.red(`\n✗ Failed to generate warehouse code: ${errMsg}\n`));
+        }
+        process.exit(1);
+      }
+    }
+
+    let enableLocation: string | undefined;
+    if (enableLocationRaw !== undefined) {
+      enableLocation = 'Y';
+    } else if (disableLocationRaw !== undefined) {
+      enableLocation = 'N';
+    }
+
+    let enableNegativeInventory: string | undefined;
+    if (enableNegativeInventoryRaw !== undefined) {
+      enableNegativeInventory = 'Y';
+    } else if (disableNegativeInventoryRaw !== undefined) {
+      enableNegativeInventory = 'N';
+    }
+
+    const data: CreateWarehouseRequest = {
+      warehouseCode: code!,
+      warehouseName: name!,
+      warehouseType: type,
+      warehouseClasses: classes,
+      address: address,
+      contact: contact,
+      phone: phone,
+      manager: manager,
+      enableLocation: enableLocation,
+      enableNegativeInventory: enableNegativeInventory,
+      remark: remark,
+    };
+
+    if (dryRun) {
+      if (jsonOutput) {
+        console.log(JSON.stringify({ success: true, dryRun: true, data }));
+      } else {
+        console.log(chalk.bold('\n🔍 Dry-Run Mode: No changes will be made\n'));
+        console.log('  Warehouse to be created:');
+        console.log(formatTable(data));
+        console.log();
+      }
+      return;
+    }
+
+    if (!jsonOutput) {
+      console.log(chalk.bold('\n📝 Creating Warehouse\n'));
+    }
+
+    const response = await apiClient.createWarehouse(context.environment, context.tenant, data);
+
+    if (!response.success) {
+      const errorMsg = response.error?.message || 'Failed to create warehouse';
+      const errorCode = response.error?.code || 'CREATE_WAREHOUSE_ERROR';
+      if (jsonOutput) {
+        console.log(JSON.stringify({ success: false, error: { code: errorCode, message: errorMsg } }));
+      } else {
+        console.error(chalk.red(`\n✗ Failed to create warehouse: ${errorMsg}`));
+        if (response.error?.code) {
+          console.error(chalk.gray(`  Error code: ${response.error.code}`));
+        }
+        console.error();
+      }
+      process.exit(1);
+    }
+
+    if (response.success) {
+      const warehouse = response.data as {
+        warehouseId?: string;
+        warehouseCode?: string;
+        warehouseName?: string;
+        warehouseType?: string;
+        warehouseClasses?: string;
+        address?: string;
+        contact?: string;
+        phone?: string;
+        manager?: string;
+        enableLocation?: boolean;
+        enableNegativeInventory?: boolean;
+        remark?: string;
+        status?: number;
+        createTime?: string;
+      } | undefined;
+
+      if (jsonOutput) {
+        console.log(JSON.stringify({ success: true, data: warehouse }));
+      } else {
+        console.log(chalk.green('✓ Warehouse created successfully\n'));
+        if (warehouse?.warehouseId) console.log(`  ${chalk.gray('Warehouse ID:')} ${chalk.green(String(warehouse.warehouseId))}`);
+        if (warehouse?.warehouseCode) console.log(`  ${chalk.gray('Code:')}   ${warehouse.warehouseCode}`);
+        if (warehouse?.warehouseName) console.log(`  ${chalk.gray('Name:')}   ${warehouse.warehouseName}`);
+        if (warehouse?.warehouseType) console.log(`  ${chalk.gray('Type:')}   ${warehouse.warehouseType}`);
+        if (warehouse?.warehouseClasses) console.log(`  ${chalk.gray('Classes:')}   ${warehouse.warehouseClasses}`);
+        if (warehouse?.address) console.log(`  ${chalk.gray('Address:')}   ${warehouse.address}`);
+        if (warehouse?.contact) console.log(`  ${chalk.gray('Contact:')}   ${warehouse.contact}`);
+        if (warehouse?.phone) console.log(`  ${chalk.gray('Phone:')}   ${warehouse.phone}`);
+        if (warehouse?.manager) console.log(`  ${chalk.gray('Manager:')}   ${warehouse.manager}`);
+        if (warehouse?.enableLocation !== undefined) {
+          console.log(`  ${chalk.gray('Enable Location:')}   ${warehouse.enableLocation ? 'Yes' : 'No'}`);
+        }
+        if (warehouse?.enableNegativeInventory !== undefined) {
+          console.log(`  ${chalk.gray('Enable Negative Inventory:')}   ${warehouse.enableNegativeInventory ? 'Yes' : 'No'}`);
+        }
+        if (warehouse?.remark) console.log(`  ${chalk.gray('Remark:')}   ${warehouse.remark}`);
+        if (warehouse?.status !== undefined) {
+          const statusText = warehouse.status === 1 ? 'Enabled' : 'Disabled';
+          console.log(`  ${chalk.gray('Status:')}   ${chalk.green(statusText)}`);
+        }
+        console.log();
+      }
+    }
+  });
+
+  return cmd;
+}

package/src/commands/liberica/warehouses/delete.ts

@@ -0,0 +1,112 @@
+import { Command } from 'commander';
+import chalk from 'chalk';
+import { configManager } from '../../../core/config/manager.js';
+import { apiClient } from '../../../core/api/client.js';
+import { Environment } from '../../../types/index.js';
+import { Warehouse } from '../../../types/warehouse.js';
+
+export function createWarehouseDeleteCommand(): Command {
+  const deleteCommand = new Command('delete');
+  deleteCommand
+    .description('Delete a warehouse')
+    .argument('<warehouseId>', 'Warehouse ID')
+    .option('--force', 'Skip confirmation and delete directly')
+    .option('--dry-run', 'Preview without making API call')
+    .option('--json', 'Output as JSON')
+    .action(async (warehouseId: string, options: Record<string, unknown>) => {
+      const context = configManager.getCurrentContext();
+      const environment = context.environment as Environment;
+      const tenant = context.tenant;
+      const dryRun = options.dryRun === true;
+      const jsonOutput = options.json === true;
+
+      if (!jsonOutput) {
+        console.log(chalk.bold(`\n🗑️  Delete Warehouse (${environment})\n`));
+        console.log(chalk.gray(`  Warehouse ID: ${warehouseId}`));
+      }
+
+      let warehouse: Warehouse | undefined;
+      try {
+        const warehouseResponse = await apiClient.getWarehouse(environment, tenant, warehouseId);
+        if (!warehouseResponse.success || !warehouseResponse.data || !('warehouseId' in warehouseResponse.data)) {
+          const errMsg = `Warehouse not found: ${warehouseId}`;
+          if (jsonOutput) {
+            console.log(JSON.stringify({ success: false, error: { code: 'NOT_FOUND', message: errMsg } }));
+          } else {
+            console.error(chalk.red(`\n✗ ${errMsg}\n`));
+          }
+          process.exit(1);
+        }
+        warehouse = warehouseResponse.data as Warehouse;
+      } catch (error) {
+        const errMsg = error instanceof Error ? error.message : 'Unknown error';
+        if (jsonOutput) {
+          console.log(JSON.stringify({ success: false, error: { code: 'FETCH_ERROR', message: `Failed to fetch warehouse: ${errMsg}` } }));
+        } else {
+          console.error(chalk.red(`\n✗ Failed to fetch warehouse: ${errMsg}\n`));
+        }
+        process.exit(1);
+      }
+
+      if (!jsonOutput) {
+        console.log(chalk.gray(`  Name: ${warehouse.warehouseName}`));
+        console.log(chalk.gray(`  Code: ${warehouse.warehouseCode}`));
+        if (warehouse.contact) {
+          console.log(chalk.gray(`  Contact: ${warehouse.contact} (${warehouse.phone || 'N/A'})`));
+        }
+        if (warehouse.address) {
+          console.log(chalk.gray(`  Address: ${warehouse.address}`));
+        }
+      }
+
+      if (dryRun) {
+        if (jsonOutput) {
+          console.log(JSON.stringify({ success: true, dryRun: true, warehouse }));
+        } else {
+          console.log(chalk.cyan('\n🔍 Dry-Run Mode: No actual deletion will be made\n'));
+          console.log(chalk.yellow('  To delete, add: --force\n'));
+        }
+        return;
+      }
+
+      if (!options.force) {
+        // TODO: add confirmation prompt here
+        return;
+      }
+
+      try {
+        const response = await apiClient.deleteWarehouse(environment, tenant, warehouseId);
+
+        if (response.success) {
+          if (jsonOutput) {
+            console.log(JSON.stringify({ success: true, warehouseId, name: warehouse.warehouseName }));
+          } else {
+            console.log(chalk.green(`\n✓ Warehouse "${warehouse.warehouseName}" (ID: ${warehouseId}) deleted\n`));
+          }
+        } else {
+          const errorMsg = response.error?.message || 'Unknown error';
+          const errorCode = response.error?.code || 'DELETE_ERROR';
+          if (jsonOutput) {
+            console.log(JSON.stringify({ success: false, error: { code: errorCode, message: errorMsg } }));
+          } else {
+            console.error(chalk.red(`\n✗ Failed to delete warehouse: ${errorMsg}`));
+            if (response.error?.code) {
+              console.error(chalk.gray(`  Error code: ${response.error.code}`));
+            }
+            console.error();
+          }
+          process.exit(1);
+        }
+      } catch (error) {
+        const errMsg = error instanceof Error ? error.message : 'Unknown error';
+        if (jsonOutput) {
+          console.log(JSON.stringify({ success: false, error: { code: 'ERROR', message: errMsg } }));
+        } else {
+          console.error(chalk.red(`\n✗ Error: ${errMsg}\n`));
+        }
+        process.exit(1);
+      }
+    });
+
+  return deleteCommand;
+}