refactor(tasks): share audit JSON payload formatting · openclaw/openclaw@8fe181c

Recent Commits to openclaw:main

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

Original file line number	Diff line number	Diff line change
`@@ -103,3 +103,40 @@ export function buildTaskSystemAuditFindings(params: {`
`103`	`103`	`},`
`104`	`104`	`};`
`105`	`105`	`}`
	`106`	`+`
	`107`	`+type TaskSystemAuditResult = ReturnType<typeof buildTaskSystemAuditFindings>;`
	`108`	`+`
	`109`	`+export function buildTaskSystemAuditJsonPayload(`
	`110`	`+result: TaskSystemAuditResult,`
	`111`	`+params: {`
	`112`	`+severityFilter?: TaskSystemAuditSeverity;`
	`113`	`+codeFilter?: TaskSystemAuditCode;`
	`114`	`+limit?: number;`
	`115`	`+},`
	`116`	`+) {`
	`117`	`+const { allFindings, filteredFindings, taskFindings, summary } = result;`
	`118`	`+const limit = typeof params.limit === "number" && params.limit > 0 ? params.limit : undefined;`
	`119`	`+const displayed = limit ? filteredFindings.slice(0, limit) : filteredFindings;`
	`120`	`+// Preserve the legacy task-only summary while adding combined task-flow counts.`
	`121`	`+const legacySummary = summarizeTaskAuditFindings(taskFindings);`
	`122`	`+return {`
	`123`	`+count: allFindings.length,`
	`124`	`+filteredCount: filteredFindings.length,`
	`125`	`+displayed: displayed.length,`
	`126`	`+filters: {`
	`127`	`+severity: params.severityFilter ?? null,`
	`128`	`+code: params.codeFilter ?? null,`
	`129`	`+limit: limit ?? null,`
	`130`	`+},`
	`131`	`+summary: {`
	`132`	`+ ...legacySummary,`
	`133`	`+taskFlows: summary.taskFlows,`
	`134`	`+combined: {`
	`135`	`+total: summary.total,`
	`136`	`+errors: summary.errors,`
	`137`	`+warnings: summary.warnings,`
	`138`	`+},`
	`139`	`+},`
	`140`	`+findings: displayed,`
	`141`	`+};`
	`142`	`+}`

Original file line number	Diff line number	Diff line change
`@@ -6,9 +6,10 @@ import { writeRuntimeJson } from "../runtime.js";`
`6`	`6`	`import { listTaskRecords } from "../tasks/runtime-internal.js";`
`7`	`7`	`import { listTaskFlowAuditFindings } from "../tasks/task-flow-registry.audit.js";`
`8`	`8`	`import { listTaskFlowRecords } from "../tasks/task-flow-runtime-internal.js";`
`9`		`-import { listTaskAuditFindings, summarizeTaskAuditFindings } from "../tasks/task-registry.audit.js";`
	`9`	`+import { listTaskAuditFindings } from "../tasks/task-registry.audit.js";`
`10`	`10`	`import type { TaskRecord } from "../tasks/task-registry.types.js";`
`11`	`11`	`import {`
	`12`	`+buildTaskSystemAuditJsonPayload,`
`12`	`13`	`buildTaskSystemAuditFindings,`
`13`	`14`	`type TaskSystemAuditCode,`
`14`	`15`	`type TaskSystemAuditSeverity,`
`@@ -73,34 +74,15 @@ function buildTasksListJsonPayload(opts: TasksListJsonArgs) {`
`73`	`74`	`function buildTasksAuditJsonPayload(opts: TasksAuditJsonArgs) {`
`74`	`75`	`const severityFilter = opts.severity?.trim() as TaskSystemAuditSeverity \| undefined;`
`75`	`76`	`const codeFilter = opts.code?.trim() as TaskSystemAuditCode \| undefined;`
`76`		`-const { allFindings, filteredFindings, taskFindings, summary } = toSystemAuditFindings({`
	`77`	`+const result = toSystemAuditFindings({`
`77`	`78`	`severityFilter,`
`78`	`79`	`codeFilter,`
`79`	`80`	`});`
`80`		`-const limit = typeof opts.limit === "number" && opts.limit > 0 ? opts.limit : undefined;`
`81`		`-const displayed = limit ? filteredFindings.slice(0, limit) : filteredFindings;`
`82`		`-// Preserve the legacy task-only summary while adding combined task-flow counts.`
`83`		`-const legacySummary = summarizeTaskAuditFindings(taskFindings);`
`84`		`-return {`
`85`		`-count: allFindings.length,`
`86`		`-filteredCount: filteredFindings.length,`
`87`		`-displayed: displayed.length,`
`88`		`-filters: {`
`89`		`-severity: severityFilter ?? null,`
`90`		`-code: codeFilter ?? null,`
`91`		`-limit: limit ?? null,`
`92`		`-},`
`93`		`-summary: {`
`94`		`- ...legacySummary,`
`95`		`-taskFlows: summary.taskFlows,`
`96`		`-combined: {`
`97`		`-total: summary.total,`
`98`		`-errors: summary.errors,`
`99`		`-warnings: summary.warnings,`
`100`		`-},`
`101`		`-},`
`102`		`-findings: displayed,`
`103`		`-};`
	`81`	`+return buildTaskSystemAuditJsonPayload(result, {`
	`82`	`+ severityFilter,`
	`83`	`+ codeFilter,`
	`84`	`+limit: opts.limit,`
	`85`	`+});`
`104`	`86`	`}`
`105`	`87`
`106`	`88`	`/** Writes task list JSON without triggering task maintenance. */`

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,6 @@ import {`
`24`	`24`	`import {`
`25`	`25`	`listTaskAuditFindings,`
`26`	`26`	`summarizeRetainedLostTaskAuditFindings,`
`27`		`-summarizeTaskAuditFindings,`
`28`	`27`	`} from "../tasks/task-registry.audit.js";`
`29`	`28`	`import {`
`30`	`29`	`getInspectableTaskAuditSummary,`
`@@ -41,6 +40,7 @@ import {`
`41`	`40`	`import { summarizeTaskRecords } from "../tasks/task-registry.summary.js";`
`42`	`41`	`import type { TaskNotifyPolicy, TaskRecord } from "../tasks/task-registry.types.js";`
`43`	`42`	`import {`
	`43`	`+buildTaskSystemAuditJsonPayload,`
`44`	`44`	`buildTaskSystemAuditFindings,`
`45`	`45`	`type TaskSystemAuditCode,`
`46`	`46`	`type TaskSystemAuditFinding,`
`@@ -499,37 +499,22 @@ export async function tasksAuditCommand(`
`499`	`499`	`configureTaskMaintenanceFromConfig();`
`500`	`500`	`const severityFilter = opts.severity?.trim() as TaskSystemAuditSeverity \| undefined;`
`501`	`501`	`const codeFilter = opts.code?.trim() as TaskSystemAuditCode \| undefined;`
`502`		`-const { allFindings, filteredFindings, taskFindings, summary } = toSystemAuditFindings({`
	`502`	`+const auditResult = toSystemAuditFindings({`
`503`	`503`	`severityFilter,`
`504`	`504`	`codeFilter,`
`505`	`505`	`});`
	`506`	`+const { filteredFindings, summary } = auditResult;`
`506`	`507`	`const limit = typeof opts.limit === "number" && opts.limit > 0 ? opts.limit : undefined;`
`507`	`508`	`const displayed = limit ? filteredFindings.slice(0, limit) : filteredFindings;`
`508`	`509`
`509`	`510`	`if (opts.json) {`
`510`		`-const legacySummary = summarizeTaskAuditFindings(taskFindings);`
`511`	`511`	`runtime.log(`
`512`	`512`	`JSON.stringify(`
`513`		`-{`
`514`		`-count: allFindings.length,`
`515`		`-filteredCount: filteredFindings.length,`
`516`		`-displayed: displayed.length,`
`517`		`-filters: {`
`518`		`-severity: severityFilter ?? null,`
`519`		`-code: codeFilter ?? null,`
`520`		`-limit: limit ?? null,`
`521`		`-},`
`522`		`-summary: {`
`523`		`- ...legacySummary,`
`524`		`-taskFlows: summary.taskFlows,`
`525`		`-combined: {`
`526`		`-total: summary.total,`
`527`		`-errors: summary.errors,`
`528`		`-warnings: summary.warnings,`
`529`		`-},`
`530`		`-},`
`531`		`-findings: displayed,`
`532`		`-},`
	`513`	`+buildTaskSystemAuditJsonPayload(auditResult, {`
	`514`	`+ severityFilter,`
	`515`	`+ codeFilter,`
	`516`	`+limit: opts.limit,`
	`517`	`+}),`
`533`	`518`	`null,`
`534`	`519`	`2,`
`535`	`520`	`),`

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

推荐订阅源

Recent Commits to openclaw:main