Skip to content

refactor: remove legacy desktop runtime facades - #207

Merged
oratis merged 36 commits into
mainfrom
codex/remove-legacy-runtime-facades
Aug 2, 2026
Merged

refactor: remove legacy desktop runtime facades#207
oratis merged 36 commits into
mainfrom
codex/remove-legacy-runtime-facades

Conversation

@oratis

@oratis oratis commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • delete the unused renderer IPC protocol and desktop native mutation/search/bash facade
  • reduce Tauri to sidecar supervision plus credential-safe read-only file and snapshot projections
  • keep legacy session data read-only while making canonical snapshot timestamps interoperable across core and desktop
  • remove the obsolete chat route alias and browser-provider escape hatch, and update architecture docs

Safety

  • no legacy session files are rewritten or deleted
  • credential files remain blocked from renderer file preview
  • workspace mutations remain exclusively behind RuntimeHost and the versioned app-server protocol

Validation

  • pnpm typecheck
  • pnpm test: 1028 passed, 12 skipped
  • pnpm lint: no errors, one pre-existing warning
  • pnpm format:check
  • pnpm docs:check
  • pnpm build
  • cargo test: 30 passed

Stacked on #206.

t added 30 commits August 1, 2026 13:32
@oratis
oratis changed the base branch from codex/conflict-safe-review-revert to main August 2, 2026 06:36
@oratis
oratis marked this pull request as ready for review August 2, 2026 06:47
@oratis

oratis commented Aug 2, 2026

Copy link
Copy Markdown
Owner Author

Review: approved ✅

−1096 lines of dead runtime. Now that #192 moved desktop behind the app server, the renderer IPC protocol and the native mutation/search/bash facade are unreferenced — and unreferenced privileged code is the worst kind to leave lying around, because it stays reachable from the Tauri command surface while nobody reviews it.

What I checked against the PR's own safety claims

Canonical snapshot timestamps being made interoperable across core and desktop is a real fix bundled here — divergent timestamp formats would have broken /rewind interop between CLI and desktop sessions.

Validation: CI green; full suite green locally at the stack tip.

@oratis
oratis merged commit 4d7a9d7 into main Aug 2, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant