Skip to content

chore(deps): update knip ✂️ and add enable treatTagHintsAsErrors#2824

Merged
ghostdevv merged 1 commit into
npmx-dev:mainfrom
cylewaitforit:knip-6.15.0-treatTagHintsAsErrors
May 31, 2026
Merged

chore(deps): update knip ✂️ and add enable treatTagHintsAsErrors#2824
ghostdevv merged 1 commit into
npmx-dev:mainfrom
cylewaitforit:knip-6.15.0-treatTagHintsAsErrors

Conversation

@cylewaitforit
Copy link
Copy Markdown
Contributor

@cylewaitforit cylewaitforit commented May 31, 2026

🔗 Linked issue

Closes: #2821

🧭 Context

Updates knip to 6.15.0 and enables treatTagHintsAsErrors. This will require that unused tags identified as unused by knip be removed.

📚 Description

This can be checked by adding an unused tag like those removed in #2786 and then running pnpm knip --production --exclude dependencies to see it exit with code 1.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs.npmx.dev Ready Ready Preview, Comment May 31, 2026 3:48pm
npmx.dev Ready Ready Preview, Comment May 31, 2026 3:48pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored May 31, 2026 3:48pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 31, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

Knip is upgraded from v6.14.2 to v6.15.0, and the new treatTagHintsAsErrors: true configuration option is enabled in knip.ts to enforce stricter validation of unnecessary tag hints within the codebase.

Changes

Knip tag hint validation

Layer / File(s) Summary
Enable stricter tag hint validation
package.json, knip.ts
Knip dependency is bumped to v6.15.0 to support the new treatTagHintsAsErrors option, which is then enabled in the KnipConfig to prevent unnecessary tag hints from remaining in the codebase.

Possibly related PRs

  • npmx-dev/npmx.dev#2786: Both PRs modify knip.ts configuration to enforce hints-as-errors strategies; this PR adds treatTagHintsAsErrors whilst the other adds treatConfigHintsAsErrors.

Suggested reviewers

  • serhalp
  • ghostdevv
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes both main changes: updating knip dependency and enabling treatTagHintsAsErrors configuration.
Linked Issues check ✅ Passed The PR fully addresses issue #2821 by updating knip to 6.15.0 and enabling treatTagHintsAsErrors in the configuration.
Out of Scope Changes check ✅ Passed All changes are directly related to the linked issue objectives: knip version bump and treatTagHintsAsErrors enablement, with no extraneous modifications.
Description check ✅ Passed The pull request description clearly relates to the changeset, providing context about the knip upgrade and the enablement of treatTagHintsAsErrors.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedknip@​6.15.0991009596100

View full report

@codecov
Copy link
Copy Markdown

codecov Bot commented May 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@github-actions
Copy link
Copy Markdown

e18e dependency analysis

No dependency warnings found.

@ghostdevv ghostdevv added this pull request to the merge queue May 31, 2026
Merged via the queue into npmx-dev:main with commit 26091b2 May 31, 2026
29 checks passed
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.

Enable treatTagHintsAsErrors in knip config

2 participants