chore(lint): enable readability lint rules · openclaw/ope...

Recent Commits to openclaw:main

steipete · 2026-05-31 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,7 @@`
`26`	`26`	`"eslint/no-sequences": "error",`
`27`	`27`	`"eslint/no-self-compare": "error",`
`28`	`28`	`"eslint/no-shadow": "off",`
	`29`	`+"eslint/no-implicit-coercion": "error",`
`29`	`30`	`"eslint/no-var": "error",`
`30`	`31`	`"eslint/no-useless-call": "error",`
`31`	`32`	`"eslint/no-useless-computed-key": "error",`
`@@ -41,6 +42,7 @@`
`41`	`42`	`"eslint/default-param-last": "error",`
`42`	`43`	`"eslint/prefer-exponentiation-operator": "error",`
`43`	`44`	`"eslint/prefer-numeric-literals": "error",`
	`45`	`+"eslint/prefer-object-has-own": "error",`
`44`	`46`	`"eslint/prefer-rest-params": "error",`
`45`	`47`	`"eslint/prefer-spread": "error",`
`46`	`48`	`"eslint/radix": "error",`
`@@ -66,6 +68,7 @@`
`66`	`68`	`"typescript/no-empty-object-type": ["error", { "allowInterfaces": "with-single-extends" }],`
`67`	`69`	`"typescript/no-explicit-any": "error",`
`68`	`70`	`"typescript/no-extraneous-class": "error",`
	`71`	`+"typescript/no-import-type-side-effects": "error",`
`69`	`72`	`"typescript/no-meaningless-void-operator": "error",`
`70`	`73`	`"typescript/no-non-null-asserted-nullish-coalescing": "error",`
`71`	`74`	`"typescript/no-unnecessary-qualifier": "error",`
`@@ -111,6 +114,7 @@`
`111`	`114`	`"unicorn/no-unnecessary-slice-end": "error",`
`112`	`115`	`"unicorn/no-useless-error-capture-stack-trace": "error",`
`113`	`116`	`"unicorn/no-useless-promise-resolve-reject": "error",`
	`117`	`+"unicorn/no-zero-fractions": "error",`
`114`	`118`	`"unicorn/prefer-date-now": "error",`
`115`	`119`	`"unicorn/prefer-dom-node-text-content": "error",`
`116`	`120`	`"unicorn/prefer-keyboard-event-key": "error",`
`@@ -183,6 +187,7 @@`
`183`	`187`	`"docs/_layouts/",`
`184`	`188`	`"extensions/diffs/assets/viewer-runtime.js",`
`185`	`189`	`"extensions/diffs-language-pack/assets/viewer-runtime.js",`
	`190`	`+"extensions/canvas/src/host/a2ui/a2ui.bundle.js",`
`186`	`191`	`"node_modules/",`
`187`	`192`	`"patches/",`
`188`	`193`	`"pnpm-lock.yaml",`

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ async function readLeaseFile(filePath: string): Promise<LeaseFile> {`
`75`	`75`	`leases: [],`
`76`	`76`	`});`
`77`	`77`	`const leases = Array.isArray(value.leases)`
`78`		`- ? value.leases.map(normalizeLease).filter((lease): lease is AcpxProcessLease => !!lease)`
	`78`	`+ ? value.leases.map(normalizeLease).filter((lease): lease is AcpxProcessLease => Boolean(lease))`
`79`	`79`	`: [];`
`80`	`80`	`return { version: 1, leases };`
`81`	`81`	`}`

Original file line number	Diff line number	Diff line change
`@@ -127,9 +127,7 @@ function readRpcRequestBody(body: unknown):`
`127`	`127`	`request: {`
`128`	`128`	`id,`
`129`	`129`	`method: rpcBody.method.trim(),`
`130`		`- ...(Object.prototype.hasOwnProperty.call(rpcBody, "params")`
`131`		`- ? { params: rpcBody.params }`
`132`		`- : {}),`
	`130`	`+ ...(Object.hasOwn(rpcBody, "params") ? { params: rpcBody.params } : {}),`
`133`	`131`	`},`
`134`	`132`	`};`
`135`	`133`	`}`

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`		`-import {`
`2`		`-type BedrockClient,`
`3`		`-type ListFoundationModelsCommandOutput,`
`4`		`-type ListInferenceProfilesCommandOutput,`
	`1`	`+import type {`
	`2`	`+BedrockClient,`
	`3`	`+ListFoundationModelsCommandOutput,`
	`4`	`+ListInferenceProfilesCommandOutput,`
`5`	`5`	`} from "@aws-sdk/client-bedrock";`
`6`	`6`	`import { createSubsystemLogger } from "openclaw/plugin-sdk/core";`
`7`	`7`	`import { formatErrorMessage } from "openclaw/plugin-sdk/error-runtime";`

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ export const ARCEE_MODEL_CATALOG: ModelDefinitionConfig[] = [`
`26`	`26`	`maxTokens: 16384,`
`27`	`27`	`cost: {`
`28`	`28`	`input: 0.25,`
`29`		`-output: 1.0,`
	`29`	`+output: 1,`
`30`	`30`	`cacheRead: 0.25,`
`31`	`31`	`cacheWrite: 0.25,`
`32`	`32`	`},`

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import { type WebSearchProviderPlugin } from "openclaw/plugin-sdk/provider-web-search-config-contract";`
	`1`	`+import type { WebSearchProviderPlugin } from "openclaw/plugin-sdk/provider-web-search-config-contract";`
`2`	`2`	`import { buildBraveWebSearchProviderBase } from "./web-search-shared.js";`
`3`	`3`
`4`	`4`	`export function createBraveWebSearchProvider(): WebSearchProviderPlugin {`

Original file line number	Diff line number	Diff line change
`@@ -38,13 +38,10 @@ export default definePluginEntry({`
`38`	`38`	`) {`
`39`	`39`	`return null;`
`40`	`40`	`}`
`41`		`-if (Object.prototype.hasOwnProperty.call(config, "browser")) {`
	`41`	`+if (Object.hasOwn(config, "browser")) {`
`42`	`42`	`return "browser configured";`
`43`	`43`	`}`
`44`		`-if (`
`45`		`-config.plugins?.entries &&`
`46`		`-Object.prototype.hasOwnProperty.call(config.plugins.entries, "browser")`
`47`		`-) {`
	`44`	`+if (config.plugins?.entries && Object.hasOwn(config.plugins.entries, "browser")) {`
`48`	`45`	`return "browser plugin configured";`
`49`	`46`	`}`
`50`	`47`	`if (hasBrowserToolReference(config)) {`

Original file line number	Diff line number	Diff line change
`@@ -5,11 +5,7 @@ import {`
`5`	`5`	`normalizeOptionalString,`
`6`	`6`	`normalizeOptionalTrimmedStringList,`
`7`	`7`	`} from "openclaw/plugin-sdk/string-coerce-runtime";`
`8`		`-import {`
`9`		`-type BrowserConfig,`
`10`		`-type BrowserProfileConfig,`
`11`		`-type OpenClawConfig,`
`12`		`-} from "../config/config.js";`
	`8`	`+import type { BrowserConfig, BrowserProfileConfig, OpenClawConfig } from "../config/config.js";`
`13`	`9`	`import { resolveGatewayPort } from "../config/paths.js";`
`14`	`10`	`import {`
`15`	`11`	`DEFAULT_BROWSER_CONTROL_PORT,`

Original file line number	Diff line number	Diff line change
`@@ -269,7 +269,7 @@ async function runBrowserLiveProbe(req: BrowserRequest, ctx: BrowserRouteContext`
`269`	`269`	`}`
`270`	`270`
`271`	`271`	`function hasQueryKey(query: BrowserRequest["query"], key: string): boolean {`
`272`		`-return Object.prototype.hasOwnProperty.call(query ?? {}, key);`
	`272`	`+return Object.hasOwn(query ?? {}, key);`
`273`	`273`	`}`
`274`	`274`
`275`	`275`	`function parseHeadlessStartOverride(params: {`

Original file line number	Diff line number	Diff line change
`@@ -133,7 +133,7 @@ function isLegacyClawdProfileConfigured(cfg: OpenClawConfig, legacyProfileDir: s`
`133`	`133`	`if (!configuredProfiles) {`
`134`	`134`	`return false;`
`135`	`135`	`}`
`136`		`-if (Object.prototype.hasOwnProperty.call(configuredProfiles, LEGACY_CLAWD_BROWSER_PROFILE_NAME)) {`
	`136`	`+if (Object.hasOwn(configuredProfiles, LEGACY_CLAWD_BROWSER_PROFILE_NAME)) {`
`137`	`137`	`return true;`
`138`	`138`	`}`
`139`	`139`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main