docs(bg-agents): reframe roadmap around workstacean integration#229
Merged
Conversation
The original Phase A/B/C plan was written before we committed to making proto a workstacean fleet agent. This update revises the roadmap to match current priorities: - Track 1 (active): SDK surface for workstacean (drives protoCLI#223) - Track 2 (deferred): Phase A reduced — model-facing agent control, in two PRs (housekeeping first, then content port) - Track 3 (dropped): Phase B TUI surface — workstacean owns the dashboard; only /tasks remains as an optional spike - Track 4 (dropped): Phase C cross-session resume — workstacean is the long-running session, not proto Notable concrete decisions captured: - Drop our task-stop.ts and adopt upstream's (Arena/Team unused). - Adopt upstream's tools/agent/ nested layout in PR-1. - Skip QwenLM#3687 — covered by PR-1's reconciliation. - Persistence stays single-JSON since Track 4 is dropped. Adds a 2026-05-03 update banner at the top so readers don't trust the older sections without context. Section numbering preserved. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
WalkthroughDocumentation update revising the background agents design roadmap from three original phases to Track 1–4 structure, prioritizing SDK surface work integration with protoLabs Studio while deferring model-facing agent control and dropping TUI and cross-session resume features. ChangesRoadmap and Cross-Cutting Concerns Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related issues
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Review rate limit: 4/5 reviews remaining, refill in 12 minutes. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/explanation/background-agents-design.md (1)
124-125: ⚡ Quick winUse fully qualified repo/issue references for durable links.
Line 124 (
protoCLI#223,workstacean#516), Line 131 (#219), Line 178 (#3642), and Lines 217–220 (protolabs/fast,protolabs/smart) are clear to maintainers but may not always render as clickable links across contexts. Preferowner/repo#numberand explicit repo links.Suggested doc tweak
-Tracked at protoCLI#223; driven by workstacean#516. +Tracked at protoLabsAI/protoCLI#223; driven by protoLabsAI/workstacean#516. -- Token usage rollup — the new `ui.modelPricing` setting (`#219`) needs +- Token usage rollup — the new `ui.modelPricing` setting (protoLabsAI/protoCLI#219) needs -Keep open the option of porting **just `/tasks`** from `#3642` as a +Keep open the option of porting **just `/tasks`** from protoLabsAI/qwen-code#3642 as a -touches the streaming path needs to be validated against `protolabs/fast` -and `protolabs/smart` before merging. +touches the streaming path needs to be validated against +`protoLabsAI/protolabs-fast` and `protoLabsAI/protolabs-smart` before merging.Also applies to: 131-132, 178-179, 217-220, 226-227
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/explanation/background-agents-design.md` around lines 124 - 125, Replace ambiguous issue and repo shorthands with fully qualified owner/repo#number or full GitHub URLs: update occurrences of "protoCLI#223", "workstacean#516", "#219", "#3642" and repo names "protolabs/fast", "protolabs/smart" to the explicit forms (e.g., owner/repo#223 or https://github.com/owner/repo/issues/223) so links render reliably across contexts; search for these tokens in background-agents-design.md and replace each shorthand with the matching owner/repo#number or canonical https://github.com/owner/repo/issues/<number> or https://github.com/protolabs/fast style URL.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@docs/explanation/background-agents-design.md`:
- Around line 124-125: Replace ambiguous issue and repo shorthands with fully
qualified owner/repo#number or full GitHub URLs: update occurrences of
"protoCLI#223", "workstacean#516", "#219", "#3642" and repo names
"protolabs/fast", "protolabs/smart" to the explicit forms (e.g., owner/repo#223
or https://github.com/owner/repo/issues/223) so links render reliably across
contexts; search for these tokens in background-agents-design.md and replace
each shorthand with the matching owner/repo#number or canonical
https://github.com/owner/repo/issues/<number> or
https://github.com/protolabs/fast style URL.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 00b13ae1-079a-4690-a77b-a5e0a13702c7
📒 Files selected for processing (1)
docs/explanation/background-agents-design.md
Code Coverage Summary
CLI Package - Full Text ReportCore Package - Full Text ReportFor detailed HTML reports, please see the 'coverage-reports-22.x-ubuntu-latest' artifact from the main CI run. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Track changes
Concrete decisions captured
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit