fix: use staging url for extensions on staging (#6140)

* fix: use staging url for extensions on staging

* fix: implement some feedback

* fix: let production be fallback for extension api

Author: khendrikse

packages/config/src/api/integrations.ts

@@ -2,26 +2,20 @@ import fetch from 'node-fetch'
 
 import { TestOptions } from '../types/options.js'
 
-type AvailableIntegration = {
-  slug: string
-  hostSiteUrl: string
-  hasBuild?: boolean
-}
+type AvailableIntegration = { slug: string; hostSiteUrl: string; hasBuild?: boolean }
 
-type GetAvailableIntegrationsOpts = {
-  testOpts: TestOptions
-  offline: boolean
-}
+type GetAvailableIntegrationsOpts = { testOpts: TestOptions; offline: boolean; extensionApiBaseUrl: string }
 
 export const getAvailableIntegrations = async function ({
   testOpts,
   offline,
+  extensionApiBaseUrl,
 }: GetAvailableIntegrationsOpts): Promise<AvailableIntegration[]> {
   if (offline) {
     return []
   }
   const { host } = testOpts
-  const baseUrl = new URL(host ? `http://${host}/` : `https://api.netlifysdk.com/`)
+  const baseUrl = new URL(host ? `http://${host}/` : extensionApiBaseUrl)
 
   try {
     const response = await fetch(`${baseUrl}integrations`)

packages/config/src/api/site_info.ts

@@ -19,6 +19,7 @@ type GetSiteInfoOpts = {
   testOpts?: TestOptions
   siteFeatureFlagPrefix: string
   token: string
+  extensionApiBaseUrl: string
 }
 /**
  * Retrieve Netlify Site information, if available.
@@ -40,6 +41,7 @@ export const getSiteInfo = async function ({
   siteFeatureFlagPrefix,
   token,
   featureFlags = {},
+  extensionApiBaseUrl,
 }: GetSiteInfoOpts) {
   const { env: testEnv = false } = testOpts
 
@@ -51,7 +53,7 @@ export const getSiteInfo = async function ({
 
     const integrations =
       mode === 'buildbot' && !offline
-        ? await getIntegrations({ siteId, testOpts, offline, accountId, token, featureFlags })
+        ? await getIntegrations({ siteId, testOpts, offline, accountId, token, featureFlags, extensionApiBaseUrl })
         : []
 
     return { siteInfo, accounts: [], addons: [], integrations }
@@ -61,7 +63,7 @@ export const getSiteInfo = async function ({
     getSite(api, siteId, siteFeatureFlagPrefix),
     getAccounts(api),
     getAddons(api, siteId),
-    getIntegrations({ siteId, testOpts, offline, accountId, token, featureFlags }),
+    getIntegrations({ siteId, testOpts, offline, accountId, token, featureFlags, extensionApiBaseUrl }),
   ]
 
   const [siteInfo, accounts, addons, integrations] = await Promise.all(promises)
@@ -117,6 +119,7 @@ type GetIntegrationsOpts = {
   offline: boolean
   token?: string
   featureFlags?: Record<string, boolean>
+  extensionApiBaseUrl: string
 }
 
 const getIntegrations = async function ({
@@ -126,14 +129,14 @@ const getIntegrations = async function ({
   offline,
   token,
   featureFlags,
+  extensionApiBaseUrl,
 }: GetIntegrationsOpts): Promise<IntegrationResponse[]> {
   if (!siteId || offline) {
     return []
   }
   const sendBuildBotTokenToJigsaw = featureFlags?.send_build_bot_token_to_jigsaw
   const { host } = testOpts
-
-  const baseUrl = new URL(host ? `http://${host}` : `https://api.netlifysdk.com`)
+  const baseUrl = new URL(host ? `http://${host}` : extensionApiBaseUrl)
 
   // if accountId isn't present, use safe v1 endpoint
   const url = accountId

packages/config/src/integrations.ts

@@ -9,6 +9,7 @@ type MergeIntegrationsOpts = {
   context: string
   testOpts?: TestOptions
   offline: boolean
+  extensionApiBaseUrl: string
 }
 
 export const mergeIntegrations = async function ({
@@ -17,8 +18,9 @@ export const mergeIntegrations = async function ({
   context,
   testOpts = {},
   offline,
+  extensionApiBaseUrl,
 }: MergeIntegrationsOpts): Promise<Integration[]> {
-  const availableIntegrations = await getAvailableIntegrations({ testOpts, offline })
+  const availableIntegrations = await getAvailableIntegrations({ testOpts, offline, extensionApiBaseUrl })
 
   // Include all API integrations, unless they have a `dev` property and we are in the `dev` context
   const resolvedApiIntegrations = apiIntegrations.filter(

packages/config/src/main.ts

@@ -49,6 +49,10 @@ 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 {
     config: configOpt,
     defaultConfig,
@@ -81,6 +85,7 @@ export const resolveConfig = async function (opts) {
     featureFlags,
     testOpts,
     token,
+    extensionApiBaseUrl,
   })
 
   const { defaultConfig: defaultConfigA, baseRelDir: baseRelDirA } = parseDefaultConfig({
@@ -131,6 +136,7 @@ export const resolveConfig = async function (opts) {
     context: context,
     testOpts,
     offline,
+    extensionApiBaseUrl,
   })
 
   const result = {