dsh-context-compression-improved 0.3.0 → 0.4.0-beta.1

package/.githooks/pre-push

@@ -0,0 +1,37 @@
+#!/bin/sh
+# lib freshness gate (runs before every push from this repo).
+#
+# The plugin ships lib/ as a committed artifact and never builds on the
+# client at install time, so a stale lib pushed to a branch is what users
+# actually download. Rebuild here and refuse the push when the committed
+# lib no longer matches the committed sources.
+#
+# Skip conditions:
+# - the branch does not track a lib directory (non-plugin branches);
+# - pnpm or the build itself fails for reasons unrelated to freshness
+#   (the push is allowed, the build problem surfaces in CI/typecheck).
+
+ROOT="$(git rev-parse --show-toplevel)"
+LIB_DIR="packages/selector/lib"
+
+# Nothing to guard on branches without the shipped artifact.
+if [ ! -f "$ROOT/$LIB_DIR/index.js" ]; then
+  exit 0
+fi
+
+echo "[pre-push] rebuilding lib to verify freshness..."
+if ! (cd "$ROOT" && pnpm build >/dev/null 2>&1); then
+  echo "[pre-push] WARNING: pnpm build failed; skipping the lib freshness check." >&2
+  exit 0
+fi
+
+drift="$(git -C "$ROOT" diff --name-only -- "$LIB_DIR"; git -C "$ROOT" diff --cached --name-only -- "$LIB_DIR")"
+if [ -n "$drift" ]; then
+  echo >&2 "[pre-push] BLOCKED: packages/selector/lib drifted from the committed sources."
+  echo >&2 "[pre-push] The build just refreshed it. Review and commit the lib changes:"
+  echo >&2 "[pre-push]   git add packages/selector/lib && git commit --amend --no-edit   (or a follow-up commit)"
+  echo >&2 "[pre-push] Then push again."
+  exit 1
+fi
+
+exit 0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-context-compression-improved",
-  "version": "0.3.0",
+  "version": "0.4.0-beta.1",
   "publishConfig": {
     "registry": "https://registry.npmjs.org",
     "access": "public",
@@ -36,6 +36,7 @@
     "client": {
       "inject": [
         "@deepseek-ai/dsh-client-locale",
+        "@deepseek-ai/dsh-client-ui-primitives",
         "@deepseek-ai/dsh-client-ui-slots",
         "@deepseek-ai/dsh-client-ui-settings"
       ],
@@ -49,7 +50,7 @@
   "packageManager": "pnpm@11.7.0",
   "engines": {
     "node": "^22.19.0 || >=24.0.0",
-    "dsh": ">=0.1.5-alpha.1 <0.2.0-0"
+    "dsh": ">=0.1.5-rc.1 <0.2.0-0"
   },
   "scripts": {
     "build": "pnpm -r --if-present run bundle",

package/packages/selector/cordis.patch.yml

@@ -6,16 +6,23 @@
       name: 'dsh-context-compression-improved'
       config:
         presetOverlay: true
-    # The estimator catalog route rides its own row so the compression stack
-    # stays loadable on profiles that have no host web server. Measured on the
-    # host source: `dsh-host-webserver.register` performs no authorization check
-    # and `ctx.get(name)` only asks whether the providing fiber is active, so
-    # this `inject` is belt-and-braces — the row is simply absent instead of
+    # The HTTP routes ride their own row so the compression stack stays loadable
+    # on profiles that have no host web server. Measured on the host source:
+    # `dsh-host-webserver.register` performs no authorization check and
+    # `ctx.get(name)` only asks whether the providing fiber is active, so this
+    # `inject` is belt-and-braces — the row is simply absent instead of
     # announcing a pending route on a webServer-less profile. The two-channel
     # registration inside the plugin covers both arrival orders on its own.
+    #
+    # `reviewQueueRoute` belongs here and not on the bundle row: it gates the
+    # R4 human-review endpoints (`/review-queue`, `/review-decide`) and was
+    # never turned on, so the review overlay polled two 404s and the panel
+    # could not open. Both flags are set on exactly one row, so each route is
+    # registered exactly once.
     - id: context-compression-improved-estimator-catalog
       name: 'dsh-context-compression-improved'
       inject:
         - webServer
       config:
         estimatorCatalogRoute: true
+        reviewQueueRoute: true

package/packages/selector/lib/client.d.ts

@@ -84,6 +84,11 @@ interface PresetOptionsSettings {
   readonly estimatorBaseUrl?: string;
   readonly estimatorApiKey?: string;
   readonly estimatorTimeoutMs?: number;
+  /** Review-mode overrides (beta); mirrors the runtime PresetOptions.reviewMode. */
+  readonly reviewMode?: boolean;
+  readonly reviewTimeoutTurns?: number;
+  readonly cacheHitDiscountAlpha?: number;
+  readonly reviewHighImpactTokens?: number;
 }
 /** Durable settings section owned by this package. */
 interface ContextCompressionSettings {
@@ -141,6 +146,25 @@ declare const zh: {
   'estimator.apiKey.set': string;
   'estimator.apiKey.clear': string;
   'estimator.apiKey.overwrite': string;
+  'review.title': string;
+  'review.description': string;
+  'review.enabled': string;
+  'review.enabled.on': string;
+  'review.enabled.off': string;
+  'review.timeoutTurns': string;
+  'review.alpha': string;
+  'review.highImpact': string;
+  'review.badge': string;
+  'review.summary.autoApplied': string;
+  'review.summary.reviewApplied': string;
+  'review.summary.expired': string;
+  'review.summary.voided': string;
+  'review.row.payback': string;
+  'review.row.expectedSaving': string;
+  'review.row.estimated': string;
+  'review.action.approve': string;
+  'review.action.reject': string;
+  'review.action.ignore': string;
   'detail.tokenpilot-inspired': string;
   'profile.custom': string;
   'profile.native': string;