Bug - Package Publishing - Published Libraries should follow NPM dual module package standards

Offen

#11,686 erstellt am 18.3.2025 4 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 42/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: nodejs, typescript
Bereich: build-system, developer-experience, release

Rechercherichtung

Überprüfe die im Issue gezeigten Paketmetadatenfelder in allen PatternFly-Paketen und reproduziere anschließend das aktuelle Problem bei der Modulauflösung mit vitest. Prüfe zuerst die Referenzen zu NPM package-examples und Vite library-mode; erledigt ist die Aufgabe, wenn jedes betroffene Paket sowohl ESM als auch CommonJS über standardmäßige Paketmetadaten unterstützt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Breaking change :boom: Stale

PatternFly libraries should follow NPM standards for publishing dual module (CommonJS/ESModule) packages.

PatternFly packages have

  "main": "dist/js/index.js",
  "module": "dist/esm/index.js",
  "types": "dist/esm/index.d.ts",

This is a really early format when es modules were being initially developed and not supported by modern tooling.

NPM documentation links to this as the right way to handle dual module packages.

{
  "type": "module",
  "exports": {
    "import": "./index.mjs",
    "require": "./index.cjs"
  }
}

Modern tooling like vite builds libraries with a combination of both for compatibility.

{
  "name": "my-lib",
  "type": "module",
  "main": "./dist/my-lib.umd.cjs",
  "types": "./dist/my-lib.d.ts",
  "module": "./dist/my-lib.js",
  "exports": {
    ".": {
      "import": "./dist/my-lib.js",
      "require": "./dist/my-lib.umd.cjs"
    }
  }
}

This is currently blocking us from using vitest to test because it gets really confused. It tries to import the code as ESM but then is unhappy that the package is not marked as type "module". If we manually add type 'module', it then runs into some code that uses a 'require', which is not valid in the scope of a module.

PatternFly packages should support both ESM and CommonJS using NPM standards for better compatibility.

This affects all PF packages not just react-core.

Vorherrschende Sprache: TypeScript
Sterne: 862
Forks: 392
Ø Merge: 4 T. 8 Std.
Gemergte PRs (30 T.): 9

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus patternfly/patternfly-react

Bug - Label - Doesn't allow ref forwarding Offen

PF Team

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

patternfly/patternfly-react#12651 · Erstellt am 14.9.2026
Bug - [Menu] - Flyout stays open when hovering a disabled item Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

patternfly/patternfly-react#12637 · Erstellt am 02.9.2026
Table - support for small toggle using ActionsColumn Offen

PF Team

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100

patternfly/patternfly-react#12617 · 1 Kommentar · Erstellt am 13.8.2026
Toolbar - add support for item minWidths Offen

PF Team

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

patternfly/patternfly-react#12610 · 1 Kommentar · 1 Reaktion · Erstellt am 03.8.2026
Bug - [LoginForm] - `autoComplete` is not exposed and cannot be defined in the LoginForm. Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

patternfly/patternfly-react#12600 · Erstellt am 28.7.2026

Alle Issues in patternfly/patternfly-react