Skip to content

Comments

chore: add knip#139

Merged
serhalp merged 1 commit intomainfrom
chore/knip
Feb 18, 2026
Merged

chore: add knip#139
serhalp merged 1 commit intomainfrom
chore/knip

Conversation

@serhalp
Copy link
Owner

@serhalp serhalp commented Feb 18, 2026

Note

Low Risk
CI/tooling-only change that adds a static analysis step and new dev dependency; no runtime or production logic changes.

Overview
Adds knip to the repo and wires it into CI via a new GitHub Actions workflow that runs on pushes to main and all PRs.

Introduces knip.config.ts with custom Nuxt 4 app/ + server/ entry globs and dependency/binary ignore lists, plus a pnpm run knip script and lockfile updates for the new dev dependency.

Written by Cursor Bugbot for commit dec049b. This will update automatically on new commits. Configure here.

@github-actions
Copy link

github-actions bot commented Feb 18, 2026

⚠️ Dependency Count

This PR adds 27 new dependencies (1371 → 1398), which exceeds the threshold of 10.

📊 Dependency Size Changes

📦 Package 📏 Size
@oxc-resolver/binding-linux-x64-gnu@11.17.1 2.5 MB
knip@5.83.1 1.6 MB
smol-toml@1.6.0 103.2 kB
oxc-resolver@11.17.1 58.8 kB
formatly@0.3.0 21.6 kB
strip-json-comments@5.0.3 8.2 kB
fd-package-json@2.0.0 7.7 kB
walk-up-path@4.0.0 6.1 kB

Total size change: 4.3 MB

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

'server/routes/**/*.ts',
'server/middleware/**/*.ts',
'server/plugins/**/*.ts',
'server/**/*.ts',
Copy link

Choose a reason for hiding this comment

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

Redundant server entry patterns subsumed by catch-all

Low Severity

The four specific server globs (server/api/**/*.ts, server/routes/**/*.ts, server/middleware/**/*.ts, server/plugins/**/*.ts) are completely redundant because server/**/*.ts on line 21 already matches every .ts file under server/, including all files those patterns would match. The specific patterns add no filtering value and only create maintenance overhead.

Fix in Cursor Fix in Web

@serhalp serhalp merged commit ae750d7 into main Feb 18, 2026
17 checks passed
@serhalp serhalp deleted the chore/knip branch February 18, 2026 01:31
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