fuli-context 0.5.0 → 0.7.3

package/src/mcp/instructions.js

@@ -1,9 +1,9 @@
 export const MCP_INSTRUCTIONS = [
-  "Each user task: if begin_task_context supplied hook-provided task context, apply. Otherwise call exactly get_collaboration_preferences with projectPath=cwd before other tools/answer as the fallback; never substitute a project action or Fuli tool. Apply all effective_preferences: personal-global everywhere plus only Fuli's matched project. Resolve relevant deferred_conflict first. Never guess personalProjectId. Writes enforce preferences in the actual payload; final text is not compliance.",
-  'For current-project facts use search_current_project_knowledge: exact active child first, then inheritable parent knowledge via outgoing PART_OF/USES_KNOWLEDGE_FROM within two hops. Child stable keys override. Never inherit project preferences or traverse RELATED_TO.',
-  'Before claiming facts unknown, use search_knowledge_graph. Use exact IDs for extra projects; mark pending. Widen a location miss to all_local_confirmed only after consent, then use rg only in the safe current repo/workspace. Never scan outside it. Live URLs need deployment evidence; changing state needs monitoring/Git MCP.',
-  'Use record_decision_trace for confirmed choices/rationale/validation. Use record_knowledge_feedback when retained knowledge is rejected, contradicted, outdated, or fails validation; preserve history.',
-  'Batch durable confirmed knowledge via capture_session_knowledge in exact scope. Never store secrets, transcripts, or guesses. Only human/source confirmation permits public review. Finish a hook-managed task with checkpoint_task_knowledge once: capture_candidates or retain_nothing; fallback Agents likewise flush or retain nothing.',
-  'When personal knowledge materially affects the answer/action, call record_knowledge_usage. Retrieval/injection never counts; it creates at most agent_confirmed and is never public-eligible.',
-  'Terminal-safe Markdown. Final MUST begin with sourceMarker.leadMarkdown; append sourceMarker.markdown unchanged only on match. On miss use noMatchSourceMarker leadMarkdown only (empty markdown). Never wrap markers in HTML or cite writes.'
+  "Each user task: apply hook-provided task context from begin_task_context. Otherwise, before other tools/answer, call exactly get_collaboration_preferences with projectPath=cwd and taskPrompt=current user request as fallback; never substitute a project action/Fuli tool. Apply all effective_preferences: personal-global everywhere plus only Fuli's matched project. Resolve relevant deferred_conflict first. Never guess personalProjectId. Writes enforce preferences in the actual payload; final text is not compliance.",
+  'Before asking the user to repeat a stable project fact or method, inspect task_knowledge_recall. On miss use search_current_project_knowledge with 1-4 focused action, artifact, target-system, or identifier queries; never use the whole conversational request as the only query. Search the exact active child first, then inheritable parent PART_OF/USES_KNOWLEDGE_FROM knowledge within two hops; child stable keys override. Never inherit project preferences or traverse RELATED_TO.',
+  'Before claiming facts unknown, use search_knowledge_graph. Use exact IDs for extra projects. Widen a miss to all_local_confirmed only after consent, then rg only in the safe current repo/workspace. Never scan outside it. Live state needs deployment/monitoring/Git MCP evidence. External/public evidence: search_connected_knowledge; agent_decide is response-only.',
+  'Use record_decision_trace for confirmed choices and record_knowledge_feedback when knowledge is rejected, contradicted, outdated, or fails validation.',
+  'Batch durable confirmed knowledge via capture_session_knowledge in exact scope; never secrets, transcripts, or guesses. Finish once with checkpoint_task_knowledge: capture_candidates or retain_nothing.',
+  'After material personal-knowledge use call record_knowledge_usage; retrieval alone never counts or permits public use.',
+  'Terminal-safe Markdown. Final MUST begin with sourceMarker.leadMarkdown; append sourceMarker.markdown unchanged on match. On miss use noMatchSourceMarker leadMarkdown only (empty markdown). Never wrap markers in HTML.'
 ].join(' ');

package/src/mcp/tool-annotations.js

@@ -5,6 +5,7 @@ const READ_TOOLS = new Set([
   'preview_common_knowledge_promotion',
   'list_knowledge_spaces',
   'list_personal_projects',
+  'list_knowledge_review_candidates',
   'preview_personal_project_action',
   'list_project_releases',
   'list_project_relations',
@@ -21,6 +22,7 @@ const WRITE_TOOLS = new Set([
   'capture_session_knowledge',
   'record_decision_trace',
   'search_knowledge_graph',
+  'search_connected_knowledge',
   'record_knowledge_usage',
   'record_knowledge_feedback',
   'apply_common_knowledge_promotion',
@@ -28,6 +30,9 @@ const WRITE_TOOLS = new Set([
   'search_human_knowledge_changes',
   'review_human_knowledge_change',
   'upsert_personal_project',
+  'start_knowledge_review',
+  'record_knowledge_review_progress',
+  'finish_knowledge_review',
   'revise_personal_knowledge',
   'reassign_personal_knowledge',
   'set_personal_preference_scope',
@@ -64,6 +69,7 @@ export function annotationsFor(name) {
       name === 'record_knowledge_feedback' ||
       name === 'checkpoint_task_knowledge',
     openWorldHint: name === 'capture_session_knowledge' ||
+      name === 'search_connected_knowledge' ||
       name === 'record_decision_trace' ||
       name === 'record_knowledge_feedback' ||
       name === 'checkpoint_task_knowledge' ||

package/src/mcp/tool-result.js

@@ -4,6 +4,7 @@ const OMITTED_KEYS = new Set([
   'store',
   'dbpath',
   'databasepath',
+  'privatepath',
   'snapshot',
   'fullsnapshot'
 ]);

package/src/server/application-lifecycle.js

@@ -1,21 +1,12 @@
-import { createApplication } from '../app/create-application.js';
 import { openFederatedGraphApplication } from '../graphiti/federated-application.js';
-import { openLocalApplication } from '../runtime-options.js';
 
 export function createServerApplication({
   app,
-  store,
-  dbPath,
   runtimeConfigPath,
-  personalSpaceName,
   closeApplicationOnShutdown
 }) {
-  const application = app ?? (store
-    ? createApplication({ store, activePersonalSpaceName: personalSpaceName })
-    : dbPath
-      ? openLocalApplication({ dbPath, personalSpaceName })
-      : openFederatedGraphApplication({ runtimeConfigPath }));
-  const owned = closeApplicationOnShutdown ?? (!app && !store);
+  const application = app ?? openFederatedGraphApplication({ runtimeConfigPath });
+  const owned = closeApplicationOnShutdown ?? !app;
   return {
     application,
     close: closeOnce(() => {

package/src/server/blocked-ports.js

@@ -0,0 +1,11 @@
+const FETCH_BLOCKED_PORTS = new Set([
+  1, 7, 9, 11, 13, 15, 17, 19, 20, 21, 22, 23, 25, 37, 42, 43, 53, 69, 77, 79, 87, 95,
+  101, 102, 103, 104, 109, 110, 111, 113, 115, 117, 119, 123, 135, 137, 139, 143, 161, 179, 389,
+  427, 465, 512, 513, 514, 515, 526, 530, 531, 532, 540, 548, 554, 556, 563, 587, 601, 636,
+  989, 990, 993, 995, 1719, 1720, 1723, 2049, 3659, 4045, 4190, 5060, 5061, 6000, 6566,
+  6665, 6666, 6667, 6668, 6669, 6697, 10080
+]);
+
+export function isFetchBlockedPort(port) {
+  return FETCH_BLOCKED_PORTS.has(Number(port));
+}

package/src/server.js

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 import { createServer as createHttpServer } from 'node:http';
+import { dirname } from 'node:path';
 import { fileURLToPath } from 'node:url';
 
 import { handleApiRequest } from './http/api-router.js';
@@ -9,6 +10,7 @@ import { sendJson } from './http/response.js';
 import { serveStatic } from './http/static-handler.js';
 import { DEFAULT_FULI_PORT } from './defaults.js';
 import { resolveGraphRuntimeOptions } from './graphiti/runtime-config.js';
+import { isFetchBlockedPort } from './server/blocked-ports.js';
 import {
   discoverLanAddresses,
   lanConsoleUrls,
@@ -16,19 +18,14 @@ import {
 } from './server/lan-access.js';
 import { createServerApplication } from './server/application-lifecycle.js';
 import { listenServer } from './server/listen.js';
+import { resolveSetupPaths } from './setup/paths.js';
+import { createSystemService } from './system/system-service.js';
 
-const FETCH_BLOCKED_PORTS = new Set([
-  1, 7, 9, 11, 13, 15, 17, 19, 20, 21, 22, 23, 25, 37, 42, 43, 53, 69, 77, 79, 87, 95, 101, 102,
-  103, 104, 109, 110, 111, 113, 115, 117, 119, 123, 135, 137, 139, 143, 161, 179, 389, 427, 465,
-  512, 513, 514, 515, 526, 530, 531, 532, 540, 548, 554, 556, 563, 587, 601, 636, 989, 990, 993,
-  995, 1719, 1720, 1723, 2049, 3659, 4045, 4190, 5060, 5061, 6000, 6566, 6665, 6666, 6667, 6668,
-  6669, 6697, 10080
-]);
+const PACKAGE_ROOT = dirname(dirname(fileURLToPath(import.meta.url)));
 
 export async function createServer(options = {}) {
   const {
     port = DEFAULT_FULI_PORT,
-    store,
     app,
     closeApplicationOnShutdown,
     isBlockedPort = isFetchBlockedPort,
@@ -47,17 +44,28 @@ export async function createServer(options = {}) {
   if (Number(port) !== 0 && isBlockedPort(port)) {
     throw new Error(`Port ${port} is blocked for local fetch`);
   }
-  const legacyRuntime = app || store || Object.hasOwn(options, 'dbPath');
-  const localOptions = legacyRuntime
-    ? { dbPath: options.dbPath, personalSpaceName: options.personalSpaceName ?? '我' }
-    : resolveServerGraphOptions(options);
+  const localOptions = app ? {} : resolveServerGraphOptions(options);
   const runtime = createServerApplication({
     app,
-    store,
     ...localOptions,
     closeApplicationOnShutdown
   });
   const application = runtime.application;
+  const externalKnowledge = options.externalKnowledge ??
+    application.externalKnowledge ?? null;
+  const connectedKnowledge = options.connectedKnowledge ??
+    application.connectedKnowledge ?? null;
+  const system = options.system ?? (!app
+    ? createSystemService({
+        paths: resolveSetupPaths({
+          dataDir: dirname(localOptions.runtimeConfigPath),
+          packageRoot: PACKAGE_ROOT
+        }),
+        packageRoot: PACKAGE_ROOT,
+        activePort: port,
+        activeLan: lanEnabled
+      })
+    : null);
 
   for (let attempt = 0; attempt < 20; attempt += 1) {
     let authority = null;
@@ -67,6 +75,9 @@ export async function createServer(options = {}) {
         request,
         response,
         app: application,
+        system,
+        externalKnowledge,
+        connectedKnowledge,
         authority,
         lanAuthorities: allowedLanAuthorities,
         lanAccessToken: lanToken
@@ -122,14 +133,15 @@ function resolveServerGraphOptions(options) {
   });
 }
 
-export function isFetchBlockedPort(port) {
-  return FETCH_BLOCKED_PORTS.has(Number(port));
-}
+export { isFetchBlockedPort } from './server/blocked-ports.js';
 
 async function handleRequest({
   request,
   response,
   app,
+  system,
+  externalKnowledge,
+  connectedKnowledge,
   authority,
   lanAuthorities,
   lanAccessToken
@@ -141,7 +153,14 @@ async function handleRequest({
     lanAuthorities,
     lanAccessToken
   })) return;
-  if (await handleApiRequest({ request, response, app })) return;
+  if (await handleApiRequest({
+    request,
+    response,
+    app,
+    system,
+    externalKnowledge,
+    connectedKnowledge
+  })) return;
   serveStatic(new URL(request.url, 'http://127.0.0.1').pathname, response);
 }
 

package/src/setup/agent-installation-status.js

@@ -17,7 +17,8 @@ export function inspectAgentInstallations(agents, context, {
     const registration = inspectRegistration(agent, context, { readText, readJson });
     const skillPairs = [
       [context.sessionSkillPath, agent.skillPath],
-      [context.projectSkillPath, agent.projectSkillPath]
+      [context.projectSkillPath, agent.projectSkillPath],
+      [context.reviewSkillPath, agent.reviewSkillPath]
     ];
     const skillsPresent = skillPairs.some(([, targetPath]) => fileExists(targetPath));
     const skillsCurrent = skillPairs.every(

package/src/setup/agents.js

@@ -9,6 +9,7 @@ import {
 } from './claude-code-config.js';
 import { connectCursor, disconnectCursor } from './cursor-config.js';
 import { connectCodex, disconnectCodex } from './codex-config.js';
+import { adaptAgentForReviewSkill } from './review-skill-adapters.js';
 
 export function discoverAgents({
   platform = process.platform,
@@ -52,7 +53,7 @@ export function discoverAgents({
       projectSkillPath: pathApi.join(homeDir, '.cursor', 'skills', projectSkillName),
       available: Boolean(commandExists('cursor'))
     }
-  ];
+  ].map((agent) => adaptAgentForReviewSkill(agent, { homeDir, pathApi }));
 }
 
 export function buildAgentCommands(agent, context) {

package/src/setup/claude-code-config.js

@@ -115,7 +115,8 @@ function withManagedHooks(settings) {
     ...(hooks.UserPromptSubmit ?? []),
     managedHookGroup('begin_task_context', {
       sessionId: '${session_id}',
-      projectPath: '${cwd}'
+      projectPath: '${cwd}',
+      taskPrompt: '${prompt}'
     }, 'Loading Fuli task context')
   ];
   hooks.Stop = [

package/src/setup/codex-bootstrap.js

@@ -9,18 +9,23 @@ import {
 } from 'node:fs';
 import { basename, dirname, join } from 'node:path';
 
+import { codexReviewCommandBridge } from './review-skill-adapters.js';
+
 const BLOCK_START = '<!-- BEGIN FULI MANAGED INSTRUCTIONS -->';
 const BLOCK_END = '<!-- END FULI MANAGED INSTRUCTIONS -->';
 
 const GLOBAL_INSTRUCTIONS = `${BLOCK_START}
 At the start of every user task, before any other tool or answer, call exactly
-\`get_collaboration_preferences\` with \`projectPath\`=current working directory. Never substitute
-project action or Fuli tool.
-Apply all returned \`effective_preferences\` before answers or tools. For write tools, enforce
-them in the actual payload; mentioning them later is not compliance.
-Before using a relevant \`deferred_conflict\`, call
-\`resolve_deferred_preference_conflict\`; ignore unrelated ones.
-The current request wins on conflict.
+\`get_collaboration_preferences\` with \`projectPath\`=current working directory and
+\`taskPrompt\`=current user request; never substitute a project action/Fuli tool. Fuli uses both
+transiently and never stores or returns them. Apply all returned \`effective_preferences\`.
+Before asking the user to repeat a stable project fact or method, inspect
+\`task_knowledge_recall\`. On miss call \`search_current_project_knowledge\` with 1-4 focused
+action/artifact/target/identifier queries; never use the full request as the only query.
+For write tools enforce them in the actual payload; mentioning them later is not compliance.
+Relevant \`deferred_conflict\`: call \`resolve_deferred_preference_conflict\` first; ignore others.
+Current request wins.
+${codexReviewCommandBridge()}
 ${BLOCK_END}`;
 
 export function installCodexBootstrap(agent, _context, {

package/src/setup/container-runtime.js

@@ -86,7 +86,7 @@ export async function ensureContainerRuntime(options = {}, dependencies = {}) {
     throw containerRuntimeError(runtime, options.platform ?? process.platform);
   }
 
-  onProgress(`正在启动 ${runtime.desktop.label}，首次启动可能需要几分钟…`);
+  onProgress(`Starting ${runtime.desktop.label}; the first launch may take a few minutes...`);
   await launch(runtime.desktop, dependencies);
 
   const attempts = Math.max(1, Math.ceil(maxWaitMs / pollIntervalMs));
@@ -100,9 +100,9 @@ export async function ensureContainerRuntime(options = {}, dependencies = {}) {
   }
 
   throw new Error(
-    `${runtime.desktop?.label ?? runtime.label} 已启动，但容器引擎未在 ` +
-    `${Math.ceil(maxWaitMs / 1000)} 秒内就绪。请打开该应用查看错误，确认 docker info ` +
-    `可以成功后重新运行 fl setup。${detailSuffix(runtime.detail)}`
+    `${runtime.desktop?.label ?? runtime.label} started, but the container engine was not ready ` +
+    `within ${Math.ceil(maxWaitMs / 1000)} seconds. Open the application to inspect errors, ` +
+    `confirm that docker info succeeds, then run fl setup again.${detailSuffix(runtime.detail)}`
   );
 }
 
@@ -124,12 +124,14 @@ export function runDockerCompose(args, runtime = null, {
 
   const detail = safeProcessDetail(result, homeDir);
   if (/port is already allocated|address already in use|bind.+failed/i.test(detail)) {
-    throw new Error(`Fuli 需要的本地端口已被占用。${detailSuffix(detail)}`);
+    throw new Error(`A local port required by Fuli is already in use.${detailSuffix(detail)}`);
   }
   if (/cannot connect|connection refused|error during connect|docker daemon|\bEOF\b/i.test(detail)) {
-    throw new Error(`容器运行时在启动 Fuli Provider 时失去连接。${detailSuffix(detail)}`);
+    throw new Error(
+      `The container runtime disconnected while starting the Fuli Provider.${detailSuffix(detail)}`
+    );
   }
-  throw new Error(`Docker Compose 无法启动 Fuli Provider。${detailSuffix(detail)}`);
+  throw new Error(`Docker Compose could not start the Fuli Provider.${detailSuffix(detail)}`);
 }
 
 export function selectDockerEnvironment({
@@ -161,19 +163,21 @@ export function containerRuntimeError(runtime, platform = process.platform) {
   }
   if (runtime.status === 'missing-compose') {
     return new Error(
-      `已找到 Docker，但缺少 Docker Compose v2。请安装 Compose v2 后重新运行 fl setup。` +
+      'Docker was found, but Docker Compose v2 is missing. Install Compose v2, then run ' +
+      'fl setup again.' +
       detailSuffix(runtime.detail)
     );
   }
   if (runtime.explicitTarget) {
     return new Error(
-      `DOCKER_HOST 或 DOCKER_CONTEXT 指向的容器引擎不可用。请修复该配置并确认 ` +
-      `docker info 可以成功后重新运行 fl setup。${detailSuffix(runtime.detail)}`
+      'The container engine selected by DOCKER_HOST or DOCKER_CONTEXT is unavailable. Fix the ' +
+      `configuration, confirm that docker info succeeds, then run fl setup again.${detailSuffix(runtime.detail)}`
     );
   }
   return new Error(
-    `已找到 Docker 和 Compose，但容器引擎没有运行。请启动 Docker Desktop、` +
-    `Rancher Desktop 或当前 Docker 服务，确认 docker info 可以成功后重新运行 fl setup。` +
+    'Docker and Compose were found, but the container engine is not running. Start Docker ' +
+    'Desktop, Rancher Desktop, or the current Docker service; confirm that docker info succeeds; ' +
+    'then run fl setup again.' +
     detailSuffix(runtime.detail)
   );
 }
@@ -330,17 +334,17 @@ async function launchDesktopRuntime(desktop, { spawnProcess = spawn } = {}) {
 
 function missingRuntimeMessage(platform) {
   if (platform === 'darwin') {
-    return `未检测到可用的 Docker Compose 容器运行时。请安装并首次打开 Rancher Desktop ` +
-      `(${RANCHER_DOWNLOAD_URL}) 或 Docker Desktop (${DOCKER_DOWNLOAD_URL})，` +
-      `然后重新运行 fl setup。`;
+    return 'No usable Docker Compose container runtime was detected. Install and open Rancher ' +
+      `Desktop (${RANCHER_DOWNLOAD_URL}) or Docker Desktop (${DOCKER_DOWNLOAD_URL}), then run ` +
+      'fl setup again.';
   }
   if (platform === 'win32') {
-    return `未检测到可用的 Docker Compose 容器运行时。请先启用 WSL 2，再安装并首次打开 ` +
-      `Rancher Desktop (${RANCHER_DOWNLOAD_URL}) 或 Docker Desktop ` +
-      `(${DOCKER_DOWNLOAD_URL})，然后重新运行 fl setup。`;
+    return 'No usable Docker Compose container runtime was detected. Enable WSL 2, then install ' +
+      `and open Rancher Desktop (${RANCHER_DOWNLOAD_URL}) or Docker Desktop ` +
+      `(${DOCKER_DOWNLOAD_URL}), and run fl setup again.`;
   }
-  return `未检测到可用的 Docker Compose 容器运行时。请安装 Docker Engine + Compose v2，` +
-    `或 Rancher Desktop (${RANCHER_DOWNLOAD_URL})，然后重新运行 fl setup。`;
+  return 'No usable Docker Compose container runtime was detected. Install Docker Engine with ' +
+    `Compose v2, or Rancher Desktop (${RANCHER_DOWNLOAD_URL}), then run fl setup again.`;
 }
 
 function defaultDockerAvailable(env) {
@@ -372,7 +376,7 @@ function safeProcessDetail(result, homeDir) {
 }
 
 function detailSuffix(detail) {
-  return nonEmpty(detail) ? ` Docker 返回：${detail}` : '';
+  return nonEmpty(detail) ? ` Docker returned: ${detail}` : '';
 }
 
 function processSucceeded(result) {