@sanqianx/project-knowledge 5.0.18 → 5.0.20-rc.1

package/_site/_test/git-validation-test.js

@@ -1,81 +1,81 @@
-// Git status remains a read-only v2 API; manual validation/state mutation is removed.
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { execFileSync } = require('child_process');
-const { spawnServer } = require('./helpers/spawn-server');
-const { makeRepo } = require('./fixtures/make-git-repos');
-
-const ROOT = path.resolve(__dirname, '..', '..');
-const DATA_DIR = fs.mkdtempSync(path.join(os.tmpdir(), `kb-data-git-validation-${process.pid}-`));
-const PORT = Number(process.env.KB_GIT_TEST_PORT || 7792);
-const BASE_URL = `http://127.0.0.1:${PORT}`;
-const PROJECT_ID = 'task-002-temp';
-
-async function waitForServer() {
-  for (let attempt = 0; attempt < 100; attempt += 1) {
-    try { if ((await fetch(`${BASE_URL}/api/health`)).ok) return; } catch {}
-    await new Promise(resolve => setTimeout(resolve, 100));
-  }
-  throw new Error('server did not start');
-}
-
-async function json(method, route, body) {
-  const response = await fetch(`${BASE_URL}${route}`, {
-    method,
-    headers: body ? { 'Content-Type': 'application/json' } : undefined,
-    body: body ? JSON.stringify(body) : undefined,
-  });
-  return { response, body: await response.json() };
-}
-
-(async () => {
-  const repo = makeRepo({ kind: 'one-commit' });
-  const spawned = spawnServer({ root: ROOT, port: PORT, dataDir: DATA_DIR, tag: 'git-validation', extraEnv: { KB_AUTOMATION_FAKE_CLAUDE: '1' } });
-  let output = '';
-  spawned.child.stdout.on('data', chunk => { output += chunk; });
-  spawned.child.stderr.on('data', chunk => { output += chunk; });
-  try {
-    await waitForServer();
-    let result = await json('PATCH', '/api/settings', { knowledge: { rootPath: path.join(DATA_DIR, 'knowledge') } });
-    assert(result.response.ok, JSON.stringify(result.body));
-    result = await json('POST', '/api/projects/import', { projectId: PROJECT_ID, localPath: repo.path, displayName: 'Git status fixture' });
-    assert(result.response.status === 201, JSON.stringify(result.body));
-
-    const statePath = path.join(DATA_DIR, 'projects', PROJECT_ID, 'state.json');
-    const stateBefore = fs.readFileSync(statePath, 'utf8');
-    result = await json('GET', `/api/projects/${PROJECT_ID}/git-status`);
-    assert(result.response.ok, JSON.stringify(result.body));
-    assert.strictEqual(result.body.repoStatus, 'ok');
-    assert.strictEqual(result.body.branch, 'main');
-    assert.strictEqual(result.body.dirty, false);
-    assert.match(result.body.headCommit, /^[a-f0-9]{40}$/i);
-    assert.strictEqual(fs.readFileSync(statePath, 'utf8'), stateBefore, 'read-only status must not mutate project state');
-
-    fs.writeFileSync(path.join(repo.path, 'untracked.txt'), 'working tree change\n', 'utf8');
-    result = await json('GET', `/api/projects/${PROJECT_ID}/git-status`);
-    assert.strictEqual(result.body.dirty, true);
-    assert(result.body.changes.some(line => line.includes('untracked.txt')));
-
-    execFileSync('git', ['add', 'untracked.txt'], { cwd: repo.path });
-    execFileSync('git', ['commit', '-m', 'status fixture'], { cwd: repo.path });
-    result = await json('GET', `/api/projects/${PROJECT_ID}/git-status`);
-    assert.strictEqual(result.body.repoStatus, 'ok');
-    assert.strictEqual(result.body.dirty, false);
-
-    result = await json('POST', `/api/projects/${PROJECT_ID}/validate-git`, {});
-    assert.strictEqual(result.response.status, 404, 'manual validation route must remain deleted');
-    result = await json('GET', '/api/projects/%5Ebad/git-status');
-    assert.strictEqual(result.response.status, 400);
-    result = await json('GET', '/api/projects/project-missing/git-status');
-    assert.strictEqual(result.response.status, 404);
-    console.log('TASK-002 git validation test passed');
-  } catch (error) {
-    if (output) process.stderr.write(output);
-    throw error;
-  } finally {
-    await spawned.cleanup();
-    repo.cleanup();
-  }
-})().catch(error => { console.error(error); process.exitCode = 1; });
+// Git status remains a read-only v2 API; manual validation/state mutation is removed.
+const assert = require('assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { execFileSync } = require('child_process');
+const { spawnServer } = require('./helpers/spawn-server');
+const { makeRepo } = require('./fixtures/make-git-repos');
+
+const ROOT = path.resolve(__dirname, '..', '..');
+const DATA_DIR = fs.mkdtempSync(path.join(os.tmpdir(), `kb-data-git-validation-${process.pid}-`));
+const PORT = Number(process.env.KB_GIT_TEST_PORT || 7792);
+const BASE_URL = `http://127.0.0.1:${PORT}`;
+const PROJECT_ID = 'task-002-temp';
+
+async function waitForServer() {
+  for (let attempt = 0; attempt < 100; attempt += 1) {
+    try { if ((await fetch(`${BASE_URL}/api/health`)).ok) return; } catch {}
+    await new Promise(resolve => setTimeout(resolve, 100));
+  }
+  throw new Error('server did not start');
+}
+
+async function json(method, route, body) {
+  const response = await fetch(`${BASE_URL}${route}`, {
+    method,
+    headers: body ? { 'Content-Type': 'application/json' } : undefined,
+    body: body ? JSON.stringify(body) : undefined,
+  });
+  return { response, body: await response.json() };
+}
+
+(async () => {
+  const repo = makeRepo({ kind: 'one-commit' });
+  const spawned = spawnServer({ root: ROOT, port: PORT, dataDir: DATA_DIR, tag: 'git-validation', extraEnv: { KB_AUTOMATION_FAKE_CLAUDE: '1' } });
+  let output = '';
+  spawned.child.stdout.on('data', chunk => { output += chunk; });
+  spawned.child.stderr.on('data', chunk => { output += chunk; });
+  try {
+    await waitForServer();
+    let result = await json('PATCH', '/api/settings', { knowledge: { rootPath: path.join(DATA_DIR, 'knowledge') } });
+    assert(result.response.ok, JSON.stringify(result.body));
+    result = await json('POST', '/api/projects/import', { projectId: PROJECT_ID, localPath: repo.path, displayName: 'Git status fixture' });
+    assert(result.response.status === 201, JSON.stringify(result.body));
+
+    const statePath = path.join(DATA_DIR, 'projects', PROJECT_ID, 'state.json');
+    const stateBefore = fs.readFileSync(statePath, 'utf8');
+    result = await json('GET', `/api/projects/${PROJECT_ID}/git-status`);
+    assert(result.response.ok, JSON.stringify(result.body));
+    assert.strictEqual(result.body.repoStatus, 'ok');
+    assert.strictEqual(result.body.branch, 'main');
+    assert.strictEqual(result.body.dirty, false);
+    assert.match(result.body.headCommit, /^[a-f0-9]{40}$/i);
+    assert.strictEqual(fs.readFileSync(statePath, 'utf8'), stateBefore, 'read-only status must not mutate project state');
+
+    fs.writeFileSync(path.join(repo.path, 'untracked.txt'), 'working tree change\n', 'utf8');
+    result = await json('GET', `/api/projects/${PROJECT_ID}/git-status`);
+    assert.strictEqual(result.body.dirty, true);
+    assert(result.body.changes.some(line => line.includes('untracked.txt')));
+
+    execFileSync('git', ['add', 'untracked.txt'], { cwd: repo.path });
+    execFileSync('git', ['commit', '-m', 'status fixture'], { cwd: repo.path });
+    result = await json('GET', `/api/projects/${PROJECT_ID}/git-status`);
+    assert.strictEqual(result.body.repoStatus, 'ok');
+    assert.strictEqual(result.body.dirty, false);
+
+    result = await json('POST', `/api/projects/${PROJECT_ID}/validate-git`, {});
+    assert.strictEqual(result.response.status, 404, 'manual validation route must remain deleted');
+    result = await json('GET', '/api/projects/%5Ebad/git-status');
+    assert.strictEqual(result.response.status, 400);
+    result = await json('GET', '/api/projects/project-missing/git-status');
+    assert.strictEqual(result.response.status, 404);
+    console.log('TASK-002 git validation test passed');
+  } catch (error) {
+    if (output) process.stderr.write(output);
+    throw error;
+  } finally {
+    await spawned.cleanup();
+    repo.cleanup();
+  }
+})().catch(error => { console.error(error); process.exitCode = 1; });

package/_site/_test/hook-worktree-test.js

@@ -1,42 +1,42 @@
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { spawnSync } = require('child_process');
-const { installHook, readHookStatus, migrateManagedHook, LEGACY_HOOK_MARKER } = 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();
-}
-
-const base = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-hook-worktree-'));
-const main = path.join(base, 'main repo');
-const linked = path.join(base, 'linked worktree');
-fs.mkdirSync(main, { recursive: true });
-git(main, ['init', '--initial-branch=main']);
-git(main, ['config', 'user.email', 'hook@example.invalid']);
-git(main, ['config', 'user.name', 'Hook Test']);
-fs.writeFileSync(path.join(main, 'README.md'), '# main\n');
-git(main, ['add', 'README.md']);
-git(main, ['commit', '-m', 'initial']);
-git(main, ['worktree', 'add', '-b', 'linked', linked]);
-
-git(linked, ['config', 'core.hooksPath', '.custom-hooks']);
-const installed = installHook({ repoPath: linked, projectId: 'project-worktree', triggerScriptPath: TRIGGER });
-const authoritativeHooks = git(linked, ['rev-parse', '--path-format=absolute', '--git-path', 'hooks']);
-assert.strictEqual(path.resolve(path.dirname(installed.hookPath)), path.resolve(authoritativeHooks));
-assert.strictEqual(readHookStatus({ repoPath: linked, projectId: 'project-worktree' }).installed, true);
-
-fs.writeFileSync(installed.hookPath, `#!/bin/sh\n${LEGACY_HOOK_MARKER} — old\nexit 0\n`);
-const migrated = migrateManagedHook({ repoPath: linked, projectId: 'project-worktree', triggerScriptPath: TRIGGER });
-assert.strictEqual(migrated.migrated, true);
-const second = migrateManagedHook({ repoPath: linked, projectId: 'project-worktree', triggerScriptPath: TRIGGER });
-assert.strictEqual(second.migrated, false);
-assert.strictEqual(second.reason, 'current');
-
-console.log('hook-worktree-test PASS');
+const assert = require('assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { spawnSync } = require('child_process');
+const { installHook, readHookStatus, migrateManagedHook, LEGACY_HOOK_MARKER } = 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();
+}
+
+const base = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-hook-worktree-'));
+const main = path.join(base, 'main repo');
+const linked = path.join(base, 'linked worktree');
+fs.mkdirSync(main, { recursive: true });
+git(main, ['init', '--initial-branch=main']);
+git(main, ['config', 'user.email', 'hook@example.invalid']);
+git(main, ['config', 'user.name', 'Hook Test']);
+fs.writeFileSync(path.join(main, 'README.md'), '# main\n');
+git(main, ['add', 'README.md']);
+git(main, ['commit', '-m', 'initial']);
+git(main, ['worktree', 'add', '-b', 'linked', linked]);
+
+git(linked, ['config', 'core.hooksPath', '.custom-hooks']);
+const installed = installHook({ repoPath: linked, projectId: 'project-worktree', triggerScriptPath: TRIGGER });
+const authoritativeHooks = git(linked, ['rev-parse', '--path-format=absolute', '--git-path', 'hooks']);
+assert.strictEqual(path.resolve(path.dirname(installed.hookPath)), path.resolve(authoritativeHooks));
+assert.strictEqual(readHookStatus({ repoPath: linked, projectId: 'project-worktree' }).installed, true);
+
+fs.writeFileSync(installed.hookPath, `#!/bin/sh\n${LEGACY_HOOK_MARKER} — old\nexit 0\n`);
+const migrated = migrateManagedHook({ repoPath: linked, projectId: 'project-worktree', triggerScriptPath: TRIGGER });
+assert.strictEqual(migrated.migrated, true);
+const second = migrateManagedHook({ repoPath: linked, projectId: 'project-worktree', triggerScriptPath: TRIGGER });
+assert.strictEqual(second.migrated, false);
+assert.strictEqual(second.reason, 'current');
+
+console.log('hook-worktree-test PASS');

package/_site/_test/index-builder-test.js

@@ -1,41 +1,41 @@
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { regenerateIndexes } = require('../lib/index-builder');
-
-const temp = fs.mkdtempSync(path.join(os.tmpdir(), 'index-builder-test-'));
-
-try {
-  const modules = path.join(temp, 'modules');
-  const changes = path.join(temp, 'changes');
-  fs.mkdirSync(modules, { recursive: true });
-  fs.mkdirSync(changes, { recursive: true });
-
-  const manyTags = Array.from({ length: 40 }, (_, index) => `tag-${index}`);
-  fs.writeFileSync(path.join(modules, 'core.md'), `---\ntitle: Core | API\ntags: [${manyTags.join(', ')}]\nsourcePaths: [${manyTags.map(tag => `src/${tag}.js`).join(', ')}]\nsymbols: [${manyTags.map(tag => `symbol-${tag}`).join(', ')}]\n---\n# Core\n`, 'utf8');
-
-  for (let index = 0; index < 105; index++) {
-    const day = String((index % 28) + 1).padStart(2, '0');
-    fs.writeFileSync(path.join(changes, `change-${String(index).padStart(3, '0')}.md`), `---\ntitle: Change ${index}\ntags: [${manyTags.join(', ')}]\naffectedModules: [core, api, ui, worker, storage, release]\nupdatedAt: 2026-07-${day}\ndevelopmentIntent: ${'Long intent '.repeat(80)}\n---\n# Change ${index}\n`, 'utf8');
-  }
-
-  regenerateIndexes(temp);
-  const moduleIndex = fs.readFileSync(path.join(modules, '00-index.md'), 'utf8');
-  const changeIndex = fs.readFileSync(path.join(changes, '00-index.md'), 'utf8');
-
-  assert(!/^Tags:/m.test(moduleIndex), 'module index must not aggregate all tags');
-  assert(!/^Tags:/m.test(changeIndex), 'change index must not aggregate all tags');
-  assert(moduleIndex.startsWith('<!-- Generated by project-knowledge.'), 'generated indexes should identify themselves as derived files');
-  assert(moduleIndex.includes('[Core \\| API](./core.md)'), 'Markdown table links should escape title separators');
-  assert(moduleIndex.includes('tag-0, tag-1, tag-2, tag-3 (+36)'), 'tags should be capped with a remainder count');
-  assert(changeIndex.includes('Items: 105 (showing 40;'), 'large change indexes should be bounded');
-  assert.equal((changeIndex.match(/^\| \[/gm) || []).length, 40, 'only the most recent 40 entries should be listed');
-  assert(Buffer.byteLength(changeIndex, 'utf8') < 30000, 'large generated indexes should stay below 30 KiB');
-  assert(Math.max(...moduleIndex.split(/\r?\n/).map(line => line.length)) < 900, 'module index lines should stay compact');
-  assert(Math.max(...changeIndex.split(/\r?\n/).map(line => line.length)) < 900, 'change index lines should stay compact');
-
-  console.log('index-builder-test: PASS');
-} finally {
-  fs.rmSync(temp, { recursive: true, force: true });
-}
+const assert = require('assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { regenerateIndexes } = require('../lib/index-builder');
+
+const temp = fs.mkdtempSync(path.join(os.tmpdir(), 'index-builder-test-'));
+
+try {
+  const modules = path.join(temp, 'modules');
+  const changes = path.join(temp, 'changes');
+  fs.mkdirSync(modules, { recursive: true });
+  fs.mkdirSync(changes, { recursive: true });
+
+  const manyTags = Array.from({ length: 40 }, (_, index) => `tag-${index}`);
+  fs.writeFileSync(path.join(modules, 'core.md'), `---\ntitle: Core | API\ntags: [${manyTags.join(', ')}]\nsourcePaths: [${manyTags.map(tag => `src/${tag}.js`).join(', ')}]\nsymbols: [${manyTags.map(tag => `symbol-${tag}`).join(', ')}]\n---\n# Core\n`, 'utf8');
+
+  for (let index = 0; index < 105; index++) {
+    const day = String((index % 28) + 1).padStart(2, '0');
+    fs.writeFileSync(path.join(changes, `change-${String(index).padStart(3, '0')}.md`), `---\ntitle: Change ${index}\ntags: [${manyTags.join(', ')}]\naffectedModules: [core, api, ui, worker, storage, release]\nupdatedAt: 2026-07-${day}\ndevelopmentIntent: ${'Long intent '.repeat(80)}\n---\n# Change ${index}\n`, 'utf8');
+  }
+
+  regenerateIndexes(temp);
+  const moduleIndex = fs.readFileSync(path.join(modules, '00-index.md'), 'utf8');
+  const changeIndex = fs.readFileSync(path.join(changes, '00-index.md'), 'utf8');
+
+  assert(!/^Tags:/m.test(moduleIndex), 'module index must not aggregate all tags');
+  assert(!/^Tags:/m.test(changeIndex), 'change index must not aggregate all tags');
+  assert(moduleIndex.startsWith('<!-- Generated by project-knowledge.'), 'generated indexes should identify themselves as derived files');
+  assert(moduleIndex.includes('[Core \\| API](./core.md)'), 'Markdown table links should escape title separators');
+  assert(moduleIndex.includes('tag-0, tag-1, tag-2, tag-3 (+36)'), 'tags should be capped with a remainder count');
+  assert(changeIndex.includes('Items: 105 (showing 40;'), 'large change indexes should be bounded');
+  assert.equal((changeIndex.match(/^\| \[/gm) || []).length, 40, 'only the most recent 40 entries should be listed');
+  assert(Buffer.byteLength(changeIndex, 'utf8') < 30000, 'large generated indexes should stay below 30 KiB');
+  assert(Math.max(...moduleIndex.split(/\r?\n/).map(line => line.length)) < 900, 'module index lines should stay compact');
+  assert(Math.max(...changeIndex.split(/\r?\n/).map(line => line.length)) < 900, 'change index lines should stay compact');
+
+  console.log('index-builder-test: PASS');
+} finally {
+  fs.rmSync(temp, { recursive: true, force: true });
+}

package/_site/_test/index-error-display-contract-test.js

@@ -1,10 +1,10 @@
-'use strict';
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const path = require('node:path');
-const source = fs.readFileSync(path.resolve(__dirname, '../../ui/product.js'), 'utf8');
-for (const field of ['upstreamMessage', 'failureKind', 'upstreamCode', 'requestId', 'attempt', 'httpStatus', 'durationMs']) {
-  assert.ok(source.includes(`esc(r.error.details.${field}`), `index detail must be escaped: ${field}`);
-}
-assert.ok(source.includes("['failed', 'index-pending'].includes(r.state)"));
-console.log('index-error-display-contract-test PASS (real rendering/export covered by engine test:web:faults)');
+'use strict';
+const assert = require('node:assert/strict');
+const fs = require('node:fs');
+const path = require('node:path');
+const source = fs.readFileSync(path.resolve(__dirname, '../../ui/product.js'), 'utf8');
+for (const field of ['upstreamMessage', 'failureKind', 'upstreamCode', 'requestId', 'attempt', 'httpStatus', 'durationMs']) {
+  assert.ok(source.includes(`esc(r.error.details.${field}`), `index detail must be escaped: ${field}`);
+}
+assert.ok(source.includes("['failed', 'index-pending'].includes(r.state)"));
+console.log('index-error-display-contract-test PASS (real rendering/export covered by engine test:web:faults)');

package/_site/_test/integration-adapters-test.js

@@ -139,85 +139,85 @@ async function main() {
   });
   const codexResult = await codexManager.operate('update', ['codex']);
   assert(codexResult[0].ok, 'Codex update should complete');
-  assert(codexCalls.some(call => call.args.join(' ') === 'plugin marketplace upgrade project-knowledge'), 'Codex should refresh its marketplace');
-  assert(codexCalls.some(call => call.args.join(' ') === 'plugin remove project-knowledge@project-knowledge'), 'Codex should remove the cached plugin during update');
-  assert(codexCalls.some(call => call.args.join(' ') === 'plugin add project-knowledge@project-knowledge'), 'Codex should install the refreshed plugin');
-
-  const chainedCodexConfig = path.join(TEMP, 'codex-config.toml');
-  const oldNotifier = Buffer.from(JSON.stringify(['node', 'C:/existing-notify.js'])).toString('base64');
-  fs.writeFileSync(chainedCodexConfig, `notify = ["node", "C:/tools/devtask-radar/connectors/codex/notify-hook.js", "--previous-notify", "${oldNotifier}"]\n`, 'utf8');
-  const chainedManager = new IntegrationManager({
-    rootDir: ROOT,
-    homeDir: path.join(TEMP, 'codex-home'),
-    bridgeHomeDir: path.join(TEMP, 'bridge-home'),
-    codexConfigFile: chainedCodexConfig,
-  });
-  const chained = await chainedManager.installCaptureCodex();
-  assert(chained.installed && !chained.conflict, 'a compatible DevTask Radar notifier should chain Bridge instead of blocking capture');
-  const chainedConfig = JSON.parse(fs.readFileSync(chainedCodexConfig, 'utf8').replace(/^\s*notify\s*=\s*/, '').trim());
-  const nextIndex = chainedConfig.indexOf('--next-base64');
+  assert(codexCalls.some(call => call.args.join(' ') === 'plugin marketplace upgrade project-knowledge'), 'Codex should refresh its marketplace');
+  assert(codexCalls.some(call => call.args.join(' ') === 'plugin remove project-knowledge@project-knowledge'), 'Codex should remove the cached plugin during update');
+  assert(codexCalls.some(call => call.args.join(' ') === 'plugin add project-knowledge@project-knowledge'), 'Codex should install the refreshed plugin');
+
+  const chainedCodexConfig = path.join(TEMP, 'codex-config.toml');
+  const oldNotifier = Buffer.from(JSON.stringify(['node', 'C:/existing-notify.js'])).toString('base64');
+  fs.writeFileSync(chainedCodexConfig, `notify = ["node", "C:/tools/devtask-radar/connectors/codex/notify-hook.js", "--previous-notify", "${oldNotifier}"]\n`, 'utf8');
+  const chainedManager = new IntegrationManager({
+    rootDir: ROOT,
+    homeDir: path.join(TEMP, 'codex-home'),
+    bridgeHomeDir: path.join(TEMP, 'bridge-home'),
+    codexConfigFile: chainedCodexConfig,
+  });
+  const chained = await chainedManager.installCaptureCodex();
+  assert(chained.installed && !chained.conflict, 'a compatible DevTask Radar notifier should chain Bridge instead of blocking capture');
+  const chainedConfig = JSON.parse(fs.readFileSync(chainedCodexConfig, 'utf8').replace(/^\s*notify\s*=\s*/, '').trim());
+  const nextIndex = chainedConfig.indexOf('--next-base64');
   assert(nextIndex >= 0, 'managed fan-out must preserve the compatible notifier as its independent branch');
-  const nextNotify = JSON.parse(Buffer.from(chainedConfig[nextIndex + 1], 'base64').toString('utf8'));
+  const nextNotify = JSON.parse(Buffer.from(chainedConfig[nextIndex + 1], 'base64').toString('utf8'));
   assert(nextNotify[1] === 'C:/tools/devtask-radar/connectors/codex/notify-hook.js', 'Radar remains the existing independent notifier');
   const bridgeIndex = chainedConfig.indexOf('--bridge-base64');
   const bridgeNotify = JSON.parse(Buffer.from(chainedConfig[bridgeIndex + 1], 'base64').toString('utf8'));
   assert(bridgeNotify[1] === path.join(TEMP, 'bridge-home', 'bin', 'bridge-hook.cjs'), 'capture must receive the payload through the current Bridge shim');
-  const chainedStatus = await chainedManager.statusCaptureCodex();
-  assert(chainedStatus.installed && !chainedStatus.conflict, 'status should recognize the compatible chained notifier');
-
-  const nestedCodexConfig = path.join(TEMP, 'nested-codex-config.toml');
-  const nestedDevTask = ['node', 'C:/tools/devtask-radar/connectors/codex/notify-hook.js', '--next-base64', oldNotifier];
+  const chainedStatus = await chainedManager.statusCaptureCodex();
+  assert(chainedStatus.installed && !chainedStatus.conflict, 'status should recognize the compatible chained notifier');
+
+  const nestedCodexConfig = path.join(TEMP, 'nested-codex-config.toml');
+  const nestedDevTask = ['node', 'C:/tools/devtask-radar/connectors/codex/notify-hook.js', '--next-base64', oldNotifier];
   const desktopExecutable = path.join(TEMP, 'OpenAI', 'Codex', 'runtimes', 'cua_node', 'test-runtime', 'bin', 'node_modules', '@oai', 'sky', 'bin', 'windows', 'codex-computer-use.exe');
   fs.mkdirSync(path.dirname(desktopExecutable), { recursive: true });
   fs.writeFileSync(desktopExecutable, '');
   fs.writeFileSync(nestedCodexConfig, `notify = ${JSON.stringify([desktopExecutable, 'turn-ended', '--previous-notify', JSON.stringify(nestedDevTask)])}\n`, 'utf8');
-  const nestedManager = new IntegrationManager({
-    rootDir: ROOT,
-    homeDir: path.join(TEMP, 'nested-codex-home'),
-    bridgeHomeDir: path.join(TEMP, 'nested-bridge-home'),
-    codexConfigFile: nestedCodexConfig,
-  });
-  const nested = await nestedManager.installCaptureCodex();
-  assert(nested.installed && !nested.conflict, 'a nested DevTask Radar notifier should be preserved through the fan-out wrapper');
-  const nestedConfig = JSON.parse(fs.readFileSync(nestedCodexConfig, 'utf8').replace(/^\s*notify\s*=\s*/, '').trim());
+  const nestedManager = new IntegrationManager({
+    rootDir: ROOT,
+    homeDir: path.join(TEMP, 'nested-codex-home'),
+    bridgeHomeDir: path.join(TEMP, 'nested-bridge-home'),
+    codexConfigFile: nestedCodexConfig,
+  });
+  const nested = await nestedManager.installCaptureCodex();
+  assert(nested.installed && !nested.conflict, 'a nested DevTask Radar notifier should be preserved through the fan-out wrapper');
+  const nestedConfig = JSON.parse(fs.readFileSync(nestedCodexConfig, 'utf8').replace(/^\s*notify\s*=\s*/, '').trim());
   assert(nestedConfig[0] === desktopExecutable, 'the active Codex desktop wrapper keeps its position');
   assert(nestedConfig.join(' ').includes('codex-notify-fanout.cjs'), 'nested notifier chain should use the Project Knowledge fan-out wrapper');
-  const nestedStatus = await nestedManager.statusCaptureCodex();
-  assert(nestedStatus.installed && !nestedStatus.conflict, 'status should recognize the managed nested fan-out wrapper');
-
-  // Codex Desktop computer-use wraps the existing notifier as a JSON array in
-  // --previous-notify. The managed fan-out is still installed and must not be
-  // misreported as a third-party conflict or wrapped a second time.
-  const desktopWrappedConfig = path.join(TEMP, 'desktop-wrapped-codex-config.toml');
+  const nestedStatus = await nestedManager.statusCaptureCodex();
+  assert(nestedStatus.installed && !nestedStatus.conflict, 'status should recognize the managed nested fan-out wrapper');
+
+  // Codex Desktop computer-use wraps the existing notifier as a JSON array in
+  // --previous-notify. The managed fan-out is still installed and must not be
+  // misreported as a third-party conflict or wrapped a second time.
+  const desktopWrappedConfig = path.join(TEMP, 'desktop-wrapped-codex-config.toml');
   const desktopWrappedArgs = [desktopExecutable, 'turn-ended', '--previous-notify', JSON.stringify(nestedConfig)];
-  fs.writeFileSync(desktopWrappedConfig, `notify = ${JSON.stringify(desktopWrappedArgs)}\n`, 'utf8');
-  const desktopWrappedManager = new IntegrationManager({
-    rootDir: ROOT,
-    homeDir: path.join(TEMP, 'desktop-wrapped-codex-home'),
-    bridgeHomeDir: path.join(TEMP, 'nested-bridge-home'),
-    codexConfigFile: desktopWrappedConfig,
-  });
-  const desktopWrappedStatus = await desktopWrappedManager.statusCaptureCodex();
-  assert(desktopWrappedStatus.installed && !desktopWrappedStatus.conflict,
-    'Codex Desktop outer wrapper should preserve managed capture health');
-  const beforeDesktopInstall = fs.readFileSync(desktopWrappedConfig, 'utf8');
-  const desktopWrappedInstall = await desktopWrappedManager.installCaptureCodex();
-  assert(desktopWrappedInstall.installed && !desktopWrappedInstall.conflict,
-    'one-click repair should recognize the nested managed fan-out');
-  assert(fs.readFileSync(desktopWrappedConfig, 'utf8') === beforeDesktopInstall,
-    'healthy Codex Desktop wrapper must remain byte-for-byte unchanged');
-
-  const bridgeOnlyConfig = path.join(TEMP, 'bridge-only-codex-config.toml');
-  const bridgeOnlyManager = new IntegrationManager({
-    rootDir: ROOT,
-    homeDir: path.join(TEMP, 'bridge-only-codex-home'),
-    bridgeHomeDir: path.join(TEMP, 'bridge-only-bridge-home'),
-    codexConfigFile: bridgeOnlyConfig,
-  });
-  const bridgeOnly = await bridgeOnlyManager.installCaptureCodex();
-  assert(bridgeOnly.installed && !bridgeOnly.conflict, 'a fresh Codex install should use the payload-preserving fan-out wrapper');
-  const bridgeOnlyArgs = JSON.parse(fs.readFileSync(bridgeOnlyConfig, 'utf8').replace(/^\s*notify\s*=\s*/, '').trim());
-  assert(bridgeOnlyArgs.join(' ').includes('codex-notify-fanout.cjs'), 'fresh Codex install should use the Project Knowledge fan-out wrapper');
+  fs.writeFileSync(desktopWrappedConfig, `notify = ${JSON.stringify(desktopWrappedArgs)}\n`, 'utf8');
+  const desktopWrappedManager = new IntegrationManager({
+    rootDir: ROOT,
+    homeDir: path.join(TEMP, 'desktop-wrapped-codex-home'),
+    bridgeHomeDir: path.join(TEMP, 'nested-bridge-home'),
+    codexConfigFile: desktopWrappedConfig,
+  });
+  const desktopWrappedStatus = await desktopWrappedManager.statusCaptureCodex();
+  assert(desktopWrappedStatus.installed && !desktopWrappedStatus.conflict,
+    'Codex Desktop outer wrapper should preserve managed capture health');
+  const beforeDesktopInstall = fs.readFileSync(desktopWrappedConfig, 'utf8');
+  const desktopWrappedInstall = await desktopWrappedManager.installCaptureCodex();
+  assert(desktopWrappedInstall.installed && !desktopWrappedInstall.conflict,
+    'one-click repair should recognize the nested managed fan-out');
+  assert(fs.readFileSync(desktopWrappedConfig, 'utf8') === beforeDesktopInstall,
+    'healthy Codex Desktop wrapper must remain byte-for-byte unchanged');
+
+  const bridgeOnlyConfig = path.join(TEMP, 'bridge-only-codex-config.toml');
+  const bridgeOnlyManager = new IntegrationManager({
+    rootDir: ROOT,
+    homeDir: path.join(TEMP, 'bridge-only-codex-home'),
+    bridgeHomeDir: path.join(TEMP, 'bridge-only-bridge-home'),
+    codexConfigFile: bridgeOnlyConfig,
+  });
+  const bridgeOnly = await bridgeOnlyManager.installCaptureCodex();
+  assert(bridgeOnly.installed && !bridgeOnly.conflict, 'a fresh Codex install should use the payload-preserving fan-out wrapper');
+  const bridgeOnlyArgs = JSON.parse(fs.readFileSync(bridgeOnlyConfig, 'utf8').replace(/^\s*notify\s*=\s*/, '').trim());
+  assert(bridgeOnlyArgs.join(' ').includes('codex-notify-fanout.cjs'), 'fresh Codex install should use the Project Knowledge fan-out wrapper');
 
   const integrationLogs = [];
   const failingManager = new IntegrationManager({