Skip to content

Remove unused dependencies #156

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
alexparish opened this issue Dec 2, 2024 · 1 comment
Open

Remove unused dependencies #156

alexparish opened this issue Dec 2, 2024 · 1 comment

Comments

@alexparish
Copy link

The package agent-base is listed in this package's dependencies but is not used in the codebase. Can it be removed?

Additionally the package temp-fs is listed in dependencies but is only referenced in tests. Can it be moved to devDependencies?

This could be classed as general housekeeping but a benefit for the consumer is a reduction in bandwidth required when installing or updating this package.

I'd be happy to submit a PR for these changes.

@mgol
Copy link

mgol commented Mar 17, 2025

temp-fs was last published 9 years ago! I'd replace it with native Node.js fs.mkdtemp(). That gives you a fresh temporary directory in which you can create as many temporary files as you need.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants