dsh-code 1.0.4 → 1.0.5

package/src/render/export.ts

@@ -6,7 +6,7 @@
  * @module @deepseek-ai/dsh-code/render/export
  */
 
-import { assertNever } from '@deepseek-ai/dsh-llm'
+import { assertNever } from '@deepseek-ai/dsh-util-values'
 import { imageLabels, type TranscriptView } from './projection.ts'
 
 /**
@@ -32,12 +32,12 @@ export function buildExportMarkdown(view: TranscriptView, sessionId: string): st
           out.push('## user', '', entry.text, ...(imageLabels(entry.images) === '' ? [] : [imageLabels(entry.images)]), '')
         }
         break
-      case 'assistant':
-        if (entry.reasoning !== '') {
-          out.push('<details><summary>thinking</summary>', '', entry.reasoning, '', '</details>', '')
-        }
-        out.push('## assistant', '', entry.text, '')
-        break
+      case 'assistant':
+        if (entry.reasoning !== '') {
+          out.push('<details><summary>thinking</summary>', '', entry.reasoning, '', '</details>', '')
+        }
+        out.push('## assistant', '', entry.text, '')
+        break
       case 'tool':
         out.push(`### tool \`${entry.name}\``, '')
         if (entry.preview !== '') out.push(`- args: ${entry.preview}`)

package/src/render/projection.ts

@@ -8,7 +8,8 @@
  */
 
 import { boundContextSummary, type ContentBlock, type ImageBlock, type MessageId } from '@deepseek-ai/dsh-llm'
-import type { SessionEvent, TodoItem } from '@deepseek-ai/dsh-session'
+import type { SessionEvent } from '@deepseek-ai/dsh-session'
+import type { TodoItem } from '@deepseek-ai/dsh-tool-todo'
 import { graphemeWidth, splitGraphemes } from './width.ts'
 // Type-only imports merge the plugin-owned SessionEventMap variants
 // (agent/inbox/spliced, command/*, compaction/*, goal/change, llm/retry*,

package/src/settings-file.ts

@@ -16,9 +16,46 @@
  * @module @deepseek-ai/dsh-code/settings-file
  */
 
+import { randomUUID } from 'node:crypto'
 import { mkdir, rename, writeFile } from 'node:fs/promises'
 import { dirname } from 'node:path'
 
+/**
+ * Run one file operation with a bounded retry: one initial try plus at
+ * most `retries` more. Creating or replacing a file can fail transiently
+ * with EPERM/EACCES while an antivirus scanner or search indexer holds
+ * it — the standard graceful-fs remedy, not a workaround for a
+ * persistent permission problem. A save that still fails leaves its
+ * uniquely named temp file behind, so repeated crashed saves accumulate
+ * distinct leftovers rather than corrupting a shared one.
+ */
+async function withTransientRetry(operation: () => Promise<void>, retries = 5): Promise<void> {
+  for (let attempt = 0; ; attempt += 1) {
+    try {
+      await operation()
+      return
+    } catch (error: unknown) {
+      const code = (error as NodeJS.ErrnoException).code
+      if (attempt >= retries || (code !== 'EPERM' && code !== 'EACCES')) throw error
+      await new Promise(resolve => setTimeout(resolve, 30 * (attempt + 1)))
+    }
+  }
+}
+
+/**
+ * Write one file atomically: create the parent directory, write to a
+ * uniquely named temp file, and rename it into place. A crash midway
+ * can never leave a half-written document behind. Unique temp names
+ * keep concurrent writers (two terminals, two chains in one process)
+ * from sharing one temp path.
+ */
+export async function writeFileAtomically(path: string, text: string): Promise<void> {
+  await mkdir(dirname(path), { recursive: true })
+  const temp = `${path}.${process.pid}.${randomUUID()}.tmp`
+  await withTransientRetry(() => writeFile(temp, text, 'utf8'))
+  await withTransientRetry(() => rename(temp, path))
+}
+
 /** The serialized persistence surface; flush() is handed to the quit sequence. */
 export interface UserSettingsPersistence {
   /**
@@ -39,12 +76,7 @@ export function createUserSettingsPersistence(): UserSettingsPersistence {
   let chain: Promise<void> = Promise.resolve()
   return {
     save(path: string, text: string): Promise<void> {
-      const write = chain.then(async () => {
-        await mkdir(dirname(path), { recursive: true })
-        const temp = `${path}.tmp`
-        await writeFile(temp, text, 'utf8')
-        await rename(temp, path)
-      })
+      const write = chain.then(() => writeFileAtomically(path, text))
       // A failed write must not break the chain for later saves.
       chain = write.catch(() => {})
       return write