-
Notifications
You must be signed in to change notification settings - Fork 3
Setup knip #237
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Setup knip #237
Conversation
2276bee to
a11cfda
Compare
🚀 Package Preview Available!Install this PR's preview build with npm: npm i @base44-preview/cli@0.0.31-pr.237.6328bcfPrefer not to change any import paths? Install using npm alias so your code still imports npm i "base44@npm:@base44-preview/cli@0.0.31-pr.237.6328bcf"Or add it to your {
"dependencies": {
"base44": "npm:@base44-preview/cli@0.0.31-pr.237.6328bcf"
}
}
Preview published to npm registry — try new features instantly! |
|
README check ran. 6 issue(s) found and applied: (1) Added missing |
Note
Description
This PR integrates Knip, a tool for detecting unused code and dependencies, into the Base44 CLI project. It includes configuration for Knip, CI/CD integration, and cleanup of unused exports and dependencies identified by the tool.
Related Issue
None
Type of Change
Changes Made
knip.json).github/workflows/knip.yml) to run Knip on PRs and main branch pushes.claude/settings.json) with hooks for typecheck, lint, and knip after edits@types/tardependency (unused type definitions)mswfrom v2.12.7 to v2.12.10.worktreesto.gitignoreRunCommandOptions,CLIError,UserError,SystemError)Testing
npm test)Checklist
Additional Notes
This refactoring improves code maintainability by identifying and removing unused code/dependencies. The Knip CI workflow will help prevent accumulation of dead code in the future. Changes are purely internal cleanup and do not affect the CLI's behavior or public API.
🤖 Generated by Claude | 2026-02-12 21:15 UTC