@sanqianx/project-knowledge 4.7.0 → 5.0.0

package/_site/_test/protected-architecture-gate-test.js

@@ -1,123 +1,123 @@
-// _site/_test/protected-architecture-gate-test.js
-//
-// G07: Protected architecture gate. Asserts the non-negotiable invariants
-// from I-01..I-14 and C-01..C-07 hold after T00-T24.
-//
-//   - Markdown authoritative (no inline knowledge JSON written by analyzers)
-//   - IndexService is the only LanceDB writer
-//   - One canonical commit reconciliation path (no second analyzer queue)
-//   - Workbench messages do NOT auto-append to Development Conversation
-//   - Knowledge Analyzer sessions do NOT auto-append to Development Conversation
-//   - CommitConversationSnapshot remains immutable
-//   - No CLAUDE.md manager is reintroduced
-//   - Migration preserves project identity, baseline, AI profile, language
-
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { StorageLayout } = require('../lib/storage-layout');
-const { IndexService } = require('../lib/index-service');
-const { CommitReconciler } = require('../lib/commit-reconciler');
-const { renderCommitPrompt } = require('../lib/commit-prompt');
-const { normalizeAutomationConfig } = require('../lib/automation-config');
-const { automationConfig: removedAutomationConfig } = (() => ({ automationConfig: undefined }))();
-void removedAutomationConfig;
-
-(async () => {
-  const dataDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-protected-'));
-  const layout = new StorageLayout({ dataDir });
-  // Test that all the protected modules load cleanly.
-  // (loading at the top ensures any missing dependency fails fast)
-  const { spawnSync } = require('child_process');
-
-  // I-01: Markdown authoritative. The KnowledgePromotionService writes
-  // Markdown files and the resulting claim is then indexed by IndexService.
-  // We assert by reading the schema of commit-prompt output — the
-  // manifest references Markdown paths, not JSON knowledge blobs.
-  {
-    const rendered = renderCommitPrompt({
-      config: { projectId: 'p', displayName: 'P', knowledgePath: '/tmp', repoPath: '/tmp', automation: {} },
-      evidence: {
-        schema: 'commit-evidence/v1', commitSha: 'a'.repeat(40), parents: [],
-        author: 'x', date: '2026-01-01', subject: 's', branch: 'main', patchBase: 'p',
-        patchMode: 'parent', files: [], patchHash: 'sha256:' + '0'.repeat(64),
-        patchBytes: 0, patchInline: true, patchChunked: false, patchOmitted: false,
-        patchLimitBytes: 0, omittedReason: null, evidenceBundle: null,
-        evidenceHash: 'sha256:' + '0'.repeat(64), patch: '',
-      },
-      requirements: [], conversationSnapshot: null,
-      existingKnowledge: { entries: [], totalBytes: 0 },
-    });
-    assert(rendered.prompt.includes('staging manifest'), 'commit-prompt must reference staging manifest (Markdown authoritative)');
-    assert(!rendered.prompt.includes('knowledge JSON') && !rendered.prompt.includes('.jsonl'),
-      'commit-prompt must not instruct writing inline JSON knowledge blobs');
-  }
-
-  // I-02: IndexService only index writer. The constructor signature does
-  // not accept any other LanceDB-writing component. The runtime exposes
-  // a single indexAdapter that IndexService owns.
-  {
-    const service = new IndexService({ layout });
-    // IndexService builds its adapter internally; verify the public API
-    // surface is what the rest of the system uses.
-    assert(typeof service.enqueue === 'function', 'IndexService exposes enqueue');
-    assert(typeof service.processProject === 'function', 'IndexService exposes processProject');
-  }
-
-  // I-04: No second analyzer queue. CommitReconciler.prepareClaim is the
-  // only path that produces an analysis claim; there is no parallel
-  // automation-queue producing equivalent artifacts.
-  {
-    const reconciler = new CommitReconciler({ layout });
-    assert(typeof reconciler.reconcile === 'function');
-    assert(typeof reconciler.processCommit === 'function');
-    assert(typeof reconciler.failWithoutClaim === 'function');
-  }
-
-  // I-13: No CLAUDE.md manager reintroduced. We assert the hook-manager
-  // buildHookBody produces output that does NOT write CLAUDE.md, and the
-  // runtime modules do not export any claudeMdManager.
-  const hookManager = require('../lib/hook-manager');
-  assert(typeof hookManager.buildHookBody === 'function');
-  // Spot-check the body content has no CLAUDE.md rewrite instructions.
-  const fakeTrigger = path.join(os.tmpdir(), 'fake-trigger-' + process.pid + '.js');
-  fs.writeFileSync(fakeTrigger, '// stub');
-  const body = hookManager.buildHookBody({
-    projectId: 'project-valid', triggerScriptPath: fakeTrigger, nodeExecutable: process.execPath,
-  });
-  assert(!body.includes('CLAUDE.md'), 'hook body must NOT touch CLAUDE.md');
-  fs.unlinkSync(fakeTrigger);
-
-  // I-11: Import establishes a Git tracking baseline; it must not
-  // automatically re-analyze the entire historical repository.
-  // (ProjectLifecycleService.importProject sets trackingStartCommit and
-  // never runs reconciliation over historical commits by itself.)
-  const { ProjectLifecycleService } = require('../lib/project-lifecycle-service');
-  assert(typeof ProjectLifecycleService.prototype.importProject === 'function');
-
-  // C-04: One AI profile resolver. Importing the resolver + checking that
-  // there is exactly one exported resolveEffectiveAiProfile function.
-  const resolver = require('../lib/ai-profile-resolver');
-  assert(typeof resolver.resolveEffectiveAiProfile === 'function');
-  assert(resolver.resolveEffectiveAiProfile({ ai: { profiles: [{ id: 'p', enabled: true }] } }, { aiProfileId: 'p' }).profileId === 'p',
-    'T01 resolver must surface the configured profile');
-
-  // C-05: Missing legacy hook is not a successful migration. We assert
-  // by reading hook-manager.migrateManagedHook semantics: a missing hook
-  // returns { reason: 'missing' } and does NOT silently mark migrated.
-  const noHookRepo = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-nohook-'));
-  spawnSync('git', ['-C', noHookRepo, 'init', '-q', '-b', 'main']);
-  assert.strictEqual(
-    require('../lib/hook-manager').migrateManagedHook({ repoPath: noHookRepo }).reason,
-    'missing',
-    'migrateManagedHook must report reason=missing for missing hook (T03 contract)',
-  );
-  fs.rmSync(noHookRepo, { recursive: true, force: true });
-
-  fs.rmSync(dataDir, { recursive: true, force: true });
-  console.log('protected-architecture-gate-test PASS');
-})().catch(error => {
-  console.error(error.stack || error.message);
-  process.exit(1);
+// _site/_test/protected-architecture-gate-test.js
+//
+// G07: Protected architecture gate. Asserts the non-negotiable invariants
+// from I-01..I-14 and C-01..C-07 hold after T00-T24.
+//
+//   - Markdown authoritative (no inline knowledge JSON written by analyzers)
+//   - IndexService is the only LanceDB writer
+//   - One canonical commit reconciliation path (no second analyzer queue)
+//   - Workbench messages do NOT auto-append to Development Conversation
+//   - Knowledge Analyzer sessions do NOT auto-append to Development Conversation
+//   - CommitConversationSnapshot remains immutable
+//   - No CLAUDE.md manager is reintroduced
+//   - Migration preserves project identity, baseline, AI profile, language
+
+const assert = require('assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { StorageLayout } = require('../lib/storage-layout');
+const { IndexService } = require('../lib/index-service');
+const { CommitReconciler } = require('../lib/commit-reconciler');
+const { renderCommitPrompt } = require('../lib/commit-prompt');
+const { normalizeAutomationConfig } = require('../lib/automation-config');
+const { automationConfig: removedAutomationConfig } = (() => ({ automationConfig: undefined }))();
+void removedAutomationConfig;
+
+(async () => {
+  const dataDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-protected-'));
+  const layout = new StorageLayout({ dataDir });
+  // Test that all the protected modules load cleanly.
+  // (loading at the top ensures any missing dependency fails fast)
+  const { spawnSync } = require('child_process');
+
+  // I-01: Markdown authoritative. The KnowledgePromotionService writes
+  // Markdown files and the resulting claim is then indexed by IndexService.
+  // We assert by reading the schema of commit-prompt output — the
+  // manifest references Markdown paths, not JSON knowledge blobs.
+  {
+    const rendered = renderCommitPrompt({
+      config: { projectId: 'p', displayName: 'P', knowledgePath: '/tmp', repoPath: '/tmp', automation: {} },
+      evidence: {
+        schema: 'commit-evidence/v1', commitSha: 'a'.repeat(40), parents: [],
+        author: 'x', date: '2026-01-01', subject: 's', branch: 'main', patchBase: 'p',
+        patchMode: 'parent', files: [], patchHash: 'sha256:' + '0'.repeat(64),
+        patchBytes: 0, patchInline: true, patchChunked: false, patchOmitted: false,
+        patchLimitBytes: 0, omittedReason: null, evidenceBundle: null,
+        evidenceHash: 'sha256:' + '0'.repeat(64), patch: '',
+      },
+      requirements: [], conversationSnapshot: null,
+      existingKnowledge: { entries: [], totalBytes: 0 },
+    });
+    assert(rendered.prompt.includes('staging manifest'), 'commit-prompt must reference staging manifest (Markdown authoritative)');
+    assert(!rendered.prompt.includes('knowledge JSON') && !rendered.prompt.includes('.jsonl'),
+      'commit-prompt must not instruct writing inline JSON knowledge blobs');
+  }
+
+  // I-02: IndexService only index writer. The constructor signature does
+  // not accept any other LanceDB-writing component. The runtime exposes
+  // a single indexAdapter that IndexService owns.
+  {
+    const service = new IndexService({ layout });
+    // IndexService builds its adapter internally; verify the public API
+    // surface is what the rest of the system uses.
+    assert(typeof service.enqueue === 'function', 'IndexService exposes enqueue');
+    assert(typeof service.processProject === 'function', 'IndexService exposes processProject');
+  }
+
+  // I-04: No second analyzer queue. CommitReconciler.prepareClaim is the
+  // only path that produces an analysis claim; there is no parallel
+  // automation-queue producing equivalent artifacts.
+  {
+    const reconciler = new CommitReconciler({ layout });
+    assert(typeof reconciler.reconcile === 'function');
+    assert(typeof reconciler.processCommit === 'function');
+    assert(typeof reconciler.failWithoutClaim === 'function');
+  }
+
+  // I-13: No CLAUDE.md manager reintroduced. We assert the hook-manager
+  // buildHookBody produces output that does NOT write CLAUDE.md, and the
+  // runtime modules do not export any claudeMdManager.
+  const hookManager = require('../lib/hook-manager');
+  assert(typeof hookManager.buildHookBody === 'function');
+  // Spot-check the body content has no CLAUDE.md rewrite instructions.
+  const fakeTrigger = path.join(os.tmpdir(), 'fake-trigger-' + process.pid + '.js');
+  fs.writeFileSync(fakeTrigger, '// stub');
+  const body = hookManager.buildHookBody({
+    projectId: 'project-valid', triggerScriptPath: fakeTrigger, nodeExecutable: process.execPath,
+  });
+  assert(!body.includes('CLAUDE.md'), 'hook body must NOT touch CLAUDE.md');
+  fs.unlinkSync(fakeTrigger);
+
+  // I-11: Import establishes a Git tracking baseline; it must not
+  // automatically re-analyze the entire historical repository.
+  // (ProjectLifecycleService.importProject sets trackingStartCommit and
+  // never runs reconciliation over historical commits by itself.)
+  const { ProjectLifecycleService } = require('../lib/project-lifecycle-service');
+  assert(typeof ProjectLifecycleService.prototype.importProject === 'function');
+
+  // C-04: One AI profile resolver. Importing the resolver + checking that
+  // there is exactly one exported resolveEffectiveAiProfile function.
+  const resolver = require('../lib/ai-profile-resolver');
+  assert(typeof resolver.resolveEffectiveAiProfile === 'function');
+  assert(resolver.resolveEffectiveAiProfile({ ai: { profiles: [{ id: 'p', enabled: true }] } }, { aiProfileId: 'p' }).profileId === 'p',
+    'T01 resolver must surface the configured profile');
+
+  // C-05: Missing legacy hook is not a successful migration. We assert
+  // by reading hook-manager.migrateManagedHook semantics: a missing hook
+  // returns { reason: 'missing' } and does NOT silently mark migrated.
+  const noHookRepo = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-nohook-'));
+  spawnSync('git', ['-C', noHookRepo, 'init', '-q', '-b', 'main']);
+  assert.strictEqual(
+    require('../lib/hook-manager').migrateManagedHook({ repoPath: noHookRepo }).reason,
+    'missing',
+    'migrateManagedHook must report reason=missing for missing hook (T03 contract)',
+  );
+  fs.rmSync(noHookRepo, { recursive: true, force: true });
+
+  fs.rmSync(dataDir, { recursive: true, force: true });
+  console.log('protected-architecture-gate-test PASS');
+})().catch(error => {
+  console.error(error.stack || error.message);
+  process.exit(1);
 });

package/_site/_test/refactor-characterization-test.js

@@ -1,36 +1,36 @@
-const assert = require('assert');
-const fs = require('fs');
-const path = require('path');
-
-const ROOT = path.resolve(__dirname, '..', '..');
-const read = relative => fs.readFileSync(path.join(ROOT, relative), 'utf8');
-const serverEntry = read('_site/server.js');
-const server = read('_site/lib/server-app.js');
-const automation = read('_site/lib/post-commit-automation.js');
-const reconciler = read('_site/lib/commit-reconciler.js');
-const contracts = read('_site/lib/contracts.js');
-const logger = read('_site/lib/structured-logger.js');
-const ui = read('ui/index.html');
-
-assert(serverEntry.includes("require('./lib/server-app')"), 'server entry should remain thin');
-for (const removed of ['/automation/simulate', '/automation/init', '/hook-install', '/hook-uninstall', '/api/raw', "'Access-Control-Allow-Origin': '*'"]) {
-  assert(!server.includes(removed), `legacy server capability must be absent: ${removed}`);
-}
-for (const removed of ['dispatchProjectInit', 'renderProjectInitPrompt', 'DEFAULT_INIT_PROMPT_TEMPLATE', 'dispatchAutomation']) {
-  assert(!automation.includes(removed), `legacy automation symbol must be absent: ${removed}`);
-}
-assert.strictEqual((automation.match(/handlePostCommitEvent/g) || []).length >= 1, true);
-assert(reconciler.includes('validateTrigger(trigger)'));
-assert(contracts.includes("Object.freeze(['git-hook'])"), 'git-hook must be the only analysis trigger');
-assert(!server.includes("reconcileProjectCommits(projectId, 'startup'"), 'startup must not submit commit analysis');
-assert(!automation.includes('dispatchPendingAutomations'), 'pending startup dispatcher must be deleted');
-assert(reconciler.includes('processCommitEvent'), 'Hook events must have an explicit commit processor');
-assert(!reconciler.includes('this.scanner.scan('), 'knowledge analysis must not scan baseline..HEAD');
-assert(logger.includes('LOG_LEVELS,') && logger.includes('new Set(LOG_LEVELS)'), 'the v2 logger must use the shared six-level contract');
-assert(logger.includes('class LogRepository'));
-assert(!ui.includes('tailwindcss.com'));
-assert(!ui.includes('vue.global'));
-assert.strictEqual((ui.match(/class="shell"/g) || []).length, 1, 'production must contain one full product shell');
-assert.strictEqual((ui.match(/id="settings-logs"/g) || []).length, 1, 'production must contain one Settings logs view');
-
-console.log('refactor-characterization-test PASS');
+const assert = require('assert');
+const fs = require('fs');
+const path = require('path');
+
+const ROOT = path.resolve(__dirname, '..', '..');
+const read = relative => fs.readFileSync(path.join(ROOT, relative), 'utf8');
+const serverEntry = read('_site/server.js');
+const server = read('_site/lib/server-app.js');
+const automation = read('_site/lib/post-commit-automation.js');
+const reconciler = read('_site/lib/commit-reconciler.js');
+const contracts = read('_site/lib/contracts.js');
+const logger = read('_site/lib/structured-logger.js');
+const ui = read('ui/index.html');
+
+assert(serverEntry.includes("require('./lib/server-app')"), 'server entry should remain thin');
+for (const removed of ['/automation/simulate', '/automation/init', '/hook-install', '/hook-uninstall', '/api/raw', "'Access-Control-Allow-Origin': '*'"]) {
+  assert(!server.includes(removed), `legacy server capability must be absent: ${removed}`);
+}
+for (const removed of ['dispatchProjectInit', 'renderProjectInitPrompt', 'DEFAULT_INIT_PROMPT_TEMPLATE', 'dispatchAutomation']) {
+  assert(!automation.includes(removed), `legacy automation symbol must be absent: ${removed}`);
+}
+assert.strictEqual((automation.match(/handlePostCommitEvent/g) || []).length >= 1, true);
+assert(reconciler.includes('validateTrigger(trigger)'));
+assert(contracts.includes("Object.freeze(['git-hook'])"), 'git-hook must be the only analysis trigger');
+assert(!server.includes("reconcileProjectCommits(projectId, 'startup'"), 'startup must not submit commit analysis');
+assert(!automation.includes('dispatchPendingAutomations'), 'pending startup dispatcher must be deleted');
+assert(reconciler.includes('processCommitEvent'), 'Hook events must have an explicit commit processor');
+assert(!reconciler.includes('this.scanner.scan('), 'knowledge analysis must not scan baseline..HEAD');
+assert(logger.includes('LOG_LEVELS,') && logger.includes('new Set(LOG_LEVELS)'), 'the v2 logger must use the shared six-level contract');
+assert(logger.includes('class LogRepository'));
+assert(!ui.includes('tailwindcss.com'));
+assert(!ui.includes('vue.global'));
+assert.strictEqual((ui.match(/class="app"/g) || []).length, 1, 'production must contain one full product shell');
+assert.strictEqual((ui.match(/id="logsDialog"/g) || []).length, 1, 'production must contain one diagnostics view');
+
+console.log('refactor-characterization-test PASS');

package/_site/_test/release-version-sync-test.js

@@ -74,7 +74,7 @@ try {
   assert.strictEqual(claude.version, '9.8.7');
   assert.strictEqual(codex.version, '9.8.7');
   assert.strictEqual(marketplace.plugins[0].version, '9.8.7');
-  assert(mcp.mcpServers['project-knowledge'].args.includes('project-knowledge@9.8.7'));
+  assert(mcp.mcpServers['project-knowledge'].args.includes('@sanqianx/project-knowledge@9.8.7'));
   assert.strictEqual(desktopLock.version, '9.8.7');
   assert.strictEqual(desktopLock.packages[''].version, '9.8.7');
   const staged = git(['diff', '--cached', '--name-only']);