docs: document the isolated session runtime - #53
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
|
Codex review: needs maintainer review before merge. Reviewed September 17, 2026, 4:01 PM ET / 20:01 UTC (Revision 28). ClawSweeper reviewWhat this changesExpands the README command reference and adds an operational guide covering isolated sessions, host execution, gateway management, diagnostics, and cleanup. Merge readiness✅ Ready for maintainer review Keep open: this remains useful documentation for the pending session stack, and no introduced blocking defect was found. The default branch and latest release do not contain this guide. Priority: P3 Review scores
Verification
How this fits togetherThe Windows package launcher routes OpenClaw commands to the host or an owned isolated agent session. Its separate clawctl commands prepare runtimes, manage the session and gateway, and collect diagnostics. flowchart TD
A[Operator commands] --> B[Windows package launcher]
B --> C[Host or isolated session selection]
C --> D[Host runtime]
C --> E[Owned agent session]
B --> F[Setup and lifecycle management]
F --> E
E --> G[OpenClaw CLI and gateway]
D --> G
Before mergeNone. Agent review detailsSecurityNone. Review metricsNone. Technical reviewBest possible solution: Keep this operator guide alongside the corresponding session implementation as the stack lands in dependency order. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR changes documentation; its command and lifecycle claims were checked against the pinned source and focused tests without executing them. Is this the best way to solve the issue? Yes: a concise README reference linked to a dedicated operational guide is a bounded way to explain the implemented stack without changing command semantics. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning medium; reviewed against 685ee93b7ebb. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (27 earlier review cycles; latest 8 shown)
|
bf83ecc to
c4efffd
Compare
c4efffd to
70f301c
Compare
70f301c to
703d552
Compare
703d552 to
9d76759
Compare
9d76759 to
6e9bc0e
Compare
6e9bc0e to
65224b9
Compare
2bbb9cf to
8fba7ba
Compare
d5d2380 to
1ba1ec6
Compare
1ba1ec6 to
27c6977
Compare
3ead480 to
db008c9
Compare
db008c9 to
9faacb9
Compare
9faacb9 to
7419c90
Compare
7419c90 to
9c9059d
Compare
d31bc96 to
e234e6c
Compare
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 97c31f3f-0c7b-43e8-b979-6418dcb7fedc
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 97c31f3f-0c7b-43e8-b979-6418dcb7fedc
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 97c31f3f-0c7b-43e8-b979-6418dcb7fedc
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 97c31f3f-0c7b-43e8-b979-6418dcb7fedc
e234e6c to
753d8c5
Compare
What Problem This Solves
Operators and maintainers need documentation that matches the implemented isolated-session lifecycle and clearly distinguishes host and guest behavior.
User Impact
The README makes lifecycle commands discoverable, and the MXC compatibility evidence explains the backend seam and ownership model that maintain the session boundary.
Why This Change Was Made
The documentation describes the implemented runtime without claiming a published MXC SDK, changing command semantics, or making local Developer Mode deployment runnable.
Documentation corrections
clawctl statusas session-only and directs gateway-state checks toclawctl gateway-service status.Current validation
Current layer head:
753d8c5b275cf0db80f86cd3d960320cdf2c395a. This layer is included in the final integrated stack tipcce2b02f3acd5791654b7a6b1a5a9c27db5ff63brebased onto685ee93b7ebbec1e784205a3544c460bea740e11.Integrated local gates: exact .NET SDK 10.0.100;
Test-DotNetQuality.ps1with 0 warnings/errors; full solution tests 642/642; NativeAOT x64 and ARM64 publishes for both launcher and session host; NativeAOT CLI, deployment, MXC, signing, runtime-input, release-identity, bundle, isolation-plugin, and packaging-relevance policy suites.Live x64 MXC evidence: final-tip Developer Mode deployment registered
OpenClaw.Gateway_0.1.2451.40134_x64__kaa03rpbbqef6from workflow payload run 35191206689;openclaw --versionreturnedOpenClaw 2026.9.4 (3a9d69d);clawctl statusconfirmed the isolated session was running. Earlier final-tip validation also exercised setup, Node.js 24.20.0 reuse, package-qualified activation, detached gateway launch, and redacted diagnostics collection.Signed package evidence: local NativeAOT x64 and ARM64 packages and a multi-architecture bundle were composed and test-signed. Elevated upgrade validation passed all four proof-release transitions (
v0.0.0.0andv0.0.0.1, standalone and bundle), retained package-family LocalState in every transition, and accepted fresh standalone and bundle installs. The temporary certificate and test package were removed, then the Developer Mode registration was restored.Layer 11 of 12. Parent: #52 - feat/session-fresh-reset. Child: #54 - improve/session-local-deployment