dsh-smooth-stream 0.4.2 → 0.5.0

package/src/plugin.ts

@@ -1,7 +1,9 @@
 import type { Context } from '@deepseek-ai/cordis'
 import type {} from '@deepseek-ai/dsh-host-webserver'
 import type { ConnectionRpcHandler } from '@deepseek-ai/dsh-client-connection'
-import { settingsNamespace } from '@deepseek-ai/dsh-settings'
+// Type-only: erased at runtime, so the host entry never link-fails on kernels
+// whose dsh-settings no longer ships the value-side helper (issue #17).
+import type { SettingsNamespace } from '@deepseek-ai/dsh-settings'
 import Schema from '@deepseek-ai/schemastery'
 import { DEFAULT_STREAM_CONFIG, type StreamConfig } from './config.ts'
 import { injectStreamConfig } from './boot-config.ts'
@@ -53,8 +55,8 @@ export const Config: Schema<Config> = Schema.object({
  */
 export const StreamSettingsSchema: Schema<StreamSettings> = Schema.object({
   enabled: Schema.boolean().default(DEFAULT_STREAM_SETTINGS.enabled),
+  controlScroll: Schema.boolean().default(DEFAULT_STREAM_SETTINGS.controlScroll),
   thinkAutoExpand: Schema.boolean().default(DEFAULT_STREAM_SETTINGS.thinkAutoExpand),
-  autoCollapse: Schema.boolean().default(DEFAULT_STREAM_SETTINGS.autoCollapse),
   debugEnabled: Schema.boolean().default(DEFAULT_STREAM_SETTINGS.debugEnabled),
   debugTuning: Schema.object({
     revealScale: Schema.number().min(0.25).max(2).default(DEFAULT_STREAM_SETTINGS.debugTuning.revealScale),
@@ -93,8 +95,14 @@ export function apply(ctx: Context, config: Config): void {
   // the durable provider as the authority, but expose this one schema through
   // the plugin's own loopback-only connection channel instead.
   ctx.inject(['settings'], (settingsCtx) => {
+    // 0.1.2 kernels dropped the `settingsNamespace()` helper — a validating
+    // identity on ≤ 0.1.1 — and take the raw string, so the rc-era brand is
+    // reproduced locally instead of statically importing a removed symbol.
+    // The namespace is a compile-time constant matching the kernel's
+    // /^[a-z][a-z0-9-]*$/ pattern.
+    const settingsNamespace = STREAM_SETTINGS_NS as SettingsNamespace
     const scope = settingsCtx.settings.register(
-      settingsNamespace(STREAM_SETTINGS_NS),
+      settingsNamespace,
       StreamSettingsSchema,
       { applies: 'live' },
     )
@@ -109,8 +117,8 @@ export function apply(ctx: Context, config: Config): void {
           installation: installation.kind,
           writable: connectionCtx.settings.writable,
           enabled: settings.enabled,
+          controlScroll: settings.controlScroll,
           thinkAutoExpand: settings.thinkAutoExpand,
-          autoCollapse: settings.autoCollapse,
           canUpgrade: installation.kind === 'npm',
         }
       }
@@ -151,13 +159,14 @@ export function apply(ctx: Context, config: Config): void {
         if (endpoint === STREAM_SETTINGS_RPC.write) {
           if (typeof payload !== 'object' || payload === null || Array.isArray(payload)
             || typeof (payload as { enabled?: unknown }).enabled !== 'boolean'
+            || typeof (payload as { controlScroll?: unknown }).controlScroll !== 'boolean'
             || typeof (payload as { thinkAutoExpand?: unknown }).thinkAutoExpand !== 'boolean'
-            || typeof (payload as { autoCollapse?: unknown }).autoCollapse !== 'boolean') {
+          ) {
             return {
               ok: false,
               error: {
                 code: 'settings-rejected',
-                message: 'enabled, thinkAutoExpand and autoCollapse must be booleans',
+                message: 'enabled, controlScroll and thinkAutoExpand must be booleans',
                 details: { ns: STREAM_SETTINGS_NS },
               },
             }
@@ -175,8 +184,8 @@ export function apply(ctx: Context, config: Config): void {
           try {
             const next = payload as {
               enabled: boolean
+              controlScroll: boolean
               thinkAutoExpand: boolean
-              autoCollapse: boolean
               debugEnabled?: unknown
               debugTuning?: unknown
             }
@@ -193,8 +202,8 @@ export function apply(ctx: Context, config: Config): void {
             }
             await scope.update({
               enabled: next.enabled,
+              controlScroll: next.controlScroll,
               thinkAutoExpand: next.thinkAutoExpand,
-              autoCollapse: next.autoCollapse,
               ...(hasDebug ? { debugEnabled: next.debugEnabled, debugTuning: next.debugTuning } : {}),
             })
           } catch {

package/src/settings-api.ts

@@ -25,10 +25,10 @@ export interface StreamSettingsView {
   writable: boolean
   /** Whether smooth-stream currently owns conversation rendering. */
   enabled: boolean
+  /** Whether smooth-stream also owns conversation bottom-follow. */
+  controlScroll: boolean
   /** Current resolved preference. */
   thinkAutoExpand: boolean
-  /** Whether finished turns fold their work process behind a summary row. */
-  autoCollapse: boolean
   /** Whether a fixed npm update command is safe to offer. */
   canUpgrade: boolean
 }

package/src/settings.ts

@@ -56,19 +56,17 @@ export interface StreamSettings {
    * Off returns all rendering ownership to the built-in UI.
    */
   enabled: boolean
+  /**
+   * Whether this plugin also takes over conversation bottom-follow. Off leaves
+   * scroll ownership to the Harness; text reveal still runs.
+   */
+  controlScroll: boolean
   /**
    * Whether a reasoning ("Think") block auto-expands while it is the
    * streaming tail. Off keeps the block collapsed — the user can still open
    * it by hand — and stops the running state from re-owning the disclosure.
    */
   thinkAutoExpand: boolean
-  /**
-   * Whether a finished turn folds its work process (thinking, tool calls,
-   * context injection, intermediate replies) behind one "已处理" summary row,
-   * leaving only the final answer visible. The summary row can always be
-   * toggled by hand, and turns still expand live while they stream.
-   */
-  autoCollapse: boolean
   /** Whether the live renderer diagnostics panel is enabled. */
   debugEnabled: boolean
   /** Values edited by the diagnostics panel. */
@@ -78,8 +76,8 @@ export interface StreamSettings {
 /** Defaults shared by the Host schema and the client-side fallback. */
 export const DEFAULT_STREAM_SETTINGS: StreamSettings = {
   enabled: true,
+  controlScroll: true,
   thinkAutoExpand: true,
-  autoCollapse: true,
   debugEnabled: false,
   debugTuning: DEFAULT_STREAM_DEBUG_TUNING,
 }