@hachej/boring-workspace 0.1.36 → 0.1.38

package/dist/server.js

@@ -1294,8 +1294,8 @@ function packagePathContainmentIssues(rootDir, pkg) {
   if (boring?.server !== false && boring?.server !== void 0) {
     push(pathPreflightIssue(rootDir, boring.server, "boring.server"));
   }
-  pi?.extensions?.forEach((value, index) => push(pathPreflightIssue(rootDir, value, `pi.extensions[${index}]`)));
-  pi?.skills?.forEach((value, index) => push(pathPreflightIssue(rootDir, value, `pi.skills[${index}]`)));
+  pi?.extensions?.forEach((value, index) => push(pathPreflightIssue(rootDir, value, `pi.extensions[${index}]`, { mustExist: true })));
+  pi?.skills?.forEach((value, index) => push(pathPreflightIssue(rootDir, value, `pi.skills[${index}]`, { mustExist: true })));
   return issues;
 }
 function discoverBoringPluginDirs(pluginDirs) {
@@ -1617,7 +1617,6 @@ var BoringPluginAssetManager = class {
   pluginDirs;
   errorRoot;
   frontTargetResolver;
-  includeLegacyFrontUrl;
   loaded = /* @__PURE__ */ new Map();
   revisions = /* @__PURE__ */ new Map();
   listeners = /* @__PURE__ */ new Set();
@@ -1628,7 +1627,6 @@ var BoringPluginAssetManager = class {
     this.pluginDirs = options.pluginDirs;
     this.errorRoot = options.errorRoot ?? join5(process.cwd(), ".pi", "extensions");
     this.frontTargetResolver = options.frontTargetResolver;
-    this.includeLegacyFrontUrl = options.includeLegacyFrontUrl ?? true;
   }
   /**
    * Replace the scanned source roots. Lets hosts re-resolve discovery inputs
@@ -1646,6 +1644,17 @@ var BoringPluginAssetManager = class {
   list() {
     return [...this.loaded.values()].map((plugin) => this.toListEntry(plugin));
   }
+  /**
+   * Plugins whose front lifecycle the SSE channel owns. Internal plugins are
+   * app code — their front is statically bundled by the host app and must
+   * never be re-imported through the hot-reload pipeline (a second module
+   * instance would carry a fresh React context identity, breaking
+   * provider ↔ panel lookups). They are loaded server-side (routes, agent
+   * tools, Pi snapshot) but excluded from SSE replay and live events.
+   */
+  listExternal() {
+    return [...this.loaded.values()].filter((plugin) => plugin.source.kind === "external").map((plugin) => this.toListEntry(plugin));
+  }
   getError(pluginId) {
     const path = this.errorPath(pluginId);
     if (!path || !existsSync4(path)) return null;
@@ -1722,9 +1731,7 @@ ${prompts.join("\n\n")}` } : {}
         } catch {
         }
       }
-      const event = { type: "boring.plugin.unload", id, revision };
-      events.push(event);
-      this.emit(event);
+      this.record(events, { type: "boring.plugin.unload", id, revision }, previous?.source);
     }
     for (const plugin of nextPlugins) {
       try {
@@ -1755,22 +1762,18 @@ ${prompts.join("\n\n")}` } : {}
           boring: plugin.boring,
           version: plugin.version,
           revision,
-          ...this.frontUrlPayload(plugin.frontUrl),
           ...frontTarget ? { frontTarget } : {},
           ...requiresRestart.length > 0 ? { requiresRestart } : {}
         };
-        events.push(event);
-        this.emit(event);
+        this.record(events, event, plugin.source);
       } catch (error) {
         const revision = this.bumpRevision(plugin.id);
         const message = error instanceof Error ? error.stack ?? error.message : String(error);
         this.writeError(plugin.id, message);
-        const event = { type: "boring.plugin.error", id: plugin.id, revision, message };
         const loadError = { id: plugin.id, revision, message };
         this.lastErrors.set(plugin.id, loadError);
         errors.push(loadError);
-        events.push(event);
-        this.emit(event);
+        this.record(events, { type: "boring.plugin.error", id: plugin.id, revision, message }, plugin.source);
       }
     }
     return { loaded: this.list(), events, errors };
@@ -1803,16 +1806,14 @@ Plugin dir: ${error.pluginDir}`;
       ...plugin.pi ? { pi: plugin.pi } : {},
       version: plugin.version,
       revision: plugin.revision,
-      ...this.frontUrlPayload(plugin.frontUrl),
       ...plugin.frontTarget ? { frontTarget: plugin.frontTarget } : {}
     };
   }
-  frontUrlPayload(frontUrl) {
-    if (!this.includeLegacyFrontUrl || !frontUrl) return {};
-    return { frontUrl };
-  }
   resolveFrontTarget(plugin, revision) {
-    if (!plugin.frontPath || !this.frontTargetResolver) return void 0;
+    if (!plugin.frontPath) return void 0;
+    if (!this.frontTargetResolver) {
+      return plugin.frontUrl ? { kind: "module-url", entryUrl: plugin.frontUrl, revision } : void 0;
+    }
     const frontEntrySubpath = typeof plugin.boring.front === "string" ? plugin.boring.front.replace(/^\.\//, "") : normalizePluginSubpath(plugin.rootDir, plugin.frontPath);
     const frontTarget = this.frontTargetResolver(plugin, {
       revision,
@@ -1821,6 +1822,16 @@ Plugin dir: ${error.pluginDir}`;
     if (!frontTarget) return void 0;
     return { ...frontTarget, revision };
   }
+  /**
+   * Append to the load result's events array and emit on the SSE channel —
+   * unless the source is internal. Internal plugins are app code: their
+   * events stay in the load result (for /reload diagnostics and restart
+   * warnings) but never reach SSE subscribers (see listExternal).
+   */
+  record(events, event, source) {
+    events.push(event);
+    if (source?.kind !== "internal") this.emit(event);
+  }
   emit(event) {
     for (const listener of [...this.listeners]) {
       try {
@@ -1905,14 +1916,13 @@ async function boringPluginRoutes(app, opts) {
       }
       write(event.type, payload);
     });
-    for (const plugin of manager.list()) {
+    for (const plugin of manager.listExternal()) {
       write("boring.plugin.load", {
         type: "boring.plugin.load",
         id: plugin.id,
         boring: plugin.boring,
         version: plugin.version,
         revision: plugin.revision,
-        ...plugin.frontUrl ? { frontUrl: plugin.frontUrl } : {},
         ...plugin.frontTarget ? { frontTarget: plugin.frontTarget } : {},
         replay: true
       });

package/dist/testing.js

@@ -1,7 +1,7 @@
 import { jsx as Ba } from "react/jsx-runtime";
 import * as Pa from "react";
 import { createElement as is, useMemo as wn, useLayoutEffect as us, isValidElement as ss, cloneElement as ds, useSyncExternalStore as Gi } from "react";
-import { h as cs, q as fs, o as ps } from "./WorkspaceProvider-CpK401MG.js";
+import { h as cs, q as fs, o as ps } from "./WorkspaceProvider-CX_4aV6Z.js";
 import { d as ms } from "./panel-DnvDNQac.js";
 import * as bs from "react-dom/test-utils";
 import ka from "react-dom";

package/dist/workspace.css

@@ -2731,6 +2731,13 @@
     -webkit-user-select: none;
     user-select: none;
   }
+  .group-hover\:inline-flex {
+    &:is(:where(.group):hover *) {
+      @media (hover: hover) {
+        display: inline-flex;
+      }
+    }
+  }
   .group-hover\:text-muted-foreground {
     &:is(:where(.group):hover *) {
       @media (hover: hover) {
@@ -2750,9 +2757,9 @@
       color: var(--accent);
     }
   }
-  .group-data-\[focused\=true\]\:opacity-100 {
+  .group-data-\[focused\=true\]\:inline-flex {
     &:is(:where(.group)[data-focused="true"] *) {
-      opacity: 100%;
+      display: inline-flex;
     }
   }
   .placeholder\:text-muted-foreground\/70 {
@@ -6758,18 +6765,43 @@
   opacity: 0.7;
 }
 
-/* Sash (resize handle) */
+/* Sash (resize handle)
+   Dockview's default 4px grab zone is easy to miss. Keep the visible divider
+   subtle, but expand the pointer target to 12px so pane resizing is forgiving. */
+.dv-shell {
+  --dv-sash-color: transparent;
+  --dv-active-sash-color: transparent;
+  --dv-active-sash-transition-delay: 0s;
+}
+
 .dv-shell .sash-container .sash,
 .dv-shell .dv-sash-container .dv-sash {
-  transition: background-color 0.2s;
+  transition: box-shadow 0.16s cubic-bezier(0.22, 1, 0.36, 1),
+    background-color 0.16s cubic-bezier(0.22, 1, 0.36, 1);
+}
+
+.dv-shell .dv-split-view-container.dv-horizontal > .dv-sash-container > .dv-sash {
+  width: 12px;
+  margin-left: -4px;
+}
+
+.dv-shell .dv-split-view-container.dv-vertical > .dv-sash-container > .dv-sash {
+  height: 12px;
+  margin-top: -4px;
+}
+
+.dv-shell .dv-split-view-container.dv-horizontal > .dv-sash-container > .dv-sash:hover,
+.dv-shell .dv-split-view-container.dv-horizontal > .dv-sash-container > .dv-sash:active {
+  box-shadow: inset 0 0 0 1px oklch(from var(--primary) l c h / 0.45),
+    inset 5px 0 0 oklch(from var(--primary) l c h / 0.35),
+    inset 6px 0 0 oklch(from var(--primary) l c h / 0.35);
 }
 
-.dv-shell .sash-container .sash:hover,
-.dv-shell .sash-container .sash.active,
-.dv-shell .dv-sash-container .dv-sash:hover,
-.dv-shell .dv-sash-container .dv-sash.dv-active {
-  background-color: var(--primary);
-  transition-delay: 0.15s;
+.dv-shell .dv-split-view-container.dv-vertical > .dv-sash-container > .dv-sash:hover,
+.dv-shell .dv-split-view-container.dv-vertical > .dv-sash-container > .dv-sash:active {
+  box-shadow: inset 0 0 0 1px oklch(from var(--primary) l c h / 0.45),
+    inset 0 5px 0 oklch(from var(--primary) l c h / 0.35),
+    inset 0 6px 0 oklch(from var(--primary) l c h / 0.35);
 }
 
 /* Drop target overlay */