fix: review improvements — schema walker, status helper, UI i18n, CI version skip

1. walkSchema: add patternProperties, if/then/else traversal + cycle detection
2. proxy-handler: extract toErrorStatus() helper, centralize 4x StatusCode casts
3. Header StableText: dynamic t() references for i18n + whitespace-nowrap on status button
4. sync-electron.yml: cancel-in-progress: false + merge two pushes into one

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

.github/workflows/sync-electron.yml

@@ -7,7 +7,7 @@ on:
 
 concurrency:
   group: sync-electron
-  cancel-in-progress: true
+  cancel-in-progress: false
 
 permissions:
   actions: write
@@ -104,8 +104,7 @@ jobs:
         if: env.CONFLICT != 'true'
         run: |
           if [ "$BUMP" = "true" ]; then
-            git push origin electron --follow-tags
-            git push origin master
+            git push origin electron master --follow-tags
           else
             git push origin electron
           fi

CHANGELOG.md

@@ -20,6 +20,12 @@
 
 ### Fixed
 
+- JSON Schema `additionalProperties` 递归注入：`injectAdditionalProperties()` 递归注入 `additionalProperties: false` 到 JSON Schema 所有 object 节点，覆盖 `properties`、`patternProperties`、`$defs`/`definitions`、`items`、`prefixItems`、组合器（`oneOf`/`anyOf`/`allOf`）、条件（`if`/`then`/`else`），含循环检测；三个端点（OpenAI/Gemini/Responses passthrough）统一调用 (#64)
+- CONNECT tunnel header 解析：循环跳过中间 header block（CONNECT 200、100 Continue），修复代理模式下 tunnel 的 `HTTP/1.1 200` 被当作真实状态码导致上游 4xx 错误被掩盖为 502 的问题 (#64)
+- 上游 HTTP 状态码透传：非流式 collect 路径从错误消息提取真实 HTTP 状态码，不再硬编码 502；提取 `toErrorStatus()` 辅助函数统一 4 处 StatusCode 转换 (#64)
+- Dashboard 中英文切换按钮宽度跳变：`StableText` 的 `reference` 从英文硬编码改为 `t()` 动态取值，按钮宽度跟随当前语言自适应
+- Dashboard "指纹更新中..." 按钮竖排显示：更新状态按钮添加 `whitespace-nowrap`，防止 CJK 字符逐字换行
+- CI 版本跳号（v1.0.28 → v1.0.30）：`sync-electron.yml` 的 `cancel-in-progress` 改为 `false`，避免 workflow 被取消后 tag 已推送但版本号未同步回 master；合并两次 `git push` 为一次减少部分推送窗口
 - 混合 plan 账号路由失败：free 和 team/plus 账号混用时，请求 plan 受限模型（如 `gpt-5.4`）可能 fallback 到不兼容的 free 账号导致 400 错误，现在严格按 plan 过滤，无匹配账号时返回明确错误而非降级 (#54)
 - `cached_tokens` / `reasoning_tokens` 透传：从 Codex API 响应的 `input_tokens_details` 和 `output_tokens_details` 中提取，传递到 OpenAI（`prompt_tokens_details`）、Anthropic（`cache_read_input_tokens`）、Gemini（`cachedContentTokenCount`）三种格式，覆盖流式和非流式模式 (#55, #58)
 - Dashboard 模型选择器使用后端 catalog 的 `isDefault` 字段，替代硬编码 `gpt-5.4`

src/routes/shared/proxy-handler.ts

@@ -54,6 +54,11 @@ export interface FormatAdapter {
  *
  * Handles: acquire, session lookup, retry, stream/collect, release, error formatting.
  */
+/** Clamp an HTTP status to a valid error StatusCode, defaulting to 502 for non-error codes. */
+function toErrorStatus(status: number): StatusCode {
+  return (status >= 400 && status < 600 ? status : 502) as StatusCode;
+}
+
 /** Check if a CodexApiError indicates the model is not supported on the account's plan. */
 function isModelNotSupportedError(err: CodexApiError): boolean {
   // Only 4xx client errors (exclude 429 rate-limit)
@@ -186,7 +191,7 @@ export async function handleProxyRequest(
               } catch (retryErr) {
                 accountPool.release(currentEntryId);
                 if (retryErr instanceof CodexApiError) {
-                  const code = (retryErr.status >= 400 && retryErr.status < 600 ? retryErr.status : 502) as StatusCode;
+                  const code = toErrorStatus(retryErr.status);
                   c.status(code);
                   return c.json(fmt.formatError(code, retryErr.message));
                 }
@@ -210,7 +215,7 @@ export async function handleProxyRequest(
             // Extract upstream status from error message (e.g. "HTTP/1.1 400 Bad Request")
             const statusMatch = msg.match(/HTTP\/[\d.]+ (\d{3})/);
             const upstreamStatus = statusMatch ? parseInt(statusMatch[1], 10) : 0;
-            const code = (upstreamStatus >= 400 && upstreamStatus < 600 ? upstreamStatus : 502) as StatusCode;
+            const code = toErrorStatus(upstreamStatus);
             c.status(code);
             return c.json(fmt.formatError(code, msg));
           }
@@ -241,7 +246,7 @@ export async function handleProxyRequest(
             continue; // re-enter model retry loop
           }
           // No other account available — return error (already released above)
-          const code = (err.status >= 400 && err.status < 600 ? err.status : 502) as StatusCode;
+          const code = toErrorStatus(err.status);
           c.status(code);
           return c.json(fmt.formatError(code, err.message));
         }

src/translation/shared-utils.ts

@@ -72,10 +72,14 @@ export function budgetToEffort(budget: number | undefined): string | undefined {
 export function injectAdditionalProperties(
   schema: Record<string, unknown>,
 ): Record<string, unknown> {
-  return walkSchema(structuredClone(schema));
+  return walkSchema(structuredClone(schema), new Set());
 }
 
-function walkSchema(node: Record<string, unknown>): Record<string, unknown> {
+function walkSchema(node: Record<string, unknown>, seen: Set<object>): Record<string, unknown> {
+  // Cycle detection — stop if we've already visited this node
+  if (seen.has(node)) return node;
+  seen.add(node);
+
   // Inject on object types that don't already specify additionalProperties
   if (node.type === "object" && node.additionalProperties === undefined) {
     node.additionalProperties = false;
@@ -86,7 +90,17 @@ function walkSchema(node: Record<string, unknown>): Record<string, unknown> {
     for (const key of Object.keys(node.properties)) {
       const prop = node.properties[key];
       if (isRecord(prop)) {
-        node.properties[key] = walkSchema(prop);
+        node.properties[key] = walkSchema(prop, seen);
+      }
+    }
+  }
+
+  // Traverse patternProperties
+  if (isRecord(node.patternProperties)) {
+    for (const key of Object.keys(node.patternProperties)) {
+      const prop = node.patternProperties[key];
+      if (isRecord(prop)) {
+        node.patternProperties[key] = walkSchema(prop, seen);
       }
     }
   }
@@ -97,7 +111,7 @@ function walkSchema(node: Record<string, unknown>): Record<string, unknown> {
       const defs = node[defsKey] as Record<string, unknown>;
       for (const key of Object.keys(defs)) {
         if (isRecord(defs[key])) {
-          defs[key] = walkSchema(defs[key] as Record<string, unknown>);
+          defs[key] = walkSchema(defs[key] as Record<string, unknown>, seen);
         }
       }
     }
@@ -105,13 +119,13 @@ function walkSchema(node: Record<string, unknown>): Record<string, unknown> {
 
   // Traverse items (array items)
   if (isRecord(node.items)) {
-    node.items = walkSchema(node.items as Record<string, unknown>);
+    node.items = walkSchema(node.items as Record<string, unknown>, seen);
   }
 
   // Traverse prefixItems
   if (Array.isArray(node.prefixItems)) {
     node.prefixItems = node.prefixItems.map((item: unknown) =>
-      isRecord(item) ? walkSchema(item) : item,
+      isRecord(item) ? walkSchema(item, seen) : item,
     );
   }
 
@@ -119,14 +133,16 @@ function walkSchema(node: Record<string, unknown>): Record<string, unknown> {
   for (const combiner of ["oneOf", "anyOf", "allOf"] as const) {
     if (Array.isArray(node[combiner])) {
       node[combiner] = (node[combiner] as unknown[]).map((entry: unknown) =>
-        isRecord(entry) ? walkSchema(entry) : entry,
+        isRecord(entry) ? walkSchema(entry, seen) : entry,
       );
     }
   }
 
-  // Traverse not
-  if (isRecord(node.not)) {
-    node.not = walkSchema(node.not as Record<string, unknown>);
+  // Traverse conditional: if, then, else
+  for (const keyword of ["if", "then", "else", "not"] as const) {
+    if (isRecord(node[keyword])) {
+      node[keyword] = walkSchema(node[keyword] as Record<string, unknown>, seen);
+    }
   }
 
   return node;

web/src/components/Header.tsx

@@ -74,7 +74,7 @@ export function Header({ onAddAccount, onCheckUpdate, onOpenUpdateModal, checkin
                 <span class="animate-ping absolute inline-flex h-full w-full rounded-full bg-primary opacity-75" />
                 <span class="relative inline-flex rounded-full h-2.5 w-2.5 bg-primary" />
               </span>
-              <StableText reference="Server Online" class="text-xs font-semibold text-primary">{t("serverOnline")}</StableText>
+              <StableText reference={t("serverOnline")} class="text-xs font-semibold text-primary">{t("serverOnline")}</StableText>
               {version && (
                 <span class="text-[0.65rem] font-mono text-primary/70 whitespace-nowrap">v{version}</span>
               )}
@@ -92,7 +92,7 @@ export function Header({ onAddAccount, onCheckUpdate, onOpenUpdateModal, checkin
               <svg class="size-3.5" viewBox="0 0 24 24" fill="currentColor">
                 <path d="M12 2l3.09 6.26L22 9.27l-5 4.87 1.18 6.88L12 17.77l-6.18 3.25L7 14.14 2 9.27l6.91-1.01L12 2z" />
               </svg>
-              <StableText reference="Star" class="text-xs font-semibold">{t("starOnGithub")}</StableText>
+              <StableText reference={t("starOnGithub")} class="text-xs font-semibold">{t("starOnGithub")}</StableText>
             </a>
             {/* Check for Updates */}
             <button
@@ -103,13 +103,13 @@ export function Header({ onAddAccount, onCheckUpdate, onOpenUpdateModal, checkin
               <svg class={`size-3.5 ${checking ? "animate-spin" : ""}`} viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2">
                 <path stroke-linecap="round" stroke-linejoin="round" d="M16.023 9.348h4.992v-.001M2.985 19.644v-4.992m0 0h4.992m-4.992 0l3.181 3.183a8.25 8.25 0 0013.803-3.7M4.031 9.865a8.25 8.25 0 0113.803-3.7l3.181 3.182M20.985 4.356v4.992" />
               </svg>
-              <StableText reference="Check for Updates" class="text-xs font-semibold">{checking ? t("checkingUpdates") : t("checkForUpdates")}</StableText>
+              <StableText reference={t("checkForUpdates")} class="text-xs font-semibold">{checking ? t("checkingUpdates") : t("checkForUpdates")}</StableText>
             </button>
             {/* Update status message */}
             {updateStatusMsg && !checking && (
               <button
                 onClick={hasUpdate && onOpenUpdateModal ? onOpenUpdateModal : onCheckUpdate}
-                class={`hidden lg:inline text-xs font-medium ${updateStatusColor} hover:underline`}
+                class={`hidden lg:inline whitespace-nowrap text-xs font-medium ${updateStatusColor} hover:underline`}
               >
                 {updateStatusMsg}
               </button>
@@ -140,7 +140,7 @@ export function Header({ onAddAccount, onCheckUpdate, onOpenUpdateModal, checkin
                   <svg class="size-3.5" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2">
                     <path stroke-linecap="round" stroke-linejoin="round" d="M10.5 6h9.75M10.5 6a1.5 1.5 0 11-3 0m3 0a1.5 1.5 0 10-3 0M3.75 6H7.5m3 12h9.75m-9.75 0a1.5 1.5 0 01-3 0m3 0a1.5 1.5 0 00-3 0m-3.75 0H7.5m9-6h3.75m-3.75 0a1.5 1.5 0 01-3 0m3 0a1.5 1.5 0 00-3 0m-9.75 0h9.75" />
                   </svg>
-                  <StableText reference="Proxy Assignment" class="text-xs font-semibold">{t("proxySettings")}</StableText>
+                  <StableText reference={t("proxySettings")} class="text-xs font-semibold">{t("proxySettings")}</StableText>
                 </a>
                 <button
                   onClick={onAddAccount}
@@ -149,7 +149,7 @@ export function Header({ onAddAccount, onCheckUpdate, onOpenUpdateModal, checkin
                   <svg class="size-3.5" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2.5">
                     <path stroke-linecap="round" stroke-linejoin="round" d="M12 4.5v15m7.5-7.5h-15" />
                   </svg>
-                  <StableText reference="Add Account">{t("addAccount")}</StableText>
+                  <StableText reference={t("addAccount")}>{t("addAccount")}</StableText>
                 </button>
               </>
             )}