@anionex/dsh-turn-rewind 0.3.7 → 0.3.9

package/src/engine.ts

@@ -58,7 +58,7 @@ const DEFAULTS = {
   maxSnapshotBytes: 512 * 1024 * 1024,
   planTtlMs: 15 * 60 * 1_000,
   staleLockMs: 30_000,
-  turnCheckpointMode: 'legacy',
+  turnCheckpointMode: 'auto',
   turnCheckpointTimeoutMs: 60_000,
   turnCheckpointMaxNewBytes: 32 * 1024 * 1024,
   turnCheckpointTrust: 'fast',
@@ -316,7 +316,12 @@ export class ChangeLedgerEngine {
           let manifest: RestorePointManifest
           // Git-native checkpoints live in the repository object database, so an
           // ordinary directory always uses the ledger's own content-addressed store.
-          if (this.config.turnCheckpointMode === 'legacy' || source.state.type === 'directory') {
+          // `auto` picks the fastest path each workspace can support: a Git
+          // worktree reuses the repository's own object database, while an
+          // ordinary directory can only use the ledger's store.
+          const gitNative = this.config.turnCheckpointMode === 'git-native'
+            || (this.config.turnCheckpointMode === 'auto' && source.state.type === 'git')
+          if (!gitNative) {
             manifest = await this.createLocked({
               cwd: source.state.root,
               kind: 'turn',
@@ -1396,8 +1401,8 @@ function requireNonEmptyString(value: string, name: string): string {
 }
 
 function checkpointMode(value: ChangeLedgerConfig['turnCheckpointMode']): ResolvedChangeLedgerConfig['turnCheckpointMode'] {
-  if (value !== 'off' && value !== 'git-native' && value !== 'legacy') {
-    throw new ChangeLedgerError('INVALID_CONFIG', 'turnCheckpointMode must be off, git-native, or legacy')
+  if (value !== 'off' && value !== 'auto' && value !== 'git-native' && value !== 'legacy') {
+    throw new ChangeLedgerError('INVALID_CONFIG', 'turnCheckpointMode must be off, auto, git-native, or legacy')
   }
   return value
 }

package/src/index.ts

@@ -6,7 +6,7 @@ import { readFileSync } from 'node:fs'
 import type { Context } from '@deepseek-ai/cordis'
 import { ChangeLedgerEngine } from './engine.js'
 import { installManageHttp, installRewindHttp, TurnCheckpointCoordinator } from './rewind-host.js'
-import { installTurnRewindSettings } from './settings.js'
+import { installTurnRewindSettings, TurnRewindSettingsSchema } from './settings.js'
 import type { ChangeLedgerConfig } from './types.js'
 
 export * from './engine.js'
@@ -32,6 +32,17 @@ function pluginVersion(): string {
   }
 }
 
+/** Volatile config values are wrappers in DSH 0.1.7 and plain values on older hosts. */
+function unwrapVolatileConfig(config: ChangeLedgerConfig): ChangeLedgerConfig {
+  const result: Record<string, unknown> = {}
+  for (const [key, value] of Object.entries(config)) {
+    result[key] = value !== null && typeof value === 'object' && typeof (value as { get?: unknown }).get === 'function'
+      ? (value as { get(): unknown }).get()
+      : value
+  }
+  return result as ChangeLedgerConfig
+}
+
 declare module '@deepseek-ai/cordis' {
   interface Context {
     changeLedger: ChangeLedgerService
@@ -40,12 +51,15 @@ declare module '@deepseek-ai/cordis' {
 
 /** Cordis service exposed as `ctx.changeLedger` for other DSH plugins. */
 export class ChangeLedgerService {
+  /** DSH 0.1.7 projects volatile Config fields into its settings form. */
+  static Config = TurnRewindSettingsSchema
   readonly engine: ChangeLedgerEngine
 
   /** Register the service and startup reconciliation. */
   constructor(ctx: Context, config: ChangeLedgerConfig = {}) {
+    const normalized = unwrapVolatileConfig(config)
     ctx.provide('changeLedger', this)
-    this.engine = new ChangeLedgerEngine(config)
+    this.engine = new ChangeLedgerEngine(normalized)
     const checkpoints = new TurnCheckpointCoordinator(this.engine)
     ctx.logger.info(`[turn-rewind] v${pluginVersion()} active; workspace modes: git worktree, ordinary directory`)
     ctx.inject(['agents'], (scope: Context) => { checkpoints.install(scope) })
@@ -55,7 +69,7 @@ export class ChangeLedgerService {
       installRewindHttp(scope, this.engine, checkpoints)
       installManageHttp(scope, this.engine)
     })
-    installTurnRewindSettings(ctx, config, this.engine)
+    installTurnRewindSettings(ctx, normalized, this.engine)
     void this.engine.initialize().then((reconciled) => {
       if (reconciled > 0) {
         ctx.logger.warn(`[change-ledger] reconciled ${reconciled} interrupted durable operation(s)`)

package/src/rewind-host.ts

@@ -678,8 +678,14 @@ async function createConversationRestart(
   // that returns undefined instead of failing the request.
   const controller = ctx.get('sessionController') as SessionControllerLike | undefined
   if (controller !== undefined) {
+    // Modern hosts require workspace membership independently of Session cwd.
+    // Preserve the source association; never guess a workspace from its path.
+    const registry = ctx.get('workspaceRegistry') as {
+      list(): readonly { readonly id: string; readonly sessionIds: readonly string[] }[]
+    } | undefined
+    const workspace = registry?.list().find(candidate => candidate.sessionIds.includes(sourceId))
     const created = checkpoint.previousTurnEndSeq === undefined
-      ? await controller.create({ cwd: checkpoint.cwd })
+      ? await controller.create(workspace === undefined ? { cwd: checkpoint.cwd } : { workspaceId: workspace.id })
       : await controller.fork({ sessionId: sourceId, atSeq: checkpoint.previousTurnEndSeq })
     return { sessionId: requiredText(created.sessionId, 'sessionController sessionId') }
   }

package/src/settings.ts

@@ -19,7 +19,7 @@ import type { ChangeLedgerConfig, ResolvedChangeLedgerConfig } from './types.js'
  * a compile-time-only brand, so a plain lowercase literal is the entire runtime
  * value — the upstream convention (`const CHAT_SETTINGS_NAMESPACE = 'ui-chat'`).
  * The literal satisfies 0.1.5's registration grammar `/^[a-z][a-z0-9-]*$/` and is
- * the exact key the browser card binds through `ctx.settingsScope.bind`.
+ * the exact key both browser settings transports use.
  */
 export const TURN_REWIND_SETTINGS_NAMESPACE = 'turn-rewind'
 
@@ -40,7 +40,7 @@ export interface TurnRewindSettings {
   /** Age after which a lock whose owner is gone may be reclaimed. */
   staleLockMs: number
   /** Automatic turn-checkpoint implementation; `off` records durable skips instead. */
-  turnCheckpointMode: 'off' | 'git-native' | 'legacy'
+  turnCheckpointMode: 'off' | 'auto' | 'git-native' | 'legacy'
   /** Maximum time one automatic checkpoint may block the first Agent step. */
   turnCheckpointTimeoutMs: number
   /** Maximum uncached worktree bytes read by one automatic Git-native checkpoint. */
@@ -49,21 +49,27 @@ export interface TurnRewindSettings {
   turnCheckpointTrust: 'fast' | 'strict'
 }
 
-/** Schemastery schema for the `turn-rewind` settings namespace. */
+/** Older Schemastery releases have no volatile marker; their settings provider owns updates. */
+function volatileWhenSupported<T>(schema: z<T>): z<T> {
+  const compatible = schema as z<T> & { volatile?: () => z<T> }
+  return typeof compatible.volatile === 'function' ? compatible.volatile() : schema
+}
+
+/** Shared schema for the legacy settings namespace and DSH 0.1.7 Config form. */
 export const TurnRewindSettingsSchema: z<TurnRewindSettings> = (() => {
   const defaults = tunableSettings(resolveConfig({}))
   return z.object({
-    maxRestorePoints: z.number().step(1).min(1).default(defaults.maxRestorePoints),
-    maxTurnCheckpointsPerSession: z.number().step(1).min(1).default(defaults.maxTurnCheckpointsPerSession),
-    maxFiles: z.number().step(1).min(1).default(defaults.maxFiles),
-    maxFileBytes: z.number().step(1).min(1).default(defaults.maxFileBytes),
-    maxSnapshotBytes: z.number().step(1).min(1).default(defaults.maxSnapshotBytes),
-    planTtlMs: z.number().step(1).min(1).default(defaults.planTtlMs),
-    staleLockMs: z.number().step(1).min(1).default(defaults.staleLockMs),
-    turnCheckpointMode: z.union(['off', 'git-native', 'legacy']).default(defaults.turnCheckpointMode),
-    turnCheckpointTimeoutMs: z.number().step(1).min(1).default(defaults.turnCheckpointTimeoutMs),
-    turnCheckpointMaxNewBytes: z.number().step(1).min(1).default(defaults.turnCheckpointMaxNewBytes),
-    turnCheckpointTrust: z.union(['fast', 'strict']).default(defaults.turnCheckpointTrust),
+    maxRestorePoints: volatileWhenSupported(z.number().step(1).min(1).default(defaults.maxRestorePoints)),
+    maxTurnCheckpointsPerSession: volatileWhenSupported(z.number().step(1).min(1).default(defaults.maxTurnCheckpointsPerSession)),
+    maxFiles: volatileWhenSupported(z.number().step(1).min(1).default(defaults.maxFiles)),
+    maxFileBytes: volatileWhenSupported(z.number().step(1).min(1).default(defaults.maxFileBytes)),
+    maxSnapshotBytes: volatileWhenSupported(z.number().step(1).min(1).default(defaults.maxSnapshotBytes)),
+    planTtlMs: volatileWhenSupported(z.number().step(1).min(1).default(defaults.planTtlMs)),
+    staleLockMs: volatileWhenSupported(z.number().step(1).min(1).default(defaults.staleLockMs)),
+    turnCheckpointMode: volatileWhenSupported(z.union(['off', 'auto', 'git-native', 'legacy']).default(defaults.turnCheckpointMode)),
+    turnCheckpointTimeoutMs: volatileWhenSupported(z.number().step(1).min(1).default(defaults.turnCheckpointTimeoutMs)),
+    turnCheckpointMaxNewBytes: volatileWhenSupported(z.number().step(1).min(1).default(defaults.turnCheckpointMaxNewBytes)),
+    turnCheckpointTrust: volatileWhenSupported(z.union(['fast', 'strict']).default(defaults.turnCheckpointTrust)),
   })
 })()
 
@@ -106,7 +112,7 @@ interface SettingsSectionHooksLike<T> {
  * {@link SettingsProvider} type at the single boundary where it is used.
  */
 interface SettingsProviderLike {
-  register<T>(ns: string, schema: z<T>, options?: { readonly base?: Partial<T> }): SettingsScopeLike<T>
+  register?<T>(ns: string, schema: z<T>, options?: { readonly base?: Partial<T> }): SettingsScopeLike<T>
   installSection?<T>(owner: Context, ns: string, schema: z<T>, entry: T, hooks: SettingsSectionHooksLike<T>): void
 }
 
@@ -142,6 +148,9 @@ export function installTurnRewindSettings(ctx: Context, config: ChangeLedgerConf
       provider.installSection(ctx, TURN_REWIND_SETTINGS_NAMESPACE, TurnRewindSettingsSchema, entry, hooks)
       return
     }
+    // DSH 0.1.7 projects the plugin's Config schema instead of registering a
+    // separate namespace. Its settings service exposes neither old method.
+    if (typeof provider.register !== 'function') return
     // Older hosts shipped that wiring only as the free `installSettingsSection`,
     // which 0.1.5 removed, so reproduce its body against `register` + `scope.get()`
     // + `scope.watch()` — present in every supported version.

package/src/types.ts

@@ -244,7 +244,7 @@ export interface ChangeLedgerConfig {
   /** Age after which a lock whose owner is gone may be reclaimed. */
   readonly staleLockMs?: number
   /** Automatic turn-checkpoint implementation. Git-native is bounded and avoids rereading clean tracked content. */
-  readonly turnCheckpointMode?: 'off' | 'git-native' | 'legacy'
+  readonly turnCheckpointMode?: 'off' | 'auto' | 'git-native' | 'legacy'
   /** Maximum time one automatic checkpoint may block the first Agent step. */
   readonly turnCheckpointTimeoutMs?: number
   /** Maximum uncached worktree bytes read by one automatic Git-native checkpoint. */
@@ -263,7 +263,7 @@ export interface ResolvedChangeLedgerConfig {
   readonly maxSnapshotBytes: number
   readonly planTtlMs: number
   readonly staleLockMs: number
-  readonly turnCheckpointMode: 'off' | 'git-native' | 'legacy'
+  readonly turnCheckpointMode: 'off' | 'auto' | 'git-native' | 'legacy'
   readonly turnCheckpointTimeoutMs: number
   readonly turnCheckpointMaxNewBytes: number
   readonly turnCheckpointTrust: 'fast' | 'strict'