Skip to content
This repository was archived by the owner on Jan 6, 2026. It is now read-only.

Remove unused dependencies#1760

Merged
NullVoxPopuli merged 1 commit intomainfrom
cleanup-unused-dependencies
Jul 6, 2025
Merged

Remove unused dependencies#1760
NullVoxPopuli merged 1 commit intomainfrom
cleanup-unused-dependencies

Conversation

@wycats
Copy link
Copy Markdown
Contributor

@wycats wycats commented Jul 5, 2025

Summary

This PR removes unused dependencies identified through code analysis:

  • bin/package.json: Removed globby, pkg-entry-points, spex
  • repo-metadata/package.json: Removed fs-extra

Analysis

I analyzed the codebase to verify which dependencies were actually unused vs false positives from the knip tool:

bin directory

  • globby, pkg-entry-points, spex - Not imported anywhere in the codebase
  • Other reported dependencies (chalk, execa, etc.) - Actually used and kept

repo-metadata directory

  • fs-extra - Not used (code uses Node.js built-in fs module instead)
  • Other reported dependencies - All actively used in the codebase

Testing

  • pnpm install completes successfully
  • pnpm build completes without errors
  • ✅ No functionality changes

This is a small cleanup that reduces the dependency footprint without affecting functionality.

- Remove globby, pkg-entry-points, spex from bin/package.json
- Remove fs-extra from repo-metadata/package.json

These dependencies were identified as unused through code analysis.
All other dependencies reported by knip were false positives.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 5, 2025

This PRmain
Dev
588K └─┬ .
169K   ├── runtime
160K   ├── syntax
100K   ├── compiler
 58K   ├── opcode-compiler
 27K   ├── manager
 24K   ├── validator
 11K   ├── program
8.9K   ├── reference
7.2K   ├── destroyable
6.3K   ├── util
4.3K   ├── node
3.4K   ├── global-context
2.5K   ├── wire-format
1.0K   ├── vm
969B   ├── encoder
844B   ├── vm-babel-plugins
606B   └── owner
588K └─┬ .
169K   ├── runtime
160K   ├── syntax
100K   ├── compiler
 58K   ├── opcode-compiler
 27K   ├── manager
 24K   ├── validator
 11K   ├── program
8.9K   ├── reference
7.2K   ├── destroyable
6.3K   ├── util
4.3K   ├── node
3.4K   ├── global-context
2.5K   ├── wire-format
1.0K   ├── vm
969B   ├── encoder
844B   ├── vm-babel-plugins
606B   └── owner
Prod
231K └─┬ .
 70K   ├── syntax
 63K   ├── runtime
 48K   ├── compiler
 18K   ├── opcode-compiler
7.9K   ├── manager
5.1K   ├── validator
4.8K   ├── program
3.6K   ├── reference
2.4K   ├── util
2.1K   ├── node
1.6K   ├── wire-format
1.5K   ├── destroyable
737B   ├── vm
594B   ├── global-context
516B   ├── encoder
469B   ├── vm-babel-plugins
155B   └── owner
231K └─┬ .
 70K   ├── syntax
 63K   ├── runtime
 48K   ├── compiler
 18K   ├── opcode-compiler
7.9K   ├── manager
5.1K   ├── validator
4.8K   ├── program
3.6K   ├── reference
2.4K   ├── util
2.1K   ├── node
1.6K   ├── wire-format
1.5K   ├── destroyable
737B   ├── vm
594B   ├── global-context
516B   ├── encoder
469B   ├── vm-babel-plugins
155B   └── owner

@NullVoxPopuli NullVoxPopuli merged commit 4cf00d1 into main Jul 6, 2025
12 checks passed
@NullVoxPopuli NullVoxPopuli deleted the cleanup-unused-dependencies branch July 6, 2025 12:22
@github-actions github-actions Bot mentioned this pull request Jul 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants