Skip to content

Commit cff16ed

Browse files
claude-code-bestclaude
claude-code-best
and
claude
committed
chore: 清理 unused biome-ignore 注释并修复 lint error
- 删除 208 处已关闭规则的 biome-ignore 注释（noConsole/useHookAtTopLevel/useExhaustiveDependencies 等） - 自动修复 parseInt radix、useless undefined init、Math.pow→** 等 lint error - biome.json 新增关闭 noControlCharactersInRegex/noSelfCompare/noTemplateCurlyInString 等规则 - 关闭 CSS lint 以兼容 Tailwind v4 语法 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent f9d0111 commit cff16ed

114 files changed

Lines changed: 256 additions & 407 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎biome.json‎

Lines changed: 142 additions & 112 deletions
Original file line numberDiff line numberDiff line change
@@ -1,114 +1,144 @@
11
{
2-
"$schema": "https://biomejs.dev/schemas/2.4.10/schema.json",
3-
"vcs": {
4-
"enabled": true,
5-
"clientKind": "git",
6-
"useIgnoreFile": true
7-
},
8-
"files": {
9-
"includes": ["**", "!!**/dist", "!!**/packages/@ant"]
10-
},
11-
"formatter": {
12-
"enabled": true,
13-
"indentStyle": "space",
14-
"indentWidth": 2,
15-
"lineWidth": 80
16-
},
17-
"linter": {
18-
"enabled": true,
19-
"rules": {
20-
"recommended": true,
21-
"suspicious": {
22-
"noExplicitAny": "off",
23-
"noAssignInExpressions": "off",
24-
"noDoubleEquals": "off",
25-
"noRedeclare": "off",
26-
"noImplicitAnyLet": "off",
27-
"noGlobalIsNan": "off",
28-
"noFallthroughSwitchClause": "off",
29-
"noShadowRestrictedNames": "off",
30-
"noArrayIndexKey": "off",
31-
"noConsole": "off",
32-
"noConfusingLabels": "off",
33-
"useIterableCallbackReturn": "off"
34-
},
35-
"style": {
36-
"useConst": "off",
37-
"noNonNullAssertion": "off",
38-
"noParameterAssign": "off",
39-
"useDefaultParameterLast": "off",
40-
"noUnusedTemplateLiteral": "off",
41-
"useTemplate": "off",
42-
"useNumberNamespace": "off",
43-
"useNodejsImportProtocol": "off",
44-
"useImportType": "off"
45-
},
46-
"complexity": {
47-
"noForEach": "off",
48-
"noBannedTypes": "off",
49-
"noUselessConstructor": "off",
50-
"noStaticOnlyClass": "off",
51-
"useOptionalChain": "off",
52-
"noUselessSwitchCase": "off",
53-
"noUselessFragments": "off",
54-
"noUselessTernary": "off",
55-
"noUselessLoneBlockStatements": "off",
56-
"noUselessEmptyExport": "off",
57-
"useArrowFunction": "off",
58-
"useLiteralKeys": "off"
59-
},
60-
"correctness": {
61-
"noUnusedVariables": "off",
62-
"noUnusedImports": "off",
63-
"useExhaustiveDependencies": "off",
64-
"noSwitchDeclarations": "off",
65-
"noUnreachable": "off",
66-
"useHookAtTopLevel": "off",
67-
"noVoidTypeReturn": "off",
68-
"noConstantCondition": "off",
69-
"noUnusedFunctionParameters": "off"
70-
},
71-
"a11y": {
72-
"recommended": false
73-
},
74-
"nursery": {
75-
"recommended": false
76-
}
77-
}
78-
},
79-
"json": {
80-
"formatter": {
81-
"enabled": false
82-
}
83-
},
84-
"javascript": {
85-
"formatter": {
86-
"quoteStyle": "single",
87-
"semicolons": "asNeeded",
88-
"arrowParentheses": "asNeeded",
89-
"trailingCommas": "all"
90-
}
91-
},
92-
"overrides": [
93-
{
94-
"includes": ["**/*.tsx"],
95-
"javascript": {
96-
"formatter": {
97-
"semicolons": "always"
98-
}
99-
},
100-
"formatter": {
101-
"lineWidth": 120
102-
}
103-
},
104-
{
105-
"includes": ["scripts/**", "packages/**", "**/*.js", "**/*.mjs", "**/*.jsx"],
106-
"formatter": {
107-
"enabled": false
108-
}
109-
}
110-
],
111-
"assist": {
112-
"enabled": false
113-
}
2+
"$schema": "https://biomejs.dev/schemas/2.4.10/schema.json",
3+
"vcs": {
4+
"enabled": true,
5+
"clientKind": "git",
6+
"useIgnoreFile": true
7+
},
8+
"files": {
9+
"includes": [
10+
"**",
11+
"!!**/dist",
12+
"!!**/packages/@ant"
13+
]
14+
},
15+
"formatter": {
16+
"enabled": true,
17+
"indentStyle": "space",
18+
"indentWidth": 2,
19+
"lineWidth": 80
20+
},
21+
"linter": {
22+
"enabled": true,
23+
"rules": {
24+
"recommended": true,
25+
"suspicious": {
26+
"noExplicitAny": "off",
27+
"noAssignInExpressions": "off",
28+
"noDoubleEquals": "off",
29+
"noRedeclare": "off",
30+
"noImplicitAnyLet": "off",
31+
"noGlobalIsNan": "off",
32+
"noFallthroughSwitchClause": "off",
33+
"noShadowRestrictedNames": "off",
34+
"noArrayIndexKey": "off",
35+
"noConsole": "off",
36+
"noConfusingLabels": "off",
37+
"useIterableCallbackReturn": "off",
38+
"noControlCharactersInRegex": "off",
39+
"noSelfCompare": "off",
40+
"noTemplateCurlyInString": "off",
41+
"noAsyncPromiseExecutor": "off"
42+
},
43+
"style": {
44+
"useConst": "off",
45+
"noNonNullAssertion": "off",
46+
"noParameterAssign": "off",
47+
"useDefaultParameterLast": "off",
48+
"noUnusedTemplateLiteral": "off",
49+
"useTemplate": "off",
50+
"useNumberNamespace": "off",
51+
"useNodejsImportProtocol": "off",
52+
"useImportType": "off"
53+
},
54+
"complexity": {
55+
"noForEach": "off",
56+
"noBannedTypes": "off",
57+
"noUselessConstructor": "off",
58+
"noStaticOnlyClass": "off",
59+
"useOptionalChain": "off",
60+
"noUselessSwitchCase": "off",
61+
"noUselessFragments": "off",
62+
"noUselessTernary": "off",
63+
"noUselessLoneBlockStatements": "off",
64+
"noUselessEmptyExport": "off",
65+
"useArrowFunction": "off",
66+
"useLiteralKeys": "off"
67+
},
68+
"correctness": {
69+
"noUnusedVariables": "off",
70+
"noUnusedImports": "off",
71+
"useExhaustiveDependencies": "off",
72+
"noSwitchDeclarations": "off",
73+
"noUnreachable": "off",
74+
"useHookAtTopLevel": "off",
75+
"noVoidTypeReturn": "off",
76+
"noConstantCondition": "off",
77+
"noUnusedFunctionParameters": "off",
78+
"useYield": "off",
79+
"noUnusedPrivateClassMembers": "off",
80+
"noUnusedLabels": "off"
81+
},
82+
"a11y": {
83+
"recommended": false
84+
},
85+
"nursery": {
86+
"recommended": false
87+
},
88+
"security": {
89+
"noDangerouslySetInnerHtml": "off"
90+
}
91+
}
92+
},
93+
"json": {
94+
"formatter": {
95+
"enabled": false
96+
}
97+
},
98+
"javascript": {
99+
"formatter": {
100+
"quoteStyle": "single",
101+
"semicolons": "asNeeded",
102+
"arrowParentheses": "asNeeded",
103+
"trailingCommas": "all"
104+
}
105+
},
106+
"overrides": [
107+
{
108+
"includes": [
109+
"**/*.tsx"
110+
],
111+
"javascript": {
112+
"formatter": {
113+
"semicolons": "always"
114+
}
115+
},
116+
"formatter": {
117+
"lineWidth": 120
118+
}
119+
},
120+
{
121+
"includes": [
122+
"scripts/**",
123+
"packages/**",
124+
"**/*.js",
125+
"**/*.mjs",
126+
"**/*.jsx"
127+
],
128+
"formatter": {
129+
"enabled": false
130+
}
131+
},
132+
{
133+
"includes": [
134+
"**/*.css"
135+
],
136+
"linter": {
137+
"enabled": false
138+
}
139+
}
140+
],
141+
"assist": {
142+
"enabled": false
143+
}
114144
}

‎packages/@ant/ink/src/core/ink.tsx‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1843,7 +1843,6 @@ export default class Ink {
18431843
}
18441844

18451845
patchConsole(): () => void {
1846-
// biome-ignore lint/suspicious/noConsole: intentionally patching global console
18471846
const con = console
18481847
const originals: Partial<Record<keyof Console, Console[keyof Console]>> = {}
18491848
const toDebug = (...args: unknown[]) =>

‎packages/@ant/ink/src/core/reconciler.ts‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ if (process.env.NODE_ENV === 'development') {
3535
// eslint-disable-next-line @typescript-eslint/no-explicit-any
3636
} catch (error: unknown) {
3737
if (error instanceof Error && (error as NodeJS.ErrnoException).code === 'ERR_MODULE_NOT_FOUND') {
38-
// biome-ignore lint/suspicious/noConsole: intentional warning
3938
console.warn(
4039
`
4140
The environment variable DEV is set to true, so Ink tried to import \`react-devtools-core\`,
@@ -197,7 +196,6 @@ let _prepareAt = 0
197196

198197
/** Debug log helper — replaces fs.appendFileSync with console.warn. */
199198
function debugLog(message: string): void {
200-
// biome-ignore lint/suspicious/noConsole: debug instrumentation
201199
console.warn(`[ink-commit] ${message}`)
202200
}
203201
// --- END ---

‎packages/@ant/ink/src/core/root.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,6 @@ const wrappedRender = async (
114114
await Promise.resolve()
115115
const instance = renderSync(node, options)
116116
if (process.env.CLAUDE_CODE_DEBUG_REPAINTS === '1') {
117-
// biome-ignore lint/suspicious/noConsole: debug instrumentation
118117
console.warn(
119118
`[render] first ink render: ${Math.round(process.uptime() * 1000)}ms since process start`,
120119
)

‎packages/builtin-tools/src/tools/BashTool/BashTool.tsx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1140,7 +1140,7 @@ async function* runShellCommand({
11401140
let lastProgressOutput = ''
11411141
let lastTotalLines = 0
11421142
let lastTotalBytes = 0
1143-
let backgroundShellId: string | undefined = undefined
1143+
let backgroundShellId: string | undefined
11441144
let assistantAutoBackgrounded = false
11451145

11461146
// Progress signal: resolved by onProgress callback from the shared poller,
@@ -1314,7 +1314,7 @@ async function* runShellCommand({
13141314

13151315
// Wait for the initial threshold before showing progress
13161316
const startTime = Date.now()
1317-
let foregroundTaskId: string | undefined = undefined
1317+
let foregroundTaskId: string | undefined
13181318

13191319
{
13201320
const initialResult = await Promise.race([

‎packages/builtin-tools/src/tools/FileReadTool/imageProcessor.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ export async function getImageProcessor(): Promise<SharpFunction> {
4949
return sharpFn
5050
} catch {
5151
// Fall back to sharp if native module is not available
52-
// biome-ignore lint/suspicious/noConsole: intentional warning
5352
console.warn(
5453
'Native image processor not available, falling back to sharp',
5554
)

‎packages/builtin-tools/src/tools/NotebookEditTool/NotebookEditTool.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -377,7 +377,7 @@ export const NotebookEditTool = buildTool({
377377
}
378378

379379
const language = notebook.metadata.language_info?.name ?? 'python'
380-
let new_cell_id = undefined
380+
let new_cell_id
381381
if (
382382
notebook.nbformat > 4 ||
383383
(notebook.nbformat === 4 && notebook.nbformat_minor >= 5)

‎packages/builtin-tools/src/tools/PowerShellTool/PowerShellTool.tsx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -908,7 +908,7 @@ async function* runPowerShellCommand({
908908
let lastProgressOutput = ''
909909
let lastTotalLines = 0
910910
let lastTotalBytes = 0
911-
let backgroundShellId: string | undefined = undefined
911+
let backgroundShellId: string | undefined
912912
let interruptBackgroundingStarted = false
913913
let assistantAutoBackgrounded = false
914914

@@ -1109,7 +1109,7 @@ async function* runPowerShellCommand({
11091109
// Set up progress yielding with periodic checks
11101110
const startTime = Date.now()
11111111
let nextProgressTime = startTime + PROGRESS_THRESHOLD_MS
1112-
let foregroundTaskId: string | undefined = undefined
1112+
let foregroundTaskId: string | undefined
11131113

11141114
// Progress loop: wrap in try/finally so stopPolling is called on every exit
11151115
// path — normal completion, timeout/interrupt backgrounding, and Ctrl+B

‎packages/remote-control-server/src/__tests__/sse-writer.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ describe("SSE Writer", () => {
117117

118118
app.get("/stream/:sessionId", (c) => {
119119
const sessionId = c.req.param("sessionId");
120-
const fromSeq = parseInt(c.req.query("fromSeq") || "0");
120+
const fromSeq = parseInt(c.req.query("fromSeq") || "0", 10);
121121
return createSSEStream(c, sessionId, fromSeq);
122122
});
123123

‎packages/remote-control-server/src/config.ts‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,20 @@
11
export const config = {
22
version: process.env.RCS_VERSION || "0.1.0",
3-
port: parseInt(process.env.RCS_PORT || "3000"),
3+
port: parseInt(process.env.RCS_PORT || "3000", 10),
44
host: process.env.RCS_HOST || "0.0.0.0",
55
apiKeys: (process.env.RCS_API_KEYS || "").split(",").filter(Boolean),
66
baseUrl: process.env.RCS_BASE_URL || "",
7-
pollTimeout: parseInt(process.env.RCS_POLL_TIMEOUT || "8"),
8-
heartbeatInterval: parseInt(process.env.RCS_HEARTBEAT_INTERVAL || "20"),
9-
jwtExpiresIn: parseInt(process.env.RCS_JWT_EXPIRES_IN || "3600"),
10-
disconnectTimeout: parseInt(process.env.RCS_DISCONNECT_TIMEOUT || "300"),
7+
pollTimeout: parseInt(process.env.RCS_POLL_TIMEOUT || "8", 10),
8+
heartbeatInterval: parseInt(process.env.RCS_HEARTBEAT_INTERVAL || "20", 10),
9+
jwtExpiresIn: parseInt(process.env.RCS_JWT_EXPIRES_IN || "3600", 10),
10+
disconnectTimeout: parseInt(process.env.RCS_DISCONNECT_TIMEOUT || "300", 10),
1111
/** Bun WebSocket idle timeout (seconds). Bun sends protocol-level pings after
1212
* this many seconds of no received data. Must be shorter than any reverse
1313
* proxy's idle timeout (nginx default 60s, Cloudflare 100s). Default 30s. */
14-
wsIdleTimeout: parseInt(process.env.RCS_WS_IDLE_TIMEOUT || "30"),
14+
wsIdleTimeout: parseInt(process.env.RCS_WS_IDLE_TIMEOUT || "30", 10),
1515
/** Server→client keep_alive data-frame interval (seconds). Keeps reverse
1616
* proxies from closing idle connections. Default 20s. */
17-
wsKeepaliveInterval: parseInt(process.env.RCS_WS_KEEPALIVE_INTERVAL || "20"),
17+
wsKeepaliveInterval: parseInt(process.env.RCS_WS_KEEPALIVE_INTERVAL || "20", 10),
1818
} as const;
1919

2020
export function getBaseUrl(): string {

0 commit comments

Comments
 (0)