Remove all instances of loadAndValidateOptions

HubSpot · Dec 18, 2024 · ad5b50d · ad5b50d
1 parent 273b804
commit ad5b50d
Show file tree

Hide file tree

Showing 54 changed files with 15 additions and 197 deletions.
diff --git a/commands/account/clean.ts b/commands/account/clean.ts
@@ -6,7 +6,6 @@ const {
 
 const { trackCommandUsage } = require('../../lib/usageTracking');
 const { i18n } = require('../../lib/lang');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { EXIT_CODES } = require('../../lib/enums/exitCodes');
 const { addTestingOptions, addConfigOptions } = require('../../lib/commonOpts');
 const { promptUser } = require('../../lib/prompts/promptUtils');
@@ -29,7 +28,6 @@ exports.describe = i18n(`${i18nKey}.describe`);
 
 exports.handler = async options => {
   const { qa } = options;
-  await loadAndValidateOptions(options, false);
 
   trackCommandUsage('accounts-clean', null);
 

diff --git a/commands/account/info.ts b/commands/account/info.ts
@@ -3,7 +3,6 @@ const { logger } = require('@hubspot/local-dev-lib/logger');
 const { getAccountConfig } = require('@hubspot/local-dev-lib/config');
 const { getAccessToken } = require('@hubspot/local-dev-lib/personalAccessKey');
 const { addConfigOptions } = require('../../lib/commonOpts');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { i18n } = require('../../lib/lang');
 const { getTableContents } = require('../../lib/ui/table');
 
@@ -13,8 +12,6 @@ exports.describe = i18n(`${i18nKey}.describe`);
 exports.command = 'info [account]';
 
 exports.handler = async options => {
-  await loadAndValidateOptions(options);
-
   const { derivedAccountId } = options;
   const config = getAccountConfig(derivedAccountId);
   // check if the provided account is using a personal access key, if not, show an error

diff --git a/commands/account/list.ts b/commands/account/list.ts
@@ -12,7 +12,6 @@ const { addConfigOptions } = require('../../lib/commonOpts');
 const { getTableContents, getTableHeader } = require('../../lib/ui/table');
 
 const { trackCommandUsage } = require('../../lib/usageTracking');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { isSandbox, isDeveloperTestAccount } = require('../../lib/accountTypes');
 
 const { i18n } = require('../../lib/lang');
@@ -81,8 +80,6 @@ const getPortalData = mappedPortalData => {
 };
 
 exports.handler = async options => {
-  await loadAndValidateOptions(options, false);
-
   const { derivedAccountId } = options;
 
   trackCommandUsage('accounts-list', null, derivedAccountId);

diff --git a/commands/account/remove.ts b/commands/account/remove.ts
@@ -13,15 +13,13 @@ const {
 const { trackCommandUsage } = require('../../lib/usageTracking');
 const { i18n } = require('../../lib/lang');
 const { selectAccountFromConfig } = require('../../lib/prompts/accountsPrompt');
-const { loadAndValidateOptions } = require('../../lib/validation');
 
 const i18nKey = 'commands.account.subcommands.remove';
 
 exports.command = 'remove [account]';
 exports.describe = i18n(`${i18nKey}.describe`);
 
 exports.handler = async options => {
-  await loadAndValidateOptions(options, false);
   const { account } = options;
   let accountToRemove = account;
 

diff --git a/commands/account/rename.ts b/commands/account/rename.ts
@@ -4,7 +4,6 @@ const { renameAccount } = require('@hubspot/local-dev-lib/config');
 
 const { addConfigOptions, addAccountOptions } = require('../../lib/commonOpts');
 const { trackCommandUsage } = require('../../lib/usageTracking');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { i18n } = require('../../lib/lang');
 
 const i18nKey = 'commands.account.subcommands.rename';
@@ -13,8 +12,6 @@ exports.command = 'rename <accountName> <newName>';
 exports.describe = i18n(`${i18nKey}.describe`);
 
 exports.handler = async options => {
-  loadAndValidateOptions(options);
-
   const { accountName, newName, derivedAccountId } = options;
 
   trackCommandUsage('accounts-rename', null, derivedAccountId);

diff --git a/commands/account/use.ts b/commands/account/use.ts
@@ -9,16 +9,13 @@ const {
 const { trackCommandUsage } = require('../../lib/usageTracking');
 const { i18n } = require('../../lib/lang');
 const { selectAccountFromConfig } = require('../../lib/prompts/accountsPrompt');
-const { loadAndValidateOptions } = require('../../lib/validation');
 
 const i18nKey = 'commands.account.subcommands.use';
 
 exports.command = 'use [account]';
 exports.describe = i18n(`${i18nKey}.describe`);
 
 exports.handler = async options => {
-  await loadAndValidateOptions(options, false);
-
   let newDefaultAccount = options.account;
 
   if (!newDefaultAccount) {

diff --git a/commands/cms/lighthouseScore.ts b/commands/cms/lighthouseScore.ts
@@ -7,7 +7,6 @@ const {
 } = require('../../lib/commonOpts');
 const { logger } = require('@hubspot/local-dev-lib/logger');
 const { getTableContents, getTableHeader } = require('../../lib/ui/table');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { promptUser } = require('../../lib/prompts/promptUtils');
 const { i18n } = require('../../lib/lang');
 const { fetchThemes } = require('@hubspot/local-dev-lib/api/designManager');
@@ -67,7 +66,6 @@ const selectTheme = async accountId => {
 };
 
 exports.handler = async options => {
-  await loadAndValidateOptions(options);
   const { target, verbose, theme, derivedAccountId } = options;
 
   const includeDesktopScore = target === 'desktop' || !verbose;

diff --git a/commands/config/set.ts b/commands/config/set.ts
@@ -1,5 +1,4 @@
 // @ts-nocheck
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { i18n } = require('../../lib/lang');
 const { trackCommandUsage } = require('../../lib/usageTracking');
 const { promptUser } = require('../../lib/prompts/promptUtils');
@@ -55,8 +54,6 @@ const handleConfigUpdate = async (accountId, options) => {
 };
 
 exports.handler = async options => {
-  await loadAndValidateOptions(options);
-
   const { derivedAccountId } = options;
 
   trackCommandUsage('config-set', null, derivedAccountId);

diff --git a/commands/customObject/create.ts b/commands/customObject/create.ts
@@ -4,10 +4,7 @@ import { inputPrompt } from '../../lib/prompts/promptUtils';
 const { logger } = require('@hubspot/local-dev-lib/logger');
 const { logError } = require('../../lib/errorHandlers/index');
 const { getAbsoluteFilePath } = require('@hubspot/local-dev-lib/path');
-const {
-  checkAndConvertToJson,
-  loadAndValidateOptions,
-} = require('../../lib/validation');
+const { checkAndConvertToJson } = require('../../lib/validation');
 const { trackCommandUsage } = require('../../lib/usageTracking');
 const {
   batchCreateObjects,
@@ -24,8 +21,6 @@ exports.handler = async options => {
   const { path, name: providedName, derivedAccountId } = options;
   let definitionPath = path;
 
-  await loadAndValidateOptions(options);
-
   trackCommandUsage('custom-object-batch-create', null, derivedAccountId);
 
   if (!definitionPath) {

diff --git a/commands/customObject/schema/create.ts b/commands/customObject/schema/create.ts
@@ -2,10 +2,7 @@
 const { logger } = require('@hubspot/local-dev-lib/logger');
 const { logError } = require('../../../lib/errorHandlers/index');
 const { getAbsoluteFilePath } = require('@hubspot/local-dev-lib/path');
-const {
-  checkAndConvertToJson,
-  loadAndValidateOptions,
-} = require('../../../lib/validation');
+const { checkAndConvertToJson } = require('../../../lib/validation');
 const { trackCommandUsage } = require('../../../lib/usageTracking');
 const { addTestingOptions } = require('../../../lib/commonOpts');
 const {
@@ -34,8 +31,6 @@ exports.describe = i18n(`${i18nKey}.describe`);
 exports.handler = async options => {
   const { path, derivedAccountId } = options;
 
-  await loadAndValidateOptions(options);
-
   trackCommandUsage('custom-object-schema-create', null, derivedAccountId);
 
   const filePath = getAbsoluteFilePath(path);

diff --git a/commands/customObject/schema/delete.ts b/commands/customObject/schema/delete.ts
@@ -4,7 +4,6 @@ import { confirmPrompt, listPrompt } from '../../../lib/prompts/promptUtils';
 import { fetchObjectSchemas } from '@hubspot/local-dev-lib/api/customObjects';
 
 const { logger } = require('@hubspot/local-dev-lib/logger');
-const { loadAndValidateOptions } = require('../../../lib/validation');
 const { trackCommandUsage } = require('../../../lib/usageTracking');
 const {
   deleteObjectSchema,
@@ -20,8 +19,6 @@ exports.describe = i18n(`${i18nKey}.describe`);
 exports.handler = async options => {
   const { name: providedName, force, derivedAccountId } = options;
 
-  await loadAndValidateOptions(options);
-
   trackCommandUsage('custom-object-schema-delete', null, derivedAccountId);
 
   let name;

diff --git a/commands/customObject/schema/fetch-all.ts b/commands/customObject/schema/fetch-all.ts
@@ -2,7 +2,6 @@
 import { inputPrompt } from '../../../lib/prompts/promptUtils';
 
 const { logger } = require('@hubspot/local-dev-lib/logger');
-const { loadAndValidateOptions } = require('../../../lib/validation');
 const { trackCommandUsage } = require('../../../lib/usageTracking');
 const {
   downloadSchemas,
@@ -18,8 +17,6 @@ exports.command = 'fetch-all [dest]';
 exports.describe = i18n(`${i18nKey}.describe`);
 
 exports.handler = async options => {
-  await loadAndValidateOptions(options);
-
   const { derivedAccountId, dest: providedDest } = options;
 
   trackCommandUsage('custom-object-schema-fetch-all', null, derivedAccountId);

diff --git a/commands/customObject/schema/fetch.ts b/commands/customObject/schema/fetch.ts
@@ -13,7 +13,6 @@ const {
 const { fetchSchema } = require('@hubspot/local-dev-lib/api/fileTransport');
 const { getCwd } = require('@hubspot/local-dev-lib/path');
 
-const { loadAndValidateOptions } = require('../../../lib/validation');
 const { trackCommandUsage } = require('../../../lib/usageTracking');
 const { i18n } = require('../../../lib/lang');
 const { logError } = require('../../../lib/errorHandlers');
@@ -26,8 +25,6 @@ exports.describe = i18n(`${i18nKey}.describe`);
 exports.handler = async options => {
   const { name: providedName, dest: providedDest, derivedAccountId } = options;
 
-  await loadAndValidateOptions(options);
-
   trackCommandUsage('custom-object-schema-fetch', null, derivedAccountId);
   let name;
 

diff --git a/commands/customObject/schema/list.ts b/commands/customObject/schema/list.ts
@@ -2,7 +2,6 @@
 const { logger } = require('@hubspot/local-dev-lib/logger');
 const { logError } = require('../../../lib/errorHandlers/index');
 
-const { loadAndValidateOptions } = require('../../../lib/validation');
 const { trackCommandUsage } = require('../../../lib/usageTracking');
 const { listSchemas } = require('../../../lib/schema');
 const { i18n } = require('../../../lib/lang');
@@ -13,8 +12,6 @@ exports.command = 'list';
 exports.describe = i18n(`${i18nKey}.describe`);
 
 exports.handler = async options => {
-  await loadAndValidateOptions(options);
-
   const { derivedAccountId } = options;
 
   trackCommandUsage('custom-object-schema-list', null, derivedAccountId);

diff --git a/commands/customObject/schema/update.ts b/commands/customObject/schema/update.ts
@@ -8,10 +8,7 @@ const { getAbsoluteFilePath } = require('@hubspot/local-dev-lib/path');
 const {
   ENVIRONMENTS,
 } = require('@hubspot/local-dev-lib/constants/environments');
-const {
-  checkAndConvertToJson,
-  loadAndValidateOptions,
-} = require('../../../lib/validation');
+const { checkAndConvertToJson } = require('../../../lib/validation');
 const { trackCommandUsage } = require('../../../lib/usageTracking');
 const { addTestingOptions } = require('../../../lib/commonOpts');
 const { CONFIG_FLAGS } = require('../../../lib/constants');
@@ -37,8 +34,6 @@ exports.describe = i18n(`${i18nKey}.describe`);
 exports.handler = async options => {
   const { path, name: providedName, derivedAccountId } = options;
 
-  await loadAndValidateOptions(options);
-
   trackCommandUsage('custom-object-schema-update', null, derivedAccountId);
 
   const filePath = getAbsoluteFilePath(path);

diff --git a/commands/fetch.ts b/commands/fetch.ts
@@ -11,10 +11,7 @@ const {
   addGlobalOptions,
 } = require('../lib/commonOpts');
 const { resolveLocalPath } = require('../lib/filesystem');
-const {
-  validateCmsPublishMode,
-  loadAndValidateOptions,
-} = require('../lib/validation');
+const { validateCmsPublishMode } = require('../lib/validation');
 const { trackCommandUsage } = require('../lib/usageTracking');
 const { i18n } = require('../lib/lang');
 
@@ -28,8 +25,6 @@ exports.describe = i18n(`${i18nKey}.describe`);
 exports.handler = async options => {
   const { src, dest } = options;
 
-  await loadAndValidateOptions(options);
-
   if (!validateCmsPublishMode(options)) {
     process.exit(EXIT_CODES.ERROR);
   }

diff --git a/commands/filemanager/fetch.ts b/commands/filemanager/fetch.ts
@@ -9,7 +9,6 @@ const {
   addGlobalOptions,
   addUseEnvironmentOptions,
 } = require('../../lib/commonOpts');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { trackCommandUsage } = require('../../lib/usageTracking');
 const { i18n } = require('../../lib/lang');
 
@@ -23,8 +22,6 @@ exports.describe = i18n(`${i18nKey}.describe`);
 exports.handler = async options => {
   const { src, includeArchived, derivedAccountId, overwrite } = options;
 
-  await loadAndValidateOptions(options);
-
   if (typeof src !== 'string') {
     logger.error(i18n(`${i18nKey}.errors.sourceRequired`));
     process.exit(EXIT_CODES.ERROR);

diff --git a/commands/filemanager/upload.ts b/commands/filemanager/upload.ts
@@ -18,7 +18,6 @@ const {
   addAccountOptions,
   addUseEnvironmentOptions,
 } = require('../../lib/commonOpts');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { trackCommandUsage } = require('../../lib/usageTracking');
 const { i18n } = require('../../lib/lang');
 
@@ -31,8 +30,6 @@ exports.describe = i18n(`${i18nKey}.describe`);
 exports.handler = async options => {
   const { src, dest, derivedAccountId } = options;
 
-  await loadAndValidateOptions(options);
-
   const absoluteSrcPath = path.resolve(getCwd(), src);
 
   let stats;

diff --git a/commands/function/deploy.ts b/commands/function/deploy.ts
@@ -14,7 +14,6 @@ const {
   buildPackage,
   getBuildStatus,
 } = require('@hubspot/local-dev-lib/api/functions');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { outputBuildLog } = require('../../lib/serverlessLogs');
 const { i18n } = require('../../lib/lang');
 const { isHubSpotHttpError } = require('@hubspot/local-dev-lib/errors/index');
@@ -25,8 +24,6 @@ exports.command = 'deploy <path>';
 exports.describe = false;
 
 exports.handler = async options => {
-  await loadAndValidateOptions(options);
-
   const { path: functionPath, derivedAccountId } = options;
   const splitFunctionPath = functionPath.split('.');
 

diff --git a/commands/function/list.ts b/commands/function/list.ts
@@ -10,7 +10,6 @@ const {
   addUseEnvironmentOptions,
 } = require('../../lib/commonOpts');
 const { trackCommandUsage } = require('../../lib/usageTracking');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { i18n } = require('../../lib/lang');
 
 const i18nKey = 'commands.function.subcommands.list';
@@ -20,8 +19,6 @@ exports.command = ['list', 'ls'];
 exports.describe = i18n(`${i18nKey}.describe`);
 
 exports.handler = async options => {
-  loadAndValidateOptions(options);
-
   const { derivedAccountId } = options;
 
   trackCommandUsage('functions-list', null, derivedAccountId);

diff --git a/commands/function/server.ts b/commands/function/server.ts
@@ -7,7 +7,6 @@ const {
 const { trackCommandUsage } = require('../../lib/usageTracking');
 const { logger } = require('@hubspot/local-dev-lib/logger');
 const { start: startTestServer } = require('@hubspot/serverless-dev-runtime');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { i18n } = require('../../lib/lang');
 
 const i18nKey = 'commands.function.subcommands.server';
@@ -16,8 +15,6 @@ exports.command = 'server <path>';
 exports.describe = false;
 
 exports.handler = async options => {
-  await loadAndValidateOptions(options);
-
   const { path: functionPath, derivedAccountId } = options;
 
   trackCommandUsage('functions-server', null, derivedAccountId);

diff --git a/commands/hubdb/clear.ts b/commands/hubdb/clear.ts
@@ -6,7 +6,6 @@ const { publishTable } = require('@hubspot/local-dev-lib/api/hubdb');
 const {
   selectHubDBTablePrompt,
 } = require('../../lib/prompts/selectHubDBTablePrompt');
-const { loadAndValidateOptions } = require('../../lib/validation');
 const { trackCommandUsage } = require('../../lib/usageTracking');
 const {
   addConfigOptions,
@@ -23,8 +22,6 @@ exports.describe = i18n(`${i18nKey}.describe`);
 exports.handler = async options => {
   const { derivedAccountId } = options;
 
-  await loadAndValidateOptions(options);
-
   trackCommandUsage('hubdb-clear', null, derivedAccountId);
 
   try {