dsh-lan-peers 0.8.2 → 0.8.4

package/client/cli-code.test.mjs

@@ -0,0 +1,108 @@
+// dsh-lan-peers · M6 之后 `lan-peers code` 必须请本机 host 发码。
+//
+// 审计/产品口径：配对码只在 daemon 进程内存里，CLI 是独立进程 —— 它自己造的码对正在跑的
+// host 无效，用户抄走的是废码。所以 code 子命令改成打本机管理接口（带 token），
+// 本机没 host / 拿不到 token 时 exit 2 并说清怎么起 host，绝不本地造一个码。
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { execFile } from 'node:child_process';
+import fs from 'node:fs';
+import http from 'node:http';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { createAdmin } from '../lib/admin.mjs';
+import { makeTempStateDir } from '../host/test/helpers.mjs';
+
+const CLI = path.join(path.dirname(fileURLToPath(import.meta.url)), 'cli.mjs');
+
+const run = (args) => new Promise((resolve) => {
+  execFile(process.execPath, [CLI, ...args], { encoding: 'utf8' }, (error, stdout, stderr) => {
+    resolve({ code: error ? error.code : 0, stdout: String(stdout || ''), stderr: String(stderr || '') });
+  });
+});
+
+/** 真 admin 挂在真 HTTP 上：token 文件由它启动时写进这个 stateDir（CLI 读同一份）。 */
+async function startAdmin(t, { dir, code = '246810' } = {}) {
+  const state = { instanceId: 'inst-self', name: 'self' };
+  const store = {
+    instanceId: 'inst-self',
+    stateDir: dir,
+    state,
+    listPeers: () => [],
+    getPeer: () => null,
+    issuePairingCode: () => {
+      const issued = { code, expiresAt: Date.now() + 600_000 };
+      state.pairingCode = issued;
+      return issued;
+    },
+  };
+  const admin = createAdmin({
+    server: { config: { logger: () => {} }, store },
+    nameOf: () => 'self',
+    rename: (n) => n,
+  });
+  const calls = [];
+  const server = http.createServer((req, res) => {
+    let body = '';
+    req.on('data', (chunk) => { body += chunk; });
+    req.on('end', () => {
+      calls.push({ url: String(req.url || ''), method: String(req.method || 'GET'), headers: req.headers });
+      const url = new URL(String(req.url || '/'), 'http://127.0.0.1');
+      Promise.resolve(admin.handle(req, res, url)).then((handled) => {
+        if (!handled) {
+          res.writeHead(404, { 'content-type': 'application/json' });
+          res.end('{"error":"not-found"}');
+        }
+      }).catch((error) => {
+        res.writeHead(500, { 'content-type': 'application/json' });
+        res.end(JSON.stringify({ error: String(error && error.message ? error.message : error) }));
+      });
+    });
+  });
+  await new Promise((resolve) => server.listen(0, '127.0.0.1', resolve));
+  t.after(() => new Promise((resolve) => server.close(resolve)));
+  return { admin, calls, state, url: 'http://127.0.0.1:' + server.address().port };
+}
+
+test('code：请本机 host 发码并原样打印（与 admin 手里的码一致）', async (t) => {
+  const dir = makeTempStateDir('lanb-cli-code-');
+  t.after(() => fs.rmSync(dir, { recursive: true, force: true }));
+  const rig = await startAdmin(t, { dir, code: '246810' });
+
+  const out = await run(['code', '--state-dir', dir, '--url', rig.url]);
+  assert.equal(out.code, 0, out.stderr);
+  assert.match(out.stdout, /配对码: 246810/);
+  assert.doesNotMatch(out.stdout + out.stderr, /token/, '不能把 token 字样漏到界面上');
+
+  const issued = rig.calls.find((call) => call.url === '/api/lan-peers/code');
+  assert.ok(issued, '必须打 POST /api/lan-peers/code: ' + JSON.stringify(rig.calls.map((c) => c.url)));
+  assert.equal(issued.method, 'POST');
+  assert.equal(issued.headers['x-lan-peers-token'], rig.admin.token, 'token 从 <stateDir>/admin-token 读，只走请求头');
+
+  // 打印出来的码就是 host 手里的那一个：GET state 说「已生成」。
+  const state = await fetch(rig.url + '/api/lan-peers/state', { headers: { 'x-lan-peers-token': rig.admin.token } });
+  const body = await state.json();
+  assert.equal(body.pairing.active, true, 'CLI 打印的码必须是本机 host 活着的那一个');
+  assert.equal(JSON.stringify(body).includes('246810'), false, 'GET state 仍不回显码');
+});
+
+test('code：本机没有 host / 拿不到 token → exit 2，且绝不本地造码打印', async (t) => {
+  const dir = makeTempStateDir('lanb-cli-code-');
+  t.after(() => fs.rmSync(dir, { recursive: true, force: true }));
+  const rig = await startAdmin(t, { dir, code: '246810' });
+  // 拿掉 token 文件：等价于「本机 host 没起 / 不是同一个进程」。
+  fs.rmSync(path.join(dir, 'admin-token'));
+
+  const out = await run(['code', '--state-dir', dir, '--url', rig.url]);
+  assert.equal(out.code, 2, '拿不到码就是 exit 2: ' + out.stdout + out.stderr);
+  assert.match(out.stderr, /请先启动本机 host/);
+  assert.match(out.stderr, /token-required/, '失败原因要如实转述');
+  assert.doesNotMatch(out.stdout, /\d{6}/, '失败时不能在 stdout 吐出任何 6 位数字（那会被当成码）');
+  assert.equal(fs.existsSync(path.join(dir, 'admin-token')), false, 'CLI 不该自己去造 token');
+});
+
+test('code --help 说明「请本机 host 发码」，不再声称写 state.json', async (t) => {
+  const out = await run(['code', '--help']);
+  assert.match(out.stdout, /请本机 host 生成一次性配对码/);
+  assert.doesNotMatch(out.stdout, /同时写入 state\.json/, 'M6 之后 state.json 不再存码');
+});

package/client/cli.mjs

@@ -7,11 +7,9 @@
 //   node cli.mjs events --peer <peerId|name> --session <sessionId> [--duration 300] [--verbose]
 //   node cli.mjs unpair --peer <peerId|name> [--local-only] [--timeout 10]
 import crypto from 'node:crypto';
-import { execFileSync } from 'node:child_process';
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
-import { fileURLToPath } from 'node:url';
 import {
   DEFAULT_ANSWER_WAIT_MS,
   MAX_ANSWER_WAIT_MS,
@@ -82,9 +80,11 @@ const USAGE = [
 ].join('\n');
 
 const CODE_USAGE = [
-  '用法: dsh-lan-peers code [--state-dir <目录>]',
+  '用法: dsh-lan-peers code [--state-dir <目录>] [--url <本机管理接口>]',
   '',
-  '生成一次性配对码，同时写入 state.json（10 分钟有效，用后即焚）。',
+  '请本机 host 生成一次性配对码（10 分钟有效，用后即焚）。',
+  'M6 起码只活在 host 进程内存里：本命令是独立进程，自己生成的码对正在跑的 host 无效，',
+  '所以必须先启动本机 host（或直接用界面面板的「生成配对码」）。',
 ].join('\n');
 
 const PAIR_USAGE = [
@@ -227,20 +227,40 @@ async function signedCall({ baseUrl, secret, senderId, method = 'GET', apiPath,
   return callPeer({ baseUrl, secret, senderId, method, apiPath, body, timeoutMs: 60_000 });
 }
 
+/**
+ * M6：配对码只活在 daemon 进程内存里（state.json 不再持久化它）。本命令是**独立进程**，
+ * 自己生成一个码打印出来，正在跑的 host 根本不知道 —— 用户抄走的是一条无效凭据
+ * （旧行为「另一个进程能写码」本身也正是要堵掉的植入路径）。所以这里改成请本机 host 发码：
+ * 它才是唯一该写这条的进程，也是唯一知道码的进程。
+ *
+ * 本机没有 host / 拿不到 token → exit 2 并说清怎么起 host，绝不本地造一个码糊弄过去。
+ */
 async function cmdCode(args) {
-  // The code lives in state.json, which the host process re-reads; issuing it via
-  // the host CLI keeps one writer instead of racing the running server.
-  const pkgRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
-  const out = execFileSync(process.execPath, [
-    path.join(pkgRoot, 'host', 'server.mjs'),
-    '--code',
-    '--state-dir',
-    stateDir(args),
-  ], { encoding: 'utf8' });
-  const match = /(\d{6})/.exec(out);
-  if (!match) throw new Error(out.trim() || 'no pairing code');
-  console.log('配对码: ' + match[1] + '   有效期: 10 分钟，一次性');
+  const dir = stateDir(args);
+  const url = String(args.url ?? '').trim() || ADMIN_DEFAULT_URL;
+  let out;
+  try {
+    out = await adminCall(url, 'code', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: '{}',
+    }, dir);
+  } catch (error) {
+    const reason = String(error?.message ?? error);
+    process.stderr.write('生成配对码失败：本机 host 未接受 ' + url + '（' + reason + '）\n'
+      + '修法：请先启动本机 host（或用界面面板的「生成配对码」）—— 码只能在 host 进程里生成\n');
+    process.exitCode = 2;
+    return;
+  }
+  const code = String(out && out.code ? out.code : '').trim();
+  if (!/^\d{6}$/.test(code)) {
+    process.stderr.write('生成配对码失败：本机 host 未按契约回 6 位码\n');
+    process.exitCode = 2;
+    return;
+  }
+  console.log('配对码: ' + code + '   有效期: 10 分钟，一次性');
   console.log('对方把它填进「添加伙伴」里即可配对（地址填本机 http://<ip>:7788）');
+  console.log('提示: 这个码只在本机 host 进程里有效，重启 host 即失效');
 }
 
 async function cmdPair(args) {
@@ -718,6 +738,10 @@ function formatDispatchLine(dispatch) {
   const status = String(data.status ?? '');
   const who = String(data.targetName ?? '') || String(data.target ?? '') || '—';
   if (status === 'skipped') return '未派单（消息里没有 @! 目标）: ' + String(data.reason ?? 'no-target');
+  // M1：同名歧义是「用户得先选一个」，不是网络错误 —— 给出可操作的重发语法。
+  if (status === 'failed' && String(data.reason ?? '') === 'ambiguous') {
+    return '派单 → ' + who + '：有多个同名成员，请用 @!名字 (短id) 重发';
+  }
   const parts = [];
   // done 也可能带 reason（契约 §1：多个 @! 只取第一个，reason=multi-target-ignored）。
   const reason = String(data.reason ?? '');
@@ -751,7 +775,7 @@ async function triggerDispatch(args, command) {
         task: command.task,
         extraTargets: command.extraTargets,
       }),
-    });
+    }, stateDir(args));
   } catch (error) {
     const reason = String(error?.message ?? error);
     process.stderr.write('消息已发出，但派单未触发：本机 host 未响应 ' + url + '（' + reason + '）\n'
@@ -828,7 +852,7 @@ async function cmdGroupAccept(dir, args) {
       method: 'POST',
       headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ acceptDispatch: value }),
-    });
+    }, dir);
   } catch (error) {
     const reason = String(error?.message ?? error);
     process.stderr.write('切换接单失败：本机 host 未接受 ' + url + '（' + reason + '）\n'
@@ -1052,16 +1076,37 @@ async function cmdGroupClaim(dir, args) {
 
 const ADMIN_DEFAULT_URL = 'http://127.0.0.1:7788';
 const ADMIN_TIMEOUT_MS = 10000;
+// S3：管理接口要 X-Lan-Peers-Token。host 启动时把它写到 <stateDir>/admin-token（0600），
+// CLI 只读不写；token 本身绝不出现在 URL、日志或错误消息里（只说去哪读）。
+const ADMIN_TOKEN_FILE = 'admin-token';
+
+function readAdminToken(dir) {
+  try {
+    return fs.readFileSync(path.join(String(dir || DEFAULT_STATE_DIR), ADMIN_TOKEN_FILE), 'utf8').trim();
+  } catch {
+    return '';
+  }
+}
 
 /** 打本机回环管理接口；非 2xx 抛 `<status> <error>` 形式的 Error（与 group-client 同款）。 */
-async function adminCall(baseUrl, apiPath, init) {
+async function adminCall(baseUrl, apiPath, init, dir) {
   const base = String(baseUrl || ADMIN_DEFAULT_URL).replace(/\/+$/, '');
+  const token = readAdminToken(dir);
+  const headers = { ...(init && init.headers ? init.headers : {}) };
+  if (token !== '') headers['X-Lan-Peers-Token'] = token;
   const reply = await fetch(base + '/api/lan-peers/' + apiPath, {
     ...init,
+    headers,
     signal: AbortSignal.timeout(ADMIN_TIMEOUT_MS),
   });
   const body = await reply.json().catch(() => ({}));
-  if (!reply.ok) throw new Error(reply.status + ' ' + String(body.error || 'admin-call-failed'));
+  if (!reply.ok) {
+    const status = reply.status;
+    const where = path.join(String(dir || DEFAULT_STATE_DIR), ADMIN_TOKEN_FILE);
+    // 401 一定是 token 缺失/不匹配：直接指路（host 启动时写入，权限 0600）。
+    const hint = status === 401 ? '（管理接口需要 token：确认本机 host 在跑，且 ' + where + ' 可读）' : '';
+    throw new Error(status + ' ' + String(body.error || 'admin-call-failed') + hint);
+  }
   return body;
 }
 
@@ -1075,7 +1120,7 @@ function reportArchiveError(error) {
 async function cmdGroupArchives(dir, args) {
   let out;
   try {
-    out = await adminCall(args.url, 'group/archives');
+    out = await adminCall(args.url, 'group/archives', undefined, dir);
   } catch (error) {
     reportArchiveError(error);
     return;
@@ -1111,7 +1156,7 @@ async function cmdGroupArchiveDelete(dir, args) {
       method: 'POST',
       headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ file: file, action: 'delete' }),
-    });
+    }, dir);
   } catch (error) {
     reportArchiveError(error);
     return;