google / google/pprof-nodejs

Consider a maintained replacement for the deep-copy test dependency

Open Beginner friendly
#359 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.