test(dependency): add boundary and contract tests for dependency refactoring

Add comprehensive tests to verify the dependency boundary refactoring
and ensure IPC contracts are properly maintained.

Changes:
- add version-management-dependency-boundary.test.ts for main process boundary tests
- add version-bridge.contract.test.ts for preload bridge contract tests
- update versionManagement.installProgress.render.test.ts with UI removal assertions

Co-Authored-By: Hagicode <noreply@hagicode.com>
Signed-off-by: newbe36524 <newbe36524@qq.com>

Author: newbe36524

src/main/__tests__/version-management-dependency-boundary.test.ts

@@ -0,0 +1,47 @@
+import assert from 'node:assert/strict';
+import fs from 'node:fs/promises';
+import path from 'node:path';
+import { describe, it } from 'node:test';
+
+const versionHandlersPath = path.resolve(process.cwd(), 'src/main/ipc/handlers/versionHandlers.ts');
+const dependencyHandlersPath = path.resolve(process.cwd(), 'src/main/ipc/handlers/dependencyHandlers.ts');
+const mainPath = path.resolve(process.cwd(), 'src/main/main.ts');
+const versionManagerPath = path.resolve(process.cwd(), 'src/main/version-manager.ts');
+const dependencyManagerPath = path.resolve(process.cwd(), 'src/main/dependency-manager.ts');
+
+describe('version management dependency boundary', () => {
+  it('removes the retired version dependency IPC channel while keeping lifecycle handlers', async () => {
+    const [versionHandlersSource, mainSource] = await Promise.all([
+      fs.readFile(versionHandlersPath, 'utf8'),
+      fs.readFile(mainPath, 'utf8'),
+    ]);
+
+    assert.equal(versionHandlersSource.includes('version:checkDependencies'), false);
+    assert.equal(mainSource.includes('version:checkDependencies'), false);
+    assert.match(versionHandlersSource, /ipcMain\.handle\('version:list'/);
+    assert.match(versionHandlersSource, /ipcMain\.handle\('version:getInstalled'/);
+    assert.match(versionHandlersSource, /ipcMain\.handle\('version:getActive'/);
+    assert.match(versionHandlersSource, /ipcMain\.handle\('version:install'/);
+    assert.match(versionHandlersSource, /ipcMain\.handle\('version:uninstall'/);
+    assert.match(versionHandlersSource, /ipcMain\.handle\('version:reinstall'/);
+    assert.match(versionHandlersSource, /ipcMain\.handle\('version:switch'/);
+    assert.match(versionHandlersSource, /ipcMain\.handle\('version:openLogs'/);
+  });
+
+  it('moves manifest dependency list loading onto the dependency manager boundary', async () => {
+    const [dependencyHandlersSource, versionManagerSource, dependencyManagerSource, mainSource] = await Promise.all([
+      fs.readFile(dependencyHandlersPath, 'utf8'),
+      fs.readFile(versionManagerPath, 'utf8'),
+      fs.readFile(dependencyManagerPath, 'utf8'),
+      fs.readFile(mainPath, 'utf8'),
+    ]);
+
+    assert.match(versionManagerSource, /async resolveVersionInstallPath\(versionId: string\): Promise<string \| null>/);
+    assert.equal(versionManagerSource.includes('async getDependencyListFromManifest('), false);
+    assert.match(dependencyManagerSource, /async getDependencyListFromManifest\(installPath: string\): Promise<DependencyCheckResult\[]>/);
+    assert.match(dependencyHandlersSource, /resolveVersionInstallPath\(versionId\)/);
+    assert.match(dependencyHandlersSource, /getDependencyListFromManifest\(installPath\)/);
+    assert.match(mainSource, /resolveVersionInstallPath\(versionId\)/);
+    assert.match(mainSource, /getDependencyListFromManifest\(installPath\)/);
+  });
+});

src/preload/__tests__/version-bridge.contract.test.ts

@@ -0,0 +1,22 @@
+import assert from 'node:assert/strict';
+import fs from 'node:fs/promises';
+import path from 'node:path';
+import { describe, it } from 'node:test';
+
+const preloadPath = path.resolve(process.cwd(), 'src/preload/index.ts');
+
+describe('version preload bridge contract', () => {
+  it('removes the retired dependency inspection bridge while keeping lifecycle bridges', async () => {
+    const source = await fs.readFile(preloadPath, 'utf8');
+
+    assert.equal(source.includes('versionCheckDependencies'), false);
+    assert.equal(source.includes('version:checkDependencies'), false);
+    assert.match(source, /versionList: \(\) => ipcRenderer\.invoke\('version:list'\)/);
+    assert.match(source, /versionGetInstalled: \(\) => ipcRenderer\.invoke\('version:getInstalled'\)/);
+    assert.match(source, /versionGetActive: \(\) => ipcRenderer\.invoke\('version:getActive'\)/);
+    assert.match(source, /versionInstall: \(versionId\) => ipcRenderer\.invoke\('version:install', versionId\)/);
+    assert.match(source, /versionReinstall: \(versionId\) => ipcRenderer\.invoke\('version:reinstall', versionId\)/);
+    assert.match(source, /versionSwitch: \(versionId\) => ipcRenderer\.invoke\('version:switch', versionId\)/);
+    assert.match(source, /versionOpenLogs: \(versionId\) => ipcRenderer\.invoke\('version:openLogs', versionId\)/);
+  });
+});

src/renderer/store/slices/__tests__/versionManagement.installProgress.render.test.ts

@@ -12,4 +12,16 @@ describe('version management install progress rendering', () => {
     assert.match(source, /installingVersionId === version\.id/);
     assert.match(source, /renderInstallTelemetry\(version\.id\)/);
   });
+
+  it('removes dependency affordances from installed version cards', async () => {
+    const source = await fs.readFile(pagePath, 'utf8');
+
+    assert.equal(source.includes('VersionDependencyGuidance'), false);
+    assert.equal(source.includes('handleToggleDependencies'), false);
+    assert.equal(source.includes('getDependencyList('), false);
+    assert.equal(source.includes('viewDependencies'), false);
+    assert.equal(source.includes('collapseDependencies'), false);
+    assert.equal(source.includes('dependencyInfo'), false);
+    assert.equal(source.includes('aiGuidance'), false);
+  });
 });