@goodandready/dsh-lanmode 0.6.2 → 0.6.4

package/README.md

@@ -1,5 +1,8 @@
 # dsh-lanmode
 
+> Adapted for DeepSeek Harness **0.1.2-alpha.1**: the host now decides LAN trust from the request `Host` header, so the plugin no longer patches the page for it.
+
+
 Open the [DeepSeek Harness](https://github.com/deepseek-ai/deepseek-harness) Web UI from another device — a phone, a tablet, the laptop on the other side of the room — and have **all of it** work, including the parts the UI switches off when the page is not `localhost`.
 
 Works whether you put a reverse proxy in front of the harness or let this plugin serve the network itself. Pick a mode, restart, done.

package/lib/assumptions.js

@@ -42,14 +42,29 @@ export async function checkAssumptions(options) {
     'webServer.port — без него не поднять прямой режим и не проверить, кто слушает сеть',
   ))
 
-  let html = ''
+  let page = { status: 0, html: '' }
   try {
-    html = await options.fetchIndex()
+    page = await options.fetchIndex()
   } catch (unreachable) {
     results.push(verdict('страница отдаётся', false, String(unreachable.message || unreachable)))
     return results
   }
 
+  // Харнесс может потребовать токен и ответить отказом. Тело отказа — не
+  // страница интерфейса, и искать в нём заплатку бессмысленно: получилось бы
+  // уверенное «её нет» там, где мы просто не смотрели.
+  if (page.status && page.status !== 200) {
+    results.push(verdict(
+      'страница отдаётся нам',
+      false,
+      'харнесс ответил ' + page.status + ' на запрос по петле без токена — проверить страницу отсюда нельзя. '
+        + 'Это не поломка плагина: заплатка могла встать, но увидеть её мы не можем',
+    ))
+    return results
+  }
+
+  const html = page.html
+
   results.push(verdict(
     'заплатка попала на страницу',
     html.includes('data-dsh-lanmode'),
@@ -72,5 +87,6 @@ export function summarize(results) {
   if (bad.length === 0) return 'точки крепления на месте: ' + results.length + ' из ' + results.length
   return 'ТОЧКИ КРЕПЛЕНИЯ УЕХАЛИ (' + bad.length + ' из ' + results.length + '): '
     + bad.map((item) => item.name).join('; ')
-    + '. Плагин может работать не так, как задумано, — вероятно, обновилось ядро'
+    + '. Плагин может работать не так, как задумано. Что именно не сошлось — в списке выше; '
+    + 'ядро при этом может быть совершенно исправным'
 }

package/lib/bridge.js

@@ -17,6 +17,7 @@
 
 import http from 'node:http'
 import https from 'node:https'
+import { forceLoopback, isConnectionBundle } from './loopback-source.js'
 
 import { allowed } from './access.js'
 import { REFUSED, isPrivileged } from './privileged.js'
@@ -57,6 +58,25 @@ function throttle(log, everyMs) {
  * снаружи выглядит как поломка харнесса. Обратный прокси перед харнессом
  * настраивают ровно так же, иначе он не годится.
  */
+/**
+ * Пояснение про ядро, которое обходится без нашей правки страницы.
+ *
+ * До 0.1.2 признак «страница открыта с этой же машины» вычислялся в браузере,
+ * и разделы настроек по сети приезжали пустыми — поэтому плагин правил это
+ * вычисление в сборке на лету. С 0.1.2 ядро решает то же самое у себя, по
+ * заголовку `Host` запроса. Мост подменяет `Host` на петлю с самого начала,
+ * значит вопрос закрыт раньше, чем страница успевает его задать.
+ *
+ * Говорим об этом один раз: иначе строка повторяется на каждый запрос сборки.
+ */
+let coreNoted = false
+
+function noteSelfSufficientCore(log) {
+  if (coreNoted) return
+  coreNoted = true
+  log('ядро само решает, доверять ли адресу, — по заголовку запроса; правка страницы не нужна')
+}
+
 function relaxTimeouts(server, streamTimeoutMs) {
   server.timeout = streamTimeoutMs
   server.requestTimeout = streamTimeoutMs
@@ -109,17 +129,40 @@ export function startDirectBridge(ctx, options) {
     req.setTimeout(streamTimeoutMs)
     res.setTimeout(streamTimeoutMs)
 
+    // Сборку с признаком «своя машина» правим на лету, всё остальное идёт
+    // кусками как шло. Для неё просим несжатый ответ: иначе пришлось бы
+    // распаковывать ради одной строки.
+    const patching = isConnectionBundle(req.url)
+    const headers = rewritten(req.headers, authority)
+    if (patching) headers['accept-encoding'] = 'identity'
+
     const upstream = http.request({
       host: '127.0.0.1',
       port: upstreamPort,
       method: req.method,
       path: req.url,
-      headers: rewritten(req.headers, authority),
+      headers,
       timeout: streamTimeoutMs || undefined,
     }, (answer) => {
       answer.setTimeout(streamTimeoutMs)
-      res.writeHead(answer.statusCode || 502, answer.headers)
-      answer.pipe(res)
+      if (!patching) {
+        res.writeHead(answer.statusCode || 502, answer.headers)
+        answer.pipe(res)
+        return
+      }
+      const parts = []
+      answer.on('data', (chunk) => parts.push(chunk))
+      answer.on('end', () => {
+        const done = forceLoopback(Buffer.concat(parts).toString('utf8'))
+        if (!done.changed) noteSelfSufficientCore(options.log)
+        const body = Buffer.from(done.source, 'utf8')
+        const out = { ...answer.headers }
+        out['content-length'] = String(body.length)
+        delete out['content-encoding']
+        res.writeHead(answer.statusCode || 502, out)
+        res.end(body)
+      })
+      answer.on('error', () => { if (!res.writableEnded) res.end() })
     })
     upstream.on('error', () => {
       if (!res.headersSent) res.writeHead(502, { 'content-type': 'text/plain; charset=utf-8' })

package/lib/index.js

@@ -326,12 +326,16 @@ function start(ctx, config) {
   // поломка обязана быть заметной.
   ctx.effect(() => {
     let alive = true
-    const port = ctx.webServer && ctx.webServer.port
-    const fetchIndex = () => fetch('http://127.0.0.1:' + port + '/').then((answer) => answer.text())
+    // Держим сам сервис, а не путь к нему: отложенное обращение через ctx
+    // в новых сборках возвращает пустоту, хотя сервис на месте.
+    const webServer = ctx.webServer
+    const port = webServer && webServer.port
+    const fetchIndex = () => fetch('http://127.0.0.1:' + port + '/')
+      .then((answer) => answer.text().then((html) => ({ status: answer.status, html })))
 
     // Даём харнессу договорить о себе: страница отдаётся не в первый миг.
     const timer = setTimeout(() => {
-      checkAssumptions({ webServer: ctx.webServer, fetchIndex })
+      checkAssumptions({ webServer, fetchIndex })
         .then((results) => {
           if (!alive) return
           state.assumptions = results

package/lib/loopback-source.js

@@ -0,0 +1,49 @@
+// Признак «страница открыта с этой же машины» — в самом источнике.
+//
+// Ядро вычисляет его в браузере один раз, при сборке объекта соединения:
+//
+//     isLoopback: pageLocation === void 0 || isLoopbackHostname(pageLocation.hostname)
+//
+// Дальше это обычное свойство обычного объекта, и подменить его на странице
+// можно — плагин так и делает. Но подмена успевает не везде: разделы настроек
+// привязываются в момент своего apply, и тот, кто привязался раньше нашего
+// клиентского кода, уже запомнил «нет». Вкладка «Модели» — как раз такой
+// случай: она читает раздел с провайдерами до нас, получает пустое хранилище
+// вместо файла, и девять заведённых провайдеров исчезают из списка вместе с
+// возможностью их выбрать.
+//
+// Гонку не выиграть перестановкой: порядок загрузки задаёт не плагин. Поэтому
+// значение правится там, где его больше никто не обгонит — в самом файле,
+// пока он идёт через мост. Одна строка, одно вхождение, остальной файл
+// нетронут.
+//
+// Настоящее значение при этом не теряется: клиентская часть плагина отдаёт его
+// тем, кому оно действительно нужно — выдаче результатов работы, где флаг
+// решает, открывать ли путь на своей машине или на серверной.
+
+/** Путь, по которому ядро отдаёт сборку с этим вычислением. */
+export const CONNECTION_BUNDLE = '/plugins/@deepseek-ai/dsh-client-connection/client.js'
+
+/** Ровно то выражение, которое считает признак. */
+const COMPUTED = 'isLoopback: pageLocation === void 0 || isLoopbackHostname(pageLocation.hostname),'
+
+/** Чем его заменяем. */
+const ALWAYS = 'isLoopback: true,'
+
+/** Тот ли это запрос: путь сверяем без строки запроса — там висит ?rev=... */
+export function isConnectionBundle(url) {
+  const path = String(url ?? '').split('?')[0]
+  return path === CONNECTION_BUNDLE
+}
+
+/**
+ * Заменить вычисление признака на «да».
+ *
+ * @param source текст сборки
+ * @returns `{ source, changed }` — исходник и признак того, что замена нашлась
+ */
+export function forceLoopback(source) {
+  const text = String(source)
+  if (!text.includes(COMPUTED)) return { source: text, changed: false }
+  return { source: text.replace(COMPUTED, ALWAYS), changed: true }
+}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@goodandready/dsh-lanmode",
-  "version": "0.6.2",
-  "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.",
+  "version": "0.6.4",
+  "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. Adapted for DeepSeek Harness 0.1.2-alpha.1.",
   "license": "MIT",
   "type": "module",
   "main": "./lib/index.js",