@sanqianx/project-knowledge 4.7.0 → 5.0.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.agents/plugins/marketplace.json +17 -17
- package/.claude-plugin/marketplace.json +1 -1
- package/CHANGELOG.md +836 -799
- package/README.md +58 -41
- package/_modules/knowledge-engine/README.md +91 -0
- package/_modules/knowledge-engine/package.json +38 -0
- package/_modules/knowledge-engine/src/analysis-contract.js +25 -0
- package/_modules/knowledge-engine/src/analysis-worker.js +54 -0
- package/_modules/knowledge-engine/src/bin.js +52 -0
- package/_modules/knowledge-engine/src/bridge-client.js +24 -0
- package/_modules/knowledge-engine/src/coordinator.js +263 -0
- package/_modules/knowledge-engine/src/headless-analyzer.js +75 -0
- package/_modules/knowledge-engine/src/memories.js +85 -0
- package/_modules/knowledge-engine/src/retrieval.js +116 -0
- package/_modules/knowledge-engine/src/runtime/README.md +7 -0
- package/_modules/knowledge-engine/src/runtime/atomic-file.js +204 -0
- package/_modules/knowledge-engine/src/runtime/automation-config.js +140 -0
- package/_modules/knowledge-engine/src/runtime/commit-conversation-binder.js +249 -0
- package/_modules/knowledge-engine/src/runtime/commit-processing-ledger.js +26 -0
- package/_modules/knowledge-engine/src/runtime/commit-prompt.js +202 -0
- package/_modules/knowledge-engine/src/runtime/commit-reconciler.js +432 -0
- package/_modules/knowledge-engine/src/runtime/content-hash.js +4 -0
- package/_modules/knowledge-engine/src/runtime/contracts.js +279 -0
- package/_modules/knowledge-engine/src/runtime/conversation-exclusions.js +204 -0
- package/_modules/knowledge-engine/src/runtime/conversation-store.js +387 -0
- package/_modules/knowledge-engine/src/runtime/evidence-bundle.js +228 -0
- package/_modules/knowledge-engine/src/runtime/git-runner.js +62 -0
- package/_modules/knowledge-engine/src/runtime/knowledge-promotion.js +444 -0
- package/_modules/knowledge-engine/src/runtime/knowledge-retrieval-service.js +402 -0
- package/_modules/knowledge-engine/src/runtime/layout.js +44 -0
- package/_modules/knowledge-engine/src/runtime/markdown-knowledge-indexer.js +230 -0
- package/_modules/knowledge-engine/src/runtime/scanner.js +298 -0
- package/_modules/knowledge-engine/src/server.js +154 -0
- package/_modules/knowledge-engine/src/store.js +236 -0
- package/_modules/knowledge-engine/ui/app.css +172 -0
- package/_modules/knowledge-engine/ui/app.js +297 -0
- package/_modules/knowledge-engine/ui/index.html +95 -0
- package/_site/README.md +30 -30
- package/_site/_test/ai-profile-resolver-test.js +137 -137
- package/_site/_test/automation-queue-test.js +14 -14
- package/_site/_test/automation-ui-test.js +40 -40
- package/_site/_test/background-task-registry-test.js +43 -43
- package/_site/_test/baseline-schema-test.js +91 -91
- package/_site/_test/chat-claudecodeui-match-test.js +92 -92
- package/_site/_test/claude-executable-discovery-test.js +62 -62
- package/_site/_test/claude-workbench-test.js +170 -170
- package/_site/_test/codex-conversation-projection-test.js +2 -1
- package/_site/_test/commit-conversation-binding-test.js +95 -95
- package/_site/_test/commit-evidence-test.js +110 -110
- package/_site/_test/conversation-api-test.js +109 -109
- package/_site/_test/conversation-store-test.js +136 -136
- package/_site/_test/desktop-browser-compat-test.js +28 -28
- package/_site/_test/desktop-hook-runtime-regression-test.js +204 -204
- package/_site/_test/explicit-commit-processor-test.js +37 -37
- package/_site/_test/fixtures/make-git-repos.js +12 -1
- package/_site/_test/folder-picker-output-test.js +19 -19
- package/_site/_test/fresh-hook-reimport-test.js +53 -0
- package/_site/_test/fresh-product-data-test.js +42 -0
- package/_site/_test/full-integration-e2e-test.js +165 -165
- package/_site/_test/git-validation-test.js +2 -2
- package/_site/_test/hook-runtime-endpoint-test.js +88 -88
- package/_site/_test/hook-status-repair-api-test.js +152 -152
- package/_site/_test/hook-trigger-test.js +85 -85
- package/_site/_test/import-preflight-api-test.js +234 -233
- package/_site/_test/index-writer-concurrency-test.js +135 -135
- package/_site/_test/integration-adapters-test.js +171 -171
- package/_site/_test/integration-surface-coverage-test.js +164 -164
- package/_site/_test/knowledge-language-control-test.js +170 -170
- package/_site/_test/knowledge-promotion-recovery-test.js +260 -260
- package/_site/_test/knowledge-query-test.js +54 -54
- package/_site/_test/knowledge-retrieval-service-test.js +77 -77
- package/_site/_test/knowledge-storage-startup-test.js +52 -52
- package/_site/_test/knowledge-store-logs-supervision-test.js +89 -89
- package/_site/_test/logging-api-test.js +89 -89
- package/_site/_test/logging-sse-no-gap-test.js +88 -88
- package/_site/_test/logging-ui-test.js +106 -104
- package/_site/_test/markdown-delta-overlay-test.js +85 -85
- package/_site/_test/markdown-maintenance-api-test.js +75 -75
- package/_site/_test/mcp-server-test.js +149 -149
- package/_site/_test/module-artifact-integrity-test.js +27 -0
- package/_site/_test/module-boundary-test.js +30 -0
- package/_site/_test/module-bridge-eventbridge-test.js +7 -0
- package/_site/_test/module-model-configuration-test.js +44 -0
- package/_site/_test/module-process-lifecycle-test.js +24 -0
- package/_site/_test/module-stream-proxy-test.js +34 -0
- package/_site/_test/non-release-ci-test.js +34 -34
- package/_site/_test/offline-boundary-isolation-test.js +32 -32
- package/_site/_test/packaged-ui-smoke-test.js +11 -9
- package/_site/_test/path-consistency-test.js +151 -151
- package/_site/_test/pending-sweep-test.js +9 -9
- package/_site/_test/post-commit-automation-test.js +87 -87
- package/_site/_test/product-diagnostics-ui-test.js +75 -0
- package/_site/_test/product-import-ui-test.js +92 -0
- package/_site/_test/project-delete-recovery-test.js +64 -64
- package/_site/_test/project-goal-editor-test.js +144 -144
- package/_site/_test/project-lifecycle-transaction-test.js +106 -106
- package/_site/_test/prompt-settings-test.js +115 -115
- package/_site/_test/protected-architecture-gate-test.js +122 -122
- package/_site/_test/refactor-characterization-test.js +36 -36
- package/_site/_test/release-version-sync-test.js +1 -1
- package/_site/_test/requirement-recorder-test.js +173 -173
- package/_site/_test/run-all-tests.js +159 -156
- package/_site/_test/shared-contracts-test.js +58 -58
- package/_site/_test/startup-analysis-disabled-test.js +10 -10
- package/_site/_test/storage-foundation-test.js +78 -78
- package/_site/_test/structured-logger-test.js +101 -101
- package/_site/_test/tracking-start-test.js +109 -108
- package/_site/_test/workbench-permission-test.js +117 -117
- package/_site/_test/workspace-ui-contract-test.js +28 -43
- package/_site/lib/ai-profile-resolver.js +78 -78
- package/_site/lib/ai-workspace.js +101 -101
- package/_site/lib/automation-config.js +140 -140
- package/_site/lib/bridge-adapter.js +1 -0
- package/_site/lib/bridge-consumer-service.js +402 -402
- package/_site/lib/claude-cli-runner.js +1510 -1510
- package/_site/lib/commit-conversation-binder.js +234 -234
- package/_site/lib/commit-processing-ledger.js +27 -27
- package/_site/lib/commit-prompt.js +202 -202
- package/_site/lib/commit-reconciler.js +406 -406
- package/_site/lib/contracts.js +284 -279
- package/_site/lib/conversation-query-service.js +144 -144
- package/_site/lib/conversation-store.js +385 -385
- package/_site/lib/data-dir.js +6 -74
- package/_site/lib/engine-connection.js +67 -0
- package/_site/lib/evidence-bundle.js +228 -228
- package/_site/lib/folder-picker-output.js +21 -21
- package/_site/lib/git-runner.js +62 -62
- package/_site/lib/github-team-store.js +1172 -1172
- package/_site/lib/hook-manager.js +203 -202
- package/_site/lib/index-service.js +173 -173
- package/_site/lib/integration-installer.js +908 -907
- package/_site/lib/kb-framework.js +176 -176
- package/_site/lib/kb-validator.js +124 -124
- package/_site/lib/knowledge-promotion.js +435 -435
- package/_site/lib/knowledge-retrieval-service.js +401 -401
- package/_site/lib/knowledge-tool-runtime.js +359 -359
- package/_site/lib/llm-client.js +161 -161
- package/_site/lib/markdown-knowledge-indexer.js +230 -230
- package/_site/lib/module-bridge.js +191 -36
- package/_site/lib/post-commit-automation.js +87 -87
- package/_site/lib/product-data.js +24 -0
- package/_site/lib/project-lifecycle-service.js +535 -497
- package/_site/lib/project-store.js +268 -262
- package/_site/lib/requirement-recorder.js +278 -278
- package/_site/lib/runtime-endpoint.js +155 -155
- package/_site/lib/scanner.js +298 -298
- package/_site/lib/server-app.js +1579 -1520
- package/_site/lib/settings-store.js +133 -113
- package/_site/lib/storage-layout.js +162 -162
- package/_site/lib/structured-logger.js +574 -574
- package/_site/scripts/folder-picker.ps1 +156 -156
- package/_site/scripts/hook-trigger.js +157 -156
- package/_site/scripts/install-module-candidates.js +69 -0
- package/_site/scripts/pack-module-candidates.js +72 -0
- package/_site/scripts/sync-release-version.js +2 -2
- package/_site/scripts/vendor-modules.js +47 -0
- package/_site/scripts/verify-module-runtime.js +45 -0
- package/_site/scripts/verify-product-runtime.js +34 -0
- package/bin/project-knowledge-mcp.js +194 -194
- package/docs/README.zh-CN.md +42 -29
- package/docs/fresh-v5-start.md +28 -0
- package/docs/project-registry-schema.md +22 -22
- package/docs/testing-strategy.md +33 -33
- package/module-runtime-manifest.json +220 -0
- package/package.json +14 -9
- package/plugins/project-knowledge/.claude-plugin/plugin.json +1 -1
- package/plugins/project-knowledge/.codex-plugin/plugin.json +1 -1
- package/plugins/project-knowledge/.mcp.json +1 -1
- package/plugins/project-knowledge/opencode/project-knowledge.md +3 -3
- package/plugins/project-knowledge/skills/project-knowledge/SKILL.md +28 -28
- package/templates/project-readme.md +34 -34
- package/ui/favicon.svg +38 -38
- package/ui/index.html +52 -149
- package/ui/product.css +9 -0
- package/ui/product.js +412 -0
- package/vendor-manifest.json +45 -0
- package/_site/_test/data-dir-migration-test.js +0 -77
- package/_site/_test/import-ui-flow-test.js +0 -171
- package/_site/_test/knowledge-migration-test.js +0 -75
- package/_site/_test/legacy-forward-compat-test.js +0 -257
- package/_site/_test/legacy-project-upgrade-e2e-test.js +0 -363
- package/_site/_test/p0-data-migration-characterization-test.js +0 -37
- package/_site/_test/p0-e2e-gate-test.js +0 -347
- package/_site/_test/project-control-panel-task14-test.js +0 -63
- package/_site/_test/project-layout-v2-migration-test.js +0 -127
- package/_site/_test/server-runtime-migration-safety-test.js +0 -27
- package/_site/_test/task15-20-ui-flow-test.js +0 -148
- package/_site/_test/ui-i18n-toggle-test.js +0 -114
- package/_site/_test/ui-smoke-test.js +0 -73
- package/_site/_test/v4122-upgrade-data-contract-test.js +0 -56
- package/_site/lib/data-state-classifier.js +0 -40
- package/_site/lib/legacy-data-manifest.js +0 -26
- package/_site/lib/migration-service.js +0 -442
- package/ui/app.css +0 -58
- package/ui/app.js +0 -607
- package/ui/i18n.js +0 -146
|
@@ -1,32 +1,32 @@
|
|
|
1
|
-
const assert = require('assert');
|
|
2
|
-
const fs = require('fs');
|
|
3
|
-
const os = require('os');
|
|
4
|
-
const path = require('path');
|
|
5
|
-
const { StorageLayout } = require('../lib/storage-layout');
|
|
6
|
-
const { ProjectStore } = require('../lib/project-store');
|
|
7
|
-
const { ConversationStore } = require('../lib/conversation-store');
|
|
8
|
-
const { CommitConversationBinder } = require('../lib/commit-conversation-binder');
|
|
9
|
-
|
|
10
|
-
(async () => {
|
|
11
|
-
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-offline-boundaries-'));
|
|
12
|
-
try {
|
|
13
|
-
const layout = new StorageLayout({ dataDir: path.join(root, 'data') });
|
|
14
|
-
const projects = new ProjectStore({ layout });
|
|
15
|
-
const projectId = 'project-offline-boundary';
|
|
16
|
-
const identity = { commonDir: path.join(root, 'repo', '.git') };
|
|
17
|
-
await projects.create(projectId, { storageName: 'offline-boundary', displayName: 'Offline Boundary', repoPath: path.join(root, 'repo'), knowledgePath: path.join(root, 'knowledge'), repoIdentity: identity }, { conversationBaselineCursor: 0 });
|
|
18
|
-
const store = new ConversationStore({ layout, projectStore: projects });
|
|
19
|
-
const binder = new CommitConversationBinder({ layout, projectStore: projects, conversationStore: store });
|
|
20
|
-
const add = (sequence, eventId, content) => store.appendEvent(projectId, { eventId, sequence, source: 'codex', eventType: 'user_prompt', role: 'user', content, turnId: eventId, repoIdentity: identity, capturedAt: '2026-08-24T00:00:00.000Z', identityConfidence: 'exact', captureStatus: 'captured' });
|
|
21
|
-
const boundary = (sha, cursor) => store.writeBoundary(projectId, { commitSha: sha, repoIdentity: identity, parentShas: [], branch: 'main', committedAt: '2026-08-24T00:00:00.000Z', bridgeCursorAtCommit: cursor, journalSequence: cursor, openTurnIdsAtCommit: [], operationId: `op-${cursor}` });
|
|
22
|
-
const b = 'b'.repeat(40); const c = 'c'.repeat(40); const d = 'd'.repeat(40);
|
|
23
|
-
await add(1, 'prompt-b', 'offline B'); boundary(b, 2);
|
|
24
|
-
await add(3, 'prompt-c', 'offline C'); boundary(c, 4);
|
|
25
|
-
await add(5, 'prompt-d', 'online D'); boundary(d, 6);
|
|
26
|
-
const snapshot = await binder.bind({ projectId, commitSha: d });
|
|
27
|
-
assert.deepEqual(snapshot.turns.map(turn => turn.turnId), ['prompt-d']);
|
|
28
|
-
assert.equal(JSON.stringify(snapshot).includes('offline B'), false);
|
|
29
|
-
assert.equal(JSON.stringify(snapshot).includes('offline C'), false);
|
|
30
|
-
} finally { fs.rmSync(root, { recursive: true, force: true }); }
|
|
31
|
-
console.log('offline-boundary-isolation-test PASS');
|
|
32
|
-
})().catch(error => { console.error(error.stack || error.message); process.exit(1); });
|
|
1
|
+
const assert = require('assert');
|
|
2
|
+
const fs = require('fs');
|
|
3
|
+
const os = require('os');
|
|
4
|
+
const path = require('path');
|
|
5
|
+
const { StorageLayout } = require('../lib/storage-layout');
|
|
6
|
+
const { ProjectStore } = require('../lib/project-store');
|
|
7
|
+
const { ConversationStore } = require('../lib/conversation-store');
|
|
8
|
+
const { CommitConversationBinder } = require('../lib/commit-conversation-binder');
|
|
9
|
+
|
|
10
|
+
(async () => {
|
|
11
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-offline-boundaries-'));
|
|
12
|
+
try {
|
|
13
|
+
const layout = new StorageLayout({ dataDir: path.join(root, 'data') });
|
|
14
|
+
const projects = new ProjectStore({ layout });
|
|
15
|
+
const projectId = 'project-offline-boundary';
|
|
16
|
+
const identity = { commonDir: path.join(root, 'repo', '.git') };
|
|
17
|
+
await projects.create(projectId, { storageName: 'offline-boundary', displayName: 'Offline Boundary', repoPath: path.join(root, 'repo'), knowledgePath: path.join(root, 'knowledge'), repoIdentity: identity }, { conversationBaselineCursor: 0 });
|
|
18
|
+
const store = new ConversationStore({ layout, projectStore: projects });
|
|
19
|
+
const binder = new CommitConversationBinder({ layout, projectStore: projects, conversationStore: store });
|
|
20
|
+
const add = (sequence, eventId, content) => store.appendEvent(projectId, { eventId, sequence, source: 'codex', eventType: 'user_prompt', role: 'user', content, turnId: eventId, repoIdentity: identity, capturedAt: '2026-08-24T00:00:00.000Z', identityConfidence: 'exact', captureStatus: 'captured' });
|
|
21
|
+
const boundary = (sha, cursor) => store.writeBoundary(projectId, { commitSha: sha, repoIdentity: identity, parentShas: [], branch: 'main', committedAt: '2026-08-24T00:00:00.000Z', bridgeCursorAtCommit: cursor, journalSequence: cursor, openTurnIdsAtCommit: [], operationId: `op-${cursor}` });
|
|
22
|
+
const b = 'b'.repeat(40); const c = 'c'.repeat(40); const d = 'd'.repeat(40);
|
|
23
|
+
await add(1, 'prompt-b', 'offline B'); boundary(b, 2);
|
|
24
|
+
await add(3, 'prompt-c', 'offline C'); boundary(c, 4);
|
|
25
|
+
await add(5, 'prompt-d', 'online D'); boundary(d, 6);
|
|
26
|
+
const snapshot = await binder.bind({ projectId, commitSha: d });
|
|
27
|
+
assert.deepEqual(snapshot.turns.map(turn => turn.turnId), ['prompt-d']);
|
|
28
|
+
assert.equal(JSON.stringify(snapshot).includes('offline B'), false);
|
|
29
|
+
assert.equal(JSON.stringify(snapshot).includes('offline C'), false);
|
|
30
|
+
} finally { fs.rmSync(root, { recursive: true, force: true }); }
|
|
31
|
+
console.log('offline-boundary-isolation-test PASS');
|
|
32
|
+
})().catch(error => { console.error(error.stack || error.message); process.exit(1); });
|
|
@@ -53,26 +53,28 @@ function buildAndInstallPackage() {
|
|
|
53
53
|
let browser;
|
|
54
54
|
try {
|
|
55
55
|
const packagedRoot = buildAndInstallPackage();
|
|
56
|
-
assert(fs.existsSync(path.join(packagedRoot, 'ui',
|
|
56
|
+
for (const file of ['index.html', 'product.js', 'product.css']) assert(fs.existsSync(path.join(packagedRoot, 'ui', file)), `packed UI must include ${file}`);
|
|
57
|
+
for (const file of ['app.js', 'app.css', 'i18n.js']) assert(!fs.existsSync(path.join(packagedRoot, 'ui', file)), `obsolete shell resource must not be shipped: ${file}`);
|
|
57
58
|
|
|
58
|
-
server = spawnServer({ root: packagedRoot, port: PORT, tag: 'packaged-ui', stdio: 'ignore' });
|
|
59
|
+
server = spawnServer({ root: packagedRoot, port: PORT, dataDir: path.join(tempRoot, 'data'), tag: 'packaged-ui', analysisOwner: 'engine', stdio: 'ignore' });
|
|
59
60
|
await waitFor(async () => {
|
|
60
61
|
try { return (await fetch(`${BASE}/api/health`)).ok ? { ok: true } : null; }
|
|
61
62
|
catch { return null; }
|
|
62
63
|
}, 'packaged server', 20000);
|
|
63
64
|
|
|
64
|
-
const
|
|
65
|
-
assert.strictEqual(
|
|
66
|
-
assert.match(String(
|
|
65
|
+
const productResponse = await fetch(`${BASE}/product.js`);
|
|
66
|
+
assert.strictEqual(productResponse.status, 200, 'packaged server must serve /product.js');
|
|
67
|
+
assert.match(String(productResponse.headers.get('content-type') || ''), /javascript/i);
|
|
67
68
|
|
|
68
|
-
browser = await launchCdpBrowser({ chrome: findChrome(), profileDir, url: `${BASE}/`, width:
|
|
69
|
+
browser = await launchCdpBrowser({ chrome: findChrome(), profileDir, url: `${BASE}/`, width: 1920, height: 1080 });
|
|
70
|
+
await browser.setViewport(1920, 1080);
|
|
69
71
|
await waitFor(async () => {
|
|
70
|
-
const ready = await browser.evaluate('
|
|
72
|
+
const ready = await browser.evaluate("document.getElementById('homeContent').textContent.includes('第一个项目')");
|
|
71
73
|
return ready ? { ok: true } : null;
|
|
72
74
|
}, 'packaged UI bootstrap', 20000);
|
|
73
75
|
|
|
74
|
-
|
|
75
|
-
assert.strictEqual(
|
|
76
|
+
assert.strictEqual(await browser.evaluate("document.getElementById('projectSelect').disabled"), true);
|
|
77
|
+
assert.strictEqual(await browser.evaluate("document.querySelectorAll('nav [data-view]').length"), 6);
|
|
76
78
|
assert.strictEqual(browser.exceptions.length, 0, `packaged UI runtime exceptions: ${JSON.stringify(browser.exceptions)}`);
|
|
77
79
|
console.log('packaged-ui-smoke-test PASS');
|
|
78
80
|
} finally {
|
|
@@ -1,151 +1,151 @@
|
|
|
1
|
-
const assert = require('assert');
|
|
2
|
-
const fs = require('fs');
|
|
3
|
-
const os = require('os');
|
|
4
|
-
const path = require('path');
|
|
5
|
-
const { spawnSync } = require('child_process');
|
|
6
|
-
const { StorageLayout } = require('../lib/storage-layout');
|
|
7
|
-
const { SettingsStore } = require('../lib/settings-store');
|
|
8
|
-
const { ProjectRegistryStore } = require('../lib/project-registry-store');
|
|
9
|
-
const { ProjectStore } = require('../lib/project-store');
|
|
10
|
-
const { ProjectLifecycleService } = require('../lib/project-lifecycle-service');
|
|
11
|
-
const { KnowledgeToolRuntime } = require('../lib/knowledge-tool-runtime');
|
|
12
|
-
const { CommitReconciler } = require('../lib/commit-reconciler');
|
|
13
|
-
const { handlePostCommitEvent } = require('../lib/post-commit-automation');
|
|
14
|
-
const hookManager = require('../lib/hook-manager');
|
|
15
|
-
const { callTool } = require('../../bin/project-knowledge-mcp');
|
|
16
|
-
const { spawnServer } = require('./helpers/spawn-server');
|
|
17
|
-
|
|
18
|
-
const ROOT = path.resolve(__dirname, '..', '..');
|
|
19
|
-
const PORT = 7834;
|
|
20
|
-
const BASE = `http://127.0.0.1:${PORT}`;
|
|
21
|
-
const temp = fs.mkdtempSync(path.join(os.tmpdir(), 'pkb-path-v2-'));
|
|
22
|
-
const dataDir = path.join(temp, 'data');
|
|
23
|
-
const rootA = path.join(temp, 'knowledge-a');
|
|
24
|
-
const rootB = path.join(temp, 'knowledge-b');
|
|
25
|
-
const oldRepo = path.join(temp, 'old-repo');
|
|
26
|
-
const movedRepo = path.join(temp, 'moved-repo');
|
|
27
|
-
const newRepo = path.join(temp, 'new-repo');
|
|
28
|
-
|
|
29
|
-
function git(cwd, args) {
|
|
30
|
-
const result = spawnSync('git', args, { cwd, encoding: 'utf8' });
|
|
31
|
-
assert.strictEqual(result.status, 0, result.stderr);
|
|
32
|
-
return String(result.stdout || '').trim();
|
|
33
|
-
}
|
|
34
|
-
|
|
35
|
-
function initializeRepo(cwd, title) {
|
|
36
|
-
fs.mkdirSync(cwd, { recursive: true });
|
|
37
|
-
git(cwd, ['init', '--initial-branch=main']);
|
|
38
|
-
git(cwd, ['config', 'user.email', 'path@example.test']);
|
|
39
|
-
git(cwd, ['config', 'user.name', 'Path Test']);
|
|
40
|
-
fs.writeFileSync(path.join(cwd, 'README.md'), `# ${title}\n`, 'utf8');
|
|
41
|
-
git(cwd, ['add', 'README.md']);
|
|
42
|
-
git(cwd, ['commit', '-m', 'baseline']);
|
|
43
|
-
}
|
|
44
|
-
|
|
45
|
-
async function waitForServer() {
|
|
46
|
-
for (let attempt = 0; attempt < 100; attempt += 1) {
|
|
47
|
-
try { if ((await fetch(`${BASE}/api/health`)).ok) return; } catch {}
|
|
48
|
-
await new Promise(resolve => setTimeout(resolve, 100));
|
|
49
|
-
}
|
|
50
|
-
throw new Error('server did not start');
|
|
51
|
-
}
|
|
52
|
-
|
|
53
|
-
(async () => {
|
|
54
|
-
let spawned;
|
|
55
|
-
try {
|
|
56
|
-
initializeRepo(oldRepo, 'Old');
|
|
57
|
-
initializeRepo(newRepo, 'New');
|
|
58
|
-
fs.mkdirSync(rootA, { recursive: true });
|
|
59
|
-
fs.mkdirSync(rootB, { recursive: true });
|
|
60
|
-
const oldKnowledge = path.join(rootA, 'old-fixed');
|
|
61
|
-
fs.mkdirSync(path.join(oldKnowledge, 'modules'), { recursive: true });
|
|
62
|
-
fs.writeFileSync(path.join(oldKnowledge, 'modules', 'identity.md'), '# Fixed identity\n\nOld project knowledge remains here.\n', 'utf8');
|
|
63
|
-
|
|
64
|
-
const layout = new StorageLayout({ dataDir });
|
|
65
|
-
const settingsStore = new SettingsStore({ layout });
|
|
66
|
-
const registryStore = new ProjectRegistryStore({ layout });
|
|
67
|
-
const projectStore = new ProjectStore({ layout });
|
|
68
|
-
await settingsStore.initialize({ knowledge: { rootPath: rootA } });
|
|
69
|
-
await registryStore.initialize();
|
|
70
|
-
const baseline = git(oldRepo, ['rev-parse', 'HEAD']);
|
|
71
|
-
const commonDir = git(oldRepo, ['rev-parse', '--path-format=absolute', '--git-common-dir']);
|
|
72
|
-
await projectStore.create('project-stable', {
|
|
73
|
-
displayName: 'Before rename', storageName: 'old-fixed', repoPath: oldRepo, knowledgePath: oldKnowledge,
|
|
74
|
-
repoIdentity: { commonDir }, legacyExtensions: { slug: 'legacy-old' },
|
|
75
|
-
}, { trackingStartCommit: baseline, index: { dirty: true, generation: 1 } });
|
|
76
|
-
await registryStore.add('project-stable', { displayNameSnapshot: 'Before rename' });
|
|
77
|
-
|
|
78
|
-
await settingsStore.updatePatch({ knowledge: { rootPath: rootB } });
|
|
79
|
-
assert.strictEqual(projectStore.readConfig('project-stable').knowledgePath, oldKnowledge, 'global root changes must not move an existing project');
|
|
80
|
-
await projectStore.updateConfig('project-stable', { displayName: 'After rename' });
|
|
81
|
-
assert.strictEqual(projectStore.readConfig('project-stable').projectId, 'project-stable');
|
|
82
|
-
|
|
83
|
-
const lifecycle = new ProjectLifecycleService({ layout, settingsStore, registryStore, projectStore, hookManager, triggerScriptPath: path.join(ROOT, '_site', 'scripts', 'hook-trigger.js') });
|
|
84
|
-
const imported = await lifecycle.importProject({ localPath: newRepo, displayName: 'New root project' });
|
|
85
|
-
assert(layout.isPathInside(rootB, imported.config.knowledgePath, { realpath: true }));
|
|
86
|
-
assert(!layout.isPathInside(rootA, imported.config.knowledgePath, { realpath: true }));
|
|
87
|
-
await settingsStore.updatePatch({ ai: { schema: 'ai-profiles/v1', profiles: [{ id: 'secret', apiKey: 'never-print-this-secret' }] } });
|
|
88
|
-
|
|
89
|
-
fs.renameSync(oldRepo, movedRepo);
|
|
90
|
-
const reconciler = new CommitReconciler({ layout, registryStore, projectStore, requireAiProfile: false });
|
|
91
|
-
reconciler.processCommitEvent = async input => ({ ok: true, projectId: input.projectId, commitSha: input.commitSha, status: 'completed', processed: [] });
|
|
92
|
-
const moved = await handlePostCommitEvent({ schema: 'hook-event/v2', projectId: 'project-stable', repoRoot: movedRepo, head: baseline, branch: 'main' }, { layout, registryStore, projectStore, reconciler });
|
|
93
|
-
assert.strictEqual(moved.ok, true);
|
|
94
|
-
assert.strictEqual(projectStore.readConfig('project-stable').repoPath, path.resolve(movedRepo));
|
|
95
|
-
assert.strictEqual(projectStore.readConfig('project-stable').projectId, 'project-stable');
|
|
96
|
-
|
|
97
|
-
const runtime = new KnowledgeToolRuntime({ layout, settingsStore, registryStore, projectStore, cwd: movedRepo });
|
|
98
|
-
const resolved = runtime.resolveProject({ repoPath: movedRepo });
|
|
99
|
-
assert.strictEqual(resolved.projectId, 'project-stable');
|
|
100
|
-
assert.strictEqual(resolved.knowledgePath, oldKnowledge);
|
|
101
|
-
assert.strictEqual(resolved.indexPath, layout.getIndexPath());
|
|
102
|
-
const mcpResolved = await callTool(runtime, 'project_knowledge_resolve', { projectId: 'project-stable' });
|
|
103
|
-
assert.strictEqual(mcpResolved.knowledgePath, resolved.knowledgePath);
|
|
104
|
-
assert.strictEqual(mcpResolved.indexPath, resolved.indexPath);
|
|
105
|
-
|
|
106
|
-
const protectedFiles = [layout.getSettingsPath(), layout.getProjectRegistryPath(), layout.getProjectConfigPath('project-stable'), layout.getProjectStatePath('project-stable')];
|
|
107
|
-
const beforeRead = new Map(protectedFiles.map(file => [file, fs.readFileSync(file, 'utf8')]));
|
|
108
|
-
const entry = await runtime.get({ projectId: 'project-stable', entry: 'modules/identity.md' });
|
|
109
|
-
assert.match(entry.chunks[0].chunk_text, /Old project knowledge/);
|
|
110
|
-
const search = await runtime.search({ projectId: 'project-stable', query: 'Old project knowledge' });
|
|
111
|
-
assert.strictEqual(search.source, 'knowledge-retrieval-service');
|
|
112
|
-
assert.strictEqual(search.backend, 'markdown-hybrid-fallback');
|
|
113
|
-
const cli = spawnSync(process.execPath, [path.join(ROOT, 'bin', 'project-knowledge-kb.js'), 'get', '--project', 'project-stable', '--entry', 'modules/identity.md', '--json'], {
|
|
114
|
-
cwd: movedRepo, encoding: 'utf8', env: { ...process.env, KB_DATA_DIR: dataDir, KB_SKIP_MIGRATION: '1' },
|
|
115
|
-
});
|
|
116
|
-
assert.strictEqual(cli.status, 0, cli.stderr);
|
|
117
|
-
assert.match(JSON.parse(cli.stdout).chunks[0].chunk_text, /Old project knowledge/);
|
|
118
|
-
const cliError = spawnSync(process.execPath, [path.join(ROOT, 'bin', 'project-knowledge-kb.js'), 'get', '--project', 'missing-project', '--entry', 'README.md', '--json'], {
|
|
119
|
-
cwd: movedRepo, encoding: 'utf8', env: { ...process.env, KB_DATA_DIR: dataDir, KB_SKIP_MIGRATION: '1' },
|
|
120
|
-
});
|
|
121
|
-
assert.notStrictEqual(cliError.status, 0);
|
|
122
|
-
assert(!cliError.stderr.includes('never-print-this-secret'));
|
|
123
|
-
assert(!/\bstack\b/i.test(cliError.stderr));
|
|
124
|
-
for (const file of protectedFiles) assert.strictEqual(fs.readFileSync(file, 'utf8'), beforeRead.get(file), `read query mutated ${file}`);
|
|
125
|
-
await runtime.close();
|
|
126
|
-
|
|
127
|
-
spawned = spawnServer({ root: ROOT, port: PORT, dataDir, tag: 'path-v2' });
|
|
128
|
-
await waitForServer();
|
|
129
|
-
const response = await fetch(`${BASE}/api/projects`);
|
|
130
|
-
const body = await response.json();
|
|
131
|
-
const serverProject = body.projects.find(project => project.projectId === 'project-stable');
|
|
132
|
-
assert.strictEqual(serverProject.config.knowledgePath, oldKnowledge);
|
|
133
|
-
assert.strictEqual(layout.getIndexPath(), path.join(dataDir, 'index', 'knowledge.lancedb'));
|
|
134
|
-
console.log('path-consistency-test PASS');
|
|
135
|
-
} finally {
|
|
136
|
-
if (spawned) {
|
|
137
|
-
try { spawned.child.kill(); } catch {}
|
|
138
|
-
await Promise.race([
|
|
139
|
-
new Promise(resolve => spawned.child.once('exit', resolve)),
|
|
140
|
-
new Promise(resolve => setTimeout(resolve, 2000)),
|
|
141
|
-
]);
|
|
142
|
-
}
|
|
143
|
-
for (let attempt = 0; attempt < 5; attempt += 1) {
|
|
144
|
-
try { fs.rmSync(temp, { recursive: true, force: true, maxRetries: 3, retryDelay: 100 }); break; }
|
|
145
|
-
catch (error) {
|
|
146
|
-
if (attempt === 4) throw error;
|
|
147
|
-
await new Promise(resolve => setTimeout(resolve, 150));
|
|
148
|
-
}
|
|
149
|
-
}
|
|
150
|
-
}
|
|
151
|
-
})().catch(error => { console.error(error); process.exitCode = 1; });
|
|
1
|
+
const assert = require('assert');
|
|
2
|
+
const fs = require('fs');
|
|
3
|
+
const os = require('os');
|
|
4
|
+
const path = require('path');
|
|
5
|
+
const { spawnSync } = require('child_process');
|
|
6
|
+
const { StorageLayout } = require('../lib/storage-layout');
|
|
7
|
+
const { SettingsStore } = require('../lib/settings-store');
|
|
8
|
+
const { ProjectRegistryStore } = require('../lib/project-registry-store');
|
|
9
|
+
const { ProjectStore } = require('../lib/project-store');
|
|
10
|
+
const { ProjectLifecycleService } = require('../lib/project-lifecycle-service');
|
|
11
|
+
const { KnowledgeToolRuntime } = require('../lib/knowledge-tool-runtime');
|
|
12
|
+
const { CommitReconciler } = require('../lib/commit-reconciler');
|
|
13
|
+
const { handlePostCommitEvent } = require('../lib/post-commit-automation');
|
|
14
|
+
const hookManager = require('../lib/hook-manager');
|
|
15
|
+
const { callTool } = require('../../bin/project-knowledge-mcp');
|
|
16
|
+
const { spawnServer } = require('./helpers/spawn-server');
|
|
17
|
+
|
|
18
|
+
const ROOT = path.resolve(__dirname, '..', '..');
|
|
19
|
+
const PORT = 7834;
|
|
20
|
+
const BASE = `http://127.0.0.1:${PORT}`;
|
|
21
|
+
const temp = fs.mkdtempSync(path.join(os.tmpdir(), 'pkb-path-v2-'));
|
|
22
|
+
const dataDir = path.join(temp, 'data');
|
|
23
|
+
const rootA = path.join(temp, 'knowledge-a');
|
|
24
|
+
const rootB = path.join(temp, 'knowledge-b');
|
|
25
|
+
const oldRepo = path.join(temp, 'old-repo');
|
|
26
|
+
const movedRepo = path.join(temp, 'moved-repo');
|
|
27
|
+
const newRepo = path.join(temp, 'new-repo');
|
|
28
|
+
|
|
29
|
+
function git(cwd, args) {
|
|
30
|
+
const result = spawnSync('git', args, { cwd, encoding: 'utf8' });
|
|
31
|
+
assert.strictEqual(result.status, 0, result.stderr);
|
|
32
|
+
return String(result.stdout || '').trim();
|
|
33
|
+
}
|
|
34
|
+
|
|
35
|
+
function initializeRepo(cwd, title) {
|
|
36
|
+
fs.mkdirSync(cwd, { recursive: true });
|
|
37
|
+
git(cwd, ['init', '--initial-branch=main']);
|
|
38
|
+
git(cwd, ['config', 'user.email', 'path@example.test']);
|
|
39
|
+
git(cwd, ['config', 'user.name', 'Path Test']);
|
|
40
|
+
fs.writeFileSync(path.join(cwd, 'README.md'), `# ${title}\n`, 'utf8');
|
|
41
|
+
git(cwd, ['add', 'README.md']);
|
|
42
|
+
git(cwd, ['commit', '-m', 'baseline']);
|
|
43
|
+
}
|
|
44
|
+
|
|
45
|
+
async function waitForServer() {
|
|
46
|
+
for (let attempt = 0; attempt < 100; attempt += 1) {
|
|
47
|
+
try { if ((await fetch(`${BASE}/api/health`)).ok) return; } catch {}
|
|
48
|
+
await new Promise(resolve => setTimeout(resolve, 100));
|
|
49
|
+
}
|
|
50
|
+
throw new Error('server did not start');
|
|
51
|
+
}
|
|
52
|
+
|
|
53
|
+
(async () => {
|
|
54
|
+
let spawned;
|
|
55
|
+
try {
|
|
56
|
+
initializeRepo(oldRepo, 'Old');
|
|
57
|
+
initializeRepo(newRepo, 'New');
|
|
58
|
+
fs.mkdirSync(rootA, { recursive: true });
|
|
59
|
+
fs.mkdirSync(rootB, { recursive: true });
|
|
60
|
+
const oldKnowledge = path.join(rootA, 'old-fixed');
|
|
61
|
+
fs.mkdirSync(path.join(oldKnowledge, 'modules'), { recursive: true });
|
|
62
|
+
fs.writeFileSync(path.join(oldKnowledge, 'modules', 'identity.md'), '# Fixed identity\n\nOld project knowledge remains here.\n', 'utf8');
|
|
63
|
+
|
|
64
|
+
const layout = new StorageLayout({ dataDir });
|
|
65
|
+
const settingsStore = new SettingsStore({ layout });
|
|
66
|
+
const registryStore = new ProjectRegistryStore({ layout });
|
|
67
|
+
const projectStore = new ProjectStore({ layout });
|
|
68
|
+
await settingsStore.initialize({ knowledge: { rootPath: rootA } });
|
|
69
|
+
await registryStore.initialize();
|
|
70
|
+
const baseline = git(oldRepo, ['rev-parse', 'HEAD']);
|
|
71
|
+
const commonDir = git(oldRepo, ['rev-parse', '--path-format=absolute', '--git-common-dir']);
|
|
72
|
+
await projectStore.create('project-stable', {
|
|
73
|
+
displayName: 'Before rename', storageName: 'old-fixed', repoPath: oldRepo, knowledgePath: oldKnowledge,
|
|
74
|
+
repoIdentity: { commonDir }, legacyExtensions: { slug: 'legacy-old' },
|
|
75
|
+
}, { trackingStartCommit: baseline, index: { dirty: true, generation: 1 } });
|
|
76
|
+
await registryStore.add('project-stable', { displayNameSnapshot: 'Before rename' });
|
|
77
|
+
|
|
78
|
+
await settingsStore.updatePatch({ knowledge: { rootPath: rootB } });
|
|
79
|
+
assert.strictEqual(projectStore.readConfig('project-stable').knowledgePath, oldKnowledge, 'global root changes must not move an existing project');
|
|
80
|
+
await projectStore.updateConfig('project-stable', { displayName: 'After rename' });
|
|
81
|
+
assert.strictEqual(projectStore.readConfig('project-stable').projectId, 'project-stable');
|
|
82
|
+
|
|
83
|
+
const lifecycle = new ProjectLifecycleService({ layout, settingsStore, registryStore, projectStore, hookManager, triggerScriptPath: path.join(ROOT, '_site', 'scripts', 'hook-trigger.js') });
|
|
84
|
+
const imported = await lifecycle.importProject({ localPath: newRepo, displayName: 'New root project' });
|
|
85
|
+
assert(layout.isPathInside(rootB, imported.config.knowledgePath, { realpath: true }));
|
|
86
|
+
assert(!layout.isPathInside(rootA, imported.config.knowledgePath, { realpath: true }));
|
|
87
|
+
await settingsStore.updatePatch({ ai: { schema: 'ai-profiles/v1', profiles: [{ id: 'secret', apiKey: 'never-print-this-secret' }] } });
|
|
88
|
+
|
|
89
|
+
fs.renameSync(oldRepo, movedRepo);
|
|
90
|
+
const reconciler = new CommitReconciler({ layout, registryStore, projectStore, requireAiProfile: false });
|
|
91
|
+
reconciler.processCommitEvent = async input => ({ ok: true, projectId: input.projectId, commitSha: input.commitSha, status: 'completed', processed: [] });
|
|
92
|
+
const moved = await handlePostCommitEvent({ schema: 'hook-event/v2', projectId: 'project-stable', repoRoot: movedRepo, head: baseline, branch: 'main' }, { layout, registryStore, projectStore, reconciler });
|
|
93
|
+
assert.strictEqual(moved.ok, true);
|
|
94
|
+
assert.strictEqual(projectStore.readConfig('project-stable').repoPath, path.resolve(movedRepo));
|
|
95
|
+
assert.strictEqual(projectStore.readConfig('project-stable').projectId, 'project-stable');
|
|
96
|
+
|
|
97
|
+
const runtime = new KnowledgeToolRuntime({ layout, settingsStore, registryStore, projectStore, cwd: movedRepo });
|
|
98
|
+
const resolved = runtime.resolveProject({ repoPath: movedRepo });
|
|
99
|
+
assert.strictEqual(resolved.projectId, 'project-stable');
|
|
100
|
+
assert.strictEqual(resolved.knowledgePath, oldKnowledge);
|
|
101
|
+
assert.strictEqual(resolved.indexPath, layout.getIndexPath());
|
|
102
|
+
const mcpResolved = await callTool(runtime, 'project_knowledge_resolve', { projectId: 'project-stable' });
|
|
103
|
+
assert.strictEqual(mcpResolved.knowledgePath, resolved.knowledgePath);
|
|
104
|
+
assert.strictEqual(mcpResolved.indexPath, resolved.indexPath);
|
|
105
|
+
|
|
106
|
+
const protectedFiles = [layout.getSettingsPath(), layout.getProjectRegistryPath(), layout.getProjectConfigPath('project-stable'), layout.getProjectStatePath('project-stable')];
|
|
107
|
+
const beforeRead = new Map(protectedFiles.map(file => [file, fs.readFileSync(file, 'utf8')]));
|
|
108
|
+
const entry = await runtime.get({ projectId: 'project-stable', entry: 'modules/identity.md' });
|
|
109
|
+
assert.match(entry.chunks[0].chunk_text, /Old project knowledge/);
|
|
110
|
+
const search = await runtime.search({ projectId: 'project-stable', query: 'Old project knowledge' });
|
|
111
|
+
assert.strictEqual(search.source, 'knowledge-retrieval-service');
|
|
112
|
+
assert.strictEqual(search.backend, 'markdown-hybrid-fallback');
|
|
113
|
+
const cli = spawnSync(process.execPath, [path.join(ROOT, 'bin', 'project-knowledge-kb.js'), 'get', '--project', 'project-stable', '--entry', 'modules/identity.md', '--json'], {
|
|
114
|
+
cwd: movedRepo, encoding: 'utf8', env: { ...process.env, KB_DATA_DIR: dataDir, KB_SKIP_MIGRATION: '1' },
|
|
115
|
+
});
|
|
116
|
+
assert.strictEqual(cli.status, 0, cli.stderr);
|
|
117
|
+
assert.match(JSON.parse(cli.stdout).chunks[0].chunk_text, /Old project knowledge/);
|
|
118
|
+
const cliError = spawnSync(process.execPath, [path.join(ROOT, 'bin', 'project-knowledge-kb.js'), 'get', '--project', 'missing-project', '--entry', 'README.md', '--json'], {
|
|
119
|
+
cwd: movedRepo, encoding: 'utf8', env: { ...process.env, KB_DATA_DIR: dataDir, KB_SKIP_MIGRATION: '1' },
|
|
120
|
+
});
|
|
121
|
+
assert.notStrictEqual(cliError.status, 0);
|
|
122
|
+
assert(!cliError.stderr.includes('never-print-this-secret'));
|
|
123
|
+
assert(!/\bstack\b/i.test(cliError.stderr));
|
|
124
|
+
for (const file of protectedFiles) assert.strictEqual(fs.readFileSync(file, 'utf8'), beforeRead.get(file), `read query mutated ${file}`);
|
|
125
|
+
await runtime.close();
|
|
126
|
+
|
|
127
|
+
spawned = spawnServer({ root: ROOT, port: PORT, dataDir, tag: 'path-v2' });
|
|
128
|
+
await waitForServer();
|
|
129
|
+
const response = await fetch(`${BASE}/api/projects`);
|
|
130
|
+
const body = await response.json();
|
|
131
|
+
const serverProject = body.projects.find(project => project.projectId === 'project-stable');
|
|
132
|
+
assert.strictEqual(serverProject.config.knowledgePath, oldKnowledge);
|
|
133
|
+
assert.strictEqual(layout.getIndexPath(), path.join(dataDir, 'index', 'knowledge.lancedb'));
|
|
134
|
+
console.log('path-consistency-test PASS');
|
|
135
|
+
} finally {
|
|
136
|
+
if (spawned) {
|
|
137
|
+
try { spawned.child.kill(); } catch {}
|
|
138
|
+
await Promise.race([
|
|
139
|
+
new Promise(resolve => spawned.child.once('exit', resolve)),
|
|
140
|
+
new Promise(resolve => setTimeout(resolve, 2000)),
|
|
141
|
+
]);
|
|
142
|
+
}
|
|
143
|
+
for (let attempt = 0; attempt < 5; attempt += 1) {
|
|
144
|
+
try { fs.rmSync(temp, { recursive: true, force: true, maxRetries: 3, retryDelay: 100 }); break; }
|
|
145
|
+
catch (error) {
|
|
146
|
+
if (attempt === 4) throw error;
|
|
147
|
+
await new Promise(resolve => setTimeout(resolve, 150));
|
|
148
|
+
}
|
|
149
|
+
}
|
|
150
|
+
}
|
|
151
|
+
})().catch(error => { console.error(error); process.exitCode = 1; });
|
|
@@ -1,9 +1,9 @@
|
|
|
1
|
-
const assert = require('assert');
|
|
2
|
-
const automation = require('../lib/post-commit-automation');
|
|
3
|
-
|
|
4
|
-
// Pending repository scans are not a recovery mechanism. A Hook event is the
|
|
5
|
-
// only authority that can request analysis.
|
|
6
|
-
assert.equal(Object.hasOwn(automation, 'dispatchPendingAutomations'), false);
|
|
7
|
-
assert.equal(Object.hasOwn(automation, 'getQueueSize'), false);
|
|
8
|
-
assert.equal(Object.hasOwn(automation, 'drainQueue'), false);
|
|
9
|
-
console.log('pending-sweep-test PASS');
|
|
1
|
+
const assert = require('assert');
|
|
2
|
+
const automation = require('../lib/post-commit-automation');
|
|
3
|
+
|
|
4
|
+
// Pending repository scans are not a recovery mechanism. A Hook event is the
|
|
5
|
+
// only authority that can request analysis.
|
|
6
|
+
assert.equal(Object.hasOwn(automation, 'dispatchPendingAutomations'), false);
|
|
7
|
+
assert.equal(Object.hasOwn(automation, 'getQueueSize'), false);
|
|
8
|
+
assert.equal(Object.hasOwn(automation, 'drainQueue'), false);
|
|
9
|
+
console.log('pending-sweep-test PASS');
|
|
@@ -1,87 +1,87 @@
|
|
|
1
|
-
// Run: node _site/_test/post-commit-automation-test.js
|
|
2
|
-
|
|
3
|
-
const assert = require('assert');
|
|
4
|
-
const fs = require('fs');
|
|
5
|
-
const path = require('path');
|
|
6
|
-
const automationConfig = require('../lib/automation-config');
|
|
7
|
-
const automation = require('../lib/post-commit-automation');
|
|
8
|
-
const { REQUIREMENT_UNAVAILABLE, renderCommitPrompt } = require('../lib/commit-prompt');
|
|
9
|
-
|
|
10
|
-
(() => {
|
|
11
|
-
const normalized = automationConfig.normalizeAutomationConfig({
|
|
12
|
-
enabled: true,
|
|
13
|
-
hookPromptTemplate: 'Legacy instruction for {{commitHash}}.',
|
|
14
|
-
initPromptTemplate: 'must be ignored',
|
|
15
|
-
postCommitEnabled: true,
|
|
16
|
-
allowReadOnlyBash: true,
|
|
17
|
-
});
|
|
18
|
-
assert.strictEqual(normalized.commitPromptTemplate, 'Legacy instruction for {{commitHash}}.', 'legacy Hook prompt should be a read-only compatibility override');
|
|
19
|
-
assert.strictEqual(normalized.legacyPromptOverride, true);
|
|
20
|
-
assert(!Object.hasOwn(normalized, 'hookPromptTemplate'));
|
|
21
|
-
assert(!Object.hasOwn(normalized, 'initPromptTemplate'));
|
|
22
|
-
assert(!Object.hasOwn(normalized, 'postCommitEnabled'));
|
|
23
|
-
assert(!Object.hasOwn(normalized, 'allowReadOnlyBash'));
|
|
24
|
-
assert(!Object.hasOwn(automationConfig, 'DEFAULT_INIT_PROMPT_TEMPLATE'), 'init prompt constant must not be exported');
|
|
25
|
-
assert(!Object.hasOwn(automation, 'dispatchProjectInit'), 'project init dispatch must not be public');
|
|
26
|
-
assert(!Object.hasOwn(automation, 'dispatchAutomation'), 'generic automation dispatch must not be public');
|
|
27
|
-
|
|
28
|
-
const staging = path.resolve('runtime', 'staging', 'run-test');
|
|
29
|
-
const evidenceRoot = path.resolve('runtime', 'runs', 'run-test', 'input', 'evidence');
|
|
30
|
-
const policy = automationConfig.buildAutomationToolPolicy({ stagingPath: staging, evidenceRoot });
|
|
31
|
-
assert.strictEqual(policy.canWriteKb, false, 'AI must never write final knowledge directly');
|
|
32
|
-
assert.strictEqual(policy.canWriteStaging, true);
|
|
33
|
-
assert(!policy.allowedTools.includes('Bash'), 'generic Bash must not be available to analysis');
|
|
34
|
-
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Write', { file_path: path.join(staging, 'manifest.json') }).behavior, 'allow');
|
|
35
|
-
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Write', { file_path: path.resolve('source.js') }).behavior, 'deny');
|
|
36
|
-
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Read', { file_path: path.join(evidenceRoot, 'patch-manifest.json') }).behavior, 'allow');
|
|
37
|
-
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Write', { file_path: path.join(evidenceRoot, 'patches', '000001.patch') }).behavior, 'deny');
|
|
38
|
-
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Read', { file_path: path.resolve('repo', 'source.js') }).behavior, 'deny');
|
|
39
|
-
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Bash', { command: 'git show' }).behavior, 'deny');
|
|
40
|
-
|
|
41
|
-
const evidence = {
|
|
42
|
-
commitSha: 'a'.repeat(40),
|
|
43
|
-
parents: ['b'.repeat(40)],
|
|
44
|
-
author: 'Test Author',
|
|
45
|
-
date: '2026-08-17T00:00:00Z',
|
|
46
|
-
subject: 'feat: exact evidence',
|
|
47
|
-
branch: 'main',
|
|
48
|
-
patchMode: 'parent',
|
|
49
|
-
patchHash: 'sha256:patch',
|
|
50
|
-
patchBytes: 42,
|
|
51
|
-
patchOmitted: false,
|
|
52
|
-
patch: 'diff --git a/a.js b/a.js\n+implemented\n',
|
|
53
|
-
files: [{ status: 'M', path: 'a.js', oldPath: null, binary: false, added: 1, deleted: 0 }],
|
|
54
|
-
};
|
|
55
|
-
const config = {
|
|
56
|
-
projectId: 'project-test',
|
|
57
|
-
displayName: 'Prompt Test',
|
|
58
|
-
repoPath: path.resolve('repo'),
|
|
59
|
-
knowledgePath: path.resolve('knowledge'),
|
|
60
|
-
automation: { commitPromptTemplate: automationConfig.DEFAULT_COMMIT_PROMPT_TEMPLATE },
|
|
61
|
-
};
|
|
62
|
-
const missing = renderCommitPrompt({ config, evidence, requirements: [], existingKnowledge: { entries: [], omitted: [] } });
|
|
63
|
-
assert(missing.prompt.includes(REQUIREMENT_UNAVAILABLE), 'unavailable requirement must use the fixed non-guessing text');
|
|
64
|
-
assert(missing.prompt.includes(evidence.patch), 'prompt must contain the actual patch');
|
|
65
|
-
assert(missing.prompt.includes('现有相关知识'), 'prompt must contain existing knowledge section');
|
|
66
|
-
assert(missing.prompt.includes('新增、更新或删除'), 'prompt must ask for add/update/delete decisions');
|
|
67
|
-
assert(!missing.prompt.includes('git-hook') && !missing.prompt.includes('startup'), 'trigger must not affect prompt content');
|
|
68
|
-
|
|
69
|
-
const requirement = { id: 'req-1', client: 'codex', sessionId: 'session-1', ts: '2026-08-17T00:00:00Z', requirement: 'Implement the exact feature.' };
|
|
70
|
-
const fromHook = renderCommitPrompt({ config, evidence, requirements: [requirement], existingKnowledge: { entries: [] } });
|
|
71
|
-
const fromStartup = renderCommitPrompt({ config, evidence, requirements: [requirement], existingKnowledge: { entries: [] } });
|
|
72
|
-
assert.strictEqual(fromHook.promptHash, fromStartup.promptHash, 'Hook and startup must render the identical prompt hash');
|
|
73
|
-
assert(fromHook.prompt.includes('Implement the exact feature.'), 'recorded user requirement must be present');
|
|
74
|
-
|
|
75
|
-
const legacy = renderCommitPrompt({
|
|
76
|
-
config: { ...config, automation: { hookPromptTemplate: 'Legacy {{commitHash}}' } },
|
|
77
|
-
evidence,
|
|
78
|
-
requirements: [],
|
|
79
|
-
existingKnowledge: { entries: [] },
|
|
80
|
-
});
|
|
81
|
-
assert(legacy.prompt.includes('Legacy ' + evidence.commitSha), 'legacy Hook prompt override should still render');
|
|
82
|
-
assert(legacy.prompt.includes(REQUIREMENT_UNAVAILABLE) && legacy.prompt.includes(evidence.patch), 'mandatory evidence sections must be appended to incomplete legacy templates');
|
|
83
|
-
|
|
84
|
-
const source = fs.readFileSync(path.join(__dirname, '..', 'lib', 'post-commit-automation.js'), 'utf8');
|
|
85
|
-
assert(!/dispatchProjectInit|renderProjectInitPrompt|project-init|startup-recovery/.test(source), 'automation source must not retain init or a third trigger name');
|
|
86
|
-
console.log('post-commit-automation-test PASS');
|
|
87
|
-
})();
|
|
1
|
+
// Run: node _site/_test/post-commit-automation-test.js
|
|
2
|
+
|
|
3
|
+
const assert = require('assert');
|
|
4
|
+
const fs = require('fs');
|
|
5
|
+
const path = require('path');
|
|
6
|
+
const automationConfig = require('../lib/automation-config');
|
|
7
|
+
const automation = require('../lib/post-commit-automation');
|
|
8
|
+
const { REQUIREMENT_UNAVAILABLE, renderCommitPrompt } = require('../lib/commit-prompt');
|
|
9
|
+
|
|
10
|
+
(() => {
|
|
11
|
+
const normalized = automationConfig.normalizeAutomationConfig({
|
|
12
|
+
enabled: true,
|
|
13
|
+
hookPromptTemplate: 'Legacy instruction for {{commitHash}}.',
|
|
14
|
+
initPromptTemplate: 'must be ignored',
|
|
15
|
+
postCommitEnabled: true,
|
|
16
|
+
allowReadOnlyBash: true,
|
|
17
|
+
});
|
|
18
|
+
assert.strictEqual(normalized.commitPromptTemplate, 'Legacy instruction for {{commitHash}}.', 'legacy Hook prompt should be a read-only compatibility override');
|
|
19
|
+
assert.strictEqual(normalized.legacyPromptOverride, true);
|
|
20
|
+
assert(!Object.hasOwn(normalized, 'hookPromptTemplate'));
|
|
21
|
+
assert(!Object.hasOwn(normalized, 'initPromptTemplate'));
|
|
22
|
+
assert(!Object.hasOwn(normalized, 'postCommitEnabled'));
|
|
23
|
+
assert(!Object.hasOwn(normalized, 'allowReadOnlyBash'));
|
|
24
|
+
assert(!Object.hasOwn(automationConfig, 'DEFAULT_INIT_PROMPT_TEMPLATE'), 'init prompt constant must not be exported');
|
|
25
|
+
assert(!Object.hasOwn(automation, 'dispatchProjectInit'), 'project init dispatch must not be public');
|
|
26
|
+
assert(!Object.hasOwn(automation, 'dispatchAutomation'), 'generic automation dispatch must not be public');
|
|
27
|
+
|
|
28
|
+
const staging = path.resolve('runtime', 'staging', 'run-test');
|
|
29
|
+
const evidenceRoot = path.resolve('runtime', 'runs', 'run-test', 'input', 'evidence');
|
|
30
|
+
const policy = automationConfig.buildAutomationToolPolicy({ stagingPath: staging, evidenceRoot });
|
|
31
|
+
assert.strictEqual(policy.canWriteKb, false, 'AI must never write final knowledge directly');
|
|
32
|
+
assert.strictEqual(policy.canWriteStaging, true);
|
|
33
|
+
assert(!policy.allowedTools.includes('Bash'), 'generic Bash must not be available to analysis');
|
|
34
|
+
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Write', { file_path: path.join(staging, 'manifest.json') }).behavior, 'allow');
|
|
35
|
+
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Write', { file_path: path.resolve('source.js') }).behavior, 'deny');
|
|
36
|
+
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Read', { file_path: path.join(evidenceRoot, 'patch-manifest.json') }).behavior, 'allow');
|
|
37
|
+
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Write', { file_path: path.join(evidenceRoot, 'patches', '000001.patch') }).behavior, 'deny');
|
|
38
|
+
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Read', { file_path: path.resolve('repo', 'source.js') }).behavior, 'deny');
|
|
39
|
+
assert.strictEqual(automationConfig.evaluateAutomationToolUse(policy, 'Bash', { command: 'git show' }).behavior, 'deny');
|
|
40
|
+
|
|
41
|
+
const evidence = {
|
|
42
|
+
commitSha: 'a'.repeat(40),
|
|
43
|
+
parents: ['b'.repeat(40)],
|
|
44
|
+
author: 'Test Author',
|
|
45
|
+
date: '2026-08-17T00:00:00Z',
|
|
46
|
+
subject: 'feat: exact evidence',
|
|
47
|
+
branch: 'main',
|
|
48
|
+
patchMode: 'parent',
|
|
49
|
+
patchHash: 'sha256:patch',
|
|
50
|
+
patchBytes: 42,
|
|
51
|
+
patchOmitted: false,
|
|
52
|
+
patch: 'diff --git a/a.js b/a.js\n+implemented\n',
|
|
53
|
+
files: [{ status: 'M', path: 'a.js', oldPath: null, binary: false, added: 1, deleted: 0 }],
|
|
54
|
+
};
|
|
55
|
+
const config = {
|
|
56
|
+
projectId: 'project-test',
|
|
57
|
+
displayName: 'Prompt Test',
|
|
58
|
+
repoPath: path.resolve('repo'),
|
|
59
|
+
knowledgePath: path.resolve('knowledge'),
|
|
60
|
+
automation: { commitPromptTemplate: automationConfig.DEFAULT_COMMIT_PROMPT_TEMPLATE },
|
|
61
|
+
};
|
|
62
|
+
const missing = renderCommitPrompt({ config, evidence, requirements: [], existingKnowledge: { entries: [], omitted: [] } });
|
|
63
|
+
assert(missing.prompt.includes(REQUIREMENT_UNAVAILABLE), 'unavailable requirement must use the fixed non-guessing text');
|
|
64
|
+
assert(missing.prompt.includes(evidence.patch), 'prompt must contain the actual patch');
|
|
65
|
+
assert(missing.prompt.includes('现有相关知识'), 'prompt must contain existing knowledge section');
|
|
66
|
+
assert(missing.prompt.includes('新增、更新或删除'), 'prompt must ask for add/update/delete decisions');
|
|
67
|
+
assert(!missing.prompt.includes('git-hook') && !missing.prompt.includes('startup'), 'trigger must not affect prompt content');
|
|
68
|
+
|
|
69
|
+
const requirement = { id: 'req-1', client: 'codex', sessionId: 'session-1', ts: '2026-08-17T00:00:00Z', requirement: 'Implement the exact feature.' };
|
|
70
|
+
const fromHook = renderCommitPrompt({ config, evidence, requirements: [requirement], existingKnowledge: { entries: [] } });
|
|
71
|
+
const fromStartup = renderCommitPrompt({ config, evidence, requirements: [requirement], existingKnowledge: { entries: [] } });
|
|
72
|
+
assert.strictEqual(fromHook.promptHash, fromStartup.promptHash, 'Hook and startup must render the identical prompt hash');
|
|
73
|
+
assert(fromHook.prompt.includes('Implement the exact feature.'), 'recorded user requirement must be present');
|
|
74
|
+
|
|
75
|
+
const legacy = renderCommitPrompt({
|
|
76
|
+
config: { ...config, automation: { hookPromptTemplate: 'Legacy {{commitHash}}' } },
|
|
77
|
+
evidence,
|
|
78
|
+
requirements: [],
|
|
79
|
+
existingKnowledge: { entries: [] },
|
|
80
|
+
});
|
|
81
|
+
assert(legacy.prompt.includes('Legacy ' + evidence.commitSha), 'legacy Hook prompt override should still render');
|
|
82
|
+
assert(legacy.prompt.includes(REQUIREMENT_UNAVAILABLE) && legacy.prompt.includes(evidence.patch), 'mandatory evidence sections must be appended to incomplete legacy templates');
|
|
83
|
+
|
|
84
|
+
const source = fs.readFileSync(path.join(__dirname, '..', 'lib', 'post-commit-automation.js'), 'utf8');
|
|
85
|
+
assert(!/dispatchProjectInit|renderProjectInitPrompt|project-init|startup-recovery/.test(source), 'automation source must not retain init or a third trigger name');
|
|
86
|
+
console.log('post-commit-automation-test PASS');
|
|
87
|
+
})();
|