@hubspot/cli 7.7.21-experimental.1 → 7.7.22-experimental.0

package/commands/__tests__/function.test.js

@@ -1,50 +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 list_1 = __importDefault(require("../function/list"));
-const deploy_1 = __importDefault(require("../function/deploy"));
-const server_1 = __importDefault(require("../function/server"));
-const function_1 = __importDefault(require("../function"));
+import yargs from 'yargs';
+import list from '../function/list.js';
+import deploy from '../function/deploy.js';
+import server from '../function/server.js';
+import functionCommands from '../function.js';
 vi.mock('../function/list');
 vi.mock('../function/deploy');
 vi.mock('../function/server');
 vi.mock('../../lib/commonOpts');
 const commandSpy = vi
-    .spyOn(yargs_1.default, 'command')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'command')
+    .mockReturnValue(yargs);
 const demandCommandSpy = vi
-    .spyOn(yargs_1.default, 'demandCommand')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'demandCommand')
+    .mockReturnValue(yargs);
 describe('commands/function', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(function_1.default.command).toEqual(['function', 'functions']);
+            expect(functionCommands.command).toEqual(['function', 'functions']);
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(function_1.default.describe).toBeDefined();
+            expect(functionCommands.describe).toBeDefined();
         });
     });
     describe('builder', () => {
-        const subcommands = [list_1.default, deploy_1.default, server_1.default];
+        const subcommands = [list, deploy, server];
         it('should demand the command takes one positional argument', () => {
-            function_1.default.builder(yargs_1.default);
+            functionCommands.builder(yargs);
             expect(demandCommandSpy).toHaveBeenCalledTimes(1);
             expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
         });
         it('should support the correct options', () => {
-            function_1.default.builder(yargs_1.default);
+            functionCommands.builder(yargs);
         });
         it('should add the correct number of sub commands', () => {
-            function_1.default.builder(yargs_1.default);
+            functionCommands.builder(yargs);
             expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
         });
         it.each(subcommands)('should attach the %s subcommand', module => {
-            function_1.default.builder(yargs_1.default);
+            functionCommands.builder(yargs);
             expect(commandSpy).toHaveBeenCalledWith(module);
         });
     });

package/commands/__tests__/getStarted.test.js

@@ -1,18 +1,13 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-const getStarted_1 = __importDefault(require("../getStarted"));
-const usageTracking_1 = require("../../lib/usageTracking");
-const promptUtils_1 = require("../../lib/prompts/promptUtils");
-const projectNameAndDestPrompt_1 = require("../../lib/prompts/projectNameAndDestPrompt");
-const github_1 = require("@hubspot/local-dev-lib/github");
-const config_1 = require("../../lib/projects/config");
-const dependencyManagement_1 = require("../../lib/dependencyManagement");
-const constants_1 = require("../../lib/constants");
-const exitCodes_1 = require("../../lib/enums/exitCodes");
-const open_1 = __importDefault(require("open"));
+import getStartedCommand from '../getStarted.js';
+import { trackCommandUsage } from '../../lib/usageTracking.js';
+import { promptUser } from '../../lib/prompts/promptUtils.js';
+import { projectNameAndDestPrompt } from '../../lib/prompts/projectNameAndDestPrompt.js';
+import { cloneGithubRepo } from '@hubspot/local-dev-lib/github';
+import { getProjectConfig, writeProjectConfig, } from '../../lib/projects/config.js';
+import { getProjectPackageJsonLocations, installPackages, } from '../../lib/dependencyManagement.js';
+import { GET_STARTED_OPTIONS } from '../../lib/constants.js';
+import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
+import open from 'open';
 vi.mock('../../lib/usageTracking');
 vi.mock('../../lib/prompts/promptUtils');
 vi.mock('../../lib/prompts/projectNameAndDestPrompt');
@@ -35,22 +30,22 @@ describe('commands/get-started', () => {
     });
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(getStarted_1.default.command).toEqual('get-started');
+            expect(getStartedCommand.command).toEqual('get-started');
         });
     });
     describe('describe', () => {
         it('should have undefined describe property', () => {
-            expect(getStarted_1.default.describe).toBeUndefined();
+            expect(getStartedCommand.describe).toBeUndefined();
         });
     });
     describe('command structure', () => {
         it('should have handler function', () => {
-            expect(getStarted_1.default.handler).toBeDefined();
-            expect(typeof getStarted_1.default.handler).toBe('function');
+            expect(getStartedCommand.handler).toBeDefined();
+            expect(typeof getStartedCommand.handler).toBe('function');
         });
         it('should have builder function', () => {
-            expect(getStarted_1.default.builder).toBeDefined();
-            expect(typeof getStarted_1.default.builder).toBe('function');
+            expect(getStartedCommand.builder).toBeDefined();
+            expect(typeof getStartedCommand.builder).toBe('function');
         });
     });
     describe('handler', () => {
@@ -72,62 +67,62 @@ describe('commands/get-started', () => {
         };
         beforeEach(() => {
             vi.clearAllMocks();
-            usageTracking_1.trackCommandUsage.mockResolvedValue(undefined);
-            projectNameAndDestPrompt_1.projectNameAndDestPrompt.mockResolvedValue({
+            trackCommandUsage.mockResolvedValue(undefined);
+            projectNameAndDestPrompt.mockResolvedValue({
                 dest: 'test-destination',
                 name: 'test-project',
             });
-            github_1.cloneGithubRepo.mockResolvedValue(true);
-            config_1.getProjectConfig.mockResolvedValue({
+            cloneGithubRepo.mockResolvedValue(true);
+            getProjectConfig.mockResolvedValue({
                 projectConfig: null,
                 projectDir: null,
             });
-            config_1.writeProjectConfig.mockResolvedValue(true);
-            dependencyManagement_1.getProjectPackageJsonLocations.mockResolvedValue(['/path/to/package/dir']);
-            dependencyManagement_1.installPackages.mockResolvedValue(undefined);
+            writeProjectConfig.mockResolvedValue(true);
+            getProjectPackageJsonLocations.mockResolvedValue(['/path/to/package/dir']);
+            installPackages.mockResolvedValue(undefined);
             process.exit = vi.fn();
         });
         describe('CMS flow', () => {
             it('should handle CMS option selection', async () => {
-                promptUtils_1.promptUser.mockResolvedValue({
-                    default: constants_1.GET_STARTED_OPTIONS.CMS,
+                promptUser.mockResolvedValue({
+                    default: GET_STARTED_OPTIONS.CMS,
                 });
-                await getStarted_1.default.handler(mockArgs);
-                expect(process.exit).toHaveBeenCalledWith(exitCodes_1.EXIT_CODES.SUCCESS);
+                await getStartedCommand.handler(mockArgs);
+                expect(process.exit).toHaveBeenCalledWith(EXIT_CODES.SUCCESS);
             });
             it('should open Design Manager when user confirms and browser is available', async () => {
-                promptUtils_1.promptUser
-                    .mockResolvedValueOnce({ default: constants_1.GET_STARTED_OPTIONS.CMS })
+                promptUser
+                    .mockResolvedValueOnce({ default: GET_STARTED_OPTIONS.CMS })
                     .mockResolvedValueOnce({ shouldOpen: true });
-                await getStarted_1.default.handler(mockArgs);
-                expect(open_1.default).toHaveBeenCalledWith(expect.stringContaining('design-manager'), { url: true });
+                await getStartedCommand.handler(mockArgs);
+                expect(open).toHaveBeenCalledWith(expect.stringContaining('design-manager'), { url: true });
             });
         });
         describe('App flow', () => {
             beforeEach(() => {
-                promptUtils_1.promptUser
+                promptUser
                     .mockResolvedValueOnce({
-                    default: constants_1.GET_STARTED_OPTIONS.APP,
+                    default: GET_STARTED_OPTIONS.APP,
                 })
                     .mockResolvedValueOnce({
                     shouldUpload: true,
                 });
             });
             it('should handle App option selection', async () => {
-                await getStarted_1.default.handler(mockArgs);
-                expect(projectNameAndDestPrompt_1.projectNameAndDestPrompt).toHaveBeenCalledWith(mockArgs);
-                expect(github_1.cloneGithubRepo).toHaveBeenCalled();
+                await getStartedCommand.handler(mockArgs);
+                expect(projectNameAndDestPrompt).toHaveBeenCalledWith(mockArgs);
+                expect(cloneGithubRepo).toHaveBeenCalled();
             });
             it('should handle upload flow when user confirms', async () => {
-                promptUtils_1.promptUser
-                    .mockResolvedValueOnce({ default: constants_1.GET_STARTED_OPTIONS.APP })
+                promptUser
+                    .mockResolvedValueOnce({ default: GET_STARTED_OPTIONS.APP })
                     .mockResolvedValueOnce({ shouldUpload: true });
                 const mockProjectConfig = {
                     name: 'test-project',
                     srcDir: 'src',
                     platformVersion: '1.0.0',
                 };
-                config_1.getProjectConfig
+                getProjectConfig
                     .mockResolvedValueOnce({
                     projectConfig: null,
                     projectDir: null,
@@ -136,8 +131,8 @@ describe('commands/get-started', () => {
                     projectConfig: mockProjectConfig,
                     projectDir: '/path/to/project',
                 });
-                await getStarted_1.default.handler(mockArgs);
-                expect(promptUtils_1.promptUser).toHaveBeenCalledWith([
+                await getStartedCommand.handler(mockArgs);
+                expect(promptUser).toHaveBeenCalledWith([
                     expect.objectContaining({
                         type: 'confirm',
                         name: 'shouldUpload',
@@ -149,11 +144,11 @@ describe('commands/get-started', () => {
         });
         describe('tracking', () => {
             it('should track command usage', async () => {
-                promptUtils_1.promptUser.mockResolvedValue({
-                    default: constants_1.GET_STARTED_OPTIONS.APP,
+                promptUser.mockResolvedValue({
+                    default: GET_STARTED_OPTIONS.APP,
                 });
-                await getStarted_1.default.handler(mockArgs);
-                expect(usageTracking_1.trackCommandUsage).toHaveBeenCalledWith('get-started', {}, mockArgs.derivedAccountId);
+                await getStartedCommand.handler(mockArgs);
+                expect(trackCommandUsage).toHaveBeenCalledWith('get-started', {}, mockArgs.derivedAccountId);
             });
         });
     });

package/commands/__tests__/hubdb.test.js

@@ -1,35 +1,30 @@
-"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 clear_1 = __importDefault(require("../hubdb/clear"));
-const create_1 = __importDefault(require("../hubdb/create"));
-const delete_1 = __importDefault(require("../hubdb/delete"));
-const fetch_1 = __importDefault(require("../hubdb/fetch"));
-const list_1 = __importDefault(require("../hubdb/list"));
-const hubdb_1 = __importDefault(require("../hubdb"));
+import yargs from 'yargs';
+import clear from '../hubdb/clear.js';
+import create from '../hubdb/create.js';
+import deleteCommand from '../hubdb/delete.js';
+import fetch from '../hubdb/fetch.js';
+import list from '../hubdb/list.js';
+import hubdbCommands from '../hubdb.js';
 vi.mock('../hubdb/clear');
 vi.mock('../hubdb/create');
 vi.mock('../hubdb/delete');
 vi.mock('../hubdb/fetch');
 vi.mock('../../lib/commonOpts');
 const commandSpy = vi
-    .spyOn(yargs_1.default, 'command')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'command')
+    .mockReturnValue(yargs);
 const demandCommandSpy = vi
-    .spyOn(yargs_1.default, 'demandCommand')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'demandCommand')
+    .mockReturnValue(yargs);
 describe('commands/hubdb', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(hubdb_1.default.command).toEqual('hubdb');
+            expect(hubdbCommands.command).toEqual('hubdb');
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(hubdb_1.default.describe).toBeDefined();
+            expect(hubdbCommands.describe).toBeDefined();
         });
     });
     describe('builder', () => {
@@ -37,18 +32,18 @@ describe('commands/hubdb', () => {
             commandSpy.mockClear();
             demandCommandSpy.mockClear();
         });
-        const subcommands = [clear_1.default, create_1.default, delete_1.default, fetch_1.default, list_1.default];
+        const subcommands = [clear, create, deleteCommand, fetch, list];
         it('should demand the command takes one positional argument', () => {
-            hubdb_1.default.builder(yargs_1.default);
+            hubdbCommands.builder(yargs);
             expect(demandCommandSpy).toHaveBeenCalledTimes(1);
             expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
         });
         it('should add the correct number of sub commands', () => {
-            hubdb_1.default.builder(yargs_1.default);
+            hubdbCommands.builder(yargs);
             expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
         });
         it.each(subcommands)('should attach the %s subcommand', module => {
-            hubdb_1.default.builder(yargs_1.default);
+            hubdbCommands.builder(yargs);
             expect(commandSpy).toHaveBeenCalledWith(module);
         });
     });

package/commands/__tests__/init.test.js

@@ -1,29 +1,24 @@
-"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 init_1 = __importDefault(require("../init"));
+import yargs from 'yargs';
+import { addConfigOptions, addTestingOptions } from '../../lib/commonOpts.js';
+import initCommand from '../init.js';
 vi.mock('../../lib/commonOpts');
 const optionsSpy = vi
-    .spyOn(yargs_1.default, 'options')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'options')
+    .mockReturnValue(yargs);
 describe('commands/init', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(init_1.default.command).toEqual('init');
+            expect(initCommand.command).toEqual('init');
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(init_1.default.describe).toBeDefined();
+            expect(initCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {
         it('should support the correct options', () => {
-            init_1.default.builder(yargs_1.default);
+            initCommand.builder(yargs);
             expect(optionsSpy).toHaveBeenCalledTimes(1);
             expect(optionsSpy).toHaveBeenCalledWith({
                 'auth-type': expect.objectContaining({
@@ -38,10 +33,10 @@ describe('commands/init', () => {
                     default: false,
                 }),
             });
-            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargs_1.default);
-            expect(commonOpts_1.addTestingOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addTestingOptions).toHaveBeenCalledWith(yargs_1.default);
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargs);
+            expect(addTestingOptions).toHaveBeenCalledTimes(1);
+            expect(addTestingOptions).toHaveBeenCalledWith(yargs);
         });
     });
 });

package/commands/__tests__/lint.test.js

@@ -1,38 +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 lint_1 = __importDefault(require("../lint"));
+import yargs from 'yargs';
+import { addConfigOptions, addAccountOptions } from '../../lib/commonOpts.js';
+import lintCommand from '../lint.js';
 vi.mock('../../lib/commonOpts');
 const positionalSpy = vi
-    .spyOn(yargs_1.default, 'positional')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'positional')
+    .mockReturnValue(yargs);
 describe('commands/lint', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(lint_1.default.command).toEqual('lint <path>');
+            expect(lintCommand.command).toEqual('lint <path>');
         });
     });
     describe('describe', () => {
         it('should not provide a description', () => {
-            expect(lint_1.default.describe).toEqual(undefined);
+            expect(lintCommand.describe).toEqual(undefined);
         });
     });
     describe('builder', () => {
         it('should support the correct positional arguments', () => {
-            lint_1.default.builder(yargs_1.default);
+            lintCommand.builder(yargs);
             expect(positionalSpy).toHaveBeenCalledTimes(1);
             expect(positionalSpy).toHaveBeenCalledWith('path', expect.objectContaining({ type: 'string' }));
         });
         it('should support the correct options', () => {
-            lint_1.default.builder(yargs_1.default);
-            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargs_1.default);
-            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargs_1.default);
+            lintCommand.builder(yargs);
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargs);
+            expect(addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(addAccountOptions).toHaveBeenCalledWith(yargs);
         });
     });
 });

package/commands/__tests__/list.test.js

@@ -1,46 +1,41 @@
-"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 list_1 = __importDefault(require("../list"));
+import yargs from 'yargs';
+import { addConfigOptions, addAccountOptions, addUseEnvironmentOptions, } from '../../lib/commonOpts.js';
+import listCommand from '../list.js';
 vi.mock('../../lib/commonOpts');
 const positionalSpy = vi
-    .spyOn(yargs_1.default, 'positional')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'positional')
+    .mockReturnValue(yargs);
 const exampleSpy = vi
-    .spyOn(yargs_1.default, 'example')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'example')
+    .mockReturnValue(yargs);
 describe('commands/list', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(list_1.default.command).toEqual(['list [path]', 'ls [path]']);
+            expect(listCommand.command).toEqual(['list [path]', 'ls [path]']);
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(list_1.default.describe).toBeDefined();
+            expect(listCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {
         it('should support the correct positional arguments', () => {
-            list_1.default.builder(yargs_1.default);
+            listCommand.builder(yargs);
             expect(positionalSpy).toHaveBeenCalledTimes(1);
             expect(positionalSpy).toHaveBeenCalledWith('path', expect.objectContaining({ type: 'string' }));
         });
         it('should support the correct options', () => {
-            list_1.default.builder(yargs_1.default);
-            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargs_1.default);
-            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargs_1.default);
-            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargs_1.default);
+            listCommand.builder(yargs);
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargs);
+            expect(addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(addAccountOptions).toHaveBeenCalledWith(yargs);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargs);
         });
         it('should provide examples', () => {
-            list_1.default.builder(yargs_1.default);
+            listCommand.builder(yargs);
             expect(exampleSpy).toHaveBeenCalledTimes(1);
         });
     });

package/commands/__tests__/logs.test.js

@@ -1,38 +1,39 @@
-"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");
+import yargs from 'yargs';
+import { addConfigOptions, addAccountOptions, addUseEnvironmentOptions, } from '../../lib/commonOpts.js';
 vi.mock('../../lib/commonOpts');
 const optionsSpy = vi
-    .spyOn(yargs_1.default, 'options')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'options')
+    .mockReturnValue(yargs);
 const conflictsSpy = vi
-    .spyOn(yargs_1.default, 'conflicts')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'conflicts')
+    .mockReturnValue(yargs);
+const exampleSpy = vi
+    .spyOn(yargs, 'example')
+    .mockReturnValue(yargs);
+const positionalSpy = vi
+    .spyOn(yargs, 'positional')
+    .mockReturnValue(yargs);
 // Import this last so mocks apply
-const logs_1 = __importDefault(require("../logs"));
+import logsCommand from '../logs.js';
 describe('commands/logs', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(logs_1.default.command).toEqual('logs [endpoint]');
+            expect(logsCommand.command).toEqual('logs [endpoint]');
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(logs_1.default.describe).toBeDefined();
+            expect(logsCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {
         it('should support the correct positional arguments', () => {
-            logs_1.default.builder(yargs_1.default);
-            expect(yargs_1.default.positional).toHaveBeenCalledTimes(1);
-            expect(yargs_1.default.positional).toHaveBeenCalledWith('endpoint', expect.objectContaining({ type: 'string' }));
+            logsCommand.builder(yargs);
+            expect(positionalSpy).toHaveBeenCalledTimes(1);
+            expect(positionalSpy).toHaveBeenCalledWith('endpoint', expect.objectContaining({ type: 'string' }));
         });
         it('should support the correct options', () => {
-            logs_1.default.builder(yargs_1.default);
+            logsCommand.builder(yargs);
             expect(optionsSpy).toHaveBeenCalledTimes(1);
             expect(optionsSpy).toHaveBeenCalledWith({
                 latest: expect.objectContaining({
@@ -50,21 +51,21 @@ describe('commands/logs', () => {
                     type: 'number',
                 }),
             });
-            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargs_1.default);
-            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargs_1.default);
-            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargs_1.default);
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargs);
+            expect(addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(addAccountOptions).toHaveBeenCalledWith(yargs);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargs);
         });
         it('should set the correct conflicts', () => {
-            logs_1.default.builder(yargs_1.default);
+            logsCommand.builder(yargs);
             expect(conflictsSpy).toHaveBeenCalledTimes(1);
             expect(conflictsSpy).toHaveBeenCalledWith('follow', 'limit');
         });
         it('should provide examples', () => {
-            logs_1.default.builder(yargs_1.default);
-            expect(yargs_1.default.example).toHaveBeenCalledTimes(1);
+            logsCommand.builder(yargs);
+            expect(exampleSpy).toHaveBeenCalledTimes(1);
         });
     });
 });

package/commands/__tests__/mcp.test.js

@@ -1,30 +1,25 @@
-"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 start_1 = __importDefault(require("../mcp/start"));
-const setup_1 = __importDefault(require("../mcp/setup"));
-const mcp_1 = __importDefault(require("../mcp"));
+import yargs from 'yargs';
+import startCommand from '../mcp/start.js';
+import setupCommand from '../mcp/setup.js';
+import mcpCommand from '../mcp.js';
 vi.mock('../mcp/start');
 vi.mock('../mcp/setup');
 vi.mock('../../lib/commonOpts');
 const commandSpy = vi
-    .spyOn(yargs_1.default, 'command')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'command')
+    .mockReturnValue(yargs);
 const demandCommandSpy = vi
-    .spyOn(yargs_1.default, 'demandCommand')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'demandCommand')
+    .mockReturnValue(yargs);
 describe('commands/mcp', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(mcp_1.default.command).toEqual('mcp');
+            expect(mcpCommand.command).toEqual('mcp');
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(mcp_1.default.describe).toBeUndefined();
+            expect(mcpCommand.describe).toBeUndefined();
         });
     });
     describe('builder', () => {
@@ -32,18 +27,18 @@ describe('commands/mcp', () => {
             commandSpy.mockClear();
             demandCommandSpy.mockClear();
         });
-        const subcommands = [start_1.default, setup_1.default];
+        const subcommands = [startCommand, setupCommand];
         it('should demand the command takes one positional argument', () => {
-            mcp_1.default.builder(yargs_1.default);
+            mcpCommand.builder(yargs);
             expect(demandCommandSpy).toHaveBeenCalledTimes(1);
             expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
         });
         it('should add the correct number of sub commands', () => {
-            mcp_1.default.builder(yargs_1.default);
+            mcpCommand.builder(yargs);
             expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
         });
         it.each(subcommands)('should attach the %s subcommand', module => {
-            mcp_1.default.builder(yargs_1.default);
+            mcpCommand.builder(yargs);
             expect(module).toBeDefined();
             expect(commandSpy).toHaveBeenCalledWith(module);
         });