Skip to content

[codex] Remove executor jsonc source sync#807

Merged
RhysSullivan merged 1 commit into
mainfrom
rs/kill-executor-jsonc
May 13, 2026
Merged

[codex] Remove executor jsonc source sync#807
RhysSullivan merged 1 commit into
mainfrom
rs/kill-executor-jsonc

Conversation

@RhysSullivan
Copy link
Copy Markdown
Owner

Summary

Remove the local app's executor.jsonc source replay and write-back path while preserving executor.jsonc plugin-manifest loading.

What changed

  • Stop creating and passing a ConfigFileSink into local plugin constructors, so source add/remove operations no longer mirror into executor.jsonc.
  • Continue loading plugins from executor.jsonc#plugins at runtime and in the Vite plugin-client bundling path.
  • Delete the boot-time config source sync module and tests that only covered source replay.
  • Update the desktop smoke test to register its OpenAPI source through executor tools instead of writing source JSONC.
  • Refresh stale desktop/MCP UI copy that implied sources should be edited in executor.jsonc.

Impact

executor.jsonc can still be used for plugin parsing, but it is no longer treated as a source of truth for sources on local startup or source mutation. Source state stays in storage.

Validation

  • bun run --filter @executor-js/local test -- src/server
  • bun run --filter @executor-js/local typecheck
  • bun run --filter @executor-js/desktop typecheck
  • bun run --filter @executor-js/plugin-mcp typecheck

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 13, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
executor-marketing c1c5ded Commit Preview URL

Branch Preview URL
May 13 2026, 06:10 PM

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 13, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
executor-cloud c1c5ded May 13 2026, 06:10 PM

@RhysSullivan RhysSullivan force-pushed the rs/kill-executor-jsonc branch from dfe53ae to c1c5ded Compare May 13, 2026 18:08
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 13, 2026

Open in StackBlitz

@executor-js/cli

npm i https://pkg.pr.new/@executor-js/cli@807

@executor-js/config

npm i https://pkg.pr.new/@executor-js/config@807

@executor-js/execution

npm i https://pkg.pr.new/@executor-js/execution@807

@executor-js/sdk

npm i https://pkg.pr.new/@executor-js/sdk@807

@executor-js/storage-core

npm i https://pkg.pr.new/@executor-js/storage-core@807

@executor-js/codemode-core

npm i https://pkg.pr.new/@executor-js/codemode-core@807

@executor-js/runtime-quickjs

npm i https://pkg.pr.new/@executor-js/runtime-quickjs@807

@executor-js/plugin-file-secrets

npm i https://pkg.pr.new/@executor-js/plugin-file-secrets@807

@executor-js/plugin-google-discovery

npm i https://pkg.pr.new/@executor-js/plugin-google-discovery@807

@executor-js/plugin-graphql

npm i https://pkg.pr.new/@executor-js/plugin-graphql@807

@executor-js/plugin-keychain

npm i https://pkg.pr.new/@executor-js/plugin-keychain@807

@executor-js/plugin-mcp

npm i https://pkg.pr.new/@executor-js/plugin-mcp@807

@executor-js/plugin-onepassword

npm i https://pkg.pr.new/@executor-js/plugin-onepassword@807

@executor-js/plugin-openapi

npm i https://pkg.pr.new/@executor-js/plugin-openapi@807

executor

npm i https://pkg.pr.new/executor@807

commit: dfe53ae

@RhysSullivan RhysSullivan marked this pull request as ready for review May 13, 2026 18:12
@RhysSullivan RhysSullivan merged commit 05d456c into main May 13, 2026
10 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