From d29cc740c20e583d2db6b8fffcc0d21bd13f14e3 Mon Sep 17 00:00:00 2001 From: Sawyer Hood Date: Tue, 25 Aug 2026 23:22:19 +0000 Subject: [PATCH 1/2] Disable the Monaco editor by default and rename it to File Editor The builtin from #2127 replaced the read-only file preview for ~86 extensions on every fresh install. Ship it opt-in instead: flip defaultEnabled to false so a fresh database registers it disabled. Existing registrations keep their stored enabled state. Rename the user-facing strings to "File Editor": the manifest bb.name shown in Settings > Plugins and the fileOpener title shown in Settings > File openers. The plugin id monaco-editor and the opener id monaco stay, so existing rows and per-extension opt-outs still match. Co-Authored-By: Claude --- apps/server/src/services/plugins/builtin-registry.ts | 2 +- apps/server/test/services/plugins/builtin-plugins.test.ts | 8 ++++++++ plugins/monaco-editor/app.tsx | 2 +- plugins/monaco-editor/package.json | 2 +- 4 files changed, 11 insertions(+), 3 deletions(-) diff --git a/apps/server/src/services/plugins/builtin-registry.ts b/apps/server/src/services/plugins/builtin-registry.ts index a55dff72a8..71d1e6c649 100644 --- a/apps/server/src/services/plugins/builtin-registry.ts +++ b/apps/server/src/services/plugins/builtin-registry.ts @@ -81,7 +81,7 @@ export const BUILTIN_PLUGINS = [ { name: "monaco-editor", pluginId: "monaco-editor", - defaultEnabled: true, + defaultEnabled: false, category: "Interface", }, { diff --git a/apps/server/test/services/plugins/builtin-plugins.test.ts b/apps/server/test/services/plugins/builtin-plugins.test.ts index f18eab9cec..31ea73a871 100644 --- a/apps/server/test/services/plugins/builtin-plugins.test.ts +++ b/apps/server/test/services/plugins/builtin-plugins.test.ts @@ -463,6 +463,14 @@ describe("builtin plugin reconciliation", () => { expect(pluginApiTester?.defaultEnabled).toBe(false); }); + it("ships the File Editor (monaco-editor) disabled on a fresh database", () => { + const monacoEditor = BUILTIN_PLUGINS.find( + (builtin) => builtin.name === "monaco-editor", + ); + + expect(monacoEditor?.defaultEnabled).toBe(false); + }); + it("ships Workflows disabled on a fresh database", async () => { const workflows = BUILTIN_PLUGINS.find( (builtin) => builtin.name === "workflows", diff --git a/plugins/monaco-editor/app.tsx b/plugins/monaco-editor/app.tsx index 456cda838b..6bee9a0a28 100644 --- a/plugins/monaco-editor/app.tsx +++ b/plugins/monaco-editor/app.tsx @@ -547,7 +547,7 @@ function NoticeAction({ export default definePluginApp((app) => { app.slots.fileOpener({ id: "monaco", - title: "Monaco", + title: "File Editor", extensions: CLAIMED_EXTENSIONS, component: MonacoFileOpener, }); diff --git a/plugins/monaco-editor/package.json b/plugins/monaco-editor/package.json index cfe6259e2f..caa80ba510 100644 --- a/plugins/monaco-editor/package.json +++ b/plugins/monaco-editor/package.json @@ -27,7 +27,7 @@ "bbPluginSdk": ">=0.4.9" }, "bb": { - "name": "Monaco editor", + "name": "File Editor", "description": "Edit files in BB with the Monaco editor instead of the read-only preview.", "branding": { "icon": "Code" From 73b057a504eab1b2dce1188714f3f5bd55b7851a Mon Sep 17 00:00:00 2001 From: Sawyer Hood Date: Tue, 25 Aug 2026 23:27:25 +0000 Subject: [PATCH 2/2] Drop monaco-editor from the package smoke's default-enabled builtins The smoke waits for every default-enabled builtin to reach "running". The File Editor now ships disabled, so the wait timed out on monaco-editor=disabled. Remove it from the list; the list documents itself as the default-enabled set. Co-Authored-By: Claude --- packages/bb-app/scripts/smoke-tarball.mjs | 1 - 1 file changed, 1 deletion(-) diff --git a/packages/bb-app/scripts/smoke-tarball.mjs b/packages/bb-app/scripts/smoke-tarball.mjs index 70565f7609..5b2745eac8 100644 --- a/packages/bb-app/scripts/smoke-tarball.mjs +++ b/packages/bb-app/scripts/smoke-tarball.mjs @@ -33,7 +33,6 @@ const EXPECTED_RUNNING_BUILTIN_PLUGINS = [ "custom-instructions", "inline-vis", "keep-awake", - "monaco-editor", "pdf-preview", "provider-retry", "secrets",