Skip to content

[codex] Integrate production dashboard stack#95339

Closed
SnowBelt wants to merge 7 commits into
openclaw:mainfrom
SnowBelt:codex/dashboard-production-grade-mainline-20260619
Closed

[codex] Integrate production dashboard stack#95339
SnowBelt wants to merge 7 commits into
openclaw:mainfrom
SnowBelt:codex/dashboard-production-grade-mainline-20260619

Conversation

@SnowBelt

@SnowBelt SnowBelt commented Jun 20, 2026

Copy link
Copy Markdown

Summary

Integrates the production-grade dashboard / Codex-style Chat stack onto current main.

Included surfaces:

  • Working Now surface
  • Chat-native Projects
  • Plan Mode cards
  • Pursue Goal UI/state
  • Approval cards
  • Tool / proof / artifact cards
  • Multi-agent work tree
  • Truth / completion diagnostics in Chat and Dashboard
  • Apple-style polish and accessibility
  • Network / remote-proof approval cards

What Problem This Solves

The dashboard stack was previously built and proven on scoped feature branches, but current main did not contain the integrated production dashboard experience. Users could not test the full Codex-style Chat/Dashboard surface from mainline without switching to old feature branches.

This PR ports the proven dashboard stack onto current main, resolves integration regressions, and keeps Control Director truth, Judge-completion, liveness, approval, and completion enforcement intact.

Evidence

Local proof on branch codex/dashboard-production-grade-mainline-20260619 at 0167f7ef76315813ee13875b5b49ffcba7feb928:

  • pnpm test ui/src/ui/views/chat.test.ts ui/src/ui/app-chat.test.ts ui/src/ui/chat/grouped-render.test.ts ui/src/ui/chat/tool-cards.test.ts ui/src/ui/chat/control-director-diagnostics.test.ts
  • pnpm tsgo:test:ui
  • pnpm ui:smoke:chat-work-surface
  • pnpm ui:smoke:chat-projects
  • pnpm ui:smoke:chat-plan-mode
  • pnpm ui:smoke:chat-pursue-goal
  • pnpm ui:smoke:chat-approval-cards
  • pnpm ui:smoke:chat-tool-proof-artifact-cards
  • pnpm ui:smoke:chat-truth-diagnostics
  • pnpm ui:smoke:chat-polish-a11y
  • pnpm ui:smoke:chat-network-remote-approval-cards
  • pnpm check:changed

Remote proof:

Live dashboard proof:

  • Verified the current-branch foreground Gateway served the integrated Control UI and the key dashboard/chat surfaces loaded successfully.

Additional shrinkwrap gate investigation:

  • Local sandbox-only shrinkwrap timeout was traced to sandboxed npm registry/log-path restrictions, not branch code.
  • pnpm deps:shrinkwrap:check passed for all 36 package directories outside the sandbox.
  • Full pnpm check:changed passed outside the sandbox on the final SHA.

Notes

  • This is a draft PR for review. Do not merge without explicit maintainer approval.
  • Android-inclusive CI is not included here; non-Android full CI passed.

@openclaw-barnacle openclaw-barnacle Bot added docs Improvements or additions to documentation channel: discord Channel integration: discord channel: msteams Channel integration: msteams channel: slack Channel integration: slack channel: telegram Channel integration: telegram channel: whatsapp-web Channel integration: whatsapp-web channel: zalouser Channel integration: zalouser app: web-ui App: web-ui gateway Gateway runtime extensions: diagnostics-otel Extension: diagnostics-otel scripts Repository scripts commands Command implementations agents Agent runtime and tooling channel: feishu Channel integration: feishu extensions: acpx size: XL triage: dirty-candidate Candidate: broad unrelated surfaces; may need splitting or cleanup. triage: needs-pr-context Candidate: external PR body lacks required problem context or evidence. labels Jun 20, 2026
@clawsweeper

clawsweeper Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Thanks for the contribution. This PR still needs the requested real-behavior proof, and the branch has been idle since that ask.

Close as stalled-unproven: this external PR has had a current real-behavior-proof ask open since June 20, 2026, with no new head commit or check activity after June 20, and the branch remains dirty with prior proof and protocol blockers still visible.

So I’m closing this for now to keep the review queue honest. Please reopen or open a fresh PR with real-behavior proof (a live run, logs, or a reproducible validation transcript) and it will be reviewed again.

Review details

Best possible solution:

Close this stale branch and let any revived dashboard work come back as smaller, current-base PRs with current-head real behavior proof and clean protocol/release-owned-file handling.

Do we have a high-confidence way to reproduce the issue?

Yes for the patch blockers: source inspection on the PR head shows the changelog rollback and task-flow schema/handler mismatches. The user-visible dashboard/chat behavior itself is not reproduced or proven by inspectable current-head artifacts.

Is this the best way to solve the issue?

No; the dashboard direction may be useful, but this broad conflicting branch is not the best landing shape after the proof ask expired. A smaller staged implementation with clean protocol contracts and current-head proof is the safer path.

Security review:

Security review cleared: No concrete malicious or supply-chain defect was identified, but workflow and approval/proof boundary changes remain merge-sensitive risks if the work is revived.

AGENTS.md: found and applied where relevant.

What I checked:

  • Live PR state and stale proof gate: Live GitHub shows the PR is open, external, unassigned, not draft, mergeable=CONFLICTING, mergeStateStatus=DIRTY, current head 63216a7, with labels including status: needs proof and rating: unranked krab. (63216a766d7b)
  • No recent head or check activity: The latest PR commit is 63216a7 at 2026-06-20T17:32:33Z and the latest status-check completion in the live rollup is 2026-06-20T19:55:17Z, both more than 14 days before this review on 2026-07-07. (63216a766d7b)
  • Proof ask age: The current status: needs proof label was re-applied at 2026-06-20T17:32:38Z after the final force-push cycle, and no current maintainer review, assignee, or requested reviewer is visible. (63216a766d7b)
  • Prior proof request remains unanswered: The existing ClawSweeper review says the PR lacks inspectable current-head screenshots, recordings, terminal output, logs, or artifacts showing the dashboard/chat changes running after the fix. (63216a766d7b)
  • Task-flow list contract still mismatches handler output: The PR head still maps taskFlows.list entries through mapTaskFlowDetail, so list responses include detail fields even though the list schema declares TaskFlowSummary entries. (src/gateway/server-methods/tasks.ts:292, 63216a766d7b)
  • Task-flow detail schema still uses closed intersect branches: The PR head defines TaskFlowDetailSchema as an intersection of two additionalProperties:false object schemas, keeping the prior generated-client/schema-validator compatibility concern live. (packages/gateway-protocol/src/schema/tasks.ts:145, 63216a766d7b)

Likely related people:

  • steipete: Authored and merged recent Control UI direction and plugin-tab seam work that the related modular-dashboard proposal explicitly builds on. (role: recent area contributor; confidence: medium; commits: bb724436150e, c730d8f1f1bb; files: ui/, src/gateway/methods/core-descriptors.ts)
  • xingzhou: Local blame on the current checked-out main attributes the central task/protocol/workflow/changelog files sampled in this sparse checkout to the recent Codex runtime commit, making this a useful routing hint but not a full ownership conclusion. (role: recent adjacent contributor; confidence: low; commits: f14eb97ae7c5; files: src/gateway/server-methods/tasks.ts, packages/gateway-protocol/src/schema/tasks.ts, .github/workflows/workflow-sanity.yml)
  • Peter Steinberger: Current main history and blame show recent top-of-changelog and repository-head work, relevant to the release-owned changelog rollback risk. (role: recent release/changelog contributor; confidence: medium; commits: b95b9e448794; files: CHANGELOG.md)

Codex review notes: model internal, reasoning high; reviewed against b95b9e448794.

@openclaw-barnacle openclaw-barnacle Bot removed the triage: needs-pr-context Candidate: external PR body lacks required problem context or evidence. label Jun 20, 2026
@clawsweeper clawsweeper Bot added rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask. P2 Normal backlog priority with limited blast radius. merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. merge-risk: 🚨 availability 🚨 May cause crashes, hangs, restart loops, stalls, or process outages. merge-risk: 🚨 automation 🚨 May affect CI, automerge, proof capture, label sync, or maintainer automation. labels Jun 20, 2026
@SnowBelt
SnowBelt force-pushed the codex/dashboard-production-grade-mainline-20260619 branch from 0167f7e to 18da6de Compare June 20, 2026 15:03
@openclaw-barnacle openclaw-barnacle Bot removed docs Improvements or additions to documentation channel: discord Channel integration: discord channel: msteams Channel integration: msteams labels Jun 20, 2026
@openclaw-barnacle openclaw-barnacle Bot added extensions: diagnostics-otel Extension: diagnostics-otel commands Command implementations channel: feishu Channel integration: feishu extensions: acpx labels Jun 20, 2026
@github-actions github-actions Bot added the dependencies-changed PR changes dependency-related files label Jun 20, 2026
@github-actions

github-actions Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Dependency graph guard cleared

This PR no longer has blocked dependency graph changes. A future dependency graph change requires a fresh /allow-dependencies-change comment after the guard blocks that new head SHA.

  • Current SHA: 63216a766d7bc20da500a887ad668951cb0a881e

@SnowBelt
SnowBelt force-pushed the codex/dashboard-production-grade-mainline-20260619 branch from 0167f7e to 874e409 Compare June 20, 2026 17:03
@openclaw-barnacle openclaw-barnacle Bot removed docs Improvements or additions to documentation channel: discord Channel integration: discord channel: msteams Channel integration: msteams channel: slack Channel integration: slack channel: telegram Channel integration: telegram channel: whatsapp-web Channel integration: whatsapp-web channel: zalouser Channel integration: zalouser extensions: diagnostics-otel Extension: diagnostics-otel commands Command implementations channel: feishu Channel integration: feishu extensions: acpx labels Jun 20, 2026
@github-actions github-actions Bot removed the dependencies-changed PR changes dependency-related files label Jun 20, 2026
@clawsweeper clawsweeper Bot removed the rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. label Jun 20, 2026
@clawsweeper

clawsweeper Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper applied the proposed close for this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent runtime and tooling app: web-ui App: web-ui gateway Gateway runtime merge-risk: 🚨 automation 🚨 May affect CI, automerge, proof capture, label sync, or maintainer automation. merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. merge-risk: 🚨 security-boundary 🚨 May affect sandboxing, authorization, credentials, or sensitive data. P2 Normal backlog priority with limited blast radius. rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. scripts Repository scripts size: XL status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask. triage: dirty-candidate Candidate: broad unrelated surfaces; may need splitting or cleanup.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant