refactor(qa): share progress formatting helpers · opencla...

Recent Commits to openclaw:main

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

Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,10 @@ import {`
`18`	`18`	`} from "../../evidence-summary.js";`
`19`	`19`	`import { startQaGatewayChild } from "../../gateway-child.js";`
`20`	`20`	`import { isTruthyOptIn } from "../../mantis-options.runtime.js";`
	`21`	`+import {`
	`22`	`+parseQaProgressBooleanEnv as parseTelegramQaProgressBooleanEnv,`
	`23`	`+sanitizeQaProgressValue as sanitizeTelegramQaProgressValue,`
	`24`	`+} from "../../progress-format.js";`
`21`	`25`	`import { DEFAULT_QA_LIVE_PROVIDER_MODE } from "../../providers/index.js";`
`22`	`26`	`import {`
`23`	`27`	`defaultQaModelForMode,`
`@@ -574,20 +578,6 @@ function readConfigRecord(root: Record<string, unknown>, key: string): Record<st`
`574`	`578`	`return value;`
`575`	`579`	`}`
`576`	`580`
`577`		`-function parseTelegramQaProgressBooleanEnv(value: string \| undefined): boolean \| undefined {`
`578`		`-const normalized = value?.trim().toLowerCase();`
`579`		`-if (!normalized) {`
`580`		`-return undefined;`
`581`		`-}`
`582`		`-if (normalized === "1" \|\| normalized === "true" \|\| normalized === "yes" \|\| normalized === "on") {`
`583`		`-return true;`
`584`		`-}`
`585`		`-if (normalized === "0" \|\| normalized === "false" \|\| normalized === "no" \|\| normalized === "off") {`
`586`		`-return false;`
`587`		`-}`
`588`		`-return undefined;`
`589`		`-}`
`590`		`-`
`591`	`581`	`function shouldLogTelegramQaLiveProgress(env: NodeJS.ProcessEnv = process.env) {`
`592`	`582`	`const override = parseTelegramQaProgressBooleanEnv(env[QA_SUITE_PROGRESS_ENV]);`
`593`	`583`	`if (override !== undefined) {`
`@@ -680,20 +670,6 @@ function writeTelegramQaProgress(enabled: boolean, message: string) {`
`680`	`670`	process.stderr.write(`${TELEGRAM_QA_PROGRESS_PREFIX} ${message}\n`);
`681`	`671`	`}`
`682`	`672`
`683`		`-function sanitizeTelegramQaProgressValue(value: string): string {`
`684`		`-let normalized = "";`
`685`		`-for (const char of value) {`
`686`		`-const code = char.codePointAt(0);`
`687`		`-if (code === undefined) {`
`688`		`-continue;`
`689`		`-}`
`690`		`-const isControl = code <= 0x1f \|\| (code >= 0x7f && code <= 0x9f);`
`691`		`-normalized += isControl ? " " : char;`
`692`		`-}`
`693`		`-normalized = normalized.replace(/\s+/gu, " ").trim();`
`694`		`-return normalized.length > 0 ? normalized : "<empty>";`
`695`		`-}`
`696`		`-`
`697`	`673`	`function formatTelegramQaProgressDetails(details: string): string {`
`698`	`674`	`const sanitized = sanitizeTelegramQaProgressValue(details);`
`699`	`675`	`if (sanitized.length <= TELEGRAM_QA_PROGRESS_DETAIL_LIMIT) {`

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,27 @@`
	`1`	`+export function parseQaProgressBooleanEnv(value: string \| undefined): boolean \| undefined {`
	`2`	`+const normalized = value?.trim().toLowerCase();`
	`3`	`+if (!normalized) {`
	`4`	`+return undefined;`
	`5`	`+}`
	`6`	`+if (normalized === "1" \|\| normalized === "true" \|\| normalized === "yes" \|\| normalized === "on") {`
	`7`	`+return true;`
	`8`	`+}`
	`9`	`+if (normalized === "0" \|\| normalized === "false" \|\| normalized === "no" \|\| normalized === "off") {`
	`10`	`+return false;`
	`11`	`+}`
	`12`	`+return undefined;`
	`13`	`+}`
	`14`	`+`
	`15`	`+export function sanitizeQaProgressValue(value: string): string {`
	`16`	`+let normalized = "";`
	`17`	`+for (const char of value) {`
	`18`	`+const code = char.codePointAt(0);`
	`19`	`+if (code === undefined) {`
	`20`	`+continue;`
	`21`	`+}`
	`22`	`+const isControl = code <= 0x1f \|\| (code >= 0x7f && code <= 0x9f);`
	`23`	`+normalized += isControl ? " " : char;`
	`24`	`+}`
	`25`	`+normalized = normalized.replace(/\s+/gu, " ").trim();`
	`26`	`+return normalized.length > 0 ? normalized : "<empty>";`
	`27`	`+}`

Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,10 @@ import {`
`27`	`27`	`normalizeQaProviderMode,`
`28`	`28`	`type QaProviderMode,`
`29`	`29`	`} from "./model-selection.js";`
	`30`	`+import {`
	`31`	`+parseQaProgressBooleanEnv as parseQaSuiteBooleanEnv,`
	`32`	`+sanitizeQaProgressValue as sanitizeQaSuiteProgressValue,`
	`33`	`+} from "./progress-format.js";`
`30`	`34`	`import { DEFAULT_QA_LIVE_PROVIDER_MODE } from "./providers/index.js";`
`31`	`35`	`import { startQaProviderServer } from "./providers/server-runtime.js";`
`32`	`36`	`import type { QaThinkingLevel } from "./qa-gateway-config.js";`
`@@ -124,20 +128,6 @@ export type QaSuiteRunParams = {`
`124`	`128`	`captureRuntimeParityCell?: boolean;`
`125`	`129`	`};`
`126`	`130`
`127`		`-function parseQaSuiteBooleanEnv(value: string \| undefined): boolean \| undefined {`
`128`		`-const normalized = value?.trim().toLowerCase();`
`129`		`-if (!normalized) {`
`130`		`-return undefined;`
`131`		`-}`
`132`		`-if (normalized === "1" \|\| normalized === "true" \|\| normalized === "yes" \|\| normalized === "on") {`
`133`		`-return true;`
`134`		`-}`
`135`		`-if (normalized === "0" \|\| normalized === "false" \|\| normalized === "no" \|\| normalized === "off") {`
`136`		`-return false;`
`137`		`-}`
`138`		`-return undefined;`
`139`		`-}`
`140`		`-`
`141`	`131`	`function shouldLogQaSuiteProgress(env: NodeJS.ProcessEnv = process.env) {`
`142`	`132`	`const override = parseQaSuiteBooleanEnv(env.OPENCLAW_QA_SUITE_PROGRESS);`
`143`	`133`	`if (override !== undefined) {`
`@@ -214,20 +204,6 @@ async function waitForQaLabReadyOrStopOwned(params: {`
`214`	`204`	`}`
`215`	`205`	`}`
`216`	`206`
`217`		`-function sanitizeQaSuiteProgressValue(value: string): string {`
`218`		`-let normalized = "";`
`219`		`-for (const char of value) {`
`220`		`-const code = char.codePointAt(0);`
`221`		`-if (code === undefined) {`
`222`		`-continue;`
`223`		`-}`
`224`		`-const isControl = code <= 0x1f \|\| (code >= 0x7f && code <= 0x9f);`
`225`		`-normalized += isControl ? " " : char;`
`226`		`-}`
`227`		`-normalized = normalized.replace(/\s+/gu, " ").trim();`
`228`		`-return normalized.length > 0 ? normalized : "<empty>";`
`229`		`-}`
`230`		`-`
`231`	`207`	`function requireQaSuiteStartLab(startLab: QaSuiteStartLabFn \| undefined): QaSuiteStartLabFn {`
`232`	`208`	`if (startLab) {`
`233`	`209`	`return startLab;`

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

推荐订阅源

Recent Commits to openclaw:main