Repository navigation
feat(app): restore the archive app and retire the orchestrator - #18
Merged
Merged
Conversation
The script deletes the orchestrator crate, the workbench UI, the task commands, and the in-memory IPC mock. It then checks out the archive shell and UI from 502827d^, the parent of the orchestrator pivot. It keeps src/format.ts because the restored UI imports it, and drops src/diff.ts because nothing does. This commit is the script's output and nothing else. Rerun the script on the parent commit and diff to check it. The tree does not build yet; the next commit holds the hand edits.
Drop lore-orchestrator from the workspace. Point the shell back at lore-core and rusqlite, and remove tauri-plugin-notification with its capability entries, since no restored code sends notifications. Remove the task, workspace, activity, merge queue, repo settings, and decision DTOs from lore-ipc with their generated bindings. Each manifest, the capability file, lore-ipc, and Cargo.lock now match 502827d^ byte for byte. The restored sources needed no edits for the pinned toolchain, clippy, eslint, or tsc.
Remove the four @fontsource-variable fonts, @tauri-apps/plugin-notification, react-markdown, remark-gfm, and sonner. No file under src, index.html, or the build config imports them after the restore. package.json now matches 502827d^, and the lockfile differs from it only by the license field newer npm writes.
This was referenced Oct 26, 2026
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.
Why
Lore is going back to being a local archive of Claude Code and Codex sessions. Commit 502827d swapped the archive app for an agent orchestrator. This PR puts the archive app back and deletes the orchestrator.
What changed
scripts/p1-restore.shdeletes the orchestrator crate, the workbench UI, the task commands, and the IPC mock. It then checks out the archive shell and UI from502827d^.lore-ipcgo back to their502827d^state. That drops the orchestrator DTOs, their bindings, and the notification plugin.Scope
lore-ipc, and the manifests. The whole tree now matches502827d^except the script,README.md,tauri.conf.json, the favicon inindex.html,src/vite-env.d.ts, and one lockfile license field.README.mdandtauri.conf.jsonto their own PR.Tradeoffs
scripts/p1-restore.shonmainand diff against the first commit. A rerun on this branch leaves the working tree clean.archive/orchestrator-providersand at commit0fab97f, which is worth a tag.Blast Radius
The desktop app becomes the archive again, and every orchestrator command is gone.
npm run devshows no sample data until P1-2 adds an archive mock.Verification
cargo fmt --all -- --check,cargo clippy --workspace --all-targets --all-features -- -D warnings, andcargo test --workspacepass, with 834 tests includingno_egress.cargo test -p lore-ipcleavescrates/lore-ipc/bindingsunchanged, andcargo build --locked -p lore-appsucceeds.npm ci,npm run lint,npm run typecheck,npm test(151 tests), andnpm run buildpass.