google / google/pprof-nodejs

Consider a maintained replacement for the deep-copy test dependency

Abierto Apto para principiantes
#359 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
JavaScript
Estrellas
291
Forks
37
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

`package.json` currently declares `deep-copy@^1.4.2` as a direct dev dependency. `ts/test/test-heap-profiler.ts` loads it with `require('deep-copy')` and uses it to clone the plain profile fixtures in the heap-profiler tests.

Would the maintainers approve replacing that declaration with an npm alias?

```json
"deep-copy": "npm:@stackline/deep-copy@1.0.0"
```

The alias would preserve the existing import key and callable CommonJS API, so the test source would not need to change. The replacement supports the repository's Node `>=18` range and has no production dependencies. I maintain `@stackline/deep-copy`; its package and compatibility documentation are available at https://www.npmjs.com/package/@stackline/deep-copy and https://alexandro.net/docs/vanilla/deep-copy/contract.html.

If this direction is acceptable, I can prepare the focused manifest/lockfile change and run the heap-profiler test suite. Keeping the current version or choosing another maintained compatible implementation are also reasonable alternatives if the project prefers them.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza con package.json y el archivo de bloqueo; después, inspecciona ts/test/test-heap-profiler.ts para confirmar el uso existente de require('deep-copy'). Si se aprueba el alias, actualiza la declaración de la dependencia y el archivo de bloqueo sin cambiar la importación del test; luego ejecuta la suite de tests de heap-profiler y verifica que los tests pasen en Node >=18.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
javascript, node.js
Área
testing-qa
Tipo de issue
Refactorización
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Activo
Claridad
Bastante claro
Aptitud para principiantes
72/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.