@xmoon76/dsh-pi-tui 0.1.1 → 0.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xmoon76/dsh-pi-tui",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "The dsh-pi-tui bundle: a TUI surface for DeepSeek Harness profiles (dsh --profile pi-tui)",
   "license": "MIT",
   "author": "XMoon <xmoonox@gmail.com>",
@@ -39,6 +39,7 @@
   "files": [
     "dist",
     "config",
+    "scripts",
     "cordis.patch.yml",
     "README.md",
     "LICENSE"
@@ -60,35 +61,36 @@
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/cordis-plugin-loader": "^1.0.2",
-    "@deepseek-ai/dsh-cmdline": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-agent": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-agent-default-model": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-llm": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-session": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-agent-presets": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-cmdline": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-commands": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-token-meter": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-skill": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-settings": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-plan-mode": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-session-persistence": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-user-approval": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-user-questions": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-subagent": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-credentials": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-goal": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-tools": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-tool-workflow": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-jobs": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-llm": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-llm-retry": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-permission-presets": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-session-title": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-plan-mode": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-sandbox-policy": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-session": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-session-persistence": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-session-query": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-jobs": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-session-title": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-settings": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-shell": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-credentials": "^0.1.0-rc.6",
-    "@deepseek-ai/dsh-agent-presets": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-skill": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-subagent": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-token-meter": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-tool-workflow": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-tools": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-user-approval": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-user-questions": "^0.1.0-rc.6",
     "@deepseek-ai/schemastery": "^3.18.1"
   },
   "devDependencies": {
-    "@xmoon76/pi-tui": "^0.84.2",
+    "@xmoon76/pi-tui": "^0.84.3",
     "@deepseek-ai/cordis-plugin-loader": "^1.0.2",
     "@deepseek-ai/dsh-agent": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-agent-default-model": "^0.1.0-rc.6",
@@ -124,8 +126,10 @@
   },
   "scripts": {
     "build": "tsdown",
+    "build:package": "pnpm --dir ../pi-tui build && pnpm build",
     "typecheck": "tsc -p tsconfig.json --noEmit",
-    "test": "node --test test/*.test.ts",
-    "clean": "rm -rf dist"
+    "test": "node --test test/*.test.ts test/*.test.mjs",
+    "clean": "rm -rf dist",
+    "smoke": "node scripts/tarball-smoke.mjs"
   }
 }

package/scripts/bench.mts

@@ -0,0 +1,261 @@
+#!/usr/bin/env node
+/**
+ * @xmoon76/dsh-pi-tui/scripts/bench — non-default performance benchmark
+ * (run explicitly: `node scripts/bench.mjs`; never part of the test suite).
+ *
+ * Builds synthetic session logs (markdown, diffs, consecutive reads, tool
+ * calls, CJK/emoji) and measures, across widths and themes:
+ *
+ *   - ingest: TranscriptFolder.apply() time per event count;
+ *   - projection: messages() p50/p95/p99 (the incremental read-grouping);
+ *   - rebuild: TuiApp.setTranscript cold (full markdown parse) vs warm
+ *     (the stage-J per-message render cache) p50/p95/p99, including the
+ *     20 Hz streaming case (one message's text replaced per frame);
+ *   - theme switch cost;
+ *   - heap: growth per warm rebuild and the settled working set.
+ *
+ * Output is a plain baseline table; run twice to compare. Requires a
+ * Node version with native TypeScript stripping (>=22.6 with
+ * --experimental-strip-types, or 24+).
+ * @module bench
+ */
+
+import xterm from '@xterm/headless'
+import { TuiApp } from '../src/tui-app.ts'
+import { TranscriptFolder } from '../src/transcript.ts'
+import type { SessionEvent } from '@deepseek-ai/dsh-session'
+import type { TranscriptMessage } from '../src/transcript.ts'
+
+const XtermTerminal = xterm.Terminal
+
+/** A minimal headless terminal for TuiApp (render target only). */
+class BenchTerminal {
+  private readonly xterm: InstanceType<typeof XtermTerminal>
+  constructor(columns: number, rows: number) {
+    this.xterm = new XtermTerminal({ cols: columns, rows, disableStdin: true, allowProposedApi: true })
+  }
+  start(): void {}
+  stop(): void {}
+  async drainInput(): Promise<void> {}
+  write(data: string): void { this.xterm.write(data) }
+  get columns(): number { return this.xterm.cols }
+  get rows(): number { return this.xterm.rows }
+  get kittyProtocolActive(): boolean { return false }
+  moveBy(): void {}
+  hideCursor(): void {}
+  showCursor(): void {}
+  clearLine(): void {}
+  clearFromCursor(): void {}
+  clearScreen(): void {}
+  setTitle(): void {}
+  setProgress(): void {}
+}
+
+// --- synthetic session content ---------------------------------------------
+
+const MARKDOWN_BLOCKS = [
+  '## Findings\n\nThe **cache** now hits for unchanged messages, so `markdown` is not re-parsed every frame.\n\n```ts\nconst hit = cached.text === message.text\n```\n\n- one\n- two\n- three',
+  '## Steps\n\n1. ingest events\n2. fold the projection\n3. render the tail\n\n> Only the visible window pays the expensive conversions.',
+  '## 结论（CJK + emoji）\n\n缓存命中后 🐋🐳 单帧成本不再随完整历史线性增长。`宽度 40/80/160` 与主题切换均按 key 失效。',
+]
+
+const DIFF_BODY = 'diff --git a/src/a.ts b/src/a.ts\nindex 111..222 100644\n--- a/src/a.ts\n+++ b/src/a.ts\n@@ -1,3 +1,4 @@\n const a = 1\n+const b = 2\n-const old = 3\n // tail'
+
+/** Build `turns` turns of events; each turn ≈ 9 events. */
+function buildEvents(turns: number): SessionEvent[] {
+  const events: SessionEvent[] = []
+  let seq = 0
+  for (let turn = 0; turn < turns; turn += 1) {
+    events.push({ type: 'turn/start', seq: seq++, time: turn * 1000, data: { turn } } as SessionEvent)
+    events.push({
+      type: 'user/message', seq: seq++, time: turn * 1000 + 1, data: {
+        content: [{ type: 'text', text: `user prompt ${turn} with CJK 你好 and emoji 🐋` }],
+        source: { kind: 'user' },
+      },
+    } as SessionEvent)
+    for (let chunk = 0; chunk < 8; chunk += 1) {
+      events.push({
+        type: 'assistant/chunk', seq: seq++, time: turn * 1000 + 2 + chunk, data: {
+          turn, step: 0, index: chunk,
+          chunk: { type: 'text-delta', index: chunk, text: MARKDOWN_BLOCKS[turn % 3]!.slice(chunk * 40, chunk * 40 + 40) },
+        },
+      } as SessionEvent)
+    }
+    events.push({
+      type: 'assistant/message', seq: seq++, time: turn * 1000 + 11, data: {
+        turn, step: 0,
+        message: { id: `msg-${turn}`, role: 'assistant', content: [{ type: 'text', text: MARKDOWN_BLOCKS[turn % 3]! }], source: { kind: 'assistant' } },
+      },
+    } as SessionEvent)
+    events.push({
+      type: 'tool/call', seq: seq++, time: turn * 1000 + 12, data: {
+        turn, step: 0, callId: `r${turn}`, name: 'read', arguments: JSON.stringify({ file: `src/file-${turn}.ts` }),
+      },
+    } as SessionEvent)
+    events.push({
+      type: 'tool/result', seq: seq++, time: turn * 1000 + 13, data: {
+        turn, step: 0, message: {
+          id: `m-${turn}`, role: 'user',
+          content: [{ type: 'tool-result', toolCallId: `r${turn}`, content: [{ type: 'text', text: DIFF_BODY }] }],
+          source: { kind: 'tool', callId: `r${turn}` },
+        },
+      },
+    } as SessionEvent)
+    events.push({
+      type: 'tool/call', seq: seq++, time: turn * 1000 + 14, data: {
+        turn, step: 0, callId: `b${turn}`, name: 'bash', arguments: JSON.stringify({ command: 'ls -la' }),
+      },
+    } as SessionEvent)
+    events.push({
+      type: 'tool/result', seq: seq++, time: turn * 1000 + 15, data: {
+        turn, step: 0, message: {
+          id: `bm-${turn}`, role: 'user',
+          content: [{ type: 'tool-result', toolCallId: `b${turn}`, content: [{ type: 'text', text: 'total 8\ndrwxr-xr-x 2 user user 4096 Aug 15 00:00 .\n-rw-r--r-- 1 user user 123 src/a.ts' }] }],
+          source: { kind: 'tool', callId: `b${turn}` },
+        },
+      },
+    } as SessionEvent)
+    events.push({ type: 'turn/end', seq: seq++, time: turn * 1000 + 16, data: { turn, reason: { kind: 'completed' } } } as SessionEvent)
+  }
+  return events
+}
+
+// --- measurement helpers ----------------------------------------------------
+
+function percentile(sorted: number[], p: number): number {
+  if (sorted.length === 0) return 0
+  const index = Math.min(sorted.length - 1, Math.max(0, Math.ceil((p / 100) * sorted.length) - 1))
+  return sorted[index]!
+}
+
+function stats(samples: number[]): { p50: number; p95: number; p99: number; mean: number } {
+  const sorted = [...samples].sort((a, b) => a - b)
+  return {
+    p50: percentile(sorted, 50),
+    p95: percentile(sorted, 95),
+    p99: percentile(sorted, 99),
+    mean: samples.reduce((sum, value) => sum + value, 0) / Math.max(1, samples.length),
+  }
+}
+
+function fmt(st: { p50: number; p95: number; p99: number; mean: number }): string {
+  return `p50 ${st.p50.toFixed(2)}ms · p95 ${st.p95.toFixed(2)}ms · p99 ${st.p99.toFixed(2)}ms · mean ${st.mean.toFixed(2)}ms`
+}
+
+function fmtMs(ms: number): string {
+  return `${ms.toFixed(2)}ms`
+}
+
+/** Run a callback `n` times and return per-call wall times. */
+function timeIt(n: number, run: () => void): number[] {
+  const samples: number[] = []
+  for (let i = 0; i < n; i += 1) {
+    const started = performance.now()
+    run()
+    samples.push(performance.now() - started)
+  }
+  return samples
+}
+
+/** Iteration counts; BENCH_FAST=1 shrinks them for quick before/after sweeps. */
+const FAST = process.env.BENCH_FAST === '1'
+const PROJ_SAMPLES = FAST ? 50 : 200
+const WARM_SAMPLES = FAST ? 50 : 200
+const STREAM_SAMPLES = FAST ? 50 : 200
+const FULL_SAMPLES = FAST ? 20 : 50
+const HEAP_REBUILDS = FAST ? 300 : 2000
+
+// --- the benchmark ----------------------------------------------------------
+
+function main(): void {
+  const rows: string[] = []
+  const row = (label: string, value: string): void => { rows.push(`${label.padEnd(58)} ${value}`) }
+
+  row('scenario', 'value')
+
+  // 1. ingest + projection
+  for (const turns of [111, 1111, 5555]) {
+    const events = buildEvents(turns)
+    const folder = new TranscriptFolder()
+    const ingest = timeIt(3, () => folder.apply(events))
+    const projection = timeIt(PROJ_SAMPLES, () => folder.messages())
+    const st = stats(projection)
+    row(`ingest ${events.length} events (${turns} turns)`, fmtMs(ingest[0]!))
+    row(`  messages() ×200 (${folder.messages().length} messages)`, fmt(st))
+  }
+
+  // 2. rebuild (cold vs warm) across widths, plus the streaming case
+  const turns = 555
+  const folder = new TranscriptFolder()
+  folder.apply(buildEvents(turns))
+  const messages = folder.messages()
+  for (const width of [40, 80, 160]) {
+    // True cold: a fresh app whose FIRST setTranscript parses every message.
+    const coldApp = new TuiApp(new BenchTerminal(width, 24) as never, { onSubmit: () => {}, onExit: () => {} })
+    coldApp.start()
+    const cold = timeIt(1, () => coldApp.setTranscript(messages))
+    coldApp.stop()
+    const terminal = new BenchTerminal(width, 24)
+    const app = new TuiApp(terminal as never, { onSubmit: () => {}, onExit: () => {} })
+    app.start()
+    app.setTranscript(messages)
+    const warm = timeIt(WARM_SAMPLES, () => app.setTranscript(messages))
+    row(`rebuild ${messages.length} messages @${width} cols (cold, fresh app)`, fmtMs(cold[0]!))
+    row(`  same content (warm cache)`, fmt(stats(warm)))
+    // 20 Hz streaming: one assistant message's text replaced per frame.
+    const streaming = [...messages]
+    const target = streaming.findIndex(message => message.kind === 'assistant')
+    const warmStream = timeIt(STREAM_SAMPLES, () => {
+      const entry = streaming[target]
+      if (entry !== undefined && entry.kind === 'assistant') entry.text += ' tail'
+      app.setTranscript(streaming)
+    })
+    row(`  streaming 1 message/frame @${width}`, fmt(stats(warmStream)))
+    // Theme switch cost (once per switch).
+    const theme = timeIt(5, () => app.applyTheme('light'))
+    row(`  theme dark→light @${width}`, fmtMs(theme[0]!))
+    app.stop()
+  }
+
+  // 3. fullscreen rebuild
+  {
+    const terminal = new BenchTerminal(120, 24)
+    const app = new TuiApp(terminal as never, { onSubmit: () => {}, onExit: () => {} })
+    app.start()
+    app.setTranscript(messages)
+    app.setFullscreen(true)
+    const full = timeIt(FULL_SAMPLES, () => app.setTranscript(messages))
+    row(`fullscreen rebuild ${messages.length} messages @120`, fmt(stats(full)))
+    app.stop()
+  }
+
+  // 4. heap (requires --expose-gc)
+  if (globalThis.gc !== undefined) {
+    const terminal = new BenchTerminal(120, 24)
+    const app = new TuiApp(terminal as never, { onSubmit: () => {}, onExit: () => {} })
+    app.start()
+    app.setTranscript(messages)
+    const gc = globalThis.gc as () => void
+    gc()
+    const before = process.memoryUsage().heapUsed
+    for (let i = 0; i < HEAP_REBUILDS; i += 1) app.setTranscript(messages)
+    gc()
+    const after = process.memoryUsage().heapUsed
+    row(`heap growth per warm rebuild (${HEAP_REBUILDS}×, gc)`, `${((after - before) / 2000).toFixed(1)} B/rebuild`)
+    gc()
+    const settled = process.memoryUsage().heapUsed
+    row(`settled heap for ${messages.length} messages @120`, `${(settled / 1024 / 1024).toFixed(1)} MiB`)
+    app.stop()
+  } else {
+    row('heap (rerun with --expose-gc)', 'skipped')
+  }
+
+  console.log(rows.join('\n'))
+}
+
+try {
+  main()
+} catch (error) {
+  console.error(`bench: ${error instanceof Error ? error.message : String(error)}`)
+  process.exit(1)
+}

package/scripts/naming-gate.mjs

@@ -0,0 +1,102 @@
+#!/usr/bin/env node
+/**
+ * @xmoon76/dsh-pi-tui/scripts/naming-gate — CI guard for the naming and
+ * packaging decisions (AGENTS.md "Naming (hard rules)"): nothing here may
+ * use the `dsh-tui` / `@deepseek-ai/dsh-tui` family, the profile stays
+ * `pi-tui`, the fork stays private and bundled, and the bundle is the only
+ * published package with its fork in `deps.onlyBundle`. Fails with a clear
+ * list when any decision regresses.
+ * @module naming-gate
+ */
+
+import { existsSync, readFileSync } from 'node:fs'
+import { spawnSync } from 'node:child_process'
+import { join, dirname } from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+const ROOT = join(dirname(fileURLToPath(import.meta.url)), '..', '..', '..')
+const failures = []
+const checks = []
+
+function check(name, ok, detail = '') {
+  checks.push(`${ok ? 'ok  ' : 'FAIL'} ${name}${detail === '' ? '' : ` — ${detail}`}`)
+  if (!ok) failures.push(`${name}${detail === '' ? '' : `: ${detail}`}`)
+}
+
+function readJson(path) {
+  return JSON.parse(readFileSync(path, 'utf8'))
+}
+
+const rootPkg = readJson(join(ROOT, 'package.json'))
+const bundlePkg = readJson(join(ROOT, 'packages', 'dsh-pi-tui', 'package.json'))
+const forkPkg = readJson(join(ROOT, 'packages', 'pi-tui', 'package.json'))
+
+// 1. Repository + profile naming: never the reserved `tui` / `dsh-tui` family.
+check('root package name is dsh-pi-tui', rootPkg.name === 'dsh-pi-tui', rootPkg.name)
+check('no @deepseek-ai/dsh-tui anywhere', !['@deepseek-ai/dsh-tui', 'dsh-tui'].some(name =>
+  [rootPkg, bundlePkg, forkPkg].some(pkg => pkg.name === name)))
+for (const pkg of [rootPkg, bundlePkg, forkPkg]) {
+  const manifest = JSON.stringify(pkg)
+  if (manifest.includes('@deepseek-ai/dsh-tui') || manifest.includes('"dsh-tui"')) {
+    check(`no dsh-tui references in ${pkg.name}`, false)
+  }
+}
+// The startup service keeps its stable id (internal Loader ids, fine as-is).
+const patch = readFileSync(join(ROOT, 'packages', 'dsh-pi-tui', 'cordis.patch.yml'), 'utf8')
+check('patch rows keep tui-startup/tui-app ids', patch.includes('id: tui-startup') && patch.includes('id: tui-app'))
+check('startup service keeps TUI_STARTUP_SERVICE',
+  readFileSync(join(ROOT, 'packages', 'dsh-pi-tui', 'src', 'startup.ts'), 'utf8').includes("export const TUI_STARTUP_SERVICE = 'tuiStartup'"))
+
+// 2. Package roles: the fork is private + bundled; the bundle is public.
+check('fork package is private (@xmoon76/pi-tui)', forkPkg.private === true, `private=${String(forkPkg.private)}`)
+check('bundle package is public (@xmoon76/dsh-pi-tui)', bundlePkg.private !== true, `private=${String(bundlePkg.private)}`)
+check('bundle declares dsh.bundle.patch', bundlePkg.dsh?.bundle?.patch !== undefined, bundlePkg.dsh?.bundle?.patch)
+
+// 3. The fork is bundled, never a runtime dependency of the published package.
+const tsdown = readFileSync(join(ROOT, 'packages', 'dsh-pi-tui', 'tsdown.config.ts'), 'utf8')
+check('tsdown deps.onlyBundle includes @xmoon76/pi-tui',
+  tsdown.includes("onlyBundle") && tsdown.includes("'@xmoon76/pi-tui'"))
+const runtimeDeps = bundlePkg.dependencies ?? {}
+check('@xmoon76/pi-tui is NOT a runtime dependency of the bundle',
+  !(runtimeDeps['@xmoon76/pi-tui'] !== undefined), runtimeDeps['@xmoon76/pi-tui'] ?? 'absent')
+const forkDeps = forkPkg.dependencies ?? {}
+check('the fork does not depend on the bundle', !(forkDeps['@xmoon76/dsh-pi-tui'] !== undefined))
+
+// 4. The vendored-version single source of truth stays in the fork manifest.
+check('fork repository.note records the vendored commit (single source of truth)',
+  typeof forkPkg.repository?.note === 'string' && forkPkg.repository.note.length > 20,
+  forkPkg.repository?.note ?? 'missing')
+
+// 5. The tarball smoke + pack gate stay wired (stage G).
+check('bundle prepack gate exists', (bundlePkg.scripts?.prepack ?? '').includes('build:package'))
+check('bundle postpack smoke exists', (bundlePkg.scripts?.postpack ?? '').includes('tarball-smoke'))
+
+// 6. The workspace has no stray dist/ or tarballs COMMITTED (dist is
+//    gitignored by design; the gate checks git tracking, not the tree).
+for (const pkg of ['pi-tui', 'dsh-pi-tui']) {
+  const tracked = spawnSync('git', ['ls-files', `packages/${pkg}/dist`], { cwd: ROOT, encoding: 'utf8' }).stdout.trim()
+  check(`packages/${pkg}/dist is not committed`, tracked === '', tracked.split('\n').filter(Boolean).slice(0, 3).join(', '))
+}
+const tgz = spawnSync('git', ['ls-files', '*.tgz'], { cwd: ROOT, encoding: 'utf8' }).stdout.trim()
+check('no tarballs committed', tgz === '', tgz.split('\n').filter(Boolean).slice(0, 3).join(', '))
+
+// 7. Release workflows reference the UN-scoped tarball filename: npm/pnpm
+//    strip the `@` scope from tgz names (xmoon76-dsh-pi-tui-<v>.tgz), so an
+//    `@xmoon76-dsh-pi-tui-*.tgz` glob in a workflow matches nothing and the
+//    gate/upload/publish step fails at runtime. Guard the exact mistake the
+//    2026-08-15 pack-gate review found.
+for (const workflow of ['ci.yml', 'release.yml']) {
+  const path = join(ROOT, '.github', 'workflows', workflow)
+  if (!existsSync(path)) continue // publishing lives inside ci.yml today
+  const text = readFileSync(path, 'utf8')
+  check(`workflow ${workflow} uses unscoped tarball globs`,
+    !text.includes('@xmoon76-dsh-pi-tui-*.tgz') && text.includes('xmoon76-dsh-pi-tui-*.tgz'))
+}
+
+for (const line of checks) console.log(line)
+if (failures.length > 0) {
+  console.error(`\nnaming-gate FAILED (${failures.length}):`)
+  for (const failure of failures) console.error(`  - ${failure}`)
+  process.exit(1)
+}
+console.log('\nnaming-gate passed')