@hubspot/cli 7.9.0-beta.2 → 7.9.0-experimental.0

package/lib/__tests__/dependencyManagement.test.js

@@ -1,5 +1,5 @@
 import util from 'util';
-import { installPackages, getProjectPackageJsonLocations, } from '../dependencyManagement.js';
+import { installPackages, updatePackages, getProjectPackageJsonLocations, isPackageInstalled, } from '../dependencyManagement.js';
 import { walk } from '@hubspot/local-dev-lib/fs';
 import path from 'path';
 import { getProjectConfig } from '../projects/config.js';
@@ -105,6 +105,54 @@ describe('lib/dependencyManagement', () => {
                 cwd: extensionsDir,
             });
         });
+        it('should install packages as dev dependencies when dev flag is true', async () => {
+            const packages = ['eslint', 'prettier'];
+            await installPackages({ packages, installLocations, dev: true });
+            expect(execMock).toHaveBeenCalledTimes(installLocations.length);
+            for (const location of installLocations) {
+                expect(execMock).toHaveBeenCalledWith(`npm install --save-dev eslint prettier`, {
+                    cwd: location,
+                });
+            }
+        });
+        it('should install packages as regular dependencies when dev flag is false', async () => {
+            const packages = ['react', 'react-dom'];
+            await installPackages({ packages, installLocations, dev: false });
+            expect(execMock).toHaveBeenCalledTimes(installLocations.length);
+            for (const location of installLocations) {
+                expect(execMock).toHaveBeenCalledWith(`npm install react react-dom`, {
+                    cwd: location,
+                });
+            }
+        });
+        it('should install packages as regular dependencies when dev flag is not provided', async () => {
+            const packages = ['axios'];
+            await installPackages({ packages, installLocations });
+            expect(execMock).toHaveBeenCalledTimes(installLocations.length);
+            for (const location of installLocations) {
+                expect(execMock).toHaveBeenCalledWith(`npm install axios`, {
+                    cwd: location,
+                });
+            }
+        });
+        it('should not use --save-dev flag when dev is true but no packages are provided', async () => {
+            await installPackages({ installLocations, dev: true });
+            expect(execMock).toHaveBeenCalledTimes(installLocations.length);
+            for (const location of installLocations) {
+                expect(execMock).toHaveBeenCalledWith(`npm install `, {
+                    cwd: location,
+                });
+            }
+        });
+        it('should not use --save-dev flag when dev is true but packages array is empty', async () => {
+            await installPackages({ packages: [], installLocations, dev: true });
+            expect(execMock).toHaveBeenCalledTimes(installLocations.length);
+            for (const location of installLocations) {
+                expect(execMock).toHaveBeenCalledWith(`npm install `, {
+                    cwd: location,
+                });
+            }
+        });
         it('should throw an error when installing the dependencies fails', async () => {
             execMock = vi.fn().mockImplementation(command => {
                 if (command === 'npm --version') {
@@ -133,6 +181,92 @@ describe('lib/dependencyManagement', () => {
             });
         });
     });
+    describe('updatePackages()', () => {
+        it('should setup a loading spinner', async () => {
+            const packages = ['package1', 'package2'];
+            await updatePackages({ packages, installLocations });
+            expect(SpinniesManager.init).toHaveBeenCalledTimes(installLocations.length);
+            expect(SpinniesManager.add).toHaveBeenCalledTimes(installLocations.length);
+            expect(SpinniesManager.succeed).toHaveBeenCalledTimes(installLocations.length);
+        });
+        it('should update the provided packages in all the provided install locations', async () => {
+            const packages = ['package1', 'package2'];
+            await updatePackages({ packages, installLocations });
+            expect(execMock).toHaveBeenCalledTimes(installLocations.length);
+            expect(SpinniesManager.add).toHaveBeenCalledTimes(installLocations.length);
+            expect(SpinniesManager.succeed).toHaveBeenCalledTimes(installLocations.length);
+            for (const location of installLocations) {
+                expect(execMock).toHaveBeenCalledWith(`npm update package1 package2`, {
+                    cwd: location,
+                });
+                expect(SpinniesManager.add).toHaveBeenCalledWith(`updatingDependencies-${location}`, {
+                    text: `Updating [package1, package2] in ${location}`,
+                });
+                expect(SpinniesManager.succeed).toHaveBeenCalledWith(`updatingDependencies-${location}`, {
+                    text: `Updated dependencies in ${location}`,
+                });
+            }
+        });
+        it('should use the provided install locations', async () => {
+            await updatePackages({ installLocations });
+            expect(execMock).toHaveBeenCalledTimes(installLocations.length);
+            expect(execMock).toHaveBeenCalledWith(`npm update `, {
+                cwd: appFunctionsDir,
+            });
+            expect(execMock).toHaveBeenCalledWith(`npm update `, {
+                cwd: extensionsDir,
+            });
+        });
+        it('should locate the projects package.json files when install locations is not provided', async () => {
+            const installLocations = [
+                path.join(appFunctionsDir, 'package.json'),
+                path.join(extensionsDir, 'package.json'),
+            ];
+            mockedWalk.mockResolvedValue(installLocations);
+            mockedGetProjectConfig.mockResolvedValue({
+                projectDir,
+                projectConfig: {
+                    srcDir,
+                },
+            });
+            await updatePackages({});
+            // It's called once per each install location, plus once to check if npm installed
+            expect(execMock).toHaveBeenCalledTimes(installLocations.length + 1);
+            expect(execMock).toHaveBeenCalledWith(`npm update `, {
+                cwd: appFunctionsDir,
+            });
+            expect(execMock).toHaveBeenCalledWith(`npm update `, {
+                cwd: extensionsDir,
+            });
+        });
+        it('should throw an error when updating the dependencies fails', async () => {
+            execMock = vi.fn().mockImplementation(command => {
+                if (command === 'npm --version') {
+                    return;
+                }
+                throw new Error('OH NO');
+            });
+            util.promisify = mockedPromisify(execMock);
+            // Mock walk to return the directory paths instead of package.json paths
+            mockedWalk.mockResolvedValue([appFunctionsDir, extensionsDir]);
+            mockedFs.existsSync.mockImplementation(filePath => {
+                const pathStr = filePath.toString();
+                if (pathStr === projectDir ||
+                    pathStr === path.join(projectDir, srcDir)) {
+                    return true;
+                }
+                return false;
+            });
+            await expect(() => updatePackages({ installLocations: [appFunctionsDir, extensionsDir] })).rejects.toThrowError(`Updating dependencies for ${appFunctionsDir} failed`);
+            expect(SpinniesManager.fail).toHaveBeenCalledTimes(installLocations.length);
+            expect(SpinniesManager.fail).toHaveBeenCalledWith(`updatingDependencies-${appFunctionsDir}`, {
+                text: `Updating dependencies for ${appFunctionsDir} failed`,
+            });
+            expect(SpinniesManager.fail).toHaveBeenCalledWith(`updatingDependencies-${extensionsDir}`, {
+                text: `Updating dependencies for ${extensionsDir} failed`,
+            });
+        });
+    });
     describe('getProjectPackageJsonFiles()', () => {
         it('should throw an error when ran outside the boundary of a project', async () => {
             mockedGetProjectConfig.mockResolvedValue({});
@@ -149,6 +283,30 @@ describe('lib/dependencyManagement', () => {
             mockedFs.existsSync.mockReturnValueOnce(false);
             await expect(() => getProjectPackageJsonLocations()).rejects.toThrowError(new RegExp(`No dependencies to install. The project ${projectName} folder might be missing component or subcomponent files.`));
         });
+        it('should throw "install" error message when isUpdate=false and no package.json files found', async () => {
+            mockedWalk.mockResolvedValue([]);
+            mockedFs.existsSync.mockImplementation(filePath => {
+                const pathStr = filePath.toString();
+                if (pathStr === projectDir ||
+                    pathStr === path.join(projectDir, srcDir)) {
+                    return true;
+                }
+                return false;
+            });
+            await expect(() => getProjectPackageJsonLocations(undefined, false)).rejects.toThrowError(new RegExp(`No dependencies to install. The project ${projectName} folder might be missing component or subcomponent files.`));
+        });
+        it('should throw "update" error message when isUpdate=true and no package.json files found', async () => {
+            mockedWalk.mockResolvedValue([]);
+            mockedFs.existsSync.mockImplementation(filePath => {
+                const pathStr = filePath.toString();
+                if (pathStr === projectDir ||
+                    pathStr === path.join(projectDir, srcDir)) {
+                    return true;
+                }
+                return false;
+            });
+            await expect(() => getProjectPackageJsonLocations(undefined, true)).rejects.toThrowError(new RegExp(`No dependencies to update. The project ${projectName} folder might be missing component or subcomponent files.`));
+        });
         it('should ignore package.json files in certain directories', async () => {
             const nodeModulesDir = path.join(appDir, 'node_modules');
             const viteDir = path.join(appDir, '.vite');
@@ -172,4 +330,118 @@ describe('lib/dependencyManagement', () => {
             expect(actual).toEqual([appFunctionsDir, extensionsDir]);
         });
     });
+    describe('isPackageInstalled()', () => {
+        const testDir = '/test/directory';
+        const readFileSyncSpy = vi.spyOn(fs, 'readFileSync');
+        const existsSyncSpy = vi.spyOn(fs, 'existsSync');
+        function mockNodeModulesExists(packageName, exists = true) {
+            existsSyncSpy.mockImplementation(filePath => {
+                const pathStr = filePath.toString();
+                return (exists && pathStr === path.join(testDir, 'node_modules', packageName));
+            });
+        }
+        beforeEach(() => {
+            vi.clearAllMocks();
+            readFileSyncSpy.mockReset();
+            existsSyncSpy.mockReset();
+        });
+        it('should return true if package is in dependencies and in node_modules', () => {
+            readFileSyncSpy.mockReturnValueOnce(JSON.stringify({
+                dependencies: {
+                    eslint: '^9.0.0',
+                },
+            }));
+            mockNodeModulesExists('eslint', true);
+            const result = isPackageInstalled(testDir, 'eslint');
+            expect(result).toBe(true);
+            expect(readFileSyncSpy).toHaveBeenCalledWith(path.join(testDir, 'package.json'), 'utf-8');
+            expect(existsSyncSpy).toHaveBeenCalledWith(path.join(testDir, 'node_modules', 'eslint'));
+        });
+        it('should return true if package is in devDependencies and in node_modules', () => {
+            readFileSyncSpy.mockReturnValueOnce(JSON.stringify({
+                devDependencies: {
+                    prettier: '^3.0.0',
+                },
+            }));
+            mockNodeModulesExists('prettier', true);
+            const result = isPackageInstalled(testDir, 'prettier');
+            expect(result).toBe(true);
+        });
+        it('should return false if package is in package.json but not in node_modules', () => {
+            readFileSyncSpy.mockReturnValueOnce(JSON.stringify({
+                dependencies: {
+                    react: '^18.0.0',
+                },
+            }));
+            mockNodeModulesExists('react', false);
+            const result = isPackageInstalled(testDir, 'react');
+            expect(result).toBe(false);
+        });
+        it('should return false if package is not in package.json but is in node_modules', () => {
+            readFileSyncSpy.mockReturnValueOnce(JSON.stringify({
+                dependencies: {
+                    typescript: '^5.0.0',
+                },
+            }));
+            mockNodeModulesExists('lodash', true);
+            const result = isPackageInstalled(testDir, 'lodash');
+            expect(result).toBe(false);
+        });
+        it('should return false if package is not in package.json and not in node_modules', () => {
+            readFileSyncSpy.mockReturnValueOnce(JSON.stringify({
+                dependencies: {},
+            }));
+            mockNodeModulesExists('nonexistent-package', false);
+            const result = isPackageInstalled(testDir, 'nonexistent-package');
+            expect(result).toBe(false);
+        });
+        it('should return false if package.json cannot be read', () => {
+            readFileSyncSpy.mockImplementationOnce(() => {
+                throw new Error('File not found');
+            });
+            const result = isPackageInstalled(testDir, 'eslint');
+            expect(result).toBe(false);
+        });
+        it('should return false if package.json has invalid JSON', () => {
+            readFileSyncSpy.mockReturnValueOnce('invalid json{');
+            const result = isPackageInstalled(testDir, 'eslint');
+            expect(result).toBe(false);
+        });
+        it('should return false if checking node_modules throws an error', () => {
+            readFileSyncSpy.mockReturnValueOnce(JSON.stringify({
+                dependencies: {
+                    eslint: '^9.0.0',
+                },
+            }));
+            existsSyncSpy.mockImplementation(() => {
+                throw new Error('Permission denied');
+            });
+            const result = isPackageInstalled(testDir, 'eslint');
+            expect(result).toBe(false);
+        });
+        it('should handle scoped packages correctly', () => {
+            readFileSyncSpy.mockReturnValueOnce(JSON.stringify({
+                dependencies: {
+                    '@typescript-eslint/parser': '^8.0.0',
+                },
+            }));
+            mockNodeModulesExists('@typescript-eslint/parser', true);
+            const result = isPackageInstalled(testDir, '@typescript-eslint/parser');
+            expect(result).toBe(true);
+            expect(existsSyncSpy).toHaveBeenCalledWith(path.join(testDir, 'node_modules', '@typescript-eslint/parser'));
+        });
+        it('should check both dependencies and devDependencies', () => {
+            readFileSyncSpy.mockReturnValueOnce(JSON.stringify({
+                dependencies: {
+                    react: '^18.0.0',
+                },
+                devDependencies: {
+                    eslint: '^9.0.0',
+                },
+            }));
+            mockNodeModulesExists('eslint', true);
+            const result = isPackageInstalled(testDir, 'eslint');
+            expect(result).toBe(true);
+        });
+    });
 });

package/lib/commonOpts.js

@@ -149,16 +149,13 @@ export function setCLILogLevel(options) {
         setLogLevel(LOG_LEVEL.ERROR);
         SpinniesManager.setDisableOutput(true);
     }
-    else if (debug) {
+    else if (debug || networkDebug) {
         setLogLevel(LOG_LEVEL.DEBUG);
+        process.env.HUBSPOT_NETWORK_LOGGING = 'true';
     }
     else {
         setLogLevel(LOG_LEVEL.LOG);
     }
-    if (networkDebug) {
-        process.env.HUBSPOT_NETWORK_LOGGING = 'true';
-        setLogLevel(LOG_LEVEL.DEBUG);
-    }
 }
 export function getCommandName(argv) {
     return String(argv && argv._ && argv._[0]) || '';

package/lib/dependencyManagement.d.ts

@@ -1,6 +1,12 @@
-export declare function installPackages({ packages, installLocations, }: {
+export declare function installPackages({ packages, installLocations, dev, }: {
     packages?: string[];
     installLocations?: string[];
+    dev?: boolean;
 }): Promise<void>;
-export declare function getProjectPackageJsonLocations(dir?: string): Promise<string[]>;
+export declare function updatePackages({ packages, installLocations, }: {
+    packages?: string[];
+    installLocations?: string[];
+}): Promise<void>;
+export declare function getProjectPackageJsonLocations(dir?: string, isUpdate?: boolean): Promise<string[]>;
+export declare function isPackageInstalled(directory: string, packageName: string): boolean;
 export declare function hasMissingPackages(directory: string): Promise<boolean>;

package/lib/dependencyManagement.js

@@ -6,19 +6,21 @@ import { walk } from '@hubspot/local-dev-lib/fs';
 import { getProjectConfig } from './projects/config.js';
 import { commands } from '../lang/en.js';
 import SpinniesManager from './ui/SpinniesManager.js';
-import { isGloballyInstalled, executeInstall, DEFAULT_PACKAGE_MANAGER, } from './npm.js';
+import { isGloballyInstalled, executeInstall, executeUpdate, DEFAULT_PACKAGE_MANAGER, } from './npm.js';
 class NoPackageJsonFilesError extends Error {
-    constructor(projectName) {
-        super(commands.project.installDeps.noPackageJsonInProject(projectName));
+    constructor(projectName, isUpdate = false) {
+        super(isUpdate
+            ? commands.project.updateDeps.noPackageJsonInProject(projectName)
+            : commands.project.installDeps.noPackageJsonInProject(projectName));
     }
 }
-export async function installPackages({ packages, installLocations, }) {
+export async function installPackages({ packages, installLocations, dev = false, }) {
     const installDirs = installLocations || (await getProjectPackageJsonLocations());
     await Promise.all(installDirs.map(async (dir) => {
-        await installPackagesInDirectory(dir, packages);
+        await installPackagesInDirectory(dir, packages, dev);
     }));
 }
-async function installPackagesInDirectory(directory, packages) {
+async function installPackagesInDirectory(directory, packages, dev = false) {
     const spinner = `installingDependencies-${directory}`;
     const relativeDir = path.relative(process.cwd(), directory);
     SpinniesManager.init();
@@ -28,7 +30,8 @@ async function installPackagesInDirectory(directory, packages) {
             : commands.project.installDeps.installingDependencies(relativeDir),
     });
     try {
-        await executeInstall(packages, null, { cwd: directory });
+        const flags = dev && packages && packages.length > 0 ? '--save-dev' : null;
+        await executeInstall(packages, flags, { cwd: directory });
         SpinniesManager.succeed(spinner, {
             text: commands.project.installDeps.installationSuccessful(relativeDir),
         });
@@ -42,26 +45,60 @@ async function installPackagesInDirectory(directory, packages) {
         });
     }
 }
-export async function getProjectPackageJsonLocations(dir) {
+export async function updatePackages({ packages, installLocations, }) {
+    const installDirs = installLocations || (await getProjectPackageJsonLocations(undefined, true));
+    await Promise.all(installDirs.map(async (dir) => {
+        await updatePackagesInDirectory(dir, packages);
+    }));
+}
+async function updatePackagesInDirectory(directory, packages) {
+    const spinner = `updatingDependencies-${directory}`;
+    const relativeDir = path.relative(process.cwd(), directory);
+    SpinniesManager.init();
+    SpinniesManager.add(spinner, {
+        text: packages && packages.length
+            ? commands.project.updateDeps.updatingDependenciesToLocation(`[${packages.join(', ')}]`, relativeDir)
+            : commands.project.updateDeps.updatingDependencies(relativeDir),
+    });
+    try {
+        await executeUpdate(packages, null, { cwd: directory });
+        SpinniesManager.succeed(spinner, {
+            text: commands.project.updateDeps.updateSuccessful(relativeDir),
+        });
+    }
+    catch (e) {
+        SpinniesManager.fail(spinner, {
+            text: commands.project.updateDeps.updatingDependenciesFailed(relativeDir),
+        });
+        throw new Error(commands.project.updateDeps.updatingDependenciesFailed(relativeDir), {
+            cause: e,
+        });
+    }
+}
+export async function getProjectPackageJsonLocations(dir, isUpdate = false) {
     const projectConfig = await getProjectConfig(dir);
     if (!projectConfig ||
         !projectConfig.projectDir ||
         !projectConfig.projectConfig) {
-        throw new Error(commands.project.installDeps.noProjectConfig);
+        throw new Error(isUpdate
+            ? commands.project.updateDeps.noProjectConfig
+            : commands.project.installDeps.noProjectConfig);
     }
     const { projectDir, projectConfig: { srcDir, name }, } = projectConfig;
     if (!(await isGloballyInstalled(DEFAULT_PACKAGE_MANAGER))) {
-        throw new Error(commands.project.installDeps.packageManagerNotInstalled(DEFAULT_PACKAGE_MANAGER));
+        throw new Error(isUpdate
+            ? commands.project.updateDeps.packageManagerNotInstalled(DEFAULT_PACKAGE_MANAGER)
+            : commands.project.installDeps.packageManagerNotInstalled(DEFAULT_PACKAGE_MANAGER));
     }
     if (!fs.existsSync(projectConfig.projectDir) ||
         !fs.existsSync(path.join(projectDir, srcDir))) {
-        throw new NoPackageJsonFilesError(name);
+        throw new NoPackageJsonFilesError(name, isUpdate);
     }
     const packageJsonFiles = (await walk(path.join(projectDir, srcDir))).filter(file => file.includes('package.json') &&
         !file.includes('node_modules') &&
         !file.includes('.vite'));
     if (packageJsonFiles.length === 0) {
-        throw new NoPackageJsonFilesError(name);
+        throw new NoPackageJsonFilesError(name, isUpdate);
     }
     const packageParentDirs = [];
     packageJsonFiles.forEach(packageJsonFile => {
@@ -70,6 +107,32 @@ export async function getProjectPackageJsonLocations(dir) {
     });
     return packageParentDirs;
 }
+function isPackageInPackageJson(directory, packageName) {
+    const packageJsonPath = path.join(directory, 'package.json');
+    try {
+        const packageJsonContent = fs.readFileSync(packageJsonPath, 'utf-8');
+        const packageJson = JSON.parse(packageJsonContent);
+        return !!((packageJson.dependencies && packageJson.dependencies[packageName]) ||
+            (packageJson.devDependencies && packageJson.devDependencies[packageName]));
+    }
+    catch (error) {
+        return false;
+    }
+}
+function isPackageInNodeModules(directory, packageName) {
+    const packagePath = path.join(directory, 'node_modules', packageName);
+    try {
+        return fs.existsSync(packagePath);
+    }
+    catch (error) {
+        return false;
+    }
+}
+export function isPackageInstalled(directory, packageName) {
+    const inPackageJson = isPackageInPackageJson(directory, packageName);
+    const actuallyInstalled = isPackageInNodeModules(directory, packageName);
+    return inPackageJson && actuallyInstalled;
+}
 export async function hasMissingPackages(directory) {
     const exec = util.promisify(execAsync);
     const { stdout } = await exec(`npm install --ignore-scripts --dry-run`, {

package/lib/npm.d.ts

@@ -7,3 +7,6 @@ export declare function getLatestCliVersion(): Promise<{
 export declare function executeInstall(packages?: string[], flags?: string | null, options?: {
     cwd?: string;
 }): Promise<void>;
+export declare function executeUpdate(packages?: string[], flags?: string | null, options?: {
+    cwd?: string;
+}): Promise<void>;

package/lib/npm.js

@@ -25,3 +25,9 @@ export async function executeInstall(packages = [], flags, options) {
     const exec = util.promisify(execAsync);
     await exec(installCommand, options);
 }
+export async function executeUpdate(packages = [], flags, options) {
+    const updateCommand = `${DEFAULT_PACKAGE_MANAGER} update${flags ? ` ${flags}` : ''} ${packages.join(' ')}`;
+    uiLogger.debug('Running', updateCommand);
+    const exec = util.promisify(execAsync);
+    await exec(updateCommand, options);
+}

package/lib/projects/__tests__/platformVersion.test.js

@@ -8,10 +8,14 @@ describe('platformVersion', () => {
             expect(isV2Project('2025.2')).toBe(true);
         });
         it('returns true if platform version is greater than the minimum', () => {
-            expect(isV2Project('2026.2')).toBe(true);
+            expect(isV2Project('2025.3')).toBe(true);
+            expect(isV2Project('2026.03')).toBe(true);
+            expect(isV2Project('2026.03-beta')).toBe(true);
         });
         it('returns false if platform version is less than the minimum', () => {
             expect(isV2Project('2025.0')).toBe(false);
+            expect(isV2Project('2025.01')).toBe(false);
+            expect(isV2Project('2025.01-beta')).toBe(false);
         });
         it('returns false if platform version is invalid', () => {
             expect(isV2Project(null)).toBe(false);

package/lib/projects/platformVersion.js

@@ -5,6 +5,6 @@ export function isV2Project(platformVersion) {
     if (platformVersion.toLowerCase() === 'unstable') {
         return true;
     }
-    const [year, minor] = platformVersion.split('.');
+    const [year, minor] = platformVersion.split(/[.-]/);
     return Number(year) >= 2025 && Number(minor) >= 2;
 }

package/package.json

@@ -1,12 +1,12 @@
 {
     "name": "@hubspot/cli",
-    "version": "7.9.0-beta.2",
+    "version": "7.9.0-experimental.0",
     "description": "The official CLI for developing on HubSpot",
     "license": "Apache-2.0",
     "repository": "https://github.com/HubSpot/hubspot-cli",
     "type": "module",
     "dependencies": {
-        "@hubspot/local-dev-lib": "3.21.1",
+        "@hubspot/local-dev-lib": "3.21.2",
         "@hubspot/project-parsing-lib": "0.10.1",
         "@hubspot/serverless-dev-runtime": "7.0.6",
         "@hubspot/theme-preview-dev-server": "0.0.10",