@sanqianx/project-knowledge 4.7.0 → 5.0.0

package/_site/_test/project-lifecycle-transaction-test.js

@@ -1,106 +1,106 @@
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { spawnSync } = require('child_process');
-const { StorageLayout } = require('../lib/storage-layout');
-const { SettingsStore } = require('../lib/settings-store');
-const { ProjectRegistryStore } = require('../lib/project-registry-store');
-const { ProjectStore } = require('../lib/project-store');
-const { ProjectLifecycleService } = require('../lib/project-lifecycle-service');
-const realHookManager = require('../lib/hook-manager');
-
-const ROOT = path.resolve(__dirname, '..', '..');
-const TRIGGER = path.join(ROOT, '_site', 'scripts', 'hook-trigger.js');
-
-function git(repo, 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`);
-  return String(result.stdout || '').trim();
-}
-
-function createRepo(root, name, withCommit = true) {
-  const repo = path.join(root, name);
-  fs.mkdirSync(repo, { recursive: true });
-  git(repo, ['init', '--initial-branch=main']);
-  git(repo, ['config', 'user.email', 'life@example.invalid']);
-  git(repo, ['config', 'user.name', 'Lifecycle Test']);
-  if (withCommit) {
-    fs.writeFileSync(path.join(repo, 'README.md'), `# ${name}\n`);
-    git(repo, ['add', 'README.md']);
-    git(repo, ['commit', '-m', 'initial']);
-  }
-  return repo;
-}
-
-(async () => {
-  const root = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-lifecycle-'));
-  const dataDir = path.join(root, 'data');
-  const knowledgeRoot = path.join(root, 'knowledge');
-  const layout = new StorageLayout({ dataDir });
-  const settings = new SettingsStore({ layout });
-  const registry = new ProjectRegistryStore({ layout });
-  const projects = new ProjectStore({ layout });
-  await settings.initialize({ knowledge: { rootPath: knowledgeRoot } });
-  await registry.initialize();
-  const lifecycle = new ProjectLifecycleService({
-    layout,
-    settingsStore: settings,
-    registryStore: registry,
-    projectStore: projects,
-    triggerScriptPath: TRIGGER,
-    bridgeAdapter: { async getHighWatermark() { return { status: 'captured', cursor: 73 }; } },
-  });
-
-  const repo = createRepo(root, 'repo-a');
-  const claude = path.join(repo, 'CLAUDE.md');
-  fs.writeFileSync(claude, '# user only\n');
-  const imported = await lifecycle.importProject({ localPath: repo, projectId: 'project-life-a', storageName: 'repo-a-stable' });
-  assert.strictEqual(imported.ok, true);
-  assert.strictEqual(imported.config.aiProfileId, null, 'import must not require an AI profile');
-  assert.strictEqual(imported.state.trackingStartCommit, git(repo, ['rev-parse', 'HEAD']));
-  assert.strictEqual(imported.state.conversationBaselineCursor, 73, 'import must freeze the current Bridge cursor as its conversation baseline');
-  assert.deepStrictEqual(fs.readdirSync(imported.config.knowledgePath), [], 'import must not create TODO knowledge files');
-  assert.strictEqual(fs.readFileSync(claude, 'utf8'), '# user only\n');
-  assert.strictEqual(realHookManager.readHookStatus({ repoPath: repo, projectId: imported.projectId }).installed, true);
-  await assert.rejects(lifecycle.importProject({ localPath: repo }), /already imported/);
-
-  const conflictRepo = createRepo(root, 'repo-conflict');
-  const conflictPath = path.join(knowledgeRoot, 'conflict-stable');
-  fs.mkdirSync(conflictPath, { recursive: true });
-  fs.writeFileSync(path.join(conflictPath, 'user.md'), 'do not overwrite');
-  await assert.rejects(lifecycle.importProject({ localPath: conflictRepo, projectId: 'project-conflict', storageName: 'conflict-stable' }), error => error.code === 'MIGRATION_TARGET_CONFLICT');
-  assert(!registry.listIds().includes('project-conflict'));
-  assert.strictEqual(fs.readFileSync(path.join(conflictPath, 'user.md'), 'utf8'), 'do not overwrite');
-
-  const failingRepo = createRepo(root, 'repo-failing');
-  const failingHooks = { ...realHookManager, installHook() { throw new Error('injected Hook failure'); } };
-  const failingLifecycle = new ProjectLifecycleService({ layout, settingsStore: settings, registryStore: registry, projectStore: projects, triggerScriptPath: TRIGGER, hookManager: failingHooks });
-  await assert.rejects(failingLifecycle.importProject({ localPath: failingRepo, projectId: 'project-failing', storageName: 'failing-stable' }));
-  assert(!registry.listIds().includes('project-failing'));
-  assert(!fs.existsSync(layout.getProjectMetadataDir('project-failing')));
-  assert(!fs.existsSync(path.join(knowledgeRoot, 'failing-stable')));
-
-  const emptyRepo = createRepo(root, 'repo-empty', false);
-  const empty = await lifecycle.importProject({ localPath: emptyRepo, projectId: 'project-empty', storageName: 'empty-stable' });
-  assert.strictEqual(empty.state.trackingMode, 'empty-repo');
-  assert.strictEqual(empty.state.trackingStartCommit, null);
-
-  const failingDelete = new ProjectLifecycleService({
-    layout, settingsStore: settings, registryStore: registry, projectStore: projects, triggerScriptPath: TRIGGER,
-    hookManager: { ...realHookManager, uninstallHook() { throw new Error('injected uninstall failure'); } },
-  });
-  await assert.rejects(failingDelete.deleteProject(imported.projectId));
-  assert(registry.listIds().includes(imported.projectId), 'Hook uninstall failure must preserve registration');
-
-  const knowledgePath = imported.config.knowledgePath;
-  const deleted = await lifecycle.deleteProject(imported.projectId);
-  assert.strictEqual(deleted.removedKnowledge, false);
-  assert(fs.existsSync(knowledgePath), 'default delete must preserve external knowledge directory');
-  assert(!registry.listIds().includes(imported.projectId));
-
-  console.log('project-lifecycle-transaction-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 { spawnSync } = require('child_process');
+const { StorageLayout } = require('../lib/storage-layout');
+const { SettingsStore } = require('../lib/settings-store');
+const { ProjectRegistryStore } = require('../lib/project-registry-store');
+const { ProjectStore } = require('../lib/project-store');
+const { ProjectLifecycleService } = require('../lib/project-lifecycle-service');
+const realHookManager = require('../lib/hook-manager');
+
+const ROOT = path.resolve(__dirname, '..', '..');
+const TRIGGER = path.join(ROOT, '_site', 'scripts', 'hook-trigger.js');
+
+function git(repo, 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`);
+  return String(result.stdout || '').trim();
+}
+
+function createRepo(root, name, withCommit = true) {
+  const repo = path.join(root, name);
+  fs.mkdirSync(repo, { recursive: true });
+  git(repo, ['init', '--initial-branch=main']);
+  git(repo, ['config', 'user.email', 'life@example.invalid']);
+  git(repo, ['config', 'user.name', 'Lifecycle Test']);
+  if (withCommit) {
+    fs.writeFileSync(path.join(repo, 'README.md'), `# ${name}\n`);
+    git(repo, ['add', 'README.md']);
+    git(repo, ['commit', '-m', 'initial']);
+  }
+  return repo;
+}
+
+(async () => {
+  const root = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-lifecycle-'));
+  const dataDir = path.join(root, 'data');
+  const knowledgeRoot = path.join(root, 'knowledge');
+  const layout = new StorageLayout({ dataDir });
+  const settings = new SettingsStore({ layout });
+  const registry = new ProjectRegistryStore({ layout });
+  const projects = new ProjectStore({ layout });
+  await settings.initialize({ knowledge: { rootPath: knowledgeRoot } });
+  await registry.initialize();
+  const lifecycle = new ProjectLifecycleService({
+    layout,
+    settingsStore: settings,
+    registryStore: registry,
+    projectStore: projects,
+    triggerScriptPath: TRIGGER,
+    bridgeAdapter: { async getHighWatermark() { return { status: 'captured', cursor: 73 }; } },
+  });
+
+  const repo = createRepo(root, 'repo-a');
+  const claude = path.join(repo, 'CLAUDE.md');
+  fs.writeFileSync(claude, '# user only\n');
+  const imported = await lifecycle.importProject({ localPath: repo, projectId: 'project-life-a', storageName: 'repo-a-stable' });
+  assert.strictEqual(imported.ok, true);
+  assert.strictEqual(imported.config.aiProfileId, null, 'import must not require an AI profile');
+  assert.strictEqual(imported.state.trackingStartCommit, git(repo, ['rev-parse', 'HEAD']));
+  assert.strictEqual(imported.state.conversationBaselineCursor, 73, 'import must freeze the current Bridge cursor as its conversation baseline');
+  assert.deepStrictEqual(fs.readdirSync(imported.config.knowledgePath), [], 'import must not create TODO knowledge files');
+  assert.strictEqual(fs.readFileSync(claude, 'utf8'), '# user only\n');
+  assert.strictEqual(realHookManager.readHookStatus({ repoPath: repo, projectId: imported.projectId }).installed, true);
+  await assert.rejects(lifecycle.importProject({ localPath: repo }), /already imported/);
+
+  const conflictRepo = createRepo(root, 'repo-conflict');
+  const conflictPath = path.join(knowledgeRoot, 'conflict-stable');
+  fs.mkdirSync(conflictPath, { recursive: true });
+  fs.writeFileSync(path.join(conflictPath, 'user.md'), 'do not overwrite');
+  await assert.rejects(lifecycle.importProject({ localPath: conflictRepo, projectId: 'project-conflict', storageName: 'conflict-stable' }), error => error.code === 'MIGRATION_TARGET_CONFLICT');
+  assert(!registry.listIds().includes('project-conflict'));
+  assert.strictEqual(fs.readFileSync(path.join(conflictPath, 'user.md'), 'utf8'), 'do not overwrite');
+
+  const failingRepo = createRepo(root, 'repo-failing');
+  const failingHooks = { ...realHookManager, installHook() { throw new Error('injected Hook failure'); } };
+  const failingLifecycle = new ProjectLifecycleService({ layout, settingsStore: settings, registryStore: registry, projectStore: projects, triggerScriptPath: TRIGGER, hookManager: failingHooks });
+  await assert.rejects(failingLifecycle.importProject({ localPath: failingRepo, projectId: 'project-failing', storageName: 'failing-stable' }));
+  assert(!registry.listIds().includes('project-failing'));
+  assert(!fs.existsSync(layout.getProjectMetadataDir('project-failing')));
+  assert(!fs.existsSync(path.join(knowledgeRoot, 'failing-stable')));
+
+  const emptyRepo = createRepo(root, 'repo-empty', false);
+  const empty = await lifecycle.importProject({ localPath: emptyRepo, projectId: 'project-empty', storageName: 'empty-stable' });
+  assert.strictEqual(empty.state.trackingMode, 'empty-repo');
+  assert.strictEqual(empty.state.trackingStartCommit, null);
+
+  const failingDelete = new ProjectLifecycleService({
+    layout, settingsStore: settings, registryStore: registry, projectStore: projects, triggerScriptPath: TRIGGER,
+    hookManager: { ...realHookManager, uninstallHook() { throw new Error('injected uninstall failure'); } },
+  });
+  await assert.rejects(failingDelete.deleteProject(imported.projectId));
+  assert(registry.listIds().includes(imported.projectId), 'Hook uninstall failure must preserve registration');
+
+  const knowledgePath = imported.config.knowledgePath;
+  const deleted = await lifecycle.deleteProject(imported.projectId);
+  assert.strictEqual(deleted.removedKnowledge, false);
+  assert(fs.existsSync(knowledgePath), 'default delete must preserve external knowledge directory');
+  assert(!registry.listIds().includes(imported.projectId));
+
+  console.log('project-lifecycle-transaction-test PASS');
+})().catch(error => {
+  console.error(error.stack || error.message);
+  process.exitCode = 1;
+});

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);
+});