@sanqianx/project-knowledge 4.7.0 → 5.0.0

package/_site/_test/import-ui-flow-test.js

@@ -1,171 +0,0 @@
-// _site/_test/import-ui-flow-test.js
-//
-// T07: end-to-end UI smoke for the Import view.
-// Verifies the preflight-driven UI: every input runs /api/projects/preflight-import,
-// the Import button stays disabled until ready=true, and problems surface
-// as a structured error list with an action hint.
-
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { spawnSync } = require('child_process');
-const { spawnServer } = require('./helpers/spawn-server');
-const { findChrome } = require('./helpers/find-chrome');
-const { launchCdpBrowser, waitFor } = require('./helpers/cdp-browser');
-
-const ROOT = path.resolve(__dirname, '..', '..');
-const sitePort = 7960 + (process.pid % 200);
-const profileDir = path.join(os.tmpdir(), 'pk-import-ui-' + process.pid);
-
-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: ${result.status}`);
-  return String(result.stdout || '').trim();
-}
-
-(async () => {
-  const dataDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-import-ui-data-' + process.pid + '-'));
-  const repo = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-import-ui-repo-' + process.pid + '-'));
-  const repoSpaces = path.join(fs.mkdtempSync(path.join(os.tmpdir(), 'pk-import-ui-spaces-' + process.pid + '-')), 'project with spaces');
-  fs.mkdirSync(repoSpaces, { recursive: true });
-  git(repo, ['init', '-q', '-b', 'main']);
-  git(repo, ['config', 'user.email', 'import-ui@example.local']);
-  git(repo, ['config', 'user.name', 'Import UI']);
-  fs.writeFileSync(path.join(repo, 'README.md'), '# import ui\n');
-  git(repo, ['add', 'README.md']);
-  git(repo, ['commit', '-q', '-m', 'baseline']);
-  git(repoSpaces, ['init', '-q', '-b', 'main']);
-  git(repoSpaces, ['config', 'user.email', 'import-ui@example.local']);
-  git(repoSpaces, ['config', 'user.name', 'Import UI']);
-  fs.writeFileSync(path.join(repoSpaces, 'README.md'), '# spaces\n');
-  git(repoSpaces, ['add', 'README.md']);
-  git(repoSpaces, ['commit', '-q', '-m', 'baseline']);
-  const server = spawnServer({
-    root: ROOT, port: sitePort, dataDir, tag: 'import-ui',
-    extraEnv: { KB_AUTOMATION_FAKE_CLAUDE: '1', KB_EMBEDDING_FAKE: '1', KB_SKIP_MIGRATION: '1' },
-  });
-  let browser;
-  // Plain Node fetch with explicit timeout. The cdp-browser requestJson helper
-  // uses a 1-second default which is too short for the server's first request
-  // after startup; we use Node's fetch + AbortSignal for direct setup calls.
-  async function serverFetch(url, init) {
-    const controller = new AbortController();
-    const timer = setTimeout(() => controller.abort(), 10_000);
-    try {
-      const response = await fetch(url, { ...init, signal: controller.signal });
-      const text = await response.text();
-      return { status: response.status, body: text ? JSON.parse(text) : {} };
-    } finally {
-      clearTimeout(timer);
-    }
-  }
-  try {
-    await waitFor(async () => {
-      try { const r = await serverFetch(`http://127.0.0.1:${sitePort}/api/health`); return r.body.ok ? { ok: true } : null; } catch { return null; }
-    }, 'isolated server', 20000);
-    await serverFetch(`http://127.0.0.1:${sitePort}/api/settings`, {
-      method: 'PATCH',
-      headers: { 'content-type': 'application/json' },
-      body: JSON.stringify({ knowledge: { rootPath: path.join(dataDir, 'knowledge') } }),
-    });
-    await serverFetch(`http://127.0.0.1:${sitePort}/api/ai-profiles`, {
-      method: 'PUT',
-      headers: { 'content-type': 'application/json' },
-      body: JSON.stringify({
-        schema: 'ai-profiles/v1', defaultProfileId: 'fake',
-        profiles: [{ id: 'fake', name: 'Fake', enabled: true, vendor: 'anthropic', model: 'fake', apiKeyUpdate: { mode: 'replace', value: 'k' } }],
-      }),
-    });
-    browser = await launchCdpBrowser({
-      chrome: findChrome(), profileDir, url: `http://127.0.0.1:${sitePort}/`, width: 1440, height: 900,
-    });
-    // Switch to Import view
-    await browser.evaluate("document.querySelector('[data-go=import]').click()");
-    await waitFor(async () => {
-      const visible = await browser.evaluate("document.getElementById('view-import').classList.contains('active')");
-      return visible ? { ok: true } : null;
-    }, 'import view visible', 20000);
-
-    // Case 1: empty path -> Import button disabled
-    {
-      const disabled = await browser.evaluate("document.getElementById('import-submit').disabled");
-      assert.strictEqual(disabled, true, 'Import button must be disabled when path is empty');
-    }
-
-    // Case 2: enter path with spaces -> preflight returns ready=true; button enabled
-    await browser.evaluate(`(() => {
-      const path = ${JSON.stringify(repoSpaces)};
-      const input = document.getElementById('import-path');
-      input.value = path;
-      input.dispatchEvent(new Event('input', { bubbles: true }));
-      input.dispatchEvent(new Event('change', { bubbles: true }));
-    })()`);
-    await waitFor(async () => {
-      const state = await browser.evaluate(`(() => ({
-        disabled: document.getElementById('import-submit').disabled,
-        git: document.getElementById('pv-git-status').textContent,
-        root: document.getElementById('pv-knowledge-root').textContent,
-        preflightHidden: document.getElementById('import-preflight').hidden,
-        errHidden: document.getElementById('import-errors').hidden,
-      }))()`);
-      return state.disabled === false && state.errHidden ? { ok: true, state } : null;
-    }, 'preflight ready for valid path', 20000);
-
-    // Case 3: ② 知识库地址 → 预览回显该地址，导入按钮保持可用（两地址即登记映射）
-    await browser.evaluate(`(() => {
-      const input = document.getElementById('import-knowledge-path');
-      input.value = 'D:\\\\Knowledge\\\\spaces-demo';
-      input.dispatchEvent(new Event('input', { bubbles: true }));
-      input.dispatchEvent(new Event('change', { bubbles: true }));
-    })()`);
-    await waitFor(async () => {
-      const state = await browser.evaluate(`(() => ({
-        disabled: document.getElementById('import-submit').disabled,
-        root: document.getElementById('pv-knowledge-root').textContent,
-      }))()`);
-      return state.disabled === false && /spaces-demo/.test(state.root) ? { ok: true } : null;
-    }, 'knowledge address reflected in preview', 20000);
-
-    // Case 5: submit form -> project imported; UI navigates to the terminal view.
-    await browser.evaluate(`(() => {
-      const form = document.getElementById('import-form');
-      if (typeof form.requestSubmit === 'function') form.requestSubmit();
-      else form.dispatchEvent(new Event('submit', { bubbles: true, cancelable: true }));
-    })()`);
-    await waitFor(async () => {
-      const visible = await browser.evaluate("document.getElementById('view-terminal').classList.contains('active')");
-      return visible ? { ok: true } : null;
-    }, 'import navigates to terminal view', 30000);
-
-    // Case 6: folder picker buttons are visible in Web mode and wired to the
-    // proxied module pickers (never clicked here — that would open a real
-    // native dialog on the host machine).
-    {
-      const pickers = await browser.evaluate(`(() => ({
-        dev: !document.getElementById('import-pick-folder').hidden,
-        knowledge: !document.getElementById('import-pick-knowledge').hidden,
-      }))()`);
-      assert.strictEqual(pickers.dev, true, 'dev-address picker must be visible');
-      assert.strictEqual(pickers.knowledge, true, 'knowledge-address picker must be visible');
-    }
-  } catch (error) {
-    console.error(error.stack || error.message);
-    throw error;
-  } finally {
-    if (browser) await browser.close();
-    server.child.kill();
-    await new Promise(resolve => server.child.once('exit', resolve));
-    fs.rmSync(dataDir, { recursive: true, force: true });
-    fs.rmSync(repo, { recursive: true, force: true });
-    fs.rmSync(repoSpaces, { recursive: true, force: true });
-  }
-  console.log('import-ui-flow-test PASS');
-})().catch(error => {
-  console.error(error.stack || error.message);
-  process.exit(1);
-});

package/_site/_test/knowledge-migration-test.js

@@ -1,75 +0,0 @@
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { StorageLayout } = require('../lib/storage-layout');
-const { SettingsStore } = require('../lib/settings-store');
-const { ProjectStore } = require('../lib/project-store');
-const { MigrationService } = require('../lib/migration-service');
-const { KnowledgeDatabase } = require('../lib/knowledge-db');
-const { EMBEDDING_DIMENSIONS } = require('../lib/knowledge-schema');
-
-(async () => {
-  const dataDir = fs.mkdtempSync(path.join(os.tmpdir(), 'knowledge-migration-v2-'));
-  const oldKnowledgeRoot = path.join(dataDir, 'old-knowledge');
-  const fixedKnowledgePath = path.join(oldKnowledgeRoot, 'api-fixed');
-  const legacyIndex = path.join(dataDir, 'knowledge.lancedb');
-  fs.mkdirSync(fixedKnowledgePath, { recursive: true });
-  fs.writeFileSync(path.join(fixedKnowledgePath, 'GOAL.md'), '# API goal\n\nAuthoritative Markdown remains external.\n', 'utf8');
-  fs.writeFileSync(path.join(dataDir, 'projects.json'), JSON.stringify({
-    api: {
-      displayName: 'API', localPath: dataDir, gitPath: dataDir, kbPath: fixedKnowledgePath,
-      trackingStartCommit: 'track-before', lastAnalyzedCommit: 'commit-before', enabled: true,
-    },
-  }, null, 2));
-  fs.writeFileSync(path.join(dataDir, 'knowledge-store.json'), JSON.stringify({ rootPath: oldKnowledgeRoot, configured: true }));
-  fs.writeFileSync(path.join(dataDir, 'ai-profiles.json'), JSON.stringify({ schema: 'ai-profiles/v1', profiles: [{ id: 'primary', apiKey: 'migration-secret-value' }] }));
-  fs.writeFileSync(path.join(dataDir, 'logging.json'), JSON.stringify({ levels: ['debug', 'info', 'error'], retentionDays: 0, maxTotalSizeMB: 128 }));
-  fs.writeFileSync(path.join(dataDir, '.hook-trigger-errors.log'), '2026-01-01 legacy hook error\n', 'utf8');
-
-  const vector = new Array(EMBEDDING_DIMENSIONS).fill(0);
-  vector[9] = 1;
-  const oldDatabase = new KnowledgeDatabase({ dbPath: legacyIndex });
-  await oldDatabase.replaceEntry('project:api', 'GOAL.md', [{ chunkOrder: 0, title: 'Legacy index', chunkText: 'A migrated derived row.', vector }]);
-  await oldDatabase.close();
-  const oldIndexRows = await (async () => {
-    const db = new KnowledgeDatabase({ dbPath: legacyIndex });
-    try { return await db.count(); } finally { await db.close(); }
-  })();
-
-  const layout = new StorageLayout({ dataDir });
-  const migration = new MigrationService({
-    layout,
-    legacyDataDir: dataDir,
-    indexValidator: async candidate => {
-      const db = new KnowledgeDatabase({ dbPath: candidate });
-      try { await db.open(); return { ok: (await db.count()) === oldIndexRows }; }
-      finally { await db.close(); }
-    },
-  });
-  const result = await migration.migrateIfNeeded({ migrationRunId: 'knowledge-index-v2' });
-  assert(result.ok && result.completed && result.migrated, JSON.stringify(result));
-  const projectId = result.projectMap.api;
-  const projectStore = new ProjectStore({ layout });
-  const config = projectStore.readConfig(projectId);
-  const state = projectStore.readState(projectId);
-  assert.strictEqual(config.knowledgePath, fixedKnowledgePath, 'legacy kbPath must be preserved exactly');
-  assert.strictEqual(state.trackingStartCommit, 'track-before');
-  assert.strictEqual(state.lastAnalyzedCommit, 'commit-before');
-  assert.strictEqual(new SettingsStore({ layout }).read().ai.profiles[0].apiKey, 'migration-secret-value');
-  const migratedDatabase = new KnowledgeDatabase({ dbPath: layout.getIndexPath() });
-  assert.strictEqual(await migratedDatabase.count(), 1, 'the validated legacy index should be copied to the one internal index path');
-  await migratedDatabase.close();
-  assert(fs.existsSync(legacyIndex), 'legacy index source must remain for rollback');
-  assert(fs.existsSync(path.join(result.recoveryDir, 'backup', 'knowledge.lancedb')), 'recovery must retain an index backup');
-  assert(fs.existsSync(path.join(dataDir, '.hook-trigger-errors.log')), 'legacy hook logs must remain at the source');
-  assert(fs.existsSync(path.join(result.recoveryDir, 'backup', '.hook-trigger-errors.log')));
-
-  const settings = new SettingsStore({ layout });
-  await settings.updatePatch({ knowledge: { rootPath: path.join(dataDir, 'new-knowledge-root') } });
-  assert.strictEqual(projectStore.readConfig(projectId).knowledgePath, fixedKnowledgePath, 'changing the global root must not move an imported project');
-  assert.strictEqual(layout.getIndexPath(), path.join(dataDir, 'index', 'knowledge.lancedb'), 'global root changes must not move the derived index');
-  const second = await migration.migrateIfNeeded();
-  assert.strictEqual(second.migrated, false);
-  console.log('knowledge-migration-test: PASS');
-})().catch(error => { console.error(error); process.exitCode = 1; });

package/_site/_test/legacy-forward-compat-test.js

@@ -1,257 +0,0 @@
-// _site/_test/legacy-forward-compat-test.js
-//
-// T08: v4.1.22 -> main forward-compatibility migration. Builds a realistic
-// two-project legacy fixture with team binding + legacy hook + tracking
-// state, runs MigrationService, and asserts every required field is
-// preserved or deterministically derived. Also runs migrateManagedHooks
-// to confirm the T03 hook state machine is the post-migration reality.
-
-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 { MigrationService } = require('../lib/migration-service');
-const { ProjectStore } = require('../lib/project-store');
-const { ProjectRegistryStore } = require('../lib/project-registry-store');
-const { Logger } = require('../lib/structured-logger');
-const { migrateManagedHooks } = require('../lib/server-app');
-const { LEGACY_HOOK_MARKER, readHookStatus } = 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: ${result.status}`);
-  return String(result.stdout || '').trim();
-}
-
-function fixture({ withTeamBinding, withLegacyHook, knowledgeLanguage, aiProfileId, enabled, repoPath: customRepoPath } = {}) {
-  const dataDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-legacy-forward-compat-'));
-  const repoPath = customRepoPath || path.join(dataDir, 'repos', 'legacy-team');
-  fs.mkdirSync(repoPath, { recursive: true });
-  git(repoPath, ['init', '-q', '-b', 'main']);
-  git(repoPath, ['config', 'user.email', 'legacy-fc@example.local']);
-  git(repoPath, ['config', 'user.name', 'Legacy FC']);
-  fs.writeFileSync(path.join(repoPath, 'README.md'), '# legacy\n');
-  git(repoPath, ['add', 'README.md']);
-  git(repoPath, ['commit', '-q', '-m', 'initial']);
-  const initialSha = git(repoPath, ['rev-parse', 'HEAD']);
-
-  if (withLegacyHook) {
-    const gitDir = git(repoPath, ['rev-parse', '--path-format=absolute', '--git-dir']);
-    const hooksDir = path.join(path.resolve(gitDir), 'hooks');
-    fs.mkdirSync(hooksDir, { recursive: true });
-    fs.writeFileSync(path.join(hooksDir, 'post-commit'), (
-      `#!/bin/sh\n${LEGACY_HOOK_MARKER} — auto-installed by KB manager (legacy)\n` +
-      `node 'C:/legacy/site/_site/scripts/hook-trigger.js' \\\n` +
-      `  --kb-root 'C:/legacy/site' --repo "${repoPath.replace(/\\/g, '/')}" >/dev/null 2>&1 || true\n` +
-      `exit 0\n`
-    ), { mode: 0o755 });
-  }
-
-  // Knowledge path
-  const kbPath = path.join(dataDir, 'knowledge', 'legacy-team');
-  fs.mkdirSync(kbPath, { recursive: true });
-
-  // Team binding
-  const teamBinding = withTeamBinding ? {
-    teamKbId: 'team-store/team-a',
-    provider: 'github',
-  } : null;
-
-  const projectEntry = {
-    schema: 'kb-project/v1',
-    displayName: 'Legacy Team',
-    localPath: repoPath,
-    gitPath: repoPath,
-    kbPath,
-    trackingStartCommit: initialSha,
-    lastAnalyzedCommit: initialSha,
-    enabled: enabled !== false,
-    aiProfileId: aiProfileId !== undefined ? aiProfileId : 'primary',
-    knowledgeLanguage: knowledgeLanguage || 'zh-CN',
-    teamBinding,
-  };
-
-  fs.writeFileSync(path.join(dataDir, 'projects.json'), JSON.stringify({
-    'legacy-team': projectEntry,
-  }, null, 2));
-  fs.writeFileSync(path.join(dataDir, 'knowledge-store.json'), JSON.stringify({
-    schema: 'knowledge-store/v1',
-    rootPath: path.join(dataDir, 'knowledge'),
-    configured: true,
-  }));
-  fs.writeFileSync(path.join(dataDir, 'ai-profiles.json'), JSON.stringify({
-    schema: 'ai-profiles/v1',
-    profiles: [{ id: 'primary', name: 'Primary', enabled: true, vendor: 'anthropic', model: 'claude-sonnet' }],
-    defaultProfileId: 'primary',
-  }));
-  return { dataDir, repoPath, kbPath, initialSha };
-}
-
-async function run() {
-  // Case 1: deterministic projectId + repoPath + knowledgePath + tracking
-  {
-    const { dataDir, repoPath, kbPath, initialSha } = fixture();
-    const layout = new StorageLayout({ dataDir });
-    const migration = new MigrationService({ layout, legacyDataDir: dataDir });
-    const result = await migration.migrateIfNeeded({ migrationRunId: 'fc-det' });
-    assert.strictEqual(result.ok, true);
-    const projectId = result.projectMap['legacy-team'];
-    assert(projectId, 'deterministic projectId must be produced');
-    const config = JSON.parse(fs.readFileSync(layout.getProjectConfigPath(projectId), 'utf8'));
-    const state = JSON.parse(fs.readFileSync(layout.getProjectStatePath(projectId), 'utf8'));
-    assert.strictEqual(config.repoPath, path.resolve(repoPath), 'repoPath must be preserved (not data-dir fallback)');
-    assert.strictEqual(config.knowledgePath, path.resolve(kbPath), 'knowledgePath must be preserved');
-    assert.strictEqual(config.enabled, true, 'enabled must be preserved');
-    assert.strictEqual(state.trackingStartCommit, initialSha, 'trackingStartCommit must be preserved');
-    assert.strictEqual(state.lastAnalyzedCommit, initialSha, 'lastAnalyzedCommit must be preserved');
-    // Idempotence: second migration must not re-migrate
-    const second = await migration.migrateIfNeeded();
-    assert.strictEqual(second.completed, true);
-    assert.strictEqual(second.migrated, false);
-    fs.rmSync(dataDir, { recursive: true, force: true });
-  }
-
-  // Case 2: knowledgeLanguage + aiProfileId + teamBinding preservation
-  {
-    const { dataDir, repoPath, kbPath } = fixture({
-      withTeamBinding: true, knowledgeLanguage: 'en-US', aiProfileId: 'primary',
-    });
-    const layout = new StorageLayout({ dataDir });
-    const migration = new MigrationService({ layout, legacyDataDir: dataDir });
-    const result = await migration.migrateIfNeeded({ migrationRunId: 'fc-team' });
-    assert.strictEqual(result.ok, true);
-    const projectId = result.projectMap['legacy-team'];
-    const config = JSON.parse(fs.readFileSync(layout.getProjectConfigPath(projectId), 'utf8'));
-    assert.strictEqual(config.knowledgeLanguage, 'en-US', 'knowledgeLanguage must be preserved');
-    assert.strictEqual(config.aiProfileId, 'primary', 'aiProfileId must be preserved');
-    assert(config.teamBinding && config.teamBinding.teamKbId === 'team-store/team-a', 'teamBinding must be preserved');
-    assert.strictEqual(config.teamBinding.provider, 'github');
-    fs.rmSync(dataDir, { recursive: true, force: true });
-  }
-
-  // Case 3: AI profile semantics via T01 — stale project aiProfileId
-  // falls back to default or first-usable. Here legacy has aiProfileId='stale'
-  // but the global default is 'primary'; the resolver must surface 'primary'
-  // when CommitReconciler runs. (The migration itself preserves the stale
-  // id; the resolver is what fixes it at runtime.)
-  {
-    const { dataDir, repoPath } = fixture({ aiProfileId: 'stale-deleted-profile' });
-    const layout = new StorageLayout({ dataDir });
-    const migration = new MigrationService({ layout, legacyDataDir: dataDir });
-    const result = await migration.migrateIfNeeded({ migrationRunId: 'fc-stale-profile' });
-    const projectId = result.projectMap['legacy-team'];
-    const config = JSON.parse(fs.readFileSync(layout.getProjectConfigPath(projectId), 'utf8'));
-    assert.strictEqual(config.aiProfileId, 'stale-deleted-profile', 'migration preserves whatever was in the legacy record; resolver handles fallback at runtime');
-    const { resolveEffectiveAiProfile } = require('../lib/ai-profile-resolver');
-    const settings = JSON.parse(fs.readFileSync(layout.getSettingsPath(), 'utf8'));
-    const resolved = resolveEffectiveAiProfile(settings, config);
-    assert.strictEqual(resolved.profileId, 'primary', 'T01 resolver must fall back to default when project id is stale');
-    assert.strictEqual(resolved.source, 'default');
-    fs.rmSync(dataDir, { recursive: true, force: true });
-  }
-
-  // Case 4: hook migration via T03 — legacy v1 hook present, must be replaced
-  // and project state must report verified v2 after migrateManagedHooks.
-  {
-    const { dataDir, repoPath } = fixture({ withLegacyHook: true });
-    const layout = new StorageLayout({ dataDir });
-    const migration = new MigrationService({ layout, legacyDataDir: dataDir });
-    const result = await migration.migrateIfNeeded({ migrationRunId: 'fc-hook' });
-    const projectId = result.projectMap['legacy-team'];
-    const projectStore = new ProjectStore({ layout });
-    const registryStore = new ProjectRegistryStore({ layout });
-    await registryStore.initialize();
-    const logger = new Logger({ layout, settingsProvider: () => ({ levels: [] }), context: { component: 'fc' } });
-    const runtime = { registryStore, projectStore, logger, layout };
-    const hooks = await migrateManagedHooks(runtime);
-    const entry = hooks.find(item => item.projectId === projectId);
-    assert(entry, 'migrateManagedHooks must process the migrated project');
-    assert.strictEqual(entry.status, 'verified');
-    assert.strictEqual(entry.reason, 'v1-migrated');
-    const status = readHookStatus({ repoPath, projectId });
-    assert.strictEqual(status.kbManaged, true);
-    assert.strictEqual(status.managedVersion, 2);
-    const finalState = projectStore.readState(projectId);
-    assert.strictEqual(finalState.hook.migrationVersion, 2);
-    await logger.close();
-    fs.rmSync(dataDir, { recursive: true, force: true });
-  }
-
-  // Case 5: disabled=false preservation
-  {
-    const { dataDir } = fixture({ enabled: false });
-    const layout = new StorageLayout({ dataDir });
-    const migration = new MigrationService({ layout, legacyDataDir: dataDir });
-    const result = await migration.migrateIfNeeded({ migrationRunId: 'fc-disabled' });
-    const projectId = result.projectMap['legacy-team'];
-    const config = JSON.parse(fs.readFileSync(layout.getProjectConfigPath(projectId), 'utf8'));
-    assert.strictEqual(config.enabled, false, 'enabled=false must be preserved');
-    fs.rmSync(dataDir, { recursive: true, force: true });
-  }
-
-  // Case 6: repoPath MUST NOT be replaced with a data-dir fallback when a
-  // valid legacy repoPath exists. (RC-style protection: a buggy migration
-  // would substitute knowledgeDir/projects/<id> for repoPath.)
-  {
-    const dataDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-legacy-no-repo-fallback-'));
-    const repoPath = path.join(dataDir, 'real', 'project');
-    fs.mkdirSync(repoPath, { recursive: true });
-    git(repoPath, ['init', '-q', '-b', 'main']);
-    git(repoPath, ['config', 'user.email', 'legacy-fc@example.local']);
-    git(repoPath, ['config', 'user.name', 'Legacy FC']);
-    fs.writeFileSync(path.join(repoPath, 'README.md'), '# legacy\n');
-    git(repoPath, ['add', 'README.md']);
-    git(repoPath, ['commit', '-q', '-m', 'initial']);
-    const initialSha = git(repoPath, ['rev-parse', 'HEAD']);
-    const kbPath = path.join(dataDir, 'knowledge', 'legacy-team');
-    fs.mkdirSync(kbPath, { recursive: true });
-    fs.writeFileSync(path.join(dataDir, 'projects.json'), JSON.stringify({
-      'legacy-team': {
-        schema: 'kb-project/v1',
-        displayName: 'Legacy Team',
-        localPath: repoPath,
-        gitPath: repoPath,
-        kbPath,
-        trackingStartCommit: initialSha,
-        lastAnalyzedCommit: initialSha,
-        enabled: true,
-        aiProfileId: 'primary',
-        knowledgeLanguage: 'zh-CN',
-      },
-    }, null, 2));
-    fs.writeFileSync(path.join(dataDir, 'knowledge-store.json'), JSON.stringify({
-      schema: 'knowledge-store/v1',
-      rootPath: path.join(dataDir, 'knowledge'),
-      configured: true,
-    }));
-    fs.writeFileSync(path.join(dataDir, 'ai-profiles.json'), JSON.stringify({
-      schema: 'ai-profiles/v1',
-      profiles: [{ id: 'primary', name: 'Primary', enabled: true, vendor: 'anthropic', model: 'claude-sonnet' }],
-      defaultProfileId: 'primary',
-    }));
-    const layout = new StorageLayout({ dataDir });
-    const migration = new MigrationService({ layout, legacyDataDir: dataDir });
-    const result = await migration.migrateIfNeeded({ migrationRunId: 'fc-no-fallback' });
-    assert(result.projectMap && result.projectMap['legacy-team'], 'migration must produce projectMap');
-    const projectId = result.projectMap['legacy-team'];
-    const config = JSON.parse(fs.readFileSync(layout.getProjectConfigPath(projectId), 'utf8'));
-    assert.strictEqual(config.repoPath, path.resolve(repoPath), 'repoPath must NOT be silently rewritten to a data-dir fallback');
-    fs.rmSync(dataDir, { recursive: true, force: true });
-  }
-
-  console.log('legacy-forward-compat-test PASS');
-}
-
-run().catch(error => {
-  console.error(error.stack || error.message);
-  process.exit(1);
-});