Skip to content

docs(dsh): confined mode works from CLI 0.1.196 - #2939

Merged
lilyshen0722 merged 1 commit into
mainfrom
wren/dsh-confined-cli-0196
Oct 28, 2026
Merged

lilyshen0722 merged 1 commit into
mainfrom
wren/dsh-confined-cli-0196

Conversation

@lilyshen0722

Copy link
Copy Markdown
Contributor

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.md changes. docs-site has no copy of this paragraph.

Tier 0 docs. Needs the lumen gate (D138) before Lily presses.

🤖 Generated with Claude Code

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
@coderabbitai

coderabbitai Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 7d091a4 and 0dce05f.

Files selected for processing: 1
  • docs/cli/dsh.md

Limit details: You’ve used all 10 included reviews currently available.

Walkthrough

The confinement guide now identifies CLI 0.1.196 as including the resolver-service fix. It records a successful confined turn on 2026-10-28 and notes that requests to non-allowlisted hosts were refused.

Changes: Confined mode documentation

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: Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Possibly related PRs

Merge Risk: Minimal · up to 0dce0

No actionable merge-blocking risk is established by the reviewed change.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title clearly and concisely states the main documentation change: confined mode works from CLI 0.1.196.
Description check Passed The description accurately describes the documentation update, the CLI version, the live test result, and the affected file.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
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 lilyshen0722 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

samxu01 pushed a commit that referenced this pull request Oct 28, 2026
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
@lilyshen0722
lilyshen0722 added this pull request to the merge queue Oct 28, 2026
Merged via the queue into main with commit 12a3260 Oct 28, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant