dsh-composer-plus 0.6.0 → 0.6.2

package/README.md

@@ -1,57 +1,85 @@
 # dsh-composer-plus
 
-Small composer helpers for [DeepSeek Harness](https://github.com/deepseek-ai/deepseek-harness) (web / desktop UI). The client half adds the UI; a small Host half serves read-only git changes and saves Auto Review grants. No external network calls.
+[![npm](https://img.shields.io/npm/v/dsh-composer-plus)](https://www.npmjs.com/package/dsh-composer-plus)
+[![test](https://github.com/furkancak1r/dsh-composer-plus/actions/workflows/test.yml/badge.svg)](https://github.com/furkancak1r/dsh-composer-plus/actions/workflows/test.yml)
+[![license: MIT](https://img.shields.io/badge/license-MIT-blue)](LICENSE)
 
-- **Input history** – in an empty composer, `↑` / `↓` walk your previously sent messages, like a terminal. An open `/` menu or an edited draft keeps the normal keys.
-- **`/plan` toggle** – `Shift+Tab` adds `/plan ` to the start of the draft, even when it already has text; press it again to remove that leading `/plan`. The rest of the draft, file references and attachments stay intact, and the caret ends up at the end. It never sends. An open `/` or `@` menu keeps `Shift+Tab` for closing itself.
-- **One-click Continue** – when a turn has stopped and the composer is empty, the Send button becomes **▶**; one click sends `continue`. Hidden on a blank chat, while running, in subagent views, or once you type or attach something.
-- **Copy deeplink** – right-click a session in the sidebar (or use its `…` menu) and pick **Copy deeplink** (label follows dsh's language: English or Chinese). You get a link like `dsh-session:InNlc3Npb24t…`; paste it into another chat and dsh attaches that session as read-only context.
-- **Allow for this chat (Auto Review)** – when Auto Review denies a call and asks you, the approval card gets an **Allow for this chat** button next to Reject / Allow once. It allows this call and every later Auto Review ask for the **same exact tool in the same chat**, with any arguments (e.g. any read-only SQL query on one database tool). Other tools, other chats and non-Auto-Review approvals still ask. A header chip (“N tool(s) allowed · Revoke”) lists the grants and revokes them in one click. Grants are saved by the plugin's Host half (`$DSH_HOME/storages/dsh-composer-plus/grants.json`, user-only permissions), so they survive a restart until you revoke them. If the tool can write, the grant covers any write too.
-- **Changes panel** – a Codex-style **Changes** page in the right sidebar (open it from the sidebar's `+` guide). Pick **Uncommitted**, **Unstaged**, **Staged** or **Branch** (your branch vs. its upstream, from their merge base) to list changed files with `+/-` counts and expand any file for its diff; **Last Turn** opens dsh's own review of the latest turn. It refreshes while visible. Git runs read-only on the Host in the chat's own folder.
-- **Background notifications** – while the dsh window is not focused, you get an OS notification titled with the chat name when a turn finishes (with the reply's first line), or when a chat waits for an approval (names the tool), a question (shows it) or a plan review. The Dock/taskbar badge counts chats waiting for you. Clicking it brings dsh to the front and opens that chat. Notifications stay in Notification Center until clicked; a newer one for the same chat replaces the older. Subagent chats don't notify on their own (the parent chat does). Text follows dsh's language (English or Chinese).
-- **Drag-and-drop queue reordering** – queued messages get a grip (⠿) at the left. Drag a row to a new position (drop line shown, `Esc` cancels), or focus the grip and press `↑` / `↓`.
+Codex-style quality-of-life features for the [DeepSeek Harness](https://github.com/deepseek-ai/deepseek-harness) (dsh) desktop and web UI: a Changes panel, input history, a plan-mode shortcut, one-click Continue, queue drag-and-drop, session deeplinks, per-chat Auto Review grants and background notifications.
 
-Tested on dsh `0.2.0-rc.2` (desktop and web) and `0.2.1-alpha.2` (web). CI runs the tests on every push.
+Unofficial community plugin. No external network calls.
 
 ## Install
 
-**Desktop app:** Settings → Plugins → **Add plugin**, paste
+**Desktop app:** Settings → Plugins → **Add plugin** → enter `dsh-composer-plus` → **Enable now**.
+You can also paste `https://github.com/furkancak1r/dsh-composer-plus` instead of the npm name.
 
-```
-https://github.com/furkancak1r/dsh-composer-plus
+**Web / CLI profile:**
+
+```sh
+dsh plugin --profile web add dsh-composer-plus
 ```
 
-then **Enable now** (or restart dsh). (The desktop profile is managed by the app, so the CLI cannot install into it.)
+Then restart dsh. (The CLI can't install into the desktop profile; the app manages it.)
 
-**Web / CLI profiles:**
+> dsh's pnpm skips versions younger than 24 hours. Right after a release, add the version: `dsh-composer-plus@0.6.2`.
 
-```sh
-dsh plugin --profile web add https://github.com/furkancak1r/dsh-composer-plus
-```
+## Features
+
+| Feature | How to use it |
+|---|---|
+| **Changes panel** | Right sidebar → `+` → **Changes**. Pick **Uncommitted**, **Unstaged**, **Staged** or **Branch** to see changed files with `+/-` counts; click a file for its diff. **Last Turn** opens dsh's own review of the latest turn. |
+| **Input history** | In an empty composer, press `↑` / `↓` to walk through messages you sent, like a terminal. |
+| **Plan mode shortcut** | `Shift+Tab` adds `/plan ` to the start of the draft (press again to remove it). If the chat is already in plan mode, `Shift+Tab` leaves it. |
+| **One-click Continue** | When a turn has stopped and the composer is empty, Send turns into **▶**. One click sends `continue`. |
+| **Queue drag-and-drop** | Queued messages get a grip (⠿). Drag to reorder, or focus the grip and press `↑` / `↓`. `Esc` cancels. |
+| **Copy deeplink** | Right-click a chat in the sidebar → **Copy deeplink**. Paste the `dsh-session:…` link into another chat to attach that chat as read-only context. |
+| **Allow for this chat** | When Auto Review asks you, the approval card gets **Allow for this chat**: the same tool in the same chat is allowed from then on, with any arguments. A header chip lists the grants and revokes them. Grants survive a restart. |
+| **Background notifications** | While dsh is in the background you get a notification when a turn finishes (first line of the reply), or when a chat needs an approval (tool name), an answer (the question) or a plan review. The Dock/taskbar badge counts waiting chats. Clicking opens that chat. |
+
+Labels follow dsh's language (English or Chinese).
+
+## Keyboard shortcuts
 
-This adds the dependency and the profile bundle. Restart dsh.
+| Key | Where | Action |
+|---|---|---|
+| `↑` / `↓` | empty composer | previous / next sent message |
+| `Shift+Tab` | composer | add/remove `/plan `, or leave plan mode |
+| `↑` / `↓` | focused queue grip | move the queued message |
+| `Esc` | while dragging | cancel the move |
 
-## How it works / limits
+An open `/` or `@` menu keeps these keys for itself.
 
-- dsh's `session.updateQueue` has no `move` action, so a reorder rotates the affected rows' text with `edit` calls (tail → head). If the agent claims a row mid-move, the applied edits are reverted and a notice is shown; nothing is lost or duplicated.
-- Rows with images/files can't be moved or crossed (queue `edit` is text-only).
-- A deeplink uses dsh's `dsh-session:` reference format. It references a session inside chats; clicking it does not open the session (dsh's `dsh://` handler only focuses the app today).
-- Session rows have no context-menu slot, so right-click finds the row's own `…` button in the DOM. If dsh changes that markup, right-click shows the normal menu again; the `…` menu item keeps working.
-- The Send button and queue rows are not slots, so ▶ and the grips are portaled into the existing DOM. If a dsh update changes that markup, they simply stop appearing; nothing else breaks.
-- “Allow for this chat” answers the existing approval request with dsh's own `allowed-once` outcome; Auto Review still runs on every call, and review failures are never auto-allowed. The button is portaled into the approval card's button row (no slot exists); if dsh changes that markup, the button disappears and Allow once / Reject keep working.
-- Notifications use the web `Notification` API, which the desktop app allows; macOS asks for permission on the first one. Banners disappear after a few seconds by default; set the app's style to **Alerts** in System Settings → Notifications to keep them on screen. A notification sent before a restart only brings dsh to the front when clicked.
-- A native `move` action is requested in [#773](https://github.com/deepseek-ai/deepseek-harness/discussions/773); a built-in Continue state in [#9269](https://github.com/deepseek-ai/deepseek-harness/discussions/9269).
+## Compatibility
 
-## Tests
+Tested on dsh `0.2.0-rc.2` (desktop and web) and `0.2.1-alpha.2` (web). CI runs the tests on every push.
+
+## Privacy and security
+
+- **Changes panel:** git runs read-only on your machine, in the chat's own folder, through dsh's authenticated `/api` channel. It never commits, stages or fetches, and only shows paths git itself listed.
+- **Allow for this chat:** grants are stored in `$DSH_HOME/storages/dsh-composer-plus/grants.json` (owner-only permissions). A grant covers **one exact tool in one chat**. If that tool can write, the grant covers any write too. Revoke it from the header chip.
+- Auto Review still runs on every call; review failures are never auto-allowed.
+
+## Troubleshooting
+
+- **No notifications on macOS:** System Settings → Notifications → DeepSeek Harness → allow. Choose **Alerts** to keep them on screen.
+- **▶, queue grips, Allow for this chat or right-click missing after a dsh update:** these attach to dsh's current HTML because dsh has no slot for them yet. If dsh changes that markup they hide themselves; nothing else breaks. Please open an [issue](https://github.com/furkancak1r/dsh-composer-plus/issues).
+- **Install fails right after a release:** add the version (`dsh-composer-plus@0.6.2`), see the install note above.
+
+## Limits
+
+- dsh's queue has no `move` action, so a reorder rewrites the affected rows' text. If the agent picks up a row mid-move, the move is undone and nothing is lost. Messages with images or files can't be moved.
+- A deeplink works inside chats; clicking it does not open the chat.
+- A notification sent before a restart only brings dsh to the front when clicked.
+- Built-in support is requested upstream: queue `move` in [#773](https://github.com/deepseek-ai/deepseek-harness/discussions/773), a Continue state in [#9269](https://github.com/deepseek-ai/deepseek-harness/discussions/9269).
+
+## Development
 
 ```sh
 npm test
 ```
 
-CI runs the same tests on every push.
-
-Runs the real `client.js` against a fake React/DOM (/plan toggle, deeplink format and right-click, Continue, queue drag, Auto Review grants including persistence across a restart, notification bodies and Dock badge) and the real `index.js` against a temporary git repository (every Changes mode, rename parsing, untracked files, refusal of paths git did not list, grants file).
+The tests run the real `client.js` against a fake React/DOM and the real `index.js` (Host half) against a temporary git repository. `node test/browser-history.mjs '<dsh web URL with ?token=…>'` checks input history in a real browser against a throwaway `dsh web` with its own `DSH_HOME`.
 
-`node test/browser-history.mjs '<dsh web URL with ?token=…>'` checks ↑/↓ history in a real browser against a throwaway `dsh web` (it sends test messages, so use a separate `DSH_HOME`). The `/plan` toggle (empty, multi-line, references, attachments, caret position), Continue and drag-and-drop were also checked by hand-scripted browser runs against a slow fake model.
+## License
 
-MIT License. Unofficial community plugin.
+MIT

package/client.js

@@ -96,12 +96,16 @@ window.__ModuleLoader__.load({
     /** Shift+Tab on every platform. */
     const isPlanKey = (event) => event.key === 'Tab' && event.shiftKey && !event.altKey && !event.ctrlKey && !event.metaKey;
 
-    /** The plan key toggles a leading `/plan` and leaves the caret at the end. Rides `conversation.input.right`
-     * because, unlike the composer dock, it also renders on a blank chat. */
-    function PlanToggle({ useInput, inputActions }) {
+    /** The plan key leaves plan mode when the chat is in it (as dsh's plan chip does: `/plan off`); otherwise it
+     * toggles a leading `/plan` and leaves the caret at the end. Rides `conversation.input.right` because, unlike
+     * the composer dock, it also renders on a blank chat. */
+    function PlanToggle({ useInput, inputActions, useProjection, exitPlanMode, notify }) {
       const draft = useInput((state) => (state === undefined ? '' : state.draft));
+      const plan = typeof useProjection === 'function' ? useProjection('plan') : undefined;
+      // Same effective target as dsh's PlanChip: a pending switch flips the folded host value.
+      const planOn = plan !== undefined && (plan.pending ? !plan.active : plan.active);
       const live = React.useRef(null);
-      live.current = { draft, inputActions };
+      live.current = { draft, inputActions, planOn, exitPlanMode, notify };
       const anchor = React.useRef(null);
 
       React.useEffect(() => {
@@ -114,8 +118,14 @@ window.__ModuleLoader__.load({
           if (seat.querySelector('[data-trigger-menu]') !== null) return; // Shift+Tab closes an open slash/@ menu
           event.preventDefault();
           event.stopPropagation();
-          const { draft, inputActions } = live.current;
-          if (event.repeat || event.isComposing || event.keyCode === 229 || inputActions === undefined) return;
+          const { draft, inputActions, planOn, exitPlanMode, notify } = live.current;
+          if (event.repeat || event.isComposing || event.keyCode === 229) return;
+          if (planOn && exitPlanMode !== undefined) {
+            exitPlanMode().then((failure) => { if (failure !== null && notify !== undefined) notify('error', failure); },
+              (reason) => { if (notify !== undefined) notify('error', reason instanceof Error ? reason.message : String(reason)); });
+            return;
+          }
+          if (inputActions === undefined) return;
           const { start, end, text } = planToggle(draft);
           // Edit only the prefix so the rest of the draft, its file chips and attachments stay intact.
           if (!inputActions.insertText(text, { ...inputActions.captureInsertion(), start, end })) return;
@@ -667,7 +677,7 @@ window.__ModuleLoader__.load({
     }
 
     return {
-      inject: ['slots', 'sessions', 'locale'],
+      inject: ['slots', 'sessions', 'locale', 'remote', 'remote.commands'],
       apply(ctx) {
         ctx.slots.inject('conversation.input.dock', () => ctx.slots.register({
           name: 'conversation.input.dock',
@@ -686,6 +696,18 @@ window.__ModuleLoader__.load({
           name: 'conversation.input.right',
           id: 'composer-plan-toggle',
           order: 999,
+          inject: (sessionId) => {
+            const commands = ctx.remote === undefined ? undefined : ctx.remote.commands;
+            const actx = ctx.sessions === undefined ? undefined : ctx.sessions.scope(sessionId);
+            return {
+              exitPlanMode: commands === undefined ? undefined : async () => {
+                const result = await commands.execute(sessionId, '/plan off', []);
+                if (!result.ok) return `${result.error.message} (${result.error.code})`;
+                return result.value === undefined ? 'unknown command: /plan off' : null;
+              },
+              notify: (level, text) => { try { actx.get('conversation').input.for(actx).notify(level, text); } catch { /* notice is best-effort */ } },
+            };
+          },
         }, PlanToggle));
         ctx.slots.inject('conversation.input.right', () => ctx.slots.register({
           name: 'conversation.input.right',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-composer-plus",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "DeepSeek Harness helpers: Codex-style Changes panel, input history, Shift+Tab /plan, one-click Continue, queue drag-and-drop, Copy deeplink, Auto Review allow-for-chat, background notifications",
   "license": "MIT",
   "type": "module",
@@ -40,7 +40,8 @@
         "@deepseek-ai/dsh-client-ui-workspace",
         "@deepseek-ai/dsh-client-ui-sidebar-right",
         "@deepseek-ai/dsh-client-ui-session",
-        "@deepseek-ai/dsh-api-remotes"
+        "@deepseek-ai/dsh-api-remotes",
+        "@deepseek-ai/dsh-api-session-controller"
       ]
     }
   },