vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-06 13:00:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: repair release validation checks

Browse Source
This commit is contained in:
Peter Steinberger
2026-05-04 23:39:37 +01:00
parent f8e080386d
commit 864b1be1b3
4 changed files with 13 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
extensions/diagnostics-otel/src/service.ts
View File

@@ -2222,6 +2222,8 @@ export function createDiagnosticsOtelService(): OpenClawPluginService {
case "diagnostic.liveness.warning":
recordLivenessWarning(evt);
return;
case "diagnostic.phase.completed":
return;
case "run.started":
recordRunStarted(evt, metadata);
return;

2
src/logging/diagnostic-phase.ts
View File

@@ -39,7 +39,7 @@ export function getCurrentDiagnosticPhase(): string | undefined {
}
export function getRecentDiagnosticPhases(limit = 8): DiagnosticPhaseSnapshot[] {
return recentPhases.slice(-Math.max(0, limit)).map((phase) => ({ ...phase }));
return recentPhases.slice(-Math.max(0, limit)).map((phase) => Object.assign({}, phase));
}
export function recordDiagnosticPhase(snapshot: DiagnosticPhaseSnapshot): void {

6
src/logging/diagnostic.test.ts
View File

@@ -397,7 +397,7 @@ describe("stuck session diagnostics threshold", () => {
}
expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining("terminalProgressStale=true"));
expect(events.filter((event) => event.type === "session.stalled").at(-1)).toMatchObject({
expect(events.findLast((event) => event.type === "session.stalled")).toMatchObject({
terminalProgressStale: true,
lastProgressReason: "codex_app_server:notification:rawResponseItem/completed",
});
@@ -754,9 +754,7 @@ describe("stuck session diagnostics threshold", () => {
expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining("phase=startup.plugins.load"));
expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining("work=[queued=main("));
expect(
events.filter((event) => event.type === "diagnostic.liveness.warning").at(-1),
).toMatchObject({
expect(events.findLast((event) => event.type === "diagnostic.liveness.warning")).toMatchObject({
phase: "startup.plugins.load",
queuedWorkLabels: [expect.stringContaining("main(")],
});

12
test/scripts/package-acceptance-workflow.test.ts
View File

@@ -528,8 +528,12 @@ describe("package artifact reuse", () => {
expect(workflow).toContain(
"docker_lanes: doctor-switch update-channel-switch upgrade-survivor published-upgrade-survivor plugins-offline plugin-update",
);
expect(workflow).toContain("published_upgrade_survivor_baselines: all-since-2026.4.23");
expect(workflow).toContain("published_upgrade_survivor_scenarios: reported-issues");
expect(workflow).toContain(
"published_upgrade_survivor_baselines: ${{ needs.resolve_target.outputs.run_release_soak == 'true' && 'all-since-2026.4.23' || '' }}",
);
expect(workflow).toContain(
"published_upgrade_survivor_scenarios: ${{ needs.resolve_target.outputs.run_release_soak == 'true' && 'reported-issues' || '' }}",
);
expect(workflow).toContain("telegram_mode: mock-openai");
expect(workflow).toContain(
"telegram_scenarios: telegram-help-command,telegram-commands-command,telegram-tools-compact-command,telegram-whoami-command,telegram-context-command,telegram-current-session-status-tool,telegram-mention-gating",
@@ -551,7 +555,7 @@ describe("package artifact reuse", () => {
"contains(fromJSON('[\"all\",\"cross-os\",\"package\"]'), needs.resolve_target.outputs.rerun_group) || (needs.resolve_target.outputs.rerun_group == 'live-e2e' && needs.resolve_target.outputs.live_suite_filter == '')",
);
expect(workflow).toContain(
"contains(fromJSON('[\"all\",\"live-e2e\"]'), needs.resolve_target.outputs.rerun_group) && needs.resolve_target.outputs.live_suite_filter == ''",
"(needs.resolve_target.outputs.rerun_group == 'live-e2e' || (needs.resolve_target.outputs.rerun_group == 'all' && needs.resolve_target.outputs.run_release_soak == 'true')) && needs.resolve_target.outputs.live_suite_filter == ''",
);
expect(workflow).toContain("- live-e2e");
expect(workflow).toContain("- qa-live");
@@ -673,7 +677,7 @@ describe("package artifact reuse", () => {
"Focused `npm-telegram` reruns require `npm_telegram_package_spec`",
);
expectTextToIncludeAll(fullReleaseDocs, [
"full pre-publish candidate",
"pre-publish candidate",
"silently skip that",
"Telegram package lane",
"| `npm-telegram` | Published-package Telegram E2E; requires `npm_telegram_package_spec`. |",