@sanqianx/project-knowledge 5.0.24 → 5.0.25

package/_site/_test/prompt-settings-test.js

@@ -1,115 +1,115 @@
-// _site/_test/prompt-settings-test.js
-//
-// T12: Prompt Settings on the canonical analysis path. Verifies:
-//   - promptOverrides from settings flow into the canonical commit
-//     prompt rendering (via commit-prompt.js / automation-config.js);
-//   - reset to default restores the original prompt hash;
-//   - malformed templates are rejected;
-//   - the canonical CommitReconciler.prepareClaim path picks up the
-//     configured override without bypassing evidence manifests or
-//     prompt hashes.
-
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { StorageLayout } = require('../lib/storage-layout');
-const { SettingsStore, defaultSettings } = require('../lib/settings-store');
-const { renderCommitPrompt } = require('../lib/commit-prompt');
-const { normalizeAutomationConfig, DEFAULT_COMMIT_PROMPT_TEMPLATE } = require('../lib/automation-config');
-
-(async () => {
-  const dataDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-prompt-settings-'));
-  const layout = new StorageLayout({ dataDir });
-  const settings = new SettingsStore({ layout });
-  await settings.initialize({ knowledge: { rootPath: '' } });
-
-  const fakeEvidence = {
-    schema: 'commit-evidence/v1',
-    commitSha: 'abc1234567890abcdef1234567890abcdef123456',
-    parents: [],
-    author: 'tester',
-    date: '2026-08-20T00:00:00Z',
-    subject: 'test commit',
-    branch: 'main',
-    patchBase: 'parent',
-    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: '',
-  };
-  const baseInput = {
-    config: { projectId: 'p', displayName: 'P', knowledgePath: '', repoPath: '', automation: {} },
-    evidence: fakeEvidence,
-    requirements: [],
-    conversationSnapshot: null,
-    existingKnowledge: { entries: [], totalBytes: 0 },
-  };
-
-  // Case 1: default prompt renders the canonical template.
-  {
-    const config = normalizeAutomationConfig({});
-    assert.strictEqual(config.commitPromptTemplate, DEFAULT_COMMIT_PROMPT_TEMPLATE);
-    const rendered = renderCommitPrompt(baseInput);
-    assert(rendered.prompt.length > 0);
-    assert(rendered.promptHash && rendered.promptHash.startsWith('sha256:'), 'promptHash must be a sha256 digest');
-    assert(rendered.prompt.includes('你正在根据一个 Git Commit'), 'default template must render the canonical Chinese prompt');
-  }
-
-  // Case 2: promptOverrides.commitPromptTemplate changes the rendered
-  // prompt deterministically.
-  {
-    const override = 'CUSTOM-PROMPT-TEMPLATE {{commitHash}} {{commitSubject}} {{commitAuthor}}';
-    const renderedDefault = renderCommitPrompt(baseInput);
-    const renderedCustom = renderCommitPrompt({
-      ...baseInput,
-      config: { ...baseInput.config, automation: { commitPromptTemplate: override } },
-    });
-    assert.notStrictEqual(renderedCustom.promptHash, renderedDefault.promptHash, 'override must change prompt hash');
-    assert(renderedCustom.prompt.includes('CUSTOM-PROMPT-TEMPLATE'), 'override template must appear in rendered prompt');
-  }
-
-  // Case 3: reset to default restores the original template + hash.
-  {
-    const originalConfig = normalizeAutomationConfig({});
-    const overrideConfig = normalizeAutomationConfig({ commitPromptTemplate: 'OVERRIDE {{commitHash}}' });
-    const resetConfig = normalizeAutomationConfig({ ...overrideConfig, commitPromptTemplate: '' });
-    assert.strictEqual(resetConfig.commitPromptTemplate, originalConfig.commitPromptTemplate, 'empty override restores default');
-  }
-
-  // Case 4: malformed template (no required tokens) — the renderer must
-  // still produce a deterministic hash; downstream code (prepareClaim)
-  // is what enforces minimum content via ensureRequiredSections.
-  {
-    const rendered = renderCommitPrompt({
-      ...baseInput,
-      config: { ...baseInput.config, automation: { commitPromptTemplate: 'NO-TOKENS-HERE' } },
-    });
-    assert(rendered.prompt.includes('NO-TOKENS-HERE'));
-    assert(rendered.promptHash.startsWith('sha256:'));
-  }
-
-  // Case 5: settings.promptOverrides is wired through settings updatePatch.
-  // Persisting a prompt override via the canonical settings API must
-  // appear in subsequent reads (the next analyzer invocation will pick
-  // up the new template via this path).
-  {
-    await settings.updatePatch({ promptOverrides: { commitPromptTemplate: 'NEW {{commitHash}} {{commitSubject}}' } });
-    const current = settings.read();
-    assert.strictEqual(current.promptOverrides.commitPromptTemplate, 'NEW {{commitHash}} {{commitSubject}}');
-  }
-
-  fs.rmSync(dataDir, { recursive: true, force: true });
-  console.log('prompt-settings-test PASS');
-})().catch(error => {
-  console.error(error.stack || error.message);
-  process.exit(1);
-});
+// _site/_test/prompt-settings-test.js
+//
+// T12: Prompt Settings on the canonical analysis path. Verifies:
+//   - promptOverrides from settings flow into the canonical commit
+//     prompt rendering (via commit-prompt.js / automation-config.js);
+//   - reset to default restores the original prompt hash;
+//   - malformed templates are rejected;
+//   - the canonical CommitReconciler.prepareClaim path picks up the
+//     configured override without bypassing evidence manifests or
+//     prompt hashes.
+
+const assert = require('assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { StorageLayout } = require('../lib/storage-layout');
+const { SettingsStore, defaultSettings } = require('../lib/settings-store');
+const { renderCommitPrompt } = require('../lib/commit-prompt');
+const { normalizeAutomationConfig, DEFAULT_COMMIT_PROMPT_TEMPLATE } = require('../lib/automation-config');
+
+(async () => {
+  const dataDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-prompt-settings-'));
+  const layout = new StorageLayout({ dataDir });
+  const settings = new SettingsStore({ layout });
+  await settings.initialize({ knowledge: { rootPath: '' } });
+
+  const fakeEvidence = {
+    schema: 'commit-evidence/v1',
+    commitSha: 'abc1234567890abcdef1234567890abcdef123456',
+    parents: [],
+    author: 'tester',
+    date: '2026-08-20T00:00:00Z',
+    subject: 'test commit',
+    branch: 'main',
+    patchBase: 'parent',
+    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: '',
+  };
+  const baseInput = {
+    config: { projectId: 'p', displayName: 'P', knowledgePath: '', repoPath: '', automation: {} },
+    evidence: fakeEvidence,
+    requirements: [],
+    conversationSnapshot: null,
+    existingKnowledge: { entries: [], totalBytes: 0 },
+  };
+
+  // Case 1: default prompt renders the canonical template.
+  {
+    const config = normalizeAutomationConfig({});
+    assert.strictEqual(config.commitPromptTemplate, DEFAULT_COMMIT_PROMPT_TEMPLATE);
+    const rendered = renderCommitPrompt(baseInput);
+    assert(rendered.prompt.length > 0);
+    assert(rendered.promptHash && rendered.promptHash.startsWith('sha256:'), 'promptHash must be a sha256 digest');
+    assert(rendered.prompt.includes('你正在根据一个 Git Commit'), 'default template must render the canonical Chinese prompt');
+  }
+
+  // Case 2: promptOverrides.commitPromptTemplate changes the rendered
+  // prompt deterministically.
+  {
+    const override = 'CUSTOM-PROMPT-TEMPLATE {{commitHash}} {{commitSubject}} {{commitAuthor}}';
+    const renderedDefault = renderCommitPrompt(baseInput);
+    const renderedCustom = renderCommitPrompt({
+      ...baseInput,
+      config: { ...baseInput.config, automation: { commitPromptTemplate: override } },
+    });
+    assert.notStrictEqual(renderedCustom.promptHash, renderedDefault.promptHash, 'override must change prompt hash');
+    assert(renderedCustom.prompt.includes('CUSTOM-PROMPT-TEMPLATE'), 'override template must appear in rendered prompt');
+  }
+
+  // Case 3: reset to default restores the original template + hash.
+  {
+    const originalConfig = normalizeAutomationConfig({});
+    const overrideConfig = normalizeAutomationConfig({ commitPromptTemplate: 'OVERRIDE {{commitHash}}' });
+    const resetConfig = normalizeAutomationConfig({ ...overrideConfig, commitPromptTemplate: '' });
+    assert.strictEqual(resetConfig.commitPromptTemplate, originalConfig.commitPromptTemplate, 'empty override restores default');
+  }
+
+  // Case 4: malformed template (no required tokens) — the renderer must
+  // still produce a deterministic hash; downstream code (prepareClaim)
+  // is what enforces minimum content via ensureRequiredSections.
+  {
+    const rendered = renderCommitPrompt({
+      ...baseInput,
+      config: { ...baseInput.config, automation: { commitPromptTemplate: 'NO-TOKENS-HERE' } },
+    });
+    assert(rendered.prompt.includes('NO-TOKENS-HERE'));
+    assert(rendered.promptHash.startsWith('sha256:'));
+  }
+
+  // Case 5: settings.promptOverrides is wired through settings updatePatch.
+  // Persisting a prompt override via the canonical settings API must
+  // appear in subsequent reads (the next analyzer invocation will pick
+  // up the new template via this path).
+  {
+    await settings.updatePatch({ promptOverrides: { commitPromptTemplate: 'NEW {{commitHash}} {{commitSubject}}' } });
+    const current = settings.read();
+    assert.strictEqual(current.promptOverrides.commitPromptTemplate, 'NEW {{commitHash}} {{commitSubject}}');
+  }
+
+  fs.rmSync(dataDir, { recursive: true, force: true });
+  console.log('prompt-settings-test PASS');
+})().catch(error => {
+  console.error(error.stack || error.message);
+  process.exit(1);
+});

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="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');
+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/registry-events-test.js

@@ -1,38 +1,38 @@
-'use strict';
-const assert = require('node:assert/strict');
-const http = require('node:http');
-const { RegistryEvents } = require('../lib/registry-events');
-const { createRequestHandler } = require('../lib/server-app');
-
-(async () => {
-  const events = new RegistryEvents(), received = [];
-  const off = events.subscribe(event => received.push(event));
-  assert.equal(received[0].type, 'registry-snapshot');
-  const runtime = { registryEvents: events, logger: { debug: async () => {}, info: async () => {} },
-    moduleBridge: { syncEngineConfiguration: async () => ({}) },
-    settingsStore: { read: () => ({ brain: { id: 'brain-default', name: 'before' }, knowledge: { rootPath: '' } }), updatePatch: async patch => ({ brain: patch.brain }) } };
-  // Exercise the real HTTP completion hook independently of business stores.
-  const handler = createRequestHandler(runtime, { host: '127.0.0.1', port: 0 });
-  const server = http.createServer(handler);
-  await new Promise(resolve => server.listen(0, '127.0.0.1', resolve));
-  try {
-    const base = `http://127.0.0.1:${server.address().port}`;
-    const reader = await fetch(`${base}/api/registry/events`);
-    const stream = reader.body.getReader();
-    const first = new TextDecoder().decode((await stream.read()).value);
-    assert.match(first, /registry-snapshot/);
-    const mutation = await fetch(`${base}/api/brain`, { method: 'PATCH', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ name: 'after' }) });
-    assert.equal(mutation.status, 200);
-    const change = new TextDecoder().decode((await stream.read()).value);
-    assert.match(change, /registry-changed/);
-    assert.equal(received[1].revision, 1);
-    const reconnect = []; events.subscribe(event => reconnect.push(event))();
-    assert.equal(reconnect[0].revision, 1);
-    assert.equal(reconnect[0].instanceId, received[0].instanceId);
-    await stream.cancel();
-    off(); events.publish(['brain']);
-    assert.equal(received.length, 2, 'unsubscribed observers receive no more updates');
-    assert.notEqual(new RegistryEvents().instanceId, events.instanceId, 'restart makes old revisions distinguishable');
-    console.log('registry-events-test PASS (real SSE, reconnect identity and unsubscribe)');
-  } finally { server.closeAllConnections(); await new Promise(resolve => server.close(resolve)); }
-})().catch(error => { console.error(error); process.exitCode = 1; });
+'use strict';
+const assert = require('node:assert/strict');
+const http = require('node:http');
+const { RegistryEvents } = require('../lib/registry-events');
+const { createRequestHandler } = require('../lib/server-app');
+
+(async () => {
+  const events = new RegistryEvents(), received = [];
+  const off = events.subscribe(event => received.push(event));
+  assert.equal(received[0].type, 'registry-snapshot');
+  const runtime = { registryEvents: events, logger: { debug: async () => {}, info: async () => {} },
+    moduleBridge: { syncEngineConfiguration: async () => ({}) },
+    settingsStore: { read: () => ({ brain: { id: 'brain-default', name: 'before' }, knowledge: { rootPath: '' } }), updatePatch: async patch => ({ brain: patch.brain }) } };
+  // Exercise the real HTTP completion hook independently of business stores.
+  const handler = createRequestHandler(runtime, { host: '127.0.0.1', port: 0 });
+  const server = http.createServer(handler);
+  await new Promise(resolve => server.listen(0, '127.0.0.1', resolve));
+  try {
+    const base = `http://127.0.0.1:${server.address().port}`;
+    const reader = await fetch(`${base}/api/registry/events`);
+    const stream = reader.body.getReader();
+    const first = new TextDecoder().decode((await stream.read()).value);
+    assert.match(first, /registry-snapshot/);
+    const mutation = await fetch(`${base}/api/brain`, { method: 'PATCH', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ name: 'after' }) });
+    assert.equal(mutation.status, 200);
+    const change = new TextDecoder().decode((await stream.read()).value);
+    assert.match(change, /registry-changed/);
+    assert.equal(received[1].revision, 1);
+    const reconnect = []; events.subscribe(event => reconnect.push(event))();
+    assert.equal(reconnect[0].revision, 1);
+    assert.equal(reconnect[0].instanceId, received[0].instanceId);
+    await stream.cancel();
+    off(); events.publish(['brain']);
+    assert.equal(received.length, 2, 'unsubscribed observers receive no more updates');
+    assert.notEqual(new RegistryEvents().instanceId, events.instanceId, 'restart makes old revisions distinguishable');
+    console.log('registry-events-test PASS (real SSE, reconnect identity and unsubscribe)');
+  } finally { server.closeAllConnections(); await new Promise(resolve => server.close(resolve)); }
+})().catch(error => { console.error(error); process.exitCode = 1; });