codemod: fix missing route transform (#70959)

vercel · Oct 10, 2024 · f3e1e0e · f3e1e0e
1 parent 8ced6ba
commit f3e1e0e
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 2 deletions.
diff --git a/...forms/__testfixtures__/next-async-request-api-dynamic-props/origin-name-01-route.input.js b/...forms/__testfixtures__/next-async-request-api-dynamic-props/origin-name-01-route.input.js
@@ -0,0 +1,3 @@
+export async function GET(req, { params }) {
+  f1(params)
+}
diff --git a/...orms/__testfixtures__/next-async-request-api-dynamic-props/origin-name-01-route.output.js b/...orms/__testfixtures__/next-async-request-api-dynamic-props/origin-name-01-route.output.js
@@ -0,0 +1,4 @@
+export async function GET(req, props) {
+  const params = await props.params;
+  f1(params)
+}
diff --git a/...ic-props/origin-name-somefile-01.input.js → ...ic-props/origin-name-01-somefile.input.js b/...ic-props/origin-name-somefile-01.input.js → ...ic-props/origin-name-01-somefile.input.js
diff --git a/...c-props/origin-name-somefile-01.output.js → ...c-props/origin-name-01-somefile.output.js b/...c-props/origin-name-somefile-01.output.js → ...c-props/origin-name-01-somefile.output.js
diff --git a/packages/next-codemod/transforms/__tests__/next-async-request-api-dynamic-props.test.js b/packages/next-codemod/transforms/__tests__/next-async-request-api-dynamic-props.test.js
@@ -43,7 +43,12 @@ describe('next-async-request-api - dynamic-props', () => {
 
     // Override test fixture input filename with `page.tsx` to always match the expected output,
     // otherwise fallback to the original filename.
-    const overrideFilename = /[\\/]origin-name-/.test(inputPath) ? inputPath : 'page.tsx'
+    const overrideFilename = /[\\/]origin-name-\d{2}-/.test(inputPath) 
+      // extract the <name> from `origin-name-<name>-<number>.input.js`
+      ? inputPath
+        .replace(/origin-name-(\d{2})-/, '')
+        .replace(/\.input\./, '.')
+      : 'page.tsx'
 
     it(`transforms correctly ${prefix}`, () => {
       runInlineTest(

diff --git a/packages/next-codemod/transforms/lib/async-request-api/next-async-dynamic-prop.ts b/packages/next-codemod/transforms/lib/async-request-api/next-async-dynamic-prop.ts
@@ -151,7 +151,7 @@ function applyUseAndRenameAccessedProp(
   return modified
 }
 
-const MATCHED_FILE_PATTERNS = /([\\/]|^)(page|layout)\.(t|j)sx?$/
+const MATCHED_FILE_PATTERNS = /([\\/]|^)(page|layout|route)\.(t|j)sx?$/
 
 function modifyTypes(
   paramTypeAnnotation: any,