dsh-context-compression-improved 0.2.0 → 0.3.0

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

@@ -1,5 +1,6 @@
 import type {} from '@deepseek-ai/dsh-client-locale/client'
-import type { ClientContext } from '@deepseek-ai/dsh-client-runtime/client'
+import type { Context as ClientContext } from '@deepseek-ai/cordis'
+import type {} from '@deepseek-ai/dsh-client-ui-session/client'
 import type {} from '@deepseek-ai/dsh-client-ui-settings/client'
 import type {} from '@deepseek-ai/dsh-client-ui-workspace/client'
 import {
@@ -10,8 +11,33 @@ import {
 import { DEFAULT_CUSTOM_COMPRESSION_POLICY } from '../profiles.ts'
 import { decodeSettings } from './decode.ts'
 import { en, zh } from './locales.ts'
-import { mergePresetOptionsPatch, presetOptionsEqual } from './preset-options.ts'
+import { planPresetOptionsOps, presetOptionsOpsAccepted } from './preset-options.ts'
 
+/**
+ * Harness 0.1.5 mounts the web core's `slots` service on the client context
+ * but no longer ships a public type for it; declare the face this plugin
+ * consumes (same shape dsh-thinking-levels and dsh-prime-memory rely on).
+ * `inject` factories may be sync (return a disposer) or generator (yield the
+ * registration); slot options are validated at runtime per slot kind, which
+ * lets one entry supply both `id` (Desktop, list) and `key` (CLI, keyed).
+ */
+interface SlotsService {
+  inject(slot: string, register: () => (() => void) | Generator<() => void>): () => void
+  register(options: Record<string, unknown>, component: unknown): () => void
+}
+declare module '@deepseek-ai/cordis' {
+  interface Context {
+    slots: SlotsService
+  }
+}
+
+// Declare all consumed services (the official client-plugin pattern, and the
+// shape dsh-thinking-levels proves on this same 0.1.5 host line): cordis holds
+// apply until `locale` / `settingsScope` are provided, so registration can use
+// them directly. Resolving them lazily via ctx.get() instead races the settings
+// client's activation — on a loss the apply early-returned and EVERY settings
+// entry (the standalone section, the plugins-tab card, the item card) silently
+// vanished.
 export const inject = ['slots', 'locale', 'settingsScope']
 const NS = 'context-compression'
 
@@ -40,69 +66,81 @@ function sameCustomPolicy(
 }
 
 export function apply(ctx: ClientContext): void {
-  ctx.effect(() => ctx.locale.register(NS, { zh, en }), 'ui-context-compression: dictionaries')
-  const scope = ctx.settingsScope.bind<ContextCompressionSettings>({ namespace: NS, decode: decodeSettings })
-  const writeAndConfirm = async (
-    write: () => Promise<void>,
-    accepts: (settings: ContextCompressionSettings) => boolean,
-  ): Promise<void> => {
-    const beforeRevision = scope.getSnapshot().revision
-    await write()
-    const after = scope.getSnapshot()
-    if (
-      after.status !== 'ready'
-      || after.value === undefined
-      || after.revision === beforeRevision
-      || !accepts(after.value)
-    ) {
-      throw new Error('Context compression settings were not saved.')
+  ctx.locale.register(NS, { zh, en })
+  const injected = (): CompressionSelectorInjected => {
+    // Bind per factory call on the caller's fiber (0.1.5: activation must
+    // never block on the settings transport; the scope disposer belongs to
+    // the calling registration's lifecycle).
+    const scope = ctx.settingsScope.bind<ContextCompressionSettings>({ namespace: NS, decode: decodeSettings })
+    const writeAndConfirm = async (
+      write: () => Promise<void>,
+      accepts: (settings: ContextCompressionSettings) => boolean,
+    ): Promise<void> => {
+      const beforeRevision = scope.getSnapshot().revision
+      await write()
+      const after = scope.getSnapshot()
+      if (
+        after.status !== 'ready'
+        || after.value === undefined
+        || after.revision === beforeRevision
+        || !accepts(after.value)
+      ) {
+        throw new Error('Context compression settings were not saved.')
+      }
+    }
+    return {
+      hooks: { compression: scope },
+      select: profile => writeAndConfirm(
+        () => scope.set('profile', profile),
+        settings => settings.profile === profile,
+      ),
+      saveCustom: custom => writeAndConfirm(
+        () => scope.set('custom', custom),
+        settings => isCustomCompressionPolicy(settings.custom)
+          && sameCustomPolicy(settings.custom, custom),
+      ),
+      resetCustom: () => writeAndConfirm(
+        () => scope.set('custom', structuredClone(DEFAULT_CUSTOM_COMPRESSION_POLICY)),
+        settings => isCustomCompressionPolicy(settings.custom)
+          && sameCustomPolicy(settings.custom, DEFAULT_CUSTOM_COMPRESSION_POLICY),
+      ),
+      saveAutoCompact: thresholdPercent => writeAndConfirm(
+        () => scope.set('autoCompact', { thresholdPercent }),
+        settings => settings.autoCompact.thresholdPercent === thresholdPercent,
+      ),
+      saveCodeSkeleton: enabled => writeAndConfirm(
+        () => scope.set('codeSkeleton', { enabled }),
+        settings => settings.codeSkeleton.enabled === enabled,
+      ),
+      savePresetOptions: options => {
+        // Path-addressed so one field write cannot delete its siblings: the
+        // previous whole-section set erased estimatorMode (and every other
+        // override) whenever the user touched a second field.
+        const ops = planPresetOptionsOps(scope.getSnapshot().value?.presetOptions, options)
+        if (ops.length === 0) return Promise.resolve()
+        return writeAndConfirm(
+          () => scope.mutate(ops),
+          settings => presetOptionsOpsAccepted(settings.presetOptions, ops),
+        )
+      },
     }
   }
-  const injected = (): CompressionSelectorInjected => ({
-    hooks: { compression: scope },
-    select: profile => writeAndConfirm(
-      () => scope.set('profile', profile),
-      settings => settings.profile === profile,
-    ),
-    saveCustom: custom => writeAndConfirm(
-      () => scope.set('custom', custom),
-      settings => isCustomCompressionPolicy(settings.custom)
-        && sameCustomPolicy(settings.custom, custom),
-    ),
-    resetCustom: () => writeAndConfirm(
-      () => scope.set('custom', structuredClone(DEFAULT_CUSTOM_COMPRESSION_POLICY)),
-      settings => isCustomCompressionPolicy(settings.custom)
-        && sameCustomPolicy(settings.custom, DEFAULT_CUSTOM_COMPRESSION_POLICY),
-    ),
-    saveAutoCompact: thresholdPercent => writeAndConfirm(
-      () => scope.set('autoCompact', { thresholdPercent }),
-      settings => settings.autoCompact.thresholdPercent === thresholdPercent,
-    ),
-    saveCodeSkeleton: enabled => writeAndConfirm(
-      () => scope.set('codeSkeleton', { enabled }),
-      settings => settings.codeSkeleton.enabled === enabled,
-    ),
-    savePresetOptions: options => {
-      // The section root is replaced by whatever is written there, so merge the
-      // patch over the stored document: writing the bare patch deleted
-      // estimatorMode (and every other override) on the next field edit.
-      const current = scope.getSnapshot().value?.presetOptions
-      const next = mergePresetOptionsPatch(current, options)
-      if (presetOptionsEqual(current, next)) return Promise.resolve()
-      return writeAndConfirm(
-        () => scope.set('presetOptions', next),
-        settings => presetOptionsEqual(settings.presetOptions, next),
-      )
-    },
-  })
-  ctx.slots.inject('settings.section', () => ctx.slots.register({
-    name: 'settings.section',
-    id: 'context-compression',
-    order: 17,
-    label: () => ctx.locale.bind(NS)('nav'),
-    locale: NS,
-    inject: injected,
-  }, ContextCompressionSettingsSection))
+  // 设置 → 上下文压缩 直挂分节（0.1.1 契约；0.1.5 官方分节也注册在此，未声明槽
+  // 的注册会在激活期抛错，故 try/catch 守卫 —— 同 dsh-prime-memory 的双槽冗余）。
+  // 只挂这一处：再注册 `settings.plugins.tab` / `settings.plugin.item` 会在
+  // 设置里同时出现独立分节和插件卡片，重复展示同一张面板。
+  try {
+    ctx.slots.inject('settings.section', () => ctx.slots.register({
+      name: 'settings.section',
+      id: 'context-compression',
+      order: 17,
+      label: () => ctx.locale.bind(NS)('nav'),
+      locale: NS,
+      inject: injected,
+    }, ContextCompressionSettingsSection))
+  } catch (error) {
+    console.warn('[dsh-context-compression-improved] settings.section 注册失败(新宿主已收编):', error)
+  }
 }
 
 export type {

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

@@ -19,7 +19,6 @@ export const zh = {
   'estimator.description': '辅助小模型零样本判断旧文件读取是否仍有引用价值，仅建议性加速历史清理；未配置或失败时自动退回纯规则通道，不影响主流程。',
   'estimator.mode': '通道',
   'estimator.mode.off': '关闭（纯规则）',
-  'estimator.followHost': '跟随宿主默认模型',
   'estimator.mode.host': '宿主模型（复用已配置供应商）',
   'estimator.mode.direct': '直连 OpenAI 兼容端点',
   'estimator.inactive': '估计器只随「TokenPilot 启发模式」提供：该模式的预设选项（去重指针、摘要定位块、读取状态语义与估计器通道）不会合并进其他 Profile，因此当前 Profile「{profile}」下没有可配置的估计器通道。选择「TokenPilot 启发模式」后，本区块会出现「通道」选择，可复用已配置供应商（宿主模型）或直连 OpenAI 兼容端点。',
@@ -120,7 +119,6 @@ export const en = {
   'estimator.description': 'A small auxiliary model zero-shots whether old file reads are still likely referenced, advisory-only for history aging; unconfigured or failing endpoints fall back to rule-only behavior.',
   'estimator.mode': 'Channel',
   'estimator.mode.off': 'Off (rule-only)',
-  'estimator.followHost': 'Follow the host default model',
   'estimator.mode.host': 'Host model (reuse configured providers)',
   'estimator.mode.direct': 'Direct OpenAI-compatible endpoint',
   'estimator.inactive': 'The estimator ships only with the TokenPilot-inspired profile: that profile’s preset options (dedupe pointers, summary locators, read-state semantics, and the estimator channel) are never merged into another profile, so the current profile “{profile}” has no estimator channel to configure. Select TokenPilot-inspired and this section gains a Channel choice — reuse configured providers (host model) or a direct OpenAI-compatible endpoint.',

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

@@ -1,21 +1,25 @@
 /**
- * Field-wise writes for the `presetOptions` section of the
+ * Path-addressed writes for the `presetOptions` section of the
  * context-compression settings namespace.
  *
- * `settingsScope.set(field, value)` replaces the value AT that field, so
- * writing the section root with a patch deleted every sibling: choosing a
- * provider after choosing a channel erased `estimatorMode` and silently turned
- * the estimator back off, and the stored document kept exactly the last field
- * the user touched. The client reads the complete section (this namespace
- * declares no secret fields), so a patch is applied over the current document
- * and the merged result is what gets written.
+ * The settings Host applies a `set` op AT its path, so a field write must name
+ * only the field it changes. Writing the whole section — `scope.set(
+ * 'presetOptions', patch)` — replaces the section root with the patch, which
+ * deleted every sibling field: choosing a provider after choosing a channel
+ * erased `estimatorMode` and silently turned the estimator back off, and the
+ * saved document kept exactly the last field the user touched.
  *
- * A patch member set to `undefined` means "clear this field" (it then
- * re-inherits the preset default); absent members stay untouched.
+ * A patch member set to `undefined` means "clear this field" (the section then
+ * re-inherits the preset default) rather than "leave it alone"; absent members
+ * are untouched by construction.
  */
 
+import type { SettingsPathOpView } from '@deepseek-ai/dsh-settings/types'
 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. */
 const PRESET_OPTION_KEYS = [
   'dedupeToolResults', 'summaryLocator', 'prefixStabilizer', 'readState', 'estimatorMode',
@@ -28,43 +32,53 @@ export type PresetOptionsPatch = {
 }
 
 /**
- * Apply one patch over the stored section.
+ * Plan the path ops one patch needs against the section currently stored.
+ *
+ * Fields already at their requested value produce no op, so a re-selected
+ * value neither rewrites the document nor reports a save the Host never
+ * committed.
  *
  * @param current - the decoded `presetOptions` section, when one is stored.
  * @param patch - the fields to write or clear.
- * @returns the complete section to store.
+ * @returns the ordered ops, empty when the patch changes nothing.
  */
-export function mergePresetOptionsPatch(
+export function planPresetOptionsOps(
   current: PresetOptionsSettings | undefined,
   patch: PresetOptionsPatch,
-): PresetOptionsSettings {
-  const source = current as Record<string, unknown> | undefined
-  const merged: Record<string, unknown> = {}
+): SettingsPathOpView[] {
+  const stored = current as Record<string, unknown> | undefined
+  const ops: SettingsPathOpView[] = []
   for (const key of PRESET_OPTION_KEYS) {
-    const stored = source?.[key]
-    if (stored !== undefined) merged[key] = stored
-  }
-  for (const key of Object.keys(patch)) {
-    const value = (patch as Record<string, unknown>)[key]
-    if (value === undefined) delete merged[key]
-    else merged[key] = value
+    if (!Object.hasOwn(patch, key)) continue
+    const value = patch[key]
+    const path = [PRESET_OPTIONS_KEY, key]
+    if (value === undefined) {
+      if (stored?.[key] !== undefined) ops.push({ op: 'unset', path })
+      continue
+    }
+    if (stored?.[key] === value) continue
+    ops.push({ op: 'set', path, value })
   }
-  return merged as PresetOptionsSettings
+  return ops
 }
 
 /**
- * Compare two sections field by field, so an unchanged patch neither rewrites
- * the document nor reports a save the Host never committed.
+ * Test whether one resolved settings document already carries every planned op.
  *
- * @param left - one section (or none).
- * @param right - the other section.
- * @returns whether every known field holds the same value.
+ * @param settings - the resolved namespace section after a write.
+ * @param ops - the ops that write submitted.
+ * @returns whether every named field holds its requested state.
  */
-export function presetOptionsEqual(
-  left: PresetOptionsSettings | undefined,
-  right: PresetOptionsSettings | undefined,
+export function presetOptionsOpsAccepted(
+  presetOptions: PresetOptionsSettings | undefined,
+  ops: readonly SettingsPathOpView[],
 ): boolean {
-  const a = left as Record<string, unknown> | undefined
-  const b = right as Record<string, unknown> | undefined
-  return PRESET_OPTION_KEYS.every(key => a?.[key] === b?.[key])
+  if (ops.length === 0) return true
+  if (presetOptions === undefined) return false
+  const stored = presetOptions as Record<string, unknown>
+  return ops.every((op) => {
+    const key = op.path[op.path.length - 1]
+    if (key === undefined) return false
+    return op.op === 'set' ? stored[key] === op.value : stored[key] === undefined
+  })
 }

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

@@ -25,15 +25,13 @@ export function ContextCompressionSettingsSection(props: CompressionProfileSelec
 }
 
 export function SettingsCompressionProfileControls({
-  useCompression, useSessions, select, saveCustom, resetCustom, saveAutoCompact, saveCodeSkeleton,
+  useCompression, select, saveCustom, resetCustom, saveAutoCompact, saveCodeSkeleton,
   savePresetOptions, t,
 }: CompressionProfileSelectorProps) {
   const state = useCompression(snapshot => snapshot)
-  const currentPreset = useSessions((sessions) => {
-    const current = sessions.current
-    return current === undefined ? undefined : sessions.byId[current]?.agentPreset
-  })
-  const selectorAvailable = currentPreset !== 'minimal'
+  // Harness 0.1.5 dropped browser-side agentPreset from session summaries, so
+  // the old Minimal-preset gate has no data source; the selector stays enabled.
+  const selectorAvailable = true
   const [saving, setSaving] = useState(false)
   const [saveError, setSaveError] = useState<string | null>(null)
   const [draft, setDraft] = useState<CustomCompressionPolicyV3 | null>(null)

package/packages/selector/src/deepseek-v4-tokenizer.ts

@@ -81,11 +81,6 @@ interface TokenizerArtifact {
  * Every bundled artifact is registered with independent asset roots, integrity
  * manifests, and cache entries: one corrupted artifact must never disable the
  * tokenizer serving the other model family.
- *
- * This module deliberately sits at the package's `src/` root rather than in
- * `src/runtime/`: the build flattens the runtime into `lib/*.js`, so only a
- * module one level below the package root resolves `../assets/` identically
- * from source and from the published artifact.
  */
 const ARTIFACTS: readonly TokenizerArtifact[] = Object.freeze([
   Object.freeze({

package/packages/selector/src/index.ts

@@ -6,18 +6,28 @@ import {
   type SettingsScope,
   type default as SettingsService,
 } from '@deepseek-ai/dsh-settings'
+import { buildEstimatorCatalog, type EstimatorCatalogDeps } from './estimator-catalog.ts'
 import {
   CONTEXT_COMPRESSION_SETTINGS_NAMESPACE,
   ContextCompressionSettingsSchema,
 } from './runtime/config.ts'
-import { buildEstimatorCatalog, type EstimatorCatalogDeps } from './estimator-catalog.ts'
+
+// The settings namespace literal and the settings schema are owned by the
+// runtime config module. Both were once inlined/replaced here to dodge a
+// cross-package dependency (ab2175a: the namespace literal was duplicated and
+// the schema was downgraded to `z.any()`); the runtime is now part of this
+// package, so the real schema is back and the daily Custom defaults are
+// published again through settings.register(). The namespace value is
+// unchanged ('context-compression').
+const CONTEXT_COMPRESSION_NAMESPACE = CONTEXT_COMPRESSION_SETTINGS_NAMESPACE as never
 import {
   decorateAgentPresets,
   resolveCompressionModulePaths,
 } from './preset-overlay.ts'
 
-// 0.1.1/0.1.2 客户端 API 前缀是 /endpoint，0.1.5 起改为 /api —— 两条绝对路径都
-// 注册（各自的 (kind, path) 表项），一份处理器服务两个前缀。
+// 0.1.1/0.1.2 clients ask for /endpoint; 0.1.5 and later ask for /api. Both
+// absolute paths are registered (each is its own (kind, path) table entry) and
+// one handler serves both prefixes. The bundled client only probes /api.
 const ESTIMATOR_CATALOG_ROUTES = [
   '/endpoint/dsh-context-compression-improved/estimator-catalog',
   '/api/dsh-context-compression-improved/estimator-catalog',
@@ -30,13 +40,13 @@ const ESTIMATOR_CATALOG_ROUTES = [
  */
 const ESTIMATOR_CATALOG_ROUTE_DEPS: readonly ['webServer'] = ['webServer']
 
-/** Runtime detection of the host web server (same pattern as dsh-perm-gate). */
+/** The minimal face of the DSH `webServer` service this plugin uses. */
 interface WebServerLike {
-  register: (spec: {
+  register(route: {
     kind: 'exact'
     path: string
-    handler: (req: unknown, res: unknown) => void
-  }) => unknown
+    handler: (req: unknown, res: unknown) => unknown
+  }): () => void
 }
 
 /** The estimator-side service the catalog handler enriches its response with. */
@@ -46,28 +56,22 @@ interface AgentDefaultModelLike {
 }
 
 function asWebServer(value: unknown): WebServerLike | undefined {
-  const register = (value as { register?: unknown } | undefined)?.register
-  if (typeof register !== 'function') return undefined
+  if (typeof value !== 'object' || value === null) return undefined
+  const candidate = value as { register?: unknown }
   // Hand back the service itself -- never a wrapper re-exporting `register`.
   // The host reads its route tables off `this` (`this.exact` / `this.prefixes`),
   // so a detached call makes `this` the wrapper and throws "Cannot read
   // properties of undefined (reading 'has')" inside the host, after which the
-  // route is simply absent and the client sees a bare 404.
-  return value as WebServerLike
+  // route is simply absent and the client sees a bare 404. The 0.1.5 branch
+  // never had that defect; take this implementation, not the 0.1.2 history's.
+  return typeof candidate.register === 'function' ? (value as WebServerLike) : undefined
 }
 
 /**
  * Serve `GET /api/dsh-context-compression-improved/estimator-catalog` — the
  * settings card's host-route dropdowns (live provider/model groups from the DSH
- * `llm` service plus the effective selection). This lives on the top-level
- * plugin context, NOT inside the isolated toolResultPruner service.
- *
- * (The isolation reason this placement was originally justified with — "a route
- * registered there can never reach `webServer` across the isolation boundary" —
- * is **unverified**: no `@deepseek-ai` package calls `.isolate(`, so there is no
- * boundary to cross here. Top-level placement is still the right choice, for a
- * reason that needs no framework rule: the route is host-wide, not
- * per-pruner-instance. Don't promote the isolation wording into a rule.)
+ * `llm` service plus the effective selection). Best effort: without the
+ * `webServer` service the plugin keeps working, only the HTTP API is missing.
  *
  * The route gates on `webServer` **alone**. `llm` and `agentDefaultModel` only
  * enrich the response and are resolved per request, so listing them here would
@@ -162,9 +166,7 @@ function registerEstimatorCatalogRoute(ctx: Context): void {
   log('warn', 'context-compression webServer not active yet — estimator catalog route pending: %s', ESTIMATOR_CATALOG_ROUTES.join(', '))
 }
 
-// Harness 0.1.1 exposed a namespace-branding helper; 0.1.2 validates the
-// same public literal at SettingsProvider.register/get instead.
-const CONTEXT_COMPRESSION_NAMESPACE = CONTEXT_COMPRESSION_SETTINGS_NAMESPACE as never
+
 
 /** Shared state forwarded through every Cordis proxy of one settings service. */
 interface SharedSettingsRegistration {
@@ -196,27 +198,15 @@ export interface Config {
   /** Add the canonical compression stack to every non-Minimal preset. */
   presetOverlay?: boolean
   /**
-   * Own the estimator catalog HTTP route. Set on the Loader row that declares
-   * `inject: [webServer]`.
-   *
-   * **Measured, and it contradicts the note this field was introduced with.**
-   * The original justification — "registering a route authorizes against the
-   * calling fiber, and a fiber that has not declared `webServer` cannot reach
-   * it, not even through `ctx.inject` or `ctx.get`" — is wrong on both halves:
-   * the host's `register` performs no authorization at all (it reads
-   * `this.exact` / `this.prefixes` and throws only on a duplicate
-   * `(kind, path)`), and `ctx.get(name, strict)` checks only that the providing
-   * fiber is active (`state === 2`), never the caller's `inject` list. The one
-   * inject-gated path is the `ctx.webServer` **property** access, which this
-   * plugin never uses: `registerEstimatorCatalogRoute` uses `ctx.get` plus its
-   * own `ctx.inject(['webServer'], …)`.
+   * Register the estimator-catalog HTTP route on this row.
    *
-   * So the row-level `inject` is **not load-bearing**; it is kept as
-   * belt-and-braces so the route row stays inactive until `webServer` exists,
-   * and the flag keeps the route off standalone Bundle rows on profiles that
-   * have no web server. The internal two-channel registration is what actually
-   * covers both arrival orders. Do not cite this comment as a rule to the
-   * 0.1.5 replay — cite the host source.
+   * The standalone Bundle patch sets this on its own row (which declares
+   * `inject: [webServer]`), so profiles without a host web server never mount a
+   * row that could only announce a pending route. The row-level `inject` is
+   * belt-and-braces: `dsh-host-webserver.register` performs no authorization
+   * check and `ctx.get(name)` only asks whether the providing fiber is active,
+   * so the two-channel registration inside this function is what actually
+   * covers both arrival orders.
    */
   estimatorCatalogRoute?: boolean
 }
@@ -239,6 +229,9 @@ export function apply(ctx: Context, config: Config = {}): void {
       acquireSettingsRegistration(settingsCtx)
     })
 
+    // The settings card's host-route dropdowns read this route; it lives on the
+    // top-level plugin fiber, not inside the isolated `toolResultPruner`
+    // service, because the route is host-wide rather than per-pruner-instance.
     if (config.estimatorCatalogRoute === true) registerEstimatorCatalogRoute(ctx)
 
     if (config.presetOverlay !== true) return
@@ -255,7 +248,7 @@ export function apply(ctx: Context, config: Config = {}): void {
       presetsCtx.effect(() => () => installation.dispose(), 'contextCompressionSelector.agentPresets()')
     })
   } catch (error) {
-    console.error('context-compression apply() failed:', error)
+    console.error('context-compression selector apply failed: %o', error)
     throw error
   }
 }
@@ -268,8 +261,10 @@ export function apply(ctx: Context, config: Config = {}): void {
 function resolveAutoCompactThresholdPercent(presetsCtx: Context): number {
   const raw = presetsCtx.get('settings')?.get(CONTEXT_COMPRESSION_NAMESPACE)
   try {
-    const parsed = ContextCompressionSettingsSchema(structuredClone(raw) as never)
-    return parsed.autoCompact.thresholdPercent
+    const record = structuredClone(raw) as Record<string, unknown> | undefined
+    const threshold = record?.autoCompact as { thresholdPercent?: number } | undefined
+    const value = typeof threshold?.thresholdPercent === 'number' ? threshold.thresholdPercent : 80
+    return Number.isFinite(value) && value >= 50 && value <= 90 ? value : 80
   } catch {
     return 80
   }

package/packages/selector/src/invariant.ts

@@ -21,8 +21,8 @@ function resemblesCompanion(prune: PruneEvent, event: SessionEvent): boolean {
   if ((event.type !== 'tool/result' && event.type !== 'user/message')
     || typeof event.surfaceOp !== 'object') return false
   const sources = new Set(event.sourceEventSeqs ?? [])
-  return (event.surfaceOp.start === prune.data.shadowedRange.start
-      && event.surfaceOp.end === prune.data.shadowedRange.end)
+  return (event.surfaceOp.startSeq === prune.data.shadowedRange.start
+      && event.surfaceOp.endSeq === prune.data.shadowedRange.end)
     || prune.data.shadowedSeqs.some(seq => sources.has(seq))
 }
 
@@ -39,11 +39,11 @@ function validateCompanion(
   }
   const { shadowedRange, shadowedSeqs } = prune.data
   if (shadowedSeqs.length === 0
-    || shadowedSeqs[0] !== shadowedRange.start
-    || shadowedSeqs.at(-1) !== shadowedRange.end) {
+    || shadowedSeqs[0] !== Number(shadowedRange.start)
+    || shadowedSeqs.at(-1) !== Number(shadowedRange.end)) {
     fail(`compaction/prune at seq ${prune.seq} has a shadowed range inconsistent with shadowedSeqs`)
   }
-  if (event.surfaceOp.start !== shadowedRange.start || event.surfaceOp.end !== shadowedRange.end) {
+  if (event.surfaceOp.startSeq !== shadowedRange.start || event.surfaceOp.endSeq !== shadowedRange.end) {
     fail(`replacement at seq ${event.seq} does not replace compaction/prune range ${shadowedRange.start}-${shadowedRange.end}`)
   }
   const sources = new Set(event.sourceEventSeqs ?? [])