Skip to content

Commit 51c36bc

Browse files
justin808claude
andcommitted
Add knip configuration for node-renderer package
The knip linter was reporting react-on-rails-pro as an unused dependency in the node-renderer package. While it's not directly imported in source files, it's a runtime dependency needed for the package to function. Changes: - Added packages/react-on-rails-pro-node-renderer workspace to knip.ts - Configured entry points (index.ts, server.ts) - Added ignoreDependencies for: * react-on-rails-pro (runtime dependency) * @honeybadger-io/js (dev/test dependency) - Ignored lib/** (build output) and tests/fixtures/** This resolves the "Unused dependencies" error in CI linting step. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent a7043fa commit 51c36bc

File tree

1 file changed

+19
-0
lines changed

1 file changed

+19
-0
lines changed

knip.ts

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,25 @@ const config: KnipConfig = {
6565
],
6666
},
6767

68+
// React on Rails Pro Node Renderer package workspace
69+
'packages/react-on-rails-pro-node-renderer': {
70+
entry: ['src/index.ts!', 'src/server.ts!'],
71+
project: ['src/**/*.[jt]s{x,}!', 'tests/**/*.[jt]s{x,}', '!lib/**'],
72+
ignore: [
73+
// Build output directories that should be ignored
74+
'lib/**',
75+
// Test fixtures
76+
'tests/fixtures/**',
77+
],
78+
ignoreDependencies: [
79+
// These are runtime dependencies needed for the package to work
80+
// even though they may not be directly imported in the source
81+
'react-on-rails-pro',
82+
// Dev dependencies used only in tests
83+
'@honeybadger-io/js',
84+
],
85+
},
86+
6887
// React on Rails Pro package workspace
6988
'packages/react-on-rails-pro': {
7089
entry: [

0 commit comments

Comments
 (0)