Skip to content
This repository was archived by the owner on Jun 22, 2026. It is now read-only.

fix(runtime): catch up openclaw upstream to d1b33a6 (#1733)#99

Merged
saml212 merged 10000 commits into
mainfrom
fleet/openclaw-sync-1733-2026-06-15
Jun 16, 2026
Merged

fix(runtime): catch up openclaw upstream to d1b33a6 (#1733)#99
saml212 merged 10000 commits into
mainfrom
fleet/openclaw-sync-1733-2026-06-15

Conversation

@saml212

@saml212 saml212 commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Catch-up merge: openclaw/openclaw@d1b33a6 → platform-runtime (openclaw#1733)

Fork was a true git-fork 127 ahead / 17,806 behind upstream. This is the curated catch-up merge (real merge-base be8b4dc8), conflicts resolved per a pre-vetted per-file triage.

Status: DRAFT — merge fully resolved (0 unmerged paths, 0 conflict markers), branch pushed. Build/typecheck/test + one ssh.ts API-mapping verification are pending a follow-up commit on this branch.

Resolution plan applied

  • Take-upstream (low fork divergence): ssh-config, ssh-tunnel, settings-runtime, docker-backend (+re-graft owned-env), etc.
  • Keep-ours (11 Rockie isolation surfaces): owner-only-tools.ts, fs-pinned-path-helper.ts, fs-pinned-write-helper.ts (all 3 deleted upstream — kept), sandbox/ssh.ts, ssh-backend.ts, docker.ts, workspace-mounts.ts, tool-policy-audit.ts, tls/gateway.ts, ffmpeg-exec.ts, local-media-access.ts.
  • Careful re-graft (thin Rockie hook over big upstream churn): bash-tools.exec*, process/exec, mcp-stdio-transport, tool-policy-pipeline, gateway/server.impl, gateway/server-node-events, sandbox/browser, supervisor/*, etc.

🔒 SECURITY HUNKS FOR REVIEW (please verify before merge)

Isolation guards grep-confirmed present post-merge:

  • buildOwnedChildEnv + secret-redactor in sandbox/ssh.ts
  • assertProductionSshConfigDoesNotMaterializeIdentityData in sandbox/ssh-backend.ts
  • read-only tenant skill-mount isolation in sandbox/docker.ts
  • owner-only-tools.ts, fs-pinned-{path,write}-helper.ts retained (not deleted)
  • token-auth in gateway/server.impl.ts

One open item: sandbox/ssh.ts symlink-escape guard — upstream renamed the API resolveBoundaryPathresolveRootPath (@openclaw/fs-safe/advanced). The follow-up commit must confirm the call is mapped to the new signature and the boundary check still throws on escape.

Follow-up before ready-for-review

  1. Verify the resolveBoundaryPathresolveRootPath mapping in ssh.ts.
  2. pnpm install + build + typecheck + test; fix breakages (incl. the lint RangeError a format hook hit on one file).
  3. Re-run the isolation-guard self-check; mark PR ready.

Refs openclaw#1733. Do not merge until the security hunks are reviewed.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.