netlify-cli 17.10.0 → 17.10.2

package/src/lib/functions/runtimes/js/builders/zisi.js

@@ -142,7 +142,7 @@ export default async function handler({ config, directory, errorExit, func, meta
     }
     else {
         // We must use esbuild for certain file extensions.
-        const mustTranspile = ['.js', '.ts', '.mts', '.cts'].includes(path.extname(func.mainFile));
+        const mustTranspile = ['.mjs', '.ts', '.mts', '.cts'].includes(path.extname(func.mainFile));
         const mustUseEsbuild = hasTypeModule || mustTranspile;
         if (mustUseEsbuild && !functionsConfig['*'].nodeBundler) {
             functionsConfig['*'].nodeBundler = 'esbuild';

package/src/lib/functions/runtimes/rust/index.js

@@ -3,7 +3,6 @@ import { dirname, extname, join, resolve } from 'path';
 import { platform } from 'process';
 import { findUp } from 'find-up';
 import toml from 'toml';
-// @ts-expect-error TS(7034) FIXME: Variable 'execa' implicitly has type 'any' in some... Remove this comment to see the full error message
 import execa from '../../../../utils/execa.js';
 import { SERVE_FUNCTIONS_FOLDER } from '../../../../utils/functions/functions.js';
 import { getPathInProject } from '../../../settings.js';
@@ -18,7 +17,6 @@ const build = async ({ func }) => {
     const crateName = await getCrateName(functionDirectory);
     const binaryName = `${crateName}${isWindows ? '.exe' : ''}`;
     const binaryPath = join(targetDirectory, 'debug', binaryName);
-    // @ts-expect-error TS(7005) FIXME: Variable 'execa' implicitly has an 'any' type.
     await execa('cargo', ['build', '--target-dir', targetDirectory], {
         cwd: functionDirectory,
     });

package/src/lib/functions/server.js

@@ -25,20 +25,31 @@ const buildClientContext = function (headers) {
     const parts = headers.authorization.split(' ');
     if (parts.length !== 2 || parts[0] !== 'Bearer')
         return;
+    const identity = {
+        url: 'https://netlify-dev-locally-emulated-identity.netlify.com/.netlify/identity',
+        token: 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzb3VyY2UiOiJuZXRsaWZ5IGRldiIsInRlc3REYXRhIjoiTkVUTElGWV9ERVZfTE9DQUxMWV9FTVVMQVRFRF9JREVOVElUWSJ9.2eSDqUOZAOBsx39FHFePjYj12k0LrxldvGnlvDu3GMI',
+        // you can decode this with https://jwt.io/
+        // just says
+        // {
+        //   "source": "netlify dev",
+        //   "testData": "NETLIFY_DEV_LOCALLY_EMULATED_IDENTITY"
+        // }
+    };
+    // This data is available on both the context root and under custom.netlify for retro-compatibility.
+    // In the future it will only be available in custom.netlify.
+    // @ts-expect-error
+    const user = jwtDecode(parts[1]);
+    const netlifyContext = JSON.stringify({
+        identity: identity,
+        user: user,
+    });
     try {
         return {
-            identity: {
-                url: 'https://netlify-dev-locally-emulated-identity.netlify.com/.netlify/identity',
-                token: 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzb3VyY2UiOiJuZXRsaWZ5IGRldiIsInRlc3REYXRhIjoiTkVUTElGWV9ERVZfTE9DQUxMWV9FTVVMQVRFRF9JREVOVElUWSJ9.2eSDqUOZAOBsx39FHFePjYj12k0LrxldvGnlvDu3GMI',
-                // you can decode this with https://jwt.io/
-                // just says
-                // {
-                //   "source": "netlify dev",
-                //   "testData": "NETLIFY_DEV_LOCALLY_EMULATED_IDENTITY"
-                // }
+            identity: identity,
+            user: user,
+            custom: {
+                netlify: Buffer.from(netlifyContext).toString('base64'),
             },
-            // @ts-expect-error
-            user: jwtDecode(parts[1]),
         };
     }
     catch {

package/src/utils/banner.js

@@ -1,8 +1,7 @@
 import boxen from 'boxen';
 import { chalk, log, NETLIFYDEVLOG } from './command-helpers.js';
-// @ts-expect-error TS(7031) FIXME: Binding element 'url' implicitly has an 'any' type... Remove this comment to see the full error message
-export const printBanner = ({ url }) => {
-    const banner = chalk.bold(`${NETLIFYDEVLOG} Server now ready on ${url}`);
+export const printBanner = (options) => {
+    const banner = chalk.bold(`${NETLIFYDEVLOG} Server now ready on ${options.url}`);
     log(boxen(banner, {
         padding: 1,
         margin: 1,

package/src/utils/build-info.js

@@ -3,28 +3,19 @@ import inquirer from 'inquirer';
 import { chalk, log } from './command-helpers.js';
 /**
  * Filters the inquirer settings based on the input
- * @param {ReturnType<typeof formatSettingsArrForInquirer>} scriptInquirerOptions
- * @param {string} input
  */
-// @ts-expect-error TS(7006) FIXME: Parameter 'scriptInquirerOptions' implicitly has a... Remove this comment to see the full error message
 const filterSettings = function (scriptInquirerOptions, input) {
-    // @ts-expect-error TS(7006) FIXME: Parameter 'scriptInquirerOption' implicitly has an... Remove this comment to see the full error message
     const filterOptions = scriptInquirerOptions.map((scriptInquirerOption) => scriptInquirerOption.name);
     // TODO: remove once https://github.com/sindresorhus/eslint-plugin-unicorn/issues/1394 is fixed
     // eslint-disable-next-line unicorn/no-array-method-this-argument
     const filteredSettings = fuzzy.filter(input, filterOptions);
     const filteredSettingNames = new Set(filteredSettings.map((filteredSetting) => (input ? filteredSetting.string : filteredSetting)));
-    // @ts-expect-error TS(7006) FIXME: Parameter 't' implicitly has an 'any' type.
     return scriptInquirerOptions.filter((t) => filteredSettingNames.has(t.name));
 };
-/** @typedef {import('@netlify/build-info').Settings} Settings */
 /**
- * @param {Settings[]} settings
- * @param {'dev' | 'build'} type The type of command (dev or build)
+ * Formats the settings to present it as an array for the inquirer input so that it can choose one
  */
-// @ts-expect-error TS(7006) FIXME: Parameter 'settings' implicitly has an 'any' type.
 const formatSettingsArrForInquirer = function (settings, type = 'dev') {
-    // @ts-expect-error TS(7006) FIXME: Parameter 'setting' implicitly has an 'any' type.
     return settings.map((setting) => {
         const cmd = type === 'dev' ? setting.devCommand : setting.buildCommand;
         return {
@@ -35,13 +26,26 @@ const formatSettingsArrForInquirer = function (settings, type = 'dev') {
     });
 };
 /**
- * Uses @netlify/build-info to detect the dev settings and port based on the framework
+ * Detects and filters the build setting for a project and a command
+ */
+export async function detectBuildSettings(command) {
+    const { project, workspacePackage } = command;
+    const buildSettings = await project.getBuildSettings(project.workspace ? workspacePackage : '');
+    return buildSettings
+        .filter((setting) => {
+        if (project.workspace && project.relativeBaseDirectory && setting.packagePath) {
+            return project.relativeBaseDirectory.startsWith(setting.packagePath);
+        }
+        return true;
+    })
+        .filter((setting) => setting.devCommand);
+}
+/**
+ * Uses `@netlify/build-info` to detect the dev settings and port based on the framework
  * and the build system that is used.
- * @param {import('../commands/base-command.js').default} command
- * @param {'dev' | 'build'} type The type of command (dev or build)
- * @returns {Promise<Settings | undefined>}
+ * @param command The base command
+ * @param type The type of command (dev or build)
  */
-// @ts-expect-error TS(7006) FIXME: Parameter 'command' implicitly has an 'any' type.
 export const detectFrameworkSettings = async (command, type = 'dev') => {
     const { relConfigFilePath } = command.netlify;
     const settings = await detectBuildSettings(command);
@@ -49,16 +53,14 @@ export const detectFrameworkSettings = async (command, type = 'dev') => {
         return settings[0];
     }
     if (settings.length > 1) {
-        /** multiple matching detectors, make the user choose */
+        // multiple matching detectors, make the user choose
         const scriptInquirerOptions = formatSettingsArrForInquirer(settings, type);
-        /** @type {{chosenSettings: Settings}} */
         const { chosenSettings } = await inquirer.prompt({
             name: 'chosenSettings',
             message: `Multiple possible ${type} commands found`,
-            // @ts-expect-error TS(2769) FIXME: No overload matches this call.
+            // @ts-expect-error is not known by the types as it uses the autocomplete plugin
             type: 'autocomplete',
-            // @ts-expect-error TS(7006) FIXME: Parameter '_' implicitly has an 'any' type.
-            source(/** @type {string} */ _, input = '') {
+            source(_, input = '') {
                 if (!input)
                     return scriptInquirerOptions;
                 // only show filtered results
@@ -78,22 +80,3 @@ command = "${chosenSettings.devCommand}"
         return chosenSettings;
     }
 };
-/**
- * Detects and filters the build setting for a project and a command
- * @param {import('../commands/base-command.js').default} command
- */
-// @ts-expect-error TS(7006) FIXME: Parameter 'command' implicitly has an 'any' type.
-export const detectBuildSettings = async (command) => {
-    const { project, workspacePackage } = command;
-    const buildSettings = await project.getBuildSettings(project.workspace ? workspacePackage : '');
-    return (buildSettings
-        // @ts-expect-error TS(7006) FIXME: Parameter 'setting' implicitly has an 'any' type.
-        .filter((setting) => {
-        if (project.workspace && project.relativeBaseDirectory && setting.packagePath) {
-            return project.relativeBaseDirectory.startsWith(setting.packagePath);
-        }
-        return true;
-    })
-        // @ts-expect-error TS(7006) FIXME: Parameter 'setting' implicitly has an 'any' type.
-        .filter((setting) => setting.devCommand));
-};

package/src/utils/command-helpers.js

@@ -158,22 +158,14 @@ export const warn = (message = '') => {
     const bang = chalk.yellow(BANG);
     log(` ${bang}   Warning: ${message}`);
 };
-/**
- * throws an error or log it
- * @param {unknown} message
- * @param {object} [options]
- * @param {boolean} [options.exit]
- */
+/** Throws an error or logs it */
 export const error = (message = '', options = {}) => {
-    const err = 
-    // @ts-expect-error TS(2358) FIXME: The left-hand side of an 'instanceof' expression m... Remove this comment to see the full error message
-    message instanceof Error
+    const err = message instanceof Error
         ? message
         : // eslint-disable-next-line unicorn/no-nested-ternary
             typeof message === 'string'
                 ? new Error(message)
-                : /** @type {Error} */ { message, stack: undefined, name: 'Error' };
-    // @ts-expect-error TS(2339) FIXME: Property 'exit' does not exist on type '{}'.
+                : { message, stack: undefined, name: 'Error' };
     if (options.exit === false) {
         const bang = chalk.red(BANG);
         if (process.env.DEBUG) {

package/src/utils/detect-server-settings.js

@@ -139,11 +139,8 @@ const handleStaticServer = async ({ devConfig, flags, workingDir }) => {
 };
 /**
  * Retrieves the settings from a framework
- * @param {import('@netlify/build-info').Settings} [settings]
- * @returns {import('./types.js').BaseServerSettings | undefined}
  */
-// @ts-expect-error TS(7006) FIXME: Parameter 'settings' implicitly has an 'any' type.
-const getSettingsFromDetectedSettings = (settings) => {
+const getSettingsFromDetectedSettings = (command, settings) => {
     if (!settings) {
         return;
     }
@@ -155,7 +152,8 @@ const getSettingsFromDetectedSettings = (settings) => {
         framework: settings.framework.name,
         env: settings.env,
         pollingStrategies: settings.pollingStrategies,
-        plugins: getPluginsToAutoInstall(settings.plugins_from_config_file, settings.plugins_recommended),
+        plugins: getPluginsToAutoInstall(command, settings.plugins_from_config_file, settings.plugins_recommended),
+        clearPublishDirectory: settings.clearPublishDirectory,
     };
 };
 /**
@@ -190,55 +188,35 @@ const handleCustomFramework = ({ devConfig, workingDir }) => {
  * @param {string} config.workingDir
  * @param {Partial<import('./types.js').BaseServerSettings>=} config.frameworkSettings
  */
-// @ts-expect-error TS(7031) FIXME: Binding element 'devConfig' implicitly has an 'any... Remove this comment to see the full error message
-const mergeSettings = async ({ devConfig, frameworkSettings = {}, workingDir }) => {
-    // @ts-expect-error TS(2339) FIXME: Property 'command' does not exist on type '{}'.
+const mergeSettings = async ({ devConfig, frameworkSettings = {}, workingDir, }) => {
     const command = devConfig.command || frameworkSettings.command;
-    // @ts-expect-error TS(2339) FIXME: Property 'frameworkPort' does not exist on type '{... Remove this comment to see the full error message
     const frameworkPort = devConfig.targetPort || frameworkSettings.frameworkPort;
-    // if the framework doesn't start a server, we use a static one
     const useStaticServer = !(command && frameworkPort);
     return {
-        // @ts-expect-error TS(2339) FIXME: Property 'baseDirectory' does not exist on type '{... Remove this comment to see the full error message
         baseDirectory: devConfig.base || frameworkSettings.baseDirectory,
         command,
         frameworkPort: useStaticServer ? await getStaticServerPort({ devConfig }) : frameworkPort,
-        // @ts-expect-error TS(2339) FIXME: Property 'dist' does not exist on type '{}'.
         dist: devConfig.publish || frameworkSettings.dist || getDefaultDist(workingDir),
-        // @ts-expect-error TS(2339) FIXME: Property 'framework' does not exist on type '{}'.
         framework: frameworkSettings.framework,
-        // @ts-expect-error TS(2339) FIXME: Property 'env' does not exist on type '{}'.
         env: frameworkSettings.env,
-        // @ts-expect-error TS(2339) FIXME: Property 'pollingStrategies' does not exist on typ... Remove this comment to see the full error message
         pollingStrategies: frameworkSettings.pollingStrategies || [],
         useStaticServer,
+        clearPublishDirectory: frameworkSettings.clearPublishDirectory,
     };
 };
 /**
  * Handles a forced framework and retrieves the settings for it
- * @param {object} config
- * @param {import('../commands/dev/types.js').DevConfig} config.devConfig
- * @param {import('@netlify/build-info').Project} config.project
- * @param {string} config.workingDir
- * @param {string=} config.workspacePackage
- * @returns {Promise<import('./types.js').BaseServerSettings>}
  */
-// @ts-expect-error TS(7031) FIXME: Binding element 'devConfig' implicitly has an 'any... Remove this comment to see the full error message
-const handleForcedFramework = async ({ devConfig, project, workingDir, workspacePackage }) => {
+const handleForcedFramework = async (options) => {
     // this throws if `devConfig.framework` is not a supported framework
-    const framework = await getFramework(devConfig.framework, project);
-    const settings = await getSettings(framework, project, workspacePackage || '');
-    const frameworkSettings = getSettingsFromDetectedSettings(settings);
-    return mergeSettings({ devConfig, workingDir, frameworkSettings });
+    const framework = await getFramework(options.devConfig.framework, options.project);
+    const settings = await getSettings(framework, options.project, options.workspacePackage || '');
+    const frameworkSettings = getSettingsFromDetectedSettings(options.command, settings);
+    return mergeSettings({ devConfig: options.devConfig, workingDir: options.workingDir, frameworkSettings });
 };
 /**
  * Get the server settings based on the flags and the devConfig
- * @param {import('../commands/dev/types.js').DevConfig} devConfig
- * @param {import('commander').OptionValues} flags
- * @param {import('../commands/base-command.js').default} command
- * @returns {Promise<import('./types.js').ServerSettings>}
  */
-// @ts-expect-error TS(7006) FIXME: Parameter 'devConfig' implicitly has an 'any' type... Remove this comment to see the full error message
 const detectServerSettings = async (devConfig, flags, command) => {
     validateProperty(devConfig, 'framework', 'string');
     /** @type {Partial<import('./types.js').BaseServerSettings>} */
@@ -251,7 +229,7 @@ const detectServerSettings = async (devConfig, flags, command) => {
         // this is the default CLI behavior
         const runDetection = !hasCommandAndTargetPort(devConfig);
         const frameworkSettings = runDetection
-            ? getSettingsFromDetectedSettings(await detectFrameworkSettings(command, 'dev'))
+            ? getSettingsFromDetectedSettings(command, await detectFrameworkSettings(command, 'dev'))
             : undefined;
         if (frameworkSettings === undefined && runDetection) {
             log(`${NETLIFYDEVWARN} No app server detected. Using simple static server`);
@@ -273,6 +251,7 @@ const detectServerSettings = async (devConfig, flags, command) => {
         validateFrameworkConfig({ devConfig });
         // this is when the user explicitly configures a framework, e.g. `framework = "gatsby"`
         settings = await handleForcedFramework({
+            command,
             devConfig,
             project: command.project,
             workingDir: command.workingDir,

package/src/utils/execa.js

@@ -2,10 +2,7 @@ import { env } from 'process';
 import execaLib from 'execa';
 // This is a thin layer on top of `execa` that allows consumers to provide an
 // alternative path to the module location, making it easier to mock its logic
-// in tests (see `tests/utils/mock-execa.js`).
-/**
- * @type {import('execa')}
- */
+// in tests (see `tests/utils/moc
 // eslint-disable-next-line import/no-mutable-exports
 let execa;
 if (env.NETLIFY_CLI_EXECA_PATH) {

package/src/utils/feature-flags.js

@@ -7,14 +7,8 @@
  * Instead, we return that the feature flag is enabled if it isn't
  * specifically set to false in the response
  * @param {*} siteInfo
- * @param {string} flagName
  *
  * @returns {boolean}
  */
 // @ts-expect-error TS(7006) FIXME: Parameter 'flagName' implicitly has an 'any' type.
-export const isFeatureFlagEnabled = (flagName, siteInfo) => {
-    if (siteInfo.feature_flags && siteInfo.feature_flags[flagName] !== false) {
-        return true;
-    }
-    return false;
-};
+export const isFeatureFlagEnabled = (flagName, siteInfo) => Boolean(siteInfo.feature_flags && siteInfo.feature_flags[flagName] !== false);

package/src/utils/framework-server.js

@@ -1,3 +1,4 @@
+import { rm } from 'node:fs/promises';
 import waitPort from 'wait-port';
 import { startSpinner, stopSpinner } from '../lib/spinner.js';
 import { error, exit, log, NETLIFYDEVERR, NETLIFYDEVLOG } from './command-helpers.js';
@@ -5,19 +6,10 @@ import { runCommand } from './shell.js';
 import { startStaticServer } from './static-server.js';
 // 10 minutes
 const FRAMEWORK_PORT_TIMEOUT = 6e5;
-/**
- * @typedef StartReturnObject
- * @property {4 | 6 | undefined=} ipVersion The version the open port was found on
- */
 /**
  * Start a static server if the `useStaticServer` is provided or a framework specific server
- * @param {object} config
- * @param {import('./types.js').ServerSettings} config.settings
- * @param {string} config.cwd
- * @returns {Promise<StartReturnObject>}
  */
-// @ts-expect-error TS(7031) FIXME: Binding element 'cwd' implicitly has an 'any' type... Remove this comment to see the full error message
-export const startFrameworkServer = async function ({ cwd, settings }) {
+export const startFrameworkServer = async function ({ cwd, settings, }) {
     if (settings.useStaticServer) {
         if (settings.command) {
             runCommand(settings.command, { env: settings.env, cwd });
@@ -29,10 +21,14 @@ export const startFrameworkServer = async function ({ cwd, settings }) {
     const spinner = startSpinner({
         text: `Waiting for framework port ${settings.frameworkPort}. This can be configured using the 'targetPort' property in the netlify.toml`,
     });
+    if (settings.clearPublishDirectory && settings.dist) {
+        await rm(settings.dist, { recursive: true, force: true });
+    }
     runCommand(settings.command, { env: settings.env, spinner, cwd });
     let port;
     try {
         port = await waitPort({
+            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
             port: settings.frameworkPort,
             host: 'localhost',
             output: 'silent',

package/src/utils/init/utils.js

@@ -7,35 +7,32 @@ import { normalizeBackslash } from '../../lib/path.js';
 import { detectBuildSettings } from '../build-info.js';
 import { chalk, error as failAndExit, log, warn } from '../command-helpers.js';
 import { getRecommendPlugins, getUIPlugins } from './plugins.js';
-// these plugins represent runtimes that are
-// expected to be "automatically" installed. Even though
-// they can be installed on package/toml, we always
-// want them installed in the site settings. When installed
-// there our build will automatically install the latest without
-// user management of the versioning.
-const pluginsToAlwaysInstall = new Set(['@netlify/plugin-nextjs']);
+const formatTitle = (title) => chalk.cyan(title);
 /**
  * Retrieve a list of plugins to auto install
- * @param {string[]=} pluginsInstalled
- * @param {string[]=} pluginsRecommended
+ * @param pluginsToAlwaysInstall these plugins represent runtimes that are
+ * expected to be "automatically" installed. Even though
+ * they can be installed on package/toml, we always
+ * want them installed in the site settings. When installed
+ * there our build will automatically install the latest without
+ * user management of the versioning.
+ * @param pluginsInstalled
+ * @param pluginsRecommended
  * @returns
  */
-export const getPluginsToAutoInstall = (pluginsInstalled = [], pluginsRecommended = []) => pluginsRecommended.reduce((acc, plugin) => pluginsInstalled.includes(plugin) && !pluginsToAlwaysInstall.has(plugin) ? acc : [...acc, plugin], 
-/** @type {string[]} */ []);
-/**
- *
- * @param {Partial<import('@netlify/build-info').Settings>} settings
- * @param {*} config
- * @param {import('../../commands/base-command.js').default} command
- */
-// @ts-expect-error TS(7006) FIXME: Parameter 'settings' implicitly has an 'any' type.
+export const getPluginsToAutoInstall = (command, pluginsInstalled = [], pluginsRecommended = []) => {
+    const nextRuntime = '@netlify/plugin-nextjs';
+    const pluginsToAlwaysInstall = new Set([nextRuntime]);
+    return pluginsRecommended.reduce((acc, plugin) => pluginsInstalled.includes(plugin) && !pluginsToAlwaysInstall.has(plugin) ? acc : [...acc, plugin], []);
+};
 const normalizeSettings = (settings, config, command) => {
-    const plugins = getPluginsToAutoInstall(settings.plugins_from_config_file, settings.plugins_recommended);
+    const plugins = getPluginsToAutoInstall(command, settings.plugins_from_config_file, settings.plugins_recommended);
     const recommendedPlugins = getRecommendPlugins(plugins, config);
     return {
         defaultBaseDir: settings.baseDirectory ?? command.project.relativeBaseDirectory ?? '',
         defaultBuildCmd: config.build.command || settings.buildCommand,
         defaultBuildDir: settings.dist,
+        // @ts-expect-error types need to be fixed on @netlify/build
         defaultFunctionsDir: config.build.functions || 'netlify/functions',
         recommendedPlugins,
     };
@@ -75,21 +72,14 @@ const getPromptInputs = ({ defaultBaseDir, defaultBuildCmd, defaultBuildDir }) =
     ].filter(Boolean);
     return inputs.filter(Boolean);
 };
-/**
- * @param {object} param0
- * @param {*} param0.config
- * @param {import('../../commands/base-command.js').default} param0.command
- */
-// @ts-expect-error TS(7031) FIXME: Binding element 'command' implicitly has an 'any' ... Remove this comment to see the full error message
 export const getBuildSettings = async ({ command, config }) => {
     const settings = await detectBuildSettings(command);
     // TODO: add prompt for asking to choose the build command
-    /** @type {Partial<import('@netlify/build-info').Settings>} */
     // eslint-disable-next-line unicorn/explicit-length-check
     const setting = settings.length > 0 ? settings[0] : {};
     const { defaultBaseDir, defaultBuildCmd, defaultBuildDir, defaultFunctionsDir, recommendedPlugins } = await normalizeSettings(setting, config, command);
     if (recommendedPlugins.length !== 0 && setting.framework?.name) {
-        log(`Configuring ${formatTitle(setting.framework?.name)} runtime...`);
+        log(`Configuring ${formatTitle(setting.framework.name)} runtime...`);
         log();
     }
     const { baseDir, buildCmd, buildDir } = await inquirer.prompt(getPromptInputs({
@@ -174,11 +164,6 @@ export const formatErrorMessage = ({ error, message }) => {
     const errorMessage = error.json ? `${error.message} - ${JSON.stringify(error.json)}` : error.message;
     return `${message} with error: ${chalk.red(errorMessage)}`;
 };
-/**
- * @param {string} title
- */
-// @ts-expect-error TS(7006) FIXME: Parameter 'title' implicitly has an 'any' type.
-const formatTitle = (title) => chalk.cyan(title);
 // @ts-expect-error TS(7031) FIXME: Binding element 'api' implicitly has an 'any' type... Remove this comment to see the full error message
 export const createDeployKey = async ({ api }) => {
     try {

package/src/utils/live-tunnel.js

@@ -6,7 +6,6 @@ import { v4 as uuidv4 } from 'uuid';
 import { fetchLatestVersion, shouldFetchLatestVersion } from '../lib/exec-fetcher.js';
 import { getPathInHome } from '../lib/settings.js';
 import { NETLIFYDEVERR, NETLIFYDEVLOG, chalk, log } from './command-helpers.js';
-// @ts-expect-error TS(7034) FIXME: Variable 'execa' implicitly has type 'any' in some... Remove this comment to see the full error message
 import execa from './execa.js';
 const PACKAGE_NAME = 'live-tunnel-client';
 const EXEC_NAME = PACKAGE_NAME;
@@ -45,7 +44,6 @@ const connectTunnel = function ({ localPort, netlifyApiToken, session }) {
         args.push('-v');
         log(execPath, args);
     }
-    // @ts-expect-error TS(7005) FIXME: Variable 'execa' implicitly has an 'any' type.
     const ps = execa(execPath, args, { stdio: 'inherit' });
     // @ts-expect-error TS(7006) FIXME: Parameter 'code' implicitly has an 'any' type.
     ps.on('close', (code) => process.exit(code));

package/src/utils/lm/requirements.js

@@ -1,9 +1,7 @@
 import semver from 'semver';
-// @ts-expect-error TS(7034) FIXME: Variable 'execa' implicitly has type 'any' in some... Remove this comment to see the full error message
 import execa from '../execa.js';
 export const checkLFSFilters = async function () {
     try {
-        // @ts-expect-error TS(7005) FIXME: Variable 'execa' implicitly has an 'any' type.
         const { stdout } = await execa('git', ['config', '--get-regexp', 'filter.lfs']);
         return stdout.length !== 0;
     }
@@ -13,7 +11,6 @@ export const checkLFSFilters = async function () {
 };
 const getHelperVersion = async function () {
     try {
-        // @ts-expect-error TS(7005) FIXME: Variable 'execa' implicitly has an 'any' type.
         const { stdout } = await execa('git-credential-netlify', ['version']);
         return stdout;
     }
@@ -28,7 +25,6 @@ export const checkHelperVersion = async function () {
 };
 export const checkGitVersion = async function () {
     try {
-        // @ts-expect-error TS(7005) FIXME: Variable 'execa' implicitly has an 'any' type.
         const { stdout } = await execa('git', ['--version']);
         return stdout.split(' ').pop();
     }
@@ -38,7 +34,6 @@ export const checkGitVersion = async function () {
 };
 const getLFSVersion = async function () {
     try {
-        // @ts-expect-error TS(7005) FIXME: Variable 'execa' implicitly has an 'any' type.
         const { stdout } = await execa('git-lfs', ['version']);
         return stdout;
     }

package/src/utils/sites/utils.js

@@ -35,7 +35,6 @@ export const validateTemplate = async ({ ghToken, templateName }) => {
     const data = await response.json();
     return { exists: true, isTemplate: data.is_template };
 };
-// @ts-expect-error TS(7006) FIXME: Parameter 'templateName' implicitly has an 'any' t... Remove this comment to see the full error message
 export const createRepo = async (templateName, ghToken, siteName) => {
     const resp = await fetch(`https://api.github.com/repos/${templateName}/generate`, {
         method: 'POST',

package/src/utils/telemetry/report-error.js

@@ -3,7 +3,6 @@ import { dirname, join } from 'path';
 import process, { version as nodejsVersion } from 'process';
 import { fileURLToPath } from 'url';
 import { isCI } from 'ci-info';
-// @ts-expect-error TS(7034) FIXME: Variable 'execa' implicitly has type 'any' in some... Remove this comment to see the full error message
 import execa from '../execa.js';
 import getGlobalConfig from '../get-global-config.js';
 import { cliVersion } from './utils.js';
@@ -46,7 +45,6 @@ export const reportError = async function (error, config = {}) {
     });
     // spawn detached child process to handle send and wait for the http request to finish
     // otherwise it can get canceled
-    // @ts-expect-error TS(7005) FIXME: Variable 'execa' implicitly has an 'any' type.
     await execa(process.execPath, [join(dirPath, 'request.js'), options], {
         detached: true,
         stdio: 'ignore',

package/src/utils/telemetry/telemetry.js

@@ -2,32 +2,24 @@ import { dirname, join } from 'path';
 import process, { version as nodejsVersion } from 'process';
 import { fileURLToPath } from 'url';
 import { isCI } from 'ci-info';
-// @ts-expect-error TS(7034) FIXME: Variable 'execa' implicitly has type 'any' in some... Remove this comment to see the full error message
 import execa from '../execa.js';
 import getGlobalConfig from '../get-global-config.js';
 import { isTelemetryDisabled, cliVersion } from './utils.js';
 import isValidEventName from './validation.js';
 const dirPath = dirname(fileURLToPath(import.meta.url));
-/**
- * @param {'track' | 'identify'} type
- * @param {object} payload
- */
-// @ts-expect-error TS(7006) FIXME: Parameter 'type' implicitly has an 'any' type.
 function send(type, payload) {
     const requestFile = join(dirPath, 'request.js');
     const options = JSON.stringify({
         data: payload,
         type,
     });
-    const args = /** @type {const} */ [process.execPath, [requestFile, options]];
+    const args = [process.execPath, [requestFile, options]];
     if (process.env.NETLIFY_TEST_TELEMETRY_WAIT === 'true') {
-        // @ts-expect-error TS(7005) FIXME: Variable 'execa' implicitly has an 'any' type.
         return execa(...args, {
             stdio: 'inherit',
         });
     }
     // spawn detached child process to handle send
-    // @ts-expect-error TS(7005) FIXME: Variable 'execa' implicitly has an 'any' type.
     execa(...args, {
         detached: true,
         stdio: 'ignore',
@@ -46,10 +38,7 @@ const eventConfig = {
 };
 /**
  * Tracks a custom event with the provided payload
- * @param {string} eventName
- * @param {{status?: string, duration?: number, [key: string]: unknown}} [payload]
  */
-// @ts-expect-error TS(7006) FIXME: Parameter 'eventName' implicitly has an 'any' type... Remove this comment to see the full error message
 export async function track(eventName, payload = {}) {
     if (isCI) {
         return;
@@ -69,7 +58,6 @@ export async function track(eventName, payload = {}) {
     if (!isValid(eventName, eventConfig)) {
         return false;
     }
-    // @ts-expect-error TS(2339) FIXME: Property 'duration' does not exist on type '{}'.
     const { duration, status, ...properties } = payload;
     const defaultData = {
         event: eventName,
@@ -81,14 +69,6 @@ export async function track(eventName, payload = {}) {
     };
     return send('track', defaultData);
 }
-/**
- * @param {object} payload
- * @param {string} payload.name
- * @param {string} payload.email
- * @param {string} payload.userId
- * @returns
- */
-// @ts-expect-error TS(7006) FIXME: Parameter 'payload' implicitly has an 'any' type.
 export async function identify(payload) {
     if (isCI) {
         return;