Skip to content

Keep Orca runtime metadata owned by the active process#288

Merged
Jinwoo-H merged 2 commits into
mainfrom
Jinwoo-H/fix-runtime-metadata-ownership
Apr 4, 2026
Merged

Keep Orca runtime metadata owned by the active process#288
Jinwoo-H merged 2 commits into
mainfrom
Jinwoo-H/fix-runtime-metadata-ownership

Conversation

@Jinwoo-H

@Jinwoo-H Jinwoo-H commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • only clear orca-runtime.json when the stopping runtime still owns the published metadata
  • stop writing placeholder runtime metadata before the RPC server has a real transport and auth token
  • add regression coverage for overlapping runtime owners sharing the same userData directory

Why

Using multiple Orca instances during development could leave a healthy Orca process running without a discoverable orca-runtime.json file. That made orca status report not_running even though the editor was still alive.

Validation

  • pnpm exec vitest run src/main/runtime/runtime-metadata.test.ts src/main/runtime/runtime-rpc.test.ts
  • pnpm exec tsc --noEmit -p tsconfig.node.json --composite false

@Jinwoo-H Jinwoo-H merged commit 2f6d96b into main Apr 4, 2026
1 check passed
thomaszdxsn pushed a commit to thomaszdxsn/orca that referenced this pull request Jun 15, 2026
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