@duke-dsh-plugins/dsh-agent-approval 1.8.0 → 1.8.2

package/client.js

@@ -200,6 +200,12 @@ window.__ModuleLoader__.load({
         //    with no glyph the icon span is absent, leaving label + chevron.
         //    Skip menu rows (handled above) and the settings dialog (its nav
         //    row carries the same label but is owned by the settings-nav icon).
+        //    The trigger renders whichever preset is CURRENTLY selected, so it
+        //    must accept BOTH of our preset names (v1.8.2): 自动审批 and 自动审查
+        //    are equally ours, and neither is in the host's permissionGlyphs —
+        //    registerReviewMenuItem only marks menu rows, so without this the
+        //    trigger showed 自动审查 with no icon at all.
+        const triggerLabels = [currentLabel, String(REVIEW_LABEL).trim()];
         const buttons = document.querySelectorAll("button");
         for (let i = 0; i < buttons.length; i++) {
           const button = buttons[i];
@@ -212,7 +218,7 @@ window.__ModuleLoader__.load({
             const s = spans[j].textContent ? spans[j].textContent.trim() : "";
             if (s.length > 0) { labelText = s; break; }
           }
-          const matches = labelText === currentLabel && button.querySelector("svg") !== null;
+          const matches = triggerLabels.indexOf(labelText) !== -1 && button.querySelector("svg") !== null;
           if (matches) button.setAttribute(PERM_TRIGGER_MARKER, "");
           else button.removeAttribute(PERM_TRIGGER_MARKER);
         }

package/index.js

@@ -948,19 +948,28 @@ export class AgentApprovalService extends TypertRemoteService {
   /**
    * Resolve the audit sidecar for one session: `agent-approval.jsonl` inside
    * the session's persistence directory (same directory as the session's own
-   * durable log, via `sessionPersistence.locate(header)` — a pure path
-   * resolution that also works for live sessions). Falls back to a
-   * plugin-owned per-session file under DSH_HOME when the seam or the
-   * location is unavailable; the fallback keeps restart-safety at the cost
-   * of not being cleaned up when the session is deleted.
+   * durable log — a pure path resolution that also works for live sessions).
+   * DSH ≤0.1.7 exposes it as `sessionPersistence.locate(header)`; DSH 0.2.0
+   * dropped `locate` and moved resolution to the JSONL backend's async
+   * `resolveCurrentLog(id)` (returns the log path, or undefined while only a
+   * historical generation exists). Falls back to a plugin-owned per-session
+   * file under DSH_HOME when the seam or the location is unavailable; the
+   * fallback keeps restart-safety at the cost of not being cleaned up when the
+   * session is deleted.
    */
   async _recordsFileOf(session) {
     const persistence = this.ctx.get("sessionPersistence");
-    if (persistence !== undefined && typeof persistence.locate === "function") {
+    if (persistence !== undefined) {
       try {
-        const loc = persistence.locate(session.header);
-        if (loc && typeof loc.path === "string" && loc.path !== "") {
-          return join(dirname(loc.path), RECORDS_SIDECAR);
+        let path;
+        if (typeof persistence.locate === "function") {
+          const loc = persistence.locate(session.header);
+          if (loc && typeof loc.path === "string") path = loc.path;
+        } else if (typeof persistence.resolveCurrentLog === "function") {
+          path = await persistence.resolveCurrentLog(session.id);
+        }
+        if (typeof path === "string" && path !== "") {
+          return join(dirname(path), RECORDS_SIDECAR);
         }
       } catch (e) {
         /* fall through to the plugin-owned fallback */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@duke-dsh-plugins/dsh-agent-approval",
-  "version": "1.8.0",
+  "version": "1.8.2",
   "description": "Agent-decided approvals for DeepSeek Harness: an 自动审批 permission mode where every sandbox escalation is judged automatically (default judge: one direct LLM call with no subagent session; optional isolated judge subagent or the TypeSafe Jev decision model), plus an opt-in 自动审查 per-call review mode (Full access base; every tool call reviewed by the Jev judge before execution, risky calls rejected with no human fallback), with a configurable judge model and a per-session audit trail in the conversation window's 审批 tab.",
   "repository": {
     "type": "git",
@@ -34,13 +34,13 @@
       "platform": "web"
     },
     "engines": {
-      "dsh": "^0.1.5-rc.3"
+      "dsh": "^0.1.5-rc.3 || ^0.2.0-rc.1"
     }
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
-    "@deepseek-ai/dsh": "^0.1.5-rc.3",
-    "@deepseek-ai/dsh-typert-protocol": "^0.1.0-rc.7 || ^0.1.7-rc.1"
+    "@deepseek-ai/dsh": "^0.1.5-rc.3 || ^0.2.0-rc.1",
+    "@deepseek-ai/dsh-typert-protocol": "^0.1.0-rc.7 || ^0.1.7-rc.1 || ^0.2.0-rc.1"
   },
   "peerDependenciesMeta": {
     "@deepseek-ai/dsh": {