[npm-check-fork] Replace package-json and throat dependencies with local implementations
Los mantenedores suelen responder en 1 día
@TheLarkInn ya está trabajando en esto.
Desde el 24/1/2026.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
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+ |
- Lenguaje dominante
- TypeScript
- Estrellas
- 6.5k
- Forks
- 708
- Merge medio
- 4 d 7 h
- PR fusionados (30 d)
- 61
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. Empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de microsoft/rushstack
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
microsoft/rushstack#5971 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
microsoft/rushstack#5902 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
microsoft/rushstack#5839 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
microsoft/rushstack#5683 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Los mantenedores suelen responder en 1 día
Todos los issues de microsoft/rushstack
Issues similares
-
module-request
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
ports get and web print 'Port N already in use, trying next...' for every busy port they skipAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
appandflow/stim#1604 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
lingdojo/kana-dojo#31060 · 1 comentario · 5 reacciones ·
Los mantenedores suelen responder en 1 día
-
SSH workspace restore rewrites relative symlinks into the deleted sync-back staging directoryAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
paperclipai/paperclip#14173 ·
Los mantenedores suelen responder en 1 día
-
needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día