@aryacode/dsh-aryacode-polish 0.10.0 → 0.11.2

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Changelog
 
+## [0.11.2] - 2026-10-28
+
+### Verified
+
+- Drag reorder now has live proof, not just unit tests: with two workspaces in the
+  Dev AryaCode desktop, dropping the second workspace on the first reordered them
+  (`workspaces.insertBefore`), and the new order survived a full app restart.
+
+### Fixed
+
+- A session is never promised a move the Host cannot make. The remote workspace
+  namespace exposes `create`, `rename`, `delete`, `insertBefore`,
+  `insertSessionBefore`, `archiveSession`, `unarchiveSession`, `pinSession` and
+  `unpinSession` — and no `attachSession` or `detachSession` — while a workspace's own
+  `insertSessionBefore` only reorders the sessions already inside it (it rejects a
+  session it does not hold). So dropping a session on another workspace is refused
+  with a plain explanation instead of firing a call that fails, and the session menu
+  no longer offers "Move to <workspace>".
+
 ## [0.10.0] - 2026-10-28
 
 ### Added

package/client/workspace.js

@@ -533,7 +533,14 @@ export function BetterWorkspace(props) {
     setDrag(null)
     setDropping(null)
     if (!moving || moving.kind !== 'session' || moving.id === targetId) return
-    if (moving.workspaceId !== workspaceId) { setNotice('A session stays in the workspace it belongs to.'); return }
+    if (moving.workspaceId !== workspaceId) {
+      // A session cannot change workspace from a plugin: the Host's remote
+      // namespace exposes create/rename/delete/insertBefore/insertSessionBefore
+      // and no attachSession or detachSession, and a workspace's own
+      // insertSessionBefore only reorders the sessions already in it.
+      setNotice('A session stays in its workspace; drag it within that workspace to reorder.')
+      return
+    }
     void reorderSession(workspaceId, moving.id, targetId)
   }
 
@@ -605,7 +612,8 @@ export function BetterWorkspace(props) {
           runtime,
           workspacesSnapshot: workspaceState,
           session: { sessionId: menu.row.id, displayTitle: menu.row.displayTitle, session: menu.row },
-        }))))
+        }))),
+      null)
     : null
 
   const children = []

package/docs/better-workspace.md

@@ -70,3 +70,26 @@ are M2/M3; the seat is already the right place for them.
 
 Still owed: "/"-in-title grouping, per-row appearance, workspace presets, and the same
 tree in the Hero picker.
+
+## Why a session cannot change workspace
+
+Drag reorder works inside a workspace and between workspaces, but a *session* cannot
+be dragged into a different workspace, and that is the Host's shape rather than an
+omission here:
+
+- `packages/api/workspace-controller/src/index.ts` exposes these remotes:
+  `create`, `initializeDefault`, `rename`, `delete`, `insertBefore`,
+  `insertSessionBefore`, `archiveSession`, `unarchiveSession`, `pinSession`,
+  `unpinSession` — and nothing that attaches or detaches a session.
+- `packages/api/workspace-controller/src/commands.ts:132` resolves the target workspace
+  and calls its own `insertSessionBefore(sessionId, beforeSessionId)`
+  (`packages/workspace/workspace/src/entity.ts:152`), which reorders that workspace's
+  `sessionIds`; `packages/workspace/workspace/tests/workspace.spec.ts:715` pins that a
+  session the workspace does not hold is rejected.
+- The entity does have `attachSession`/`detachSession`
+  (`packages/workspace/workspace/src/types.ts:134` region), but no remote surfaces them.
+
+So the tree refuses with "A session stays in its workspace; drag it within that
+workspace to reorder." rather than attempting a call that throws. If the Host ever
+exposes attach/detach, the gesture becomes a one-line change here — the drag payload
+already carries `{ kind: 'session', id, workspaceId }`.

package/lib/client.js

@@ -53,7 +53,7 @@ window.__ModuleLoader__.load({
     ]
     const ROW_ID = "polish"
     const ROW_KEY = "@aryacode/dsh-aryacode-polish" + '#' + ROW_ID
-    const CLIENT_VERSION = "0.10.0"
+    const CLIENT_VERSION = "0.11.2"
 
 /* html2canvas 1.4.1 — MIT, vendored from vendor/html2canvas.min.js.
  * A UMD build, so it assigns itself to globalThis; face.js reads it lazily. */
@@ -734,7 +734,14 @@ window.__ModuleLoader__.load({
         setDrag(null)
         setDropping(null)
         if (!moving || moving.kind !== 'session' || moving.id === targetId) return
-        if (moving.workspaceId !== workspaceId) { setNotice('A session stays in the workspace it belongs to.'); return }
+        if (moving.workspaceId !== workspaceId) {
+          // A session cannot change workspace from a plugin: the Host's remote
+          // namespace exposes create/rename/delete/insertBefore/insertSessionBefore
+          // and no attachSession or detachSession, and a workspace's own
+          // insertSessionBefore only reorders the sessions already in it.
+          setNotice('A session stays in its workspace; drag it within that workspace to reorder.')
+          return
+        }
         void reorderSession(workspaceId, moving.id, targetId)
       }
 
@@ -806,7 +813,8 @@ window.__ModuleLoader__.load({
               runtime,
               workspacesSnapshot: workspaceState,
               session: { sessionId: menu.row.id, displayTitle: menu.row.displayTitle, session: menu.row },
-            }))))
+            }))),
+          null)
         : null
 
       const children = []

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aryacode/dsh-aryacode-polish",
-  "version": "0.10.0",
+  "version": "0.11.2",
   "description": "AryaCode Polish: one install for the small things that make AryaCode feel finished. One Plugins row, one switch per quality-of-life feature.",
   "keywords": [
     "dsh",