Human-gated AI development workflows

Worker dispatch ready

KAN-10PR createdArchived

Add archive and delete actions for workflows

Repository

danubii/openclaw-orchestrator

ai/KAN-10-add-archive-and-delete-actions-for-workflows

Created

Apr 25, 08:35 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:35 PM UTCDuration 2s

Completed

Context Build

Apr 25, 08:38 PM UTCDuration 3m 41s

Completed

Codex Plan

Apr 25, 08:38 PM UTCDuration 3m 41s

Completed

Plan Approval

Apr 25, 08:40 PM UTCDuration 0s

Completed

Codex Implementation

Apr 25, 10:28 PM UTCDuration 4m 22s

Completed

Test Run

Apr 25, 10:30 PM UTCDuration 2m 19s

Completed

Cross Repository Integration Test

Queued

Pending

Smoke Test

Apr 25, 10:33 PM UTCDuration 2m 36s

Completed

Claude Review

Apr 25, 10:37 PM UTCDuration 3m 44s

Completed

Review Approval

Apr 25, 10:40 PM UTCDuration 0s

Completed

Git Push Branch

Apr 25, 10:40 PM UTCDuration 10s

Completed

Test Deploy

Apr 25, 10:42 PM UTCDuration 2s

Completed

Manual Qa

Apr 25, 10:45 PM UTCDuration 0s

Completed

Pull Request

Apr 25, 10:45 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/1003

Approval history

QaApproved

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

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

TestDeployApproved

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

Deploy to test approved after branch push.

ReviewApproved

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

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

PlanApproved

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

Repositories

1 target

danubii/openclaw-orchestrator

ai/KAN-10-add-archive-and-delete-actions-for-workflows

Completed

Target: main

QA checklist

3/4 complete

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

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

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

Capture manual regression notesOptional

Review findings

8 total / 4 priority

No Playwright coverage for detail-page lifecycle actions

Medium

web/tests/e2e/project-repository-workflow.spec.ts:303-413

The approved plan (§9 Integration Test Strategy) and implementation report both state that lifecycle actions were added to the workflow detail page. The new Playwright test only exercises the overview. A regression in the detail-page archive/restore/delete or the post-delete navigation back to the overview would go undetected.

Add a second Playwright test (or extend the existing one) that navigates to /workflows/:id, archives from the detail lifecycle panel, verifies the Archived badge appears, restores, and then confirms delete navigates back to /.

Restore endpoint is missing try/catch for InvalidOperationException

Medium

src/OpenClaw.Orchestrator.Api/Endpoints/WorkflowEndpoints.cs:57-61

The archive and delete endpoints both wrap the orchestrator call in try/catch for InvalidOperationException and return BadRequest. The restore endpoint does not. If a future guard (e.g., blocking restore of deleted workflows) throws InvalidOperationException, it will propagate as an unhandled 500 instead of a structured 400.

Wrap the RestoreAsync call in the same try/catch { return Results.BadRequest(new { error = ex.Message }); } pattern used by archive and delete.

Variable shadowing: active (workflow count) vs active (boolean cleanup flag) in WorkflowDashboard

Medium

web/features/workflows/workflow-dashboard.tsx:47

The component-scope constant `const active = visibleWorkflows.filter(...).length` (a number) is shadowed by `let active = true` inside the useEffect cleanup closure (a boolean). A future reader of the effect body may incorrectly assume `active` refers to the running-workflow count and introduce bugs when modifying the cleanup logic.

Rename the useEffect cleanup flag to `let mounted = true` / `mounted = false` to eliminate the shadowing.

WorkflowDashboard fires redundant client-side API fetch on every lifecycle action

Medium

web/features/workflows/workflow-dashboard.tsx:45-66

Converting WorkflowDashboard to a client component introduced a useEffect that calls getWorkflows(view) on every change to the workflows prop or view. After each archive/restore/delete action, router.refresh() already causes the server component to re-fetch and pass a fresh prop; the concurrent client fetch results in three state updates and two redundant API calls per action. The optimistic update from onWorkflowUpdated is also immediately overwritten by the queueMicrotask reset before it can be seen.

Remove the getWorkflows(view) call from the useEffect. Rely solely on the server prop (updated via router.refresh()) for the authoritative list and on onWorkflowUpdated for the optimistic in-flight state. If a client refresh is needed independently, wire it to an explicit user action.

Default smoke profile JSON duplicated in three independent locations

Low

The npm ci --prefix web smoke step was added to DevelopmentSeeder.cs, RepositoryConfigService.cs, and WorkflowOrchestratorTests.cs independently. Future profile changes must be applied in all three places.

Extract the default test profile JSON into a shared constant or base class accessible to all three locations.

SeedWorkflowAsync uses Random.Shared.Next for JiraTicketKey with limited key space

Low

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

With only 9,000 possible values (1000–9999), concurrent or repeated test runs against a shared database could produce duplicate JiraTicketKeys. This is unlikely in practice but is avoidable.

Use `Guid.NewGuid().ToString("N")[..8]` or a counter to guarantee uniqueness across runs.

Document deliberate exclusion of ReadyForPr and WaitingForPlanApproval from delete eligibility

Suggestion

src/OpenClaw.Orchestrator.Infrastructure/Services/WorkflowOrchestrator.cs:1379-1386

IsDeleteEligibleStatus excludes WaitingForPlanApproval and ReadyForPr. The plan listed these as open questions. Without a comment, a future developer may add them inadvertently or question the decision.

Add a single-line comment explaining that WaitingForPlanApproval and ReadyForPr are intentionally excluded pending product policy decisions on non-terminal manual-gate states.

Simplify WorkflowDashboard to remove client-side data refetch

Suggestion

web/features/workflows/workflow-dashboard.tsx:45-66

The current useEffect fetches data from the API that the server already fetched and passed via props. router.refresh() handles the authoritative state update; the local optimistic update via onWorkflowUpdated covers the in-flight gap. The extra fetch is unnecessary complexity.

Delete the getWorkflows(view) call and the queueMicrotask reset from the useEffect. Update visibleWorkflows only from the workflows prop (via useState initialisation) and via onWorkflowUpdated/onWorkflowDeleted callbacks.

Live logs

100 entries / offline

22:32:36smokelibnss3 is already the newest version (2:3.98-1ubuntu0.1).

22:32:36smokelibpango-1.0-0 is already the newest version (1.52.1+ds-1build1).

22:32:36smokelibx11-6 is already the newest version (2:1.8.7-1build1).

22:32:36smokelibxcb1 is already the newest version (1.15-1ubuntu2).

22:32:36smokelibxcomposite1 is already the newest version (1:0.4.5-1build3).

22:32:36smokelibxdamage1 is already the newest version (1:1.1.6-1build1).

22:32:36smokelibxext6 is already the newest version (2:1.3.4-1build2).

22:32:36smokelibxfixes3 is already the newest version (1:6.0.0-2build1).

22:32:36smokelibxkbcommon0 is already the newest version (1.6.0-1build1).

22:32:36smokelibxrandr2 is already the newest version (2:1.5.2-2build1).

22:32:36smokexvfb is already the newest version (2:21.1.12-1ubuntu1.5).

22:32:36smokefonts-noto-color-emoji is already the newest version (2.047-0ubuntu0.24.04.1).

22:32:36smokefonts-unifont is already the newest version (1:15.1.01-1build1).

22:32:36smokelibfontconfig1 is already the newest version (2.15.0-1.1ubuntu2).

22:32:36smokelibfreetype6 is already the newest version (2.13.2+dfsg-1ubuntu0.1).

22:32:36smokexfonts-cyrillic is already the newest version (1:1.0.5+nmu1).

22:32:36smokexfonts-scalable is already the newest version (1:1.0.3-1.3).

22:32:36smokefonts-liberation is already the newest version (1:2.1.5-3).

22:32:36smokefonts-ipafont-gothic is already the newest version (00303-21ubuntu1).

22:32:36smokefonts-wqy-zenhei is already the newest version (0.9.45-8).

22:32:36smokefonts-tlwg-loma-otf is already the newest version (1:0.7.3-1).

22:32:36smokefonts-freefont-ttf is already the newest version (20211204+svn4273-2).

22:32:36smoke0 upgraded, 0 newly installed, 0 to remove and 1 not upgraded.

22:32:36smokeDownloading 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:32:41smoke| | 0% of 170.4 MiB

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

22:32:43smoke|■■■■■■■■■■■■■■■■ | 20% of 170.4 MiB

22:32:44smoke|■■■■■■■■■■■■■■■■■■■■■■■■ | 30% of 170.4 MiB

22:32:44smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 40% of 170.4 MiB

22:32:45smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 50% of 170.4 MiB

22:32:45smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 60% of 170.4 MiB

22:32:46smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 70% of 170.4 MiB

22:32:47smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 80% of 170.4 MiB

22:32:48smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 90% of 170.4 MiB

22:32:49smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■| 100% of 170.4 MiB

22:32:51smokeChrome for Testing 147.0.7727.15 (playwright chromium v1217) downloaded to /workspace/repo-10-danubii-openclaw-orchestrator/web/node_modules/playwright-core/.local-browsers/chromium-1217

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

22:32:57smoke| | 0% of 2.3 MiB

22:32:57smoke|■■■■■■■■ | 10% of 2.3 MiB

22:32:57smoke|■■■■■■■■■■■■■■■■ | 20% of 2.3 MiB

22:32:57smoke|■■■■■■■■■■■■■■■■■■■■■■■■ | 30% of 2.3 MiB

22:32:57smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 40% of 2.3 MiB

22:32:57smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 50% of 2.3 MiB

22:32:57smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 60% of 2.3 MiB

22:32:57smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 70% of 2.3 MiB

22:32:57smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 80% of 2.3 MiB

22:32:57smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 90% of 2.3 MiB

22:32:57smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■| 100% of 2.3 MiB

22:32:57smokeFFmpeg (playwright ffmpeg v1011) downloaded to /workspace/repo-10-danubii-openclaw-orchestrator/web/node_modules/playwright-core/.local-browsers/ffmpeg-1011

22:32:57smokeDownloading 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:33:04smoke| | 0% of 112 MiB

22:33:05smoke|■■■■■■■■ | 10% of 112 MiB

22:33:05smoke|■■■■■■■■■■■■■■■■ | 20% of 112 MiB

22:33:06smoke|■■■■■■■■■■■■■■■■■■■■■■■■ | 30% of 112 MiB

22:33:06smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 40% of 112 MiB

22:33:06smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 50% of 112 MiB

22:33:07smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 60% of 112 MiB

22:33:07smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 70% of 112 MiB

22:33:08smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 80% of 112 MiB

22:33:08smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ | 90% of 112 MiB

22:33:08smoke|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■| 100% of 112 MiB

22:33:09smokeChrome Headless Shell 147.0.7727.15 (playwright chromium-headless-shell v1217) downloaded to /workspace/repo-10-danubii-openclaw-orchestrator/web/node_modules/playwright-core/.local-browsers/chromium_headless_shell-1217

22:33:10smoke> /workspace/repo-10-danubii-openclaw-orchestrator/web/node_modules/.bin/playwright test

22:33:21smokeRunning 2 tests using 1 worker

22:33:28smoke··

22:33:28smoke 2 passed (16.1s)

22:33:33smokeSmokeTest completed with exit code 0.

22:33:33workerQueued ClaudeReview.

22:33:33workerWorker claimed ClaudeReview job cc8fcb4a-53cf-489d-9173-df2596074432.

22:37:18claudeClaudeReview completed with exit code 0.

22:40:31gitReview approved. Queued branch push for 1 repository.

22:40:32workerWorker claimed GitPushBranch job a8bc34f7-e080-4ca0-8dc4-bf5e884afc7c.

22:40:32gitstderr: Cloning into 'repository'...

22:40:33gitstderr: HEAD is now at de4fb97 Allow overriding medium review findings

22:40:37gitstderr: Switched to a new branch 'ai/KAN-10-add-archive-and-delete-actions-for-workflows'

22:40:37git[ai/KAN-10-add-archive-and-delete-actions-for-workflows d79e27c] KAN-10: automated implementation

22:40:37git 27 files changed, 2620 insertions(+), 77 deletions(-)

22:40:37git create mode 100644 src/OpenClaw.Orchestrator.Infrastructure/Persistence/Migrations/20260425204852_AddWorkflowLifecycleMetadata.Designer.cs

22:40:37git create mode 100644 src/OpenClaw.Orchestrator.Infrastructure/Persistence/Migrations/20260425204852_AddWorkflowLifecycleMetadata.cs

22:40:37git create mode 100644 web/features/workflows/workflow-lifecycle-actions.tsx

22:40:37git create mode 100644 web/scripts/run-e2e.mjs

22:40:42gitstderr: remote:

22:40:42gitstderr: remote: Create a pull request for 'ai/KAN-10-add-archive-and-delete-actions-for-workflows' on GitHub by visiting:

22:40:42gitstderr: remote: https://github.com/Danubii/openclaw-orchestrator/pull/new/ai/KAN-10-add-archive-and-delete-actions-for-workflows

22:40:42gitstderr: remote:

22:40:42gitstderr: To https://github.com/Danubii/openclaw-orchestrator.git

22:40:42gitstderr: * [new branch] HEAD -> ai/KAN-10-add-archive-and-delete-actions-for-workflows

22:40:42gitbranch 'ai/KAN-10-add-archive-and-delete-actions-for-workflows' set up to track 'origin/ai/KAN-10-add-archive-and-delete-actions-for-workflows'.

22:40:42gitGitPushBranch completed with exit code 0.

22:42:46systemQueued test deployment to QA Sandbox.

22:42:46workerWorker claimed MockTestDeploy job a20327bb-15f1-4620-bcdd-dc2caf1b861b.

22:42:48deploy10.0.202

22:42:48deploy10.9.7

22:42:48deployMockTestDeploy completed with exit code 0.

22:45:14qaSmoke test critical user path: completed.

22:45:14qaVerify deployment health endpoint: completed.

22:45:14qaCheck generated release notes: completed.

22:45:14qaQA approved. Creating PR handoff records.

22:45:14githubPR handoff records created: https://github.com/danubii/openclaw-orchestrator/pull/1003

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