chore: remove unused release tooling deps#93
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 184b934f94
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
This PR removes unused release-related development dependencies, deletes an unused Jest environment patch helper, and updates the release script to avoid relying on fs-extra.
Changes:
- Replace
fs-extrausage inscripts/release.jswithnode:fs+JSON.parse. - Delete the unused
scripts/patch-node-env.cjshelper. - Remove several release/tooling dev dependencies and update
pnpm-lock.yamlaccordingly.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| scripts/release.js | Switches package.json reading from fs-extra to built-in node:fs. |
| scripts/patch-node-env.cjs | Removes an unused Jest environment patch file. |
| package.json | Drops unused devDependencies related to release tooling. |
| pnpm-lock.yaml | Removes lock entries corresponding to the dropped dependencies. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
This PR removes unused release-related development dependencies and deletes the unused
patch-node-envhelper. It also updatesscripts/release.jsto readpackage.jsonwith the built-innode:fsAPI so the release script still works after droppingfs-extra.Testing
Related Links