next-codemod(upgrade): optional catch when missing dev script (vercel#71598)

imprakharshukla · devjiwonchoi · web-flow · commit e9b18d6493ce · 2024-10-23T07:09:07.000Z
### Why? During the next-codemod upgrade, when the `dev` script is missing, it throws while reading `.includes` the undefined value. Fixes vercel#71597 --------- Co-authored-by: Jiwon Choi <devjiwonchoi@gmail.com>
diff --git a/packages/next-codemod/bin/upgrade.ts b/packages/next-codemod/bin/upgrade.ts
@@ -412,7 +412,7 @@ function isUsingAppDir(projectPath: string): boolean {
  */
 async function suggestTurbopack(packageJson: any): Promise<void> {
   const devScript: string = packageJson.scripts['dev']
-  if (devScript.includes('--turbopack')) return
+  if (devScript?.includes('--turbopack')) return
 
   const responseTurbopack = await prompts(
     {

Original file line number	Diff line number	Diff line change
`@@ -412,7 +412,7 @@ function isUsingAppDir(projectPath: string): boolean {`
`412`	`412`	`*/`
`413`	`413`	`async function suggestTurbopack(packageJson: any): Promise<void> {`
`414`	`414`	`const devScript: string = packageJson.scripts['dev']`
`415`		`- if (devScript.includes('--turbopack')) return`
	`415`	`+ if (devScript?.includes('--turbopack')) return`
`416`	`416`
`417`	`417`	`const responseTurbopack = await prompts(`
`418`	`418`	`{`