Skip to content

docs: specify Gateway managed mode and its upstream changes - #3977

Draft
stevenlee-oai wants to merge 2 commits into
mainfrom
dev/stevenlee/managed-mode-spec
Draft

stevenlee-oai wants to merge 2 commits into
mainfrom
dev/stevenlee/managed-mode-spec

Conversation

@stevenlee-oai

@stevenlee-oai stevenlee-oai commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Human merge required. Agents must not merge this RFC or enable automatic merging. A human must manually click Merge.

The Gateway managed mode guide explains why OCE enables the mode by default on supported Compute Drivers. A Gateway-local edit leaves OCE’s draft unchanged, so the next deployment overwrites it. The guide shows an admin how to edit, save, and deploy a lasting change from the OCE Console with managed mode on.

The guide identifies the three blocked actions, the exceptions for node terminals and scheduled Agent turns, and what happens to local edits on redeploy in either mode. It links the OpenClaw changes and distinguishes Gateway administration, model tool policy, and filesystem access. RFC 21 points to the guide for shipped behavior.

The docs check validated 296 pages and 4,713 links. The site and search index built, formatting passed, and Chromium rendered the steps and restrictions table and followed the link to the Configuration tab guide.

@clawsweeper

clawsweeper Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

🦞👀
ClawSweeper picked this up.

Pull request received. I will update this pull request when review starts.

ClawSweeper review complete

ClawSweeper finished reviewing this revision. The review result is being finalized.

View the workflow run.

@clawsweeper clawsweeper Bot added clawsweeper:intake:docs Preliminary submission classification; full source review pending. P3 Low-risk cleanup, docs, polish, ergonomics, or speculative feature. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. and removed clawsweeper:intake:docs Preliminary submission classification; full source review pending. labels Oct 28, 2026
@clawsweeper

clawsweeper Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Codex review: needs maintainer review before merge.

What this changes

This PR expands the Gateway managed-mode reference with upstream implementation links, deployment behavior, and enforcement limits.

Example: An operator edits native configuration with gatewayManagedMode=false and then redeploys.

  • Before: The reference explicitly described restoring saved configuration when managed mode was re-enabled.
  • After: The reference explains that redeployment restores OCE configuration in either mode and Kubernetes Pod replacement also discards local configuration edits.

Review scores

Measure Result What it means
Overall readiness 🐚 platinum hermit (4/6) Useful, focused documentation supported by current source and passing head checks, with no actionable findings.
Proof confidence 🌊 off-meta tidepool Not applicable: Documentation-only changes require no new runtime demonstration; the body reports documentation checks, a build, formatting, and rendered-table inspection.
Patch quality 🐚 platinum hermit (4/6) No actionable review findings were identified.

Product

Kind: Docs · Worth it: Yes · Fix scope: Complete
User problem: Operators need a precise account of managed-mode restrictions, local-edit persistence, and the difference between administration controls and model tool policy.
Reason: The update makes existing operator-facing boundaries explicit without adding settings or changing platform contracts.

Merge readiness

✅ Ready for maintainer review

Keep this PR open for maintainer triage: it usefully clarifies existing behavior, and review found no actionable defects.

Priority: P3
Reviewed head: c05199c078c1f6539d52563b3b51487f2ae39178

Before merge

None.

Findings

None.

Agent review details

How this fits together

OCC saves the Agent deployment setting, Compute Drivers launch the Gateway with enforcement variables, and OpenClaw accepts or refuses native administration requests.

flowchart LR
  A[Installation administrator] --> B[Agent draft]
  B --> C[Authorized deployment]
  C --> D[Compute Driver]
  D --> E[Gateway controls]
  F[Native administrator] --> E
Loading

Technical review

Best possible solution:

Land the focused reference update when its author marks this PR ready.

Do we have a high-confidence way to reproduce the issue?

No runtime bug is claimed; source inspection supports the documented controls and deployment boundaries.

Is this the best way to solve the issue?

The existing feature reference is the appropriate owner, and linking it from the RFC preserves historical decisions without duplicating current behavior.

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning medium; reviewed against a55641b2b163.

Provenance checked

  • Gateway managed-mode reference keeps the original intent (110aea3: Expose deployment-level controls while retaining trusted reloads, node terminals, and separate filesystem protection; subsequent changes clarified local tool edits and supported-Driver defaults.)
  • RFC administration companion keeps the original intent (92ee883: Define managed Gateway administration and distinguish the initial restrictions from stronger filesystem guarantees; this PR adds only a link to current supported behavior.)

Security

None.

Evidence

What I checked:

Likely related people:

  • stevenlee-oai: Raw commit 110aea3 adds docs/reference/agents/gateway-managed-mode.md:1 relative to its recorded parents. This identifies author metadata, not feature responsibility or a PR merger. (role: source-line author; confidence: high; commits: 110aea3888bb; files: docs/reference/agents/gateway-managed-mode.md)

Labels

Label changes:

  • add P3: This is a focused documentation clarification with no runtime behavior change.
  • add rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🌊 off-meta tidepool and patch quality is 🐚 platinum hermit.
  • add status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR.

Label justifications:

  • P3: This is a focused documentation clarification with no runtime behavior change.
  • rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🌊 off-meta tidepool and patch quality is 🐚 platinum hermit.
  • status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR.

Rating scale

6/6 🦀 challenger crab · 5/6 🦞 diamond lobster · 4/6 🐚 platinum hermit · 3/6 🦐 gold shrimp · 2/6 🦪 silver shellfish · 1/6 🧂 unranked krab. Overall follows the weaker of proof and patch quality; ✨ marks media proof (a screenshot, video, or linked artifact) that directly shows the changed behavior.

Workflow

ClawSweeper edits this one comment on every review. Comment @clawsweeper re-review for a fresh review only. Repair and merge execution lanes are retired.

Reviewed October 28, 2026, 3:20 PM ET / 19:20 UTC.

This branch has not been deployed

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

Labels

P3 Low-risk cleanup, docs, polish, ergonomics, or speculative feature. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant