bindable decorator should not be giving a runtime error

[dkent600]

bindable decorator should not be giving a runtime error

[3cp]

This setting does fix the issue. I will put it in our WIP vite skeleton.

export default defineConfig({
  esbuild: {
    target: 'es2022',
  },

[dkent600]

Fixed by #1980 (comment)

Thanks for everyone's help!

[Sayan751]

Hi @dkent600, it seems like a vite+esbuild issue. As the latest esbuild added support for the native decorators, it is worth updating the version of esbuild. Moreover, as the parameter decorators are not yet supported by the new decorator specs, those instances needs to be refactored. The Au2 release notes mention this.

I have tried the aforementioned changes in your repo. Here is a change.patch that you can apply and check.

However, it seems that the decorators are still not transpiled. Unfortunately, I am not much familiar with vite and/or esbuild. @3cp @bigopon Do you guys notice anything configuration change that needs to be made in order to transpile the code view vite/esbuild?

[bigopon]

relax it to "^..."?.

Yes we should, completely missed it, thanks @3cp

@dkent600 I tried to debug by removing everything and only had these:

• main.ts

  @dec
  class App {}
  
  function dec() {
    console.log('???');
  }
  
  document.body.textContent = String(new App());

• vite.config.ts

  export default defineConfig({
    server: {
      port: 9000,
      strictPort: true,
    },
  })

And it still couldn't compile the decorator! Will check further.

[3cp]

Sounds like

    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,

is the temporary solution here before esbuild follows tsc behaviour.

@3cp I tried that before (see above) but it results in the runtime error:

TypeError: Cannot read properties of undefined (reading 'addInitializer')

Oh yes. Probably esbuild's decorator helper (in the compiled code) is for legacy decorator spec (honours experimentalDecorators), while Aurelia's decorator is using latest spec.

[bigopon]

It seems the version of vite in the repo is quite old, maybe let's test it with the latest first @dkent600 ? it's 5.2.11

[dkent600]

I have tried the aforementioned changes in your repo. Here is a change.patch that you can apply and check.

Thanks @Sayan751! FYI I went ahead and committed those changes to master, here: dkent600/douglaskent@fb3c171

[dkent600]

It seems the version of vite in the repo is quite old, maybe let's test it with the latest first @dkent600 ? it's 5.2.11

@bigopon The version that @Sayan751 is testing with is 5.2.11 which is only three weeks old.

[dkent600]

@Sayan751 @bigopon FYI I updated the vite config to use @aurelia/vite-plugin and made changes to the vite.config. The code is compiling again, but the vite build of the app is having this problem at runtime when it hits @customElement({ name: "resume", template }):

Uncaught SyntaxError SyntaxError: Invalid or unexpected token
at (program) (c:\Users\dkent\Documents\Projects\douglaskent\src\pages\resume\resume.ts:18:1)

The changes are here: dkent600/douglaskent@e1c46f9

PS: There is also a production build issue from rollup. I will investigate.

[dkent600]

Uncaught SyntaxError SyntaxError: Invalid or unexpected token
at (program) (c:\Users\dkent\Documents\Projects\douglaskent\src\pages\resume\resume.ts:18:1)

Both the above error and the production build issue from rollup go away if I restore the following to tsconfig.json:

    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,

Of course that causes other issues to ensue, namely in @customElement:

TypeError: Cannot read properties of undefined (reading 'addInitializer')

[bigopon]

I'm not sure what's causing the prod issue, since everything looks fine.

"experimentalDecorators": true,
"emitDecoratorMetadata": true,

We should not need these 2, since it's for the old decorators code, we no longer have it.

[Sayan751]

It seems like a configuration issue to me, since the latest version of esbuild can transpile the decorators. Maybe something in the in vite config or rollup is missing.

[3cp]

@bigopon a side issue. Our plugin-conventions is pinned to one typescript version, should we relax it to "^..."?.

"node_modules/@aurelia/plugin-conventions": {
      "version": "2.0.0-beta.17",
      "resolved": "https://registry.npmjs.org/@aurelia/plugin-conventions/-/plugin-conventions-2.0.0-beta.17.tgz",
      "integrity": "sha512-m14ML6Atfl5B389e1Hxj27H/f1lTm33HyIvgdLvUp+jLUaQnTlf5bqiSmNQLHlrhcnskH+nKfG6ngJ3umEPBgw==",
      "dev": true,
      "dependencies": {
        "@aurelia/kernel": "2.0.0-beta.17",
        "@aurelia/metadata": "2.0.0-beta.17",
        "@aurelia/platform": "2.0.0-beta.17",
        "@aurelia/runtime": "2.0.0-beta.17",
        "@aurelia/runtime-html": "2.0.0-beta.17",
        "modify-code": "^2.1.3",
        "parse5": "^5.1.1",
        "typescript": "5.4.2"
      },
      "engines": {
        "node": ">=14.17.0"
      }
    },

[3cp]

@dkent600 you patched our html import to .html?raw, I don't think that would ever work with au2.

Our plugin already rewrote html file into js code, it's not raw text anymore.

My bad, missed your condition.

[dkent600]

For what it is worth, running npx tsc has no problem, with tsconfig changes:

    //"noEmit": true,
    "outDir": "dist/tsc",
    // "experimentalDecorators": true,
    // "emitDecoratorMetadata": true,

[bigopon]

I quite give up on this ... but the good news is if you use true for enableConventions in the viteconfig, you'll not have to worry about the decorators.
(it doesn't compile with even simplest app & config whatsoever)

[dkent600]

I quite give up on this ... but the good news is if you use true for enableConventions in the viteconfig, you'll not have to worry about the decorators. (it doesn't compile with even simplest app & config whatsoever)

@bigopon I have set enableConventions to true in vite.config but the vite compile still fails on the decorator with:

Uncaught SyntaxError SyntaxError: Invalid or unexpected token
at (program) (c:\Users\dkent\Documents\Projects\douglaskent\src\pages\resume\resume.ts:18:1)

This can be seen in this commit: dkent600/douglaskent@fdd152c

[3cp]

I tried too. I think the setup somehow skipped typescript compilation which Vite should do automatically for .ts files. So even conventions handled the decorator, user code still fails with decorator.

I could not figure out what happened.

[bigopon]

Not that this helps, but I'll just post anyway. I saw that the presence of

@esbuild-plugins/node-globals-polyfill

force npm to install esbuild wasm, removing it causes npm to take normal esbuild package.

Btw, when I change to main.js, and the index.html to:

<!DOCTYPE html>
<html>
  <head>
    <script type="module" src="src/main.js"></script>
  </head>
  <body>
  </body>
</html>

it still doesn't build ... well...!

[dkent600]

I quite give up on this ... but the good news is if you use true for enableConventions in the viteconfig, you'll not have to worry about the decorators. (it doesn't compile with even simplest app & config whatsoever)

@bigopon Assuming this issue is addressed by Vite, which do you generally recommend for `enableConventions'? true or false? I am not clear what is its purpose.

[dkent600]

I created a branch for this issue:

https://github.com/dkent600/douglaskent/tree/decorators-issue

And a ticket for vite: vitejs/vite#17308 (I hope I didn't make any mistakes with the facts of the matter)

[dkent600]

I have a feeling that you guys and Vite may not be aligned about what should be happening here. In the given scenario, should the decorator syntax be getting transpiled for the browser, and if so, to what?

Please see: vitejs/vite#17308 (comment)

[3cp]

Sounds like

    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,

is the temporary solution here before esbuild follows tsc behaviour.

[dkent600]

Sounds like

    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,

is the temporary solution here before esbuild follows tsc behaviour.

@3cp I tried that before (see above) but it results in the runtime error:

TypeError: Cannot read properties of undefined (reading 'addInitializer')

[dkent600]

Status: Awaiting the fix in vite to address this issue that is apparently due to vite not using the latest version of esbuild.

[bigopon]

Thanks @dkent600 , I'm not sure what vite does internally, because even with overrides in package json to ensure esbuild version 0.21.3, it still didn't compile, as shown in my comment above.

[dkent600]

Thanks @dkent600 , I'm not sure what vite does internally, because even with overrides in package json to ensure esbuild version 0.21.3, it still didn't compile, as shown in my comment above.

Agreed @bigopon , I had the same thought. But they are claiming it fixes the issue. Will see...

[dkent600]

Confirmed this setting does fix the issue: #1980 (comment)

If there has yet been a Vite release of what was supposed to be a fix, it is not sufficient.