@changfenhuang/dsh-genui 0.11.3 → 0.11.4

package/src/plugin/fence-feedback.ts

@@ -8,8 +8,10 @@
  *
  * The loop is deliberately narrow, matching the contract agreed on the issue:
  * - **默认开启。** 插件配置中的 `fenceFeedback: false` 可以关闭回合转向。
- * - **Bounded.** At most one correction per turn AND at most one per fence
- *   body per process, so a correction that is itself wrong cannot loop.
+ * - **修正上限。** 每个 turn 最多发送两条 correction；render failure 和 delivery reminder 共用上限，
+ *   每个 fence body 在当前回合中最多修正一次，新回合可重新修正。
+ * - **仅含 reasoning 的恢复。** 已验证的 GenUI 回合若只以 reasoning block 结束，使用宿主的
+ *   `EMPTY_RESPONSE` retry policy。
  * - **Never for subagents.** A child session's fence belongs to a parent reply.
  * - **Exact fence matching.** Only an info string of exactly `dsh-ui` opens a
  *   fence, so ` ```dsh-ui-dark `, indented prose, or a mention of the name is
@@ -25,8 +27,8 @@
 
 import type { Context } from '@deepseek-ai/cordis'
 import type {} from '@deepseek-ai/dsh-agent'
-import type { SessionEvent } from '@deepseek-ai/dsh-session'
-import type { UserMessage } from '@deepseek-ai/dsh-session'
+import type { GenerateOptions, StreamChunk } from '@deepseek-ai/dsh-llm'
+import type { Session, SessionEvent, SessionStore, UserMessage } from '@deepseek-ai/dsh-session'
 import { createHash, randomUUID } from 'node:crypto'
 import { droppedNodeFailure } from './genui-diagnostic.ts'
 import { resolveFence } from '../shared/fence-resolve.ts'
@@ -38,6 +40,12 @@ export const FEEDBACK_SOURCE_KIND = `plugin:${FEEDBACK_PLUGIN_NAME}` as const
 
 /** Marker prefix inside the correction text: `[genui-fence-repair #<fingerprint>]`. */
 const MARKER_PREFIX = '[genui-fence-repair #'
+/**
+ * 同一个 turn 内 fence-feedback correction 的共享上限。
+ * Render failure 和 nothing-delivered reminder 共用该 budget；各自的 ledger 独立阻止重复修正。
+ */
+export const MAX_CORRECTIONS_PER_TURN = 2
+
 /** Marker prefix written by older plugin versions. */
 const LEGACY_MARKER_PREFIX = '[genui 自修 #'
 
@@ -177,21 +185,64 @@ export function createFeedbackMessage(text: string, sessionFormatVersion: number
 
 /** Per-session bookkeeping for the loop (process lifetime). */
 interface SessionFeedback {
-  /** Latest assistant reply text of the current turn. */
+  /** Latest assistant reply text of the current turn (fences are read from here). */
   text: string
-  /** Fence fingerprints already corrected in this process. */
-  corrected: Set<string>
-  /** Turn that already received its one correction. */
-  lastCorrectedTurn: number | undefined
+  /**
+   * A `validate_dsh_ui` call happened in this turn. This is the FORMAL signal
+   * that the turn is GenUI-related — the loop never guesses intent by scanning
+   * the reasoning block.
+   */
+  validatedThisTurn: boolean
+  /** 子代理会话的输出不会直接交给用户。 */
+  isSubagent: boolean
+  /**
+   * The turn delivered something formal: a non-empty body text, or a
+   * `render_ui` call whose RESULT reports success. A failed render is not an
+   * answer, and a call whose result has not arrived yet proves nothing.
+   */
+  deliveredThisTurn: boolean
+  /**
+   * `render_ui` calls of this turn whose result has not arrived yet. The
+   * result — not the call — decides delivery, so an outstanding call blocks
+   * steering at the boundary instead of being guessed about.
+   */
+  pendingRenders: Set<string>
+  /** Fence fingerprints already corrected for a RENDER failure in the current turn. */
+  correctedSpec: Set<string>
+  /**
+   * Turns already given the "nothing was delivered" reminder. Kept separate from
+   * {@link correctedSpec}: a missing delivery must not consume the render-failure
+   * ledger (nor the other way round).
+   */
+  deliveryRemindedTurns: Set<number>
+  /** Corrections already steered in {@link correctionsTurn}. */
+  correctionsThisTurn: number
+  /** Turn {@link correctionsThisTurn} counts. */
+  correctionsTurn: number | undefined
+  /** 从 session event 中恢复的当前 turn。 */
+  currentTurn: number | undefined
+  /** 已计入 correction budget 的 message ID。 */
+  accountedCorrectionMessageIds: Set<string>
 }
 
 /** What the pure planner needs to decide whether a correction may be sent. */
 export interface FenceFeedbackPlanInput {
+  /** Latest assistant reply text of the current turn. */
   readonly text: string
   readonly turn: number
-  readonly lastCorrectedTurn: number | undefined
-  readonly corrected: ReadonlySet<string>
+  /** Fence bodies already corrected for a render failure in the current turn. */
+  readonly correctedSpec: ReadonlySet<string>
+  /** Turns already given the delivery reminder. */
+  readonly deliveryRemindedTurns?: ReadonlySet<number> | undefined
+  /** A `validate_dsh_ui` call happened this turn (formal GenUI signal). */
+  readonly validatedThisTurn?: boolean | undefined
+  /** The turn already delivered a body or a successful `render_ui` result. */
+  readonly deliveredThisTurn?: boolean | undefined
   readonly aborted: boolean
+  /** Corrections already steered in this turn (shared hard cap). */
+  readonly correctionsThisTurn?: number | undefined
+  /** Turn {@link correctionsThisTurn} counts (stale counts are ignored). */
+  readonly correctionsTurn?: number | undefined
 }
 
 /** A correction the caller must account for before steering. */
@@ -199,26 +250,100 @@ export interface FenceFeedbackPlan {
   readonly text: string
   readonly fingerprints: readonly string[]
   readonly turn: number
+  /**
+   * `render` corrects a fence that failed to resolve; `delivery` reminds the
+   * model that the turn produced nothing formal. They share the per-turn budget
+   * but keep separate ledgers.
+   */
+  readonly kind: 'render' | 'delivery'
 }
 
 /**
- * Decide whether this turn boundary should steer a fence correction — the pure
- * core of the loop, so every bound (one per turn, one per fence, cancellation)
- * is testable without a host.
+ * Decide whether this turn boundary should steer a correction — the pure core of
+ * the loop, so every bound (per-turn cap, per-fence ledger, cancellation) is
+ * testable without a host.
  *
- * @param input - reply text, turn identity, and the session's accounting.
+ * The decision is driven by FORMAL events only: fences in the reply body, a
+ * `validate_dsh_ui` call, a delivered body text or a successful `render_ui`
+ * result. The reasoning block is
+ * never read here — a draft inside the thinking block is not proof that the model
+ * chose to deliver it, so it must not change any decision.
+ *
+ * @param input - reply text, turn identity, formal signals, and the accounting.
  * @returns the correction to send, or null when the loop must stay silent.
  */
 export function planFenceFeedback(input: FenceFeedbackPlanInput): FenceFeedbackPlan | null {
   if (input.aborted) return null
-  if (input.lastCorrectedTurn === input.turn) return null
-  if (input.text.trim() === '') return null
-  const failures = fenceFailures(input.text).filter(failure => !input.corrected.has(failure.fingerprint))
-  if (failures.length === 0) return null
-  return {
-    text: fenceCorrectionText(failures),
-    fingerprints: failures.map(failure => failure.fingerprint),
-    turn: input.turn,
+  const used = input.correctionsTurn === input.turn ? (input.correctionsThisTurn ?? 0) : 0
+  if (used >= MAX_CORRECTIONS_PER_TURN) return null
+  const failures = fenceFailures(input.text).filter(failure => !input.correctedSpec.has(failure.fingerprint))
+  if (failures.length > 0) {
+    return {
+      text: fenceCorrectionText(failures),
+      fingerprints: failures.map(failure => failure.fingerprint),
+      turn: input.turn,
+      kind: 'render',
+    }
+  }
+  // No un-corrected render failure. If the turn formally validated a spec and
+  // then delivered nothing, remind the model once for this turn.
+  if (input.validatedThisTurn !== true || input.deliveredThisTurn === true) return null
+  if (input.deliveryRemindedTurns?.has(input.turn) === true) return null
+  return { text: missingBodyCorrectionText(input.turn, used + 1), fingerprints: [], turn: input.turn, kind: 'delivery' }
+}
+
+/**
+ * Correction for a validated GenUI turn that reached the boundary without any
+ * formal delivery.
+ *
+ * @param turn - turn that reached the boundary.
+ * @param attempt - correction number within the shared turn budget.
+ * @returns the message text to steer into the running turn.
+ */
+export function missingBodyCorrectionText(turn: number, attempt = 1): string {
+  const head = `${MARKER_PREFIX}turn-${turn}]\n\n[genui-fence-repair]\nstatus=nothing_delivered\nfences=0\nnext=emit_fence_in_body\nrepeat_rendered_content=false\nreply_language=conversation\n\n`
+  const emphasis = attempt <= 1 ? '' : `（第 ${attempt} 次提醒）`
+  return `${head}本轮尚未产生正式回答，也没有通过支持的通道交付结果${emphasis}。请根据用户当前请求完成正式答复；需要 UI 时，在回答正文输出你最终选定的 dsh-ui 围栏，或明确调用 render_ui。可以修改或放弃此前候选；不能完成时，请在正文说明原因。\n`
+}
+
+/**
+ * 将 GenUI 回合中仅含 reasoning 的完整响应转换为宿主已有的可重试空响应错误。
+ *
+ * @param options - LLM stream waterfall 拦截的请求。
+ * @param source - 本次请求的 provider stream。
+ * @returns 原始 stream；仅符合条件的终止 stop 会被改写。
+ */
+async function* retryReasoningOnlyGenuiStream(
+  options: GenerateOptions,
+  source: AsyncIterable<StreamChunk>,
+): AsyncGenerator<StreamChunk> {
+  const { EMPTY_RESPONSE_CODE, isAgentLoopRequest } = await import('@deepseek-ai/dsh-llm')
+  if (!isAgentLoopRequest(options)) {
+    yield* source
+    return
+  }
+
+  let hasReasoningBlock = false
+  let hasOtherBlock = false
+  for await (const chunk of source) {
+    if (chunk.type === 'block-end') {
+      if (chunk.block.type === 'reasoning') hasReasoningBlock = true
+      else hasOtherBlock = true
+    }
+    if (chunk.type === 'finish' && chunk.reason.kind === 'stop' && hasReasoningBlock && !hasOtherBlock) {
+      yield {
+        type: 'finish',
+        reason: {
+          kind: 'error',
+          failure: {
+            message: 'GenUI turn completed with reasoning only and no deliverable response',
+            code: EMPTY_RESPONSE_CODE,
+          },
+        },
+      }
+    } else {
+      yield chunk
+    }
   }
 }
 
@@ -235,9 +360,77 @@ function textOfContent(content: unknown): string {
     .join('\n')
 }
 
-/** Fingerprints this loop already recorded inside a steered correction. */
-function markersIn(text: string): string[] {
-  const out: string[] = []
+type RenderResultStatus = 'rendered' | 'invalid'
+
+/** 读取 render_ui result protocol 中明确返回的 status。 */
+function renderResultStatus(content: unknown): RenderResultStatus | undefined {
+  const lines = textOfContent(content).split(/\r?\n/u)
+  if (lines[0]?.trim() !== '[genui-render]') return undefined
+  for (const line of lines.slice(1)) {
+    if (line === 'status=rendered') return 'rendered'
+    if (line === 'status=invalid') return 'invalid'
+  }
+  return undefined
+}
+
+interface ObservedToolResult {
+  readonly callId: string
+  readonly isError: boolean
+  readonly content: unknown
+}
+
+/** 从 legacy wrapper 与 Session format v4 中提取 tool result 数据。 */
+function observedToolResult(message: unknown): ObservedToolResult | null {
+  if (typeof message !== 'object' || message === null) return null
+  const record = message as { toolCallId?: unknown; isError?: unknown; content?: unknown }
+  if (typeof record.toolCallId === 'string') {
+    return { callId: record.toolCallId, isError: record.isError === true, content: record.content }
+  }
+  if (!Array.isArray(record.content)) return null
+  const block = record.content.find(part => typeof part === 'object' && part !== null
+    && (part as { type?: unknown }).type === 'tool-result') as {
+      toolCallId?: unknown
+      isError?: unknown
+      content?: unknown
+    } | undefined
+  if (typeof block?.toolCallId !== 'string') return null
+  return { callId: block.toolCallId, isError: block.isError === true, content: block.content }
+}
+
+/** Tool whose SUCCESSFUL result is a formal delivery. */
+const DELIVERY_TOOL = 'render_ui'
+
+/**
+ * Whether one assistant message delivered a non-empty body text.
+ *
+ * Deliberately narrow: a non-empty text block. A `render_ui` call is decided
+ * by its `tool/result`, not by the call appearing in a message — a call that
+ * failed (or whose result has not arrived) is not a delivery. Other tools
+ * (validate_dsh_ui, bash, …) may succeed without producing any answer, so
+ * they are NOT counted as delivery.
+ *
+ * @param content - assistant content blocks.
+ * @returns true when this message delivered a body text.
+ */
+function deliveredBodyText(content: unknown): boolean {
+  if (!Array.isArray(content)) return false
+  return content.some(block => {
+    if (typeof block !== 'object' || block === null) return false
+    const record = block as { type?: unknown; text?: unknown }
+    return record.type === 'text' && typeof record.text === 'string' && record.text.trim() !== ''
+  })
+}
+
+
+interface FeedbackMarkers {
+  readonly renderFingerprints: string[]
+  readonly deliveryTurns: number[]
+}
+
+/** 分类当前与 legacy correction message 中的 marker。 */
+function feedbackMarkersIn(text: string): FeedbackMarkers {
+  const renderFingerprints: string[] = []
+  const deliveryTurns: number[] = []
   let cursor = 0
   while (cursor < text.length) {
     const current = text.indexOf(MARKER_PREFIX, cursor)
@@ -248,10 +441,23 @@ function markersIn(text: string): string[] {
     const index = useLegacy ? legacy : current
     const end = text.indexOf(']', index + prefix.length)
     if (end < 0) break
-    out.push(text.slice(index + prefix.length, end))
+    const marker = text.slice(index + prefix.length, end)
+    const turn = /^turn-(\d+)$/u.exec(marker)
+    if (turn !== null) deliveryTurns.push(Number(turn[1]))
+    else renderFingerprints.push(marker)
     cursor = end + 1
   }
-  return out
+  return { renderFingerprints, deliveryTurns }
+}
+
+/** 恢复一条持久化 correction 消耗的 turn budget。 */
+function accountReplayedCorrection(state: SessionFeedback, turn: number): void {
+  if (state.correctionsTurn !== turn) {
+    state.correctionsTurn = turn
+    state.correctionsThisTurn = 1
+    return
+  }
+  state.correctionsThisTurn = Math.min(MAX_CORRECTIONS_PER_TURN, state.correctionsThisTurn + 1)
 }
 
 /** Identify this plugin's source across current and migrated session shapes. */
@@ -261,18 +467,29 @@ function isFeedbackSource(source: { kind?: unknown; plugin?: unknown } | undefin
 }
 
 /**
- * 根据插件配置启用围栏反馈流程。
+ * 注册 GenUI 回合跟踪、宿主重试判定和可选的围栏修正流程。
  *
- * @param ctx - the host context.
- * @param enabled - the plugin config flag; the loop is inert when false.
+ * @param ctx - 宿主 Context。
+ * @param enabled - 是否启用同回合围栏修正。
  */
 export function installFenceFeedback(ctx: Context, enabled: boolean): void {
-  if (!enabled) return
   const sessions = new Map<string, SessionFeedback>()
   const stateOf = (sessionId: string): SessionFeedback => {
     let state = sessions.get(sessionId)
     if (state === undefined) {
-      state = { text: '', corrected: new Set(), lastCorrectedTurn: undefined }
+      state = {
+        text: '',
+        validatedThisTurn: false,
+        isSubagent: false,
+        deliveredThisTurn: false,
+        pendingRenders: new Set(),
+        correctedSpec: new Set(),
+        deliveryRemindedTurns: new Set(),
+        correctionsThisTurn: 0,
+        correctionsTurn: undefined,
+        currentTurn: undefined,
+        accountedCorrectionMessageIds: new Set(),
+      }
       sessions.set(sessionId, state)
     }
     return state
@@ -282,53 +499,155 @@ export function installFenceFeedback(ctx: Context, enabled: boolean): void {
     sessions.delete(String(session.id))
   })
 
-  ctx.on('session/event', (session, event: SessionEvent) => {
+  const resetTurnState = (state: SessionFeedback, turn: number | undefined): void => {
+    state.text = ''
+    state.validatedThisTurn = false
+    state.deliveredThisTurn = false
+    state.pendingRenders.clear()
+    state.currentTurn = turn
+  }
+
+  const observeEvent = (session: Session, event: SessionEvent): void => {
     const sessionId = String(session.id)
+    if (event.type === 'turn/start') {
+      // The formal turn boundary: whatever happened in the previous turn is
+      // settled and this turn starts clean. The `user/message` reset below is
+      // only a fallback for direct prompts.
+      const state = stateOf(sessionId)
+      state.isSubagent = session.header.parentSession !== undefined
+      // 去重只约束当前回合，新一轮仍可修正相同错误。
+      state.correctedSpec.clear()
+      resetTurnState(state, (event.data as { turn?: number }).turn)
+      return
+    }
     if (event.type === 'assistant/message') {
-      const text = textOfContent((event.data as { message?: { content?: unknown } }).message?.content)
-      if (extractDshUiFences(text).length === 0) {
-        const state = sessions.get(sessionId)
-        if (state !== undefined) state.text = ''
-        return
+      const content = (event.data as { message?: { content?: unknown } }).message?.content
+      const text = textOfContent(content)
+      const state = stateOf(sessionId)
+      state.isSubagent = session.header.parentSession !== undefined
+      // Fences are read from the BODY only: a draft in the reasoning block is not
+      // a delivery and must not become one.
+      state.text = extractDshUiFences(text).length > 0 ? text : ''
+      // Delivery is sticky for the turn: once a body text exists, the turn has
+      // answered and must never be corrected into publishing again. A
+      // `render_ui` call in the content proves nothing here — its result does.
+      state.deliveredThisTurn = state.deliveredThisTurn || deliveredBodyText(content)
+      return
+    }
+    if (event.type === 'tool/call') {
+      const data = event.data as { name?: unknown; callId?: unknown }
+      const state = stateOf(sessionId)
+      state.isSubagent = session.header.parentSession !== undefined
+      // validate_dsh_ui is the FORMAL signal that this turn is GenUI-related.
+      if (data.name === 'validate_dsh_ui') state.validatedThisTurn = true
+      // The result — not the call — decides whether render_ui delivered.
+      if (data.name === DELIVERY_TOOL && typeof data.callId === 'string') {
+        state.pendingRenders.add(data.callId)
+      }
+      return
+    }
+    if (event.type === 'tool/result') {
+      const data = event.data as { message?: unknown; error?: unknown }
+      const result = observedToolResult(data.message)
+      if (result === null) return
+      const state = stateOf(sessionId)
+      state.isSubagent = session.header.parentSession !== undefined
+      if (!state.pendingRenders.delete(result.callId)) return
+      if (data.error === undefined && result.isError !== true && renderResultStatus(result.content) === 'rendered') {
+        state.deliveredThisTurn = true
       }
-      stateOf(sessionId).text = text
       return
     }
     if (event.type !== 'user/message') return
-    const data = event.data as { content?: unknown; source?: { kind?: unknown; plugin?: unknown } }
+    const data = event.data as { id?: unknown; content?: unknown; source?: { kind?: unknown; plugin?: unknown } }
     if (isFeedbackSource(data.source)) {
-      // Our own correction (re-observed after a plugin reload): adopt its
-      // fingerprints so a second boundary cannot repeat it.
-      const fingerprints = markersIn(textOfContent(data.content))
-      if (fingerprints.length === 0) return
+      const markers = feedbackMarkersIn(textOfContent(data.content))
+      if (markers.renderFingerprints.length === 0 && markers.deliveryTurns.length === 0) return
       const state = stateOf(sessionId)
-      for (const fingerprint of fingerprints) state.corrected.add(fingerprint)
+      state.isSubagent = session.header.parentSession !== undefined
+      for (const fingerprint of markers.renderFingerprints) state.correctedSpec.add(fingerprint)
+      for (const turn of markers.deliveryTurns) state.deliveryRemindedTurns.add(turn)
+      if (typeof data.id === 'string' && !state.accountedCorrectionMessageIds.has(data.id)) {
+        state.accountedCorrectionMessageIds.add(data.id)
+        const turn = markers.deliveryTurns[0] ?? state.currentTurn
+        if (turn !== undefined) accountReplayedCorrection(state, turn)
+      }
       return
     }
-    // A genuine user prompt starts a new turn: the previous reply is settled.
+    // A DIRECT human prompt starts a new turn: the previous reply is settled.
+    // Synthetic context that rides the same user surface — `agent.inject()`
+    // notices, team member messages — carries a different source kind and
+    // arrives MID-TURN: resetting here used to wipe the turn's validation and
+    // delivery state before the boundary could use it.
+    if (data.source?.kind !== 'user') return
     const state = sessions.get(sessionId)
-    if (state !== undefined) state.text = ''
+    if (state !== undefined) {
+      state.isSubagent = session.header.parentSession !== undefined
+      // turn/start 先于用户消息；清掉正文状态时保留正式回合身份。
+      resetTurnState(state, state.currentTurn)
+    }
+  }
+
+  const restoreHistory = (session: Session): void => {
+    if (sessions.has(String(session.id))) return
+    for (const event of session.snapshotEvents()) observeEvent(session, event)
+  }
+  ctx.on('session/event', (session, event) => {
+    // 服务注入是异步的；首条实时事件也先接管历史，避免丢失旧纠错次数。
+    restoreHistory(session)
+    observeEvent(session, event)
   })
+  ctx.on('session/created', restoreHistory)
+  const restoreSessions = (sessionCtx: Context): void => {
+    const store = sessionCtx.reflect.get('sessions') as SessionStore | undefined
+    if (store === undefined) return
+    for (const session of store.list()) restoreHistory(session)
+  }
+  // 宿主不重播历史；安装时同步接管已有会话，异步注入负责晚到的服务。
+  restoreSessions(ctx)
+  ctx.inject(['sessions'], restoreSessions)
+
+  ctx.on('llm/stream', (options, next) => {
+    if (options.sessionId === undefined || options.purpose !== undefined) return next()
+    const state = sessions.get(String(options.sessionId))
+    if (state?.validatedThisTurn !== true || state.isSubagent) return next()
+    return retryReasoningOnlyGenuiStream(options, next())
+  }, { global: true })
 
   ctx.on('agent/turn-stopping', ({ agent, turn, signal }): void => {
+    if (!enabled) return
     // A child session's fence belongs to a parent reply, and an aborted turn is
     // on its way out: never steer into either.
     if (agent.session.header.parentSession !== undefined) return
     const state = sessions.get(String(agent.session.id))
     if (state === undefined) return
+    state.currentTurn = turn
+    // A render_ui result still outstanding could deliver (or fail) after this
+    // boundary fires: steering now would race the late result, so stay silent
+    // and let the next boundary decide on settled facts.
+    if (state.pendingRenders.size > 0) return
+    const usedThisTurn = state.correctionsTurn === turn ? state.correctionsThisTurn : 0
     const plan = planFenceFeedback({
       text: state.text,
       turn,
-      lastCorrectedTurn: state.lastCorrectedTurn,
-      corrected: state.corrected,
+      correctedSpec: state.correctedSpec,
+      deliveryRemindedTurns: state.deliveryRemindedTurns,
+      validatedThisTurn: state.validatedThisTurn,
+      deliveredThisTurn: state.deliveredThisTurn,
       aborted: signal.aborted,
+      correctionsThisTurn: usedThisTurn,
+      correctionsTurn: state.correctionsTurn,
     })
     if (plan === null) return
     // Account BEFORE sending: a re-entrant boundary must not deliver twice.
-    for (const fingerprint of plan.fingerprints) state.corrected.add(fingerprint)
-    state.lastCorrectedTurn = plan.turn
+    for (const fingerprint of plan.fingerprints) state.correctedSpec.add(fingerprint)
+    if (plan.kind === 'delivery') state.deliveryRemindedTurns.add(plan.turn)
+    state.correctionsTurn = plan.turn
+    state.correctionsThisTurn = usedThisTurn + 1
     try {
-      agent.steer(createFeedbackMessage(plan.text, agent.session.header.version))
+      const message = createFeedbackMessage(plan.text, agent.session.header.version)
+      state.accountedCorrectionMessageIds.add(message.id)
+      agent.steer(message)
     } catch (error) {
       ctx.logger?.warn?.(`dsh-genui: fence feedback steering failed (${error instanceof Error ? error.message : String(error)})`)
     }

package/src/plugin/index.ts

@@ -119,12 +119,13 @@ Rules:
 - JSON 严格：坏组件被丢弃，坏围栏变代码块；≥3 节点或含 table 时调 validate_dsh_ui，按诊断修改并重验；小围栏字段存疑也先验证。
 - warning=block_markdown：按 replacement 改写并重验。
 - 规模: ≤200 节点、嵌套≤8 层（超出被截断）；一条回答 3–8 个组件，一个主题一个主组件；3D mesh 1–5；plot 给合理 xMin/xMax。
-- LOCAL-FIRST + actions: UI 能自己做的状态变化（判卷、判题、重置、展开、选中）就地完成，零往返；action 只用于必须模型参与的事。交互组件带 "action":"name"，交互以 [genui-action] name + 组件数据回传，届时重渲染更新 UI；无 action 的按钮禁用。
-- Durable state: 交互状态按「会话+内容指纹」持久化——刷新/重放恢复；重渲染相同内容保留，新内容重置。
+- LOCAL-FIRST + actions: UI 能自己做的状态变化（判卷、判题、重置、展开、选中）就地完成，零往返；action 只用于必须模型参与的事。交互以 [genui-action] name + 组件数据回传，届时重渲染更新 UI；无 action 的按钮禁用。
+- Durable state: 交互状态按「会话+内容指纹」持久化——刷新/重放恢复；相同内容保留，新内容重置。
 - 卷子模式: 每题一个 radio（group+answer+explanation）+ 一个 submit（groups 全列），本地判分。
 - Secrets ban: 不索取密码、API Key、Token、恢复码；需要时拒绝并解释。
-- Tool channel: render_ui 工具把同一 spec 渲染为工具行卡片（交付物型界面用）；围栏用于回答内联 UI。
-- Panel: "panel":true 只渲染进会话面板 dock 并原地更新；"append":true 追加合并（同标签 tabs 追加/新标签加入/尾部追加）；上限 200 节点/200 次追加，满了发 replace 重建。面板组件来的 [genui-action] 只回一个 panel:true 围栏 + 至多一行 10 字内确认，不解释、不用普通围栏。`
+- Tool channel: render_ui 工具把同一 spec 渲染为工具行卡片；围栏用于回答内联 UI。
+- 围栏位置：\`dsh-ui\` 只写在**回答正文**；写在 reasoning/思考块里不渲染、用户看不到——思考里验证好 spec，正文再输出同一份。
+- Panel: "panel":true 只渲染进会话面板 dock 并原地更新；"append":true 追加合并；上限 200 节点/200 次追加，满了发 replace 重建。面板来的 [genui-action] 只回一个 panel:true 围栏 + 至多一行 10 字内确认。`
 
 /**
  * Register the GenUI output-language section and the render_ui tool.
@@ -184,8 +185,8 @@ function bundledSkillProvider(): SkillProvider {
 export interface GenuiPluginConfig {
   /**
    * 在最终 dsh-ui 围栏无法渲染的回合中请求模型发送一次修正版（issue #160）。
-   * 默认开启，设置为 false 可以关闭。每回合和每个围栏正文最多请求一次，子代理不触发，
-   * 每次请求会消耗模型步数。
+   * 默认开启，设置为 false 可以关闭同回合围栏修正。GenUI 回合的 reasoning-only 响应仍会交给宿主重试策略处理。
+   * 每回合和每个围栏正文最多请求一次，子代理不触发，每次修正请求会消耗模型步数。
    */
   fenceFeedback?: boolean
 }