chore(openspec): sync & archive fix-session-not-found-on-load, fix-session-time-tracking,

   harness-event-bus, logging-system

   Sync delta specs from 4 changes into main spec tree:
   - harness-event-bus: new event bus architecture — 3 new capabilities (harness-event-bus,
   ui-backend-slim, logging-system), 7 modified specs
   - fix-session-not-found-on-load: handleSession fallback + rebuildIndex empty-session fix
   (session-management)
   - logging-system: file-based Logger, phase progress logging (chiff-progress-display,
   eval-progress-feedback)
   - fix-session-time-tracking: no delta specs

   Archive: 2026-01-16-{fix-session-not-found-on-load, fix-session-time-tracking, harness-event-bus,
   logging-system}

Author: 王璨

openspec/changes/archive/2026-01-16-fix-session-not-found-on-load/.openspec.yaml

@@ -0,0 +1,2 @@
+schema: spec-driven
+created: 2026-06-16

openspec/changes/archive/2026-01-16-fix-session-not-found-on-load/design.md

@@ -0,0 +1,43 @@
+## Context
+
+当前架构中，`SessionManager.listSessions()` 按设计过滤 `messageCount === 0` 的 session。`WebUiBackend.pushSessionList()` 在发送 session 列表给客户端时，会通过 `getCurrentMetadata()` 将当前活跃 session 补回列表，即使它的 messageCount 为 0。但 `handleSession` 的 `"load"` 分支直接使用 `listSessions()` 查询目标 session，没有相同的 fallback，导致用户无法点击加载一个刚创建的空 session。
+
+此外，`SessionStore.rebuildIndex()` 在重建索引时跳过 `messages.length === 0` 的 session 文件，与 `updateIndex()` 行为不一致，会在索引损坏恢复时丢失合法存在的空 session。
+
+## Goals / Non-Goals
+
+**Goals:**
+- 用户点击新建的空 session 时能正常切换（不会报 "Session not found"）
+- `rebuildIndex` 与 `updateIndex` 行为一致，不丢弃合法存在的空 session
+
+**Non-Goals:**
+- 不改变 `listSessions()` 的过滤语义（0-msg session 仍不出现于常规列表中）
+- 不改变 session 创建、持久化、删除的现有流程
+- 不改变前端侧任何行为
+
+## Decisions
+
+### Decision 1: 在 `handleSession` load 分支添加 fallback，而非修改 `listSessions()`
+
+**选择**: 在 `WebUiBackend.handleSession` 的 `"load"` case 中，当 `listSessions()` 找不到匹配项时，fallback 检查 `getCurrentMetadata()`。
+
+**理由**:
+- `listSessions()` 过滤 0-msg 是 spec 明确定义的行为，广泛用于侧边栏列表渲染等场景，不应改动
+- 与 `pushSessionList` 已有的补救逻辑保持一致，模式统一
+- 改动最小，仅影响一个 handler 分支
+
+**备选方案**: 修改 `listSessions()` 不过滤 0-msg，改为在 UI 层过滤 → 影响面大，需要追踪所有 `listSessions()` 调用点并确认每个调用点的预期行为。
+
+### Decision 2: 移除 `rebuildIndex` 中的 empty-messages 跳过逻辑
+
+**选择**: `rebuildIndex` 不应跳过 `messages.length === 0` 的 session 文件，仅跳过解析失败的损坏文件。
+
+**理由**:
+- `updateIndex`（通过 `save()` → `updateIndex()`）会正常写入 0-msg session，它们在磁盘上合法存在
+- 如果 index.json 损坏触发 `rebuildIndex`，不应丢失这些 session
+- 过滤逻辑应由 `SessionManager.listSessions()` 在应用层处理，而非索引层
+
+## Risks / Trade-offs
+
+- **[低风险] 索引包含更多条目**: 0-msg session 出现在索引中，但 `listSessions()` 仍会过滤它们，实际影响仅体现在 `rebuildIndex` 的场景中，该场景本身很少触发
+- **[无风险] Fallback 逻辑**: 仅当 `listSessions()` 匹配数为 0 时触发，且仅匹配当前活跃 session。若同时满足 "当前 session 不在列表中" 和 "请求 id 不匹配当前 session"，则仍返回 "not found"，行为正确

openspec/changes/archive/2026-01-16-fix-session-not-found-on-load/proposal.md

@@ -0,0 +1,22 @@
+## Why
+
+新建 session 后（messageCount=0），点击左侧 session 列表切换回该 session 时，服务端 `handleSession` 的 `load` 分支直接使用 `listSessions()` 查询 session，而该方法按设计过滤掉 `messageCount===0` 的 session。`pushSessionList` 已有将当前 session 补回列表的补救逻辑，但 `load` 分支缺少相同处理，导致返回 "Session not found" 错误。
+
+## What Changes
+
+- 在 `WebUiBackend.handleSession` 的 `"load"` 分支中，当 `listSessions()` 找不到匹配 session 时，增加 fallback：检查请求的 id 是否匹配当前活跃 session（`getCurrentMetadata()`），若匹配则直接使用当前 session 的 metadata 继续加载流程。
+- 在 `SessionStore.rebuildIndex` 中，移除对 `messages.length === 0` 的过滤，保持索引与实际磁盘文件一致（0-msg session 既然被 persistEmptySession 写入磁盘，就应在索引中可见）。
+
+## Capabilities
+
+### New Capabilities
+<!-- No new capabilities — this is a bug fix on existing behavior. -->
+
+### Modified Capabilities
+- `session-management`: `handleSession` load 分支的行为现在与 `pushSessionList` 一致——当 `listSessions()` 因 messageCount=0 过滤掉当前 session 时，fallback 到当前 session metadata。`rebuildIndex` 不再跳过 messages 为空的 session 文件，保持索引完整性。
+
+## Impact
+
+- `src/ui/web/web-backend.ts` — `handleSession` "load" case，增加 fallback 逻辑
+- `src/session/store.ts` — `rebuildIndex`，移除 empty-messages 跳过逻辑
+- `src/session/manager.ts` — `listSessions()` 行为不变（spec 已明确其过滤语义）

openspec/changes/archive/2026-01-16-fix-session-not-found-on-load/specs/session-management/spec.md

@@ -0,0 +1,47 @@
+## ADDED Requirements
+
+### Requirement: handleSession load fallback to current session
+
+The `WebUiBackend.handleSession` `"load"` handler SHALL, when `listSessions()` returns zero matches for the requested session ID, check whether the request ID matches the current active session via `sessionManager.getCurrentMetadata()`. If the current session metadata exists and its `id` starts with the requested ID prefix, the handler SHALL proceed with loading the current session instead of returning a "Session not found" error.
+
+This ensures consistency with `pushSessionList()`, which already includes the current session in the client-facing list even when its `messageCount` is 0.
+
+#### Scenario: Load current empty session via sidebar click
+
+- **WHEN** the client sends `{ type: "session", action: "load", id: "<currentSessionId>" }` and the current session has `messageCount === 0` (thus excluded from `listSessions()` output)
+- **THEN** the handler finds no match in `listSessions()` but detects that `getCurrentMetadata()?.id` starts with the requested ID
+- **AND** proceeds with the normal load flow (abort → save → loadSession → clear_conversation → ready)
+- **AND** does NOT return "Session not found" error
+
+#### Scenario: Load non-existent session still returns error
+
+- **WHEN** the client sends `{ type: "session", action: "load", id: "NONEXIST" }` and no session with that ID exists (neither in `listSessions()` nor as current session)
+- **THEN** the handler returns `{ type: "error", text: "Session not found: NONEXIST" }`
+
+#### Scenario: Ambiguous prefix match still returns error
+
+- **WHEN** the client sends `{ type: "session", action: "load", id: "00" }` and `listSessions()` matches more than one session with that prefix
+- **THEN** the handler returns `{ type: "error", text: "Ambiguous session ID prefix. ..." }` without checking the current session fallback
+
+### Requirement: rebuildIndex preserves empty-message sessions
+
+`SessionStore.rebuildIndex()` SHALL include session files with `messages.length === 0` in the rebuilt index, as long as the file contains valid `metadata` with a valid `id` field. Only files that fail to parse as valid JSON or lack valid `metadata.id` SHALL be skipped.
+
+The existing fix-up for `messageCount === 0 && messages.length > 0` SHALL remain unchanged.
+
+#### Scenario: Empty session preserved in rebuilt index
+
+- **WHEN** `rebuildIndex()` scans a directory containing a valid session file with `messages: []` and valid metadata
+- **THEN** that session SHALL appear in the rebuilt index with its original metadata
+
+#### Scenario: Corrupted session file still skipped
+
+- **WHEN** `rebuildIndex()` scans a directory containing a file with invalid JSON or missing `metadata.id`
+- **THEN** that file SHALL be silently skipped
+
+## MODIFIED Requirements
+
+### Requirement: Current session preserved in list when empty
+
+- **WHEN** the current active session has `messageCount === 0`
+- **THEN** that session is excluded from `listSessions()` output; the `pushSessionList` server method adds it back when sending to the client, using `currentSessionId` to identify it; the `handleSession` `"load"` handler also falls back to `getCurrentMetadata()` when `listSessions()` returns no matches

openspec/changes/archive/2026-01-16-fix-session-not-found-on-load/tasks.md

@@ -0,0 +1,14 @@
+## 1. Fix `handleSession` load handler
+
+- [x] 1.1 In `src/ui/web/web-backend.ts`, in `handleSession` `"load"` case, after line `if (matches.length === 0)` and before returning "Session not found" error, add fallback: check if `sessionManager.getCurrentMetadata()?.id.startsWith(cmd.id!)`. If so, set `match` to the current metadata and proceed with load flow.
+- [x] 1.2 Verify via unit or manual test: create a new session, click it in sidebar — should load successfully without "Session not found" error.
+
+## 2. Fix `rebuildIndex` empty-message filtering
+
+- [x] 2.1 In `src/session/store.ts` `rebuildIndex()`, remove the condition `Array.isArray(raw.messages) && raw.messages.length > 0` that guards pushing to `entries`. Push as long as `raw?.metadata && typeof raw.metadata.id === "string"`.
+- [x] 2.2 Verify via unit or manual test: delete `index.json`, restart — 0-message sessions should still appear in the index (though filtered from the sidebar by `listSessions()`).
+
+## 3. Validation
+
+- [x] 3.1 Run `npm test` to ensure existing tests pass.
+- [x] 3.2 Run `npm run typecheck` to ensure no type errors.

openspec/changes/archive/2026-01-16-fix-session-time-tracking/design.md

@@ -0,0 +1,248 @@
+## 设计
+
+### 现状问题
+
+```
+当前流程 (有 bug):
+
+createSession / loadSession
+    │
+    │  activeSince = Date.now()  ← 从此刻开始计时
+    │
+    ▼
+  [用户阅读历史, 思考, 打字...]  ← 空闲时间也被计入!
+    │
+    ▼
+  agent.prompt() → agent_start → ... processing ... → agent_end
+    │
+    ▼
+  saveSession:
+    accumulatedMs += Date.now() - activeSince  ← 包含了空闲时间
+```
+
+### 修复后流程（后端 — 已完成）
+
+```
+修复后:
+
+createSession / loadSession
+    │
+    │  activeSince = null  ← 不自动启动
+    │
+    ▼
+  [用户阅读历史, 思考, 打字...]  ← 不计入
+    │
+    ▼
+  agent_start
+    │
+    │  activeSince = Date.now()  ← 仅此时启动
+    │
+    ▼
+  [... agent 处理中 ...]
+    │
+    ▼
+  agent_end
+    │
+    │  accumulatedMs += Date.now() - activeSince
+    │  activeSince = null  ← 暂停
+    │
+    ▼
+  saveSession:
+    if (activeSince !== null) { ... }  ← null 时不累积
+    totalActiveMs = accumulatedMs
+```
+
+### SessionManager 新增 API
+
+```typescript
+// 启动计时（agent 开始处理时调用）
+startActiveTimer(): void {
+    if (this.activeSince === null) {
+        this.activeSince = Date.now();
+    }
+}
+
+// 暂停计时并累积（agent 结束处理时调用）
+stopActiveTimer(): void {
+    if (this.activeSince !== null) {
+        this.accumulatedMs += Date.now() - this.activeSince;
+        this.activeSince = null;
+        if (this.current) {
+            this.current.totalActiveMs = this.accumulatedMs;
+        }
+    }
+}
+```
+
+### Harness 接入点
+
+在 `src/core/harness.ts` 的 agent 事件订阅中：
+
+```typescript
+// agent_start (约 line 1095):
+if (event.type === "agent_start") {
+    this.sessionManager.startActiveTimer();
+    this.ui.startAssistantMessage();
+}
+
+// agent_end (约 line 1089):
+if (event.type === "agent_end") {
+    this.sessionManager.stopActiveTimer();
+    this.ui.setProcessing(false);
+    this.sessionManager.trySaveSession(this.agent);
+}
+```
+
+### SessionManager 现有改动
+
+1. `createSession()`: 移除 `this.activeSince = Date.now()`（两处：line 97 附近和 line 115 附近）
+2. `loadSession()`: 移除 `this.activeSince = Date.now()`（line 239），保留 `this.accumulatedMs = ...`
+
+---
+
+### 前端修复：方案 B — 后端推送统一时钟源
+
+#### 问题分析
+
+```
+当前前端数据流：
+
+  backend: sessions 事件（仅 agent_end 后发一次）
+           │
+           ▼
+  App.tsx: sessionActiveMs = sessions.find(...).totalActiveMs  ← 冻结值
+           │                              │
+           ▼                              ▼
+       Sidebar                        ChatView
+  formatDuration(              sessionTime/1000 + elapsed
+    s.totalActiveMs)           ↑                     ↑
+                               │                     │
+                          冻结，不走表           rAF 每帧+1，走表
+                                                 but double-count!
+
+  结果：
+  - Sidebar: frozen (processing 期间不走)
+  - ChatView: ≈ 2x real time (double-count)
+  - 差距 = processing 时长，线性扩大
+```
+
+#### 修复架构
+
+```
+修复后前端数据流：
+
+  backend: 1s interval ──► session_time { totalActiveMs } ──► agent_end 时停止
+                               │
+                               ▼
+  App.tsx: 更新 sessions state 中当前 session 的 totalActiveMs
+           │                              │
+           ▼                              ▼
+       Sidebar                        ChatView
+  formatDuration(              formatTime(
+    s.totalActiveMs)              sessionTime / 1000)
+       │                              │
+       └──────── 同一个值 ─────────────┘
+               永远一致，实时走表
+```
+
+#### Protocol 新增
+
+```typescript
+// src/ui/shared/types.ts ServerEvent union 新增:
+| { type: "session_time"; totalActiveMs: number }
+```
+
+#### WebUiBackend 改动
+
+```typescript
+// WebUiBackend 新增字段
+private sessionTimeInterval: ReturnType<typeof setInterval> | null = null;
+
+startAssistantMessage(): void {
+  this.currentAssistant = { thinking: "", text: "", tools: [] };
+  this.broadcast({ type: "assistant_start" });
+  // ★ 启动 session_time 定时广播
+  this.startSessionTimeBroadcast();
+}
+
+finishAssistantMessage(): void {
+  // ★ 先发最后一次，再停止
+  this.broadcastSessionTime();
+  this.stopSessionTimeBroadcast();
+  this.currentAssistant = null;
+  this.broadcast({ type: "assistant_end" });
+  // ... 现有 sessions 广播 ...
+}
+
+private startSessionTimeBroadcast(): void {
+  if (this.sessionTimeInterval) return;
+  this.sessionTimeInterval = setInterval(() => {
+    const sm = this.harness.sessionManager;
+    if (!sm) return;
+    this.wsServer.broadcast({
+      type: "session_time",
+      totalActiveMs: sm.getTotalActiveMs(),
+    });
+  }, 1000);
+}
+
+private stopSessionTimeBroadcast(): void {
+  if (this.sessionTimeInterval) {
+    clearInterval(this.sessionTimeInterval);
+    this.sessionTimeInterval = null;
+  }
+}
+
+private broadcastSessionTime(): void {
+  const sm = this.harness.sessionManager;
+  if (!sm) return;
+  this.wsServer.broadcast({
+    type: "session_time",
+    totalActiveMs: sm.getTotalActiveMs(),
+  });
+}
+```
+
+#### App.tsx 改动
+
+```typescript
+case "session_time":
+  setSessions((prev) => prev.map((s) =>
+    s.id === currentSessionId
+      ? { ...s, totalActiveMs: event.totalActiveMs }
+      : s
+  ));
+  break;
+```
+
+#### ChatView 改动
+
+```typescript
+// 删除 elapsed state 和 rAF effect:
+// ❌ const [elapsed, setElapsed] = useState(0);
+// ❌ useEffect(() => { ... rAF tick ... }, [turnStartRef]);
+
+// WaitingBubble: 去掉 + elapsed
+// 旧: ({formatTime(Math.floor(sessionTime / 1000) + elapsed)})
+// 新: ({formatTime(Math.floor(sessionTime / 1000))})
+
+// ThinkingBlock: 同上
+// 旧: `Thinking... (${formatTime(Math.floor(sessionTime / 1000) + elapsed)})`
+// 新: `Thinking... (${formatTime(Math.floor(sessionTime / 1000))})`
+
+// turnStartRef prop 可保留但不再在 ChatView 内使用
+// （仍由 App.tsx 管理，供 future use 或其他组件）
+```
+
+### 边界情况处理
+
+| 场景 | 行为 |
+|------|------|
+| 加载 session 后直接退出 | `activeSince = null`，不累积，totalActiveMs 不变 |
+| agent 处理中被 abort | `agent_end` 正常触发 → `stopActiveTimer()` 累积已处理时间；interval 在 `finishAssistantMessage` 中停止 |
+| 连续多个 turn | 每个 turn 的 `agent_start`/`agent_end` 独立计时；interval 随 turn 启停 |
+| `saveSession` 在 `activeSince = null` 时被调用 | 跳过 accumulation，仅保存当前 accumulatedMs |
+| `getTotalActiveMs()` 在 `activeSince = null` 时被调用 | 返回 accumulatedMs（不含 live 部分） |
+| 已有 session 的 totalActiveMs 包含历史误差 | 不回溯修正，新逻辑从加载后开始生效 |
+| WebSocket 断开期间 | interval 继续在后端运行但 broadcast 无接收者；重连后 `pushSessionList` 发送最新值 |
+| session_time 事件乱序到达 | 后端按序发送，前端幂等处理（总是更新当前 session） |