refactor(config): hide default constants · openclaw/openc...

Recent Commits to openclaw:main

vincentkoc · 2026-06-17 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -1,23 +1,24 @@`
`1`	`1`	`// Verifies default port config values and overrides.`
`2`	`2`	`import { describe, expect, it } from "vitest";`
`3`		`-import {`
`4`		`-DEFAULT_BROWSER_CDP_PORT_RANGE_END,`
`5`		`-DEFAULT_BROWSER_CDP_PORT_RANGE_START,`
`6`		`-deriveDefaultBrowserCdpPortRange,`
`7`		`-} from "./port-defaults.js";`
	`3`	`+import { deriveDefaultBrowserCdpPortRange } from "./port-defaults.js";`
	`4`	`+`
	`5`	`+const EXPECTED_BROWSER_CDP_PORT_RANGE = {`
	`6`	`+start: 18800,`
	`7`	`+end: 18899,`
	`8`	`+};`
`8`	`9`
`9`	`10`	`describe("port defaults", () => {`
`10`	`11`	`it("derives the browser CDP range from the control port", () => {`
`11`	`12`	`expect(deriveDefaultBrowserCdpPortRange(18791)).toEqual({`
`12`		`-start: DEFAULT_BROWSER_CDP_PORT_RANGE_START,`
`13`		`-end: DEFAULT_BROWSER_CDP_PORT_RANGE_END,`
	`13`	`+start: EXPECTED_BROWSER_CDP_PORT_RANGE.start,`
	`14`	`+end: EXPECTED_BROWSER_CDP_PORT_RANGE.end,`
`14`	`15`	`});`
`15`	`16`	`});`
`16`	`17`
`17`	`18`	`it("keeps the default browser CDP range wide when derived ports would overflow", () => {`
`18`	`19`	`expect(deriveDefaultBrowserCdpPortRange(65440)).toEqual({`
`19`		`-start: DEFAULT_BROWSER_CDP_PORT_RANGE_START,`
`20`		`-end: DEFAULT_BROWSER_CDP_PORT_RANGE_END,`
	`20`	`+start: EXPECTED_BROWSER_CDP_PORT_RANGE.start,`
	`21`	`+end: EXPECTED_BROWSER_CDP_PORT_RANGE.end,`
`21`	`22`	`});`
`22`	`23`	`});`
`23`	`24`	`});`

Original file line number	Diff line number	Diff line change
`@@ -14,9 +14,9 @@ function derivePort(base: number, offset: number, fallback: number): number {`
`14`	`14`	`}`
`15`	`15`
`16`	`16`	`/** Default browser-CDP sidecar port range used when no browser-control-relative range is safe. */`
`17`		`-export const DEFAULT_BROWSER_CDP_PORT_RANGE_START = 18800;`
	`17`	`+const DEFAULT_BROWSER_CDP_PORT_RANGE_START = 18800;`
`18`	`18`	`/** Inclusive end of the default browser-CDP sidecar port range. */`
`19`		`-export const DEFAULT_BROWSER_CDP_PORT_RANGE_END = 18899;`
	`19`	`+const DEFAULT_BROWSER_CDP_PORT_RANGE_END = 18899;`
`20`	`20`	`const DEFAULT_BROWSER_CDP_PORT_RANGE_SPAN =`
`21`	`21`	`DEFAULT_BROWSER_CDP_PORT_RANGE_END - DEFAULT_BROWSER_CDP_PORT_RANGE_START;`
`22`	`22`

Original file line number	Diff line number	Diff line change
`@@ -5,12 +5,14 @@ import path from "node:path";`
`5`	`5`	`import { fileURLToPath } from "node:url";`
`6`	`6`	`import { describe, expect, it } from "vitest";`
`7`	`7`	`import { FIELD_HELP } from "./schema.help.js";`
`8`		`-import {`
`9`		`-describeTalkSilenceTimeoutDefaults,`
`10`		`-TALK_SILENCE_TIMEOUT_MS_BY_PLATFORM,`
`11`		`-} from "./talk-defaults.js";`
	`8`	`+import { describeTalkSilenceTimeoutDefaults } from "./talk-defaults.js";`
`12`	`9`
`13`	`10`	`const repoRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "../..");`
	`11`	`+const EXPECTED_TALK_SILENCE_TIMEOUT_MS_BY_PLATFORM = {`
	`12`	`+macos: 700,`
	`13`	`+android: 700,`
	`14`	`+ios: 900,`
	`15`	`+} as const;`
`14`	`16`
`15`	`17`	`function readRepoFile(relativePath: string): string {`
`16`	`18`	`try {`
`@@ -45,13 +47,13 @@ describe("talk silence timeout defaults", () => {`
`45`	`47`	`);`
`46`	`48`
`47`	`49`	`expect(macDefaults).toContain(`
`48`		-`static let silenceTimeoutMs = ${TALK_SILENCE_TIMEOUT_MS_BY_PLATFORM.macos}`,
	`50`	+`static let silenceTimeoutMs = ${EXPECTED_TALK_SILENCE_TIMEOUT_MS_BY_PLATFORM.macos}`,
`49`	`51`	`);`
`50`	`52`	`expect(iosDefaults).toContain(`
`51`		-`static let silenceTimeoutMs = ${TALK_SILENCE_TIMEOUT_MS_BY_PLATFORM.ios}`,
	`53`	+`static let silenceTimeoutMs = ${EXPECTED_TALK_SILENCE_TIMEOUT_MS_BY_PLATFORM.ios}`,
`52`	`54`	`);`
`53`	`55`	`expect(androidDefaults).toContain(`
`54`		-`const val defaultSilenceTimeoutMs = ${TALK_SILENCE_TIMEOUT_MS_BY_PLATFORM.android}L`,
	`56`	+`const val defaultSilenceTimeoutMs = ${EXPECTED_TALK_SILENCE_TIMEOUT_MS_BY_PLATFORM.android}L`,
`55`	`57`	`);`
`56`	`58`	`});`
`57`	`59`	`});`

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/** Platform-specific silence windows for talk/voice turn segmentation. */`
`2`		`-export const TALK_SILENCE_TIMEOUT_MS_BY_PLATFORM = {`
	`2`	`+const TALK_SILENCE_TIMEOUT_MS_BY_PLATFORM = {`
`3`	`3`	`macos: 700,`
`4`	`4`	`android: 700,`
`5`	`5`	`ios: 900,`

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

推荐订阅源

Recent Commits to openclaw:main