Skip to content

[AI-assisted] Filter soft-archived cards by default in workboard list CLI#94883

Closed
aniruddhaadak80 wants to merge 1 commit into
openclaw:mainfrom
aniruddhaadak80:fix/workboard-cli-hide-archived
Closed

[AI-assisted] Filter soft-archived cards by default in workboard list CLI#94883
aniruddhaadak80 wants to merge 1 commit into
openclaw:mainfrom
aniruddhaadak80:fix/workboard-cli-hide-archived

Conversation

@aniruddhaadak80

Copy link
Copy Markdown
Contributor

Summary

This PR updates the \openclaw workboard list\ CLI command to filter out soft-archived cards by default (cards where \metadata.archivedAt\ is present). It also introduces a new --include-archived\ option to explicitly show archived cards if desired.

Real behavior proof

  • Setup tested: Local unit test suite.
  • Command run: \pnpm test:extension workboard\
  • Observed result:
    The new unit test case \ ilters out archived cards by default in list command\ passed successfully. Output:
    \
    ✓ extensions extensions/workboard/src/cli.test.ts (5 tests) 23ms
    \\

@openclaw-barnacle openclaw-barnacle Bot added plugin: workboard size: S triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup. labels Jun 19, 2026
@clawsweeper

clawsweeper Bot commented Jun 19, 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 contains the safer JSON-preserving Workboard CLI archived-card fix from the merged canonical PR, with tests, docs, and beta release provenance; this conflicting branch has no unique remaining value.

Root-cause cluster
Relationship: superseded
Canonical: #94562
Summary: The same Workboard CLI archived-card visibility bug is already fixed by the merged canonical PR and present on current main.

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 merged JSON-preserving main implementation and close this stale, conflicting duplicate branch.

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

Yes for the historical bug: v2026.6.10 source shows workboard list only filtered by status while the tool and slash-command paths hid archived cards. Current main no longer reproduces the failure because the text-output filter is already implemented.

Is this the best way to solve the issue?

No, not as a branch to merge now. The general CLI-layer fix was right, but this branch filters JSON output too and current main already has the better tests-and-docs-backed implementation.

Security review:

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

AGENTS.md: found and applied where relevant.

What I checked:

Likely related people:

  • steipete: Merged PR metadata and local history show this account introduced the Workboard CLI, slash command, tests, and docs surfaces involved in the archived-card list behavior. (role: introduced Workboard CLI feature; confidence: high; commits: ed46e62bcc5d, 61031d1b1cec; files: extensions/workboard/src/cli.ts, extensions/workboard/src/command.ts, extensions/workboard/src/tools.ts)
  • ZengWen-DT: This account authored the merged canonical PR that added the JSON-preserving Workboard CLI archived-card fix now present on current main. (role: canonical fix author; confidence: high; commits: 06fe3bab81db, d15e89a83e61; files: extensions/workboard/src/cli.ts, extensions/workboard/src/cli.test.ts, docs/cli/workboard.md)
  • vincentkoc: The merged canonical PR discussion records this account landing the squash commit that fixed the Workboard CLI archived-card behavior. (role: merger of canonical fix; 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 d1b917120a47; fix evidence: release v2026.6.11-beta.1, commit d15e89a83e61.

@clawsweeper clawsweeper Bot added rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. 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. rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. and removed rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. labels Jun 19, 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: S status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask. triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant