[Pkg-javascript-devel] Bug#1005287: Bug#1005287: node-csstype: Build drops "; " in typescript declarations

[Julien Puydt]

Le jeudi 10 février 2022 à 16:20 +0100, Yadd a écrit :
> On 10/02/2022 15:38, Yadd wrote:
> > Package: node-csstype
> > Version: 3.0.10-1
> > Severity: grave
> > Justification: renders package unusable
> > 
> > debian/rules launches `ts-node --files build.ts --start` which
> > (only)
> > modifies indes.d.ts and index.js.flow. The result is unusable with
> > tsc:
> > 
> >    /usr/share/nodejs/csstype/index.d.ts:18305:8 - error TS1005: ';'
> > expected.
> > 
> > All final ";" are dropped.
> 
> Removing override_dh_auto_build fixes the problem (using upstream 
> index.d.ts and index.js.flow which are readable)
> 
> @Julien, could you take a look ?

I think the problem is that in index.d.ts we have:

  export type SvgAttributes = <empty>

  export type Globals = <stuff>

while upstream has:

  export type SvgAttributes = <stuff>

  export type Globals = <stuff>

so the missing ';' is a red herring: we don't get the SvgAttributes
like we should, and that's the real problem.

I see that debian/nodejs/extlinks does have mdn-browser-compat-data,
but it creates node_modules/@mdn/browser-compat-data -- I don't know
why it does that, but it's what's breaking things as far a I can tell.

Do you know how to fix it?

J.Puydt