dsh-remote-vs 0.9.0 → 0.9.2

package/src/client/selection-popup.ts

@@ -33,15 +33,15 @@
 import { useEffect, useRef, useState, type RefObject } from 'react'
 
 /** The floating "add to conversation" action: payload + viewport anchor. */
-export interface SelectionPopup {
-  insert: string
+export interface SelectionPopup<T> {
+  insert: T
   left: number
   top: number
 }
 
-export interface SelectionPopupOptions {
+export interface SelectionPopupOptions<T> {
   /** Commit the payload into the composer draft (button click). */
-  onCommit(insert: string): void
+  onCommit(insert: T): void
   /**
    * The DOM surface that must stay on screen for the popup to live: the
    * CodeMirror host. Called lazily (refs are null until the content loads).
@@ -49,20 +49,20 @@ export interface SelectionPopupOptions {
   getSurface(): HTMLElement | null
 }
 
-export interface SelectionPopupControls {
+export interface SelectionPopupControls<T> {
   /** The current popup (null = hidden). */
-  popup: SelectionPopup | null
+  popup: SelectionPopup<T> | null
   /** Attach to the portaled button element. */
   buttonRef: RefObject<HTMLButtonElement>
   /** Anchor the popup above a selection (viewport-clamped). */
-  show(insert: string, left: number, top: number): void
+  show(insert: T, left: number, top: number): void
   /** Hide the popup (idempotent). */
   hide(): void
   /** The button's click: commit the stored payload, then hide. */
   commit(): void
 }
 
-export function useSelectionPopup(options: SelectionPopupOptions): SelectionPopupControls {
+export function useSelectionPopup<T>(options: SelectionPopupOptions<T>): SelectionPopupControls<T> {
   // Latest-callback refs: the dismissal listeners live for the mount's
   // lifetime, so they must not capture stale closures across renders.
   const onCommitRef = useRef(options.onCommit)
@@ -70,16 +70,16 @@ export function useSelectionPopup(options: SelectionPopupOptions): SelectionPopu
   onCommitRef.current = options.onCommit
   getSurfaceRef.current = options.getSurface
 
-  const [popup, setPopup] = useState<SelectionPopup | null>(null)
+  const [popup, setPopup] = useState<SelectionPopup<T> | null>(null)
   /** Live mirror for click/event-time reads (no re-render race). */
-  const popupRef = useRef<SelectionPopup | null>(null)
+  const popupRef = useRef<SelectionPopup<T> | null>(null)
   /** The portaled button itself (for the outside-click guard). */
   const buttonRef = useRef<HTMLButtonElement>(null)
   /** The surface visibility observer (created lazily on open). */
   const observerRef = useRef<IntersectionObserver | null>(null)
 
-  const show = (insert: string, left: number, top: number): void => {
-    const next: SelectionPopup = {
+  const show = (insert: T, left: number, top: number): void => {
+    const next: SelectionPopup<T> = {
       insert,
       left: Math.min(Math.max(left, 80), window.innerWidth - 80),
       top,

package/src/workspace-title.ts

@@ -5,8 +5,8 @@
  * the workspace REGISTRY, so this string is the only label the user ever sees
  * there. It used to be built from the remote PATH's last segment, which threw
  * away the one name the user had chosen: a workspace created as
- * "P710_userdebug" showed up as `… > P710`, and two bindings that differ only
- * deeper down (`…/userdebug/P710` and `…/userdebug/P710_fac`) became
+ * "build_variant_a" showed up as `… > build`, and two bindings that differ only
+ * deeper down (`…/userdebug/build_a` and `…/userdebug/build_a_fac`) became
  * indistinguishable.
  *
  * The alias stays in front: it is what tells two hosts apart, and the part after