Skip to content

feat(packagejson): autocompletion for script names.#229

Merged
pi0 merged 7 commits into
unjs:mainfrom
igalklebanov:add-node-scripts
Jun 26, 2025
Merged

feat(packagejson): autocompletion for script names.#229
pi0 merged 7 commits into
unjs:mainfrom
igalklebanov:add-node-scripts

Conversation

@igalklebanov

Copy link
Copy Markdown
Member

Hey 👋

Thanks for this awesome package! Have been using it in various projects, it's my go-to for package.json and tsconfig.json types and utilities.

This PR adds autocompletion for package.json's scripts object keys - npm and pnpm life cycle script names, and some common scripts from across the ecosystem.

@codecov

codecov Bot commented Jun 23, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.15%. Comparing base (afdf7cb) to head (1ed8b10).
Report is 74 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #229      +/-   ##
==========================================
+ Coverage   89.40%   91.15%   +1.75%     
==========================================
  Files           5       10       +5     
  Lines         434      260     -174     
  Branches       37       60      +23     
==========================================
- Hits          388      237     -151     
+ Misses         46       23      -23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread src/packagejson/types.ts
@igalklebanov igalklebanov requested a review from pi0 June 26, 2025 11:45
Comment thread package.json Outdated
Comment thread package.json Outdated

@pi0 pi0 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@pi0 pi0 merged commit a96072a into unjs:main Jun 26, 2025
3 checks passed
@igalklebanov igalklebanov deleted the add-node-scripts branch June 27, 2025 10:25
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.

2 participants