Merge branch 'release/next' into fix/suspense-request

.changeset/early-tomatoes-beg.md

@@ -0,0 +1,5 @@
+---
+'@ice/plugin-jsx-plus': patch
+---
+
+fix: allow decorators-legacy syntax in js file

.changeset/unlucky-boats-type.md

@@ -0,0 +1,5 @@
+---
+'@ice/runtime': patch
+---
+
+fix: throw error for better debugging

.github/workflows/ci.yml

@@ -22,7 +22,7 @@ jobs:
           node-version: ${{ matrix.node-version }}
           registry-url: https://registry.npmjs.org/
       - name: Install pnpm
-        uses: pnpm/action-setup@v2.2.2
+        uses: pnpm/action-setup@v4
         with:
           run_install: false
       - name: Get pnpm store directory

.github/workflows/coverage.yml

@@ -31,7 +31,7 @@ jobs:
     strategy:
       matrix:
         node-version: [16.x]
-    
+
     steps:
       - uses: actions/checkout@v3
         with:
@@ -42,7 +42,7 @@ jobs:
           node-version: ${{ matrix.node-version }}
           registry-url: https://registry.npmjs.org/
       - name: Install pnpm
-        uses: pnpm/action-setup@v2.2.2
+        uses: pnpm/action-setup@v4
         with:
           run_install: false
       - name: Get pnpm store directory

.github/workflows/release.yml

@@ -19,7 +19,7 @@ jobs:
         uses: actions/checkout@v3
 
       - name: Install pnpm
-        uses: pnpm/action-setup@v2
+        uses: pnpm/action-setup@v4
 
       - name: Use Node.js ${{ matrix.node-version }}
         uses: actions/setup-node@v3

.github/workflows/version.yml

@@ -19,7 +19,7 @@ jobs:
         uses: actions/checkout@v3
 
       - name: Install pnpm
-        uses: pnpm/action-setup@v2
+        uses: pnpm/action-setup@v4
 
       - name: Use Node.js ${{ matrix.node-version }}
         uses: actions/setup-node@v3

.github/workflows/website.yml

@@ -19,7 +19,7 @@ jobs:
           fetch-depth: 0
 
       - name: Install pnpm
-        uses: pnpm/action-setup@v2
+        uses: pnpm/action-setup@v4
 
       - name: Use Node.js 16.x
         uses: actions/setup-node@v3

packages/plugin-jsx-plus/src/index.ts

@@ -29,6 +29,7 @@ const babelTransformOptions = {
       'topLevelAwait',
       'classProperties',
       'classPrivateMethods',
+      'decorators-legacy', // allowing decorators by default
     ],
     generatorOpts: {
       decoratorsBeforeExport: true,
@@ -107,7 +108,6 @@ const plugin: Plugin<JSXPlusOptions> = (options: JSXPlusOptions = {}) => ({
           if (/\.tsx?$/.test(id)) {
             // When routes file is a typescript file, add ts parser plugins.
             options.parserOpts.plugins.push('typescript');
-            options.parserOpts.plugins.push('decorators-legacy'); // allowing decorators by default
           }
 
           const { code, map } = transformSync(source, options);

packages/runtime/src/routes.tsx

@@ -50,7 +50,8 @@ export async function loadRouteModule(route: RouteModule, routeModulesCache = {}
     return routeModule;
   } catch (error) {
     console.error(`Failed to load route module: ${id}.`);
-    console.error(error);
+    // Re-throw error for better debugging.
+    throw error;
   }
 }
 

packages/runtime/tests/routes.test.tsx

@@ -138,17 +138,20 @@ describe('routes', () => {
   });
 
   it('load error module', async () => {
-    const routeModule = await loadRouteModules([{
-      id: 'error',
-      // @ts-ignore
-      lazy: async () => {
-        throw new Error('err');
-        return {};
-      },
-    }], {});
-    expect(routeModule).toStrictEqual({
-      error: undefined,
-    });
+    // `toThrowError` seems not working with async function.
+    try {
+      await loadRouteModules([{
+        id: 'error',
+        // @ts-ignore
+        lazy: async () => {
+          throw new Error('err');
+          return {};
+        },
+      }], {});
+      expect(true).toBe(false);
+    } catch (err) {
+      expect(true).toBe(true);
+    }
   });
 
   it('load async route', async () => {