What about types?

[eatsjobs]

as per title I saw there are not types nor @types/uhtml package. am I missing something?

[WebReflection]

you TS folks need to talk 😅
#112

[eatsjobs]

@WebReflection I see. I opened an MR #116 would mind review it?
Types were buggy cause some types were(are) pointing to relative .js files and placing types in a specific folder was breaking the resolution.

[WebReflection]

@eatsjobs I really don't like all that noise in the repo ... I mean, locally in my editor, is there really nothing we can do to have all types correctly resolved in a folder a part? this should also create types only once because the cjs artifact is a 1:1 representation of the esm one so one types folder from esm should satisfy both cases.

[eatsjobs]

Hey @WebReflection unfortunately not that I'm aware of. The problem are the types with dynamic import like
typeof import("./relative.js").MyObject. If we move the types in another folder this will not work unless you rewrite the relative dynamic import somehow. you're using rollup have you tried this already? https://www.npmjs.com/package/rollup-plugin-dts

[WebReflection]

@eatsjobs

This project is in maintenance mode.

nope, and not planning to ... I wish there was https://www.npmjs.com/package/rollup-plugin-jsdoc for JSDoc TS that would do whatever the dts plugin does at the end ... we can't have it all, it seams ... btw, with latest changes with exports, do you still have issues with types? talking about 4.5.1

[eatsjobs]

https://www.npmjs.com/package/uhtml?activeTab=code cannot see the .d.ts file in the npm package. am I missed something? I don't see a "file" field in the package.json 🤔

[eatsjobs]

Since my changes are not in main even if you merge it maybe you rebased? anyway I re-prepare the "same" MR #119
@WebReflection by the way yes solves my typings issue

[WebReflection]

@eatsjobs I've reverted your change because it breaks on postbuild ... I think if I try it again it will break again and that's a no-go to me as I run build before publishing and the errors block my flow.

[eatsjobs]

@WebReflection Ok I can work on that

[eatsjobs]

@WebReflection Once you release 4.5.2 we can easily see if the types are working here https://stackblitz.com/edit/vitejs-vite-udoa4c?file=src%2Fcounter.ts

[eatsjobs]

I'd like to leave also this article here cause I think is related. https://blog.isquaredsoftware.com/2023/08/esm-modernization-lessons/ what a time to be a package maintainer!

[iacore]

as per title I saw there are not types nor @types/uhtml package. am I missing something?

Types should have worked already without the .d.ts files. Did it not work for you?

update: I see you mention stackblitz. I use the built-in Typescript LSP from VS Code.

[iacore]

Types are broken again on v4.5.5. uhtml/ssr has the wrong type file.

v4.5.2 is fine. Both Deno and tsc on Node.JS accept that.

To test this yourself, save the following two files as a.ts and b.ts, and run both with deno check a.ts or deno check b.ts.

This works. If you change initSsr() to initSsr.a(), it will show a type error, which is expected.

import initSsr from 'https://esm.sh/uhtml@4.5.2/ssr';

const { html, render } = initSsr()

const abc = [1,2,3]

console.log(html`hi, i can count to three! ${abc.map((x,i) => html`${x}${i==2?'!':', '}`)}`.toDOM().toString())

This already shows a type error without modification. ("initSsr is not callable")

import initSsr from 'https://esm.sh/uhtml@4.5.5/ssr';

const { html, render } = initSsr()

const abc = [1,2,3]

console.log(html`hi, i can count to three! ${abc.map((x,i) => html`${x}${i==2?'!':', '}`)}`.toDOM().toString())

[eatsjobs]

as per title I saw there are not types nor @types/uhtml package. am I missing something?

Types should have worked already without the .d.ts files. Did it not work for you?

update: I see you mention stackblitz. I use the built-in Typescript LSP from VS Code.

no they don't work with latest version of typescript and a vanilla-ts vite template like you can test it in stackblitz link