component-library-react/dist/index.d.ts fixen

[Robbert]

Zie https://unpkg.com/browse/@utrecht/[email protected]/dist/

vs.

https://unpkg.com/browse/@utrecht/[email protected]/dist/

Er ontbreken een heleboel declaration files.

[matijs]

De declaration files ontbreken niet, ze staan genest in ./src

De tsconfig.json in /packages/component-library-react bevat zowel compilerOptions.paths als references.

Deze references lijken niet op te worden gepakt door tsc en, naar het lijkt, ook niet door rollup-plugin-typescript2 (rpt2). Bovendien is er alleen een referentie voor @utrecht/button-react maar ontbreekt de referentie voor @utrecht/calendar-react. Het is niet helemaal duidelijk of paths überhaupt iets doet als er references zijn.

De component packages (@utrecht/button-react en @utrecht/calendar-react) zijn genest in de ./packages directory in /packages/component-library-react en dat lijkt (?) ervoor te zorgen dat TypeScript zelf bepaalt wat de waarde voor rootDir wordt. De hoogst gelegen gemeenschappelijke root voor @utrecht/component-library-react en @utrecht/button-react en @utrecht/calendar-react is /packages/component-library-react/ wat ervoor lijkt te zorgen dat ./src opeens wordt meegenomen naar outDir. (Voorheen was ./src zelf de hoogst gelegen automatisch bepaalde rootDir waardoor dat niet zo was.

Nog wat losse bevindingen

• In de repository wordt TypeScript 4 gebruikt. In TypeScript 5 is Bundler een mogelijke waarde voor moduleResolution helaas
• Het entry point waarbij rollup-plugin-postcss wordt gebruikt lijkt css.tsx te moeten zijn maar is ook index.tsx vermoedelijk moet dat rechtgetrokken worden
• rollup-plugin-peer-deps-external + outputGlobals lijkt dubbelop
• tsconfig.base.json#compilerOptions.module zou NodeNext moeten zijn omdat moduleResolution dat ook is maar dat veranderen is geen oplossing
• Soms klaagt tsc over een onClick prop in ./packages/calendar-react/src/index.tsc#L176 soms ook niet 🤷