Skip to content

fix: exclude dev dependencies from DEPENDENCIES.md#34

Merged
carlos-alm merged 2 commits into
mainfrom
fix/deps-omit-dev
Feb 23, 2026
Merged

fix: exclude dev dependencies from DEPENDENCIES.md#34
carlos-alm merged 2 commits into
mainfrom
fix/deps-omit-dev

Conversation

@carlos-alm

Copy link
Copy Markdown
Contributor

Summary

  • Change deps:tree script from npm ls --all to npm ls --all --omit=dev
  • DEPENDENCIES.md now only lists production dependencies (303 lines vs 855)
  • Removes misleading inclusion of devDependencies (vitest, biome, husky, tree-sitter grammars, etc.)

Test plan

  • Run npm run deps:tree and verify output contains only production deps
  • Verify no devDependencies (vitest, biome, husky) appear in DEPENDENCIES.md

Change deps:tree script from `npm ls --all` to `npm ls --all --omit=dev`
so DEPENDENCIES.md only lists production dependencies. Previously it
included all devDependencies, making the file misleading.
@greptile-apps

greptile-apps Bot commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Modified the deps:tree npm script to add the --omit=dev flag, ensuring DEPENDENCIES.md reflects only production dependencies. The change reduced the file from 855 to 303 lines by excluding development tools like @biomejs/biome, vitest, husky, @commitlint/cli, and tree-sitter grammar packages that are only used during development.

  • Updated npm ls --all to npm ls --all --omit=dev in package.json:33
  • Regenerated DEPENDENCIES.md with production dependencies only
  • Correctly excludes devDependencies while preserving optionalDependencies like @huggingface/transformers and @modelcontextprotocol/sdk

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The change is a single-line modification to an npm script that adds a standard --omit=dev flag. The resulting DEPENDENCIES.md regeneration is correct and properly excludes devDependencies while preserving production and optional dependencies. No logic changes, no security implications.
  • No files require special attention

Important Files Changed

Filename Overview
package.json Added --omit=dev flag to deps:tree script to exclude devDependencies from DEPENDENCIES.md output
DEPENDENCIES.md Regenerated with production dependencies only, reduced from 855 to 303 lines by excluding devDependencies

Last reviewed commit: 63c6923

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@carlos-alm carlos-alm merged commit ec72021 into main Feb 23, 2026
15 checks passed
@carlos-alm carlos-alm deleted the fix/deps-omit-dev branch February 23, 2026 02:09
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

Successfully merging this pull request may close these issues.

1 participant