Skip to content

Conversation

@smnhgn
Copy link
Member

@smnhgn smnhgn commented Oct 16, 2025

Folgende Änderungen wurden vorerst umgesetzt:

  • Entfernen von commonjs-Syntax (require, module.exports, __dirname etc.)
  • Umstellung auf primäres ESModule ("type": "module" in package.json)
  • Build-Umstellung von babel (nur commonjs) auf typescript (commonjs und esmodule)
  • Upgrade auf Node v24.10.0
  • Umstellung von node-fetch und abort-controller auf native fetch und AbortController
  • Korrekte Package-Exporte in package.json ("main", "module", "types", "exports")
  • Umstellung Package-Name von tableaux-aggregator auf grud-aggregator
  • Major-Version von 8.8.0 auf 9.0.0

Anmerkung:

  • Typisierung ist nicht Teil dieses PRs und sollte graduell (Datei für Datei) nachgezogen werden!

Copy link
Member

@zingmane zingmane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finde ich alles schon mal sehr gute Änderungen, die das Projekt auf jeden Fall nach vorne bringt.
Mit dem renaming weiß ich nicht so Recht. Das bringt vermutlich auch etliche Schwierigkeiten in anderen Projekten mit sich, weil wir das per github URL importieren.

Ach wusste garnicht, dass das Repo schon (länger) so hieß...
Dann ist es nur konsequent, auch das package so zu benennen. 👍

Copy link
Member

@Zwergal Zwergal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finde ich auch alles gut.
Sollten wir aber auch in einem der Projekte wie exportcenter testen, ob es da mit esm wie gehabt genutzt werden kann.

@smnhgn
Copy link
Member Author

smnhgn commented Oct 17, 2025

Also für andere Projekte sollte sich bzgl. cjs und esm hoffentlich nichts ändern.
Bisher wurde das Projekt nur für commonjs exportiert und jetzt für commonjs und esm.
In anderen Projekten waren wir ja gezwungen commonjs zu verwenden (d.h. bspw. ts-projekt das zu cjs compiliert), weil commonjs-apps/packages kein esm importieren können. (esm kann sowohl cjs als auch esm importieren).
Mit den Änderungen können wir den aggregator auch in "moderneren" esm-Projekten nutzen (bspw. ts-projekt das zu esm compiliert, oder reines js mit esm).

Ich habe nur an einigen Stellen die Syntax des JS Source-Code angepasst, damit wir dort einheitlich esm haben.
Vorher war das ja so ein Misch-Masch aus esm und commonjs.
Getestet habe ich das ganze schonmal lokal beim aggregator in buschjost.

In anderen Projekten würde ich dann auch nochmal stichpunktartig testen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants