dsh-brief-sidebar 0.3.0 → 0.3.1

package/src/client/brief/dock-shadow.tsx

@@ -1,74 +1,74 @@
-/**
- * Hiding the official todo dock.
- *
- * `conversation.input.dock` is a LIST slot, and its cells are keyed by each
- * entry's `id`. The slot core keeps one live entry per cell — entries are
- * sorted by `priority` ascending (then by `order`) and the first live entry of
- * a cell is the one the outlet renders, so a SECOND registration with the same
- * `id` at a LOWER priority shadows the first. The official todo entry
- * (`conversation-todo-dock` in `dsh-client-ui-conversation`) registers
- * `{ id: 'todo', order: 0 }` with the default priority 0, so
- * `{ id: 'todo', priority: -1 }` wins the cell and the official panel never
- * renders while this plugin is mounted.
- *
- * This is the same mechanism `dsh-input-traffic` uses on the sibling `queue`
- * cell (`{ id: 'queue', priority: -1 }`), so it is a proven seam on this host
- * generation, not a guess.
- *
- * Binding to a built-in id is a real coupling: if upstream renames the cell,
- * the shadow silently stops winning and the official dock simply reappears
- * (fail-open — the data is still shown, just twice). `slots.inject` is used
- * rather than a bare `register` because the slot is declared by a built-in
- * entry whose lifetime we do not control: `inject` waits for the declaration
- * and re-installs the entry if the declaration is ever recreated.
- *
- * Renders nothing on purpose: the todo UI now lives in this plugin's sidebar
- * tab. Return a summary chip here instead to keep a one-line entry point in the
- * composer band.
- */
-import type { ReactNode } from 'react'
-import type { Context } from '@deepseek-ai/cordis'
-
-/** The built-in cell id of the official todo dock. */
-export const TODO_CELL_ID = 'todo'
-
-/** Lower than the built-in's default 0 — the winner of the `todo` cell. */
-export const TODO_SHADOW_PRIORITY = -1
-
-/** Structural view of the client slots service (avoids a second type graph). */
-export interface SlotsLike {
-  inject(key: string, callback: () => (() => void) | undefined): () => void
-  register(options: Record<string, unknown>, component: (props: unknown) => ReactNode): () => void
-}
-
-/** The replacement entry: deliberately empty. */
-function TodoDockShadow(): ReactNode {
-  return null
-}
-
-/**
- * Register the shadowing entry.
- *
- * The caller wraps this in `ctx.effect(...)` so the returned disposer rides the
- * plugin fiber: unloading the plugin restores the official dock.
- *
- * @param ctx - the client root context.
- * @returns the disposer, or undefined when the slots service is unreachable.
- */
-export function registerTodoDockShadow(ctx: Context): (() => void) | undefined {
-  const slots = ctx.get('slots') as SlotsLike | undefined
-  if (slots === undefined || typeof slots.inject !== 'function') return undefined
-
-  return slots.inject('conversation.input.dock', () =>
-    slots.register(
-      {
-        name: 'conversation.input.dock',
-        id: TODO_CELL_ID,
-        order: 0,
-        priority: TODO_SHADOW_PRIORITY,
-        registrant: 'dsh-brief-sidebar',
-      },
-      TodoDockShadow,
-    ),
-  )
-}
+/**
+ * Hiding the official todo dock.
+ *
+ * `conversation.input.dock` is a LIST slot, and its cells are keyed by each
+ * entry's `id`. The slot core keeps one live entry per cell — entries are
+ * sorted by `priority` ascending (then by `order`) and the first live entry of
+ * a cell is the one the outlet renders, so a SECOND registration with the same
+ * `id` at a LOWER priority shadows the first. The official todo entry
+ * (`conversation-todo-dock` in `dsh-client-ui-conversation`) registers
+ * `{ id: 'todo', order: 0 }` with the default priority 0, so
+ * `{ id: 'todo', priority: -1 }` wins the cell and the official panel never
+ * renders while this plugin is mounted.
+ *
+ * This is the same mechanism `dsh-input-traffic` uses on the sibling `queue`
+ * cell (`{ id: 'queue', priority: -1 }`), so it is a proven seam on this host
+ * generation, not a guess.
+ *
+ * Binding to a built-in id is a real coupling: if upstream renames the cell,
+ * the shadow silently stops winning and the official dock simply reappears
+ * (fail-open — the data is still shown, just twice). `slots.inject` is used
+ * rather than a bare `register` because the slot is declared by a built-in
+ * entry whose lifetime we do not control: `inject` waits for the declaration
+ * and re-installs the entry if the declaration is ever recreated.
+ *
+ * Renders nothing on purpose: the todo UI now lives in this plugin's sidebar
+ * tab. Return a summary chip here instead to keep a one-line entry point in the
+ * composer band.
+ */
+import type { ReactNode } from 'react'
+import type { Context } from '@deepseek-ai/cordis'
+
+/** The built-in cell id of the official todo dock. */
+export const TODO_CELL_ID = 'todo'
+
+/** Lower than the built-in's default 0 — the winner of the `todo` cell. */
+export const TODO_SHADOW_PRIORITY = -1
+
+/** Structural view of the client slots service (avoids a second type graph). */
+export interface SlotsLike {
+  inject(key: string, callback: () => (() => void) | undefined): () => void
+  register(options: Record<string, unknown>, component: (props: unknown) => ReactNode): () => void
+}
+
+/** The replacement entry: deliberately empty. */
+function TodoDockShadow(): ReactNode {
+  return null
+}
+
+/**
+ * Register the shadowing entry.
+ *
+ * The caller wraps this in `ctx.effect(...)` so the returned disposer rides the
+ * plugin fiber: unloading the plugin restores the official dock.
+ *
+ * @param ctx - the client root context.
+ * @returns the disposer, or undefined when the slots service is unreachable.
+ */
+export function registerTodoDockShadow(ctx: Context): (() => void) | undefined {
+  const slots = ctx.get('slots') as SlotsLike | undefined
+  if (slots === undefined || typeof slots.inject !== 'function') return undefined
+
+  return slots.inject('conversation.input.dock', () =>
+    slots.register(
+      {
+        name: 'conversation.input.dock',
+        id: TODO_CELL_ID,
+        order: 0,
+        priority: TODO_SHADOW_PRIORITY,
+        registrant: 'dsh-brief-sidebar',
+      },
+      TodoDockShadow,
+    ),
+  )
+}

package/src/client/brief/use-todos.ts

@@ -1,27 +1,27 @@
-/**
- * Reading the live `todos` projection — the thin, todo-specific wrapper around
- * the generic face resolver (`../use-projection`). The projection key is
- * registered by `dsh-tool-todo`; the host pushes finished whole values and the
- * client never folds it. Narrowing lives in `./board` (`readTodos`).
- */
-import { useMemo } from 'react'
-import type { Context } from '@deepseek-ai/cordis'
-import { readTodos } from './board'
-import type { TodoItem } from './board'
-import { useProjectionValue } from '../use-projection'
-
-/** The projection key registered by `dsh-tool-todo`. */
-export const TODOS_KEY = 'todos'
-
-/**
- * Subscribe to one session's todo list.
- *
- * @param ctx - the client root context.
- * @param sessionId - the session whose projection is shown.
- * @returns the list, `[]` for a present-but-empty projection, or `undefined`
- *   while the capability is absent.
- */
-export function useTodos(ctx: Context | undefined, sessionId: string | undefined): readonly TodoItem[] | undefined {
-  const raw = useProjectionValue(ctx, sessionId, TODOS_KEY)
-  return useMemo(() => readTodos(raw), [raw])
-}
+/**
+ * Reading the live `todos` projection — the thin, todo-specific wrapper around
+ * the generic face resolver (`../use-projection`). The projection key is
+ * registered by `dsh-tool-todo`; the host pushes finished whole values and the
+ * client never folds it. Narrowing lives in `./board` (`readTodos`).
+ */
+import { useMemo } from 'react'
+import type { Context } from '@deepseek-ai/cordis'
+import { readTodos } from './board'
+import type { TodoItem } from './board'
+import { useProjectionValue } from '../use-projection'
+
+/** The projection key registered by `dsh-tool-todo`. */
+export const TODOS_KEY = 'todos'
+
+/**
+ * Subscribe to one session's todo list.
+ *
+ * @param ctx - the client root context.
+ * @param sessionId - the session whose projection is shown.
+ * @returns the list, `[]` for a present-but-empty projection, or `undefined`
+ *   while the capability is absent.
+ */
+export function useTodos(ctx: Context | undefined, sessionId: string | undefined): readonly TodoItem[] | undefined {
+  const raw = useProjectionValue(ctx, sessionId, TODOS_KEY)
+  return useMemo(() => readTodos(raw), [raw])
+}

package/src/client/file-open.ts

@@ -1,108 +1,108 @@
-/**
- * Opening a produced file's preview in the right Sidebar — the same channel
- * the conversation flow uses for its produced-file chips and inline mentions
- * (ui-chat `openFile` → `ctx.sidebarRight.openResource` with a
- * `dsh-resource://file/…` address).
- *
- * The address grammar is ported verbatim from the harness's browser-safe
- * `@deepseek-ai/dsh-util-workspace-path` (`file-address.ts` / `index.ts`),
- * because the plugin does not compile against that package: a relative path,
- * or an absolute path inside the session's workspace, is addressed under the
- * session's scope (`dsh-resource://file/session/<id>/<path>`); an absolute
- * path elsewhere keeps its absolute spelling in the same session's address.
- * Which tab type claims the address is the Sidebar's decision, not ours.
- *
- * The `sidebarRight` service is consumed structurally and treated as OPTIONAL:
- * where it is absent the section renders plain rows instead of links, the same
- * graceful degradation as the deliverables projection itself.
- */
-import type { Context } from '@deepseek-ai/cordis'
-
-/** Structural view of the right-Sidebar service this plugin needs. */
-export interface SidebarRightLike {
-  openResource(url: string, options?: { params?: { line?: number } }): void
-}
-
-/** Structural view of the sessions service slice the opener reads. */
-interface SessionsListLike {
-  list?: { getSnapshot(): { byId?: Record<string, { cwd?: string }> } }
-}
-
-/** The address scheme every file resource opens with. */
-const FILE_ADDRESS_PREFIX = 'dsh-resource://file/'
-
-/** Whether a path uses a Windows drive or UNC prefix. */
-function isWindowsStylePath(value: string): boolean {
-  return /^[A-Za-z]:[/\\]/.test(value) || value.startsWith('\\\\')
-}
-
-/**
- * Whether a path is absolute in either spelling the Host accepts: POSIX (`/a/b`)
- * or Windows drive or UNC.
- */
-export function isAbsoluteWorkspacePath(path: string): boolean {
-  return path.startsWith('/') || isWindowsStylePath(path)
-}
-
-/** Component-encode one id or path segment, keeping `:` literal for drive letters. */
-function encodeSegment(segment: string): string {
-  return encodeURIComponent(segment).replace(/%3A/gi, ':')
-}
-
-/** Encode a `/`-separated path segment by segment. */
-function encodePath(path: string): string {
-  return path.split('/').map(encodeSegment).join('/')
-}
-
-/**
- * Build the address of a file read through one Session.
- * @param sessionId - the Session whose Host workspace resolves the path.
- * @param path - absolute or workspace-relative path; backslashes are normalized
- *   to `/`, and leading `./` prefixes are dropped.
- */
-function sessionFileAddress(sessionId: string, path: string): string {
-  const normalized = path.replace(/\\/g, '/').replace(/^(?:\.\/)+/, '')
-  return `${FILE_ADDRESS_PREFIX}session/${encodeSegment(sessionId)}/${encodePath(normalized)}`
-}
-
-/**
- * The address for a path as a caller holds it.
- * @param sessionId - the Session the path is read in.
- * @param cwd - that Session's workspace root, when known.
- * @param path - absolute or workspace-relative path, in either separator spelling.
- */
-export function fileAddressFor(sessionId: string, cwd: string | undefined, path: string): string {
-  const normalized = path.replace(/\\/g, '/')
-  if (!isAbsoluteWorkspacePath(normalized)) return sessionFileAddress(sessionId, normalized)
-  const root = cwd === undefined ? '' : cwd.replace(/\\/g, '/').replace(/\/+$/, '')
-  if (root !== '' && normalized === root) return sessionFileAddress(sessionId, '')
-  if (root !== '' && normalized.startsWith(`${root}/`)) {
-    return sessionFileAddress(sessionId, normalized.slice(root.length + 1))
-  }
-  return sessionFileAddress(sessionId, normalized)
-}
-
-/**
- * Build a sidebar file opener for one session, or `undefined` when the
- * environment cannot open previews (no `sidebarRight` service, no session).
- * The cwd is re-read per call inside the opener, so a workspace switch between
- * renders is picked up on the next click.
- *
- * @param ctx - the client root context.
- * @param sessionId - the session whose workspace resolves paths.
- * @returns an opener that routes a produced path to the Sidebar preview.
- */
-export function sidebarFileOpener(
-  ctx: Context | undefined,
-  sessionId: string | undefined,
-): ((path: string) => void) | undefined {
-  if (ctx === undefined || sessionId === undefined) return undefined
-  const bar = ctx.get('sidebarRight') as SidebarRightLike | undefined
-  if (bar === undefined || typeof bar.openResource !== 'function') return undefined
-
-  return (path: string) => {
-    const sessions = ctx.get('sessions') as SessionsListLike | undefined
-    const cwd = sessions?.list?.getSnapshot?.().byId?.[sessionId]?.cwd
-    bar.openResource(fileAddressFor(sessionId, cwd, path))
-  }
-}
+/**
+ * Opening a produced file's preview in the right Sidebar — the same channel
+ * the conversation flow uses for its produced-file chips and inline mentions
+ * (ui-chat `openFile` → `ctx.sidebarRight.openResource` with a
+ * `dsh-resource://file/…` address).
+ *
+ * The address grammar is ported verbatim from the harness's browser-safe
+ * `@deepseek-ai/dsh-util-workspace-path` (`file-address.ts` / `index.ts`),
+ * because the plugin does not compile against that package: a relative path,
+ * or an absolute path inside the session's workspace, is addressed under the
+ * session's scope (`dsh-resource://file/session/<id>/<path>`); an absolute
+ * path elsewhere keeps its absolute spelling in the same session's address.
+ * Which tab type claims the address is the Sidebar's decision, not ours.
+ *
+ * The `sidebarRight` service is consumed structurally and treated as OPTIONAL:
+ * where it is absent the section renders plain rows instead of links, the same
+ * graceful degradation as the deliverables projection itself.
+ */
+import type { Context } from '@deepseek-ai/cordis'
+
+/** Structural view of the right-Sidebar service this plugin needs. */
+export interface SidebarRightLike {
+  openResource(url: string, options?: { params?: { line?: number } }): void
+}
+
+/** Structural view of the sessions service slice the opener reads. */
+interface SessionsListLike {
+  list?: { getSnapshot(): { byId?: Record<string, { cwd?: string }> } }
+}
+
+/** The address scheme every file resource opens with. */
+const FILE_ADDRESS_PREFIX = 'dsh-resource://file/'
+
+/** Whether a path uses a Windows drive or UNC prefix. */
+function isWindowsStylePath(value: string): boolean {
+  return /^[A-Za-z]:[/\\]/.test(value) || value.startsWith('\\\\')
+}
+
+/**
+ * Whether a path is absolute in either spelling the Host accepts: POSIX (`/a/b`)
+ * or Windows drive or UNC.
+ */
+export function isAbsoluteWorkspacePath(path: string): boolean {
+  return path.startsWith('/') || isWindowsStylePath(path)
+}
+
+/** Component-encode one id or path segment, keeping `:` literal for drive letters. */
+function encodeSegment(segment: string): string {
+  return encodeURIComponent(segment).replace(/%3A/gi, ':')
+}
+
+/** Encode a `/`-separated path segment by segment. */
+function encodePath(path: string): string {
+  return path.split('/').map(encodeSegment).join('/')
+}
+
+/**
+ * Build the address of a file read through one Session.
+ * @param sessionId - the Session whose Host workspace resolves the path.
+ * @param path - absolute or workspace-relative path; backslashes are normalized
+ *   to `/`, and leading `./` prefixes are dropped.
+ */
+function sessionFileAddress(sessionId: string, path: string): string {
+  const normalized = path.replace(/\\/g, '/').replace(/^(?:\.\/)+/, '')
+  return `${FILE_ADDRESS_PREFIX}session/${encodeSegment(sessionId)}/${encodePath(normalized)}`
+}
+
+/**
+ * The address for a path as a caller holds it.
+ * @param sessionId - the Session the path is read in.
+ * @param cwd - that Session's workspace root, when known.
+ * @param path - absolute or workspace-relative path, in either separator spelling.
+ */
+export function fileAddressFor(sessionId: string, cwd: string | undefined, path: string): string {
+  const normalized = path.replace(/\\/g, '/')
+  if (!isAbsoluteWorkspacePath(normalized)) return sessionFileAddress(sessionId, normalized)
+  const root = cwd === undefined ? '' : cwd.replace(/\\/g, '/').replace(/\/+$/, '')
+  if (root !== '' && normalized === root) return sessionFileAddress(sessionId, '')
+  if (root !== '' && normalized.startsWith(`${root}/`)) {
+    return sessionFileAddress(sessionId, normalized.slice(root.length + 1))
+  }
+  return sessionFileAddress(sessionId, normalized)
+}
+
+/**
+ * Build a sidebar file opener for one session, or `undefined` when the
+ * environment cannot open previews (no `sidebarRight` service, no session).
+ * The cwd is re-read per call inside the opener, so a workspace switch between
+ * renders is picked up on the next click.
+ *
+ * @param ctx - the client root context.
+ * @param sessionId - the session whose workspace resolves paths.
+ * @returns an opener that routes a produced path to the Sidebar preview.
+ */
+export function sidebarFileOpener(
+  ctx: Context | undefined,
+  sessionId: string | undefined,
+): ((path: string) => void) | undefined {
+  if (ctx === undefined || sessionId === undefined) return undefined
+  const bar = ctx.get('sidebarRight') as SidebarRightLike | undefined
+  if (bar === undefined || typeof bar.openResource !== 'function') return undefined
+
+  return (path: string) => {
+    const sessions = ctx.get('sessions') as SessionsListLike | undefined
+    const cwd = sessions?.list?.getSnapshot?.().byId?.[sessionId]?.cwd
+    bar.openResource(fileAddressFor(sessionId, cwd, path))
+  }
+}