dsh-brief-sidebar 0.3.0 → 0.3.1

package/src/client/summary/DeliverablesSection.tsx

@@ -1,192 +1,192 @@
-/**
- * The deliverables section: the `dshSummaryDeliverables` projection drawn as
- * one part of the summary tab.
- *
- * Unlike the progress section this is an OPTIONAL capability: where the host
- * half never registered the unit (older composition, key absent), the section
- * hides entirely — absence is not an error state here, and the official
- * "本次产出" row in the conversation flow still covers the turn-end view.
- * When the projection is live, the latest turn's produced paths update
- * mid-turn: every successful mutation result publishes a frame, so the reader
- * sees files appear while the turn is still running, which is exactly the gap
- * the official turn-tail row has.
- *
- * Codeplan artifacts (paths under `.agents/plans/<任务名>/`) are ordinary
- * produced entries — they are marked with a pill, not split into a separate
- * section.
- *
- * Colours come from `--dsw-alias-*` tokens only (skin contract).
- */
-import { createElement } from 'react'
-import type { CSSProperties, ReactNode } from 'react'
-import type { Context } from '@deepseek-ai/cordis'
-import { DELIVERABLES_KEY } from '../../projection/keys'
-import { useProjectionValue } from '../use-projection'
-import { sidebarFileOpener } from '../file-open'
-import { basename, readDeliverables, splitCodeplanPath } from './deliverables'
-import { expandCount } from '../locales'
-
-export interface DeliverablesSectionProps {
-  /** The DSH locale lookup, passed down from `apply`. */
-  t: (key: string) => string
-  /** The client root context (the tab body's only way to reach services). */
-  ctx?: Context
-  /** The session this tab is scoped to. */
-  scope?: { readonly sessionId?: string }
-}
-
-const SECTION_STYLE: CSSProperties = {
-  flexShrink: 0,
-  borderBottom: '1px solid var(--dsw-alias-border-secondary)',
-}
-
-const HEADER_STYLE: CSSProperties = {
-  padding: '10px 12px 0',
-  color: 'var(--dsw-alias-label-secondary)',
-  fontSize: 12,
-  fontWeight: 600,
-}
-
-const LIST_STYLE: CSSProperties = {
-  listStyle: 'none',
-  margin: 0,
-  padding: '6px 0',
-  display: 'flex',
-  flexDirection: 'column',
-}
-
-const ROW_STYLE: CSSProperties = {
-  display: 'flex',
-  alignItems: 'flex-start',
-  gap: 8,
-  padding: '6px 12px',
-}
-
-const TOTAL_STYLE: CSSProperties = {
-  padding: '6px 12px 10px',
-  color: 'var(--dsw-alias-label-tertiary)',
-  fontSize: 12,
-}
-
-const PATH_STYLE: CSSProperties = {
-  minWidth: 0,
-  overflowWrap: 'anywhere',
-  color: 'var(--dsw-alias-label-secondary)',
-}
-
-/** Row-as-button reset: the row itself is the click target, content unchanged. */
-const BUTTON_STYLE: CSSProperties = {
-  ...PATH_STYLE,
-  background: 'none',
-  border: 'none',
-  font: 'inherit',
-  fontSize: 'inherit',
-  color: 'var(--dsw-alias-label-secondary)',
-  padding: 0,
-  textAlign: 'left',
-  cursor: 'pointer',
-}
-
-/** The codeplan pill: same form as the status pills, tertiary emphasis. */
-const TAG_STYLE: CSSProperties = {
-  flexShrink: 0,
-  color: 'var(--dsw-alias-label-tertiary)',
-  border: '1px solid var(--dsw-alias-label-tertiary)',
-  borderRadius: 999,
-  padding: '0 6px',
-  fontSize: 11,
-  lineHeight: '16px',
-  whiteSpace: 'nowrap',
-}
-
-/**
- * One produced-file row. When the sidebar opener is available the row is a
- * button that routes the path to the Sidebar preview (the same channel the
- * conversation's produced-file chips use); otherwise it degrades to a plain
- * span. A codeplan artifact carries the pill either way.
- */
-function PathRow(props: {
-  path: string
-  t: (key: string) => string
-  open?: (path: string) => void
-}): ReactNode {
-  const { path, t, open } = props
-  const plan = splitCodeplanPath(path)
-  const title = plan === null ? path : `${t('deliverable.codeplanTag')}: ${plan.task} · ${path}`
-  const label = basename(path)
-  const content = plan === null ? null : createElement('span', { style: TAG_STYLE }, t('deliverable.codeplanTag'))
-  if (open === undefined) {
-    return createElement(
-      'li',
-      { style: ROW_STYLE },
-      content,
-      createElement('span', { style: PATH_STYLE, title }, label),
-    )
-  }
-  return createElement(
-    'li',
-    { style: ROW_STYLE },
-    content,
-    createElement('button', { style: BUTTON_STYLE, title, onClick: () => { open(path) } }, label),
-  )
-}
-
-/**
- * The deliverables section. The subscription is unconditional (a plain
- * listener, not IO).
- * @param props - translation, client context, session scope.
- */
-export function DeliverablesSection(props: DeliverablesSectionProps): ReactNode {
-  const { t, ctx, scope } = props
-
-  const view = readDeliverables(useProjectionValue(ctx, scope?.sessionId, DELIVERABLES_KEY))
-
-  // Capability absent — the optional unit never registered. Hide, don't nag.
-  if (view === undefined) return null
-
-  // Optional click-to-preview: rows degrade to plain text where the sidebar
-  // service is absent (the opener resolves the session cwd per click).
-  const open = sidebarFileOpener(ctx, scope?.sessionId)
-
-  const latestPaths = view.latest?.paths ?? []
-
-  if (latestPaths.length === 0 && view.sessionTotal === 0) {
-    return createElement(
-      'div',
-      { style: SECTION_STYLE, 'data-dsh-brief-sidebar': 'deliverables-section' },
-      createElement('div', { style: HEADER_STYLE }, t('section.deliverables')),
-      createElement(
-        'div',
-        {
-          style: {
-            margin: '0 auto',
-            padding: 24,
-            maxWidth: 460,
-            textAlign: 'center',
-            color: 'var(--dsw-alias-label-tertiary)',
-            lineHeight: 1.7,
-            fontSize: 12,
-          },
-        },
-        createElement('div', { style: { marginBottom: 6, color: 'var(--dsw-alias-label-secondary)' } }, t('deliverables.empty')),
-        createElement('div', null, t('deliverables.emptyHint')),
-      ),
-    )
-  }
-
-  return createElement(
-    'div',
-    { style: SECTION_STYLE, 'data-dsh-brief-sidebar': 'deliverables-section' },
-    createElement('div', { style: HEADER_STYLE }, t('section.deliverables')),
-    createElement(
-      'ul',
-      { style: LIST_STYLE },
-      latestPaths.map(path => createElement(PathRow, { key: path, path, t, open })),
-    ),
-    createElement(
-      'div',
-      { style: TOTAL_STYLE },
-      expandCount(t('deliverables.sessionTotal'), view.sessionTotal),
-    ),
-  )
-}
+/**
+ * The deliverables section: the `dshSummaryDeliverables` projection drawn as
+ * one part of the summary tab.
+ *
+ * Unlike the progress section this is an OPTIONAL capability: where the host
+ * half never registered the unit (older composition, key absent), the section
+ * hides entirely — absence is not an error state here, and the official
+ * "本次产出" row in the conversation flow still covers the turn-end view.
+ * When the projection is live, the latest turn's produced paths update
+ * mid-turn: every successful mutation result publishes a frame, so the reader
+ * sees files appear while the turn is still running, which is exactly the gap
+ * the official turn-tail row has.
+ *
+ * Codeplan artifacts (paths under `.agents/plans/<任务名>/`) are ordinary
+ * produced entries — they are marked with a pill, not split into a separate
+ * section.
+ *
+ * Colours come from `--dsw-alias-*` tokens only (skin contract).
+ */
+import { createElement } from 'react'
+import type { CSSProperties, ReactNode } from 'react'
+import type { Context } from '@deepseek-ai/cordis'
+import { DELIVERABLES_KEY } from '../../projection/keys'
+import { useProjectionValue } from '../use-projection'
+import { sidebarFileOpener } from '../file-open'
+import { basename, readDeliverables, splitCodeplanPath } from './deliverables'
+import { expandCount } from '../locales'
+
+export interface DeliverablesSectionProps {
+  /** The DSH locale lookup, passed down from `apply`. */
+  t: (key: string) => string
+  /** The client root context (the tab body's only way to reach services). */
+  ctx?: Context
+  /** The session this tab is scoped to. */
+  scope?: { readonly sessionId?: string }
+}
+
+const SECTION_STYLE: CSSProperties = {
+  flexShrink: 0,
+  borderBottom: '1px solid var(--dsw-alias-border-secondary)',
+}
+
+const HEADER_STYLE: CSSProperties = {
+  padding: '10px 12px 0',
+  color: 'var(--dsw-alias-label-secondary)',
+  fontSize: 12,
+  fontWeight: 600,
+}
+
+const LIST_STYLE: CSSProperties = {
+  listStyle: 'none',
+  margin: 0,
+  padding: '6px 0',
+  display: 'flex',
+  flexDirection: 'column',
+}
+
+const ROW_STYLE: CSSProperties = {
+  display: 'flex',
+  alignItems: 'flex-start',
+  gap: 8,
+  padding: '6px 12px',
+}
+
+const TOTAL_STYLE: CSSProperties = {
+  padding: '6px 12px 10px',
+  color: 'var(--dsw-alias-label-tertiary)',
+  fontSize: 12,
+}
+
+const PATH_STYLE: CSSProperties = {
+  minWidth: 0,
+  overflowWrap: 'anywhere',
+  color: 'var(--dsw-alias-label-secondary)',
+}
+
+/** Row-as-button reset: the row itself is the click target, content unchanged. */
+const BUTTON_STYLE: CSSProperties = {
+  ...PATH_STYLE,
+  background: 'none',
+  border: 'none',
+  font: 'inherit',
+  fontSize: 'inherit',
+  color: 'var(--dsw-alias-label-secondary)',
+  padding: 0,
+  textAlign: 'left',
+  cursor: 'pointer',
+}
+
+/** The codeplan pill: same form as the status pills, tertiary emphasis. */
+const TAG_STYLE: CSSProperties = {
+  flexShrink: 0,
+  color: 'var(--dsw-alias-label-tertiary)',
+  border: '1px solid var(--dsw-alias-label-tertiary)',
+  borderRadius: 999,
+  padding: '0 6px',
+  fontSize: 11,
+  lineHeight: '16px',
+  whiteSpace: 'nowrap',
+}
+
+/**
+ * One produced-file row. When the sidebar opener is available the row is a
+ * button that routes the path to the Sidebar preview (the same channel the
+ * conversation's produced-file chips use); otherwise it degrades to a plain
+ * span. A codeplan artifact carries the pill either way.
+ */
+function PathRow(props: {
+  path: string
+  t: (key: string) => string
+  open?: (path: string) => void
+}): ReactNode {
+  const { path, t, open } = props
+  const plan = splitCodeplanPath(path)
+  const title = plan === null ? path : `${t('deliverable.codeplanTag')}: ${plan.task} · ${path}`
+  const label = basename(path)
+  const content = plan === null ? null : createElement('span', { style: TAG_STYLE }, t('deliverable.codeplanTag'))
+  if (open === undefined) {
+    return createElement(
+      'li',
+      { style: ROW_STYLE },
+      content,
+      createElement('span', { style: PATH_STYLE, title }, label),
+    )
+  }
+  return createElement(
+    'li',
+    { style: ROW_STYLE },
+    content,
+    createElement('button', { style: BUTTON_STYLE, title, onClick: () => { open(path) } }, label),
+  )
+}
+
+/**
+ * The deliverables section. The subscription is unconditional (a plain
+ * listener, not IO).
+ * @param props - translation, client context, session scope.
+ */
+export function DeliverablesSection(props: DeliverablesSectionProps): ReactNode {
+  const { t, ctx, scope } = props
+
+  const view = readDeliverables(useProjectionValue(ctx, scope?.sessionId, DELIVERABLES_KEY))
+
+  // Capability absent — the optional unit never registered. Hide, don't nag.
+  if (view === undefined) return null
+
+  // Optional click-to-preview: rows degrade to plain text where the sidebar
+  // service is absent (the opener resolves the session cwd per click).
+  const open = sidebarFileOpener(ctx, scope?.sessionId)
+
+  const latestPaths = view.latest?.paths ?? []
+
+  if (latestPaths.length === 0 && view.sessionTotal === 0) {
+    return createElement(
+      'div',
+      { style: SECTION_STYLE, 'data-dsh-brief-sidebar': 'deliverables-section' },
+      createElement('div', { style: HEADER_STYLE }, t('section.deliverables')),
+      createElement(
+        'div',
+        {
+          style: {
+            margin: '0 auto',
+            padding: 24,
+            maxWidth: 460,
+            textAlign: 'center',
+            color: 'var(--dsw-alias-label-tertiary)',
+            lineHeight: 1.7,
+            fontSize: 12,
+          },
+        },
+        createElement('div', { style: { marginBottom: 6, color: 'var(--dsw-alias-label-secondary)' } }, t('deliverables.empty')),
+        createElement('div', null, t('deliverables.emptyHint')),
+      ),
+    )
+  }
+
+  return createElement(
+    'div',
+    { style: SECTION_STYLE, 'data-dsh-brief-sidebar': 'deliverables-section' },
+    createElement('div', { style: HEADER_STYLE }, t('section.deliverables')),
+    createElement(
+      'ul',
+      { style: LIST_STYLE },
+      latestPaths.map(path => createElement(PathRow, { key: path, path, t, open })),
+    ),
+    createElement(
+      'div',
+      { style: TOTAL_STYLE },
+      expandCount(t('deliverables.sessionTotal'), view.sessionTotal),
+    ),
+  )
+}

package/src/client/summary/deliverables.ts

@@ -1,93 +1,93 @@
-/**
- * The `dshSummaryDeliverables` projection — reading and classifying it.
- *
- * The Host computes this projection (this plugin's own node half registers the
- * unit) and pushes finished whole values; the client never folds it. This
- * module is the read side of that contract and is deliberately PURE — no
- * React, no context, no IO — so the narrowing and the codeplan classification
- * can be tested directly.
- */
-import type { DeliverablesView } from '../../projection/keys'
-
-/**
- * Narrow a raw projection value into a deliverables view.
- *
- * Two outcomes are kept apart on purpose, mirroring the store's contract:
- * - `undefined` — the capability is ABSENT (the host half never registered —
- *   e.g. an older composition — or no frame carried the key yet);
- * - a view — the projection is live. A view with `latest: null` and
- *   `sessionTotal: 0` IS the empty state; the fold starts publishing as soon
- *   as the unit registers, so "present but empty" is a real value here, not a
- *   sentinel.
- *
- * @param value - the raw `unknown` snapshot of the deliverables face.
- * @returns the validated view, or `undefined` when the capability is absent.
- */
-export function readDeliverables(value: unknown): DeliverablesView | undefined {
-  if (value === null || typeof value !== 'object' || Array.isArray(value)) return undefined
-  const raw = value as { latest?: unknown; sessionPaths?: unknown; sessionTotal?: unknown }
-  if (raw.latest === undefined || raw.sessionPaths === undefined || raw.sessionTotal === undefined) {
-    return undefined
-  }
-  if (!isBucket(raw.latest) || !Array.isArray(raw.sessionPaths) || typeof raw.sessionTotal !== 'number') {
-    return undefined
-  }
-  const paths = raw.sessionPaths.filter((path): path is string => typeof path === 'string')
-  return {
-    latest: raw.latest === null ? null : { turn: raw.latest.turn, paths: raw.latest.paths },
-    sessionPaths: paths,
-    sessionTotal: raw.sessionTotal,
-  }
-}
-
-/** Whether a value is a well-formed latest-turn bucket (or explicitly null). */
-function isBucket(value: unknown): value is { turn: number; paths: readonly string[] } | null {
-  if (value === null) return true
-  if (typeof value !== 'object') return false
-  const bucket = value as { turn?: unknown; paths?: unknown }
-  return typeof bucket.turn === 'number' && Array.isArray(bucket.paths)
-}
-
-/**
- * The path prefix that identifies a codeplan artifact. The codeplan skill
- * writes its four fixed artifacts under `$workspace\.agents\plans\<任务名>\`;
- * produced paths may spell the separators either way.
- */
-const CODEPLAN_SEGMENT = '.agents/plans/'
-
-/**
- * Normalize path separators so one spelling of the codeplan prefix matches.
- * Only classification uses this; the displayed path keeps the tool's exact
- * spelling.
- */
-function normalized(path: string): string {
-  return path.replaceAll('\\', '/')
-}
-
-/** Whether a produced path is a codeplan artifact. */
-export function isCodeplanPath(path: string): boolean {
-  return normalized(path).includes(CODEPLAN_SEGMENT)
-}
-
-/**
- * Split a codeplan path into its task name and file name.
- *
- * @param path - a produced path; need not actually be a codeplan artifact.
- * @returns task and file names, or `null` when the path is not under the
- *   codeplan plans directory (or names no file beneath the task folder).
- */
-export function splitCodeplanPath(path: string): { task: string; file: string } | null {
-  const marker = normalized(path).indexOf(CODEPLAN_SEGMENT)
-  if (marker < 0) return null
-  const rest = normalized(path).slice(marker + CODEPLAN_SEGMENT.length)
-  const slash = rest.indexOf('/')
-  if (slash <= 0 || slash === rest.length - 1) return null
-  return { task: rest.slice(0, slash), file: rest.slice(slash + 1) }
-}
-
-/** The final path segment, under either separator. */
-export function basename(path: string): string {
-  const normalizedPath = normalized(path)
-  const slash = normalizedPath.lastIndexOf('/')
-  return slash < 0 ? normalizedPath : normalizedPath.slice(slash + 1)
-}
+/**
+ * The `dshSummaryDeliverables` projection — reading and classifying it.
+ *
+ * The Host computes this projection (this plugin's own node half registers the
+ * unit) and pushes finished whole values; the client never folds it. This
+ * module is the read side of that contract and is deliberately PURE — no
+ * React, no context, no IO — so the narrowing and the codeplan classification
+ * can be tested directly.
+ */
+import type { DeliverablesView } from '../../projection/keys'
+
+/**
+ * Narrow a raw projection value into a deliverables view.
+ *
+ * Two outcomes are kept apart on purpose, mirroring the store's contract:
+ * - `undefined` — the capability is ABSENT (the host half never registered —
+ *   e.g. an older composition — or no frame carried the key yet);
+ * - a view — the projection is live. A view with `latest: null` and
+ *   `sessionTotal: 0` IS the empty state; the fold starts publishing as soon
+ *   as the unit registers, so "present but empty" is a real value here, not a
+ *   sentinel.
+ *
+ * @param value - the raw `unknown` snapshot of the deliverables face.
+ * @returns the validated view, or `undefined` when the capability is absent.
+ */
+export function readDeliverables(value: unknown): DeliverablesView | undefined {
+  if (value === null || typeof value !== 'object' || Array.isArray(value)) return undefined
+  const raw = value as { latest?: unknown; sessionPaths?: unknown; sessionTotal?: unknown }
+  if (raw.latest === undefined || raw.sessionPaths === undefined || raw.sessionTotal === undefined) {
+    return undefined
+  }
+  if (!isBucket(raw.latest) || !Array.isArray(raw.sessionPaths) || typeof raw.sessionTotal !== 'number') {
+    return undefined
+  }
+  const paths = raw.sessionPaths.filter((path): path is string => typeof path === 'string')
+  return {
+    latest: raw.latest === null ? null : { turn: raw.latest.turn, paths: raw.latest.paths },
+    sessionPaths: paths,
+    sessionTotal: raw.sessionTotal,
+  }
+}
+
+/** Whether a value is a well-formed latest-turn bucket (or explicitly null). */
+function isBucket(value: unknown): value is { turn: number; paths: readonly string[] } | null {
+  if (value === null) return true
+  if (typeof value !== 'object') return false
+  const bucket = value as { turn?: unknown; paths?: unknown }
+  return typeof bucket.turn === 'number' && Array.isArray(bucket.paths)
+}
+
+/**
+ * The path prefix that identifies a codeplan artifact. The codeplan skill
+ * writes its four fixed artifacts under `$workspace\.agents\plans\<任务名>\`;
+ * produced paths may spell the separators either way.
+ */
+const CODEPLAN_SEGMENT = '.agents/plans/'
+
+/**
+ * Normalize path separators so one spelling of the codeplan prefix matches.
+ * Only classification uses this; the displayed path keeps the tool's exact
+ * spelling.
+ */
+function normalized(path: string): string {
+  return path.replaceAll('\\', '/')
+}
+
+/** Whether a produced path is a codeplan artifact. */
+export function isCodeplanPath(path: string): boolean {
+  return normalized(path).includes(CODEPLAN_SEGMENT)
+}
+
+/**
+ * Split a codeplan path into its task name and file name.
+ *
+ * @param path - a produced path; need not actually be a codeplan artifact.
+ * @returns task and file names, or `null` when the path is not under the
+ *   codeplan plans directory (or names no file beneath the task folder).
+ */
+export function splitCodeplanPath(path: string): { task: string; file: string } | null {
+  const marker = normalized(path).indexOf(CODEPLAN_SEGMENT)
+  if (marker < 0) return null
+  const rest = normalized(path).slice(marker + CODEPLAN_SEGMENT.length)
+  const slash = rest.indexOf('/')
+  if (slash <= 0 || slash === rest.length - 1) return null
+  return { task: rest.slice(0, slash), file: rest.slice(slash + 1) }
+}
+
+/** The final path segment, under either separator. */
+export function basename(path: string): string {
+  const normalizedPath = normalized(path)
+  const slash = normalizedPath.lastIndexOf('/')
+  return slash < 0 ? normalizedPath : normalizedPath.slice(slash + 1)
+}