migrate: add Tauri shell and Rust core#36
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 28f8ece028
ℹ️ 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".
| .stdin(Stdio::null()) | ||
| .stdout(Stdio::null()) | ||
| .stderr(Stdio::null()); | ||
| command.spawn().map_err(|error| error.to_string())?; |
There was a problem hiding this comment.
Reap spawned URL opener processes
open_external_url spawns rundll32/open/xdg-open and immediately drops the Child handle without waiting or reaping it. On Unix, Rust does not auto-wait dropped children, so each external-link click can leave a zombie until the app exits, which can accumulate in long-running sessions and eventually exhaust process table resources. Please either wait/reap in a background task or use a launch path that does not leave unreaped children.
Useful? React with 👍 / 👎.
Summary
Adds the first Tauri migration slice beside Electron while preserving the React renderer and Python FastAPI backend. Introduces a small Rust local operations core invoked by Python through JSON stdin/stdout.
Changes
Testing
Notes