@liguoshuai/pi-web-chat 2.14.0 → 2.14.5

package/docs/ARCHITECTURE.md

@@ -1,6 +1,6 @@
 # 架构设计文档
 
-> pi-web-chat 的技术架构、数据流、关键设计决策与扩展点说明（对应 v2.12.7 版本）。
+> pi-web-chat 的技术架构、数据流、关键设计决策与扩展点说明（对应 v2.14.5 版本）。
 
 ---
 

package/docs/CHANGELOG.md

@@ -7,6 +7,28 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+## [2.14.5] - 2026-09-05
+
+### Fixed
+- 修复 CI `publish-npm` 与 `publish-release` 的依赖判定：将 `!cancelled()` 改为 `success()`，测试失败时不会发布 NPM 包与 GitHub Release。
+- 修正 README、各客户端 README、UED 原型、架构文档中残留的旧版本号（2.12.7 / 2.12.9 / 2.13.0），并同步全仓版本至 2.14.5。
+
+### Docs
+- 精简 `CLAUDE.md`，明确以 `AGENTS.md` 为唯一权威来源，补齐铁律四（CI 绿勾闭环）。
+
+## [2.14.4] - 2026-09-05
+
+### Performance
+- **添加 HTTP gzip 压缩中间件**：使用 Node.js 内置 `zlib` 实现（零外部依赖），对 `app.js` (123KB→~30KB)、`style.css` (46KB→~10KB) 等文本资源自动 gzip，传输体积减少约 70%。
+- **添加静态资源 Cache-Control 头**：常规资源缓存 1 天，带哈希文件名的资源缓存 1 年（immutable），减少浏览器重复拉取。
+
+### Refactor
+- **提取路径遍历安全校验为 `validateSessionFile()` helper**：`routes.js` 中 GET /api/session 和 DELETE /api/session 的重复安全校验逻辑（~30 行 × 2）合并为一处，降低维护风险。
+- **提取环境变量数值解析为 `parseEnvNum()` helper**：`config.js` 中 4 处重复的 IIFE 模式合并为一个带 `{ integer }` 选项的 helper 函数。
+
+### Fixed
+- 修复 `agent.js` `saveTimingData()` 中注释与代码不一致：注释写 "Keep last 200 turns" 但实际保留 1000 条。
+
 ## [2.14.0] - 2026-09-04
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@liguoshuai/pi-web-chat",
-  "version": "2.14.0",
+  "version": "2.14.5",
   "description": "A ChatGPT/Gemini-style web UI for the pi coding agent, powered by pi's RPC mode.",
   "type": "module",
   "main": "server.js",
@@ -42,8 +42,8 @@
   "dependencies": {
     "express": "^4.21.2",
     "ws": "^8.18.0",
-    "@liguoshuai/pi-chat-protocol": "2.14.0",
-    "@liguoshuai/pi-chat-server": "2.14.0"
+    "@liguoshuai/pi-chat-protocol": "2.14.5",
+    "@liguoshuai/pi-chat-server": "2.14.5"
   },
   "scripts": {
     "build": "node --check server.js && node --check bin/pi-web-chat.js && node --check public/app.js",

package/public/app.js

@@ -604,7 +604,8 @@ function stopStreamingTimer() {
 function sameSession(a, b) {
   if (!a || !b) return false;
   if (a === b) return true;
-  return a.replace(/^[.~]\//, "") === b.replace(/^[.~]\//, "");
+  const norm = (s) => s.replace(/^(?:\.\.?|~)\//, "").replace(/^~$/, "").replace(/\/+$/, "");
+  return norm(a) === norm(b);
 }
 
 // ---- Sidebar / sessions ----
@@ -771,7 +772,7 @@ async function deleteSession(file, title) {
     }
     showToast("会话已删除");
     if (state.currentSessionFile === file) {
-      startNewSession(false);
+      startNewSession();
     } else {
       await refreshSessions();
     }
@@ -831,13 +832,13 @@ async function syncSessionHistory(file, force = false) {
       renderModelPill();
     }
 
-    const topName = data.sessionName || (data.header?.id ? baseName(file) : "新对话");
+    const topName = data.sessionName || data.firstUser || "新对话";
     $("#topSessionName").textContent = topName;
 
     // Only overwrite chat if not actively backfilling
     if (!state.isBackfilling && (force || !state.streaming)) {
       clearChat();
-      const msgs = reconstructFromEntries(data.entries || []);
+      const msgs = reconstructFromEntries(data.entries || [], data.timing || null);
       showEmptyState(msgs.length === 0);
       for (const m of msgs) {
         appendMessageNode(m.role, m);
@@ -883,7 +884,7 @@ function parseEntryTimestamp(ts) {
   return isNaN(parsed) ? null : parsed;
 }
 
-function reconstructFromEntries(entries) {
+function reconstructFromEntries(entries, timing = null) {
   // Map toolResults by toolCallId so we can attach them to their toolCall in the assistant message
   const toolResults = new Map();
   for (const e of entries) {
@@ -900,6 +901,7 @@ function reconstructFromEntries(entries) {
 
   const out = [];
   let lastUserTs = null;
+  let assistantMsgCount = 0;
   for (const e of entries) {
     if (e.type !== "message") continue;
     const m = e.message;
@@ -921,18 +923,35 @@ function reconstructFromEntries(entries) {
       out.push({ role: "user", text: extractContentText(m.content), images, ts: msgTs });
     } else if (m.role === "assistant") {
       let turnDurationMs = null;
-      if (lastUserTs && msgTs && msgTs >= lastUserTs) {
+      // Try timing data first, then fall back to timestamp heuristic
+      const turnTiming = timing ? timing[assistantMsgCount] : null;
+      if (turnTiming?.turnDuration != null) {
+        turnDurationMs = turnTiming.turnDuration;
+      } else if (lastUserTs && msgTs && msgTs >= lastUserTs) {
         const diff = msgTs - lastUserTs;
         if (diff > 0 && diff < 15 * 60 * 1000) {
           turnDurationMs = diff;
         }
       }
+      let thinkingIdx = 0;
       const rawContent = Array.isArray(m.content) ? m.content : (m.content ? [{ type: "text", text: String(m.content) }] : []);
       const content = rawContent.map(part => {
+        if (part && part.type === "thinking") {
+          // Apply persisted thinking duration from timing data
+          let thinkingDurationMs = null;
+          if (turnTiming?.thinkingDurations && thinkingIdx < turnTiming.thinkingDurations.length) {
+            thinkingDurationMs = turnTiming.thinkingDurations[thinkingIdx];
+          }
+          thinkingIdx++;
+          return { ...part, durationMs: thinkingDurationMs };
+        }
         if (part && part.type === "toolCall") {
           const res = toolResults.get(part.id);
           let durationMs = null;
-          if (res) {
+          // Try timing data first
+          if (turnTiming?.toolDurations && part.id && turnTiming.toolDurations[part.id] != null) {
+            durationMs = turnTiming.toolDurations[part.id];
+          } else if (res) {
             const startTs = parseEntryTimestamp(part.timestamp || m.timestamp || e.timestamp);
             const endTs = res.entryTs || parseEntryTimestamp(res.timestamp);
             if (startTs && endTs && endTs >= startTs) {
@@ -952,6 +971,7 @@ function reconstructFromEntries(entries) {
         content.push({ type: "text", text: `⚠️ **生成失败**: ${errMsg}` });
       }
       out.push({ role: "assistant", content, ts: msgTs, turnDurationMs, usage: m.usage });
+      assistantMsgCount++;
     }
     // toolResult entries are attached directly to assistant toolCall parts, so they don't produce standalone messages
   }
@@ -1959,9 +1979,8 @@ function handlePiMessage(obj) {
       const newUrl = window.location.pathname + "?session=" + encodeURIComponent(sessionFile);
       window.history.replaceState({ session: sessionFile }, "", newUrl);
     } catch {}
-    if ($("#topSessionName").textContent === "新对话") {
-      $("#topSessionName").textContent = baseName(sessionFile);
-    }
+    // Do not set baseName(sessionFile) as title — it's a session ID, not a user-friendly summary.
+    // The proper title will be set by syncSessionHistory (via firstUser) or by the user prompt.
     refreshSessions();
   }
 
@@ -2365,7 +2384,11 @@ function updateState(d) {
     renderThinkingPill();
   }
   updateEmptyStateModelInfo();
-  $("#topSessionName").textContent = d?.sessionName || (d?.sessionFile ? baseName(d.sessionFile) : "新对话");
+  // Only update top bar title from state if sessionName is explicitly available;
+  // otherwise keep current title (set by syncSessionHistory or user prompt) to avoid showing session file name.
+  if (d?.sessionName) {
+    $("#topSessionName").textContent = d.sessionName;
+  }
 
   // Sync streaming state upon state updates (e.g. after reconnect)
   if (d && typeof d.isStreaming === "boolean") {
@@ -3173,7 +3196,7 @@ async function init() {
   await loadServerConfig();
 
   // event listeners
-  $("#btnNew").addEventListener("click", () => startNewSession(true));
+  $("#btnNew").addEventListener("click", () => startNewSession());
 
   $("#sendBtn").addEventListener("click", () => {
     if (state.streaming) {
@@ -3246,7 +3269,7 @@ async function init() {
 
     if (isCmdOrCtrl && e.shiftKey && (e.key.toLowerCase() === "n" || e.key.toLowerCase() === "o")) {
       e.preventDefault();
-      startNewSession(true);
+      startNewSession();
       return;
     }
     if ((isCmdOrCtrl && e.key.toLowerCase() === "k") || (e.ctrlKey && e.key === "/")) {