dsh-context-compression-improved 0.5.1 → 0.5.3

package/packages/selector/src/profiles.ts

@@ -135,11 +135,6 @@ export interface PresetOptionsSettings {
   readonly estimatorBaseUrl?: string
   readonly estimatorApiKey?: string
   readonly estimatorTimeoutMs?: number
-  /** Review-mode overrides (beta); mirrors the runtime PresetOptions.reviewMode. */
-  readonly reviewMode?: boolean
-  readonly reviewTimeoutTurns?: number
-  readonly cacheHitDiscountAlpha?: number
-  readonly reviewHighImpactTokens?: number
   /** Advisory advisor channel; `''` (the default) keeps the advisor off. */
   readonly advisorMode?: '' | 'host' | 'direct'
   readonly advisorTimeoutMs?: number
@@ -160,12 +155,15 @@ export function decodePresetOptionsSettings(value: unknown): PresetOptionsSettin
   const allowed = new Set([
     'dedupeToolResults', 'summaryLocator', 'prefixStabilizer', 'readState', 'estimatorMode',
     'estimatorProvider', 'estimatorModel', 'estimatorBaseUrl', 'estimatorApiKey', 'estimatorTimeoutMs',
+    // Legacy keys of the retired review gate: still ACCEPTED so an existing
+    // settings document keeps decoding (rejecting them would report the whole
+    // card unreadable), and ignored — nothing reads them any more.
     'reviewMode', 'reviewTimeoutTurns', 'cacheHitDiscountAlpha', 'reviewHighImpactTokens',
     'advisorMode', 'advisorTimeoutMs', 'advisorRefreshTurns', 'advisorScoreThreshold', 'advisorSampleLimit',
     'advisorMinTokens',
   ])
   if (Object.keys(value).some(key => !allowed.has(key))) return undefined
-  for (const key of ['dedupeToolResults', 'summaryLocator', 'prefixStabilizer', 'readState', 'reviewMode'] as const) {
+  for (const key of ['dedupeToolResults', 'summaryLocator', 'prefixStabilizer', 'readState'] as const) {
     const entry = value[key]
     if (entry !== undefined && typeof entry !== 'boolean') return undefined
   }
@@ -187,23 +185,6 @@ export function decodePresetOptionsSettings(value: unknown): PresetOptionsSettin
       || estimatorTimeoutMs < 100 || estimatorTimeoutMs > 60_000)) {
     return undefined
   }
-  const reviewTimeoutTurns = value.reviewTimeoutTurns
-  if (reviewTimeoutTurns !== undefined
-    && (typeof reviewTimeoutTurns !== 'number' || !Number.isSafeInteger(reviewTimeoutTurns) || reviewTimeoutTurns < 1)) {
-    return undefined
-  }
-  const cacheHitDiscountAlpha = value.cacheHitDiscountAlpha
-  if (cacheHitDiscountAlpha !== undefined
-    && (typeof cacheHitDiscountAlpha !== 'number' || !Number.isFinite(cacheHitDiscountAlpha)
-      || cacheHitDiscountAlpha <= 0 || cacheHitDiscountAlpha >= 1)) {
-    return undefined
-  }
-  const reviewHighImpactTokens = value.reviewHighImpactTokens
-  if (reviewHighImpactTokens !== undefined
-    && (typeof reviewHighImpactTokens !== 'number' || !Number.isSafeInteger(reviewHighImpactTokens)
-      || reviewHighImpactTokens < 0)) {
-    return undefined
-  }
   const advisorTimeoutMs = value.advisorTimeoutMs
   if (advisorTimeoutMs !== undefined
     && (typeof advisorTimeoutMs !== 'number' || !Number.isSafeInteger(advisorTimeoutMs)
@@ -246,10 +227,6 @@ export function decodePresetOptionsSettings(value: unknown): PresetOptionsSettin
   if (value.estimatorBaseUrl !== undefined) decoded.estimatorBaseUrl = value.estimatorBaseUrl as string
   if (value.estimatorApiKey !== undefined) decoded.estimatorApiKey = value.estimatorApiKey as string
   if (estimatorTimeoutMs !== undefined) decoded.estimatorTimeoutMs = estimatorTimeoutMs as number
-  if (value.reviewMode !== undefined) decoded.reviewMode = value.reviewMode as boolean
-  if (reviewTimeoutTurns !== undefined) decoded.reviewTimeoutTurns = reviewTimeoutTurns as number
-  if (cacheHitDiscountAlpha !== undefined) decoded.cacheHitDiscountAlpha = cacheHitDiscountAlpha as number
-  if (reviewHighImpactTokens !== undefined) decoded.reviewHighImpactTokens = reviewHighImpactTokens as number
   if (advisorMode !== undefined) decoded.advisorMode = advisorMode as '' | 'host' | 'direct'
   if (advisorTimeoutMs !== undefined) decoded.advisorTimeoutMs = advisorTimeoutMs as number
   if (advisorRefreshTurns !== undefined) decoded.advisorRefreshTurns = advisorRefreshTurns as number

package/packages/selector/src/pruner/state.ts

@@ -8,25 +8,12 @@
 import type { Session } from '@deepseek-ai/dsh-session'
 import type { DedupeTable } from '../runtime/tokenpilot/dedup.ts'
 import type { EstimatorFailures } from '../runtime/tokenpilot/estimator.ts'
-import type { ReviewQueue, ReviewQueueStore } from '../runtime/tokenpilot/review-queue.ts'
 import type { SideChannel } from '../runtime/tokenpilot/sidechannel.ts'
 import type {
   ContextCompressionSettings,
   ResolvedConfig,
 } from '../runtime/types.ts'
 
-/** Four-state per-session outcome counters behind the floating-window summary row. */
-export interface ReviewSessionSummary {
-  /** Rewrites that landed through the automatic path while review mode served this session. */
-  autoApplied: number
-  /** Approved proposals whose merged batch executed with an applied receipt. */
-  reviewApplied: number
-  /** Pending proposals that expired unhandled at a turn boundary. */
-  expired: number
-  /** Approved proposals voided at the apply point (digest mismatch et al). */
-  voided: number
-}
-
 /** Mutable per-session state bag used inside {@link ToolResultPruner}. */
 export interface PrunerState {
   /** Resolved immutable deployment configuration. */
@@ -54,20 +41,10 @@ export interface PrunerState {
   readonly tailTrimBoundaryAttempts: WeakMap<Session, object>
   /** Last effective policy audit key emitted for each Session. */
   readonly policyResolutionAudits: WeakMap<Session, string>
-  /**
-   * TokenPilot-inspired R4: shared review-queue store. Starts as the in-memory
-   * fail-open fallback; swapped to the storageDomain-backed adapter when (and
-   * if) that seam opens successfully.
-   */
-  reviewStore: ReviewQueueStore
-  /** Per-session review queue carrying the frozen timeout policy. */
-  readonly reviewQueues: WeakMap<Session, ReviewQueue>
-  /** Last observed turn index per Session: the monotonic clock for review expiries. */
-  readonly reviewClocks: WeakMap<Session, number>
+  /** Last observed turn index per Session: the monotonic clock for advisory records. */
+  readonly turnClocks: WeakMap<Session, number>
   /** Estimator-reported remaining turns Ŝ per Session; advisory only. */
   readonly estimatorRemainingTurns: WeakMap<Session, number>
   /** Per-session advisor side channel, constructed once with the advisor overrides. */
   readonly advisorChannels: WeakMap<Session, SideChannel>
-  /** Four-state outcome counters per Session (floating-window summary row). */
-  readonly reviewSummaries: WeakMap<Session, ReviewSessionSummary>
 }