feat: support syntax error

fix demo by removing async call.
The runtime needs to wait for compile and copy commonjs first before running.

Author: unional

package.json

@@ -1,7 +1,7 @@
 {
   "private": true,
   "scripts": {
-    "build": "turbo run build --filter=./packages/*",
+    "build": "turbo run build --filter=./packages/* --force",
     "clean": "turbo run clean",
     "compile": "tester compile",
     "node": "pnpm --filter node",

packages/tester/ts/logic/compile.ts

@@ -15,38 +15,35 @@ export function runCompile(ctx: {
   projectPath: string,
 }) {
   return {
-    compileRaw: compileProject(ctx)
+    compileRaw: compileProject(ctx)!
   }
 }
 
 export type RunCompileContext = ReturnType<typeof runCompile>
 
-async function compileProject(ctx: { project: string, projectPath: string }) {
-  return new Promise<string>(a => {
-    cp.exec(`pnpm ${ctx.project} build`, (_err, stdout) => {
-      a(stdout)
-    })
-  })
-    .then((raw) => {
-      copyCommonJSPackageJson(ctx)
-      return raw
-    })
+function compileProject(ctx: { project: string, projectPath: string }) {
+  try {
+    const raw = cp.execSync(`pnpm ${ctx.project} build`).toString()
+    copyCommonJSPackageJson(ctx)
+    return raw
+  } catch (e: any) {
+    return e.stdout.toString()
+  }
 }
 
 export function processCompileResults({ compileRaw, subjects, moduleTypes }: {
   moduleTypes: string[]
 } & RunCompileContext & TestSubjectsContext) {
 
+  const results = extractCompileErrors(compileRaw)
+  const compileErrors = results.flatMap(result => result.importType === 'all'
+    ? subjects
+      .find(s => s.name === result.subject)!.files
+      .map(f => ({ ...result, importType: f.importType }))
+    : result)
+
   return {
-    compile: compileRaw
-      .then(extractCompileErrors)
-      .then(results => results.flatMap(result => result.importType === 'all'
-        ? subjects
-          .find(s => s.name === result.subject)!.files
-          .map(f => ({ ...result, importType: f.importType }))
-        : result
-      ))
-      .then(compileErrors => buildCompileResults({ compileErrors, moduleTypes, subjects }))
+    compile: buildCompileResults({ compileErrors, moduleTypes, subjects })
   }
 }
 

packages/tester/ts/testResults.ts

@@ -133,12 +133,15 @@ function extractRuntimeErrorKey(error: Error) {
     }
     return 'ref'
   }
+  if (/SyntaxError: /.test(message)) {
+    return 'syntax'
+  }
   return error.name
 }
 
 export function extractRuntimeErrorMessage(error?: Error) {
   if (!error) return ''
-  const match = /\n\n([^\n]*)/.exec(error.message)
+  const match = error.message.split('\n').map(line => /Error: (.*)$/.exec(line)).find(x => x)
   return match ? match[1] : error.message
 }
 

tests/node-allow/test-result.4.9.4.md

@@ -0,0 +1,3 @@
+import { default as m } from 'named-cjs'
+
+m.inc(1)

tests/node-allow/ts/named-cjs.default-as.all.ts

@@ -0,0 +1,3 @@
+import { default as m } from 'named-es-cjs'
+
+m.inc(1)

tests/node-allow/ts/named-es-cjs.default-as.all.ts

@@ -0,0 +1,3 @@
+import { default as m } from 'named-esm-cjs'
+
+m.inc(1)

tests/node-allow/ts/named-esm-cjs.default-as.all.ts

@@ -0,0 +1,3 @@
+import m from 'named-esm-cjs'
+
+m.inc(1)

tests/node-allow/ts/named-esm-cjs.default.all.ts

@@ -0,0 +1,3 @@
+import { default as m } from 'named-cjs'
+
+m.inc(1)