Skip to content

fix(workboard): hide archived cards in cli list by default#95451

Closed
arkyu2077 wants to merge 1 commit into
openclaw:mainfrom
arkyu2077:pr-factory-94555
Closed

fix(workboard): hide archived cards in cli list by default#95451
arkyu2077 wants to merge 1 commit into
openclaw:mainfrom
arkyu2077:pr-factory-94555

Conversation

@arkyu2077

Copy link
Copy Markdown
Contributor

Summary

  • filter archived workboard cards out of openclaw workboard list by default
  • add an explicit --include-archived escape hatch for callers that want the old full listing
  • align CLI list behavior with the chat command and tool defaults

Testing

  • pnpm vitest run extensions/workboard/src/cli.test.ts

Closes #94555

@openclaw-barnacle openclaw-barnacle Bot added plugin: workboard size: XS triage: needs-pr-context Candidate: external PR body lacks required problem context or evidence. labels Jun 21, 2026
@clawsweeper

clawsweeper Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Thanks for the context here. I did a careful shell check against current main, and this is already implemented.

Close: current main already implements the Workboard CLI archived-card fix through the merged canonical path, and it does so with the safer JSON-compatible behavior that this branch lacks.

Root-cause cluster
Relationship: superseded
Canonical: #94562
Summary: This PR is a duplicate Workboard CLI archived-list fix candidate; the canonical PR has merged and current main now contains the safer JSON-compatible implementation.

Members:

Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything.

So I’m closing this as already implemented rather than keeping a duplicate issue open.

Review details

Best possible solution:

Keep the current main implementation from the merged canonical PR and close this obsolete branch rather than repairing a dirty duplicate.

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

No for current main: the archived-card CLI mismatch is visible in the v2026.6.10 source, but current main has the compact-text filter plus JSON preservation. The original issue remains source-backed against the release tag, not current main.

Is this the best way to solve the issue?

No for this branch as a landing path. The best fix is already on main: filter compact text by default, keep JSON full-list behavior, and document/test that compatibility boundary.

Security review:

Security review cleared: The diff only changes Workboard CLI filtering and a focused unit test; it adds no dependency, workflow, secret, install, package-resolution, or code-execution surface.

AGENTS.md: found and applied where relevant.

What I checked:

Likely related people:

  • steipete: The Workboard CLI, docs, and CLI tests were introduced in the worker-dispatch CLI history. (role: introduced Workboard CLI feature; confidence: high; commits: ed46e62bcc5d, f8b566b9c1df, fee64cfa3edb; files: extensions/workboard/src/cli.ts, extensions/workboard/src/cli.test.ts, docs/cli/workboard.md)
  • ZengWen-DT: Authored the merged Workboard CLI archive-visibility fix that preserved JSON output and updated tests/docs. (role: canonical fix author; confidence: high; commits: c2fec1243ff7, d561242ce065; files: extensions/workboard/src/cli.ts, extensions/workboard/src/cli.test.ts, docs/cli/workboard.md)
  • Wynne668: Committed the merge commit for the canonical Workboard CLI fix. (role: canonical fix merger; confidence: medium; commits: d15e89a83e61; files: extensions/workboard/src/cli.ts, extensions/workboard/src/cli.test.ts, docs/cli/workboard.md)

Codex review notes: model internal, reasoning high; reviewed against 78f7de01c1aa; fix evidence: commit d15e89a83e61, main fix timestamp 2026-06-24T10:57:06+08:00.

@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. labels Jun 21, 2026
@clawsweeper

clawsweeper Bot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper applied the proposed close for this PR.

@clawsweeper clawsweeper Bot closed this Jun 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. P2 Normal backlog priority with limited blast radius. plugin: workboard rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. size: XS status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask. triage: needs-pr-context Candidate: external PR body lacks required problem context or evidence.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: openclaw workboard list CLI does not filter archived cards (CLI/API inconsistency)

1 participant