@hubspot/cli 7.6.0-beta.2 → 7.6.0-beta.4

package/commands/customObject/schema/__tests__/update.test.js

@@ -0,0 +1,45 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../../../lib/commonOpts");
+const update_1 = __importDefault(require("../update"));
+vi.mock('yargs');
+vi.mock('../../../../lib/commonOpts');
+describe('commands/customObject/schema/update', () => {
+    let yargsMock = yargs_1.default;
+    beforeEach(() => {
+        yargsMock = {
+            positional: vi.fn().mockReturnThis(),
+            option: vi.fn().mockReturnThis(),
+        };
+    });
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(update_1.default.command).toEqual('update [name]');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(update_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            update_1.default.builder(yargsMock);
+            expect(yargsMock.positional).toHaveBeenCalledTimes(1);
+            expect(yargsMock.positional).toHaveBeenCalledWith('name', {
+                describe: expect.any(String),
+                type: 'string',
+            });
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/fetch.js

@@ -79,4 +79,3 @@ const fetchCommand = {
     builder,
 };
 exports.default = fetchCommand;
-module.exports = fetchCommand;

package/commands/filemanager/__tests__/fetch.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/filemanager/__tests__/fetch.test.js

@@ -0,0 +1,37 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../../lib/commonOpts");
+const fetch_1 = __importDefault(require("../fetch"));
+vi.mock('../../../lib/commonOpts');
+describe('commands/filemanager/fetch', () => {
+    const yargsMock = yargs_1.default;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(fetch_1.default.command).toEqual('fetch <src> [dest]');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(fetch_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            fetch_1.default.builder(yargsMock);
+            expect(commonOpts_1.addGlobalOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addGlobalOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addOverwriteOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addOverwriteOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/filemanager/__tests__/upload.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/filemanager/__tests__/upload.test.js

@@ -0,0 +1,35 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../../lib/commonOpts");
+const upload_1 = __importDefault(require("../upload"));
+vi.mock('../../../lib/commonOpts');
+describe('commands/filemanager/upload', () => {
+    const yargsMock = yargs_1.default;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(upload_1.default.command).toEqual('upload <src> <dest>');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(upload_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            upload_1.default.builder(yargsMock);
+            expect(commonOpts_1.addGlobalOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addGlobalOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/getStarted.js

@@ -128,6 +128,7 @@ async function handler(args) {
         logger_1.uiLogger.log(en_1.commands.getStarted.prompts.projectCreated.description);
         logger_1.uiLogger.log(' ');
         // 5. Install dependencies - Ask user if they want to install dependencies
+        const installLocations = await (0, dependencyManagement_1.getProjectPackageJsonLocations)(projectDest);
         const { shouldInstallDependencies } = await (0, promptUtils_1.promptUser)([
             {
                 type: 'confirm',
@@ -138,24 +139,17 @@ async function handler(args) {
         ]);
         if (shouldInstallDependencies) {
             try {
-                // Change to the project directory to install dependencies
-                const originalCwd = process.cwd();
-                process.chdir(projectDest);
-                try {
-                    await (0, dependencyManagement_1.installPackages)({});
-                    logger_1.uiLogger.log(' ');
-                    logger_1.uiLogger.success(en_1.commands.getStarted.logs.dependenciesInstalled);
-                    logger_1.uiLogger.log(' ');
-                }
-                finally {
-                    // Always restore the original working directory
-                    process.chdir(originalCwd);
-                }
+                await (0, dependencyManagement_1.installPackages)({
+                    installLocations: installLocations,
+                });
+                logger_1.uiLogger.log(' ');
+                logger_1.uiLogger.success(en_1.commands.getStarted.logs.dependenciesInstalled);
+                logger_1.uiLogger.log(' ');
             }
             catch (err) {
                 logger_1.uiLogger.log(' ');
                 logger_1.uiLogger.error(en_1.commands.getStarted.errors.installDepsFailed);
-                console.log(err);
+                (0, errorHandlers_1.logError)(err);
                 logger_1.uiLogger.log(' ');
             }
         }

package/commands/hubdb/__tests__/clear.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/hubdb/__tests__/clear.test.js

@@ -0,0 +1,33 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../../lib/commonOpts");
+const clear_1 = __importDefault(require("../clear"));
+vi.mock('../../../lib/commonOpts');
+describe('commands/hubdb/clear', () => {
+    const yargsMock = yargs_1.default;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(clear_1.default.command).toEqual('clear [table-id]');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(clear_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            clear_1.default.builder(yargsMock);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/hubdb/__tests__/create.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/hubdb/__tests__/create.test.js

@@ -0,0 +1,33 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../../lib/commonOpts");
+const create_1 = __importDefault(require("../create"));
+vi.mock('../../../lib/commonOpts');
+describe('commands/hubdb/create', () => {
+    const yargsMock = yargs_1.default;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(create_1.default.command).toEqual('create');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(create_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            create_1.default.builder(yargsMock);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/hubdb/__tests__/delete.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/hubdb/__tests__/delete.test.js

@@ -0,0 +1,33 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../../lib/commonOpts");
+const delete_1 = __importDefault(require("../delete"));
+vi.mock('../../../lib/commonOpts');
+describe('commands/hubdb/delete', () => {
+    const yargsMock = yargs_1.default;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(delete_1.default.command).toEqual('delete [table-id]');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(delete_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            delete_1.default.builder(yargsMock);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/hubdb/__tests__/fetch.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/hubdb/__tests__/fetch.test.js

@@ -0,0 +1,33 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../../lib/commonOpts");
+const fetch_1 = __importDefault(require("../fetch"));
+vi.mock('../../../lib/commonOpts');
+describe('commands/hubdb/fetch', () => {
+    const yargsMock = yargs_1.default;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(fetch_1.default.command).toEqual('fetch [table-id] [dest]');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(fetch_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            fetch_1.default.builder(yargsMock);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/hubdb/__tests__/list.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/hubdb/__tests__/list.test.js

@@ -0,0 +1,101 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const hubdb_1 = require("@hubspot/local-dev-lib/api/hubdb");
+const commonOpts_1 = require("../../../lib/commonOpts");
+const logger_1 = require("../../../lib/ui/logger");
+const testUtils_1 = require("../../../lib/testUtils");
+const usageTracking_1 = require("../../../lib/usageTracking");
+const list_1 = __importDefault(require("../list"));
+vi.mock('../../../lib/commonOpts');
+vi.mock('@hubspot/local-dev-lib/api/hubdb');
+vi.mock('../../../lib/ui/logger', () => ({
+    uiLogger: {
+        success: vi.fn(),
+        log: vi.fn(),
+        error: vi.fn(),
+        exit: vi.fn(),
+    },
+}));
+vi.mock('../../../lib/errorHandlers');
+vi.mock('../../../lib/usageTracking');
+const mockExit = vi
+    .spyOn(process, 'exit')
+    .mockImplementation(() => undefined);
+describe('commands/hubdb/list', () => {
+    const yargsMock = yargs_1.default;
+    const mockedFetchTables = hubdb_1.fetchTables;
+    const mockedLogger = logger_1.uiLogger;
+    const mockedTrackCommandUsage = usageTracking_1.trackCommandUsage;
+    beforeEach(() => {
+        mockedFetchTables.mockReset();
+        mockedTrackCommandUsage.mockReset();
+        vi.mocked(mockedLogger.success).mockReset();
+        vi.mocked(mockedLogger.log).mockReset();
+        vi.mocked(mockedLogger.error).mockReset();
+    });
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(list_1.default.command).toEqual(['list', 'ls']);
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(list_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            list_1.default.builder(yargsMock);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+    describe('handler', () => {
+        const mockArgs = {
+            derivedAccountId: 123456789,
+            d: false,
+            debug: false,
+            _: [],
+            $0: 'hs',
+        };
+        const mockTables = {
+            results: [
+                {
+                    id: 1,
+                    label: 'Test Table',
+                    name: 'test_table',
+                    columnCount: 5,
+                    rowCount: 10,
+                },
+            ],
+            total: 1,
+        };
+        it('should fetch and display tables', async () => {
+            mockedFetchTables.mockReturnValue((0, testUtils_1.mockHubSpotHttpResponse)(mockTables));
+            await list_1.default.handler(mockArgs);
+            expect(mockedFetchTables).toHaveBeenCalledTimes(1);
+            expect(mockedFetchTables).toHaveBeenCalledWith(mockArgs.derivedAccountId);
+            expect(mockedLogger.success).toHaveBeenCalled();
+            expect(mockedLogger.log).toHaveBeenCalled();
+            expect(mockExit).toHaveBeenCalledWith(0);
+        });
+        it('should handle errors appropriately', async () => {
+            const error = new Error('Test error');
+            mockedFetchTables.mockRejectedValue(error);
+            // this is a hack because the system exit is being mocked, so the return value from fetchTables is weird and can't be unpacked by the handler
+            try {
+                await list_1.default.handler(mockArgs);
+            }
+            catch (e) { }
+            expect(mockExit).toHaveBeenCalledWith(1);
+        });
+    });
+});

package/commands/logs.js

@@ -127,4 +127,3 @@ const logsCommand = {
     handler,
 };
 exports.default = logsCommand;
-module.exports = logsCommand;

package/commands/mcp/__tests__/setup.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/mcp/__tests__/setup.test.js

@@ -0,0 +1,31 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const setup_1 = __importDefault(require("../setup"));
+vi.mock('../../../lib/commonOpts');
+describe('commands/mcp/setup', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(setup_1.default.command).toEqual(['setup', 'update']);
+        });
+    });
+    describe('describe', () => {
+        it('should be undefined to keep the command hidden', () => {
+            expect(setup_1.default.describe).toBeUndefined();
+        });
+    });
+    describe('builder', () => {
+        it('should be defined as a function', () => {
+            expect(setup_1.default.builder).toBeDefined();
+            expect(typeof setup_1.default.builder).toBe('function');
+        });
+    });
+    describe('handler', () => {
+        it('should be defined', () => {
+            expect(setup_1.default.handler).toBeDefined();
+            expect(typeof setup_1.default.handler).toBe('function');
+        });
+    });
+});

package/commands/mcp/__tests__/start.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/mcp/__tests__/start.test.js

@@ -0,0 +1,32 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const start_1 = __importDefault(require("../start"));
+vi.mock('yargs');
+vi.mock('../../../lib/commonOpts');
+describe('commands/mcp/start', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(start_1.default.command).toEqual('start');
+        });
+    });
+    describe('describe', () => {
+        it('should be undefined to keep the command hidden', () => {
+            expect(start_1.default.describe).toBeUndefined();
+        });
+    });
+    describe('builder', () => {
+        it('should be defined as a function', () => {
+            expect(start_1.default.builder).toBeDefined();
+            expect(typeof start_1.default.builder).toBe('function');
+        });
+    });
+    describe('handler', () => {
+        it('should be defined', () => {
+            expect(start_1.default.handler).toBeDefined();
+            expect(typeof start_1.default.handler).toBe('function');
+        });
+    });
+});

package/commands/mcp/setup.js

@@ -37,6 +37,7 @@ function setupBuilder(yargs) {
     })
         .option('add-docs-search', {
         type: 'boolean',
+        hidden: true,
     });
     return yargs;
 }

package/commands/mcp/start.d.ts

@@ -1,3 +1,6 @@
 import { CommonArgs, YargsCommandModule } from '../../types/Yargs';
-declare const mcpStartCommand: YargsCommandModule<unknown, CommonArgs>;
+interface McpStartArgs extends CommonArgs {
+    aiAgent: string;
+}
+declare const mcpStartCommand: YargsCommandModule<unknown, McpStartArgs>;
 export default mcpStartCommand;

package/commands/mcp/start.js

@@ -24,9 +24,9 @@ async function handler(args) {
         process.exit(exitCodes_1.EXIT_CODES.ERROR);
     }
     (0, usageTracking_1.trackCommandUsage)('mcp-start', {}, args.derivedAccountId);
-    await startMcpServer();
+    await startMcpServer(args.aiAgent);
 }
-async function startMcpServer() {
+async function startMcpServer(aiAgent) {
     try {
         const serverPath = path_1.default.join(__dirname, '..', '..', 'mcp-server', 'server.js');
         // Check if server file exists
@@ -36,11 +36,13 @@ async function startMcpServer() {
         }
         logger_1.uiLogger.info(en_1.commands.mcp.start.startingServer);
         logger_1.uiLogger.info(en_1.commands.mcp.start.stopInstructions);
+        const args = [serverPath];
         // Start the server using ts-node
-        const child = (0, child_process_1.spawn)('node', [serverPath], {
+        const child = (0, child_process_1.spawn)(`node`, args, {
             stdio: 'inherit',
             env: {
                 ...process.env,
+                HUBSPOT_MCP_AI_AGENT: aiAgent || 'unknown',
             },
         });
         // Handle server process events
@@ -63,6 +65,9 @@ async function startMcpServer() {
     }
 }
 function startBuilder(yargs) {
+    yargs.option('ai-agent', {
+        type: 'string',
+    });
     return yargs;
 }
 const builder = (0, yargsUtils_1.makeYargsBuilder)(startBuilder, command, describe, {

package/commands/open.js

@@ -8,16 +8,18 @@ const exitCodes_1 = require("../lib/enums/exitCodes");
 const yargsUtils_1 = require("../lib/yargsUtils");
 const separator = ' => ';
 async function createListPrompt(accountId) {
-    return (0, promptUtils_1.promptUser)([
-        {
-            type: 'rawlist',
-            name: 'open',
-            pageSize: 20,
-            message: (0, lang_1.i18n)('commands.open.selectLink'),
-            choices: (0, links_1.getSiteLinksAsArray)(accountId).map(l => `${l.shortcut}${separator}${l.url}`),
-            filter: val => val.split(separator)[0],
-        },
-    ]);
+    return (0, promptUtils_1.promptUser)({
+        name: 'open',
+        type: 'rawlist',
+        pageSize: 20,
+        message: (0, lang_1.i18n)('commands.open.selectLink'),
+        choices: (0, links_1.getSiteLinksAsArray)(accountId)
+            .filter(l => !!l.url)
+            .map(l => ({
+            name: `${l.shortcut}${separator}${l.url}`,
+            value: l.shortcut,
+        })),
+    });
 }
 const command = 'open [shortcut]';
 const describe = (0, lang_1.i18n)('commands.open.describe');
@@ -25,8 +27,8 @@ async function handler(args) {
     const { shortcut, list, derivedAccountId } = args;
     (0, usageTracking_1.trackCommandUsage)('open', undefined, derivedAccountId);
     if (shortcut === undefined && !list) {
-        const choice = await createListPrompt(derivedAccountId);
-        (0, links_1.openLink)(derivedAccountId, choice.open);
+        const { open } = await createListPrompt(derivedAccountId);
+        (0, links_1.openLink)(derivedAccountId, open);
     }
     else if (list) {
         (0, links_1.logSiteLinks)(derivedAccountId);

package/commands/project/__tests__/add.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/project/__tests__/add.test.js

@@ -0,0 +1,48 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const add_1 = __importDefault(require("../add"));
+const constants_1 = require("../../../lib/constants");
+vi.mock('../../../lib/commonOpts');
+describe('commands/project/add', () => {
+    const yargsMock = yargs_1.default;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(add_1.default.command).toEqual('add');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(add_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            add_1.default.builder(yargsMock);
+            expect(yargsMock.example).toHaveBeenCalledTimes(2);
+            expect(yargsMock.options).toHaveBeenCalledWith({
+                auth: expect.objectContaining({
+                    type: 'string',
+                    choices: expect.arrayContaining([constants_1.staticAuth, constants_1.oAuth]),
+                }),
+                distribution: expect.objectContaining({
+                    type: 'string',
+                    choices: expect.arrayContaining([
+                        constants_1.privateDistribution,
+                        constants_1.marketplaceDistribution,
+                    ]),
+                }),
+                features: expect.objectContaining({
+                    type: 'array',
+                }),
+                name: expect.objectContaining({
+                    type: 'string',
+                }),
+                type: expect.objectContaining({ type: 'string' }),
+            });
+        });
+    });
+});