dsh-ptc-plus 0.5.3 → 0.5.6

package/internal/session-log-view.js

@@ -9,9 +9,38 @@ import { deepEqualJson, isJsonValue } from '@deepseek-ai/dsh-util-values'
 import { deepFreeze, isRecord } from './record-utils.js'
 import { advanceRuntimeHistory, createRuntimeHistory } from './recovery-tips.js'
 
-import { isHostRuntimeContextSource } from './message-sources.js'
+import { isHostRuntimeContextSource, isPtcMessageSource } from './message-sources.js'
+import { BINDING_ADOPTION_FORM, readBindingAdoption } from './binding-adoption.js'
 const SYSTEM_PROMPT_CLEARED = 'Current runtime context: none. Earlier runtime-context snapshots no longer apply.'
 
+/**
+ * A PTC binding-adoption statement, readable or not.
+ *
+ * The fold has to recognize the statement even when this build cannot resolve its
+ * wording: the fact it keeps is what a restored checkpoint replays, so a fact that
+ * dropped the form could not reproduce the record the fold appended.
+ */
+function isBindingAdoptionStatement(message) {
+  const source = message?.source
+  return isPtcMessageSource(source) && source.form === BINDING_ADOPTION_FORM
+}
+
+/**
+ * An adoption statement this build cannot resolve, kept as the statement it is.
+ *
+ * Both the panel and the `/binding` command answer from this projection, and the
+ * answer has to be this session's *newest* statement. A projection that dropped a
+ * statement it could not resolve would let the record the session superseded answer
+ * instead, so the statement is retained with no declined list — the absence is what
+ * marks it unreadable. It never reaches the model-visible surface, which is built
+ * from resolvable runtime messages only.
+ */
+function unreadableBindingAdoption(message) {
+  if (!isBindingAdoptionStatement(message)) return undefined
+  if (readBindingAdoption(message) !== undefined) return undefined
+  return Object.freeze({ form: BINDING_ADOPTION_FORM })
+}
+
 /** DSH's empty aggregate uses an exact owner marker without a snapshot form. */
 export function systemPromptSnapshotSections(message) {
   const source = message?.source
@@ -72,7 +101,8 @@ export function runtimeMessageFacts(records, initial = {}) {
 
 function presentationEvent(event) {
   const recognized = event?.type === 'user/message'
-    && (systemPromptSnapshotSections(event.data) !== undefined || readRuntimeMessage(event.data) !== undefined)
+    && (systemPromptSnapshotSections(event.data) !== undefined || readRuntimeMessage(event.data) !== undefined
+      || isBindingAdoptionStatement(event.data))
   const data = recognized ? { source: event.data.source, content: event.data.content }
     : event?.type === 'user/message' && typeof event.data?.source?.kind === 'string'
       ? { source: { kind: event.data.source.kind } } : undefined
@@ -86,11 +116,14 @@ function applyLogEvent(state, event, index = event?.seq,
   timeline = advanceSessionTimeline(state.timeline, event, index)) {
   const sections = event?.type === 'user/message' ? systemPromptSnapshotSections(event.data) : undefined
   const ptc = event?.type === 'user/message' ? readRuntimeMessage(event.data) : undefined
+  // An adoption statement this build cannot resolve is still this session's statement,
+  // so it stays in the projection as the newest one rather than disappearing.
+  const unresolvedAdoption = ptc === undefined ? unreadableBindingAdoption(event?.data) : undefined
   const contextStep = state.contextStep + (isContextStep(event) ? 1 : 0)
   const surfaceNodes = advanceSessionSurface(state.surfaceNodes, event)
   let visibleRuntimeMessages = state.visibleRuntimeMessages
-  if (timeline === state.timeline && sections === undefined && ptc === undefined && contextStep === state.contextStep
-    && surfaceNodes === state.surfaceNodes) return state
+  if (timeline === state.timeline && sections === undefined && ptc === undefined && unresolvedAdoption === undefined
+    && contextStep === state.contextStep && surfaceNodes === state.surfaceNodes) return state
   const next = { ...state, timeline, contextStep,
     logFacts: [...state.logFacts, deepFreeze({ index, event: presentationEvent(event) })],
   }
@@ -110,6 +143,11 @@ function applyLogEvent(state, event, index = event?.seq,
     const record = Object.freeze({ ...ptc, index, contextStep: state.contextStep, producer: 'ptc-plus' })
     next.ptcMessages = [...state.ptcMessages, record]
     next.runtimeMessagesBySeq = { ...next.runtimeMessagesBySeq, [`k${event.seq}`]: record }
+  } else if (unresolvedAdoption !== undefined) {
+    // Listed as this session's newest statement, but not as a resolvable runtime
+    // message: the ordered surface below is built from `runtimeMessagesBySeq`.
+    next.ptcMessages = [...state.ptcMessages,
+      Object.freeze({ ...unresolvedAdoption, index, contextStep: state.contextStep, producer: 'ptc-plus' })]
   }
   if (event?.surfaceOp === 'append') {
     const record = next.runtimeMessagesBySeq[`k${event.seq}`]
@@ -133,7 +171,11 @@ export function createSessionLogProjection(options = {}) {
   const replayBudget = admissionReplayBudget(options.replayBudgetBytes)
   return Object.freeze({
     key: SESSION_LOG_PROJECTION_KEY,
-    stateVersion: 13,
+    // 14 records the binding-adoption statement this build cannot resolve: the fact it
+    // keeps now carries the form its own replay needs, and `ptcMessages` gained a record
+    // for it. A checkpoint written by an earlier fold answers a different question, so
+    // the version moves and stored checkpoints are refolded rather than admitted.
+    stateVersion: 14,
     stateSchema: Object.freeze({ parse(value) {
       if (!isJsonValue(value) || !isRecord(value) || !isRecord(value.timeline)
         || typeof value.timeline.openTurn !== 'boolean' || typeof value.timeline.found !== 'boolean'

package/internal/settings-compat.js

@@ -74,6 +74,12 @@ export function installSettingsSectionCompat({
       && typeof writeNamespace === 'string' && writeNamespace.length > 0
       ? Object.freeze({
         update: (patch, expectedRevision) => provider.update(writeNamespace, patch, expectedRevision),
+        // The service applies field edits as `{ op: 'set' | 'unset', path }`; `update` can only
+        // restate values, so expose the primitive when it exists instead of narrowing the callers
+        // to a writer that cannot remove a field.
+        ...(typeof provider.mutate === 'function'
+          ? { mutate: (ops, expectedRevision) => provider.mutate(writeNamespace, ops, expectedRevision) }
+          : {}),
         read: () => provider.describe?.().find(descriptor => descriptor.ns === writeNamespace),
       }) : undefined
     const unbind = onProvider(provider, writer)

package/internal/user-bindings-owner.js

@@ -2,11 +2,14 @@ import { PTC_MESSAGE_SOURCE_KIND } from './message-sources.js'
 import { sessionExecutionAvailable } from './host-tool-sdk.js'
 import { bindingToggleMessage } from './binding-toggle-receipt.js'
 import {
-  bindingAdoptionMessage,
+  BINDING_ADOPTION_RECORD,
   clearedBindingAdoption,
   declinedBindingEntries,
   withBindingAdoption,
 } from './binding-adoption.js'
+// Namespace import: the record writer is one public surface of the host, and the module
+// load must not depend on a named export that a future release could rename.
+import * as dshSession from '@deepseek-ai/dsh-session'
 import { RPC_CONTRACTS } from './rpc-contract.js'
 import { randomUUID } from 'node:crypto'
 import { dirname } from 'node:path'
@@ -232,6 +235,11 @@ export function createUserBindingsOwner(ctx, options = {}) {
   let disposed = false
   const codeConsole = new UserBindingConsole(currentOptions)
   const commandRegistrations = new Map()
+  // The host's writer for a fact that belongs in the log but not on the model-visible
+  // surface. Every supported release publishes it, so an adoption decision is taken when
+  // it is made instead of waiting for a step to carry a message.
+  const appendRecord = typeof options.appendRecord === 'function' ? options.appendRecord
+    : typeof dshSession.appendPluginRecord === 'function' ? dshSession.appendPluginRecord : undefined
   const sessionAgents = new Map()
   const sessionGenerations = new Map()
   const agentLifecycles = new WeakMap()
@@ -331,8 +339,17 @@ export function createUserBindingsOwner(ctx, options = {}) {
     if (session === undefined || typeof session.append !== 'function') {
       throw new Error('binding adoption requires a live session')
     }
-    if (session.append('user/message', bindingAdoptionMessage(adoption), { surfaceOp: 'append' }) === undefined) {
-      throw new Error('the session did not record the binding adoption decision')
+    if (appendRecord === undefined) {
+      throw new Error('ptc-plus: this DSH release publishes no plugin-record writer, so a binding adoption decision cannot be recorded')
+    }
+    // Adoption is this session's own statement and the catalog the model reads already
+    // reflects it, so it is taken the moment the control is used: durable through the log,
+    // outside the model-visible surface, and requiring no prior step. A session that cannot
+    // take the record reports that instead of a decision nothing stored.
+    try {
+      appendRecord(session, BINDING_ADOPTION_RECORD, adoption)
+    } catch (error) {
+      throw new Error(`the session did not record the binding adoption decision: ${error?.message ?? String(error)}`)
     }
     return { kind: 'success', text: adoptionReceipt(adoption) }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-ptc-plus",
-  "version": "0.5.3",
+  "version": "0.5.6",
   "icon": "assets/ptc-plus-icon.svg",
   "description": "A session-bound agent-native REPL for DeepSeek Harness PTC mode.",
   "type": "module",
@@ -43,7 +43,7 @@
   ],
   "engines": {
     "node": "^22.19.0 || >=24.0.0",
-    "dsh": "*"
+    "dsh": ">=0.2.1-0 <0.2.3"
   },
   "allowScripts": {
     "esbuild": false
@@ -51,8 +51,8 @@
   "dependencies": {
     "@babel/code-frame": "^7.29.7",
     "@babel/parser": "^7.29.0",
-    "@deepseek-ai/dsh-util-values": "^0.2.0-rc.2 || ^0.2.1-alpha.1",
-    "@deepseek-ai/schemastery": "^3.18.3",
+    "@deepseek-ai/dsh-util-values": "^0.2.2-alpha.2",
+    "@deepseek-ai/schemastery": "~3.18.6-alpha.2",
     "@jridgewell/trace-mapping": "^0.3.31",
     "acorn": "^8.15.0"
   },
@@ -174,36 +174,50 @@
     "@codemirror/language": "^6.12.4",
     "@codemirror/state": "^6.7.4",
     "@codemirror/view": "^6.43.11",
-    "@deepseek-ai/dsh": "next",
-    "@deepseek-ai/dsh-agent-loop": "next",
-    "@deepseek-ai/dsh-api-gateway": "next",
-    "@deepseek-ai/dsh-api-remotes": "next",
-    "@deepseek-ai/dsh-atomic-write": "next",
-    "@deepseek-ai/dsh-client-locale": "next",
-    "@deepseek-ai/dsh-client-store": "next",
-    "@deepseek-ai/dsh-client-test-runtime": "next",
-    "@deepseek-ai/dsh-client-ui-conversation": "next",
-    "@deepseek-ai/dsh-client-ui-primitives": "next",
-    "@deepseek-ai/dsh-client-ui-renderer": "next",
-    "@deepseek-ai/dsh-client-ui-session": "next",
-    "@deepseek-ai/dsh-client-ui-slots": "next",
-    "@deepseek-ai/dsh-commands": "next",
-    "@deepseek-ai/dsh-compaction-image-offload": "next",
-    "@deepseek-ai/dsh-home-paths": "next",
-    "@deepseek-ai/dsh-llm": "next",
-    "@deepseek-ai/dsh-session-format-catalog": "next",
-    "@deepseek-ai/dsh-session-format-v0-to-v1": "next",
-    "@deepseek-ai/dsh-session-persistence-jsonl": "next",
-    "@deepseek-ai/dsh-settings": "next",
-    "@deepseek-ai/dsh-skill-filesystem": "next",
-    "@deepseek-ai/dsh-tool-cordis": "next",
-    "@deepseek-ai/dsh-tools": "next",
-    "@deepseek-ai/dsh-typert-protocol": "next",
-    "@deepseek-ai/dsh-typert-registry": "next",
-    "@deepseek-ai/dsh-util-workspace-path": "next",
+    "@deepseek-ai/cordis": "4.0.6-alpha.2",
+    "@deepseek-ai/cordis-plugin-group": "1.0.6-alpha.2",
+    "@deepseek-ai/cordis-plugin-include": "1.0.11-alpha.2",
+    "@deepseek-ai/cordis-plugin-loader": "1.0.7-alpha.2",
+    "@deepseek-ai/cordis-plugin-timer": "1.1.8-alpha.2",
+    "@deepseek-ai/cosmokit": "1.8.7-alpha.2",
+    "@deepseek-ai/dsh": "alpha",
+    "@deepseek-ai/dsh-agent-loop": "alpha",
+    "@deepseek-ai/dsh-agent-loop-testkit": "alpha",
+    "@deepseek-ai/dsh-api-gateway": "alpha",
+    "@deepseek-ai/dsh-api-remotes": "alpha",
+    "@deepseek-ai/dsh-atomic-write": "alpha",
+    "@deepseek-ai/dsh-client-locale": "alpha",
+    "@deepseek-ai/dsh-client-store": "alpha",
+    "@deepseek-ai/dsh-client-test-runtime": "alpha",
+    "@deepseek-ai/dsh-client-ui-conversation": "alpha",
+    "@deepseek-ai/dsh-client-ui-primitives": "alpha",
+    "@deepseek-ai/dsh-client-ui-renderer": "alpha",
+    "@deepseek-ai/dsh-client-ui-session": "alpha",
+    "@deepseek-ai/dsh-client-ui-slots": "alpha",
+    "@deepseek-ai/dsh-commands": "alpha",
+    "@deepseek-ai/dsh-compaction-image-offload": "alpha",
+    "@deepseek-ai/dsh-home-paths": "alpha",
+    "@deepseek-ai/dsh-llm": "alpha",
+    "@deepseek-ai/dsh-scope": "alpha",
+    "@deepseek-ai/dsh-session": "alpha",
+    "@deepseek-ai/dsh-session-format-catalog": "alpha",
+    "@deepseek-ai/dsh-session-format-v0-to-v1": "alpha",
+    "@deepseek-ai/dsh-session-persistence-jsonl": "alpha",
+    "@deepseek-ai/dsh-settings": "alpha",
+    "@deepseek-ai/dsh-skill": "alpha",
+    "@deepseek-ai/dsh-skill-filesystem": "alpha",
+    "@deepseek-ai/dsh-spill-policy": "alpha",
+    "@deepseek-ai/dsh-tool-cordis": "alpha",
+    "@deepseek-ai/dsh-tool-executor-native": "alpha",
+    "@deepseek-ai/dsh-tool-executor-ptc": "alpha",
+    "@deepseek-ai/dsh-tools": "alpha",
+    "@deepseek-ai/dsh-typert-protocol": "alpha",
+    "@deepseek-ai/dsh-typert-registry": "alpha",
+    "@deepseek-ai/dsh-util-workspace-path": "alpha",
     "@lezer/highlight": "^1.2.3",
     "@radix-ui/react-select": "^2.2.6",
     "@shikijs/langs": "^4.3.1",
+    "@tanstack/react-virtual": "^3.14.24",
     "@testing-library/react": "^16.3.3",
     "amaro": "1.1.11",
     "anser": "^2.3.5",

package/src/client-authoring-view.js

@@ -69,7 +69,11 @@ export function createAuthoringView(React, deps) {
   }) {
     const input = useInput?.(snapshot => snapshot)
     const available = useBindingCommand(snapshot => snapshot)
-    const capabilityAvailable = useCapabilityCommand?.(snapshot => snapshot) === true
+    // The capability command source publishes nothing until a session observes it, and
+    // this entry is the surface that keeps it observed while mounted. Its value decides
+    // no branch here: whether `/capabilities` is listed is the command's own answer when
+    // it runs, so the refresh shortcut stays usable and the refusal carries its reason.
+    useCapabilityCommand?.(() => undefined)
     const settings = usePtcSettings(snapshot => snapshot)
     const [review, view] = useBindingReview(sessionId)
     const anchorRef = React.useRef(null)
@@ -113,8 +117,7 @@ export function createAuthoringView(React, deps) {
     const pluginEnabled = featureEnabled(settings, 'plugin') && (hasDraft || featureEnabled(settings, 'ptcMenu'))
     const canPrefill = typeof useInput === 'function' && typeof inputActions?.setDraft === 'function'
     const canAuthor = quickAccess && available && canPrefill
-    const canRefresh = pluginEnabled && settings.status === 'ready' && settings.writable === true
-      && capabilityAvailable && canPrefill
+    const canRefresh = pluginEnabled && settings.status === 'ready' && settings.writable === true && canPrefill
     const menuOpen = pluginEnabled && view.reachable && !managing && !capabilitiesOpen && !settingsOpen
       && menu !== null && menu.key === view.candidateKey
     // The dialog owns the management session; losing the input surface only
@@ -411,8 +414,7 @@ export function createAuthoringView(React, deps) {
     const actionButton = (id, copy) => h('button', {
       type: 'button', role: 'menuitem', className: 'ptcPlusMenuButton',
       disabled: id === 'refresh' && !canRefresh,
-      title: id === 'refresh' && !capabilityAvailable ? t('capabilities.refreshUnavailable')
-        : id === 'settings' ? settingsPath(t) : t(copy),
+      title: id === 'settings' ? settingsPath(t) : t(copy),
       onClick: () => selectMenuItem(id),
     }, actionCopy(id, copy))
     const actionSections = [
@@ -496,7 +498,6 @@ export function createAuthoringView(React, deps) {
         }) : null,
       SessionCapabilityControls && capabilitiesOpen && view.reachable
         ? h(SessionCapabilityControls, { sessionId, open: true, t,
-          enabled: pluginEnabled && settings.status === 'ready' && settings.writable === true,
           onClose: () => {
             setCapabilitiesOpen(false)
             queueMicrotask(() => anchorRef.current?.querySelector('.ptcPlusAuthorButton')
@@ -747,8 +748,6 @@ export function createAuthoringView(React, deps) {
       node.outcome?.kind !== 'error' ? null : h('p', { className: 'ptcPlusMessage ptcPlusDanger' },
         node.outcome.text ?? t('bindings.commandFailed')),
       canOpen ? h(ActionButton, { onClick: () => openBindingReview(review) }, t('bindings.reviewOpen')) : null,
-      !current.mounted && (candidate !== null || matches && projection.phase === 'ready')
-        ? h('p', { className: 'ptcPlusMessage' }, t('bindings.reviewUnavailable')) : null,
       candidate === null ? null : h('details', { className: 'ptcPlusBindingSourceDetails' },
         h('summary', null, t('tool.source')), h(BindingCandidateContent, { candidate, t })))
   }