[npm-check-fork] Replace package-json and throat dependencies with local implementations
Maintainer antworten meist innerhalb von 1 Tag
@TheLarkInn arbeitet bereits daran.
Seit 24.1.2026.
Bewertung
Dieses Issue wurde noch nicht bewertet.
Beschreibung
Summary
The @rushstack/npm-check-fork package depends on package-json and throat, which should be replaced with existing Rush Stack utilities:
- package-json: ESM-only module with a heavy dependency tree (~500KB). We only use it to fetch basic registry metadata.
- throat: Concurrency limiter that duplicates functionality already in
@rushstack/node-core-library
Details
Proposed solution:
- Replace
package-jsonwith a minimalNpmRegistryClientclass using the existingWebClientfrom@rushstack/rush-lib - Replace
throatwithAsync.forEachAsyncfrom@rushstack/node-core-library
Benefits:
- Eliminates ESM compatibility issues with CJS consumers
- Reduces bundle size and dependency footprint
- Integrates with Rush's existing proxy configuration
- Uses utilities already in the codebase
- Reduces chances of CVE's from upstream dependencies
Standard questions
| Question | Answer |
|---|---|
| Package name: | @rushstack/npm-check-fork |
| Package version? | 5.0.x |
| Operating system? | All |
| Would you consider contributing a PR? | Yes |
Node.js version (node -v)? |
18.x+ |
- Vorherrschende Sprache
- TypeScript
- Sterne
- 6.5k
- Forks
- 708
- Ø Merge
- 4 T. 7 Std.
- Gemergte PRs (30 T.)
- 61
Entwicklungsumgebung
Die Einrichtungsdateien dieses Projekts haben wir noch nicht geprüft. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
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 microsoft/rushstack
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
microsoft/rushstack#5971 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
microsoft/rushstack#5902 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
microsoft/rushstack#5839 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
microsoft/rushstack#5683 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in microsoft/rushstack
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
rohitg00/agentmemory#1428 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
boxlite-ai/boxlite#1729 ·
Maintainer antworten meist innerhalb von 1 Tag
-
detectors enhancement good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
SM260845/readme-gen#1 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
angular/angularfire#3774 ·
Maintainer antworten meist innerhalb von 2 Tagen