fix(app:macos): 【 OpenClaw ⇄ clawdbot 】- Peekaboo Bridge discovery after the OpenClaw rename#6033
Conversation
apps/macos/Sources/OpenClaw/PeekabooBridgeHostCoordinator.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e0ea25ac1f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
apps/macos/Sources/OpenClaw/PeekabooBridgeHostCoordinator.swift
Outdated
Show resolved
Hide resolved
d3613da to
3e310ee
Compare
|
Short update: this fixes a real-world Peekaboo Bridge discovery issue. It keeps OpenClaw as the primary socket path and now repairs the legacy clawdbot symlink if a stale file/dir/wrong link exists. Lint passes (pnpm lint). |
apps/macos/Sources/OpenClaw/PeekabooBridgeHostCoordinator.swift
Outdated
Show resolved
Hide resolved
d111003 to
d7a7b8e
Compare
|
@greptile-apps I addressed both points: the legacy directory now sets explicit 0700 permissions, and symlink target comparison is normalized to avoid repeated recreation. Please take another look. |
Pull Request SummaryThis PR fixes a compatibility issue caused by the OpenClaw rename from "clawdbot". The main problem was that older clients still look for the Peekaboo Bridge socket at the legacy path What this PR does:File changed: Key changes:
Impact:
The implementation is clean and follows Swift best practices with proper error handling and resource management. |
|
Here's an updated PR description that better reflects the actual implementation: SummaryThis PR ensures backward compatibility for the Peekaboo Bridge socket after the OpenClaw rename by implementing automatic legacy symlink management. What Changed
Implementation Details
Backwards CompatibilityThis ensures older clients that still reference the Testing
AI assistance
|
|
This pull request has been automatically marked as stale due to inactivity. |
bfc1ccb to
f92900f
Compare
This comment was marked as spam.
This comment was marked as spam.
…ter the OpenClaw rename (#6033) * fix(mac): keep OpenClaw bridge socket and harden legacy symlink * fix(mac): add clawdis legacy Peekaboo bridge symlink * macos: include moltbot in PeekabooBridge legacy socket paths * changelog: note peekaboo legacy socket compatibility paths --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
…ter the OpenClaw rename (openclaw#6033) * fix(mac): keep OpenClaw bridge socket and harden legacy symlink * fix(mac): add clawdis legacy Peekaboo bridge symlink * macos: include moltbot in PeekabooBridge legacy socket paths * changelog: note peekaboo legacy socket compatibility paths --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
…ter the OpenClaw rename (openclaw#6033) * fix(mac): keep OpenClaw bridge socket and harden legacy symlink * fix(mac): add clawdis legacy Peekaboo bridge symlink * macos: include moltbot in PeekabooBridge legacy socket paths * changelog: note peekaboo legacy socket compatibility paths --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
…ter the OpenClaw rename (openclaw#6033) * fix(mac): keep OpenClaw bridge socket and harden legacy symlink * fix(mac): add clawdis legacy Peekaboo bridge symlink * macos: include moltbot in PeekabooBridge legacy socket paths * changelog: note peekaboo legacy socket compatibility paths --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
…ter the OpenClaw rename (openclaw#6033) * fix(mac): keep OpenClaw bridge socket and harden legacy symlink * fix(mac): add clawdis legacy Peekaboo bridge symlink * macos: include moltbot in PeekabooBridge legacy socket paths * changelog: note peekaboo legacy socket compatibility paths --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
…ter the OpenClaw rename (openclaw#6033) * fix(mac): keep OpenClaw bridge socket and harden legacy symlink * fix(mac): add clawdis legacy Peekaboo bridge symlink * macos: include moltbot in PeekabooBridge legacy socket paths * changelog: note peekaboo legacy socket compatibility paths --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
Summary
Testing
AI assistance