dsh-multi-folder 0.2.0 → 0.2.1

package/docs/design.md

@@ -200,9 +200,13 @@ window.__ModuleLoader__.load({
     renders directly ABOVE the composer card, in the same band as the
     git-branch chip. The entry receives the dock owner share (`{ session,
     input }`) plus the standard `useSessions` / `useWorkspaces` selector hooks,
-    so the hero phase (`composerPhase === 'blank' && (openState === 'open' ||
-    blank)`) and the target workspace come from framework props instead of DOM
-    probing. The row stays **in flow** — `display:flex` with the official hero
+    so the hero phase and the target workspace come from framework props instead
+    of DOM probing. Detection is DSH-version-adaptive: a shell whose
+    `SessionSnapshot` carries `composerPhase` uses
+    `composerPhase === 'blank' && (openState === 'open' || blank)`, while DSH
+    0.1.2 (no `composerPhase`) uses the settled-blank fallback
+    `blank && !running && !promptAttempted && (openState === 'open' || blank)`.
+    The row stays **in flow** — `display:flex` with the official hero
     row's 20px indent, no absolute positioning — so the framework's list-slot
     arrangement keeps it clear of every other plugin's dock row. It renders
     only on the session-creation page; an active session keeps its entry in the

package/lib/client.js

@@ -214,12 +214,13 @@ window.__ModuleLoader__.load({
 
     // ------------------------------------------------------------- plugin
     var name = 'dsh-multi-folder';
-    var inject = ['remote', 'remote.commands', 'slots', 'workspaces', 'connection', 'sessions', 'locale'];
+    var inject = ['remote', 'remote.commands', 'slots', 'workspaces', 'uiWorkspace', 'connection', 'sessions', 'locale'];
 
     function apply(ctx) {
       var slots = ctx.slots;
       var remote = ctx.remote;
       var workspaces = ctx.workspaces;
+      var uiWorkspace = ctx.uiWorkspace;
       var connection = ctx.connection;
       var sessions = ctx.sessions;
       var locale = ctx.locale;
@@ -398,7 +399,18 @@ window.__ModuleLoader__.load({
       }
 
       function addDirectory() {
-        workspaces.pickDirectory().then(function (path) {
+        // DSH 0.1.2 moved the directory picker onto `uiWorkspace`; earlier
+        // shells kept it on `workspaces`. Pick whichever service actually
+        // exposes it (and call it with the right receiver), surfacing a clear
+        // error when neither does instead of throwing synchronously.
+        var picker = (uiWorkspace && typeof uiWorkspace.pickDirectory === 'function')
+          ? uiWorkspace
+          : (workspaces && typeof workspaces.pickDirectory === 'function') ? workspaces : null;
+        if (picker === null) {
+          patch({ error: 'multi-folder: the directory picker service is unavailable' });
+          return;
+        }
+        picker.pickDirectory().then(function (path) {
           if (path === null || path === undefined) return;
           var snapshot = getSnapshot();
           if (snapshot.sessionId) {
@@ -763,10 +775,22 @@ window.__ModuleLoader__.load({
       /** The shell's own hero predicate, read from the dock's owner share: a
        *  blank conversation with an open session. A still-loading blank
        *  session already lists as blank, so it may enter the hero phase
-       *  before the composer snapshot settles to `open`. */
+       *  before the composer snapshot settles to `open`.
+       *
+       *  DSH 0.1.2 exposes no `composerPhase` on SessionSnapshot; the hero
+       *  phase there is a settled blank session (mirroring ConversationRoot's
+       *  `hero = sessionId === undefined || (shellPhase === 'blank' && ...)`),
+       *  so the blank/lifecycle fields are used as the fallback. Newer DSH
+       *  shells that do carry `composerPhase` keep their original check. */
       function isHeroPhase(session, blank) {
         if (!session) return false;
-        return session.composerPhase === 'blank' && (session.openState === 'open' || blank === true);
+        if (session.composerPhase !== undefined) {
+          return session.composerPhase === 'blank' && (session.openState === 'open' || blank === true);
+        }
+        return session.blank === true
+          && !session.running
+          && !session.promptAttempted
+          && (blank === true || session.openState === 'open');
       }
 
       /** The workspace path a session belongs to, from the workspaces list. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-multi-folder",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "DeepSeek Harness plugin: secondary working directories for a project. The agent keeps the primary workspace as cwd, gains equal write/exec permissions on configured secondary directories under workspace-write mode, and is notified of configuration changes at the next message boundary. Configurable from the session header AND from the session-creation page (before the first message) through a sessionless multiFolder remote API.",
   "keywords": [
     "dsh-plugin",