Consider a maintained replacement for the deep-copy test dependency
- Dominant language
- JavaScript
- Stars
- 291
- Forks
- 37
- PR merge metrics
- No merged PRs in 30d
Description
`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.
Contributor guide
Research direction
Start with package.json and the lockfile, then inspect ts/test/test-heap-profiler.ts to confirm the existing require('deep-copy') usage. If the alias is approved, update the dependency declaration and lockfile without changing the test import, then run the heap-profiler test suite and verify the tests pass on Node >=18.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, node.js
- Domain
- testing-qa
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 72/100