Human-gated AI development workflows

Worker dispatch ready

KAN-11PR createdArchived

Show workflow step durations and live timers

Repository

danubii/openclaw-orchestrator

ai/KAN-11-show-workflow-step-durations-and-live-timers

Created

Apr 25, 08:22 PM UTC

Updated Apr 26, 09:22 AM UTC

Owner

dev-user

Cross Repository Integration Test / PR created

Mode

Feature

PlanWaitForApproval

Run timeline

13/14 steps complete

Ticket Intake

Apr 25, 08:22 PM UTCDuration 2s

Completed

Context Build

Apr 25, 08:26 PM UTCDuration 4m 27s

Completed

Codex Plan

Apr 25, 08:26 PM UTCDuration 4m 27s

Completed

Plan Approval

Apr 25, 08:29 PM UTCDuration 0s

Completed

Codex Implementation

Apr 25, 10:32 PM UTCDuration 9m 39s

Completed

Test Run

Apr 25, 10:34 PM UTCDuration 2m 27s

Completed

Cross Repository Integration Test

Queued

Pending

Smoke Test

Apr 25, 10:36 PM UTCDuration 1m 46s

Completed

Claude Review

Apr 25, 10:41 PM UTCDuration 5m 07s

Completed

Review Approval

Apr 25, 10:43 PM UTCDuration 0s

Completed

Git Push Branch

Apr 25, 10:43 PM UTCDuration 15s

Completed

Test Deploy

Apr 25, 10:44 PM UTCDuration 0s

Completed

Manual Qa

Apr 25, 10:47 PM UTCDuration 0s

Completed

Pull Request

Apr 25, 10:47 PM UTCDuration 0s

Completed

Artifact viewer

PR_DESCRIPTION.md · v1

Loading artifact

Lifecycle

Archived workflow

Archived Apr 26, 09:22 AM UTC by dev-user.

Gate control

Cross Repository Integration Test

PlanningImplementation

Current status does not require a manual gate action.

https://github.com/danubii/openclaw-orchestrator/pull/1004

Approval history

QaApproved

dev-user / Apr 25, 10:47 PM UTC

QA approved after automated green test/smoke/review and test deploy gates.

TestDeployApproved

dev-user / Apr 25, 10:44 PM UTC

Deploy to test approved after branch push.

ReviewApproved

dev-user / Apr 25, 10:43 PM UTC

Approved: no blocker/high findings after green tests and smoke. Medium findings accepted for this run.

ReviewChanges requested

dev-user / Apr 25, 10:22 PM UTC

Return Claude high findings to implementation.

ReviewChanges requested

dev-user / Apr 25, 09:38 PM UTC

fix the 2 medium notes.

PlanApproved

dev-user / Apr 25, 08:29 PM UTC

Also create a flashing dot on the current running tasks in the Dashboard, and make sure live updates are working properly both in the workflows and also in the dashboard.

Repositories

1 target

danubii/openclaw-orchestrator

ai/KAN-11-show-workflow-step-durations-and-live-timers

Completed

Target: main

QA checklist

3/4 complete

Smoke test critical user pathRequired / Apr 25, 10:47 PM UTC

Verify deployment health endpointRequired / Apr 25, 10:47 PM UTC

Check generated release notesRequired / Apr 25, 10:47 PM UTC

Capture manual regression notesOptional

Review findings

9 total / 4 priority

No unit tests for WorkflowTiming helper class

Medium

src/OpenClaw.Orchestrator.Infrastructure/Services/WorkflowTiming.cs:1-77

WorkflowTiming.cs contains all backend timing logic (DurationSeconds, ElapsedSeconds, HasLiveTimer, CurrentStep, CompletedAt, TotalElapsedSeconds) but is only exercised through full orchestrator integration tests. Edge cases such as negative time deltas, empty step lists, CurrentStep tie-breaking, and CompletedAt falling back to workflow.UpdatedAt are untested in isolation.

Add WorkflowTimingTests.cs with focused xUnit tests for each public method. Use DateTimeOffset literals to cover: (a) SecondsBetween clamping when endedAt < startedAt, (b) CurrentStep returning Running before open-gate before Failed before Pending before last Completed, (c) CompletedAt returning workflow.UpdatedAt when no steps have CompletedAt set.

Playwright timing test relies on fallback mode, breaks when backend is reachable

Medium

web/tests/e2e/project-repository-workflow.spec.ts:266-279

The test navigates to /workflows/timing-demo and expects 'Duration 7m 00s'. This only works when the API is unavailable (ECONNREFUSED) and the server falls back to hardcoded fixture data. In any environment where the backend is running, the route returns 404 or a real workflow that lacks those exact timing values, causing the test to fail.

Replace the page.goto('/workflows/timing-demo') approach with page.route('**/api/workflows/**', route => route.fulfill({ json: deterministicWorkflowFixture })) so the test is environment-independent. The fixture should have durationSeconds:420 on the CodexPlan step and a running PlanApproval step.

Waiting/gate workflow states do not join SignalR groups on dashboard

Medium

web/features/workflows/workflow-dashboard.tsx:305-309

isRunningWorkflowStatus in workflow-dashboard.tsx only includes automated-worker states (Planning, ContextBuilding, Implementing, Testing, Reviewing, PushingBranch, DeployingToTest). Workflows in WaitingForPlanApproval, WaitingForReviewApproval, WaitingForQaApproval, ReadyForPr, or QaRejected never join SignalR groups. When a human approves a gate while the user watches the dashboard, the live timer does not switch from elapsed to completed duration until the next navigation or worker activity.

Consider whether WaitingFor* and ReadyForPr statuses should be included in isRunningWorkflowStatus. If the product accepts the local-tick-only behaviour for gate steps, document it explicitly in a code comment. If not, add the waiting states so the dashboard refreshes when gate approvals arrive.

WorkerJobProcessor timestamp hygiene changes are untested

Medium

src/OpenClaw.Orchestrator.Infrastructure/Services/WorkerJobProcessor.cs:1992-2002

WorkerJobProcessor.cs now sets job.StartedAt ??= agentRun?.StartedAt ?? step.StartedAt ?? now (line ~1995), step.StartedAt ??= job.StartedAt (line ~2002), and sets StartedAt on plan-approval, review-approval, QA, and test-deploy gate steps when they open. These writes are the basis for live gate timers and failed-step duration calculations but have no targeted tests.

Add test cases in WorkflowOrchestratorTests.cs (or a dedicated WorkerJobProcessorTimingTests.cs) that: (1) fail a job with no prior StartedAt and assert step.StartedAt is populated from agentRun.StartedAt; (2) complete a planning job and assert the PlanApproval gate step receives a non-null StartedAt equal to the completion timestamp.

formatElapsedSeconds(0) renders 'Duration 0s' for instantaneous steps

Low

web/lib/utils.ts:31-52

Steps where startedAt equals completedAt produce durationSeconds:0 and display 'Duration 0s'. The CrossRepositoryIntegrationTest step in the fallback data demonstrates this. The label looks like missing data rather than a valid sub-second completion.

Either omit the duration label for steps with durationSeconds < 2 (return null from timingLabel), or format sub-second completions as '<1s'. Decide on the threshold in the product and apply it uniformly.

runTimingLabel called twice per integration run and deployment element

Low

web/features/workflows/workflow-detail-client.tsx:766-769

workflow-detail-client.tsx calls runTimingLabel(latestDeployment, nowTick) in a conditional check and then again inside the JSX to render the value. The same pattern appears for integrationRun. Since nowTick changes every second during live timing, this doubles pure function calls unnecessarily and can make the code harder to follow.

Extract to a local variable: const deploymentTiming = runTimingLabel(latestDeployment, nowTick); then use {deploymentTiming ? <div>...{deploymentTiming}</div> : null}. Apply the same pattern for integrationRun (~line 858).

useNowTick does not reset to null when enabled transitions from true to false

Low

web/lib/hooks.ts:6-20

When enabled changes from true to false, the interval is cleared but now retains its last Date value. Components that later check now !== null (implicitly, via passing now to elapsedSeconds) will receive a stale timestamp rather than null. No current caller is affected because they gate on hasVisibleTimers first, but future callers could be surprised.

Add setNow(null) in the effect cleanup when enabled is false, or add a separate cleanup branch: if (!enabled) { setNow(null); return; }. This makes the contract explicit: null means 'not ticking', Date means 'ticking'.

Add WorkflowTiming unit test file

Suggestion

tests/OpenClaw.Orchestrator.Application.Tests/WorkflowOrchestratorTests.cs

Pure static helper deserves its own test class rather than only integration coverage.

Create tests/OpenClaw.Orchestrator.Application.Tests/WorkflowTimingTests.cs with parameterised xUnit theory tests for each public method in WorkflowTiming.cs.

Document QaRejected exclusion from IsTerminal

Suggestion

src/OpenClaw.Orchestrator.Infrastructure/Services/WorkflowTiming.cs:44-49

WorkflowTiming.IsTerminal does not include QaRejected, so total elapsed keeps ticking for rejected workflows. The intent (QA rejection is resubmittable, not truly final) is not captured anywhere in the code.

Add an inline comment: // QaRejected is intentionally excluded: a rejected workflow can be resubmitted, so elapsed time continues to accumulate.

Live logs

100 entries / offline

22:35:33smokeSetting up libatk-bridge2.0-0t64:amd64 (2.52.0-1build1) ...

22:35:33smokeSetting up libglx-mesa0:amd64 (25.2.8-0ubuntu0.24.04.1) ...

22:35:33smokeSetting up libglx0:amd64 (1.7.0-1build1) ...

22:35:33smokeSetting up x11-xkb-utils (7.7+8build2) ...

22:35:33smokeSetting up libgl1:amd64 (1.7.0-1build1) ...

22:35:33smokeSetting up xserver-common (2:21.1.12-1ubuntu1.5) ...

22:35:33smokeSetting up xvfb (2:21.1.12-1ubuntu1.5) ...

22:35:33smokeProcessing triggers for libc-bin (2.39-0ubuntu8.7) ...

22:35:33smokeDownloading Chrome for Testing 147.0.7727.15 (playwright chromium v1217) from https://cdn.playwright.dev/builds/cft/147.0.7727.15/linux64/chrome-linux64.zip

22:35:41smoke| | 0% of 170.4 MiB

22:35:42smoke|■■■■■■■■ | 10% of 170.4 MiB

22:35:42smoke|■■■■■■■■■■■■■■■■ | 20% of 170.4 MiB

22:35:43smoke|■■■■■■■■■■■■■■■■■■■■■■■■ | 30% of 170.4 MiB

22:35:43smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 40% of 170.4 MiB

22:35:44smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 50% of 170.4 MiB

22:35:44smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 60% of 170.4 MiB

22:35:45smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 70% of 170.4 MiB

22:35:45smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 80% of 170.4 MiB

22:35:46smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 90% of 170.4 MiB

22:35:46smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■| 100% of 170.4 MiB

22:35:49smokeChrome for Testing 147.0.7727.15 (playwright chromium v1217) downloaded to /root/.cache/ms-playwright/chromium-1217

22:35:49smokeDownloading FFmpeg (playwright ffmpeg v1011) from https://cdn.playwright.dev/dbazure/download/playwright/builds/ffmpeg/1011/ffmpeg-linux.zip

22:35:53smoke| | 0% of 2.3 MiB

22:35:53smoke|■■■■■■■■ | 10% of 2.3 MiB

22:35:53smoke|■■■■■■■■■■■■■■■■ | 20% of 2.3 MiB

22:35:53smoke|■■■■■■■■■■■■■■■■■■■■■■■■ | 30% of 2.3 MiB

22:35:53smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 40% of 2.3 MiB

22:35:53smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 50% of 2.3 MiB

22:35:53smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 60% of 2.3 MiB

22:35:53smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 70% of 2.3 MiB

22:35:53smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 80% of 2.3 MiB

22:35:53smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 90% of 2.3 MiB

22:35:53smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■| 100% of 2.3 MiB

22:35:53smokeFFmpeg (playwright ffmpeg v1011) downloaded to /root/.cache/ms-playwright/ffmpeg-1011

22:35:53smokeDownloading Chrome Headless Shell 147.0.7727.15 (playwright chromium-headless-shell v1217) from https://cdn.playwright.dev/builds/cft/147.0.7727.15/linux64/chrome-headless-shell-linux64.zip

22:36:00smoke| | 0% of 112 MiB

22:36:01smoke|■■■■■■■■ | 10% of 112 MiB

22:36:02smoke|■■■■■■■■■■■■■■■■ | 20% of 112 MiB

22:36:02smoke|■■■■■■■■■■■■■■■■■■■■■■■■ | 30% of 112 MiB

22:36:02smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 40% of 112 MiB

22:36:02smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 50% of 112 MiB

22:36:03smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 60% of 112 MiB

22:36:03smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 70% of 112 MiB

22:36:03smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 80% of 112 MiB

22:36:04smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 90% of 112 MiB

22:36:04smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■| 100% of 112 MiB

22:36:05smokeChrome Headless Shell 147.0.7727.15 (playwright chromium-headless-shell v1217) downloaded to /root/.cache/ms-playwright/chromium_headless_shell-1217

22:36:05smoke> web@0.1.0 test:e2e

22:36:05smoke> playwright test

22:36:15smokeRunning 2 tests using 1 worker

22:36:21smokestderr: [WebServer] Failed to proxy http://127.0.0.1:5256/api/artifacts/00000000-0000-0000-0000-000000000401 Error: connect ECONNREFUSED 127.0.0.1:5256

22:36:21smokestderr: [WebServer] at <unknown> (Error: connect ECONNREFUSED 127.0.0.1:5256) {

22:36:21smokestderr: [WebServer] errno: [33m-111[39m,

22:36:21smokestderr: [WebServer] code: [32m'ECONNREFUSED'[39m,

22:36:21smokestderr: [WebServer] syscall: [32m'connect'[39m,

22:36:21smokestderr: [WebServer] address: [32m'127.0.0.1'[39m,

22:36:21smokestderr: [WebServer] port: [33m5256[39m

22:36:21smokestderr: [WebServer] }

22:36:21smokestderr: [WebServer] Error: connect ECONNREFUSED 127.0.0.1:5256

22:36:21smokestderr: [WebServer] at <unknown> (Error: connect ECONNREFUSED 127.0.0.1:5256) {

22:36:21smokestderr: [WebServer] errno: [33m-111[39m,

22:36:21smokestderr: [WebServer] code: [32m'ECONNREFUSED'[39m,

22:36:21smokestderr: [WebServer] syscall: [32m'connect'[39m,

22:36:21smokestderr: [WebServer] address: [32m'127.0.0.1'[39m,

22:36:21smokestderr: [WebServer] port: [33m5256[39m

22:36:21smokestderr: [WebServer] }

22:36:22smoke··

22:36:22smoke 2 passed (16.2s)

22:36:23smokeSmokeTest completed with exit code 0.

22:36:23workerQueued ClaudeReview.

22:36:23workerWorker claimed ClaudeReview job 8ce74082-5a07-4ca2-b963-93d78ecb51b8.

22:41:30claudeClaudeReview completed with exit code 0.

22:43:07gitReview approved. Queued branch push for 1 repository.

22:43:08workerWorker claimed GitPushBranch job 6f7cfd4f-58f2-4f96-8141-c36b40bc69ed.

22:43:10gitstderr: Cloning into 'repository'...

22:43:14gitstderr: HEAD is now at de4fb97 Allow overriding medium review findings

22:43:15gitstderr: Switched to a new branch 'ai/KAN-11-show-workflow-step-durations-and-live-timers'

22:43:15git[ai/KAN-11-show-workflow-step-durations-and-live-timers 0421460] KAN-11: automated implementation

22:43:15git 15 files changed, 811 insertions(+), 126 deletions(-)

22:43:15git create mode 100644 src/OpenClaw.Orchestrator.Infrastructure/Services/WorkflowTiming.cs

22:43:15git create mode 100644 web/lib/hooks.ts

22:43:23gitstderr: remote:

22:43:23gitstderr: remote: Create a pull request for 'ai/KAN-11-show-workflow-step-durations-and-live-timers' on GitHub by visiting:

22:43:23gitstderr: remote: https://github.com/Danubii/openclaw-orchestrator/pull/new/ai/KAN-11-show-workflow-step-durations-and-live-timers

22:43:23gitstderr: remote:

22:43:23gitstderr: To https://github.com/Danubii/openclaw-orchestrator.git

22:43:23gitstderr: * [new branch] HEAD -> ai/KAN-11-show-workflow-step-durations-and-live-timers

22:43:23gitbranch 'ai/KAN-11-show-workflow-step-durations-and-live-timers' set up to track 'origin/ai/KAN-11-show-workflow-step-durations-and-live-timers'.

22:43:23gitGitPushBranch completed with exit code 0.

22:44:53systemQueued test deployment to QA Sandbox.

22:44:54workerWorker claimed MockTestDeploy job 2e7c6b8f-8186-4dc8-82c3-8b0dbefa38be.

22:44:54deploy10.0.202

22:44:54deploy10.9.7

22:44:54deployMockTestDeploy completed with exit code 0.

22:47:15qaSmoke test critical user path: completed.

22:47:15qaVerify deployment health endpoint: completed.

22:47:15qaCheck generated release notes: completed.

22:47:15qaQA approved. Creating PR handoff records.

22:47:15githubPR handoff records created: https://github.com/danubii/openclaw-orchestrator/pull/1004

09:22:27systemWorkflow archived by dev-user.