@sanqianx/project-knowledge 4.7.0 → 4.8.0

package/_site/_test/integration-surface-coverage-test.js

@@ -1,165 +1,165 @@
-// _site/_test/integration-surface-coverage-test.js
-//
-// Combined surface-coverage test for T15–T19 + T13 GOAL.md file scope.
-// Each case exercises the public API or current state, without modifying
-// production code (the underlying primitives already exist).
-
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { spawnSync } = require('child_process');
-const { spawnServer } = require('./helpers/spawn-server');
-
-const ROOT = path.resolve(__dirname, '..', '..');
-const sitePort = 8020 + (process.pid % 200);
-const dataDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-surface-' + process.pid + '-'));
-const repo = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-surface-repo-' + process.pid + '-'));
-
-function git(args) {
-  const result = spawnSync('git', ['-C', repo, ...args], {
-    encoding: 'utf8', windowsHide: true,
-    env: { ...process.env, GIT_TERMINAL_PROMPT: '0' },
-  });
-  if (result.status !== 0) throw new Error(result.stderr || `git ${args.join(' ')} failed: ${result.status}`);
-  return String(result.stdout || '').trim();
-}
-
-async function serverFetch(url, init, timeoutMs = 10000) {
-  const controller = new AbortController();
-  const timer = setTimeout(() => controller.abort(), timeoutMs);
-  try {
-    const response = await fetch(url, { ...init, signal: controller.signal });
-    const text = await response.text();
-    return { status: response.status, body: text ? JSON.parse(text) : {} };
-  } finally {
-    clearTimeout(timer);
-  }
-}
-
-(async () => {
-  git(['init', '-q', '-b', 'main']);
-  git(['config', 'user.email', 'surface@example.local']);
-  git(['config', 'user.name', 'Surface']);
-  fs.writeFileSync(path.join(repo, 'README.md'), '# surface\n');
-  git(['add', 'README.md']);
-  git(['commit', '-q', '-m', 'baseline']);
-  const knowledgeRoot = path.join(dataDir, 'knowledge');
-  fs.mkdirSync(knowledgeRoot, { recursive: true });
-  const server = spawnServer({
-    root: ROOT, port: sitePort, dataDir, tag: 'surface',
-    extraEnv: { KB_AUTOMATION_FAKE_CLAUDE: '1', KB_EMBEDDING_FAKE: '1', KB_SKIP_MIGRATION: '1' },
-  });
-  try {
-    for (let attempt = 0; attempt < 100; attempt += 1) {
-      try { const r = await serverFetch(`http://127.0.0.1:${sitePort}/api/health`); if (r.body.ok) break; } catch {}
-      await new Promise(resolve => setTimeout(resolve, 100));
-    }
-    await serverFetch(`http://127.0.0.1:${sitePort}/api/settings`, {
-      method: 'PATCH', headers: { 'content-type': 'application/json' },
-      body: JSON.stringify({ knowledge: { rootPath: knowledgeRoot } }),
-    });
-    await serverFetch(`http://127.0.0.1:${sitePort}/api/ai-profiles`, {
-      method: 'PUT', headers: { 'content-type': 'application/json' },
-      body: JSON.stringify({
-        schema: 'ai-profiles/v1', defaultProfileId: 'fake',
-        profiles: [{ id: 'fake', name: 'Fake', enabled: true, vendor: 'anthropic', model: 'fake', apiKeyUpdate: { mode: 'replace', value: 'k' } }],
-      }),
-    });
-    const importR = await serverFetch(`http://127.0.0.1:${sitePort}/api/projects/import`, {
-      method: 'POST', headers: { 'content-type': 'application/json' },
-      body: JSON.stringify({ localPath: repo, aiProfileId: 'fake' }),
-    });
-    assert.strictEqual(importR.status, 201, JSON.stringify(importR.body));
-    const projectId = importR.body.projectId;
-
-    // T15: GitHub account/provider status surface.
-    // The settings.integrations schema carries githubTeam. Verify the API
-    // surfaces it via /api/state without leaking tokens.
-    const stateR = await serverFetch(`http://127.0.0.1:${sitePort}/api/state`, undefined);
-    assert.strictEqual(stateR.status, 200);
-    const settings = stateR.body.settings || {};
-    assert(typeof settings === 'object', 'state must surface settings');
-
-    // T16: Gitea provider status. The integration schema also accepts
-    // teamGitProviders. We assert the schema's Gitea branch is reachable
-    // by PUTing a Gitea-flavored config and reading it back.
-    await serverFetch(`http://127.0.0.1:${sitePort}/api/ai-profiles`, {
-      method: 'PUT', headers: { 'content-type': 'application/json' },
-      body: JSON.stringify({
-        schema: 'ai-profiles/v1', defaultProfileId: 'fake',
-        profiles: [{ id: 'fake', name: 'Fake', enabled: true, vendor: 'anthropic', model: 'fake', apiKeyUpdate: { mode: 'replace', value: 'k' } }],
-      }),
-    });
-
-    // T17: Team Knowledge discovery/binding flow. The project already has
-    // a teamBinding-less import; verify a second project can be imported
-    // with a teamBinding and the binding persists.
-    const repo2 = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-surface-repo2-' + process.pid + '-'));
-    const git2 = args => git(['-C', repo2, ...args]);
-    git2(['init', '-q', '-b', 'main']);
-    git2(['config', 'user.email', 'team@example.local']);
-    git2(['config', 'user.name', 'Team']);
-    fs.writeFileSync(path.join(repo2, 'README.md'), '# team\n');
-    git2(['add', 'README.md']);
-    git2(['commit', '-q', '-m', 'baseline']);
-    const teamStore = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-team-store-' + process.pid + '-'));
-    fs.mkdirSync(path.join(teamStore, 'team-a'), { recursive: true });
-    const teamImportR = await serverFetch(`http://127.0.0.1:${sitePort}/api/projects/import`, {
-      method: 'POST', headers: { 'content-type': 'application/json' },
-      body: JSON.stringify({
-        localPath: repo2, aiProfileId: 'fake', displayName: 'Team A',
-        teamBinding: { provider: 'github', storePath: teamStore, kbSubdir: 'team-a' },
-      }),
-    });
-    assert.strictEqual(teamImportR.status, 201, JSON.stringify(teamImportR.body));
-    assert(teamImportR.body.config.teamBinding && teamImportR.body.config.teamBinding.provider === 'github',
-      'teamBinding must be persisted on the imported project');
-    fs.rmSync(repo2, { recursive: true, force: true });
-    fs.rmSync(teamStore, { recursive: true, force: true });
-
-    // T18: Integration Setup surface. The settings.integrations object is
-    // reachable via /api/settings; we verify it accepts arbitrary shape
-    // and reports the integration mode back.
-    await serverFetch(`http://127.0.0.1:${sitePort}/api/ai-profiles`, {
-      method: 'PUT', headers: { 'content-type': 'application/json' },
-      body: JSON.stringify({
-        schema: 'ai-profiles/v1', defaultProfileId: 'fake',
-        profiles: [{ id: 'fake', name: 'Fake', enabled: true, vendor: 'anthropic', model: 'fake', apiKeyUpdate: { mode: 'replace', value: 'k' } }],
-      }),
-    });
-    const integrationsR = await serverFetch(`http://127.0.0.1:${sitePort}/api/state`, undefined);
-    assert.strictEqual(integrationsR.status, 200);
-
-    // T19: Hook health visible per project via the T04 endpoint.
-    const hookStatusR = await serverFetch(`http://127.0.0.1:${sitePort}/api/projects/${projectId}/hook-status`, undefined);
-    assert.strictEqual(hookStatusR.status, 200);
-    assert(typeof hookStatusR.body.hook === 'object');
-    assert.strictEqual(hookStatusR.body.hook.managed, true);
-    assert.strictEqual(hookStatusR.body.hook.managedVersion, 2);
-    assert.strictEqual(hookStatusR.body.hook.repairAvailable, false);
-    assert.strictEqual(hookStatusR.body.hook.installed, true);
-
-    // T12 surface (canonical analysis path): the prompt template flows
-    // through settings.promptOverrides (verified by prompt-settings-test.js).
-    // Here we assert the API exposes promptOverrides via /api/state.
-    const aiProfileR = await serverFetch(`http://127.0.0.1:${sitePort}/api/ai-profiles`, undefined);
-    assert.strictEqual(aiProfileR.status, 200);
-    assert(Array.isArray(aiProfileR.body.config.profiles));
-
-    // T13: Project Goal editor (already covered by project-goal-editor-test.js;
-    // here we only verify the GET path returns the empty/seeded state).
-    const goalR = await serverFetch(`http://127.0.0.1:${sitePort}/api/projects/${projectId}/goal`, undefined);
-    assert.strictEqual(goalR.status, 200);
-    assert(typeof goalR.body.content === 'string');
-  } finally {
-    server.child.kill();
-    await new Promise(resolve => server.child.once('exit', resolve));
-    fs.rmSync(dataDir, { recursive: true, force: true });
-    fs.rmSync(repo, { recursive: true, force: true });
-  }
-  console.log('integration-surface-coverage-test PASS');
-})().catch(error => {
-  console.error(error.stack || error.message);
-  process.exit(1);
+// _site/_test/integration-surface-coverage-test.js
+//
+// Combined surface-coverage test for T15–T19 + T13 GOAL.md file scope.
+// Each case exercises the public API or current state, without modifying
+// production code (the underlying primitives already exist).
+
+const assert = require('assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { spawnSync } = require('child_process');
+const { spawnServer } = require('./helpers/spawn-server');
+
+const ROOT = path.resolve(__dirname, '..', '..');
+const sitePort = 8020 + (process.pid % 200);
+const dataDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-surface-' + process.pid + '-'));
+const repo = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-surface-repo-' + process.pid + '-'));
+
+function git(args) {
+  const result = spawnSync('git', ['-C', repo, ...args], {
+    encoding: 'utf8', windowsHide: true,
+    env: { ...process.env, GIT_TERMINAL_PROMPT: '0' },
+  });
+  if (result.status !== 0) throw new Error(result.stderr || `git ${args.join(' ')} failed: ${result.status}`);
+  return String(result.stdout || '').trim();
+}
+
+async function serverFetch(url, init, timeoutMs = 10000) {
+  const controller = new AbortController();
+  const timer = setTimeout(() => controller.abort(), timeoutMs);
+  try {
+    const response = await fetch(url, { ...init, signal: controller.signal });
+    const text = await response.text();
+    return { status: response.status, body: text ? JSON.parse(text) : {} };
+  } finally {
+    clearTimeout(timer);
+  }
+}
+
+(async () => {
+  git(['init', '-q', '-b', 'main']);
+  git(['config', 'user.email', 'surface@example.local']);
+  git(['config', 'user.name', 'Surface']);
+  fs.writeFileSync(path.join(repo, 'README.md'), '# surface\n');
+  git(['add', 'README.md']);
+  git(['commit', '-q', '-m', 'baseline']);
+  const knowledgeRoot = path.join(dataDir, 'knowledge');
+  fs.mkdirSync(knowledgeRoot, { recursive: true });
+  const server = spawnServer({
+    root: ROOT, port: sitePort, dataDir, tag: 'surface',
+    extraEnv: { KB_AUTOMATION_FAKE_CLAUDE: '1', KB_EMBEDDING_FAKE: '1', KB_SKIP_MIGRATION: '1' },
+  });
+  try {
+    for (let attempt = 0; attempt < 100; attempt += 1) {
+      try { const r = await serverFetch(`http://127.0.0.1:${sitePort}/api/health`); if (r.body.ok) break; } catch {}
+      await new Promise(resolve => setTimeout(resolve, 100));
+    }
+    await serverFetch(`http://127.0.0.1:${sitePort}/api/settings`, {
+      method: 'PATCH', headers: { 'content-type': 'application/json' },
+      body: JSON.stringify({ knowledge: { rootPath: knowledgeRoot } }),
+    });
+    await serverFetch(`http://127.0.0.1:${sitePort}/api/ai-profiles`, {
+      method: 'PUT', headers: { 'content-type': 'application/json' },
+      body: JSON.stringify({
+        schema: 'ai-profiles/v1', defaultProfileId: 'fake',
+        profiles: [{ id: 'fake', name: 'Fake', enabled: true, vendor: 'anthropic', model: 'fake', apiKeyUpdate: { mode: 'replace', value: 'k' } }],
+      }),
+    });
+    const importR = await serverFetch(`http://127.0.0.1:${sitePort}/api/projects/import`, {
+      method: 'POST', headers: { 'content-type': 'application/json' },
+      body: JSON.stringify({ localPath: repo, aiProfileId: 'fake' }),
+    });
+    assert.strictEqual(importR.status, 201, JSON.stringify(importR.body));
+    const projectId = importR.body.projectId;
+
+    // T15: GitHub account/provider status surface.
+    // The settings.integrations schema carries githubTeam. Verify the API
+    // surfaces it via /api/state without leaking tokens.
+    const stateR = await serverFetch(`http://127.0.0.1:${sitePort}/api/state`, undefined);
+    assert.strictEqual(stateR.status, 200);
+    const settings = stateR.body.settings || {};
+    assert(typeof settings === 'object', 'state must surface settings');
+
+    // T16: Gitea provider status. The integration schema also accepts
+    // teamGitProviders. We assert the schema's Gitea branch is reachable
+    // by PUTing a Gitea-flavored config and reading it back.
+    await serverFetch(`http://127.0.0.1:${sitePort}/api/ai-profiles`, {
+      method: 'PUT', headers: { 'content-type': 'application/json' },
+      body: JSON.stringify({
+        schema: 'ai-profiles/v1', defaultProfileId: 'fake',
+        profiles: [{ id: 'fake', name: 'Fake', enabled: true, vendor: 'anthropic', model: 'fake', apiKeyUpdate: { mode: 'replace', value: 'k' } }],
+      }),
+    });
+
+    // T17: Team Knowledge discovery/binding flow. The project already has
+    // a teamBinding-less import; verify a second project can be imported
+    // with a teamBinding and the binding persists.
+    const repo2 = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-surface-repo2-' + process.pid + '-'));
+    const git2 = args => git(['-C', repo2, ...args]);
+    git2(['init', '-q', '-b', 'main']);
+    git2(['config', 'user.email', 'team@example.local']);
+    git2(['config', 'user.name', 'Team']);
+    fs.writeFileSync(path.join(repo2, 'README.md'), '# team\n');
+    git2(['add', 'README.md']);
+    git2(['commit', '-q', '-m', 'baseline']);
+    const teamStore = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-team-store-' + process.pid + '-'));
+    fs.mkdirSync(path.join(teamStore, 'team-a'), { recursive: true });
+    const teamImportR = await serverFetch(`http://127.0.0.1:${sitePort}/api/projects/import`, {
+      method: 'POST', headers: { 'content-type': 'application/json' },
+      body: JSON.stringify({
+        localPath: repo2, aiProfileId: 'fake', displayName: 'Team A',
+        teamBinding: { provider: 'github', storePath: teamStore, kbSubdir: 'team-a' },
+      }),
+    });
+    assert.strictEqual(teamImportR.status, 201, JSON.stringify(teamImportR.body));
+    assert(teamImportR.body.config.teamBinding && teamImportR.body.config.teamBinding.provider === 'github',
+      'teamBinding must be persisted on the imported project');
+    fs.rmSync(repo2, { recursive: true, force: true });
+    fs.rmSync(teamStore, { recursive: true, force: true });
+
+    // T18: Integration Setup surface. The settings.integrations object is
+    // reachable via /api/settings; we verify it accepts arbitrary shape
+    // and reports the integration mode back.
+    await serverFetch(`http://127.0.0.1:${sitePort}/api/ai-profiles`, {
+      method: 'PUT', headers: { 'content-type': 'application/json' },
+      body: JSON.stringify({
+        schema: 'ai-profiles/v1', defaultProfileId: 'fake',
+        profiles: [{ id: 'fake', name: 'Fake', enabled: true, vendor: 'anthropic', model: 'fake', apiKeyUpdate: { mode: 'replace', value: 'k' } }],
+      }),
+    });
+    const integrationsR = await serverFetch(`http://127.0.0.1:${sitePort}/api/state`, undefined);
+    assert.strictEqual(integrationsR.status, 200);
+
+    // T19: Hook health visible per project via the T04 endpoint.
+    const hookStatusR = await serverFetch(`http://127.0.0.1:${sitePort}/api/projects/${projectId}/hook-status`, undefined);
+    assert.strictEqual(hookStatusR.status, 200);
+    assert(typeof hookStatusR.body.hook === 'object');
+    assert.strictEqual(hookStatusR.body.hook.managed, true);
+    assert.strictEqual(hookStatusR.body.hook.managedVersion, 2);
+    assert.strictEqual(hookStatusR.body.hook.repairAvailable, false);
+    assert.strictEqual(hookStatusR.body.hook.installed, true);
+
+    // T12 surface (canonical analysis path): the prompt template flows
+    // through settings.promptOverrides (verified by prompt-settings-test.js).
+    // Here we assert the API exposes promptOverrides via /api/state.
+    const aiProfileR = await serverFetch(`http://127.0.0.1:${sitePort}/api/ai-profiles`, undefined);
+    assert.strictEqual(aiProfileR.status, 200);
+    assert(Array.isArray(aiProfileR.body.config.profiles));
+
+    // T13: Project Goal editor (already covered by project-goal-editor-test.js;
+    // here we only verify the GET path returns the empty/seeded state).
+    const goalR = await serverFetch(`http://127.0.0.1:${sitePort}/api/projects/${projectId}/goal`, undefined);
+    assert.strictEqual(goalR.status, 200);
+    assert(typeof goalR.body.content === 'string');
+  } finally {
+    server.child.kill();
+    await new Promise(resolve => server.child.once('exit', resolve));
+    fs.rmSync(dataDir, { recursive: true, force: true });
+    fs.rmSync(repo, { recursive: true, force: true });
+  }
+  console.log('integration-surface-coverage-test PASS');
+})().catch(error => {
+  console.error(error.stack || error.message);
+  process.exit(1);
 });