Skip to content

Commit 8ea0779

Browse files
deps(knip): configuring knip and removing dead code (#225)
Co-authored-by: Elliott de Launay <edelauna@gmail.com>
1 parent 43935ed commit 8ea0779

7 files changed

Lines changed: 83 additions & 203 deletions

File tree

apps/cli/package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,11 @@
4343
"@roo-code/config-typescript": "workspace:^",
4444
"@types/node": "24.2.1",
4545
"@types/react": "18.3.23",
46+
"@vitest/coverage-v8": "3.2.6",
4647
"ink-testing-library": "4.0.0",
4748
"rimraf": "6.0.1",
4849
"tsup": "8.5.0",
49-
"@vitest/coverage-v8": "3.2.6",
50+
"tsx": "4.19.4",
5051
"vitest": "3.2.6"
5152
}
5253
}

knip.json

Lines changed: 72 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,83 @@
11
{
2-
"$schema": "https://unpkg.com/knip@latest/schema.json",
3-
"ignore": [
4-
"**/__tests__/**",
5-
"apps/vscode-e2e/**",
6-
"src/extension/api.ts",
7-
"src/activate/**",
8-
"src/workers/countTokens.ts",
9-
"src/extension.ts",
10-
"scripts/**",
11-
"apps/cli/scripts/**"
12-
],
2+
"$schema": "https://unpkg.com/knip@5/schema.json",
3+
"ignore": ["**/__tests__/**", "apps/vscode-e2e/**", "scripts/**", "apps/cli/scripts/**"],
4+
"ignoreDependencies": ["lint-staged", "only-allow", "ovsx"],
5+
"ignoreExportsUsedInFile": true,
136
"workspaces": {
147
"src": {
15-
"entry": ["extension.ts"],
16-
"project": ["**/*.ts"]
8+
"entry": ["extension.ts", "extension/api.ts", "workers/countTokens.ts"],
9+
"project": ["**/*.ts", "!**/__tests__/**"],
10+
"ignoreDependencies": [
11+
"@ai-sdk/amazon-bedrock",
12+
"@ai-sdk/baseten",
13+
"@ai-sdk/deepseek",
14+
"@ai-sdk/fireworks",
15+
"@ai-sdk/google",
16+
"@ai-sdk/google-vertex",
17+
"@ai-sdk/mistral",
18+
"@ai-sdk/xai",
19+
"@roo-code/config-typescript",
20+
"@types/mocha",
21+
"@types/node-cache",
22+
"@types/vscode",
23+
"@vscode/codicons",
24+
"esbuild-wasm",
25+
"sambanova-ai-provider",
26+
"tree-sitter-wasms",
27+
"vscode-material-icons",
28+
"zhipu-ai-provider"
29+
]
1730
},
1831
"webview-ui": {
1932
"entry": ["src/index.tsx"],
20-
"project": ["src/**/*.{ts,tsx}", "../src/shared/*.ts"]
33+
"project": ["src/**/*.{ts,tsx}", "../src/shared/*.ts"],
34+
"ignoreDependencies": [
35+
"@roo-code/config-typescript",
36+
"@types/katex",
37+
"@types/stacktrace-js",
38+
"babel-plugin-react-compiler",
39+
"katex",
40+
"react-compiler-runtime",
41+
"rehype-highlight",
42+
"source-map",
43+
"tailwindcss",
44+
"tailwindcss-animate",
45+
"vscode"
46+
]
2147
},
22-
"packages/{build,cloud,ipc,telemetry,types}": {
48+
"apps/cli": {
49+
"project": ["src/**/*.ts", "src/**/*.tsx"],
50+
"ignoreDependencies": ["@vscode/ripgrep", "cross-spawn"]
51+
},
52+
"packages/config-typescript": {
53+
"ignoreUnresolved": ["vitest/globals"]
54+
},
55+
"packages/core": {
56+
"project": ["src/**/*.ts"],
57+
"ignoreDependencies": ["esbuild"]
58+
},
59+
"packages/vscode-shim": {
60+
"project": ["src/**/*.ts"]
61+
},
62+
"packages/cloud": {
63+
"project": ["src/**/*.ts"],
64+
"ignoreDependencies": ["@types/vscode", "ioredis", "p-wait-for"]
65+
},
66+
"packages/telemetry": {
67+
"project": ["src/**/*.ts"],
68+
"ignoreDependencies": ["@types/vscode"]
69+
},
70+
"packages/{build,ipc,types}": {
2371
"project": ["src/**/*.ts"]
2472
}
73+
},
74+
"rules": {
75+
"classMembers": "off",
76+
"duplicates": "warn",
77+
"enumMembers": "warn",
78+
"exports": "warn",
79+
"nsExports": "warn",
80+
"types": "warn",
81+
"nsTypes": "warn"
2582
}
2683
}

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
"install:vsix:nightly": "pnpm install --frozen-lockfile && pnpm clean && pnpm vsix:nightly && node scripts/install-vsix.js --nightly",
2525
"code-server:install": "node scripts/code-server.js",
2626
"changeset:version": "cp CHANGELOG.md src/CHANGELOG.md && changeset version && cp -vf src/CHANGELOG.md .",
27-
"knip": "knip --include files",
27+
"knip": "knip",
2828
"npm:publish:types": "pnpm --filter @roo-code/types npm:publish"
2929
},
3030
"devDependencies": {

0 commit comments

Comments
 (0)