@hubspot/cli 7.6.2-beta.0 → 7.7.0-beta.0

package/lang/en.js

@@ -6,7 +6,7 @@ import { ARCHIVED_HUBSPOT_CONFIG_YAML_FILE_NAME, GLOBAL_CONFIG_PATH, DEFAULT_HUB
 import { uiAccountDescription, uiBetaTag, uiCommandReference, uiLink, UI_COLORS, } from '../lib/ui/index.js';
 import { getProjectDetailUrl, getProjectSettingsUrl, getLocalDevUiUrl, getAppAllowlistUrl, } from '../lib/projects/urls.js';
 import { getProductUpdatesUrl } from '../lib/links.js';
-import { APP_DISTRIBUTION_TYPES, APP_AUTH_TYPES, PROJECT_CONFIG_FILE, PROJECT_WITH_APP, } from '../lib/constants.js';
+import { APP_DISTRIBUTION_TYPES, APP_AUTH_TYPES, PROJECT_CONFIG_FILE, PROJECT_WITH_APP, LEGACY_PUBLIC_APP_FILE, } from '../lib/constants.js';
 import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
 export const commands = {
     generalErrors: {
@@ -262,6 +262,10 @@ export const commands = {
                         describe: 'Enable or disable automatic opening of the browser',
                     },
                 },
+                errors: {
+                    invalidBoolean: (commandName, value) => `Invalid boolean value "${value}" for --${commandName}. Valid values are: true, false`,
+                    invalidHTTPTimeout: `Invalid HTTP timeout value. Must be a number greater than 3000.`,
+                },
             },
         },
     },
@@ -1117,15 +1121,15 @@ export const commands = {
             },
             deprecationWarning: (oldCommand, newCommand) => `The ${oldCommand} command is deprecated and will be removed. Use ${newCommand} going forward.`,
             migrationStatus: {
-                inProgress: () => `Converting app configuration to ${chalk.bold('public-app.json')} component definition ...`,
+                inProgress: () => `Converting app configuration to ${chalk.bold(LEGACY_PUBLIC_APP_FILE)} component definition ...`,
                 success: () => `${chalk.bold('Your app was converted and build #1 is deployed')}`,
-                done: () => 'Converting app configuration to public-app.json component definition ... DONE',
-                failure: () => 'Converting app configuration to public-app.json component definition ... FAILED',
+                done: () => `Converting app configuration to ${LEGACY_PUBLIC_APP_FILE} component definition ... DONE`,
+                failure: () => `Converting app configuration to ${LEGACY_PUBLIC_APP_FILE} component definition ... FAILED`,
             },
             warning: {
                 title: () => `${chalk.bold('You are about to migrate an app to the projects framework')}`,
                 projectConversion: () => `${chalk.bold('The selected app will be converted to a project component.')}`,
-                appConfig: () => `All supported app configuration will be moved to the ${chalk.bold('public-app.json')} component definition file. Future updates to those features must be made through the project build and deploy pipeline, not the developer account UI.`,
+                appConfig: () => `All supported app configuration will be moved to the ${chalk.bold(LEGACY_PUBLIC_APP_FILE)} component definition file. Future updates to those features must be made through the project build and deploy pipeline, not the developer account UI.`,
                 buildAndDeploy: 'This will create a new project with a single app component and immediately build and deploy it to your developer account (build #1).',
                 existingApps: () => `${chalk.bold('This will not affect existing app users or installs.')}`,
                 copyApp: 'We strongly recommend making a copy of your app to test this process in a development app before replacing production.',
@@ -1159,10 +1163,10 @@ export const commands = {
                 },
             },
             cloneStatus: {
-                inProgress: () => `Cloning app configuration to ${chalk.bold('public-app.json')} component definition ...`,
-                done: 'Cloning app configuration to public-app.json component definition ... DONE',
+                inProgress: () => `Cloning app configuration to ${chalk.bold(LEGACY_PUBLIC_APP_FILE)} component definition ...`,
+                done: `Cloning app configuration to ${LEGACY_PUBLIC_APP_FILE} component definition ... DONE`,
                 success: (dest) => `Your cloned project was created in ${dest}`,
-                failure: 'Cloning app configuration to public-app.json component definition ... FAILED',
+                failure: `Cloning app configuration to ${LEGACY_PUBLIC_APP_FILE} component definition ... FAILED`,
             },
             errors: {
                 invalidAccountTypeTitle: () => `${chalk.bold('Developer account not targeted')}`,
@@ -1905,18 +1909,41 @@ export const commands = {
             example: (name) => `Create a test account config file with the name "${name}"`,
         },
         delete: {
-            describe: 'Delete a test account config file.',
+            describe: 'Delete a test account from your HubSpot account and CLI config',
             pathPrompt: '[--path] What is the path to the test account config?',
+            info: {
+                deletionCanceled: 'Deletion canceled by user',
+                accountNotFoundWithId: (id) => `No account was found with ID ${id}`,
+                replaceDefaultAccount: (testAccountId, parentAccountName) => `The removed test account ${chalk.bold(testAccountId)} was the default account. Replaced default account to parent account: ${chalk.bold(parentAccountName)}`,
+            },
+            prompts: {
+                selectTestAccounts: 'Select test account(s) to delete',
+                confirmDeletion: 'All data for the account will be permanently deleted. Any connected apps will have their access tokens revoked. Do you wish to proceed?',
+            },
             errors: {
-                failedToDelete: 'Failed to delete test account',
+                failedToDelete: (testAccountToDelete) => `Failed to delete test account with ID ${testAccountToDelete}`,
+                failedToSelectAccount: 'Failed to select a test account to delete',
+                noAccountsToDelete: (accountId) => `There are no test accounts associated with ${uiAccountDescription(accountId)} to delete. Try running ${uiCommandReference('hs account use')} to change your default account`,
+                failedToDeleteFromConfig: (testAccountToDelete) => `Failed to delete test account with ID ${testAccountToDelete} from the CLI config`,
+                failedToFetchTestAccounts: 'Failed to fetch developer test accounts',
+                testAccountNotFound: (nameOrId) => `Test account${nameOrId ? ` ${chalk.bold(nameOrId)}` : ''} not found in config. \nTry running ${uiCommandReference('hs account auth')} to add the account to config or visit ${uiLink('developer test accounts', 'https://app.hubspot.com/l/developer-test-accounts/')} to delete the test account.`,
+                parentAccountNotFound: (testAccountId) => `Parent account of test account ${chalk.bold(testAccountId)} not found in config. \nTry running ${uiCommandReference('hs account auth')} to add the parent account to config or visit ${uiLink('developer test accounts', 'https://app.hubspot.com/l/developer-test-accounts/')} to delete the test account.`,
             },
             success: {
-                testAccountDeleted: (testAccountId) => `Test account with id ${testAccountId} successfully deleted`,
+                testAccountDeletedFromHubSpot: (testAccountToDelete) => `Successfully deleted test account with ID ${testAccountToDelete}`,
+                testAccountDeletedFromConfig: (accountId) => `Successfully deleted test account with ID ${accountId} from the CLI config`,
+            },
+            options: {
+                name: 'The name of the test account (in your CLI config) to delete',
+                id: 'The id of the test account',
             },
-            positionals: {
-                testAccountId: 'The id of the test account',
+            examples: {
+                withPositionalID: (testAccountToDelete) => `Delete a test account with id "${testAccountToDelete}" using positional argument`,
+                withPositionalName: (testAccountToDelete) => `Delete a test account with name "${testAccountToDelete}" using positional argument`,
+                withID: (testAccountToDelete) => `Delete a test account with the id "${testAccountToDelete}"`,
+                withName: (testAccountToDelete) => `Delete a test account with the name "${testAccountToDelete}"`,
+                withoutId: 'Delete a test account via a prompt',
             },
-            example: (testAccountId) => `Delete a test account with the id "${testAccountId}"`,
         },
     },
     secrets: {
@@ -2765,6 +2792,7 @@ export const lib = {
             compressed: (byteCount) => `Project files compressed: ${byteCount} bytes`,
             compressing: (path) => `Compressing build files to "${path}"`,
             fileFiltered: (filename) => `Ignore rule triggered for "${filename}"`,
+            legacyFileDetected: (filename, platformVersion) => `The ${chalk.bold(filename)} file is not supported on platform version ${chalk.bold(platformVersion)} and will be ignored.`,
         },
     },
     boxen: {
@@ -2883,25 +2911,26 @@ export const lib = {
         },
         setAllowUsageTracking: {
             fieldName: 'usage tracking',
-            success: (isEnabled) => `Allow usage tracking set to: "${isEnabled}"`,
+            success: (isEnabled) => `Successfully updated ${chalk.bold('allow usage tracking')} to ${chalk.bold(isEnabled)}`,
         },
         setAllowAutoUpdates: {
             fieldName: 'auto updates',
-            success: (isEnabled) => `Allow auto updates set to: "${isEnabled}"`,
+            success: (isEnabled) => `Successfully updated ${chalk.bold('allow auto updates')} to ${chalk.bold(isEnabled)}`,
         },
         setDefaultCmsPublishMode: {
             promptMessage: 'Select CMS publish mode to be used as the default',
             error: (validModes) => `The provided CMS publish mode is invalid. Valid values are ${validModes}.`,
-            success: (mode) => `Default mode updated to: ${mode}`,
+            success: (mode) => `Successfully updated ${chalk.bold('default CMS publish mode')} to ${chalk.bold(mode)}`,
         },
         setHttpTimeout: {
             promptMessage: 'Enter http timeout duration',
-            success: (timeout) => `HTTP timeout set to: ${timeout}`,
+            success: (timeout) => `Successfully updated ${chalk.bold('HTTP timeout')} to ${chalk.bold(timeout)}`,
+            error: (timeout) => `Invalid HTTP timeout value "${timeout}". Must be a number greater than 3000.`,
         },
         setAutoOpenBrowser: {
             fieldName: 'auto open browser',
-            enabled: 'Auto opening your browser has been enabled',
-            disabled: 'Auto opening your browser has been disabled',
+            enabled: `Successfully updated ${chalk.bold('auto open browser')} to ${chalk.bold('enabled')}`,
+            disabled: `Successfully updated ${chalk.bold('auto open browser')} to ${chalk.bold('disabled')}`,
         },
     },
     commonOpts: {

package/lib/__tests__/yargsUtils.test.js

@@ -1,6 +1,19 @@
-import { hasFlag, makeYargsBuilder, getExclusiveConflicts, } from '../yargsUtils.js';
+import { hasFlag, makeYargsBuilder, strictEnforceBoolean, } from '../yargsUtils.js';
 import * as commonOpts from '../commonOpts.js';
 vi.mock('../commonOpts');
+vi.mock('../../lang/en.js', () => ({
+    commands: {
+        config: {
+            subcommands: {
+                set: {
+                    errors: {
+                        invalidBoolean: (option, value) => `Invalid boolean value "${value}" for --${option}. Valid values are: true, false`,
+                    },
+                },
+            },
+        },
+    },
+}));
 const argvWithFlag = ['hs', 'command', '--test'];
 const argvWithoutFlag = ['hs', 'command'];
 describe('lib/yargsUtils', () => {
@@ -36,15 +49,76 @@ describe('lib/yargsUtils', () => {
             expect(commonOpts.addCustomHelpOutput).toHaveBeenCalled();
         });
     });
-    describe('getExclusiveConflicts()', () => {
-        it('should return an object where each option conflicts with all others', () => {
-            const options = ['option1', 'option2', 'option3'];
-            const result = getExclusiveConflicts(options);
-            expect(result).toEqual({
-                option1: ['option2', 'option3'],
-                option2: ['option1', 'option3'],
-                option3: ['option1', 'option2'],
+    describe('strictEnforceBoolean()', () => {
+        it('should validate valid boolean values (true/false, case-insensitive)', () => {
+            const validArgs = [
+                ['hs', 'config', 'set', '--allow-usage-tracking=true'],
+                ['hs', 'config', 'set', '--allow-usage-tracking=false'],
+                ['hs', 'config', 'set', '--allow-usage-tracking=TRUE'],
+                ['hs', 'config', 'set', '--allow-usage-tracking=False'],
+            ];
+            validArgs.forEach(args => {
+                expect(() => strictEnforceBoolean(args, ['allow-usage-tracking'])).not.toThrow();
+                expect(strictEnforceBoolean(args, ['allow-usage-tracking'])).toBe(true);
             });
         });
+        it('should reject invalid boolean values with descriptive error messages', () => {
+            const invalidCases = [
+                {
+                    args: ['hs', 'config', 'set', '--allow-usage-tracking=yes'],
+                    value: 'yes',
+                },
+                {
+                    args: ['hs', 'config', 'set', '--allow-usage-tracking=1'],
+                    value: '1',
+                },
+                {
+                    args: ['hs', 'config', 'set', '--auto-open-browser=maybe'],
+                    value: 'maybe',
+                },
+            ];
+            invalidCases.forEach(({ args, value }) => {
+                const option = args[3].split('=')[0].replace('--', '');
+                expect(() => strictEnforceBoolean(args, [option])).toThrow(`Invalid boolean value "${value}" for --${option}. Valid values are: true, false`);
+            });
+        });
+        it('should support multiple config values in a single command', () => {
+            const args = [
+                'hs',
+                'config',
+                'set',
+                '--allow-usage-tracking=true',
+                '--auto-open-browser=false',
+                '--allow-auto-updates=true',
+                '--http-timeout=5000',
+                '--default-cms-publish-mode=draft',
+            ];
+            const booleanOptions = [
+                'allow-usage-tracking',
+                'auto-open-browser',
+                'allow-auto-updates',
+            ];
+            expect(() => strictEnforceBoolean(args, booleanOptions)).not.toThrow();
+            expect(strictEnforceBoolean(args, booleanOptions)).toBe(true);
+        });
+        it('should error when one of multiple values is invalid', () => {
+            const args = [
+                'hs',
+                'config',
+                'set',
+                '--allow-usage-tracking=true',
+                '--auto-open-browser=invalid',
+                '--http-timeout=5000',
+            ];
+            const booleanOptions = ['allow-usage-tracking', 'auto-open-browser'];
+            expect(() => strictEnforceBoolean(args, booleanOptions)).toThrow('Invalid boolean value "invalid" for --auto-open-browser. Valid values are: true, false');
+        });
+        it('should maintain backwards compatability', () => {
+            expect(() => strictEnforceBoolean(['hs', 'config', 'set', '--allow-usage-tracking=true'], ['allow-usage-tracking'])).not.toThrow();
+            // No values provided (interactive mode)
+            expect(() => strictEnforceBoolean(['hs', 'config', 'set'], ['allow-usage-tracking'])).not.toThrow();
+            // Flag without equals sign (succeed with true)
+            expect(() => strictEnforceBoolean(['hs', 'config', 'set', '--allow-usage-tracking'], ['allow-usage-tracking'])).not.toThrow();
+        });
     });
 });

package/lib/configOptions.js

@@ -79,6 +79,13 @@ async function enterTimeout() {
             message: lib.configOptions.setHttpTimeout.promptMessage,
             type: 'input',
             default: 30000,
+            validate: (timeout) => {
+                const timeoutNum = parseInt(timeout, 10);
+                if (isNaN(timeoutNum) || timeoutNum < 3000) {
+                    return lib.configOptions.setHttpTimeout.error(timeout);
+                }
+                return true;
+            },
         },
     ]);
     return timeout;

package/lib/constants.d.ts

@@ -125,3 +125,9 @@ export declare const CONFIG_LOCAL_STATE_FLAGS: {
 };
 export declare const EMPTY_PROJECT = "empty";
 export declare const PROJECT_WITH_APP = "app";
+export declare const LEGACY_SERVERLESS_FILE = "serverless.json";
+export declare const LEGACY_PUBLIC_APP_FILE = "public-app.json";
+export declare const LEGACY_PRIVATE_APP_FILE = "app.json";
+export declare const THEME_FILE = "theme.json";
+export declare const CMS_ASSETS_FILE = "cms-assets.json";
+export declare const LEGACY_CONFIG_FILES: string[];

package/lib/constants.js

@@ -117,3 +117,13 @@ export const CONFIG_LOCAL_STATE_FLAGS = {
 };
 export const EMPTY_PROJECT = 'empty';
 export const PROJECT_WITH_APP = 'app';
+export const LEGACY_SERVERLESS_FILE = 'serverless.json';
+export const LEGACY_PUBLIC_APP_FILE = 'public-app.json';
+export const LEGACY_PRIVATE_APP_FILE = 'app.json';
+export const THEME_FILE = 'theme.json';
+export const CMS_ASSETS_FILE = 'cms-assets.json';
+export const LEGACY_CONFIG_FILES = [
+    LEGACY_SERVERLESS_FILE,
+    LEGACY_PRIVATE_APP_FILE,
+    LEGACY_PUBLIC_APP_FILE,
+];

package/lib/doctor/DiagnosticInfoBuilder.js

@@ -9,15 +9,16 @@ import { getAccessToken } from '@hubspot/local-dev-lib/personalAccessKey';
 import { walk } from '@hubspot/local-dev-lib/fs';
 import util from 'util';
 import { exec as execAsync } from 'node:child_process';
+import { CMS_ASSETS_FILE, LEGACY_PRIVATE_APP_FILE, LEGACY_PUBLIC_APP_FILE, LEGACY_SERVERLESS_FILE, PROJECT_CONFIG_FILE, THEME_FILE, } from '../constants.js';
 // This needs to be hardcoded since we are using it in the TS type
 const hubspotCli = '@hubspot/cli';
 const configFiles = [
-    'serverless.json',
-    'hsproject.json',
-    'app.json',
-    'public-app.json',
-    'theme.json',
-    'cms-assets.json',
+    LEGACY_SERVERLESS_FILE,
+    PROJECT_CONFIG_FILE,
+    LEGACY_PRIVATE_APP_FILE,
+    LEGACY_PUBLIC_APP_FILE,
+    THEME_FILE,
+    CMS_ASSETS_FILE,
 ];
 export class DiagnosticInfoBuilder {
     accountId;

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

@@ -67,6 +67,8 @@ describe('AppDevModeInterface', () => {
             componentRoot: '/test/path',
             componentConfigPath: '/test/path/config.json',
             configUpdatedSinceLastUpload: false,
+            removed: false,
+            parsingErrors: [],
         },
         componentDeps: {},
         metaFilePath: '/test/path',

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

@@ -61,37 +61,66 @@ describe('LocalDevWebsocketServer', () => {
             expect(mockWebSocketServer.on).toHaveBeenCalledWith('connection', expect.any(Function));
             expect(logger.log).toHaveBeenCalled();
         });
-        it('should accept connection from valid origin', async () => {
-            isPortManagerServerRunning.mockResolvedValue(true);
-            requestPorts.mockResolvedValue({
-                'local-dev-ui-websocket-server': 1234,
+        describe('valid origins', () => {
+            const validOrigins = [
+                'https://app.hubspot.com',
+                'https://app.hubspotqa.com',
+                'https://local.hubspot.com',
+                'https://local.hubspotqa.com',
+                'https://app-na2.hubspot.com',
+                'https://app-na2.hubspotqa.com',
+                'https://app-na3.hubspot.com',
+                'https://app-na3.hubspotqa.com',
+                'https://app-ap1.hubspot.com',
+                'https://app-ap1.hubspotqa.com',
+                'https://app-eu1.hubspot.com',
+                'https://app-eu1.hubspotqa.com',
+            ];
+            validOrigins.forEach(origin => {
+                it(`should accept connection from ${origin}`, async () => {
+                    isPortManagerServerRunning.mockResolvedValue(true);
+                    requestPorts.mockResolvedValue({
+                        'local-dev-ui-websocket-server': 1234,
+                    });
+                    await server.start();
+                    // Get the connection callback
+                    const connectionCallback = mockWebSocketServer.on.mock
+                        .calls[0][1];
+                    // Simulate connection from valid origin
+                    connectionCallback(mockWebSocket, {
+                        headers: { origin },
+                    });
+                    expect(mockWebSocket.on).toHaveBeenCalledWith('message', expect.any(Function));
+                    expect(mockLocalDevProcess.addStateListener).toHaveBeenCalledWith('projectNodes', expect.any(Function));
+                    expect(mockWebSocket.close).not.toHaveBeenCalled();
+                });
             });
-            await server.start();
-            // Get the connection callback
-            const connectionCallback = mockWebSocketServer.on.mock.calls[0][1];
-            // Simulate connection from valid origin
-            connectionCallback(mockWebSocket, {
-                headers: { origin: 'https://app.hubspot.com' },
-            });
-            expect(mockWebSocket.on).toHaveBeenCalledWith('message', expect.any(Function));
-            expect(mockLocalDevProcess.addStateListener).toHaveBeenCalledWith('projectNodes', expect.any(Function));
-            expect(mockWebSocket.close).not.toHaveBeenCalled();
         });
-        it('should reject connection from invalid origin', async () => {
-            isPortManagerServerRunning.mockResolvedValue(true);
-            requestPorts.mockResolvedValue({
-                'local-dev-ui-websocket-server': 1234,
-            });
-            await server.start();
-            // Get the connection callback
-            const connectionCallback = mockWebSocketServer.on.mock.calls[0][1];
-            // Simulate connection from invalid origin
-            connectionCallback(mockWebSocket, {
-                headers: { origin: 'https://malicious-site.com' },
+        describe('invalid origins', () => {
+            const invalidOrigins = [
+                'https://malicious-site.com',
+                'https://app.malicious-site.com',
+                'https://app.hubspot.com.evil.com',
+            ];
+            invalidOrigins.forEach(origin => {
+                it(`should reject connection from "${origin}"`, async () => {
+                    isPortManagerServerRunning.mockResolvedValue(true);
+                    requestPorts.mockResolvedValue({
+                        'local-dev-ui-websocket-server': 1234,
+                    });
+                    await server.start();
+                    // Get the connection callback
+                    const connectionCallback = mockWebSocketServer.on.mock
+                        .calls[0][1];
+                    // Simulate connection from invalid origin
+                    connectionCallback(mockWebSocket, {
+                        headers: { origin },
+                    });
+                    expect(mockWebSocket.close).toHaveBeenCalledWith(1008, lib.LocalDevWebsocketServer.errors.originNotAllowed(origin));
+                    expect(mockWebSocket.on).not.toHaveBeenCalled();
+                    expect(mockLocalDevProcess.addStateListener).not.toHaveBeenCalled();
+                });
             });
-            expect(mockWebSocket.close).toHaveBeenCalledWith(1008, lib.LocalDevWebsocketServer.errors.originNotAllowed('https://malicious-site.com'));
-            expect(mockWebSocket.on).not.toHaveBeenCalled();
-            expect(mockLocalDevProcess.addStateListener).not.toHaveBeenCalled();
         });
         it('should reject connection with no origin header', async () => {
             isPortManagerServerRunning.mockResolvedValue(true);
@@ -119,7 +148,7 @@ describe('LocalDevWebsocketServer', () => {
             const connectionCallback = mockWebSocketServer.on.mock.calls[0][1];
             // Simulate connection from valid origin
             connectionCallback(mockWebSocket, {
-                headers: { origin: 'https://app.hubspot.com' },
+                headers: { origin: 'https://app-na3.hubspot.com' },
             });
             expect(mockLocalDevProcess.sendDevServerMessage).toHaveBeenCalledWith(LOCAL_DEV_SERVER_MESSAGE_TYPES.WEBSOCKET_SERVER_CONNECTED);
         });
@@ -213,7 +242,7 @@ describe('LocalDevWebsocketServer', () => {
                 headers: { origin: 'https://app.hubspot.com' },
             });
             connectionCallback(mockWebSocket2, {
-                headers: { origin: 'https://app.hubspotqa.com' },
+                headers: { origin: 'https://app-na2.hubspotqa.com' },
             });
             connectionCallback(mockWebSocket3, {
                 headers: { origin: 'https://local.hubspot.com' },
@@ -238,7 +267,7 @@ describe('LocalDevWebsocketServer', () => {
                 headers: { origin: 'https://app.hubspot.com' },
             });
             connectionCallback(mockWebSocket2, {
-                headers: { origin: 'https://app.hubspotqa.com' },
+                headers: { origin: 'https://app-eu1.hubspotqa.com' },
             });
             // Each websocket should receive project data
             expect(mockWebSocket1.send).toHaveBeenCalledWith(JSON.stringify({
@@ -270,7 +299,7 @@ describe('LocalDevWebsocketServer', () => {
                 headers: { origin: 'https://app.hubspot.com' },
             });
             connectionCallback(mockWebSocket2, {
-                headers: { origin: 'https://app.hubspotqa.com' },
+                headers: { origin: 'https://app-ap1.hubspotqa.com' },
             });
             // Get all the close callbacks for both connections (there should be 2 per connection)
             const closeCallbacks1 = mockWebSocket1.on.mock.calls
@@ -296,7 +325,7 @@ describe('LocalDevWebsocketServer', () => {
                 headers: { origin: 'https://app.hubspot.com' },
             });
             connectionCallback(mockWebSocket2, {
-                headers: { origin: 'https://app.hubspotqa.com' },
+                headers: { origin: 'https://local.hubspotqa.com' },
             });
             // Get the projectNodes listeners that were registered
             const projectNodesListeners = mockLocalDevProcess.addStateListener.mock.calls
@@ -312,6 +341,8 @@ describe('LocalDevWebsocketServer', () => {
                         componentRoot: '/test/path',
                         componentConfigPath: '/test/path/config.json',
                         configUpdatedSinceLastUpload: false,
+                        removed: false,
+                        parsingErrors: [],
                     },
                     componentDeps: {},
                     metaFilePath: '/test/path',

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

@@ -30,6 +30,8 @@ describe('isDeployedProjectUpToDateWithLocal', () => {
             componentRoot: '/local/path',
             componentConfigPath: '/local/path/config.json',
             configUpdatedSinceLastUpload: false,
+            removed: false,
+            parsingErrors: [],
         },
         componentDeps: {},
         metaFilePath: '/local/path',

package/lib/projects/__tests__/upload.test.d.ts

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

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

@@ -0,0 +1,82 @@
+import fs from 'fs-extra';
+import os from 'os';
+import path from 'path';
+import { vi } from 'vitest';
+import { validateSourceDirectory } from '../upload.js';
+import { uiLogger } from '../../ui/logger.js';
+import { lib } from '../../../lang/en.js';
+import { useV3Api } from '../platformVersion.js';
+import ProjectValidationError from '../../errors/ProjectValidationError.js';
+import { walk } from '@hubspot/local-dev-lib/fs';
+// Mock dependencies
+vi.mock('../../ui/logger.js');
+vi.mock('../platformVersion.js');
+vi.mock('@hubspot/local-dev-lib/fs');
+describe('lib/projects/upload', () => {
+    describe('validateSourceDirectory', () => {
+        let tempDir;
+        let srcDir;
+        let projectConfig;
+        beforeEach(() => {
+            tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'upload-test-'));
+            srcDir = path.join(tempDir, 'src');
+            fs.mkdirSync(srcDir, { recursive: true });
+            projectConfig = {
+                name: 'test-project',
+                srcDir: 'src',
+                platformVersion: '2025.2',
+            };
+            vi.clearAllMocks();
+        });
+        afterEach(() => {
+            fs.removeSync(tempDir);
+        });
+        it('should throw ProjectValidationError when source directory is empty', async () => {
+            vi.mocked(walk).mockResolvedValue([]);
+            await expect(validateSourceDirectory(srcDir, projectConfig, tempDir)).rejects.toThrow(ProjectValidationError);
+            expect(walk).toHaveBeenCalledWith(srcDir, ['node_modules']);
+        });
+        it('should warn about legacy files in V3 projects', async () => {
+            vi.mocked(useV3Api).mockReturnValue(true);
+            const legacyFilePath = path.join(srcDir, 'app', 'serverless.json');
+            vi.mocked(walk).mockResolvedValue([legacyFilePath]);
+            await validateSourceDirectory(srcDir, projectConfig, tempDir);
+            expect(uiLogger.warn).toHaveBeenCalledWith(lib.projectUpload.handleProjectUpload.legacyFileDetected('src/app/serverless.json', '2025.2'));
+        });
+        it('should warn about multiple legacy files', async () => {
+            vi.mocked(useV3Api).mockReturnValue(true);
+            const filePaths = [
+                path.join(srcDir, 'app1', 'serverless.json'),
+                path.join(srcDir, 'app2', 'app.json'),
+                path.join(srcDir, 'app3', 'public-app.json'),
+            ];
+            vi.mocked(walk).mockResolvedValue(filePaths);
+            await validateSourceDirectory(srcDir, projectConfig, tempDir);
+            expect(uiLogger.warn).toHaveBeenCalledTimes(3);
+            expect(uiLogger.warn).toHaveBeenCalledWith(lib.projectUpload.handleProjectUpload.legacyFileDetected('src/app1/serverless.json', '2025.2'));
+            expect(uiLogger.warn).toHaveBeenCalledWith(lib.projectUpload.handleProjectUpload.legacyFileDetected('src/app2/app.json', '2025.2'));
+            expect(uiLogger.warn).toHaveBeenCalledWith(lib.projectUpload.handleProjectUpload.legacyFileDetected('src/app3/public-app.json', '2025.2'));
+        });
+        it('should not warn about non-legacy files', async () => {
+            vi.mocked(useV3Api).mockReturnValue(true);
+            const filePaths = [
+                path.join(srcDir, 'component.js'),
+                path.join(srcDir, 'config.json'),
+            ];
+            vi.mocked(walk).mockResolvedValue(filePaths);
+            await validateSourceDirectory(srcDir, projectConfig, tempDir);
+            expect(uiLogger.warn).not.toHaveBeenCalled();
+        });
+        it('should not warn about legacy files in non-V3 projects', async () => {
+            vi.mocked(useV3Api).mockReturnValue(false);
+            projectConfig.platformVersion = '2025.1';
+            const filePaths = [
+                path.join(srcDir, 'app', 'serverless.json'),
+                path.join(srcDir, 'app', 'app.json'),
+            ];
+            vi.mocked(walk).mockResolvedValue(filePaths);
+            await validateSourceDirectory(srcDir, projectConfig, tempDir);
+            expect(uiLogger.warn).not.toHaveBeenCalled();
+        });
+    });
+});

package/lib/projects/localDev/AppDevModeInterface.d.ts

@@ -15,6 +15,8 @@ declare class AppDevModeInterface {
     private get appNode();
     private get appData();
     private set appData(value);
+    private isStaticAuthApp;
+    private isOAuthApp;
     private getAppInstallUrl;
     private fetchAppData;
     private checkMarketplaceAppInstalls;

package/lib/projects/localDev/AppDevModeInterface.js

@@ -54,6 +54,12 @@ class AppDevModeInterface {
         }
         this.localDevState.setAppDataForUid(this.appNode.uid, appData);
     }
+    isStaticAuthApp() {
+        return (this.appNode?.config.auth.type.toLowerCase() === APP_AUTH_TYPES.STATIC);
+    }
+    isOAuthApp() {
+        return (this.appNode?.config.auth.type.toLowerCase() === APP_AUTH_TYPES.OAUTH);
+    }
     // @TODO: Restore test account auto install functionality
     // private isAutomaticallyInstallable(): boolean {
     //   const targetTestingAccount = getAccountConfig(
@@ -71,11 +77,11 @@ class AppDevModeInterface {
     //   return (
     //     isTestAccount &&
     //     hasCorrectParent &&
-    //     this.appNode?.config.auth.type === APP_AUTH_TYPES.STATIC
+    //     this.isStaticAuthApp()
     //   );
     // }
     async getAppInstallUrl() {
-        if (this.appNode?.config.auth.type.toLowerCase() === APP_AUTH_TYPES.OAUTH) {
+        if (this.appNode && this.isOAuthApp()) {
             return getOauthAppInstallUrl({
                 targetAccountId: this.localDevState.targetTestingAccountId,
                 env: this.localDevState.env,
@@ -177,13 +183,16 @@ class AppDevModeInterface {
         //     );
         //   }
         // }
+        const staticAuthInstallOptions = this.isStaticAuthApp()
+            ? {
+                testingAccountId: this.localDevState.targetTestingAccountId,
+                projectAccountId: this.localDevState.targetProjectAccountId,
+                projectName: this.localDevState.projectConfig.name,
+                appUid: this.appNode.uid,
+            }
+            : undefined;
         const installUrl = await this.getAppInstallUrl();
-        await installAppBrowserPrompt(installUrl, isReinstall, {
-            testingAccountId: this.localDevState.targetTestingAccountId,
-            projectAccountId: this.localDevState.targetProjectAccountId,
-            projectName: this.localDevState.projectConfig.name,
-            appUid: this.appNode.uid,
-        });
+        await installAppBrowserPrompt(installUrl, isReinstall, staticAuthInstallOptions);
     }
     async checkTestAccountAppInstallation() {
         if (!this.appNode || !this.appData) {

package/lib/projects/localDev/LocalDevWebsocketServer.d.ts

@@ -3,7 +3,6 @@ declare class LocalDevWebsocketServer {
     private server?;
     private debug?;
     private localDevProcess;
-    private ALLOWED_ORIGINS;
     constructor(localDevProcess: LocalDevProcess, debug?: boolean);
     private log;
     private logError;