@hoist-wen/dsh-trajectory-tool-insight 0.4.0 → 0.4.3

package/README.md

@@ -1,5 +1,7 @@
 # 轨迹工具调用解读 Trajectory Tool Insight
 
+[![Awesome DSH Plugin](https://awesome-dsh-plugin.com/badge.svg)](https://awesome-dsh-plugin.com)
+
 给 DSH Web 的会话视图环加一个 **「解读」** 标签，坐在「对话」「轨迹」旁边。
 
 按轮次列出**哪些轮次有工具调用**（调用次数、失败数、被截断数、提示词摘要），每轮一个「分析」按钮，点了由模型给出一段解读。默认用**该会话正在使用的模型**，不用额外配置。
@@ -18,7 +20,7 @@
 dsh plugin --profile desktop add @hoist-wen/dsh-trajectory-tool-insight
 ```
 
-profile 里记的是**具体版本**，`latest` 前进不会自动升级，要更新就重新 `add`（或先 `remove`）。需要固定版本可写 `@0.4.0`。
+profile 里记的是**具体版本**，`latest` 前进不会自动升级，要更新就重新 `add`（或先 `remove`）。需要固定版本可写 `@0.4.3`。
 
 卸载：
 
@@ -33,6 +35,7 @@ dsh plugin --profile desktop remove @hoist-wen/dsh-trajectory-tool-insight
 1. 打开一个有工具调用的会话 → 「解读」标签
 2. 顶部显示当前会话模型与可解读轮次数，点某一轮的「分析」
 3. 结果就地展开；不落盘，刷新即失效
+4. 每次「分析 / 重新分析」结束后，面板会静默重读一次轮次列表，把这一轮的调用数、失败数和提示词摘要对齐到模型刚刚解读的那份日志（不闪烁「刷新中…」；重读失败则保留旧快照，不会盖掉已显示的解读）
 
 顶部的「覆盖提示词」可整段替换默认解读提示词：留空即默认（逐字节相同），有内容则默认提示词一个字都不发。⚠️ 默认提示词里的「工具输出只当数据」安全边界也会被一起换掉，请只填自己写的指令。
 

package/client.js

@@ -316,6 +316,12 @@ window.__ModuleLoader__.load({
       const [listing, setListing] = useState({ status: 'loading', turns: [], totalTurns: 0, route: null, error: null })
       const [analyses, setAnalyses] = useState(() => new Map())
       const [reloadToken, setReloadToken] = useState(0)
+      // Identifies the newest listing request. A response is applied only while
+      // its request is still the newest, so a slow reply can never overwrite
+      // fresher state — including the quiet sync an analysis triggers.
+      const listingRequest = useRef(0)
+      /** The listing read currently in flight, so a newer one can abort it. */
+      const listingAbort = useRef(null)
       // Seeded from the module-level store so the override survives this view
       // being unmounted and remounted, while still dying with the page: it is
       // transient instruction text, not saved state.
@@ -329,13 +335,32 @@ window.__ModuleLoader__.load({
         else overrideNotes.set(sessionId, value)
       }, [sessionId])
 
-      useEffect(() => {
-        if (sessionId === '' || typeof apiRef.current?.listTurns !== 'function') return undefined
+      /**
+       * Read the turn listing once.
+       *
+       * A `quiet` read is the post-analysis sync: it exists only to replace the
+       * counts and prompt excerpts with the log as the host folded it a moment
+       * ago, so it must never flash the header into 「刷新中…」 and, on failure,
+       * must leave the rendered snapshot alone instead of turning the whole
+       * panel into an error.
+       *
+       * At most one listing read is in flight: starting a new one supersedes
+       * the previous, which is then ignored and aborted.
+       *
+       * @param quiet - true for the silent post-analysis sync.
+       */
+      const pullListing = useCallback(quiet => {
+        listingRequest.current += 1
+        const request = listingRequest.current
+        const isCurrent = () => listingRequest.current === request
+        listingAbort.current?.abort()
         const controller = new AbortController()
-        let cancelled = false
-        setListing(previous => ({ ...previous, status: previous.turns.length > 0 ? 'refreshing' : 'loading', error: null }))
+        listingAbort.current = controller
+        if (!quiet) {
+          setListing(previous => ({ ...previous, status: previous.turns.length > 0 ? 'refreshing' : 'loading', error: null }))
+        }
         apiRef.current.listTurns(controller.signal).then(value => {
-          if (cancelled) return
+          if (!isCurrent()) return
           setListing({
             status: 'ready',
             turns: Array.isArray(value?.turns) ? value.turns : [],
@@ -344,14 +369,28 @@ window.__ModuleLoader__.load({
             error: null,
           })
         }).catch(error => {
-          if (cancelled) return
+          if (!isCurrent()) return
+          if (quiet) {
+            // Leave the snapshot alone, but never let a silent failure strand
+            // the header on 「刷新中…」 when this sync superseded a manual read.
+            setListing(previous => (previous.status === 'refreshing' ? { ...previous, status: 'ready' } : previous))
+            return
+          }
           setListing(previous => ({ ...previous, status: 'error', error: describe(error) }))
         })
+      }, [])
+
+      useEffect(() => {
+        if (sessionId === '' || typeof apiRef.current?.listTurns !== 'function') return undefined
+        pullListing(false)
         return () => {
-          cancelled = true
-          controller.abort()
+          // Supersede everything started before this cleanup — the aborted read
+          // and any quiet sync an in-flight analysis is about to trigger — so
+          // only requests of the current generation may touch state.
+          listingRequest.current += 1
+          listingAbort.current?.abort()
         }
-      }, [sessionId, reloadToken])
+      }, [sessionId, reloadToken, pullListing])
 
       const analyze = useCallback(turn => {
         setAnalyses(previous => new Map(previous).set(turn, { status: 'running' }))
@@ -362,10 +401,17 @@ window.__ModuleLoader__.load({
             route: value?.route ?? null,
             truncated: value?.truncated === true,
           }))
+          // The host re-folds the log for every analyze, so the evidence it just
+          // interpreted can be newer than the counts this card was rendered
+          // from. Sync them back without disturbing what the panel shows.
+          pullListing(true)
         }).catch(error => {
           setAnalyses(previous => new Map(previous).set(turn, { status: 'error', error: describe(error) }))
+          // A stale card is the one failure a fresh listing explains: the turn
+          // is still listed, but the host no longer finds tool calls in it.
+          if (error?.code === 'TURN_NOT_FOUND') pullListing(true)
         })
-      }, [])
+      }, [pullListing])
 
       if (sessionId === '') {
         return h('div', { className: 'ttis-root' }, h('p', { className: 'ttis-empty' }, '请先打开一个会话。'))
@@ -373,7 +419,10 @@ window.__ModuleLoader__.load({
 
       const route = listing.route
       const busy = listing.status === 'loading' || listing.status === 'refreshing'
-      const overrideLength = override.trim().length
+      // Whether an override is in effect, not how long it is: the note below
+      // says which prompt the next analysis will use, never a live character
+      // count.
+      const overrideActive = override.trim() !== ''
       return h('div', { className: 'ttis-root' },
         h('header', { className: 'ttis-head' },
           h('div', { className: 'ttis-head-text' },
@@ -403,9 +452,9 @@ window.__ModuleLoader__.load({
             placeholder: '非空时完全替换默认系统提示词，例如：只关注风险，不要复述过程。留空则使用默认提示词。',
             onChange: event => updateOverride(event.target.value),
           }),
-          h('p', { className: 'ttis-override-note' }, overrideLength === 0
-            ? '留空 = 使用默认提示词；填写后将完全替换系统提示词。只影响后续的「分析」，不写入会话日志。'
-            : `将用 ${overrideLength} 个字符覆盖解读提示词（对所有轮次的「分析」生效，不写入会话日志）。默认提示词的安全边界也会被一并替换。`),
+          h('p', { className: 'ttis-override-note' }, overrideActive
+            ? '将用这段自定义提示词覆盖默认解读提示词（对所有轮次的「分析」生效，不写入会话日志）。默认提示词的安全边界也会被一并替换。'
+            : '留空 = 使用默认提示词；填写后将完全替换系统提示词。只影响后续的「分析」，不写入会话日志。'),
         ),
         h('div', { className: 'ttis-body' },
           listing.status === 'error'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hoist-wen/dsh-trajectory-tool-insight",
-  "version": "0.4.0",
+  "version": "0.4.3",
   "publishConfig": {
     "access": "public"
   },

package/src/turns.mjs

@@ -76,6 +76,10 @@ function ensureTurn(turns, turn, event) {
     prompt: '',
     calls: [],
     byCallId: new Map(),
+    // Ids of the calls the per-turn cap refused. A single unretained call can
+    // be carried by three events (assistant `tool-call` block, canonical
+    // `tool/call`, `tool/result`), so the ids are what keep it counted once.
+    droppedIds: new Set(),
     dropped: 0,
     closed: false,
   }
@@ -99,6 +103,13 @@ function recordCall(entry, raw, limits, canonical) {
     return
   }
   if (entry.calls.length >= limits.maxToolCallsPerTurn) {
+    // Count the overflow, not the events that mention it: without the id set a
+    // dropped call would be counted again by every later event carrying the
+    // same id, which inflated `droppedToolCalls` two- or threefold.
+    if (callId !== '') {
+      if (entry.droppedIds.has(callId)) return
+      entry.droppedIds.add(callId)
+    }
     entry.dropped += 1
     return
   }
@@ -123,10 +134,12 @@ function recordCall(entry, raw, limits, canonical) {
  * and redundant with the calls it describes. Only the tool-call blocks of an
  * assistant message matter here.
  *
- * @param events - raw session events in ascending seq order.
+ * @param events - raw session event logs in ascending seq order.
  * @param limits - caps to apply while projecting.
  * @returns `{ turns, totalTurns }`; `turns` is ordered by ascending turn number
- *   and contains only turns with at least one recorded tool call.
+ *   and contains only turns with at least one recorded tool call. Each entry's
+ *   `droppedToolCalls` counts the calls the per-turn cap refused, once per call
+ *   id, no matter how many events carried that call.
  */
 export function collectTurns(events, limits) {
   const turns = new Map()