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
Conversation
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.tsAPI-mapping verification are pending a follow-up commit on this branch.Resolution plan applied
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.🔒 SECURITY HUNKS FOR REVIEW (please verify before merge)
Isolation guards grep-confirmed present post-merge:
buildOwnedChildEnv+ secret-redactor insandbox/ssh.tsassertProductionSshConfigDoesNotMaterializeIdentityDatainsandbox/ssh-backend.tssandbox/docker.tsowner-only-tools.ts,fs-pinned-{path,write}-helper.tsretained (not deleted)gateway/server.impl.tsOne open item:
sandbox/ssh.tssymlink-escape guard — upstream renamed the APIresolveBoundaryPath→resolveRootPath(@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
resolveBoundaryPath→resolveRootPathmapping inssh.ts.pnpm install+ build + typecheck + test; fix breakages (incl. the lintRangeErrora format hook hit on one file).Refs openclaw#1733. Do not merge until the security hunks are reviewed.