refactor(macos): delete orphaned OpenClaw* dirs from RemoteClawKit (#2622)#2623
Merged
alexey-pelykh merged 1 commit intomainfrom Apr 27, 2026
Merged
Conversation
…2622) Three legacy directories under apps/shared/RemoteClawKit/ paralleled the active RemoteClaw* sources but were never declared in Package.swift and had zero importers anywhere in the repo. They were partial-rename leftovers from the macOS native-app rebrand work (PR #124 / #222). Deleted (16 git-tracked files, 3622 lines): - Sources/OpenClawKit/ (6 .swift files, dup of Sources/RemoteClawKit/) - Sources/OpenClawChatUI/ (5 .swift files, dup of Sources/RemoteClawChatUI/) - Tests/OpenClawKitTests/ (5 .swift files, dup of Tests/RemoteClawKitTests/) Spot-checked file pairs are byte-identical with their RemoteClaw* siblings. Package.swift unchanged — already only declared RemoteClaw* targets. scripts/ci/check-rebrand-leakage.sh --all → exit 0. find apps -name '*OpenClaw*' -not -path '*/.build/*' → only the 2 Android Kotlin protocol files (intentional cross-implementation wire-protocol naming retain). AC items requiring `swift build` / `swift test` success against the shared/macos targets are pre-existing broken on main (missing symbols LoopbackHost, GatewayDeviceAuthPayload, GatewayConnectChallengeSupport, ThrowingContinuationSupport — none defined anywhere in apps/, fork-sync debt). The deleted dirs were never compiled, so this deletion is structurally independent of the build state. Closes #2622 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Deletes 16 git-tracked dead-code Swift files in 3 orphaned
OpenClaw*directories under
apps/shared/RemoteClawKit/. The dirs werepartial-rename leftovers from the macOS native-app rebrand work
(PR #124 / #222) — never declared in
Package.swift, never compiled,zero importers.
Sources/OpenClawKit/Sources/RemoteClawKit/Sources/OpenClawChatUI/Sources/RemoteClawChatUI/Tests/OpenClawKitTests/Tests/RemoteClawKitTests/Total: 16 files, -3622 lines.
Relationship to active siblings: The deletion criterion is zero
importers + path absent from
Package.swift, satisfied for all 16files (verified). SHA-256 comparison against
mainshows the 16 filesare a mix:
RemoteClaw*sibling (true stale duplicates)OpenClawChatUI/ChatComposer.swiftis 33 lines behind active)OpenClawKit/TalkConfigParsing.swift,OpenClawKit/AnyCodable+Helpers.swift, threeOpenClawKitTests/*files)The "no active sibling" group are orphan originals from the pre-rebrand
state — never reactivated, never compiled, never tested. Git history
preserves them if recovery is ever needed.
Closes #2622.
Acceptance criteria
git diff --diff-filter=D --name-only main..HEAD | wc -l→ 16git diff main..HEAD -- apps/shared/RemoteClawKit/Package.swiftis emptybash scripts/ci/check-rebrand-leakage.sh --allexits 0 — output:No rebrand leakage detected.find apps -name '*OpenClaw*' -not -path '*/.build/*'returns only the 2 Android Kotlin files —OpenClawProtocolConstants.ktand its test (intentional cross-implementation wire-protocol naming retain, per issue Out-of-scope)Pre-existing build state —
swift build/swift testACsThe original issue lists
cd apps/shared/RemoteClawKit && swift build(and the macos counterpart) as ACs. These commands fail on
mainBEFOREthis deletion with missing-symbols errors:
None of these symbols are defined anywhere in
apps/. The deletedOpenClaw* files USE
LoopbackHost(referenced, same as activeRemoteClawKit/GatewayNodeSession.swift) but do NOT define any of thesesymbols. Verified by:
.swiftfiles inapps/for(struct|enum|class) LoopbackHostetc. — zero definition sitesmainand this branch — identical errors, same filesPackage.swiftand never enter the build graphConclusion: this deletion is structurally independent of the
build state. It cannot break the build (the deleted dirs were never
compiled) and cannot fix it (the missing symbols aren't defined there).
The build break is fork-rebrand-in-progress state from upstream sync,
out of scope for this PR. Swift CI is not enforced in
.github/workflows/,so this state is local-only.
Verification
Test plan
OpenClawKit|OpenClawChatUI|OpenClawKitTestsreferences in tracked filesAI/Vibe-Coded PR
(Opus 4.7, 1M context) using the
/docommand (workflow-do→workflow-investigate→ execute → fresh-context validate +polish subprocess gates →
workflow-submit)(zero importers, byte-identical to active siblings) and that the
4 deletion-dependent ACs all PASS. The 3 build-related ACs require
pre-existing fork-sync repair (out of scope; documented above)
.tmp/workitem-2622.md,.tmp/validate-intent.task.jsonl,.tmp/polish-intent.task.jsonl,.tmp/phase00-polish-intent.task.jsonl(gitignored)
of the source tree
🤖 Generated with Claude Code