@netlify/build 34.3.0 → 35.0.0

package/lib/install/missing.js

@@ -1,9 +1,8 @@
-import { promises as fs } from 'fs';
-import { normalize, resolve } from 'path';
-import { execa } from 'execa';
+import { promises as fs } from 'node:fs';
+import { normalize } from 'node:path';
+import { fileURLToPath } from 'node:url';
 import { pathExists } from 'path-exists';
 import { isFile } from 'path-type';
-import { logArray, logSubHeader } from '../log/logger.js';
 import { logInstallMissingPlugins, logInstallIntegrations } from '../log/messages/install.js';
 import { addExactDependencies } from './main.js';
 // Automatically install plugins if not already installed.
@@ -21,40 +20,28 @@ export const installMissingPlugins = async function ({ missingPlugins, autoPlugi
     await addExactDependencies({ packageRoot: autoPluginsDir, isLocal: mode !== 'buildbot', packages });
 };
 export const installIntegrationPlugins = async function ({ integrations, autoPluginsDir, mode, logs, context, testOpts, pluginsEnv, buildDir, }) {
-    const integrationsToBuild = integrations.filter((integration) => typeof integration.dev !== 'undefined' && context === 'dev');
-    if (integrationsToBuild.length) {
-        logSubHeader(logs, 'Building extensions');
-        logArray(logs, integrationsToBuild.map(({ slug, dev: { path } }) => `${slug} from ${path}`));
-    }
     const packages = (await Promise.all(integrations.map((integration) => getIntegrationPackage({ integration, context, testOpts, buildDir, pluginsEnv })))).filter(Boolean);
-    logInstallIntegrations(logs, integrations.filter((integration) => integrationsToBuild.every((compiledIntegration) => integration.slug !== compiledIntegration.slug)));
+    logInstallIntegrations(logs, integrations);
     if (packages.length === 0) {
         return;
     }
     await createAutoPluginsDir(logs, autoPluginsDir);
     await addExactDependencies({ packageRoot: autoPluginsDir, isLocal: mode !== 'buildbot', packages });
 };
-const getIntegrationPackage = async function ({ integration: { version, dev }, context, testOpts = {}, buildDir, pluginsEnv, }) {
-    if (typeof version !== 'undefined') {
-        return `${version}/packages/buildhooks.tgz`;
-    }
-    if (typeof dev !== 'undefined' && context === 'dev') {
-        const { path } = dev;
-        const integrationDir = testOpts.cwd ? resolve(testOpts.cwd, path) : resolve(buildDir, path);
-        try {
-            const res = await execa('npm', ['run', 'build'], { cwd: integrationDir, env: pluginsEnv });
-            // This is horrible and hacky, but `npm run build` will
-            // return status code 0 even if it fails
-            if (!res.stdout.includes('Build complete!')) {
-                throw new Error(res.stdout);
-            }
-        }
-        catch (e) {
-            throw new Error(`Failed to build integration. Error:\n\n${e.stack}`);
-        }
+const getIntegrationPackage = async function ({ integration: { buildPlugin } }) {
+    if (buildPlugin === null) {
         return undefined;
     }
-    return undefined;
+    switch (buildPlugin.packageURL.protocol) {
+        case 'http:':
+        // fallthrough
+        case 'https:':
+            return buildPlugin.packageURL.toString();
+        case 'file:':
+            return fileURLToPath(buildPlugin.packageURL);
+        default:
+            throw new Error(`unsupported build plugin package URL: ${buildPlugin.packageURL.toString()}`);
+    }
 };
 // We pin the version without using semver ranges ^ nor ~
 const getPackage = function ({ packageName, expectedVersion }) {

package/lib/log/theme.d.ts

@@ -1,19 +1,20 @@
+import * as colors from 'ansis';
 /**
- * Color theme. Please use this instead of requiring chalk directly,
+ * Color theme. Please use this instead of requiring ansis directly,
  * to ensure consistent colors.
  */
 export declare const THEME: {
-    header: import("chalk").ChalkInstance;
-    subHeader: import("chalk").ChalkInstance;
-    highlightWords: import("chalk").ChalkInstance;
-    errorHeader: import("chalk").ChalkInstance;
-    errorSubHeader: import("chalk").ChalkInstance;
-    errorLine: import("chalk").ChalkInstance;
-    errorHighlightWords: import("chalk").ChalkInstance;
-    warningHeader: import("chalk").ChalkInstance;
-    warningSubHeader: import("chalk").ChalkInstance;
-    warningLine: import("chalk").ChalkInstance;
-    warningHighlightWords: import("chalk").ChalkInstance;
-    dimWords: import("chalk").ChalkInstance;
+    header: colors.Ansis;
+    subHeader: colors.Ansis;
+    highlightWords: colors.Ansis;
+    errorHeader: colors.Ansis;
+    errorSubHeader: colors.Ansis;
+    errorLine: colors.Ansis;
+    errorHighlightWords: colors.Ansis;
+    warningHeader: colors.Ansis;
+    warningSubHeader: colors.Ansis;
+    warningLine: colors.Ansis;
+    warningHighlightWords: colors.Ansis;
+    dimWords: colors.Ansis;
     none: (string: any) => any;
 };

package/lib/log/theme.js

@@ -1,27 +1,27 @@
-import chalk from 'chalk';
+import * as colors from 'ansis';
 /**
- * Color theme. Please use this instead of requiring chalk directly,
+ * Color theme. Please use this instead of requiring ansis directly,
  * to ensure consistent colors.
  */
 export const THEME = {
     // Main headers
-    header: chalk.cyanBright.bold,
+    header: colors.cyanBright.bold,
     // Single lines used as sub-headers
-    subHeader: chalk.cyan.bold,
+    subHeader: colors.cyan.bold,
     // One of several words that should be highlighted inside a line
-    highlightWords: chalk.cyan,
+    highlightWords: colors.cyan,
     // Same for errors
-    errorHeader: chalk.redBright.bold,
-    errorSubHeader: chalk.red.bold,
-    errorLine: chalk.redBright,
-    errorHighlightWords: chalk.redBright.bold,
+    errorHeader: colors.redBright.bold,
+    errorSubHeader: colors.red.bold,
+    errorLine: colors.redBright,
+    errorHighlightWords: colors.redBright.bold,
     // Same for warnings
-    warningHeader: chalk.yellowBright.bold,
-    warningSubHeader: chalk.yellow.bold,
-    warningLine: chalk.yellowBright,
-    warningHighlightWords: chalk.yellowBright.bold,
+    warningHeader: colors.yellowBright.bold,
+    warningSubHeader: colors.yellow.bold,
+    warningLine: colors.yellowBright,
+    warningHighlightWords: colors.yellowBright.bold,
     // One of several words that should be dimmed inside a line
-    dimWords: chalk.gray,
+    dimWords: colors.gray,
     // No colors
     none: (string) => string,
 };

package/lib/plugins/resolve.js

@@ -1,4 +1,4 @@
-import { join, resolve } from 'path';
+import { join } from 'node:path';
 import { addErrorInfo } from '../error/info.js';
 import { installMissingPlugins, installIntegrationPlugins } from '../install/missing.js';
 import { resolvePath, tryResolvePath } from '../utils/resolve.js';
@@ -39,8 +39,7 @@ export const resolvePluginsPath = async function ({ pluginsOptions, siteInfo, bu
         mode,
         logs,
     });
-    let integrationPluginOptions = [];
-    integrationPluginOptions = await handleIntegrations({
+    const integrationPluginOptions = await handleIntegrations({
         integrations,
         autoPluginsDir,
         mode,
@@ -118,9 +117,9 @@ const handleMissingPlugins = async function ({ pluginsOptions, autoPluginsDir, m
     return Promise.all(pluginsOptions.map((pluginOptions) => resolveMissingPluginPath({ pluginOptions, autoPluginsDir })));
 };
 const handleIntegrations = async function ({ integrations, autoPluginsDir, mode, logs, buildDir, context, testOpts, pluginsEnv, }) {
-    const toInstall = integrations.filter((integration) => integration.has_build);
+    const integrationsWithBuildPlugins = integrations.filter((integration) => integration.has_build);
     await installIntegrationPlugins({
-        integrations: toInstall,
+        integrations: integrationsWithBuildPlugins,
         autoPluginsDir,
         mode,
         logs,
@@ -129,26 +128,25 @@ const handleIntegrations = async function ({ integrations, autoPluginsDir, mode,
         buildDir,
         pluginsEnv,
     });
-    if (toInstall.length === 0) {
-        return [];
-    }
-    return Promise.all(toInstall.map((integration) => resolveIntegration({
-        integration,
-        autoPluginsDir,
-        buildDir,
-        context,
-        testOpts,
-    })));
-};
-const resolveIntegration = async function ({ integration, autoPluginsDir, buildDir, context, testOpts }) {
-    if (typeof integration.dev !== 'undefined' && context === 'dev') {
-        const { path } = integration.dev;
-        const integrationDir = testOpts.cwd ? resolve(testOpts.cwd, path) : resolve(buildDir, path);
-        const pluginPath = await resolvePath(`${integrationDir}/.ntli/build`, buildDir);
-        return { pluginPath, packageName: `${integration.slug}`, isIntegration: true, integration, loadedFrom: 'local' };
-    }
-    const pluginPath = await resolvePath(`${integration.slug}-buildhooks`, autoPluginsDir);
-    return { pluginPath, packageName: `${integration.slug}-buildhooks`, isIntegration: true, integration };
+    return Promise.all(integrationsWithBuildPlugins.map(async (integration) => {
+        // TODO(ndhoule): When developing locally, the user can accidentally set an extension name in
+        // their netlify.toml that points to a build plugin package that has a mismatched name. This
+        // will result in a failed install here. The solution is non-obvious: make sure your
+        // `netlify.toml#integrations[].name` matches the extension that `netlify.toml#integrations[].dev.path`
+        // points at.
+        //
+        // (We could, for example, detect a mismatch by untarring the local build plugin package in
+        // memory and comparing its `package.json#name` to `integration.slug`, and throw a descriptive
+        // error if they don't match.)
+        const packageName = `${integration.slug}-buildhooks`;
+        return {
+            integration,
+            isIntegration: true,
+            loadedFrom: integration.buildPlugin.origin === 'local' ? 'local' : undefined,
+            packageName,
+            pluginPath: await resolvePath(packageName, autoPluginsDir),
+        };
+    }));
 };
 // Resolve the plugins that just got automatically installed
 const resolveMissingPluginPath = async function ({ pluginOptions, pluginOptions: { packageName }, autoPluginsDir }) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/build",
-  "version": "34.3.0",
+  "version": "35.0.0",
   "description": "Netlify build module",
   "type": "module",
   "exports": "./lib/index.js",
@@ -67,9 +67,9 @@
   "license": "MIT",
   "dependencies": {
     "@bugsnag/js": "^8.0.0",
-    "@netlify/blobs": "^10.0.6",
+    "@netlify/blobs": "^10.0.7",
     "@netlify/cache-utils": "^6.0.3",
-    "@netlify/config": "^23.2.0",
+    "@netlify/config": "^24.0.0",
     "@netlify/edge-bundler": "14.2.2",
     "@netlify/functions-utils": "^6.2.0",
     "@netlify/git-utils": "^6.0.2",
@@ -79,7 +79,7 @@
     "@netlify/zip-it-and-ship-it": "14.1.0",
     "@sindresorhus/slugify": "^2.0.0",
     "ansi-escapes": "^7.0.0",
-    "chalk": "^5.0.0",
+    "ansis": "^4.1.0",
     "clean-stack": "^5.0.0",
     "execa": "^8.0.0",
     "fdir": "^6.0.1",
@@ -156,5 +156,5 @@
   "engines": {
     "node": ">=18.14.0"
   },
-  "gitHead": "2a186393af7f577cc6bafa5d3962ea0833cd084c"
+  "gitHead": "5100d463cb9b606b91a6a9b5c3fbe9c143716701"
 }