@netlify/config 21.0.3 → 21.0.5

package/lib/api/site_info.js

@@ -75,7 +75,11 @@ const getIntegrations = async function ({ siteId, accountId, testOpts, offline,
         return [];
     }
     const sendBuildBotTokenToJigsaw = featureFlags?.send_build_bot_token_to_jigsaw;
-    const { host } = testOpts;
+    const { host, setBaseUrl } = testOpts;
+    // We only use this for testing
+    if (host && setBaseUrl) {
+        setBaseUrl(extensionApiBaseUrl);
+    }
     const baseUrl = new URL(host ? `http://${host}` : extensionApiBaseUrl);
     // if accountId isn't present, use safe v1 endpoint
     const url = accountId

package/lib/integrations.d.ts

@@ -1,6 +1,9 @@
 import { IntegrationResponse } from './types/api.js';
 import { Integration } from './types/integrations.js';
 import { TestOptions } from './types/options.js';
+export declare const NETLIFY_API_STAGING_BASE_URL = "api-staging.netlify.com";
+export declare const EXTENSION_API_BASE_URL = "https://api.netlifysdk.com";
+export declare const EXTENSION_API_STAGING_BASE_URL = "https://api-staging.netlifysdk.com";
 type MergeIntegrationsOpts = {
     configIntegrations?: {
         name: string;

package/lib/integrations.js

@@ -1,4 +1,7 @@
 import { getAvailableIntegrations } from './api/integrations.js';
+export const NETLIFY_API_STAGING_BASE_URL = 'api-staging.netlify.com';
+export const EXTENSION_API_BASE_URL = 'https://api.netlifysdk.com';
+export const EXTENSION_API_STAGING_BASE_URL = 'https://api-staging.netlifysdk.com';
 export const mergeIntegrations = async function ({ configIntegrations = [], apiIntegrations, context, testOpts = {}, offline, extensionApiBaseUrl, }) {
     const availableIntegrations = await getAvailableIntegrations({ testOpts, offline, extensionApiBaseUrl });
     // Include all API integrations, unless they have a `dev` property and we are in the `dev` context
@@ -23,11 +26,7 @@ export const mergeIntegrations = async function ({ configIntegrations = [], apiI
         if (!integration) {
             return undefined;
         }
-        return {
-            slug: integration.slug,
-            version: integration.hostSiteUrl,
-            has_build: !!integration.hasBuild,
-        };
+        return { slug: integration.slug, version: integration.hostSiteUrl, has_build: !!integration.hasBuild };
     })
         .filter((i) => typeof i !== 'undefined');
     return [...resolvedApiIntegrations, ...resolvedConfigIntegrations];

package/lib/main.js

@@ -9,7 +9,7 @@ import { getEnv } from './env/main.js';
 import { resolveConfigPaths } from './files.js';
 import { getHeadersPath, addHeaders } from './headers.js';
 import { getInlineConfig } from './inline_config.js';
-import { mergeIntegrations } from './integrations.js';
+import { EXTENSION_API_BASE_URL, EXTENSION_API_STAGING_BASE_URL, mergeIntegrations, NETLIFY_API_STAGING_BASE_URL, } from './integrations.js';
 import { logResult } from './log/main.js';
 import { mergeConfigs } from './merge.js';
 import { normalizeBeforeConfigMerge, normalizeAfterConfigMerge } from './merge_normalize.js';
@@ -35,11 +35,15 @@ export const resolveConfig = async function (opts) {
         return parsedCachedConfig;
     }
     // TODO(kh): remove this mapping and get the extensionApiHost from the opts
-    const extensionApiBaseUrl = host === 'api-staging.netlify.com' ? 'https://api-staging.netlifysdk.com' : 'https://api.netlifysdk.com';
+    const extensionApiBaseUrl = host === NETLIFY_API_STAGING_BASE_URL ? EXTENSION_API_STAGING_BASE_URL : EXTENSION_API_BASE_URL;
     const { config: configOpt, defaultConfig, inlineConfig, configMutations, cwd, context, repositoryRoot, base, branch, siteId, accountId, deployId, buildId, baseRelDir, mode, debug, logs, featureFlags, } = await normalizeOpts(optsA);
     let { siteInfo, accounts, addons, integrations } = parsedCachedConfig || {};
     // If we have cached site info, we don't need to fetch it again
     const useCachedSiteInfo = Boolean(featureFlags?.use_cached_site_info && siteInfo && accounts && addons && integrations);
+    // I'm adding some debug logging to see if the logic is working as expected
+    if (featureFlags?.use_cached_site_info_logging) {
+        console.log('Checking site information', { useCachedSiteInfo, siteInfo, accounts, addons, integrations });
+    }
     if (!useCachedSiteInfo) {
         const updatedSiteInfo = await getSiteInfo({
             api,
@@ -134,13 +138,7 @@ const addLegacyFunctionsDirectory = (config) => {
     if (!config.functionsDirectory) {
         return config;
     }
-    return {
-        ...config,
-        build: {
-            ...config.build,
-            functions: config.functionsDirectory,
-        },
-    };
+    return { ...config, build: { ...config.build, functions: config.functionsDirectory } };
 };
 /**
  * Try to load the configuration file in two passes.
@@ -198,24 +196,10 @@ const loadConfig = async function ({ configOpt, cwd, context, repositoryRoot, pa
  * Load configuration file and normalize it, merge contexts, etc.
  */
 const getFullConfig = async function ({ configOpt, cwd, context, repositoryRoot, packagePath, branch, defaultConfig, inlineConfig, baseRelDir, configBase, base, logs, featureFlags, }) {
-    const configPath = await getConfigPath({
-        configOpt,
-        cwd,
-        repositoryRoot,
-        packagePath,
-        configBase,
-    });
+    const configPath = await getConfigPath({ configOpt, cwd, repositoryRoot, packagePath, configBase });
     try {
         const config = await parseConfig(configPath);
-        const configA = mergeAndNormalizeConfig({
-            config,
-            defaultConfig,
-            inlineConfig,
-            context,
-            branch,
-            logs,
-            packagePath,
-        });
+        const configA = mergeAndNormalizeConfig({ config, defaultConfig, inlineConfig, context, branch, logs, packagePath });
         const { config: configB, buildDir, base: baseA, } = await resolveFiles({ packagePath, config: configA, repositoryRoot, base, baseRelDir });
         const headersPath = getHeadersPath(configB);
         const configC = await addHeaders({ config: configB, headersPath, logs });

package/lib/types/options.d.ts

@@ -2,4 +2,5 @@ export type ModeOption = 'buildbot' | 'cli' | 'require';
 export type TestOptions = {
     env?: boolean;
     host?: string;
+    setBaseUrl?: (url: string) => void;
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/config",
-  "version": "21.0.3",
+  "version": "21.0.5",
   "description": "Netlify config module",
   "type": "module",
   "exports": "./lib/index.js",
@@ -74,7 +74,7 @@
     "is-plain-obj": "^4.0.0",
     "js-yaml": "^4.0.0",
     "map-obj": "^5.0.0",
-    "netlify": "^13.3.3",
+    "netlify": "^13.3.4",
     "node-fetch": "^3.3.1",
     "omit.js": "^2.0.2",
     "p-locate": "^6.0.0",
@@ -95,5 +95,5 @@
   "engines": {
     "node": "^14.16.0 || >=16.0.0"
   },
-  "gitHead": "180306dc6d37736969b5fa951f8588c2c85e6109"
+  "gitHead": "0e4628f57a0a7d02594b5070074445a4ff69b809"
 }