Merge pull request #38 from Activer007/codex/p0-startup-decoupling

# Conflicts:
#	src/main/libs/externalAgentEnvironment.ts

Author: freestylefly

.github/workflows/labeler.yml

@@ -1,7 +1,7 @@
 name: Labeler
 
 on:
-  pull_request:
+  pull_request_target:
     types: [opened, synchronize, reopened]
 
 env:
@@ -12,9 +12,9 @@ jobs:
     runs-on: ubuntu-latest
     permissions:
       contents: read
+      issues: write
       pull-requests: write
     steps:
-      - uses: actions/checkout@v4
       - uses: actions/labeler@v5
         with:
           configuration-path: .github/labeler.yml

CHANGELOG.md

@@ -0,0 +1,26 @@
+# Changelog
+
+本文档记录 WeSight 的重要变更。
+
+发布说明应从对应版本条目生成。
+
+## 2026.6.2 - 2026-06-03
+
+### 新增
+
+- 新增 `docs/release-process.md`，定义最小发布流程。
+- 新增根目录 `CHANGELOG.md`，作为发布说明来源。
+
+### 变更
+
+- 明确 changelog 条目是 release note 的生成来源。
+- 明确门禁失败时不得发布稳定版；若共享候选版本，必须记录为已知问题或阻断项。
+
+### 修复
+
+- 本次为文档更新，不包含产品修复。
+
+### 已知问题
+
+- 当前基线下 `npm run lint` 尚未通过。
+- 当前基线下 `npm test` 尚未通过。

docs/release-process.md

@@ -0,0 +1,37 @@
+# 发布规范
+
+本文档定义 WeSight 的最小发布流程。
+
+## 发布门禁
+
+发布稳定版前，必须运行并记录结果：
+
+```bash
+npm run build
+npm run lint
+npm test
+```
+
+任一门禁失败时，不得发布稳定版。若仍需要共享候选版本，必须把失败项写入已知问题或阻断项。
+
+## Changelog
+
+每个版本都必须在 `CHANGELOG.md` 中记录：
+
+- 版本
+- 日期
+- 新增
+- 变更
+- 修复
+- 已知问题
+
+GitHub release note 应从 `CHANGELOG.md` 的对应版本条目生成。
+
+## 发布审查
+
+打 tag 前确认：
+
+- 版本号与包元数据一致。
+- 已知问题明确、可行动。
+- 发布说明只描述对开发者或用户有意义的项目级变化。
+- 涉及安全、签名、校验和平台差异时，已有对应检查记录。

src/main/libs/externalAgentEnvironment.test.ts

@@ -112,3 +112,18 @@ test('does not treat WeSight placeholders as local CLI credentials', () => {
 
   expect(result.authStatus).toBe('logged_out');
 });
+
+test('limits probes to requested app types', async () => {
+  writeExecutable('codex', '#!/bin/sh\necho "codex-test 1.0.0"\n');
+
+  const { snapshot, report } = await getExternalAgentEnvironmentSnapshot({ appTypes: ['codex'] });
+
+  expect(snapshot.engines).toHaveLength(1);
+  expect(snapshot.engines[0]).toMatchObject({
+    appType: 'codex',
+    found: true,
+    path: path.join(tempDir, 'codex'),
+    version: 'codex-test 1.0.0',
+  });
+  expect(report.metrics.map(metric => metric.command)).toEqual(['codex']);
+});

src/main/libs/externalAgentEnvironment.ts

@@ -1033,11 +1033,14 @@ const readBaseSnapshotInputs = (): {
   return { appDir, settingsPath, dbPath, settings };
 };
 
-export function getPlaceholderExternalAgentEnvironmentSnapshot(): ExternalAgentEnvironmentSnapshot {
+export function getPlaceholderExternalAgentEnvironmentSnapshot(
+  options: ExternalAgentEnvironmentProbeOptions = {},
+): ExternalAgentEnvironmentSnapshot {
   const { appDir, settingsPath, dbPath, settings } = readBaseSnapshotInputs();
+  const commands = listAgentEngineCommands(options);
   return {
     ccSwitch: buildCcSwitchSnapshot(appDir, settingsPath, dbPath, settings),
-    engines: AGENT_ENGINE_COMMANDS.map(({ engine, appType, command }) => (
+    engines: commands.map(({ engine, appType, command }) => (
       buildPlaceholderCommandStatus(engine, appType, command, settings, dbPath)
     )),
   };

src/main/libs/performanceMetrics.test.ts

@@ -63,6 +63,26 @@ test('keeps the first startup timing value by default', () => {
   expect(getPerformanceSnapshot().startupTimings.first_paint_ms).toBe(120);
 });
 
+test('includes startup and selected engine timings in performance snapshot', () => {
+  resetPerformanceMetricsForTesting();
+
+  markTimingValue('app_ready_ms', 12);
+  markTimingValue('window_created_ms', 45);
+  markTimingValue('first_paint_ms', 90);
+  markTimingValue('first_interactive_ms', 130);
+  markTimingValue('selected_engine_detect_ms', 18);
+  markTimingValue('selected_engine_ready_ms', 18);
+
+  expect(getPerformanceSnapshot().startupTimings).toMatchObject({
+    app_ready_ms: 12,
+    window_created_ms: 45,
+    first_paint_ms: 90,
+    first_interactive_ms: 130,
+    selected_engine_detect_ms: 18,
+    selected_engine_ready_ms: 18,
+  });
+});
+
 test('keeps a bounded slow DB operation ring buffer', () => {
   resetPerformanceMetricsForTesting();
   setDbSlowThresholdForTesting(1);

src/main/main.ts

@@ -1427,6 +1427,23 @@ const resolveCoworkAgentEngine = (): CoworkAgentEngine => {
   return isCoworkAgentEngine(configured) ? configured : CoworkAgentEngineValue.YdCowork;
 };
 
+const ensureSelectedEngineReadyForStartup = async (engine: CoworkAgentEngine): Promise<void> => {
+  if (isOpenClawCoworkAgentEngine(engine)) {
+    const status = await getOpenClawEngineManager().ensureReady();
+    if (status.phase === 'error' || status.phase === 'not_installed') {
+      throw new Error(status.message || 'OpenClaw CLI is not ready.');
+    }
+    return;
+  }
+
+  if (engine === CoworkAgentEngineValue.Hermes) {
+    const status = await getHermesEngineManager().ensureReady();
+    if (status.phase === 'error' || status.phase === 'not_installed') {
+      throw new Error(status.message || 'Hermes Agent CLI is not ready.');
+    }
+  }
+};
+
 type FeishuIMAgentEngine =
   | typeof CoworkAgentEngineValue.OpenClaw
   | typeof CoworkAgentEngineValue.Hermes
@@ -1676,6 +1693,13 @@ const isExternalAgentProviderAppType = (value: unknown): value is ExternalAgentP
   || value === 'deepseek_tui'
 );
 
+const normalizeAgentEngineSnapshotAppTypes = (value: unknown): ExternalAgentProviderAppType[] => {
+  if (!Array.isArray(value)) {
+    return [];
+  }
+  return Array.from(new Set(value.filter(isExternalAgentProviderAppType)));
+};
+
 const AGENT_ENGINE_SNAPSHOT_TTL_MS = 30_000;
 
 interface AgentEngineSnapshotResponse {
@@ -1774,6 +1798,20 @@ const getCachedAgentEngineSnapshot = (options: { forceRefresh?: boolean } = {}):
   };
 };
 
+const getFilteredAgentEngineSnapshot = async (
+  appTypes: ExternalAgentProviderAppType[],
+): Promise<AgentEngineSnapshotResponse> => {
+  const { snapshot, report } = await getExternalAgentEnvironmentSnapshot({ appTypes });
+  const mergedSnapshot = mergeCodexAppStatus(snapshot);
+  summarizeAgentEngineProbeReport(report);
+  return {
+    success: true,
+    snapshot: mergedSnapshot,
+    refreshing: false,
+    cachedAt: Date.now(),
+  };
+};
+
 const getOpenClawConfigSync = (): OpenClawConfigSync => {
   if (!openClawConfigSync) {
     openClawConfigSync = new OpenClawConfigSync({
@@ -5110,8 +5148,12 @@ if (!gotTheLock) {
     }
   });
 
-  ipcMain.handle('cowork:agentEngines:list', async (_event, input: { forceRefresh?: unknown } = {}) => {
+  ipcMain.handle('cowork:agentEngines:list', async (_event, input: { forceRefresh?: unknown; appTypes?: unknown } = {}) => {
     try {
+      const appTypes = normalizeAgentEngineSnapshotAppTypes(input?.appTypes);
+      if (appTypes.length > 0) {
+        return await getFilteredAgentEngineSnapshot(appTypes);
+      }
       return getCachedAgentEngineSnapshot({ forceRefresh: input?.forceRefresh === true });
     } catch (error) {
       return {
@@ -7677,6 +7719,7 @@ if (!gotTheLock) {
     | 'enterprise_sync'
     | 'runtime_forwarders'
     | 'openclaw_config_sync'
+    | 'openclaw_proxy_config_sync'
     | 'hermes_config_sync'
     | 'selected_engine'
     | 'scheduled_tasks'
@@ -7704,6 +7747,7 @@ if (!gotTheLock) {
     'enterprise_sync',
     'runtime_forwarders',
     'openclaw_config_sync',
+    'openclaw_proxy_config_sync',
     'hermes_config_sync',
     'selected_engine',
     'scheduled_tasks',
@@ -8060,12 +8104,13 @@ if (!gotTheLock) {
     }, { degradedOnError: true });
     await runStartupService('selected_engine', async () => {
       const selectedEngineDetectStartedAt = nowMs();
-      const selectedEngine = resolveCoworkAgentEngine();
-      markTiming('selected_engine_detect_ms', selectedEngineDetectStartedAt);
-      if (isOpenClawCoworkAgentEngine(selectedEngine)) {
-        await ensureOpenClawRunningForCowork();
+      try {
+        const selectedEngine = resolveCoworkAgentEngine();
+        await ensureSelectedEngineReadyForStartup(selectedEngine);
+      } finally {
+        markTiming('selected_engine_detect_ms', selectedEngineDetectStartedAt);
+        markTiming('selected_engine_ready_ms', selectedEngineDetectStartedAt);
       }
-      markTiming('selected_engine_ready_ms', selectedEngineDetectStartedAt);
     }, { degradedOnError: true });
 
     await runStartupService('scheduled_tasks', () => {
@@ -8128,14 +8173,16 @@ if (!gotTheLock) {
 
     // Re-sync OpenClaw config after proxy is ready so that providers that route
     // through the proxy (e.g. github-copilot) get the correct baseUrl.
-    if (isOpenClawCoworkAgentEngine(resolveCoworkAgentEngine())) {
-      const proxyResync = await syncOpenClawConfig({
-        reason: 'proxy-ready',
-      });
-      if (proxyResync.changed) {
-        console.log('[Main] OpenClaw config updated after proxy ready, gateway will restart to pick up new config');
+    await runStartupService('openclaw_proxy_config_sync', async () => {
+      if (isOpenClawCoworkAgentEngine(resolveCoworkAgentEngine())) {
+        const proxyResync = await syncOpenClawConfig({
+          reason: 'proxy-ready',
+        });
+        if (proxyResync.changed) {
+          console.log('[Main] OpenClaw config updated after proxy ready, gateway will restart to pick up new config');
+        }
       }
-    }
+    }, { degradedOnError: true });
 
     // Auto-reconnect IM bots that were enabled before restart.
     await runStartupService('im_gateways', async () => {

src/main/preload.ts

@@ -351,7 +351,10 @@ contextBridge.exposeInMainWorld('electron', {
       memoryUserMemoriesMaxItems?: number;
     }) =>
       ipcRenderer.invoke('cowork:config:set', config),
-    listAgentEngines: (input?: { forceRefresh?: boolean }) =>
+    listAgentEngines: (input?: {
+      forceRefresh?: boolean;
+      appTypes?: Array<'claude' | 'codex' | 'hermes' | 'openclaw' | 'opencode' | 'grok' | 'qwen' | 'deepseek_tui'>;
+    }) =>
       ipcRenderer.invoke('cowork:agentEngines:list', input),
     getRuntimeMetricsSummary: (filters: any) =>
       ipcRenderer.invoke(CoworkIpcChannel.RuntimeMetricsSummary, filters),