epic: [Epic] Agent integration & docs#3852
Merged
Merged
Conversation
Add .claude/skills/protomaker-cli.md so agents use the CLI (via Bash + --help discovery) as the primary board interface instead of MCP tools. Documents the full command surface and a feature->PR loop workflow. Co-authored-by: Automaker <automaker@localhost> Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
* feat(docker): install beads_rust `br` CLI in the server image (#3835) Adds the `br` binary (Dicklesworthstone/beads_rust v0.2.11) alongside `gh` in the server runtime stage. SHA256-pinned, multi-arch (amd64 + arm64). Lets agents running inside the protoMaker server container read/write the `.beads/` issue-tracker store mounted from the host repo. Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com> * fix(docker): bump base from Bookworm to Trixie so the `br` binary loads (#3836) Follow-up to #3835. The `beads_rust` v0.2.11 amd64 binary is dynamically linked against GLIBC 2.38/2.39, but `node:22-slim` is Debian Bookworm (glibc 2.36) — running `br --version` in the built image fails with: br: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.38' not found br: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.39' not found `node:22-trixie-slim` ships glibc 2.41, which satisfies both. Trixie is Debian's current stable. All apt packages we install (gh, gosu, Playwright/Chromium deps, etc.) exist in Trixie. Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com> * docs: add CLI how-to guide and command reference Add a how-to walkthrough for driving the board from the CLI and a comprehensive reference covering all command groups and flags. Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com> --------- Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com> Co-authored-by: Automaker <automaker@localhost> Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
|
👀 Quinn is reviewing — verdict (PASS / WARN / FAIL) + findings to follow. |
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis PR introduces a comprehensive CLI documentation suite for protomaker and updates Docker infrastructure. It adds an AI skill descriptor, a practical workflow how-to guide, and a detailed command reference with global flags and exit codes. The Docker configuration upgrades Node.js base images to trixie-slim and installs the beads_rust binary into the server stage. Changesprotomaker CLI Documentation Suite
Docker Base Image and Binary Installation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![protoquinn[bot]](https://avatars.githubusercontent.com/in/3282126?s=60&v=4){kind=small}
There was a problem hiding this comment.
QA Audit — PR #3852 | epic: [Epic] Agent integration & docs
VERDICT: PASS
CI Status
- ci-complete: ✅ success
- test: ✅ success
- build: ✅ success
- deploy: ⏳ in_progress (non-blocking)
- checks: ⏳ in_progress (non-blocking)
Diff Review
- Epic merge rolling up 3 child features: agent skill (#3834), Diataxis docs (#3843), bin-agents
- Main artifact:
.claude/skills/protomaker-cli.md— 200-line agent-facing CLI reference covering feature management, agent lifecycle, PR operations, auto-mode, queue, board/query, context files, and sitrep - Entirely additive; no existing code modified — zero regression risk
- Documentation-only change warrants no additional test coverage
Observations
- CLAWPATCH: unavailable (repo not mounted / invalid base ref) — skipped; N/A given additive doc nature
- No unresolved CodeRabbit threads
- All child feature PRs confirmed complete per epic description
— Quinn, QA Engineer
|
Submitted APPROVE review on #3852. |
Contributor
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.
Epic: [Epic] Agent integration & docs
All child features completed. This PR merges the epic branch into main.
Features included:
Auto-generated by CompletionDetectorService.
Summary by CodeRabbit
Documentation
Chores