@sanqianx/project-knowledge 5.0.9-rc.12 → 5.0.9-rc.13

package/.claude-plugin/marketplace.json

@@ -9,7 +9,7 @@
   "plugins": [
     {
       "name": "project-knowledge",
-      "version": "5.0.9-rc.12",
+      "version": "5.0.9-rc.13",
       "source": "./plugins/project-knowledge",
       "description": "Search durable project decisions and implementation history through MCP."
     }

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [5.0.9-rc.13] - 2026-10-11
+
+- Give native Agent session creation and input setup a bounded 90-second proxy
+  budget. Ordinary API reads/abort/permissions retain 10 seconds; SSE streaming
+  and folder-picker contracts are unchanged.
+- Record safe proxy failure categories and deadlines without raw errors or URL
+  queries. Preserve MODULE_UNREACHABLE compatibility and add optional details.
+- Reproduce the installed OpenCode cold-start failure with a real delayed HTTP
+  server; verify the targeted budget, unrelated routes and credential redaction.
+- Retain rc.12 real acceptance and failed OpenCode results separately; require
+  a newly downloaded candidate and cold-client retest before user upgrade.
+
 ## [5.0.9-rc.12] - 2026-10-11
 
 - Vendor engine 0.2.3-rc.6 to address the Windows transient state-file rename

package/_site/_test/module-agent-start-proxy-test.js

@@ -0,0 +1,65 @@
+'use strict';
+const assert = require('node:assert/strict');
+const http = require('node:http');
+const { Readable } = require('node:stream');
+const { EventEmitter } = require('node:events');
+const { ModuleBridge } = require('../lib/module-bridge');
+
+async function budget(method, pathname) {
+  const timers = [], originalTimer = global.setTimeout, originalFetch = global.fetch;
+  const req = Readable.from([]); req.method = method; req.headers = {};
+  const res = new EventEmitter();
+  res.writeHead = () => {}; res.end = () => { res.writableEnded = true; };
+  try {
+    global.setTimeout = (fn, ms, ...args) => { timers.push(ms); return originalTimer(fn, ms, ...args); };
+    global.fetch = async () => new Response('{}', { headers: { 'Content-Type': 'application/json' } });
+    await new ModuleBridge().proxy(req, res, 'http://127.0.0.1:5760' + pathname);
+    return timers[0];
+  } finally { global.setTimeout = originalTimer; global.fetch = originalFetch; }
+}
+
+(async () => {
+  const upstream = http.createServer((req, res) => {
+    const timer = setTimeout(() => { res.writeHead(200, { 'Content-Type': 'application/json' }); res.end('{"accepted":true}'); }, 11000);
+    res.on('close', () => clearTimeout(timer));
+  });
+  await new Promise(resolve => upstream.listen(0, '127.0.0.1', resolve));
+  const bridge = new ModuleBridge();
+  const target = `http://127.0.0.1:${upstream.address().port}/api/claude-workbench/v1/sessions/cold-client/input`;
+  const shell = http.createServer((req, res) => bridge.proxy(req, res, target));
+  await new Promise(resolve => shell.listen(0, '127.0.0.1', resolve));
+  try {
+    const response = await fetch(`http://127.0.0.1:${shell.address().port}/send`, { method: 'POST', body: '{}' });
+    assert.equal(response.status, 200, 'cold native client startup must not be reported as unreachable after ten seconds');
+    assert.equal((await response.json()).accepted, true);
+  } finally {
+    for (const server of [shell, upstream]) { server.closeAllConnections(); await new Promise(resolve => server.close(resolve)); }
+  }
+  assert.equal(await budget('POST', '/api/claude-workbench/v1/sessions'), 90000);
+  assert.equal(await budget('POST', '/api/claude-workbench/v1/sessions/cold-client/input?scope=project'), 90000);
+  for (const [method, route] of [
+    ['GET', '/api/claude-workbench/v1/sessions'],
+    ['POST', '/api/claude-workbench/v1/sessions/cold-client/abort'],
+    ['POST', '/api/claude-workbench/v1/sessions/cold-client/permission'],
+    ['POST', '/api/engine/runs'],
+    ['POST', '/api/sessions/cold-client/input'],
+  ]) assert.equal(await budget(method, route), 10000, 'unrelated APIs must keep their ordinary timeout');
+  assert.equal(await budget('POST', '/api/claude-workbench/v1/system/pick-folder'), 600000);
+  const originalFetch = global.fetch;
+  let warning, failure;
+  const request = Readable.from([]); request.method = 'GET'; request.headers = {};
+  const response = new EventEmitter(); response.writeHead = () => {};
+  response.end = bytes => { response.writableEnded = true; failure = JSON.parse(bytes); };
+  try {
+    global.fetch = async () => { throw new Error('upstream leaked credential: secret-proxy-fixture'); };
+    const safeBridge = new ModuleBridge({ logger: { warn(...args) { warning = args; } } });
+    await safeBridge.proxy(request, response, 'http://127.0.0.1:5760/api/health?apiKey=secret-proxy-fixture');
+    assert.equal(failure.error, 'MODULE_UNREACHABLE');
+    assert.equal(failure.details.failureKind, 'NETWORK_ERROR');
+    assert.equal(failure.details.timeoutMs, 10000);
+    assert.equal(warning[0], 'module.proxy.failed');
+    assert.equal(warning[2].context.path, '/api/health');
+    assert.ok(!JSON.stringify([failure, warning]).includes('secret-proxy-fixture'), 'diagnostics omit queries, raw errors and credentials');
+  } finally { global.fetch = originalFetch; }
+  console.log('module-agent-start-proxy-test PASS');
+})().catch(error => { console.error(error); process.exitCode = 1; });

package/_site/lib/module-bridge.js

@@ -31,6 +31,9 @@ const { once } = require('events');
 const { DomainError } = require('./contracts');
 
 const PROXY_TIMEOUT_MS = 10000;
+// Native client startup includes knowledge retrieval and a local server/CLI
+// handshake. It has a bounded setup budget, separate from ordinary reads/SSE.
+const AGENT_START_TIMEOUT_MS = 90000;
 // Registration drives a full embedding pass in vector-hub (remote API); a
 // fresh knowledge folder can easily exceed the interactive proxy budget.
 const REGISTER_TIMEOUT_MS = 60000;
@@ -131,7 +134,11 @@ class ModuleBridge {
     // The native folder pickers hold the request open until the user closes
     // the dialog — they need a human-scale budget, not the API budget.
     const isPicker = /\/(system\/)?pick-folder/.test(target);
-    const timeoutMs = isPicker ? PICKER_TIMEOUT_MS : PROXY_TIMEOUT_MS;
+    const targetUrl = new URL(target);
+    const pathname = targetUrl.pathname;
+    const isAgentStart = req.method === 'POST' && /^\/api\/claude-workbench\/v1\/sessions(?:\/[^/]+\/input)?\/?$/.test(pathname);
+    const timeoutMs = isPicker ? PICKER_TIMEOUT_MS : isAgentStart ? AGENT_START_TIMEOUT_MS : PROXY_TIMEOUT_MS;
+    const startedAt = Date.now();
     let upstream;
     const controller = new AbortController();
     const timer = setTimeout(() => controller.abort(), timeoutMs);
@@ -148,8 +155,13 @@ class ModuleBridge {
       clearTimeout(timer);
       res.off?.('close', disconnected);
       if (res.destroyed) return;
+      const details = { failureKind: controller.signal.aborted ? 'UPSTREAM_TIMEOUT' : 'NETWORK_ERROR', timeoutMs };
+      this.logger.warn('module.proxy.failed', '模块请求未完成，请检查连接或客户端启动。', {
+        component: 'module-bridge', phase: 'proxy', durationMs: Date.now() - startedAt,
+        context: { ...details, method: req.method, path: pathname },
+      });
       res.writeHead(502, { 'Content-Type': 'application/json; charset=utf-8' });
-      res.end(JSON.stringify({ ok: false, error: 'MODULE_UNREACHABLE', target }));
+      res.end(JSON.stringify({ ok: false, error: 'MODULE_UNREACHABLE', target: targetUrl.origin + pathname, details }));
       return;
     }
     const contentType = upstream.headers.get('content-type') || 'application/json; charset=utf-8';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sanqianx/project-knowledge",
-  "version": "5.0.9-rc.12",
+  "version": "5.0.9-rc.13",
   "description": "Knowledge base manager with Git integration, AI-driven analysis, and bilingual (zh-CN/en-US) knowledge output",
   "main": "_site/server.js",
   "scripts": {

package/plugins/project-knowledge/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/claude-code-plugin-manifest.json",
   "name": "project-knowledge",
-  "version": "5.0.9-rc.12",
+  "version": "5.0.9-rc.13",
   "description": "Read durable project decisions and implementation history from Project Knowledge.",
   "author": {
     "name": "SanQianX",

package/plugins/project-knowledge/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "project-knowledge",
-  "version": "5.0.9-rc.12",
+  "version": "5.0.9-rc.13",
   "description": "Read durable project decisions and implementation history from Project Knowledge.",
   "author": {
     "name": "SanQianX",

package/plugins/project-knowledge/.mcp.json

@@ -6,7 +6,7 @@
       "args": [
         "-y",
         "--package",
-        "@sanqianx/project-knowledge@5.0.9-rc.12",
+        "@sanqianx/project-knowledge@5.0.9-rc.13",
         "project-knowledge-mcp"
       ]
     }