@netlify/config 20.13.1 → 20.13.2

package/lib/api/site_info.js

@@ -13,37 +13,18 @@ import { ERROR_CALL_TO_ACTION } from '../log/messages.js';
  */
 export const getSiteInfo = async function ({ api, siteId, mode, siteFeatureFlagPrefix, context, offline = false, testOpts = {}, }) {
     const { env: testEnv = false } = testOpts;
-    if (api === undefined || testEnv || offline) {
+    if (api === undefined || mode === 'buildbot' || testEnv) {
         const siteInfo = siteId === undefined ? {} : { id: siteId };
-        return { siteInfo, accounts: [], addons: [], integrations: [] };
-    }
-    const siteInfo = await getSite(api, siteId, siteFeatureFlagPrefix);
-    const featureFlags = siteInfo.feature_flags;
-    const useV2Endpoint = featureFlags?.cli_integration_installations_meta;
-    if (useV2Endpoint) {
-        const promises = [
-            getAccounts(api),
-            getAddons(api, siteId),
-            getIntegrations({ siteId, testOpts, offline, accountId: siteInfo.account_id, featureFlags }),
-        ];
-        const [accounts, addons, integrations] = await Promise.all(promises);
-        if (siteInfo.use_envelope) {
-            const envelope = await getEnvelope({ api, accountId: siteInfo.account_slug, siteId, context });
-            siteInfo.build_settings.env = envelope;
-        }
-        return { siteInfo, accounts, addons, integrations };
-    }
-    if (mode === 'buildbot') {
-        const siteInfo = siteId === undefined ? {} : { id: siteId };
-        const integrations = await getIntegrations({ siteId, testOpts, offline, featureFlags });
+        const integrations = mode === 'buildbot' && !offline ? await getIntegrations({ siteId, testOpts, offline }) : [];
         return { siteInfo, accounts: [], addons: [], integrations };
     }
     const promises = [
+        getSite(api, siteId, siteFeatureFlagPrefix),
         getAccounts(api),
         getAddons(api, siteId),
-        getIntegrations({ siteId, testOpts, offline, featureFlags }),
+        getIntegrations({ siteId, testOpts, offline }),
     ];
-    const [accounts, addons, integrations] = await Promise.all(promises);
+    const [siteInfo, accounts, addons, integrations] = await Promise.all(promises);
     if (siteInfo.use_envelope) {
         const envelope = await getEnvelope({ api, accountId: siteInfo.account_slug, siteId, context });
         siteInfo.build_settings.env = envelope;
@@ -59,7 +40,7 @@ const getSite = async function (api, siteId, siteFeatureFlagPrefix = null) {
         return { ...site, id: siteId };
     }
     catch (error) {
-        return throwUserError(`Failed retrieving site data for site ${siteId}: ${error.message}. ${ERROR_CALL_TO_ACTION}`);
+        throwUserError(`Failed retrieving site data for site ${siteId}: ${error.message}. ${ERROR_CALL_TO_ACTION}`);
     }
 };
 const getAccounts = async function (api) {
@@ -68,7 +49,7 @@ const getAccounts = async function (api) {
         return Array.isArray(accounts) ? accounts : [];
     }
     catch (error) {
-        return throwUserError(`Failed retrieving user account: ${error.message}. ${ERROR_CALL_TO_ACTION}`);
+        throwUserError(`Failed retrieving user account: ${error.message}. ${ERROR_CALL_TO_ACTION}`);
     }
 };
 const getAddons = async function (api, siteId) {
@@ -80,28 +61,22 @@ const getAddons = async function (api, siteId) {
         return Array.isArray(addons) ? addons : [];
     }
     catch (error) {
-        return throwUserError(`Failed retrieving addons for site ${siteId}: ${error.message}. ${ERROR_CALL_TO_ACTION}`);
+        throwUserError(`Failed retrieving addons for site ${siteId}: ${error.message}. ${ERROR_CALL_TO_ACTION}`);
     }
 };
-const getIntegrations = async function ({ siteId, accountId, testOpts, offline, featureFlags, }) {
+const getIntegrations = async function ({ siteId, testOpts, offline, }) {
     if (!siteId || offline) {
         return [];
     }
     const { host } = testOpts;
     const baseUrl = new URL(host ? `http://${host}` : `https://api.netlifysdk.com`);
-    const useV2Endpoint = featureFlags?.cli_integration_installations_meta;
-    const url = useV2Endpoint
-        ? `${baseUrl}team/${accountId}/integrations/installations/meta`
-        : `${baseUrl}site/${siteId}/integrations/safe`;
     try {
-        const response = await fetch(url);
+        const response = await fetch(`${baseUrl}site/${siteId}/integrations/safe`);
         const integrations = await response.json();
         return Array.isArray(integrations) ? integrations : [];
     }
     catch (error) {
-        // Integrations should not block the build if they fail to load
-        // TODO: We should consider blocking the build as integrations are a critical part of the build process
-        // https://linear.app/netlify/issue/CT-1214/implement-strategy-in-builds-to-deal-with-integrations-that-we-fail-to
+        // for now, we'll just ignore errors, as this is early days
         return [];
     }
 };

package/lib/main.js

@@ -39,6 +39,7 @@ export const resolveConfig = async function (opts) {
         mode,
         offline,
         siteFeatureFlagPrefix,
+        featureFlags,
         testOpts,
     });
     const { defaultConfig: defaultConfigA, baseRelDir: baseRelDirA } = parseDefaultConfig({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/config",
-  "version": "20.13.1",
+  "version": "20.13.2",
   "description": "Netlify config module",
   "type": "module",
   "exports": "./lib/index.js",
@@ -94,5 +94,5 @@
   "engines": {
     "node": "^14.16.0 || >=16.0.0"
   },
-  "gitHead": "600d5f1a934b57f39746083cce925ecfe9025fad"
+  "gitHead": "6573e4d27bb60c0b1438ace279791389330286ad"
 }