@sanqianx/project-knowledge 4.7.0 → 5.0.0

package/_site/_test/ai-profile-resolver-test.js

@@ -1,137 +1,137 @@
-// _site/_test/ai-profile-resolver-test.js
-//
-// Tests for the shared effective AI profile resolver. The resolver is the
-// single source of truth for "which AI profile does this project use right
-// now" — replacing the inconsistent fallback logic that used to live in
-// Workbench start-session / send-input, the analyzer, the CommitReconciler,
-// and import default assignment.
-//
-// Resolution order (per T01):
-//   1. projectConfig.aiProfileId when usable
-//   2. settings.ai.defaultProfileId when usable
-//   3. first usable configured profile in stable configured order
-//   4. otherwise AI_PROFILE_REQUIRED
-
-const assert = require('assert');
-const { resolveEffectiveAiProfile, isUsable, profileById } = require('../lib/ai-profile-resolver');
-
-const PROFILE_A = { id: 'profile-a', name: 'A', enabled: true, vendor: 'anthropic', model: 'm' };
-const PROFILE_B = { id: 'profile-b', name: 'B', enabled: true, vendor: 'anthropic', model: 'm' };
-const PROFILE_DISABLED = { id: 'profile-disabled', name: 'Disabled', enabled: false, vendor: 'anthropic', model: 'm' };
-const PROFILE_ENABLED_UNDEFINED = { id: 'profile-default-enabled', name: 'DefaultEnabled', vendor: 'anthropic', model: 'm' };
-
-function settings(profiles, defaultProfileId) {
-  return { ai: { schema: 'ai-profiles/v1', profiles, defaultProfileId: defaultProfileId || null } };
-}
-
-(async () => {
-  // ---- Case 1: explicit project profile is preferred ----
-  {
-    const s = settings([PROFILE_A, PROFILE_B], 'profile-b');
-    const r = resolveEffectiveAiProfile(s, { aiProfileId: 'profile-a' });
-    assert.strictEqual(r.source, 'project');
-    assert.strictEqual(r.profileId, 'profile-a');
-    assert.strictEqual(r.profile.id, 'profile-a');
-  }
-
-  // ---- Case 2: default profile fallback when project has none ----
-  {
-    const s = settings([PROFILE_A, PROFILE_B], 'profile-b');
-    const r = resolveEffectiveAiProfile(s, { aiProfileId: null });
-    assert.strictEqual(r.source, 'default');
-    assert.strictEqual(r.profileId, 'profile-b');
-  }
-
-  // ---- Case 3: first-usable fallback when default is missing/disabled ----
-  {
-    const s = settings([PROFILE_DISABLED, PROFILE_A, PROFILE_B], 'profile-disabled');
-    const r = resolveEffectiveAiProfile(s, { aiProfileId: null });
-    assert.strictEqual(r.source, 'first-usable');
-    assert.strictEqual(r.profileId, 'profile-a');
-  }
-
-  // ---- Case 4: stale project profile id falls back safely ----
-  {
-    const s = settings([PROFILE_A, PROFILE_B], 'profile-b');
-    const r = resolveEffectiveAiProfile(s, { aiProfileId: 'profile-deleted' });
-    assert.strictEqual(r.source, 'default');
-    assert.strictEqual(r.profileId, 'profile-b');
-  }
-
-  // ---- Case 5: disabled project profile falls back to default ----
-  {
-    const s = settings([PROFILE_DISABLED, PROFILE_B], 'profile-b');
-    const r = resolveEffectiveAiProfile(s, { aiProfileId: 'profile-disabled' });
-    assert.strictEqual(r.source, 'default');
-    assert.strictEqual(r.profileId, 'profile-b');
-  }
-
-  // ---- Case 6: no usable profile produces one clear error ----
-  {
-    const s = settings([PROFILE_DISABLED], null);
-    let threw = null;
-    try { resolveEffectiveAiProfile(s, { aiProfileId: null }); }
-    catch (error) { threw = error; }
-    assert(threw, 'resolver must throw when no usable profile exists');
-    assert.strictEqual(threw.code, 'AI_PROFILE_REQUIRED');
-    assert.strictEqual(threw.status, 409);
-    assert(threw.details && typeof threw.details.hint === 'string');
-  }
-
-  // ---- Case 7: enabled=undefined (legacy profiles) is treated as usable ----
-  {
-    const s = settings([PROFILE_ENABLED_UNDEFINED], 'profile-default-enabled');
-    const r = resolveEffectiveAiProfile(s, { aiProfileId: null });
-    assert.strictEqual(r.source, 'default');
-    assert.strictEqual(r.profileId, 'profile-default-enabled');
-  }
-
-  // ---- Case 8: empty settings produces AI_PROFILE_REQUIRED ----
-  {
-    let threw = null;
-    try { resolveEffectiveAiProfile({ ai: { profiles: [] } }, { aiProfileId: null }); }
-    catch (error) { threw = error; }
-    assert(threw, 'empty settings must throw AI_PROFILE_REQUIRED');
-    assert.strictEqual(threw.code, 'AI_PROFILE_REQUIRED');
-  }
-
-  // ---- Case 9: null/undefined inputs are tolerated ----
-  {
-    const r = resolveEffectiveAiProfile({ ai: { profiles: [PROFILE_A] } }, null);
-    assert.strictEqual(r.source, 'first-usable');
-    assert.strictEqual(r.profileId, 'profile-a');
-  }
-
-  // ---- Case 10: isUsable + profileById helpers ----
-  {
-    assert.strictEqual(isUsable(PROFILE_A), true);
-    assert.strictEqual(isUsable(PROFILE_DISABLED), false);
-    assert.strictEqual(isUsable(null), false);
-    const s = settings([PROFILE_A, PROFILE_B]);
-    assert.strictEqual(profileById(s, 'profile-a').id, 'profile-a');
-    assert.strictEqual(profileById(s, 'profile-unknown'), null);
-    assert.strictEqual(profileById(s, ''), null);
-  }
-
-  // ---- Case 11: deterministic first-usable order ----
-  {
-    const s = settings([PROFILE_A, PROFILE_B], null);
-    const r1 = resolveEffectiveAiProfile(s, { aiProfileId: null });
-    const r2 = resolveEffectiveAiProfile(s, { aiProfileId: null });
-    assert.strictEqual(r1.profileId, 'profile-a');
-    assert.strictEqual(r2.profileId, 'profile-a');
-  }
-
-  // ---- Case 12: default profile that is enabled=undefined still wins over first-usable ----
-  {
-    const s = settings([PROFILE_A, PROFILE_ENABLED_UNDEFINED], 'profile-default-enabled');
-    const r = resolveEffectiveAiProfile(s, { aiProfileId: null });
-    assert.strictEqual(r.source, 'default');
-    assert.strictEqual(r.profileId, 'profile-default-enabled');
-  }
-
-  console.log('ai-profile-resolver-test PASS');
-})().catch(error => {
-  console.error(error.stack || error.message);
-  process.exit(1);
-});
+// _site/_test/ai-profile-resolver-test.js
+//
+// Tests for the shared effective AI profile resolver. The resolver is the
+// single source of truth for "which AI profile does this project use right
+// now" — replacing the inconsistent fallback logic that used to live in
+// Workbench start-session / send-input, the analyzer, the CommitReconciler,
+// and import default assignment.
+//
+// Resolution order (per T01):
+//   1. projectConfig.aiProfileId when usable
+//   2. settings.ai.defaultProfileId when usable
+//   3. first usable configured profile in stable configured order
+//   4. otherwise AI_PROFILE_REQUIRED
+
+const assert = require('assert');
+const { resolveEffectiveAiProfile, isUsable, profileById } = require('../lib/ai-profile-resolver');
+
+const PROFILE_A = { id: 'profile-a', name: 'A', enabled: true, vendor: 'anthropic', model: 'm' };
+const PROFILE_B = { id: 'profile-b', name: 'B', enabled: true, vendor: 'anthropic', model: 'm' };
+const PROFILE_DISABLED = { id: 'profile-disabled', name: 'Disabled', enabled: false, vendor: 'anthropic', model: 'm' };
+const PROFILE_ENABLED_UNDEFINED = { id: 'profile-default-enabled', name: 'DefaultEnabled', vendor: 'anthropic', model: 'm' };
+
+function settings(profiles, defaultProfileId) {
+  return { ai: { schema: 'ai-profiles/v1', profiles, defaultProfileId: defaultProfileId || null } };
+}
+
+(async () => {
+  // ---- Case 1: explicit project profile is preferred ----
+  {
+    const s = settings([PROFILE_A, PROFILE_B], 'profile-b');
+    const r = resolveEffectiveAiProfile(s, { aiProfileId: 'profile-a' });
+    assert.strictEqual(r.source, 'project');
+    assert.strictEqual(r.profileId, 'profile-a');
+    assert.strictEqual(r.profile.id, 'profile-a');
+  }
+
+  // ---- Case 2: default profile fallback when project has none ----
+  {
+    const s = settings([PROFILE_A, PROFILE_B], 'profile-b');
+    const r = resolveEffectiveAiProfile(s, { aiProfileId: null });
+    assert.strictEqual(r.source, 'default');
+    assert.strictEqual(r.profileId, 'profile-b');
+  }
+
+  // ---- Case 3: first-usable fallback when default is missing/disabled ----
+  {
+    const s = settings([PROFILE_DISABLED, PROFILE_A, PROFILE_B], 'profile-disabled');
+    const r = resolveEffectiveAiProfile(s, { aiProfileId: null });
+    assert.strictEqual(r.source, 'first-usable');
+    assert.strictEqual(r.profileId, 'profile-a');
+  }
+
+  // ---- Case 4: stale project profile id falls back safely ----
+  {
+    const s = settings([PROFILE_A, PROFILE_B], 'profile-b');
+    const r = resolveEffectiveAiProfile(s, { aiProfileId: 'profile-deleted' });
+    assert.strictEqual(r.source, 'default');
+    assert.strictEqual(r.profileId, 'profile-b');
+  }
+
+  // ---- Case 5: disabled project profile falls back to default ----
+  {
+    const s = settings([PROFILE_DISABLED, PROFILE_B], 'profile-b');
+    const r = resolveEffectiveAiProfile(s, { aiProfileId: 'profile-disabled' });
+    assert.strictEqual(r.source, 'default');
+    assert.strictEqual(r.profileId, 'profile-b');
+  }
+
+  // ---- Case 6: no usable profile produces one clear error ----
+  {
+    const s = settings([PROFILE_DISABLED], null);
+    let threw = null;
+    try { resolveEffectiveAiProfile(s, { aiProfileId: null }); }
+    catch (error) { threw = error; }
+    assert(threw, 'resolver must throw when no usable profile exists');
+    assert.strictEqual(threw.code, 'AI_PROFILE_REQUIRED');
+    assert.strictEqual(threw.status, 409);
+    assert(threw.details && typeof threw.details.hint === 'string');
+  }
+
+  // ---- Case 7: enabled=undefined (legacy profiles) is treated as usable ----
+  {
+    const s = settings([PROFILE_ENABLED_UNDEFINED], 'profile-default-enabled');
+    const r = resolveEffectiveAiProfile(s, { aiProfileId: null });
+    assert.strictEqual(r.source, 'default');
+    assert.strictEqual(r.profileId, 'profile-default-enabled');
+  }
+
+  // ---- Case 8: empty settings produces AI_PROFILE_REQUIRED ----
+  {
+    let threw = null;
+    try { resolveEffectiveAiProfile({ ai: { profiles: [] } }, { aiProfileId: null }); }
+    catch (error) { threw = error; }
+    assert(threw, 'empty settings must throw AI_PROFILE_REQUIRED');
+    assert.strictEqual(threw.code, 'AI_PROFILE_REQUIRED');
+  }
+
+  // ---- Case 9: null/undefined inputs are tolerated ----
+  {
+    const r = resolveEffectiveAiProfile({ ai: { profiles: [PROFILE_A] } }, null);
+    assert.strictEqual(r.source, 'first-usable');
+    assert.strictEqual(r.profileId, 'profile-a');
+  }
+
+  // ---- Case 10: isUsable + profileById helpers ----
+  {
+    assert.strictEqual(isUsable(PROFILE_A), true);
+    assert.strictEqual(isUsable(PROFILE_DISABLED), false);
+    assert.strictEqual(isUsable(null), false);
+    const s = settings([PROFILE_A, PROFILE_B]);
+    assert.strictEqual(profileById(s, 'profile-a').id, 'profile-a');
+    assert.strictEqual(profileById(s, 'profile-unknown'), null);
+    assert.strictEqual(profileById(s, ''), null);
+  }
+
+  // ---- Case 11: deterministic first-usable order ----
+  {
+    const s = settings([PROFILE_A, PROFILE_B], null);
+    const r1 = resolveEffectiveAiProfile(s, { aiProfileId: null });
+    const r2 = resolveEffectiveAiProfile(s, { aiProfileId: null });
+    assert.strictEqual(r1.profileId, 'profile-a');
+    assert.strictEqual(r2.profileId, 'profile-a');
+  }
+
+  // ---- Case 12: default profile that is enabled=undefined still wins over first-usable ----
+  {
+    const s = settings([PROFILE_A, PROFILE_ENABLED_UNDEFINED], 'profile-default-enabled');
+    const r = resolveEffectiveAiProfile(s, { aiProfileId: null });
+    assert.strictEqual(r.source, 'default');
+    assert.strictEqual(r.profileId, 'profile-default-enabled');
+  }
+
+  console.log('ai-profile-resolver-test PASS');
+})().catch(error => {
+  console.error(error.stack || error.message);
+  process.exit(1);
+});

package/_site/_test/automation-queue-test.js

@@ -1,14 +1,14 @@
-// Run: node _site/_test/automation-queue-test.js
-
-const assert = require('assert');
-const fs = require('fs');
-const path = require('path');
-const automation = require('../lib/post-commit-automation');
-
-(() => {
-  assert.strictEqual(fs.existsSync(path.join(__dirname, '..', 'lib', 'automation-queue.js')), false, 'legacy in-memory automation queue must be deleted');
-  assert.strictEqual(fs.existsSync(path.join(__dirname, '..', 'lib', 'commit-automation-store.js')), false, 'legacy commit automation state must not remain a second pending authority');
-  assert.strictEqual(typeof automation.reconcileProjectCommits, 'function', 'Git-backed reconciler should be the single automation authority');
-  assert.strictEqual(Object.hasOwn(automation, 'dispatchPendingAutomations'), false, 'startup must not dispatch pending commit analysis');
-  console.log('automation-queue-test PASS');
-})();
+// Run: node _site/_test/automation-queue-test.js
+
+const assert = require('assert');
+const fs = require('fs');
+const path = require('path');
+const automation = require('../lib/post-commit-automation');
+
+(() => {
+  assert.strictEqual(fs.existsSync(path.join(__dirname, '..', 'lib', 'automation-queue.js')), false, 'legacy in-memory automation queue must be deleted');
+  assert.strictEqual(fs.existsSync(path.join(__dirname, '..', 'lib', 'commit-automation-store.js')), false, 'legacy commit automation state must not remain a second pending authority');
+  assert.strictEqual(typeof automation.reconcileProjectCommits, 'function', 'Git-backed reconciler should be the single automation authority');
+  assert.strictEqual(Object.hasOwn(automation, 'dispatchPendingAutomations'), false, 'startup must not dispatch pending commit analysis');
+  console.log('automation-queue-test PASS');
+})();

package/_site/_test/automation-ui-test.js

@@ -1,40 +1,40 @@
-const assert = require('assert');
-const fs = require('fs');
-const path = require('path');
-
-const ROOT = path.resolve(__dirname, '..', '..');
-const html = fs.readFileSync(path.join(ROOT, 'ui', 'index.html'), 'utf8');
-const app = fs.readFileSync(path.join(ROOT, 'ui', 'app.js'), 'utf8');
-const server = fs.readFileSync(path.join(ROOT, '_site', 'lib', 'server-app.js'), 'utf8');
-
-assert.strictEqual((html.match(/class="shell"/g) || []).length, 1, 'production UI must have one full product shell');
-assert.strictEqual((html.match(/id="settings-logs"/g) || []).length, 1, 'production UI must have one Settings logs section');
-assert(html.includes('id="view-terminal"') && html.includes('id="view-search"') && html.includes('id="view-import"'), 'Terminal, Search and Import must remain reachable');
-
-const forbiddenUi = [
-  /\/api\/projects\/[^"' ]+\/hook/,
-  /\/api\/projects\/[^"' ]+\/analy/,
-  /\/api\/projects\/[^"' ]+\/scan/,
-  /\/api\/projects\/[^"' ]+\/init/,
-  /simulate/i,
-  /installHook\s*\(/,
-  /uninstallHook\s*\(/,
-  /reinstallHook\s*\(/,
-  /manual analysis/i,
-];
-for (const pattern of forbiddenUi) assert(!pattern.test(html + app), 'removed manual automation control/call remains: ' + pattern);
-
-for (const route of [
-  '/api/projects/:projectId/hook/install',
-  '/api/projects/:projectId/hook/uninstall',
-  '/api/projects/:projectId/analyze/initial',
-  '/api/projects/:projectId/analyze/incremental',
-  '/api/projects/:projectId/simulate',
-]) {
-  assert(!server.includes(route), 'removed backend route remains: ' + route);
-}
-
-assert(!html.includes('vue.global.prod.js'), 'product shell must not require the old Vue runtime');
-assert(!html.includes('tailwind-browser.js'), 'product shell must not require the old browser Tailwind runtime');
-
-console.log('automation UI test PASS');
+const assert = require('assert');
+const fs = require('fs');
+const path = require('path');
+
+const ROOT = path.resolve(__dirname, '..', '..');
+const html = fs.readFileSync(path.join(ROOT, 'ui', 'index.html'), 'utf8');
+const app = fs.readFileSync(path.join(ROOT, 'ui', 'product.js'), 'utf8');
+const server = fs.readFileSync(path.join(ROOT, '_site', 'lib', 'server-app.js'), 'utf8');
+
+assert.strictEqual((html.match(/class="app"/g) || []).length, 1, 'production UI must have one full product shell');
+assert.strictEqual((html.match(/id="logsDialog"/g) || []).length, 1, 'production UI must have one diagnostics section');
+assert(html.includes('id="view-work"') && html.includes('id="view-search"') && html.includes('id="importDialog"'), 'Work, Search and Import must remain reachable');
+
+const forbiddenUi = [
+  /\/api\/projects\/[^"' ]+\/hook/,
+  /\/api\/projects\/[^"' ]+\/analy/,
+  /\/api\/projects\/[^"' ]+\/scan/,
+  /\/api\/projects\/[^"' ]+\/init/,
+  /simulate/i,
+  /installHook\s*\(/,
+  /uninstallHook\s*\(/,
+  /reinstallHook\s*\(/,
+  /manual analysis/i,
+];
+for (const pattern of forbiddenUi) assert(!pattern.test(html + app), 'removed manual automation control/call remains: ' + pattern);
+
+for (const route of [
+  '/api/projects/:projectId/hook/install',
+  '/api/projects/:projectId/hook/uninstall',
+  '/api/projects/:projectId/analyze/initial',
+  '/api/projects/:projectId/analyze/incremental',
+  '/api/projects/:projectId/simulate',
+]) {
+  assert(!server.includes(route), 'removed backend route remains: ' + route);
+}
+
+assert(!html.includes('vue.global.prod.js'), 'product shell must not require the old Vue runtime');
+assert(!html.includes('tailwind-browser.js'), 'product shell must not require the old browser Tailwind runtime');
+
+console.log('automation UI test PASS');

package/_site/_test/background-task-registry-test.js

@@ -1,43 +1,43 @@
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { activeTaskPromises, isProjectBusy, taskForProject } = require('../lib/server-app');
-
-function deferred() {
-  let resolve;
-  let reject;
-  const promise = new Promise((yes, no) => { resolve = yes; reject = no; });
-  return { promise, resolve, reject };
-}
-
-(async () => {
-  const records = [];
-  const runtime = {
-    activeTasks: new Map(),
-    logger: {
-      async debug(event, message, context) { records.push({ level: 'debug', event, message, ...context }); },
-      async error(event, message, context) { records.push({ level: 'error', event, message, ...context }); },
-    },
-  };
-  const longGate = deferred();
-  const long = taskForProject(runtime, 'project-a', 'op-long', () => longGate.promise, 'startup');
-  const fast = taskForProject(runtime, 'project-a', 'op-fast', async () => { throw new Error('fast failure'); }, 'git-hook');
-  assert.strictEqual(runtime.activeTasks.get('project-a').size, 2, 'overlapping operations must both remain registered');
-  assert.strictEqual(activeTaskPromises(runtime.activeTasks).length, 2, 'shutdown snapshot must include every operation promise');
-  assert.strictEqual(isProjectBusy('project-a', { readState: () => ({ analysis: { activeClaim: null } }) }, runtime.activeTasks), true);
-  await assert.rejects(fast, /fast failure/);
-  assert.strictEqual(runtime.activeTasks.get('project-a').size, 1, 'fast terminal must remove only its own operation');
-  assert(runtime.activeTasks.get('project-a').has('op-long'));
-  assert.strictEqual(activeTaskPromises(runtime.activeTasks).length, 1);
-  longGate.resolve('done');
-  assert.strictEqual(await long, 'done');
-  assert.strictEqual(runtime.activeTasks.has('project-a'), false, 'empty project task map must be removed');
-  assert(records.some(record => record.event === 'background.operation_failed' && record.operationId === 'op-fast'));
-  assert(records.some(record => record.event === 'background.operation_completed' && record.operationId === 'op-long'));
-
-  console.log('background-task-registry-test PASS');
-})().catch(error => {
-  console.error(error.stack || error.message);
-  process.exitCode = 1;
-});
+const assert = require('assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { activeTaskPromises, isProjectBusy, taskForProject } = require('../lib/server-app');
+
+function deferred() {
+  let resolve;
+  let reject;
+  const promise = new Promise((yes, no) => { resolve = yes; reject = no; });
+  return { promise, resolve, reject };
+}
+
+(async () => {
+  const records = [];
+  const runtime = {
+    activeTasks: new Map(),
+    logger: {
+      async debug(event, message, context) { records.push({ level: 'debug', event, message, ...context }); },
+      async error(event, message, context) { records.push({ level: 'error', event, message, ...context }); },
+    },
+  };
+  const longGate = deferred();
+  const long = taskForProject(runtime, 'project-a', 'op-long', () => longGate.promise, 'startup');
+  const fast = taskForProject(runtime, 'project-a', 'op-fast', async () => { throw new Error('fast failure'); }, 'git-hook');
+  assert.strictEqual(runtime.activeTasks.get('project-a').size, 2, 'overlapping operations must both remain registered');
+  assert.strictEqual(activeTaskPromises(runtime.activeTasks).length, 2, 'shutdown snapshot must include every operation promise');
+  assert.strictEqual(isProjectBusy('project-a', { readState: () => ({ analysis: { activeClaim: null } }) }, runtime.activeTasks), true);
+  await assert.rejects(fast, /fast failure/);
+  assert.strictEqual(runtime.activeTasks.get('project-a').size, 1, 'fast terminal must remove only its own operation');
+  assert(runtime.activeTasks.get('project-a').has('op-long'));
+  assert.strictEqual(activeTaskPromises(runtime.activeTasks).length, 1);
+  longGate.resolve('done');
+  assert.strictEqual(await long, 'done');
+  assert.strictEqual(runtime.activeTasks.has('project-a'), false, 'empty project task map must be removed');
+  assert(records.some(record => record.event === 'background.operation_failed' && record.operationId === 'op-fast'));
+  assert(records.some(record => record.event === 'background.operation_completed' && record.operationId === 'op-long'));
+
+  console.log('background-task-registry-test PASS');
+})().catch(error => {
+  console.error(error.stack || error.message);
+  process.exitCode = 1;
+});