@goodandready/dsh-lanmode 0.3.1 → 0.4.0

package/README.md

@@ -35,11 +35,23 @@ Everything happens through the web server's official index tap: one script is in
 
 | Piece | Setting | What it does |
 |---|---|---|
-| Settings | `settings` | Hands the two settings packages a connection whose `isLoopback` reads `true`. Everything downstream — the shared mirror, every namespace scope, the core's own pages — then behaves as it does on localhost. |
+| Settings | `settings` | Hands every package a connection whose `isLoopback` reads `true`. The shared mirror, every namespace scope, the core's own pages and every plugin's settings section then behave as they do on localhost. |
 | `crypto.randomUUID` | `randomUuid` | Provides an RFC 4122 v4 implementation over `crypto.getRandomValues`, which insecure origins do have. A no-op where the real one exists. |
 | `navigator.clipboard` | `clipboard` | Provides a `writeText` fallback so the copy buttons keep working. A no-op where the real one exists. |
 
-The settings substitution is deliberately narrow. Three packages read that flag, and the third is deliverables, where it decides whether a produced file may be opened locally — forcing it there would ask the Host to open paths on the server's desktop. Only the settings packages see the substitute.
+One package is excluded on purpose: deliverables, where the flag decides whether a produced file may be opened locally. Forcing it there would ask the Host to open paths on the server's desktop. Nothing else in the web UI reads the flag.
+
+The exclusion list replaced an allow list, and the reason is worth writing down. A namespace scope is bound like this:
+
+```js
+bind(spec) {
+  const ctx = this.ctx                      // the caller's context
+  const connection = ctx.get('connection')
+  ... connection.isLoopback ? 'host' : 'memory'
+}
+```
+
+`this.ctx` belongs to whichever plugin calls `bind`, not to the settings package. Handing the substitute to the settings packages alone therefore fixed the shared mirror and the core's own pages, while every plugin's own settings section still went to memory mode and reported that the harness had not announced its settings.
 
 ## Two modes
 

package/lib/shim.js

@@ -67,35 +67,83 @@
   // ------------------------------------------------------------- настройки
   if (!options.settings) return
 
-  // Флаг читают трое, и третьему подменять нельзя: в результатах работы он
-  // решает, можно ли открыть файл локально, и с подменой браузер просил бы
-  // открыть путь на машине сервера. Поэтому — только два настроечных пакета.
-  var TARGETS = [
-    '@deepseek-ai/dsh-client-ui-settings',
-    '@deepseek-ai/dsh-client-ui-settings-general',
+  // Флаг живёт на объекте соединения, и читают его в двух совершенно разных
+  // местах.
+  //
+  // Общий справочник разделов заводится в apply настроечного пакета — там
+  // хватало подмены контекста. Но каждый отдельный раздел привязывается иначе:
+  //
+  //     bind(spec) {
+  //       const ctx = this.ctx                      // контекст вызывающего
+  //       const connection = ctx.get('connection')
+  //       ... connection.isLoopback ? 'host' : 'memory'
+  //     }
+  //
+  // this.ctx — контекст того плагина, который зовёт bind. Подменять контекст
+  // каждому плагину бесполезно: сервис запоминает не заместителя, а исходный
+  // контекст, через который его завели. Поэтому подменяется не контекст, а сам
+  // объект соединения, и ровно один раз: он один на весь интерфейс, и после
+  // подмены одинаковый ответ видят все — и ядро, и любой плагин со своим
+  // разделом настроек.
+  //
+  // Исключение одно: выдача результатов работы. Там флаг решает, можно ли
+  // открыть файл локально, и с подменой браузер просил бы открыть путь на
+  // машине сервера. Этому пакету возвращается настоящее значение. Больше флаг
+  // во всём интерфейсе не читает никто.
+  var EXCLUDED = [
+    '@deepseek-ai/dsh-client-ui-deliverables',
   ]
 
-  function connectionWithForcedFlag(connection) {
+  /** Настоящий ответ по тому же правилу, что и у ядра: только свои адреса. */
+  function realLoopback() {
+    var host = location.hostname
+    if (host === 'localhost' || host === '[::1]' || host === '::1') return true
+    return /^127\./.test(host)
+  }
+
+  // Подмена ставится при первой же возможности: как только у кого-то из
+  // пакетов появился контекст, из которого достаётся соединение.
+  var forced = false
+
+  function forceOnConnection(ctx) {
+    if (forced) return
+    var connection = null
+    try { connection = ctx && ctx.get && ctx.get('connection') } catch (noService) { connection = null }
+    if (!connection || typeof connection !== 'object') return
+    try {
+      Object.defineProperty(connection, 'isLoopback', {
+        configurable: true,
+        get: function () { return loopbackAnswer },
+      })
+      forced = true
+    } catch (cannotDefine) {
+      // Свойство неподатливо — оставляем как было: страница будет вести себя
+      // так же, как без плагина.
+    }
+  }
+
+  function connectionWithRealFlag(connection) {
     if (!connection || typeof connection !== 'object') return connection
     return new Proxy(connection, {
       get: function (target, prop) {
-        if (prop === 'isLoopback') return loopbackAnswer
+        if (prop === 'isLoopback') return realLoopback()
         var value = Reflect.get(target, prop, target)
         return typeof value === 'function' ? value.bind(target) : value
       },
     })
   }
 
-  function ctxWithForcedConnection(ctx) {
+  /** Контекст исключённого пакета: ему возвращается настоящее значение. */
+  function ctxWithRealConnection(ctx) {
     return new Proxy(ctx, {
       get: function (target, prop) {
         if (prop === 'get') {
           return function (nameRequested) {
             var value = target.get(nameRequested)
-            return nameRequested === 'connection' ? connectionWithForcedFlag(value) : value
+            return nameRequested === 'connection' ? connectionWithRealFlag(value) : value
           }
         }
-        if (prop === 'connection') return connectionWithForcedFlag(Reflect.get(target, prop, target))
+        if (prop === 'connection') return connectionWithRealFlag(Reflect.get(target, prop, target))
         var value = Reflect.get(target, prop, target)
         return typeof value === 'function' ? value.bind(target) : value
       },
@@ -103,8 +151,8 @@
   }
 
   function wrap(registration) {
-    if (!registration || TARGETS.indexOf(registration.id) === -1) return registration
-    if (typeof registration.factory !== 'function') return registration
+    if (!registration || typeof registration.factory !== 'function') return registration
+    var excluded = EXCLUDED.indexOf(registration.id) !== -1
     var factory = registration.factory
     var patched = {}
     for (var key in registration) patched[key] = registration[key]
@@ -113,7 +161,12 @@
       if (!moduleExports || typeof moduleExports.apply !== 'function') return moduleExports
       var originalApply = moduleExports.apply
       moduleExports.apply = function (ctx) {
-        return originalApply.call(this, ctxWithForcedConnection(ctx))
+        // Исключённый пакет читает соединение раньше, чем его успели подменить,
+        // — но полагаться на это нельзя, поэтому ему всегда отдаётся
+        // заместитель с настоящим значением.
+        if (excluded) return originalApply.call(this, ctxWithRealConnection(ctx))
+        forceOnConnection(ctx)
+        return originalApply.call(this, ctx)
       }
       return moduleExports
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-lanmode",
-  "version": "0.3.1",
+  "version": "0.4.0",
   "description": "LAN and reverse-proxy access for the DeepSeek Harness Web UI: returns the settings service on pages that are not localhost, fills in the Web APIs the browser withholds on plain HTTP, and can open a listener of its own so nothing else is needed.",
   "license": "MIT",
   "type": "module",