Consider a maintained replacement for the deep-copy test dependency
- Vorherrschende Sprache
- JavaScript
- Sterne
- 291
- Forks
- 37
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
`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.
Beitragsleitfaden
Rechercherichtung
Beginne mit package.json und der Lockdatei und prüfe anschließend ts/test/test-heap-profiler.ts, um die bestehende Verwendung von require('deep-copy') zu bestätigen. Wenn der Alias genehmigt ist, aktualisiere die Abhängigkeitsdeklaration und die Lockdatei, ohne den Testimport zu ändern, führe anschließend die heap-profiler-Testsuite aus und verifiziere, dass die Tests unter Node >=18 bestehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, node.js
- Bereich
- testing-qa
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 72/100