chore(lint): enable structured clone rules · openclaw/openclaw@59694e8

Recent Commits to openclaw:main

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

File tree

- - embedded-agent-runner/run

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,9 @@`
`52`	`52`	`"eslint/unicode-bom": "error",`
`53`	`53`	`"eslint/yoda": "error",`
`54`	`54`	`"import/no-absolute-path": "error",`
	`55`	`+"import/first": "error",`
`55`	`56`	`"import/no-empty-named-blocks": "error",`
	`57`	`+"import/no-duplicates": "error",`
`56`	`58`	`"import/no-self-import": "error",`
`57`	`59`	`"node/no-exports-assign": "error",`
`58`	`60`	`"eslint-plugin-unicorn/prefer-set-size": "error",`
`@@ -81,7 +83,7 @@`
`81`	`83`	`"typescript/no-unnecessary-type-constraint": "error",`
`82`	`84`	`"typescript/no-unnecessary-type-conversion": "error",`
`83`	`85`	`"typescript/no-unnecessary-type-parameters": "error",`
`84`		`-"typescript/no-unsafe-type-assertion": "off",`
	`86`	`+"typescript/no-unsafe-type-assertion": "error",`
`85`	`87`	`"typescript/no-useless-default-assignment": "error",`
`86`	`88`	`"typescript/no-useless-empty-export": "error",`
`87`	`89`	`"typescript/no-wrapper-object-types": "error",`
`@@ -134,6 +136,8 @@`
`134`	`136`	`"unicorn/prefer-prototype-methods": "error",`
`135`	`137`	`"unicorn/prefer-regexp-test": "error",`
`136`	`138`	`"unicorn/prefer-set-size": "error",`
	`139`	`+"unicorn/prefer-set-has": "error",`
	`140`	`+"unicorn/prefer-structured-clone": "error",`
`137`	`141`	`"unicorn/prefer-string-starts-ends-with": "error",`
`138`	`142`	`"unicorn/prefer-string-slice": "error",`
`139`	`143`	`"unicorn/require-array-join-separator": "error",`

Original file line number	Diff line number	Diff line change
`@@ -16,12 +16,12 @@ function inMemoryIO(`
`16`	`16`	`} {`
`17`	`17`	`const store: CodexPluginsConfigBlock = {`
`18`	`18`	`enabled: options.enabled,`
`19`		`-plugins: JSON.parse(JSON.stringify(initial)),`
	`19`	`+plugins: structuredClone(initial),`
`20`	`20`	`};`
`21`	`21`	`return {`
`22`		`-current: () => JSON.parse(JSON.stringify(store.plugins ?? {})),`
`23`		`-currentConfig: () => JSON.parse(JSON.stringify(store)),`
`24`		`-readConfig: () => Promise.resolve(JSON.parse(JSON.stringify(store))),`
	`22`	`+current: () => structuredClone(store.plugins ?? {}),`
	`23`	`+currentConfig: () => structuredClone(store),`
	`24`	`+readConfig: () => Promise.resolve(structuredClone(store)),`
`25`	`25`	`mutate: async (update) => {`
`26`	`26`	`update(store);`
`27`	`27`	`},`

Original file line number	Diff line number	Diff line change
`@@ -112,12 +112,12 @@ function inMemoryCodexPluginsIO(`
`112`	`112`	`} {`
`113`	`113`	`const store: CodexPluginsConfigBlock = {`
`114`	`114`	`enabled: options.enabled,`
`115`		`-plugins: JSON.parse(JSON.stringify(initial)),`
	`115`	`+plugins: structuredClone(initial),`
`116`	`116`	`};`
`117`	`117`	`return {`
`118`		`-current: () => JSON.parse(JSON.stringify(store.plugins ?? {})),`
`119`		`-currentConfig: () => JSON.parse(JSON.stringify(store)),`
`120`		`-readConfig: () => Promise.resolve(JSON.parse(JSON.stringify(store))),`
	`118`	`+current: () => structuredClone(store.plugins ?? {}),`
	`119`	`+currentConfig: () => structuredClone(store),`
	`120`	`+readConfig: () => Promise.resolve(structuredClone(store)),`
`121`	`121`	`mutate: async (update) => {`
`122`	`122`	`update(store);`
`123`	`123`	`},`

Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,11 @@ import {`
`8`	`8`	`} from "./inbound-job.js";`
`9`	`9`	`import { createBaseDiscordMessageContext } from "./message-handler.test-harness.js";`
`10`	`10`
	`11`	`+function jsonRoundTrip<T>(value: T): T {`
	`12`	`+const serialized = JSON.stringify(value);`
	`13`	`+return JSON.parse(serialized) as T;`
	`14`	`+}`
	`15`	`+`
`11`	`16`	`describe("buildDiscordInboundJob", () => {`
`12`	`17`	`it("prefers route session key, then base session key, then channel id for queueing", async () => {`
`13`	`18`	`const routed = await createBaseDiscordMessageContext({`
`@@ -88,7 +93,7 @@ describe("buildDiscordInboundJob", () => {`
`88`	`93`	`},`
`89`	`94`	`ownerId: "user-1",`
`90`	`95`	`});`
`91`		`-const serializedPayload = JSON.parse(JSON.stringify(job.payload));`
	`96`	`+const serializedPayload = jsonRoundTrip(job.payload);`
`92`	`97`	`expect(serializedPayload.threadChannel).toEqual({`
`93`	`98`	`id: "thread-1",`
`94`	`99`	`name: "codex",`
`@@ -125,7 +130,7 @@ describe("buildDiscordInboundJob", () => {`
`125`	`130`	`parent: undefined,`
`126`	`131`	`ownerId: undefined,`
`127`	`132`	`});`
`128`		`-const serializedPayload = JSON.parse(JSON.stringify(job.payload));`
	`133`	`+const serializedPayload = jsonRoundTrip(job.payload);`
`129`	`134`	`expect(serializedPayload.threadChannel).toEqual({`
`130`	`135`	`id: "thread-1",`
`131`	`136`	`});`

Original file line number	Diff line number	Diff line change
`@@ -842,7 +842,8 @@ function resolveGoogleGemini3RetryThinkingLevel(modelId: string): GoogleThinking`
`842`	`842`	`function cloneGoogleGenerateContentRequest(`
`843`	`843`	`params: GoogleGenerateContentRequest,`
`844`	`844`	`): GoogleGenerateContentRequest {`
`845`		`-return JSON.parse(JSON.stringify(params)) as GoogleGenerateContentRequest;`
	`845`	`+const serialized = JSON.stringify(params);`
	`846`	`+return JSON.parse(serialized) as GoogleGenerateContentRequest;`
`846`	`847`	`}`
`847`	`848`
`848`	`849`	`export function buildGoogleGemini3FirstResponseRetryParams(params: {`

Original file line number	Diff line number	Diff line change
`@@ -192,7 +192,8 @@ async function loadBindingsFromPluginState(params: {`
`192`	`192`	`}`
`193`	`193`
`194`	`194`	`function toPluginJsonValue<T>(value: T): T {`
`195`		`-return JSON.parse(JSON.stringify(value)) as T;`
	`195`	`+const serialized = JSON.stringify(value);`
	`196`	`+return JSON.parse(serialized) as T;`
`196`	`197`	`}`
`197`	`198`
`198`	`199`	`async function persistBindingsSnapshot(params: {`

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,8 @@ export function resolveMSTeamsSqliteStateEnv(`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`export function toPluginJsonValue<T>(value: T): T {`
`44`		`-return JSON.parse(JSON.stringify(value)) as T;`
	`44`	`+const serialized = JSON.stringify(value);`
	`45`	`+return JSON.parse(serialized) as T;`
`45`	`46`	`}`
`46`	`47`
`47`	`48`	`export function resolveMSTeamsSqliteStateDir(`

Original file line number	Diff line number	Diff line change
`@@ -358,7 +358,7 @@ function extractLiveDockerPackageScripts(packageJson) {`
`358`	`358`	`}`
`359`	`359`
`360`	`360`	`function stripLiveDockerPackageScripts(packageJson) {`
`361`		`-const clone = JSON.parse(JSON.stringify(packageJson));`
	`361`	`+const clone = structuredClone(packageJson);`
`362`	`362`	`const scripts = clone.scripts;`
`363`	`363`	`if (!scripts \|\| typeof scripts !== "object" \|\| Array.isArray(scripts)) {`
`364`	`364`	`return clone;`
`@@ -377,7 +377,7 @@ function extractPackageScripts(packageJson) {`
`377`	`377`	`}`
`378`	`378`
`379`	`379`	`function stripPackageScripts(packageJson) {`
`380`		`-const clone = JSON.parse(JSON.stringify(packageJson));`
	`380`	`+const clone = structuredClone(packageJson);`
`381`	`381`	`delete clone.scripts;`
`382`	`382`	`return clone;`
`383`	`383`	`}`

Original file line number	Diff line number	Diff line change
`@@ -295,7 +295,8 @@ function toJsonSafe(value: unknown): unknown {`
`295`	`295`	`return null;`
`296`	`296`	`}`
`297`	`297`	`try {`
`298`		`-return JSON.parse(JSON.stringify(value)) as unknown;`
	`298`	`+const serialized = JSON.stringify(value);`
	`299`	`+return serialized === undefined ? null : (JSON.parse(serialized) as unknown);`
`299`	`300`	`} catch {`
`300`	`301`	`if (value instanceof Error) {`
`301`	`302`	`return { name: value.name, message: value.message };`

Original file line number	Diff line number	Diff line change
`@@ -271,7 +271,8 @@ function toJsonSafe(value: unknown): unknown {`
`271`	`271`	`return null;`
`272`	`272`	`}`
`273`	`273`	`try {`
`274`		`-return JSON.parse(JSON.stringify(value)) as unknown;`
	`274`	`+const serialized = JSON.stringify(value);`
	`275`	`+return serialized === undefined ? null : (JSON.parse(serialized) as unknown);`
`275`	`276`	`} catch {`
`276`	`277`	`if (value instanceof Error) {`
`277`	`278`	`return { name: value.name, message: value.message };`

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

推荐订阅源

Recent Commits to openclaw:main

File tree