Skip to content

Conversation

@Boshen
Copy link
Member

@Boshen Boshen commented Nov 17, 2025

Summary

  • Install knip as dev dependency
  • Add knip configuration with VitePress support
  • Add pnpm knip script to package.json

Configuration

The knip configuration includes:

  • Entry points for VitePress config and theme
  • Support for Vue components and markdown files
  • Proper handling of dynamically loaded files (Hero.vue via Vite alias)
  • Ignores for internal exports used in markdown files that knip can't parse

Test plan

  • Run pnpm knip - exits cleanly with no false positives
  • Verified Hero.vue is no longer flagged as unused (loaded via Vite alias)
  • Verified team.ts exports are properly ignored (used in markdown files)

🤖 Generated with Claude Code

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@netlify
Copy link

netlify bot commented Nov 17, 2025

Deploy Preview for oxc-project ready!

Name Link
🔨 Latest commit 87d77e7
🔍 Latest deploy log https://app.netlify.com/projects/oxc-project/deploys/691ad6a275682a0008989a71
😎 Deploy Preview https://deploy-preview-638--oxc-project.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Boshen Boshen merged commit 1e5227e into main Nov 17, 2025
9 checks passed
@Boshen Boshen deleted the add-knip branch November 17, 2025 08:27
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