@jmoyers/harness 0.1.8 → 0.1.10

package/src/mux/input-shortcuts.ts

@@ -9,6 +9,7 @@ type MuxGlobalShortcutAction =
   | 'mux.conversation.critique.open-or-create'
   | 'mux.conversation.next'
   | 'mux.conversation.previous'
+  | 'mux.conversation.titles.refresh-all'
   | 'mux.conversation.interrupt'
   | 'mux.conversation.archive'
   | 'mux.conversation.takeover'
@@ -47,6 +48,7 @@ const ACTION_ORDER: readonly MuxGlobalShortcutAction[] = [
   'mux.conversation.critique.open-or-create',
   'mux.conversation.next',
   'mux.conversation.previous',
+  'mux.conversation.titles.refresh-all',
   'mux.conversation.interrupt',
   'mux.conversation.archive',
   'mux.conversation.takeover',
@@ -68,6 +70,7 @@ const DEFAULT_MUX_SHORTCUT_BINDINGS_RAW: Readonly<
   'mux.conversation.critique.open-or-create': ['ctrl+g'],
   'mux.conversation.next': ['ctrl+j'],
   'mux.conversation.previous': ['ctrl+k'],
+  'mux.conversation.titles.refresh-all': ['ctrl+r'],
   'mux.conversation.interrupt': [],
   'mux.conversation.archive': [],
   'mux.conversation.takeover': ['ctrl+l'],
@@ -580,6 +583,9 @@ function withDefaultBindings(
     'mux.conversation.previous':
       overrides?.['mux.conversation.previous'] ??
       DEFAULT_MUX_SHORTCUT_BINDINGS_RAW['mux.conversation.previous'],
+    'mux.conversation.titles.refresh-all':
+      overrides?.['mux.conversation.titles.refresh-all'] ??
+      DEFAULT_MUX_SHORTCUT_BINDINGS_RAW['mux.conversation.titles.refresh-all'],
     'mux.conversation.interrupt':
       overrides?.['mux.conversation.interrupt'] ??
       DEFAULT_MUX_SHORTCUT_BINDINGS_RAW['mux.conversation.interrupt'],
@@ -625,6 +631,9 @@ export function resolveMuxShortcutBindings(
       ),
       'mux.conversation.next': parseBindingsForAction(rawByAction['mux.conversation.next']),
       'mux.conversation.previous': parseBindingsForAction(rawByAction['mux.conversation.previous']),
+      'mux.conversation.titles.refresh-all': parseBindingsForAction(
+        rawByAction['mux.conversation.titles.refresh-all'],
+      ),
       'mux.conversation.interrupt': parseBindingsForAction(
         rawByAction['mux.conversation.interrupt'],
       ),

package/src/mux/live-mux/critique-review.ts

@@ -8,6 +8,10 @@ interface CritiqueReviewAgentAvailability {
 }
 
 type CritiqueReviewCommandInput =
+  | {
+      readonly mode: 'unstaged';
+      readonly agent: CritiqueReviewAgent | null;
+    }
   | {
       readonly mode: 'staged';
       readonly agent: CritiqueReviewAgent | null;
@@ -54,7 +58,7 @@ export function buildCritiqueReviewCommand(input: CritiqueReviewCommandInput): s
   const tokens = ['critique', 'review'];
   if (input.mode === 'staged') {
     tokens.push('--staged');
-  } else {
+  } else if (input.mode === 'base-branch') {
     const normalizedBaseBranch = normalizeBranchName(input.baseBranch) ?? 'main';
     tokens.push(normalizedBaseBranch, 'HEAD');
   }

package/src/mux/live-mux/git-parsing.ts

@@ -98,6 +98,30 @@ export function shouldShowGitHubPrActions(input: {
   return normalizedTrackedBranch !== 'main';
 }
 
+function normalizeTrackedBranchForActions(value: string | null): string | null {
+  const trimmed = value?.trim() ?? '';
+  if (
+    trimmed.length === 0 ||
+    trimmed === '(detached)' ||
+    trimmed === '(loading)' ||
+    trimmed === 'HEAD'
+  ) {
+    return null;
+  }
+  return trimmed;
+}
+
+export function resolveGitHubTrackedBranchForActions(input: {
+  projectTrackedBranch: string | null;
+  currentBranch: string | null;
+}): string | null {
+  const trackedBranch = normalizeTrackedBranchForActions(input.projectTrackedBranch);
+  if (trackedBranch !== null) {
+    return trackedBranch;
+  }
+  return normalizeTrackedBranchForActions(input.currentBranch);
+}
+
 export function parseCommitCount(output: string): number | null {
   const trimmed = output.trim();
   if (trimmed.length === 0 || !/^\d+$/u.test(trimmed)) {

package/src/mux/live-mux/global-shortcut-handlers.ts

@@ -14,6 +14,7 @@ interface HandleGlobalShortcutOptions {
   conversationsHas: (sessionId: string) => boolean;
   queueControlPlaneOp: (task: () => Promise<void>, label: string) => void;
   archiveConversation: (sessionId: string) => Promise<void>;
+  refreshAllConversationTitles: () => Promise<void>;
   interruptConversation: (sessionId: string) => Promise<void>;
   takeoverConversation: (sessionId: string) => Promise<void>;
   openAddDirectoryPrompt: () => void;
@@ -37,6 +38,7 @@ export function handleGlobalShortcut(options: HandleGlobalShortcutOptions): bool
     conversationsHas,
     queueControlPlaneOp,
     archiveConversation,
+    refreshAllConversationTitles,
     interruptConversation,
     takeoverConversation,
     openAddDirectoryPrompt,
@@ -104,6 +106,12 @@ export function handleGlobalShortcut(options: HandleGlobalShortcutOptions): bool
     }
     return true;
   }
+  if (shortcut === 'mux.conversation.titles.refresh-all') {
+    queueControlPlaneOp(async () => {
+      await refreshAllConversationTitles();
+    }, 'shortcut-refresh-conversation-titles');
+    return true;
+  }
   if (shortcut === 'mux.conversation.interrupt') {
     const targetConversationId = resolveConversationForAction();
     if (targetConversationId !== null && conversationsHas(targetConversationId)) {

package/src/mux/render-frame.ts

@@ -51,7 +51,7 @@ export function buildRenderRows(
   const statusText =
     statusRowDetailText === undefined || statusRowDetailText.length === 0
       ? defaultStatus
-      : `${defaultStatus}  ${statusRowDetailText}`;
+      : `${statusRowDetailText}  ${defaultStatus}`;
   const status = padOrTrimDisplay(statusText, layout.cols);
   rows.push(status);
   return rows;

package/src/pty/pty_host.ts

@@ -68,6 +68,8 @@ class PtySession extends EventEmitter {
   private nextProbeId = 1;
   private outputWindow = Buffer.alloc(0);
   private static readonly MAX_OUTPUT_WINDOW_BYTES = 8192;
+  private static readonly MAX_PENDING_ROUNDTRIP_PROBES = 64;
+  private static readonly ROUNDTRIP_PROBE_MAX_AGE_NS = 5_000_000_000n;
 
   constructor(child: ChildProcessWithoutNullStreams) {
     super();
@@ -101,6 +103,12 @@ class PtySession extends EventEmitter {
         matchPayloads: PtySession.buildMatchPayloads(payload),
         startedAtNs: perfNowNs(),
       });
+      if (this.pendingRoundtripProbes.length > PtySession.MAX_PENDING_ROUNDTRIP_PROBES) {
+        this.pendingRoundtripProbes.splice(
+          0,
+          this.pendingRoundtripProbes.length - PtySession.MAX_PENDING_ROUNDTRIP_PROBES,
+        );
+      }
       this.nextProbeId += 1;
     }
 
@@ -148,12 +156,22 @@ class PtySession extends EventEmitter {
       );
     }
 
+    const maxMatchPayloadLength = this.compactPendingRoundtripProbes(perfNowNs());
+    if (this.pendingRoundtripProbes.length === 0) {
+      return;
+    }
+    const searchWindowLength = Math.max(
+      1,
+      Math.min(this.outputWindow.length, chunk.length + Math.max(1, maxMatchPayloadLength) - 1),
+    );
+    const searchWindow = this.outputWindow.subarray(this.outputWindow.length - searchWindowLength);
+
     let idx = 0;
     while (idx < this.pendingRoundtripProbes.length) {
       const probe = this.pendingRoundtripProbes[idx];
       if (
         probe !== undefined &&
-        probe.matchPayloads.some((matchPayload) => this.outputWindow.includes(matchPayload))
+        probe.matchPayloads.some((matchPayload) => searchWindow.includes(matchPayload))
       ) {
         recordPerfDuration('pty.keystroke.roundtrip', probe.startedAtNs, {
           'probe-id': probe.probeId,
@@ -166,6 +184,33 @@ class PtySession extends EventEmitter {
     }
   }
 
+  private compactPendingRoundtripProbes(nowNs: bigint): number {
+    if (this.pendingRoundtripProbes.length > PtySession.MAX_PENDING_ROUNDTRIP_PROBES) {
+      this.pendingRoundtripProbes.splice(
+        0,
+        this.pendingRoundtripProbes.length - PtySession.MAX_PENDING_ROUNDTRIP_PROBES,
+      );
+    }
+    let maxMatchPayloadLength = 1;
+    let idx = 0;
+    while (idx < this.pendingRoundtripProbes.length) {
+      const probe = this.pendingRoundtripProbes[idx];
+      if (probe === undefined) {
+        idx += 1;
+        continue;
+      }
+      if (nowNs - probe.startedAtNs > PtySession.ROUNDTRIP_PROBE_MAX_AGE_NS) {
+        this.pendingRoundtripProbes.splice(idx, 1);
+        continue;
+      }
+      for (const matchPayload of probe.matchPayloads) {
+        maxMatchPayloadLength = Math.max(maxMatchPayloadLength, matchPayload.length);
+      }
+      idx += 1;
+    }
+    return maxMatchPayloadLength;
+  }
+
   private static buildMatchPayloads(payload: Buffer): Buffer[] {
     if (!payload.includes(0x0a)) {
       return [payload];

package/src/services/control-plane.ts

@@ -245,6 +245,28 @@ export class ControlPlaneService {
     return parseConversationRecord(result['conversation']);
   }
 
+  async refreshConversationTitle(conversationId: string): Promise<{
+    status: 'updated' | 'unchanged' | 'skipped';
+    reason: string | null;
+  }> {
+    const result = await this.client.sendCommand({
+      type: 'conversation.title.refresh',
+      conversationId,
+    });
+    const status = result['status'];
+    if (status !== 'updated' && status !== 'unchanged' && status !== 'skipped') {
+      throw new Error('control-plane conversation.title.refresh returned malformed status');
+    }
+    const reason = result['reason'];
+    if (reason !== null && reason !== undefined && typeof reason !== 'string') {
+      throw new Error('control-plane conversation.title.refresh returned malformed reason');
+    }
+    return {
+      status,
+      reason: reason ?? null,
+    };
+  }
+
   async archiveConversation(conversationId: string): Promise<void> {
     await this.client.sendCommand({
       type: 'conversation.archive',

package/src/services/runtime-control-actions.ts

@@ -21,6 +21,13 @@ interface RuntimeGatewayRenderTraceResult {
   readonly message: string;
 }
 
+interface RuntimeConversationTitleRefreshResult {
+  readonly status: 'updated' | 'unchanged' | 'skipped';
+  readonly reason: string | null;
+}
+
+const THREAD_TITLE_AGENT_TYPES = new Set(['codex', 'claude', 'cursor']);
+
 interface RuntimeControlActionsOptions<TConversation extends RuntimeConversationControlState> {
   readonly conversationById: (sessionId: string) => TConversation | undefined;
   readonly interruptSession: (sessionId: string) => Promise<RuntimeInterruptResult>;
@@ -43,6 +50,11 @@ interface RuntimeControlActionsOptions<TConversation extends RuntimeConversation
   readonly sessionName: string | null;
   readonly setTaskPaneNotice: (message: string) => void;
   readonly setDebugFooterNotice: (message: string) => void;
+  readonly listConversationIdsForTitleRefresh?: () => readonly string[];
+  readonly conversationAgentTypeForTitleRefresh?: (sessionId: string) => string | null;
+  readonly refreshConversationTitle?: (
+    sessionId: string,
+  ) => Promise<RuntimeConversationTitleRefreshResult>;
 }
 
 export class RuntimeControlActions<TConversation extends RuntimeConversationControlState> {
@@ -109,6 +121,63 @@ export class RuntimeControlActions<TConversation extends RuntimeConversationCont
     }
   }
 
+  async refreshAllConversationTitles(): Promise<void> {
+    const listConversationIds = this.options.listConversationIdsForTitleRefresh;
+    const resolveAgentType = this.options.conversationAgentTypeForTitleRefresh;
+    const refreshConversationTitle = this.options.refreshConversationTitle;
+    if (
+      listConversationIds === undefined ||
+      resolveAgentType === undefined ||
+      refreshConversationTitle === undefined
+    ) {
+      this.setNotices(this.scopeMessage('thread-title', 'refresh unavailable'));
+      this.options.markDirty();
+      return;
+    }
+    const allConversationIds = listConversationIds();
+    const eligibleConversationIds = allConversationIds.filter((sessionId) => {
+      const agentType = resolveAgentType(sessionId)?.trim().toLowerCase();
+      return agentType !== undefined && THREAD_TITLE_AGENT_TYPES.has(agentType);
+    });
+    if (eligibleConversationIds.length === 0) {
+      this.setNotices(this.scopeMessage('thread-title', 'no agent threads to refresh'));
+      this.options.markDirty();
+      return;
+    }
+    const total = eligibleConversationIds.length;
+    let updated = 0;
+    let unchanged = 0;
+    let skipped = 0;
+    this.setNotices(this.scopeMessage('thread-title', `refreshing names 0/${String(total)}`));
+    this.options.markDirty();
+    for (let index = 0; index < eligibleConversationIds.length; index += 1) {
+      const sessionId = eligibleConversationIds[index]!;
+      try {
+        const result = await refreshConversationTitle(sessionId);
+        if (result.status === 'updated') {
+          updated += 1;
+        } else if (result.status === 'unchanged') {
+          unchanged += 1;
+        } else {
+          skipped += 1;
+        }
+      } catch {
+        skipped += 1;
+      }
+      this.setNotices(
+        this.scopeMessage('thread-title', `refreshing names ${String(index + 1)}/${String(total)}`),
+      );
+      this.options.markDirty();
+    }
+    this.setNotices(
+      this.scopeMessage(
+        'thread-title',
+        `refreshed ${String(updated)} updated ${String(unchanged)} unchanged ${String(skipped)} skipped`,
+      ),
+    );
+    this.options.markDirty();
+  }
+
   private scopeMessage(prefix: string, message: string): string {
     if (this.options.sessionName === null) {
       return `[${prefix}] ${message}`;

package/src/services/runtime-navigation-input.ts

@@ -17,6 +17,7 @@ interface RuntimeNavigationWorkspaceActions {
   toggleGatewayStatusTimeline(): Promise<void>;
   toggleGatewayRenderTrace(conversationId: string | null): Promise<void>;
   archiveConversation(sessionId: string): Promise<void>;
+  refreshAllConversationTitles(): Promise<void>;
   interruptConversation(sessionId: string): Promise<void>;
   takeoverConversation(sessionId: string): Promise<void>;
   closeDirectory(directoryId: string): Promise<void>;
@@ -157,6 +158,9 @@ export class RuntimeNavigationInput {
       archiveConversation: async (sessionId) => {
         await options.workspaceActions.archiveConversation(sessionId);
       },
+      refreshAllConversationTitles: async () => {
+        await options.workspaceActions.refreshAllConversationTitles();
+      },
       interruptConversation: async (sessionId) => {
         await options.workspaceActions.interruptConversation(sessionId);
       },

package/src/services/runtime-rail-input.ts

@@ -15,6 +15,7 @@ interface RuntimeRailWorkspaceActions {
   toggleGatewayStatusTimeline(): Promise<void>;
   toggleGatewayRenderTrace(conversationId: string | null): Promise<void>;
   archiveConversation(sessionId: string): Promise<void>;
+  refreshAllConversationTitles(): Promise<void>;
   interruptConversation(sessionId: string): Promise<void>;
   takeoverConversation(sessionId: string): Promise<void>;
   closeDirectory(directoryId: string): Promise<void>;
@@ -142,6 +143,9 @@ export class RuntimeRailInput {
         archiveConversation: async (sessionId) => {
           await options.runtimeWorkspaceActions.archiveConversation(sessionId);
         },
+        refreshAllConversationTitles: async () => {
+          await options.runtimeWorkspaceActions.refreshAllConversationTitles();
+        },
         interruptConversation: async (sessionId) => {
           await options.runtimeWorkspaceActions.interruptConversation(sessionId);
         },

package/src/services/runtime-workspace-actions.ts

@@ -30,6 +30,7 @@ interface RuntimeWorkspaceControlActions {
   toggleGatewayProfiler(): Promise<void>;
   toggleGatewayStatusTimeline(): Promise<void>;
   toggleGatewayRenderTrace(conversationId: string | null): Promise<void>;
+  refreshAllConversationTitles(): Promise<void>;
 }
 
 interface RuntimeWorkspaceTaskPaneActions {
@@ -135,6 +136,10 @@ export class RuntimeWorkspaceActions {
     await this.options.controlActions.toggleGatewayRenderTrace(conversationId);
   }
 
+  async refreshAllConversationTitles(): Promise<void> {
+    await this.options.controlActions.refreshAllConversationTitles();
+  }
+
   runTaskPaneAction(action: TaskPaneAction): void {
     this.options.taskPaneActions.runTaskPaneAction(action);
   }

package/src/ui/global-shortcut-input.ts

@@ -23,6 +23,7 @@ interface GlobalShortcutInputOptions {
   readonly conversationsHas: (sessionId: string) => boolean;
   readonly queueControlPlaneOp: (task: () => Promise<void>, label: string) => void;
   readonly archiveConversation: (sessionId: string) => Promise<void>;
+  readonly refreshAllConversationTitles: () => Promise<void>;
   readonly interruptConversation: (sessionId: string) => Promise<void>;
   readonly takeoverConversation: (sessionId: string) => Promise<void>;
   readonly openAddDirectoryPrompt: () => void;
@@ -70,6 +71,7 @@ export class GlobalShortcutInput {
       conversationsHas: this.options.conversationsHas,
       queueControlPlaneOp: this.options.queueControlPlaneOp,
       archiveConversation: this.options.archiveConversation,
+      refreshAllConversationTitles: this.options.refreshAllConversationTitles,
       interruptConversation: this.options.interruptConversation,
       takeoverConversation: this.options.takeoverConversation,
       openAddDirectoryPrompt: this.options.openAddDirectoryPrompt,