fix: bundle dynamic API routes correctly with split-api-routes (#2154)

* chore: add repro test

* fix: include files that include []

---------

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>

Author: Skn0tt

cypress/e2e/default/api.cy.ts

@@ -6,6 +6,14 @@ describe('API routes', () => {
   })
 })
 
+describe('Dynamic API routes', () => {
+  it('are bundled correctly', () => {
+    cy.request('/api/shows/1').then((response) => {
+      expect(response.status).to.equal(200)
+    })
+  })
+})
+
 describe('Extended API routes', () => {
   it('returns HTTP 202 Accepted for background route', () => {
     cy.request('/api/hello-background').then((response) => {

packages/runtime/src/helpers/config.ts

@@ -96,6 +96,12 @@ export const hasManuallyAddedModule = ({
   )
 /* eslint-enable camelcase */
 
+/**
+ * Transforms `/api/shows/[id].js` into `/api/shows/*id*.js`,
+ * so that the file `[id].js` is matched correctly.
+ */
+const escapeGlob = (path: string) => path.replace(/\[/g, '*').replace(/]/g, '*')
+
 export const configureHandlerFunctions = async ({
   netlifyConfig,
   publish,
@@ -173,7 +179,7 @@ export const configureHandlerFunctions = async ({
       netlifyConfig.functions[functionName] ||= { included_files: [] }
       netlifyConfig.functions[functionName].node_bundler = 'none'
       netlifyConfig.functions[functionName].included_files ||= []
-      netlifyConfig.functions[functionName].included_files.push(...includedFiles)
+      netlifyConfig.functions[functionName].included_files.push(...includedFiles.map(escapeGlob))
     }
   } else {
     configureFunction('_api_*')