@capawesome/cli 1.14.0-dev.1f912c9.1755635879 → 1.14.0-dev.3b0fc7e.1755934102

package/README.md

@@ -1,5 +1,9 @@
 # cli
 
+[![npm version](https://img.shields.io/npm/v/@capawesome/cli)](https://www.npmjs.com/package/@capawesome/cli)
+[![npm downloads](https://img.shields.io/npm/dm/@capawesome/cli)](https://www.npmjs.com/package/@capawesome/cli)
+[![license](https://img.shields.io/npm/l/@capawesome/cli)](https://github.com/capawesome-team/cli/blob/main/LICENSE)
+
 💻 The Capawesome Cloud Command Line Interface (CLI) can be used to manage [Live Updates](https://capawesome.io/cloud/) from the command line.
 
 ## Installation

package/dist/commands/apps/bundles/create.js

@@ -1,7 +1,3 @@
-import { defineCommand, defineOptions } from '@robingenz/zli';
-import consola from 'consola';
-import { createReadStream } from 'fs';
-import { z } from 'zod';
 import { MAX_CONCURRENT_UPLOADS } from '../../../config/index.js';
 import appBundleFilesService from '../../../services/app-bundle-files.js';
 import appBundlesService from '../../../services/app-bundles.js';
@@ -9,13 +5,17 @@ import appsService from '../../../services/apps.js';
 import authorizationService from '../../../services/authorization-service.js';
 import organizationsService from '../../../services/organizations.js';
 import { createBufferFromPath, createBufferFromReadStream, createBufferFromString, isPrivateKeyContent, } from '../../../utils/buffer.js';
-import { getMessageFromUnknownError } from '../../../utils/error.js';
+import { formatPrivateKey } from '../../../utils/private-key.js';
 import { fileExistsAtPath, getFilesInDirectoryAndSubdirectories, isDirectory } from '../../../utils/file.js';
 import { createHash } from '../../../utils/hash.js';
 import { generateManifestJson } from '../../../utils/manifest.js';
 import { prompt } from '../../../utils/prompt.js';
 import { createSignature } from '../../../utils/signature.js';
 import zip from '../../../utils/zip.js';
+import { defineCommand, defineOptions } from '@robingenz/zli';
+import consola from 'consola';
+import { createReadStream } from 'fs';
+import { z } from 'zod';
 export default defineCommand({
     description: 'Create a new app bundle.',
     options: defineOptions(z.object({
@@ -194,13 +194,17 @@ export default defineCommand({
         if (privateKey) {
             if (isPrivateKeyContent(privateKey)) {
                 // Handle plain text private key content
-                privateKeyBuffer = createBufferFromString(privateKey);
+                const formattedPrivateKey = formatPrivateKey(privateKey);
+                privateKeyBuffer = createBufferFromString(formattedPrivateKey);
             }
             else if (privateKey.endsWith('.pem')) {
                 // Handle file path
                 const fileExists = await fileExistsAtPath(privateKey);
                 if (fileExists) {
-                    privateKeyBuffer = await createBufferFromPath(privateKey);
+                    const keyBuffer = await createBufferFromPath(privateKey);
+                    const keyContent = keyBuffer.toString('utf8');
+                    const formattedPrivateKey = formatPrivateKey(keyContent);
+                    privateKeyBuffer = createBufferFromString(formattedPrivateKey);
                 }
                 else {
                     consola.error('Private key file not found.');
@@ -285,9 +289,7 @@ export default defineCommand({
                     // No-op
                 });
             }
-            const message = getMessageFromUnknownError(error);
-            consola.error(message);
-            process.exit(1);
+            throw error;
         }
     },
 });

package/dist/commands/apps/bundles/create.test.js

@@ -1,17 +1,18 @@
 import { DEFAULT_API_BASE_URL } from '../../../config/consts.js';
 import authorizationService from '../../../services/authorization-service.js';
 import { fileExistsAtPath, isDirectory } from '../../../utils/file.js';
-import userConfig from '../../../utils/userConfig.js';
+import userConfig from '../../../utils/user-config.js';
 import consola from 'consola';
 import nock from 'nock';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import createBundleCommand from './create.js';
 // Mock dependencies
-vi.mock('@/utils/userConfig.js');
+vi.mock('@/utils/user-config.js');
 vi.mock('@/services/authorization-service.js');
 vi.mock('@/utils/file.js');
 vi.mock('@/utils/zip.js');
 vi.mock('@/utils/buffer.js');
+vi.mock('@/utils/private-key.js');
 vi.mock('@/utils/hash.js');
 vi.mock('@/utils/signature.js');
 vi.mock('consola');
@@ -26,7 +27,9 @@ describe('apps-bundles-create', () => {
         mockUserConfig.read.mockReturnValue({ token: 'test-token' });
         mockAuthorizationService.hasAuthorizationToken.mockReturnValue(true);
         mockAuthorizationService.getCurrentAuthorizationToken.mockReturnValue('test-token');
-        vi.spyOn(process, 'exit').mockImplementation(() => undefined);
+        vi.spyOn(process, 'exit').mockImplementation((code) => {
+            throw new Error(`Process exited with code ${code}`);
+        });
     });
     afterEach(() => {
         nock.cleanAll();
@@ -36,9 +39,8 @@ describe('apps-bundles-create', () => {
         const appId = 'app-123';
         const options = { appId, path: './dist', artifactType: 'zip', rollout: 1 };
         mockAuthorizationService.hasAuthorizationToken.mockReturnValue(false);
-        await createBundleCommand.action(options, undefined);
+        await expect(createBundleCommand.action(options, undefined)).rejects.toThrow('Process exited with code 1');
         expect(mockConsola.error).toHaveBeenCalledWith('You must be logged in to run this command.');
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
     it('should create bundle with self-hosted URL', async () => {
         const appId = 'app-123';
@@ -84,9 +86,8 @@ describe('apps-bundles-create', () => {
         const nonexistentPath = './nonexistent';
         const options = { appId, path: nonexistentPath, artifactType: 'zip', rollout: 1 };
         mockFileExistsAtPath.mockResolvedValue(false);
-        await createBundleCommand.action(options, undefined);
+        await expect(createBundleCommand.action(options, undefined)).rejects.toThrow('Process exited with code 1');
         expect(mockConsola.error).toHaveBeenCalledWith('The path does not exist.');
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
     it('should validate manifest artifact type requires directory', async () => {
         const appId = 'app-123';
@@ -99,9 +100,8 @@ describe('apps-bundles-create', () => {
         };
         mockFileExistsAtPath.mockResolvedValue(true);
         mockIsDirectory.mockResolvedValue(false);
-        await createBundleCommand.action(options, undefined);
+        await expect(createBundleCommand.action(options, undefined)).rejects.toThrow('Process exited with code 1');
         expect(mockConsola.error).toHaveBeenCalledWith('The path must be a folder when creating a bundle with an artifact type of `manifest`.');
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
     it('should validate manifest artifact type cannot use URL', async () => {
         const appId = 'app-123';
@@ -112,9 +112,8 @@ describe('apps-bundles-create', () => {
             artifactType: 'manifest',
             rollout: 1,
         };
-        await createBundleCommand.action(options, undefined);
+        await expect(createBundleCommand.action(options, undefined)).rejects.toThrow('Process exited with code 1');
         expect(mockConsola.error).toHaveBeenCalledWith('It is not yet possible to provide a URL when creating a bundle with an artifact type of `manifest`.');
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
     it('should handle API error during creation', async () => {
         const appId = 'app-123';
@@ -130,10 +129,8 @@ describe('apps-bundles-create', () => {
             .post(`/v1/apps/${appId}/bundles`)
             .matchHeader('Authorization', `Bearer ${testToken}`)
             .reply(400, { message: 'Invalid bundle data' });
-        await createBundleCommand.action(options, undefined);
+        await expect(createBundleCommand.action(options, undefined)).rejects.toThrow();
         expect(scope.isDone()).toBe(true);
-        expect(mockConsola.error).toHaveBeenCalled();
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
     it('should handle private key file path', async () => {
         const appId = 'app-123';
@@ -164,11 +161,14 @@ describe('apps-bundles-create', () => {
         // Mock utility functions
         const mockZip = await import('../../../utils/zip.js');
         const mockBuffer = await import('../../../utils/buffer.js');
+        const mockPrivateKey = await import('../../../utils/private-key.js');
         const mockHash = await import('../../../utils/hash.js');
         const mockSignature = await import('../../../utils/signature.js');
         vi.mocked(mockZip.default.isZipped).mockReturnValue(true);
         vi.mocked(mockBuffer.createBufferFromPath).mockResolvedValue(testBuffer);
         vi.mocked(mockBuffer.isPrivateKeyContent).mockReturnValue(false);
+        vi.mocked(mockPrivateKey.formatPrivateKey).mockReturnValue('formatted-private-key');
+        vi.mocked(mockBuffer.createBufferFromString).mockReturnValue(testBuffer);
         vi.mocked(mockHash.createHash).mockResolvedValue(testHash);
         vi.mocked(mockSignature.createSignature).mockResolvedValue(testSignature);
         const scope = nock(DEFAULT_API_BASE_URL)
@@ -209,12 +209,14 @@ describe('apps-bundles-create', () => {
         // Mock utility functions
         const mockZip = await import('../../../utils/zip.js');
         const mockBuffer = await import('../../../utils/buffer.js');
+        const mockPrivateKey = await import('../../../utils/private-key.js');
         const mockHash = await import('../../../utils/hash.js');
         const mockSignature = await import('../../../utils/signature.js');
         vi.mocked(mockZip.default.isZipped).mockReturnValue(true);
         vi.mocked(mockBuffer.createBufferFromPath).mockResolvedValue(testBuffer);
         vi.mocked(mockBuffer.createBufferFromString).mockReturnValue(testBuffer);
         vi.mocked(mockBuffer.isPrivateKeyContent).mockReturnValue(true);
+        vi.mocked(mockPrivateKey.formatPrivateKey).mockReturnValue('formatted-private-key');
         vi.mocked(mockHash.createHash).mockResolvedValue(testHash);
         vi.mocked(mockSignature.createSignature).mockResolvedValue(testSignature);
         const scope = nock(DEFAULT_API_BASE_URL)
@@ -250,9 +252,8 @@ describe('apps-bundles-create', () => {
         // Mock utility functions
         const mockBuffer = await import('../../../utils/buffer.js');
         vi.mocked(mockBuffer.isPrivateKeyContent).mockReturnValue(false);
-        await createBundleCommand.action(options, undefined);
+        await expect(createBundleCommand.action(options, undefined)).rejects.toThrow('Process exited with code 1');
         expect(mockConsola.error).toHaveBeenCalledWith('Private key file not found.');
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
     it('should handle invalid private key format', async () => {
         const appId = 'app-123';
@@ -264,11 +265,12 @@ describe('apps-bundles-create', () => {
             artifactType: 'zip',
             rollout: 1,
         };
+        mockFileExistsAtPath.mockResolvedValue(true);
+        mockIsDirectory.mockResolvedValue(false);
         // Mock utility functions
         const mockBuffer = await import('../../../utils/buffer.js');
         vi.mocked(mockBuffer.isPrivateKeyContent).mockReturnValue(false);
-        await createBundleCommand.action(options, undefined);
+        await expect(createBundleCommand.action(options, undefined)).rejects.toThrow('Process exited with code 1');
         expect(mockConsola.error).toHaveBeenCalledWith('Private key must be either a path to a .pem file or the private key content as plain text.');
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
 });

package/dist/commands/apps/bundles/delete.js

@@ -1,11 +1,11 @@
-import { defineCommand, defineOptions } from '@robingenz/zli';
-import consola from 'consola';
-import { z } from 'zod';
 import appBundlesService from '../../../services/app-bundles.js';
 import appsService from '../../../services/apps.js';
+import authorizationService from '../../../services/authorization-service.js';
 import organizationsService from '../../../services/organizations.js';
-import { getMessageFromUnknownError } from '../../../utils/error.js';
 import { prompt } from '../../../utils/prompt.js';
+import { defineCommand, defineOptions } from '@robingenz/zli';
+import consola from 'consola';
+import { z } from 'zod';
 export default defineCommand({
     description: 'Delete an app bundle.',
     options: defineOptions(z.object({
@@ -14,6 +14,10 @@ export default defineCommand({
     })),
     action: async (options, args) => {
         let { appId, bundleId } = options;
+        if (!authorizationService.hasAuthorizationToken()) {
+            consola.error('You must be logged in to run this command.');
+            process.exit(1);
+        }
         // Prompt for missing arguments
         if (!appId) {
             const organizations = await organizationsService.findAll();
@@ -56,17 +60,10 @@ export default defineCommand({
             return;
         }
         // Delete bundle
-        try {
-            await appBundlesService.delete({
-                appId,
-                appBundleId: bundleId,
-            });
-            consola.success('Bundle deleted successfully.');
-        }
-        catch (error) {
-            const message = getMessageFromUnknownError(error);
-            consola.error(message);
-            process.exit(1);
-        }
+        await appBundlesService.delete({
+            appId,
+            appBundleId: bundleId,
+        });
+        consola.success('Bundle deleted successfully.');
     },
 });

package/dist/commands/apps/bundles/delete.test.js

@@ -1,13 +1,13 @@
 import { DEFAULT_API_BASE_URL } from '../../../config/consts.js';
 import authorizationService from '../../../services/authorization-service.js';
 import { prompt } from '../../../utils/prompt.js';
-import userConfig from '../../../utils/userConfig.js';
+import userConfig from '../../../utils/user-config.js';
 import consola from 'consola';
 import nock from 'nock';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import deleteBundleCommand from './delete.js';
 // Mock dependencies
-vi.mock('@/utils/userConfig.js');
+vi.mock('@/utils/user-config.js');
 vi.mock('@/utils/prompt.js');
 vi.mock('@/services/authorization-service.js');
 vi.mock('consola');
@@ -20,6 +20,7 @@ describe('apps-bundles-delete', () => {
         vi.clearAllMocks();
         mockUserConfig.read.mockReturnValue({ token: 'test-token' });
         mockAuthorizationService.getCurrentAuthorizationToken.mockReturnValue('test-token');
+        mockAuthorizationService.hasAuthorizationToken.mockReturnValue(true);
         vi.spyOn(process, 'exit').mockImplementation(() => undefined);
     });
     afterEach(() => {
@@ -132,9 +133,7 @@ describe('apps-bundles-delete', () => {
             .delete(`/v1/apps/${appId}/bundles/${bundleId}`)
             .matchHeader('Authorization', `Bearer ${testToken}`)
             .reply(404, { message: 'Bundle not found' });
-        await deleteBundleCommand.action(options, undefined);
+        await expect(deleteBundleCommand.action(options, undefined)).rejects.toThrow();
         expect(scope.isDone()).toBe(true);
-        expect(mockConsola.error).toHaveBeenCalled();
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
 });

package/dist/commands/apps/bundles/update.js

@@ -1,12 +1,11 @@
-import { defineCommand, defineOptions } from '@robingenz/zli';
-import consola from 'consola';
-import { z } from 'zod';
 import appBundlesService from '../../../services/app-bundles.js';
 import appsService from '../../../services/apps.js';
 import authorizationService from '../../../services/authorization-service.js';
 import organizationsService from '../../../services/organizations.js';
-import { getMessageFromUnknownError } from '../../../utils/error.js';
 import { prompt } from '../../../utils/prompt.js';
+import { defineCommand, defineOptions } from '@robingenz/zli';
+import consola from 'consola';
+import { z } from 'zod';
 export default defineCommand({
     description: 'Update an app bundle.',
     options: defineOptions(z.object({
@@ -78,22 +77,15 @@ export default defineCommand({
             });
         }
         // Update bundle
-        try {
-            await appBundlesService.update({
-                appId,
-                appBundleId: bundleId,
-                maxAndroidAppVersionCode: androidMax,
-                maxIosAppVersionCode: iosMax,
-                minAndroidAppVersionCode: androidMin,
-                minIosAppVersionCode: iosMin,
-                rolloutPercentage: rollout,
-            });
-            consola.success('Bundle updated successfully.');
-        }
-        catch (error) {
-            const message = getMessageFromUnknownError(error);
-            consola.error(message);
-            process.exit(1);
-        }
+        await appBundlesService.update({
+            appId,
+            appBundleId: bundleId,
+            maxAndroidAppVersionCode: androidMax,
+            maxIosAppVersionCode: iosMax,
+            minAndroidAppVersionCode: androidMin,
+            minIosAppVersionCode: iosMin,
+            rolloutPercentage: rollout,
+        });
+        consola.success('Bundle updated successfully.');
     },
 });

package/dist/commands/apps/bundles/update.test.js

@@ -1,13 +1,13 @@
 import { DEFAULT_API_BASE_URL } from '../../../config/consts.js';
 import authorizationService from '../../../services/authorization-service.js';
 import { prompt } from '../../../utils/prompt.js';
-import userConfig from '../../../utils/userConfig.js';
+import userConfig from '../../../utils/user-config.js';
 import consola from 'consola';
 import nock from 'nock';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import updateBundleCommand from './update.js';
 // Mock dependencies
-vi.mock('@/utils/userConfig.js');
+vi.mock('@/utils/user-config.js');
 vi.mock('@/utils/prompt.js');
 vi.mock('@/services/authorization-service.js');
 vi.mock('consola');
@@ -21,7 +21,9 @@ describe('apps-bundles-update', () => {
         mockUserConfig.read.mockReturnValue({ token: 'test-token' });
         mockAuthorizationService.hasAuthorizationToken.mockReturnValue(true);
         mockAuthorizationService.getCurrentAuthorizationToken.mockReturnValue('test-token');
-        vi.spyOn(process, 'exit').mockImplementation(() => undefined);
+        vi.spyOn(process, 'exit').mockImplementation((code) => {
+            throw new Error(`Process exited with code ${code}`);
+        });
     });
     afterEach(() => {
         nock.cleanAll();
@@ -32,9 +34,8 @@ describe('apps-bundles-update', () => {
         const bundleId = 'bundle-456';
         const options = { appId, bundleId };
         mockAuthorizationService.hasAuthorizationToken.mockReturnValue(false);
-        await updateBundleCommand.action(options, undefined);
+        await expect(updateBundleCommand.action(options, undefined)).rejects.toThrow('Process exited with code 1');
         expect(mockConsola.error).toHaveBeenCalledWith('You must be logged in to run this command.');
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
     it('should update bundle with provided options', async () => {
         const appId = 'app-123';
@@ -114,10 +115,8 @@ describe('apps-bundles-update', () => {
             .patch(`/v1/apps/${appId}/bundles/${bundleId}`)
             .matchHeader('Authorization', `Bearer ${testToken}`)
             .reply(404, { message: 'Bundle not found' });
-        await updateBundleCommand.action(options, undefined);
+        await expect(updateBundleCommand.action(options, undefined)).rejects.toThrow();
         expect(scope.isDone()).toBe(true);
-        expect(mockConsola.error).toHaveBeenCalled();
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
     it('should handle error when no organizations exist', async () => {
         const testToken = 'test-token';

package/dist/commands/apps/channels/create.js

@@ -1,11 +1,12 @@
-import { defineCommand, defineOptions } from '@robingenz/zli';
-import consola from 'consola';
-import { z } from 'zod';
 import appChannelsService from '../../../services/app-channels.js';
 import appsService from '../../../services/apps.js';
+import authorizationService from '../../../services/authorization-service.js';
 import organizationsService from '../../../services/organizations.js';
 import { getMessageFromUnknownError } from '../../../utils/error.js';
 import { prompt } from '../../../utils/prompt.js';
+import { defineCommand, defineOptions } from '@robingenz/zli';
+import consola from 'consola';
+import { z } from 'zod';
 export default defineCommand({
     description: 'Create a new app channel.',
     options: defineOptions(z.object({
@@ -19,6 +20,10 @@ export default defineCommand({
     })),
     action: async (options, args) => {
         let { appId, bundleLimit, ignoreErrors, name } = options;
+        if (!authorizationService.hasAuthorizationToken()) {
+            consola.error('You must be logged in to run this command.');
+            process.exit(1);
+        }
         // Validate the app ID
         if (!appId) {
             const organizations = await organizationsService.findAll();
@@ -62,9 +67,14 @@ export default defineCommand({
             consola.info(`Channel ID: ${response.id}`);
         }
         catch (error) {
-            const message = getMessageFromUnknownError(error);
-            consola.error(message);
-            process.exit(ignoreErrors ? 0 : 1);
+            if (ignoreErrors) {
+                const message = getMessageFromUnknownError(error);
+                consola.error(message);
+                process.exit(0);
+            }
+            else {
+                throw error;
+            }
         }
     },
 });

package/dist/commands/apps/channels/create.test.js

@@ -1,22 +1,29 @@
 import { DEFAULT_API_BASE_URL } from '../../../config/consts.js';
+import authorizationService from '../../../services/authorization-service.js';
 import { prompt } from '../../../utils/prompt.js';
-import userConfig from '../../../utils/userConfig.js';
+import userConfig from '../../../utils/user-config.js';
 import consola from 'consola';
 import nock from 'nock';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import createChannelCommand from './create.js';
 // Mock dependencies
-vi.mock('@/utils/userConfig.js');
+vi.mock('@/utils/user-config.js');
 vi.mock('@/utils/prompt.js');
+vi.mock('@/services/authorization-service.js');
 vi.mock('consola');
 describe('apps-channels-create', () => {
     const mockUserConfig = vi.mocked(userConfig);
     const mockPrompt = vi.mocked(prompt);
     const mockConsola = vi.mocked(consola);
+    const mockAuthorizationService = vi.mocked(authorizationService);
     beforeEach(() => {
         vi.clearAllMocks();
         mockUserConfig.read.mockReturnValue({ token: 'test-token' });
-        vi.spyOn(process, 'exit').mockImplementation(() => undefined);
+        mockAuthorizationService.getCurrentAuthorizationToken.mockReturnValue('test-token');
+        mockAuthorizationService.hasAuthorizationToken.mockReturnValue(true);
+        vi.spyOn(process, 'exit').mockImplementation((code) => {
+            throw new Error(`Process exited with code ${code}`);
+        });
     });
     afterEach(() => {
         nock.cleanAll();
@@ -96,10 +103,9 @@ describe('apps-channels-create', () => {
             .post('/v1/apps/app-123/channels')
             .matchHeader('Authorization', 'Bearer test-token')
             .reply(400, { message: 'Channel name already exists' });
-        await createChannelCommand.action(options, undefined);
+        await expect(createChannelCommand.action(options, undefined)).rejects.toThrow('Process exited with code 0');
         expect(scope.isDone()).toBe(true);
         expect(mockConsola.error).toHaveBeenCalled();
-        expect(process.exit).toHaveBeenCalledWith(0);
     });
     it('should handle error without ignoreErrors flag', async () => {
         const options = { appId: 'app-123', name: 'production' };
@@ -107,9 +113,7 @@ describe('apps-channels-create', () => {
             .post('/v1/apps/app-123/channels')
             .matchHeader('Authorization', 'Bearer test-token')
             .reply(400, { message: 'Channel name already exists' });
-        await createChannelCommand.action(options, undefined);
+        await expect(createChannelCommand.action(options, undefined)).rejects.toThrow();
         expect(scope.isDone()).toBe(true);
-        expect(mockConsola.error).toHaveBeenCalled();
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
 });

package/dist/commands/apps/channels/delete.js

@@ -1,12 +1,12 @@
-import { defineCommand, defineOptions } from '@robingenz/zli';
-import consola from 'consola';
-import { isCI } from 'std-env';
-import { z } from 'zod';
 import appChannelsService from '../../../services/app-channels.js';
 import appsService from '../../../services/apps.js';
+import authorizationService from '../../../services/authorization-service.js';
 import organizationsService from '../../../services/organizations.js';
-import { getMessageFromUnknownError } from '../../../utils/error.js';
 import { prompt } from '../../../utils/prompt.js';
+import { defineCommand, defineOptions } from '@robingenz/zli';
+import consola from 'consola';
+import { isCI } from 'std-env';
+import { z } from 'zod';
 export default defineCommand({
     description: 'Delete an app channel.',
     options: defineOptions(z.object({
@@ -22,6 +22,10 @@ export default defineCommand({
     })),
     action: async (options, args) => {
         let { appId, channelId, name } = options;
+        if (!authorizationService.hasAuthorizationToken()) {
+            consola.error('You must be logged in to run this command.');
+            process.exit(1);
+        }
         if (!appId) {
             const organizations = await organizationsService.findAll();
             if (organizations.length === 0) {
@@ -63,18 +67,11 @@ export default defineCommand({
                 return;
             }
         }
-        try {
-            await appChannelsService.delete({
-                appId,
-                id: channelId,
-                name,
-            });
-            consola.success('Channel deleted successfully.');
-        }
-        catch (error) {
-            const message = getMessageFromUnknownError(error);
-            consola.error(message);
-            process.exit(1);
-        }
+        await appChannelsService.delete({
+            appId,
+            id: channelId,
+            name,
+        });
+        consola.success('Channel deleted successfully.');
     },
 });

package/dist/commands/apps/channels/delete.test.js

@@ -1,13 +1,13 @@
 import { DEFAULT_API_BASE_URL } from '../../../config/consts.js';
 import authorizationService from '../../../services/authorization-service.js';
 import { prompt } from '../../../utils/prompt.js';
-import userConfig from '../../../utils/userConfig.js';
+import userConfig from '../../../utils/user-config.js';
 import consola from 'consola';
 import nock from 'nock';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import deleteChannelCommand from './delete.js';
 // Mock dependencies
-vi.mock('@/utils/userConfig.js');
+vi.mock('@/utils/user-config.js');
 vi.mock('@/utils/prompt.js');
 vi.mock('@/services/authorization-service.js');
 vi.mock('consola');
@@ -23,7 +23,10 @@ describe('apps-channels-delete', () => {
         vi.clearAllMocks();
         mockUserConfig.read.mockReturnValue({ token: 'test-token' });
         mockAuthorizationService.getCurrentAuthorizationToken.mockReturnValue('test-token');
-        vi.spyOn(process, 'exit').mockImplementation(() => undefined);
+        mockAuthorizationService.hasAuthorizationToken.mockReturnValue(true);
+        vi.spyOn(process, 'exit').mockImplementation((code) => {
+            throw new Error(`Process exited with code ${code}`);
+        });
     });
     afterEach(() => {
         nock.cleanAll();
@@ -132,9 +135,7 @@ describe('apps-channels-delete', () => {
             .delete(`/v1/apps/${appId}/channels/${channelId}`)
             .matchHeader('Authorization', `Bearer ${testToken}`)
             .reply(404, { message: 'Channel not found' });
-        await deleteChannelCommand.action(options, undefined);
+        await expect(deleteChannelCommand.action(options, undefined)).rejects.toThrow();
         expect(scope.isDone()).toBe(true);
-        expect(mockConsola.error).toHaveBeenCalled();
-        expect(process.exit).toHaveBeenCalledWith(1);
     });
 });