dsh-context-compression-improved 0.5.1 → 0.5.3

package/packages/selector/src/client/EstimatorControls.tsx

@@ -275,104 +275,3 @@ export function EstimatorControls({ options, disabled, save, settle, t }: Estima
     </section>
   )
 }
-
-interface ReviewModeControlsProps {
-  options: PresetOptionsSettings
-  disabled: boolean
-  save: (options: PresetOptionsPatch) => Promise<void>
-  settle: (operation: () => Promise<void>) => void
-  t: (key: ContextCompressionLocaleKey) => string
-}
-
-/**
- * TokenPilot-inspired review-mode card (beta). When enabled, edge/high-impact
- * reduction candidates queue for manual approval and execute in one merged
- * batch at the next turn boundary; the numeric fields tune the benefit model.
- * Numeric drafts commit on blur and only when they parse to a value the
- * runtime schema accepts, so an invalid keystroke never disables the panel.
- */
-export function ReviewModeControls({ options, disabled, save, settle, t }: ReviewModeControlsProps) {
-  const [turnsDraft, setTurnsDraft] = useState(String(options.reviewTimeoutTurns ?? 6))
-  const [alphaDraft, setAlphaDraft] = useState(String(options.cacheHitDiscountAlpha ?? 0.1))
-  const [highImpactDraft, setHighImpactDraft] = useState(String(options.reviewHighImpactTokens ?? 4000))
-  const reviewMode = options.reviewMode ?? false
-  const commit = (patch: PresetOptionsPatch) => {
-    settle(() => save(patch))
-  }
-  const commitTurns = (): void => {
-    const next = Number(turnsDraft)
-    if (!Number.isSafeInteger(next) || next < 1 || next === (options.reviewTimeoutTurns ?? 6)) return
-    commit({ reviewTimeoutTurns: next })
-  }
-  const commitAlpha = (): void => {
-    const next = Number(alphaDraft)
-    if (!Number.isFinite(next) || next <= 0 || next >= 1 || next === (options.cacheHitDiscountAlpha ?? 0.1)) return
-    commit({ cacheHitDiscountAlpha: next })
-  }
-  const commitHighImpact = (): void => {
-    const next = Number(highImpactDraft)
-    if (!Number.isSafeInteger(next) || next < 0 || next === (options.reviewHighImpactTokens ?? 4000)) return
-    commit({ reviewHighImpactTokens: next })
-  }
-  return (
-    <section className={css.autoCompact} aria-labelledby="context-compression-review-title">
-      <h3 id="context-compression-review-title" className={css.autoCompactTitle}>{t('review.title')}</h3>
-      <p className={css.customNote}>{t('review.description')}</p>
-      <label className={css.field}>
-        <span>{t('review.enabled')}</span>
-        <select
-          value={reviewMode ? 'on' : 'off'}
-          disabled={disabled}
-          onChange={(event) => { settle(() => save({ reviewMode: event.currentTarget.value === 'on' })) }}
-        >
-          <option value="off">{t('review.enabled.off')}</option>
-          <option value="on">{t('review.enabled.on')}</option>
-        </select>
-      </label>
-      {reviewMode ? (
-        <>
-          <label className={css.field}>
-            <span>{t('review.timeoutTurns')}</span>
-            <input
-              type="number"
-              min={1}
-              step={1}
-              value={turnsDraft}
-              disabled={disabled}
-              onChange={(event) => { setTurnsDraft(event.currentTarget.value) }}
-              onBlur={commitTurns}
-              onKeyDown={(event) => { if (event.key === 'Enter') event.currentTarget.blur() }}
-            />
-          </label>
-          <label className={css.field}>
-            <span>{t('review.alpha')}</span>
-            <input
-              type="number"
-              min={0.01}
-              max={0.99}
-              step={0.05}
-              value={alphaDraft}
-              disabled={disabled}
-              onChange={(event) => { setAlphaDraft(event.currentTarget.value) }}
-              onBlur={commitAlpha}
-              onKeyDown={(event) => { if (event.key === 'Enter') event.currentTarget.blur() }}
-            />
-          </label>
-          <label className={css.field}>
-            <span>{t('review.highImpact')}</span>
-            <input
-              type="number"
-              min={0}
-              step={500}
-              value={highImpactDraft}
-              disabled={disabled}
-              onChange={(event) => { setHighImpactDraft(event.currentTarget.value) }}
-              onBlur={commitHighImpact}
-              onKeyDown={(event) => { if (event.key === 'Enter') event.currentTarget.blur() }}
-            />
-          </label>
-        </>
-      ) : null}
-    </section>
-  )
-}

package/packages/selector/src/client/index.ts

@@ -12,7 +12,6 @@ import { DEFAULT_CUSTOM_COMPRESSION_POLICY } from '../profiles.ts'
 import { decodeSettings } from './decode.ts'
 import { en, zh } from './locales.ts'
 import { planPresetOptionsOps, presetOptionsOpsAccepted } from './preset-options.ts'
-import { renderReviewOverlay } from './ReviewOverlay.tsx'
 
 /**
  * Harness 0.1.5 mounts the web core's `slots` service on the client context
@@ -142,22 +141,6 @@ export function apply(ctx: ClientContext): void {
   } catch (error) {
     console.warn('[dsh-context-compression-improved] settings.section 注册失败(新宿主已收编):', error)
   }
-
-  // TokenPilot-inspired R4：审查浮窗挂在 shell.overlay（dsh-tidychat 先例：
-  // 该层默认点击穿透，卡片自持指针事件）。reviewMode 关闭或无 pending 时组件
-  // 渲染 null —— 与 0.1.2 宿主（无此 slot）同构的降级语义：注册失败不影响设置卡。
-  // （本文件是 .ts：元素构造在 ReviewOverlay.renderReviewOverlay，不能内联 JSX。）
-  try {
-    ctx.slots.inject('shell.overlay', () => ctx.slots.register(
-      { name: 'shell.overlay', id: 'context-compression-review' },
-      () => {
-        const scope = ctx.settingsScope.bind<ContextCompressionSettings>({ namespace: NS, decode: decodeSettings })
-        return renderReviewOverlay(scope, ctx.locale.bind(NS) as (key: string) => string)
-      },
-    ))
-  } catch (error) {
-    console.warn('[dsh-context-compression-improved] shell.overlay 注册失败(宿主无浮层或已收编):', error)
-  }
 }
 
 export type {

package/packages/selector/src/client/locales.ts

@@ -35,25 +35,6 @@ export const zh = {
   'estimator.apiKey.set': '已设置 · 输入新值覆盖',
   'estimator.apiKey.clear': '清除',
   'estimator.apiKey.overwrite': '已设置保密值，输入新值并失焦即可覆盖。',
-  'review.title': '人工审查（beta）',
-  'review.description': '开启后，边缘区间与高影响的压缩候选不再自动执行，而是进入待审队列并在你批准后的下一个回合边界批量执行；未处理的提案超过过期轮数后自动作废。仅随「TokenPilot 启发模式」提供，默认关闭。',
-  'review.enabled': '审查模式',
-  'review.enabled.on': '开（提案等待人工批准）',
-  'review.enabled.off': '关（默认，全自动）',
-  'review.timeoutTurns': '提案过期轮数',
-  'review.alpha': '缓存命中折扣 α',
-  'review.highImpact': '高影响门槛（tokens）',
-  'review.badge': '待审',
-  'review.summary.autoApplied': '自动应用',
-  'review.summary.reviewApplied': '审查应用',
-  'review.summary.expired': '已过期',
-  'review.summary.voided': '已作废',
-  'review.row.payback': '回本轮数',
-  'review.row.expectedSaving': '预期节省',
-  'review.row.estimated': '估计',
-  'review.action.approve': '批准',
-  'review.action.reject': '驳回',
-  'review.action.ignore': '本会话忽略',
   'detail.tokenpilot-inspired': '在平衡模式之上叠加去重指针、恢复豁免、摘要定位块、前缀稳定与读取状态语义；估计器需另行配置端点',
   'profile.custom': 'Custom／实验模式',
   'profile.native': '原生对照',
@@ -154,25 +135,6 @@ export const en = {
   'estimator.apiKey.set': 'Set · type a new value to overwrite',
   'estimator.apiKey.clear': 'Clear',
   'estimator.apiKey.overwrite': 'A secret is stored; type a new value and blur to overwrite it.',
-  'review.title': 'Review mode (beta)',
-  'review.description': 'When enabled, edge-band and high-impact reduction candidates no longer apply automatically: they queue for manual approval and execute in one merged batch at the next turn boundary after approval. Unhandled proposals expire after the configured number of turns. Ships with the TokenPilot-inspired profile only, off by default.',
-  'review.enabled': 'Review mode',
-  'review.enabled.on': 'On (proposals wait for manual approval)',
-  'review.enabled.off': 'Off (default, fully automatic)',
-  'review.timeoutTurns': 'Proposal expiry (turns)',
-  'review.alpha': 'Cache-hit discount α',
-  'review.highImpact': 'High-impact threshold (tokens)',
-  'review.badge': 'Review',
-  'review.summary.autoApplied': 'Auto-applied',
-  'review.summary.reviewApplied': 'Review-applied',
-  'review.summary.expired': 'Expired',
-  'review.summary.voided': 'Voided',
-  'review.row.payback': 'Payback',
-  'review.row.expectedSaving': 'Expected saving',
-  'review.row.estimated': 'estimated',
-  'review.action.approve': 'Approve',
-  'review.action.reject': 'Reject',
-  'review.action.ignore': 'Ignore',
   'detail.tokenpilot-inspired': 'Layered on Balanced: dedupe pointers, recovery exemption, summary locators, prefix stabilization, and read-state semantics; the estimator needs an endpoint configured separately',
   'profile.custom': 'Custom / Experimental',
   'profile.native': 'Native baseline',

package/packages/selector/src/client/preset-options.ts

@@ -20,11 +20,12 @@ import type { PresetOptionsSettings } from '../profiles.ts'
 /** The namespace key holding every tokenpilot-inspired sub-capability override. */
 const PRESET_OPTIONS_KEY = 'presetOptions'
 
-/** Every field a patch may address, in the schema's own order. */
+/** Every field a patch may address, in the schema's own order. The retired
+ *  review-gate keys are deliberately absent: nothing writes them any more, and
+ *  a stored document that still carries them is tolerated by the decoders. */
 const PRESET_OPTION_KEYS = [
   'dedupeToolResults', 'summaryLocator', 'prefixStabilizer', 'readState', 'estimatorMode',
   'estimatorProvider', 'estimatorModel', 'estimatorBaseUrl', 'estimatorApiKey', 'estimatorTimeoutMs',
-  'reviewMode', 'reviewTimeoutTurns', 'cacheHitDiscountAlpha', 'reviewHighImpactTokens',
   'advisorMode', 'advisorTimeoutMs', 'advisorRefreshTurns', 'advisorScoreThreshold', 'advisorSampleLimit',
   'advisorMinTokens',
 ] as const

package/packages/selector/src/client/settings-section.tsx

@@ -16,7 +16,7 @@ import {
 } from '../profiles.ts'
 import type { CompressionProfileSelectorProps } from './CompressionProfileSelector.tsx'
 import { AutoCompactThresholdControls, CodeSkeletonControls } from './CompressionProfileControls.tsx'
-import { EstimatorControls, EstimatorInactiveNotice, ReviewModeControls } from './EstimatorControls.tsx'
+import { EstimatorControls, EstimatorInactiveNotice } from './EstimatorControls.tsx'
 import { CustomPolicyEditor, editableCustom } from './CustomPolicyEditor.tsx'
 
 /** Full-page Settings surface backed by the same durable selector state. */
@@ -105,22 +105,13 @@ export function SettingsCompressionProfileControls({
       {current !== 'tokenpilot-inspired' ? (
         <EstimatorInactiveNotice profile={t(`profile.${current}`)} t={t} />
       ) : (
-        <>
-          <EstimatorControls
-            options={state.value?.presetOptions ?? {}}
-            disabled={busy || !state.writable || !selectorAvailable}
-            save={savePresetOptions}
-            settle={settle}
-            t={t}
-          />
-          <ReviewModeControls
-            options={state.value?.presetOptions ?? {}}
-            disabled={busy || !state.writable || !selectorAvailable}
-            save={savePresetOptions}
-            settle={settle}
-            t={t}
-          />
-        </>
+        <EstimatorControls
+          options={state.value?.presetOptions ?? {}}
+          disabled={busy || !state.writable || !selectorAvailable}
+          save={savePresetOptions}
+          settle={settle}
+          t={t}
+        />
       )}
       <div className={css.pricing}>{t('pricing.disclosure')}</div>
       {current !== 'custom' || draft === null || !selectorAvailable ? null : (

package/packages/selector/src/index.ts

@@ -11,7 +11,7 @@ import {
   CONTEXT_COMPRESSION_SETTINGS_NAMESPACE,
   ContextCompressionSettingsSchema,
 } from './runtime/config.ts'
-import { resolveReviewPruner, type ReviewPrunerFace } from './runtime/tokenpilot/review-registry.ts'
+
 import { getAdvisorState } from './runtime/tokenpilot/advisor-state.ts'
 
 // The settings namespace literal and the settings schema are owned by the
@@ -35,62 +35,23 @@ const ESTIMATOR_CATALOG_ROUTES = [
   '/api/dsh-context-compression-improved/estimator-catalog',
 ] as const
 
-// TokenPilot-inspired R4: the review pipeline's client↔runtime channel, dual
-// prefixed like the catalog route so 0.1.2 clients keep working.
-const REVIEW_QUEUE_ROUTES = [
-  '/endpoint/dsh-context-compression-improved/review-queue',
-  '/api/dsh-context-compression-improved/review-queue',
-] as const
-const REVIEW_DECIDE_ROUTES = [
-  '/endpoint/dsh-context-compression-improved/review-decide',
-  '/api/dsh-context-compression-improved/review-decide',
-] as const
-
 // Advisory advisor: one read-only report route, dual prefixed like the others.
 const ADVISOR_REPORT_ROUTES = [
   '/endpoint/dsh-context-compression-improved/advisor-report',
   '/api/dsh-context-compression-improved/advisor-report',
 ] as const
 
-/**
- * The review faces of the pruner service the routes consume. Owned by the
- * review registry, which the routes also fall back to when no top-level
- * `toolResultPruner` service exists — the preset-scoped case in production.
- */
-type ReviewPrunerLike = ReviewPrunerFace
-
 /** Minimal face of the agents service: session id → agent (carrying the session). */
 interface AgentsServiceLike {
   get?(id: unknown): { session?: unknown } | undefined
 }
 
-/**
- * Resolve the review pipeline for the top-level routes.
- *
- * A top-level `toolResultPruner` service wins when a deployment actually mounts
- * one, but in production every pruner lives inside an agent preset's isolated
- * group, so the registry is the path that resolves. Without the fallback the
- * queue route answered 503 "review pipeline unavailable" on every request while
- * the review pipeline itself was running normally.
- */
-function reviewPrunerOf(readService: (name: string) => unknown): ReviewPrunerLike | undefined {
-  const candidate = readService('toolResultPruner') as {
-    listReviewProposals?: unknown
-    decideReviewProposal?: unknown
-  } | undefined
-  if (typeof candidate?.listReviewProposals === 'function'
-    && typeof candidate?.decideReviewProposal === 'function') {
-    return candidate as unknown as ReviewPrunerLike
-  }
-  return resolveReviewPruner()
-}
-
 function sessionFor(readService: (name: string) => unknown, sessionId: string): unknown {
   const agents = readService('agents') as AgentsServiceLike | undefined
   return typeof agents?.get === 'function' ? agents.get(sessionId)?.session : undefined
 }
 
-function reviewJson(res: unknown, status: number, body: unknown): void {
+function jsonResponse(res: unknown, status: number, body: unknown): void {
   const resTyped = res as {
     writeHead: (code: number, headers?: Record<string, string>) => void
     end: (body?: string) => void
@@ -99,218 +60,9 @@ function reviewJson(res: unknown, status: number, body: unknown): void {
   resTyped.end(JSON.stringify(body))
 }
 
-function readRequestBody(req: unknown): Promise<string> {
-  return new Promise((resolve, reject) => {
-    const typed = req as {
-      on?: (event: string, listener: (chunk?: Buffer) => void) => void
-    }
-    let data = ''
-    try {
-      typed.on?.('data', chunk => {
-        data += String(chunk ?? '')
-        if (data.length > 64 * 1024) {
-          data = ''
-          resolve('')
-        }
-      })
-      typed.on?.('end', () => resolve(data))
-      typed.on?.('error', reject)
-    } catch (error) {
-      reject(error instanceof Error ? error : new Error(String(error)))
-    }
-  })
-}
-
-/**
- * Serve the review pipeline's two HTTP routes (best effort, mirroring the
- * estimator-catalog registration):
- *
- * - `GET .../review-queue?sessionId=…` → the session's pending proposals with
- *   their benefit numbers. Sanitized by construction: the queue never holds
- *   message content, and the response carries ids/seqs/counts only (digests
- *   stay in the runtime — the client cannot need them).
- * - `POST .../review-decide` `{sessionId, proposalId, decision}` → one human
- *   decision. Invalid body → 400; unknown/not-pending proposal → 404; review
- *   mode off for the session → 503.
- */
-function registerReviewQueueRoutes(ctx: Context): void {
-  const readService = (name: string): unknown => {
-    try {
-      return (ctx as unknown as { get: (service: string) => unknown }).get(name)
-    } catch {
-      return undefined
-    }
-  }
-  const log = (level: 'info' | 'warn', message: string, ...args: unknown[]): void => {
-    console[level](message, ...args)
-  }
-  const registered = (): void => {
-    log('info', 'context-compression review queue routes registered: %s / %s', REVIEW_QUEUE_ROUTES.join(', '), REVIEW_DECIDE_ROUTES.join(', '))
-  }
-
-  type SanitizedProposal = {
-    sessionId: string
-    id: string
-    kind: string
-    items: readonly { seq: number, kind: string, component: string, tokensBefore: number, tokensAfter: number }[]
-    benefit: {
-      recoveredTokens: number
-      penaltyTokens: number
-      paybackTurns?: number
-      expectedSaving?: number
-    }
-    enqueuedTurn: number
-    lastTurnIndex: number
-  }
-
-  const getHandler = (req: unknown, res: unknown): void => {
-    const pruner = reviewPrunerOf(readService)
-    if (pruner === undefined || pruner.listAllReviewProposals === undefined) {
-      reviewJson(res, 503, { ok: false, error: 'review pipeline unavailable' })
-      return
-    }
-    let sessionId = ''
-    try {
-      const url = new URL(String((req as { url?: string }).url ?? ''), 'http://localhost')
-      sessionId = url.searchParams.get('sessionId') ?? ''
-    } catch {
-      sessionId = ''
-    }
-    const sanitize = (
-      sid: string,
-      proposal: {
-        id: string
-        kind: string
-        items: readonly { seq: number, kind: string, component: string, tokensBefore: number, tokensAfter: number }[]
-        benefit: { recoveredTokens: number, penaltyTokens: number, paybackTurns?: number, expectedSaving?: number }
-        enqueuedTurn: number
-        lastTurnIndex: number
-      },
-    ): SanitizedProposal => ({
-      sessionId: sid,
-      id: proposal.id,
-      kind: proposal.kind,
-      items: proposal.items.map(item => ({
-        seq: item.seq,
-        kind: item.kind,
-        component: item.component,
-        tokensBefore: item.tokensBefore,
-        tokensAfter: item.tokensAfter,
-      })),
-      benefit: {
-        recoveredTokens: proposal.benefit.recoveredTokens,
-        penaltyTokens: proposal.benefit.penaltyTokens,
-        ...(proposal.benefit.paybackTurns === undefined ? {} : { paybackTurns: proposal.benefit.paybackTurns }),
-        ...(proposal.benefit.expectedSaving === undefined ? {} : { expectedSaving: proposal.benefit.expectedSaving }),
-      },
-      enqueuedTurn: proposal.enqueuedTurn,
-      lastTurnIndex: proposal.lastTurnIndex,
-    })
-
-    // Without a sessionId the read aggregates every session with live
-    // proposals — the client carries no session id of its own.
-    if (sessionId === '') {
-      const pending = pruner.listAllReviewProposals()
-        .flatMap(entry => entry.proposals.map(proposal => sanitize(entry.sessionId, proposal)))
-      reviewJson(res, 200, { ok: true, total: pending.length, pending })
-      return
-    }
-
-    const session = sessionFor(readService, sessionId)
-    if (session === undefined) {
-      reviewJson(res, 404, { ok: false, error: 'unknown session' })
-      return
-    }
-    const pending = pruner.listReviewProposals(session).map(proposal => sanitize(sessionId, proposal))
-    const summary = pruner.reviewSummary?.(session)
-    reviewJson(res, 200, {
-      ok: true,
-      sessionId,
-      total: pending.length,
-      pending,
-      ...(summary === undefined ? {} : { summary }),
-    })
-  }
-
-  const decideHandler = async (req: unknown, res: unknown): Promise<void> => {
-    const pruner = reviewPrunerOf(readService)
-    if (pruner === undefined) {
-      reviewJson(res, 503, { ok: false, error: 'review pipeline unavailable' })
-      return
-    }
-    let body: unknown
-    try {
-      body = JSON.parse(await readRequestBody(req))
-    } catch {
-      body = undefined
-    }
-    if (typeof body !== 'object' || body === null) {
-      reviewJson(res, 400, { ok: false, error: 'invalid JSON body' })
-      return
-    }
-    const record = body as { sessionId?: unknown, proposalId?: unknown, decision?: unknown }
-    if (typeof record.sessionId !== 'string' || record.sessionId === ''
-      || typeof record.proposalId !== 'string' || record.proposalId === '') {
-      reviewJson(res, 400, { ok: false, error: 'sessionId and proposalId are required' })
-      return
-    }
-    if (record.decision !== 'approved' && record.decision !== 'rejected' && record.decision !== 'ignored') {
-      reviewJson(res, 400, { ok: false, error: 'decision must be approved, rejected, or ignored' })
-      return
-    }
-    const session = sessionFor(readService, record.sessionId)
-    if (session === undefined) {
-      reviewJson(res, 404, { ok: false, error: 'unknown session' })
-      return
-    }
-    const outcome = pruner.decideReviewProposal(session, record.proposalId, record.decision)
-    if (outcome === undefined) {
-      reviewJson(res, 503, { ok: false, error: 'review mode is off for this session' })
-      return
-    }
-    if (!outcome.ok) {
-      reviewJson(res, 404, { ok: false, error: outcome.reason })
-      return
-    }
-    reviewJson(res, 200, { ok: true, sessionId: record.sessionId, proposalId: record.proposalId, decision: record.decision })
-  }
-
-  const register = (webServer: WebServerLike): void => {
-    const table: ReadonlyArray<{ path: string, handler: (req: unknown, res: unknown) => unknown }> = [
-      ...[...REVIEW_QUEUE_ROUTES].map(path => ({ path, handler: getHandler })),
-      ...[...REVIEW_DECIDE_ROUTES].map(path => ({ path, handler: (req: unknown, res: unknown) => { void decideHandler(req, res) } })),
-    ]
-    const disposers = table
-      .map(entry => webServer.register({ kind: 'exact', path: entry.path, handler: entry.handler }))
-      .filter((off): off is () => void => typeof off === 'function')
-    ctx.effect(
-      () => () => { for (const off of disposers) off() },
-      'contextCompressionSelector.review routes',
-    )
-    registered()
-  }
-
-  const active = asWebServer(readService('webServer'))
-  if (active !== undefined) {
-    register(active)
-    return
-  }
-
-  ctx.inject(['webServer'], (injected) => {
-    const webServer = asWebServer((injected as { webServer?: unknown }).webServer)
-    if (webServer === undefined) {
-      log('warn', 'context-compression webServer exposes no register() — review routes not registered')
-      return
-    }
-    register(webServer)
-  })
-
-  log('warn', 'context-compression webServer not active yet — review routes pending: %s', REVIEW_QUEUE_ROUTES.join(', '))
-}
-
 /**
  * Serve the advisory advisor's read-only report route (same registration
- * skeleton as the review routes):
+ * skeleton as the estimator-catalog route):
  *
  * `GET .../advisor-report?sessionId=…` → the session's prefix-decay figure,
  * the todolist-bound task summary, and the score distribution. Content-free
@@ -334,7 +86,7 @@ function registerAdvisorReportRoute(ctx: Context): void {
   const getHandler = (req: unknown, res: unknown): void => {
     const agents = readService('agents') as AgentsServiceLike | undefined
     if (typeof agents?.get !== 'function') {
-      reviewJson(res, 503, { ok: false, error: 'advisor report unavailable' })
+      jsonResponse(res, 503, { ok: false, error: 'advisor report unavailable' })
       return
     }
     let sessionId = ''
@@ -345,16 +97,16 @@ function registerAdvisorReportRoute(ctx: Context): void {
       // Malformed URL: fall through with the empty sessionId already set.
     }
     if (sessionId === '') {
-      reviewJson(res, 400, { ok: false, error: 'sessionId is required' })
+      jsonResponse(res, 400, { ok: false, error: 'sessionId is required' })
       return
     }
     const session = sessionFor(readService, sessionId)
     if (session === undefined) {
-      reviewJson(res, 404, { ok: false, error: 'unknown session' })
+      jsonResponse(res, 404, { ok: false, error: 'unknown session' })
       return
     }
     const state = getAdvisorState(session as Parameters<typeof getAdvisorState>[0])
-    reviewJson(res, 200, {
+    jsonResponse(res, 200, {
       ok: true,
       sessionId,
       advisor: {
@@ -364,6 +116,18 @@ function registerAdvisorReportRoute(ctx: Context): void {
         decayTurn: state.lastDecay?.turn ?? null,
         scores: [...state.scores].map(([seq, entry]) => ({ seq, score: entry.score, turn: entry.turn })),
         lowRelevanceSeqs: [...state.recertified.keys()],
+        // The benefit model's label of the last landed batch — advice, never a
+        // gate: the batch it describes landed regardless of the band.
+        lastAdvice: state.lastAdvice === undefined
+          ? null
+          : {
+              band: state.lastAdvice.band,
+              turn: state.lastAdvice.turn,
+              itemSeqs: state.lastAdvice.itemSeqs,
+              recoveredTokens: state.lastAdvice.recoveredTokens,
+              penaltyTokens: state.lastAdvice.penaltyTokens,
+              paybackTurns: state.lastAdvice.paybackTurns ?? null,
+            },
       },
     })
   }
@@ -574,18 +338,12 @@ export interface Config {
    * covers both arrival orders.
    */
   estimatorCatalogRoute?: boolean
-  /**
-   * Register the review pipeline's HTTP routes (pending-queue read + decide
-   * write) on this row. Same Bundle opt-in semantics as
-   * `estimatorCatalogRoute`; without the routes the floating window has no
-   * transport and simply never appears.
-   */
-  reviewQueueRoute?: boolean
   /**
    * Register the advisory advisor's read-only HTTP report route (decay
-   * figure, task summary, score distribution). Same Bundle opt-in semantics
-   * as `reviewQueueRoute`; the advisor itself stays off until the user turns
-   * it on through the `presetOptions.advisor*` settings keys.
+   * figure, task summary, score distribution, last benefit-model advice).
+   * Same Bundle opt-in semantics as `estimatorCatalogRoute`; the advisor
+   * itself stays off until the user turns it on through the
+   * `presetOptions.advisor*` settings keys.
    */
   advisorReportRoute?: boolean
 }
@@ -594,7 +352,6 @@ export interface Config {
 export const Config: z<Config> = z.object({
   presetOverlay: z.boolean().default(false),
   estimatorCatalogRoute: z.boolean().default(false),
-  reviewQueueRoute: z.boolean().default(false),
   advisorReportRoute: z.boolean().default(false),
 })
 
@@ -615,11 +372,7 @@ export function apply(ctx: Context, config: Config = {}): void {
     // service, because the route is host-wide rather than per-pruner-instance.
     if (config.estimatorCatalogRoute === true) registerEstimatorCatalogRoute(ctx)
 
-    // TokenPilot-inspired R4: the review pipeline's client transport (see the
-    // config JSDoc for the opt-in semantics).
-    if (config.reviewQueueRoute === true) registerReviewQueueRoutes(ctx)
-
-    // Advisory advisor: read-only decay/score report (opt-in, like review).
+    // Advisory advisor: read-only decay/score/advice report (opt-in).
     if (config.advisorReportRoute === true) registerAdvisorReportRoute(ctx)
 
     if (config.presetOverlay !== true) return