Skip to content

[guide] docs: document the Claude Remote Control pin-off (#5607) — CHANGELOG v4.1.0 Security + security-model.md Layer 4 - #5628

Merged
kubestellar-prow[bot] merged 1 commit into
v4from
guide/docs-remote-control-pin
Sep 2, 2026
Merged

[guide] docs: document the Claude Remote Control pin-off (#5607) — CHANGELOG v4.1.0 Security + security-model.md Layer 4#5628
kubestellar-prow[bot] merged 1 commit into
v4from
guide/docs-remote-control-pin

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

Documentation Fix

Documents the Claude Remote Control pin-off (#5607, PR #5619, commit e26a861, shipped in v4.1.0), which previously appeared nowhere in the docs (grep -rni "remote control" src/docs docs CHANGELOG.md returned nothing).

Files/cluster claimed: CHANGELOG.md (v4.1.0 ### Security section only) and src/docs/security-model.md (Layer 4 — agent sandboxing, one new bullet). Disjoint from open PRs #5559 (UPGRADE.md), #5625 (pkg/convergence code), #4032 (proxy code).

Changes:

  • CHANGELOG.md — new Security entry under 2026-09-02 (v4.1.0): the bridge auto-start via server-side rollout flag since Claude Code ~2.1.226, the fleet-wide exposure of --dangerously-skip-permissions sessions under the shared claude.ai account, the add-if-missing remoteControlAtStartup: false pin, the operator opt-out (true in /data/home/.claude/settings.json, never clobbered, Info receipt), and the hasUsedRemoteControl launch-time warning.
  • src/docs/security-model.md — new Layer 4 bullet documenting the same mechanism alongside the other per-agent launch controls.

Fixes #5627


Filed by guide agent (ACMM L5 — hold-gated mode). Hold-gated: human review required.

— hive: agent=guide backend=copilot model=claude-sonnet-4-6

…HANGELOG and security model

PR #5619 (e26a861, shipped in v4.1.0) pins the Claude Code Remote
Control bridge off at every claude-CLI launch, closing a fleet-wide
session-exposure vector introduced by a server-side rollout flag in
Claude Code ~2.1.226. The change was user-visible and security-relevant
but appeared nowhere in the docs:

- CHANGELOG.md: add a Security entry to the 2026-09-02 (v4.1.0)
  section describing the exposure, the add-if-missing pin, the operator
  opt-out (set remoteControlAtStartup: true), and the
  hasUsedRemoteControl launch-time warning.
- src/docs/security-model.md: add a Layer 4 (agent sandboxing) bullet
  documenting the pin alongside the other per-agent launch controls.

Fixes #5627

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: sec-check <sec-check@hive.kubestellar.io>
@kubestellar-hive kubestellar-hive Bot added the hold label Sep 2, 2026
@kubestellar-prow kubestellar-prow Bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Sep 2, 2026
@kubestellar-prow kubestellar-prow Bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 2, 2026
@kubestellar-hive kubestellar-hive Bot added documentation Improvements or additions to documentation agent/guide Approved by a Hive merger/owner for auto-merge on green CI hive/hosted-available-oke-11-placeholder-r05x Approved by a Hive merger/owner for auto-merge on green CI labels Sep 2, 2026

@clubanderson clubanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reviewed. Approve. 54 checks, zero failures.

I checked this more carefully than a docs PR would normally warrant, because it documents a security change that already shipped — and the content is right where it matters most.

The never-clobber semantics are stated correctly. The merge is add-if-missing only; an operator who wants the bridge sets remoteControlAtStartup: true in the same file and hive never overrides it. Getting this wrong in either direction would be worse than no documentation — an operator would either believe they had opted out when they had not, or believe hive would override a choice it actually respects.

It explains the mechanism, not just the setting. The bridge auto-starts whenever no explicit value exists because the CLI falls through to a server-side rollout flag — which is why a routine image upgrade could flip it on fleet-wide with no local change, and why the in-pane toggle was not a fix (session-scoped, so every relaunch brought it back). A reader can now reason about their own exposure rather than just copy a setting.

It covers both delivery paths — the shared /data/home/.claude/settings.json for the plain claude backend, and the inference settings seed for gateway agents — and the hasUsedRemoteControl launch-time warning. The Info receipt on an operator-enabled bridge is the right touch: an enabled bridge stays explainable rather than looking like the bug returning.

One correction to something I said on #5627: I flagged the #5607 citation as a dead reference. That was my error — I queried the pulls endpoint, which 404s for an issue number. #5607 is the issue, #5619 is the PR, and citing the issue in a CHANGELOG entry is the right call. No change needed here.

The security-model.md entry sits correctly in Layer 4 alongside the other agent-isolation controls, and reads consistently with its neighbours.

@clubanderson clubanderson added approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. labels Sep 2, 2026
@kubestellar-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubestellar-prow
kubestellar-prow Bot merged commit a8619b0 into v4 Sep 2, 2026
56 checks passed
@kubestellar-prow
kubestellar-prow Bot deleted the guide/docs-remote-control-pin branch September 2, 2026 10:01
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution! Your PR has been merged.

Check out what's new:

Stay connected: Slack #kubestellar-dev | Multi-Cluster Survey

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent/guide Approved by a Hive merger/owner for auto-merge on green CI approved Indicates a PR has been approved by an approver from all required OWNERS files. copilot dco-signoff: yes Indicates the PR's author has signed the DCO. documentation Improvements or additions to documentation hive/hosted-available-oke-11-placeholder-r05x Approved by a Hive merger/owner for auto-merge on green CI hold lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[guide] Claude Remote Control pin-off (#5607, in v4.1.0) is undocumented — no CHANGELOG entry, no security-model coverage

1 participant