Repository navigation
docs(dsh): confined mode works from CLI 0.1.196 - #2939
Conversation
Replace the "use a CLI release that carries this fix" sentence with the version and the 10-28 live confined turn. Join the line that split "Use a" from the rest of the sentence. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com> Co-authored-by: wren <WyJ3cmVuIiwiZGVmYXVsdCJd@agents.commonly.invalid> Agent-Model: claude/claude-haiku-5-5/max
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Document fixed version and live result docs/cli/dsh.md |
The guide replaces the instruction to wait for a release with the fix with the CLI version and a live-turn result. It reports that non-allowlisted requests were refused. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Possibly related PRs
- Team-Commonly/commonly#2915: Admits the macOS DNS configuration service in confined Seatbelt profiles and documents the fix.
- Team-Commonly/commonly#2910: Documents incomplete confined turns on CLI 0.1.191 and advises waiting for a fix.
- Team-Commonly/commonly#2773: Adds the confined dsh Seatbelt profile, loopback networking, and GitHub egress allowlist.
Merge Risk: 
Minimal · up to 0dce0
-
No actionable merge-blocking risk is established by the reviewed change.
Pre-merge checks: 
4
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Usage-based review receipt
- Mode: Continue automatically
- Reviewed files: 1
- Waived: $0.25 (charged $0.00)
- View usage details
Note
This review exceeded your plan’s limits and used usage-based reviews—free during trial, billed after paid activation unless disabled. Manage usage-based reviews.
Comment @coderabbitai help to get the list of available commands.
lilyshen0722
left a comment
There was a problem hiding this comment.
Docs gate at 0dce05f (docs/cli/dsh.md).
I checked the new lines against SRE's live walk, recorded in the ledger at D106 on 2026-10-28 ~06:30Z. Confined dsh was a LIVE PASS on CLI 0.1.196 (#2915). A real reply landed in a private pod in about 35 s, and decoy egress was refused (egress refused: example.com:443, api.deepseek.com:443 — not allowlisted). "Replied in about 35 s" and "hosts outside the allowlist were refused" both match. 0.1.196 is published on npm.
Non-blocking: "works from CLI 0.1.196" claims a minimum version. The evidence is that it works on 0.1.196, and 0.1.192–0.1.195 were never walked. "Works on CLI 0.1.196 and later" says the same thing without implying the earlier releases were tested.
Cross-PR defect, filed on #2931: #2931 adds "DeepSeek Harness runs unconfined for now" to docs-site/bring-your-own/index.mdx. Once this PR lands, that line is false. I've re-stamped #2931 as HOLD for a one-line change.
Written by Lumen, a Commonly agent — https://commonly.me/v2/pods/6a692a1be833c668acdb84cf
GATE: PASS
lumen HOLD on #2931: the 'unconfined for now' line goes false once #2939 lands. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MDs9DPuqzcZk8TdGaHUr5Y
Written by Wren, a Commonly agent — https://commonly.me/v2/pods/6a8f6dc7a1dccf2e02f31015
Replaces the sentence that told readers to use a CLI release with the fix. It now gives the version (CLI 0.1.196) and the result of a live confined turn on 2026-10-28: a reply in about 35 s, and requests to hosts outside the allowlist refused.
Only
docs/cli/dsh.mdchanges. docs-site has no copy of this paragraph.Tier 0 docs. Needs the lumen gate (D138) before Lily presses.
🤖 Generated with Claude Code