@adhdev/daemon-core 0.9.82-rc.209 → 0.9.82-rc.210

package/dist/providers/spec/types.d.ts

@@ -1,86 +1,9 @@
 export type Size = number | string;
-export interface Section {
-    id: string;
-    from_top?: Size;
-    from_bottom?: Size;
-    until?: {
-        section: string;
-    };
-    /** Scan the screen for the first (or last, if anchor_last=true) line
-     *  matching this regex and use that line as the section start.
-     *  Combine with `lines`, `until_regex`, or leave alone (defaults to
-     *  end-of-screen) to control how far it extends. */
-    anchor_regex?: string;
-    anchor_flags?: string;
-    /** If true, use the LAST matching line instead of the first. Useful
-     *  when the anchor pattern appears multiple times (e.g. separator
-     *  lines) and you want the one closest to the bottom. */
-    anchor_last?: boolean;
-    /** Optional neighbouring-line guards that must also match for the
-     *  anchor to be accepted. Reduces false positives when the anchor
-     *  pattern could appear elsewhere on screen (e.g. `>` in body text).
-     *  `prev` checks the line immediately above; `next` the line below. */
-    anchor_context?: {
-        prev?: string;
-        prev_flags?: string;
-        next?: string;
-        next_flags?: string;
-    };
-    /** When used with anchor_regex: take at most this many lines from the
-     *  anchor point. */
-    lines?: number;
-    /** When used with anchor_regex: extend until the first line matching
-     *  this regex (exclusive). Takes precedence over `lines`. */
-    until_regex?: string;
-    until_regex_flags?: string;
-}
-export interface SectionRegex {
-    section?: string;
-    regex: string;
-    flags?: string;
-    /**
-     * Optional cursor-position guards. When present, the state is only
-     * considered matched if the terminal cursor row/column satisfies the
-     * bounds (0-based, inclusive). Missing or undefined means "no constraint".
-     *
-     * Use case: distinguish modal zone from body zone for TUIs that use
-     * cursor position rather than distinct text to locate the active prompt
-     * (e.g. Antigravity cursor lands in modal_zone rows 8-31 when approval
-     * is visible, never in body rows 0-7). Without this guard, body text
-     * containing "Do you want to proceed?" could false-positive a modal match.
-     */
-    cursor_row_min?: number;
-    cursor_row_max?: number;
-    cursor_col_min?: number;
-    cursor_col_max?: number;
-}
 export interface SectionPattern {
     section?: string;
     pattern: string;
     flags?: string;
 }
-export interface ModalButtonsRule {
-    section?: string;
-    /** Single pattern (original). Use `patterns` for multi-format fallback. */
-    pattern?: string;
-    flags?: string;
-    /** Ordered list of pattern alternatives. Evaluated in order; first that
-     *  yields >= min_count buttons wins. Mutually exclusive with `pattern`. */
-    patterns?: Array<{
-        pattern: string;
-        flags?: string;
-    }>;
-    key_for_index: string;
-    min_count?: number;
-    continuation_lines?: boolean;
-}
-export interface SpecState {
-    id: string;
-    label: string;
-    when: SectionRegex;
-    extract_title?: SectionRegex;
-    modal_buttons?: ModalButtonsRule;
-}
 export type ControlAction = {
     type: 'send_keys';
     keys: string;
@@ -117,23 +40,10 @@ export interface DelegateTrigger {
  * Native history config — three modes, picked by which fields are present:
  *
  *   1. `reader`: built-in reader id (claude-cli / codex-cli / antigravity-cli / hermes-cli).
- *      Backwards-compatible path used by the four shipped providers.
  *
- *   2. `source`: declarative source descriptor. The daemon's spec native-history
- *      executor reads the on-disk store (jsonl file / sqlite db) directly using
- *      paths + jsonpath maps from this block. New providers should prefer this
- *      mode — no daemon changes required.
+ *   2. `source`: declarative source descriptor.
  *
- *   3. `override_path`: relative path to a provider-supplied reader module
- *      (e.g. "./native_history/index.js"). Escape hatch for formats too
- *      exotic for the declarative executor. The module must default-export
- *      a function with the signature
- *        (input: NativeHistoryInput) => NativeHistoryResult | null
- *      and runs under provider-script-root sandboxing (same gate as
- *      scripts.js).
- *
- * The three modes are mutually exclusive — set exactly one. Loader rejects
- * specs that set zero or more than one.
+ *   3. `override_path`: relative path to a provider-supplied reader module.
  */
 export interface NativeHistoryConfig {
     reader?: 'claude-cli' | 'codex-cli' | 'antigravity-cli' | 'hermes-cli';
@@ -141,20 +51,6 @@ export interface NativeHistoryConfig {
     override_path?: string;
 }
 export type NativeHistorySource = NativeHistoryJsonlSource | NativeHistorySqliteSource;
-/**
- * JSONL source — each line in the resolved file is one record.
- *
- * `path` accepts ~/, environment variables (${HOME}), and a small set of
- * runtime variables:
- *   {cwd}              — the session's working directory, raw
- *   {cwd_dashed}       — cwd with `/` replaced by `-` (claude's per-project key)
- *   {session_id}       — providerSessionId, when known
- *   {yyyy} {mm} {dd}   — UTC date components
- *
- * When `path` resolves to a directory, the daemon picks the newest file
- * matching `file_pattern` whose mtime is inside `recent_window_ms` (default
- * 5min) — prevents the wrong session's transcript from leaking through.
- */
 export interface NativeHistoryJsonlSource {
     kind: 'jsonl';
     path: string;
@@ -167,12 +63,6 @@ export interface NativeHistoryJsonlSource {
     };
     message_map: NativeHistoryMessageMap;
 }
-/**
- * SQLite source — provider runs two queries on a read-only handle: one to
- * pick the session row (`session_query` — first row's first column is the
- * sessionId), then one to fetch messages (`message_query` — `?` is bound
- * to the sessionId). Output rows are projected through `message_map`.
- */
 export interface NativeHistorySqliteSource {
     kind: 'sqlite';
     path: string;
@@ -180,50 +70,200 @@ export interface NativeHistorySqliteSource {
     message_query: string;
     message_map: NativeHistoryMessageMap;
 }
-/**
- * Maps a source record (jsonl line object / sqlite row) to a daemon
- * native-history message. Values are jsonpath-lite strings: `$.foo.bar`
- * or `$.items[0].text`. Literal strings (no leading `$`) pass through.
- *
- * `content_strip` lets a spec drop wrapper tags that the agent injects
- * into its own transcript (claude's `<local-command-caveat>`, agy's
- * `<USER_REQUEST>` / `<ADDITIONAL_METADATA>` / `<USER_SETTINGS_CHANGE>`).
- * Each entry is a tag name; the executor removes `<tag>…</tag>` segments
- * non-greedily before the value is handed to the dashboard. After all
- * strips, leading/trailing whitespace is trimmed. If the result is empty
- * the message is dropped.
- */
 export interface NativeHistoryMessageMap {
     role: string;
     content: string;
-    /**
-     * Tags whose entire `<tag>…</tag>` segment is removed from content.
-     * Use for noise wrappers the agent ships alongside the real message
-     * (claude's `<local-command-caveat>`, agy's `<ADDITIONAL_METADATA>`).
-     */
     content_strip?: string[];
-    /**
-     * Tags whose open/close markers are removed but inner content is kept.
-     * Use when the *real* user/assistant message is wrapped in a tag
-     * the agent uses for routing (agy wraps user input in `<USER_REQUEST>`).
-     */
     content_unwrap?: string[];
     timestamp_ms?: string;
     kind?: string;
 }
+export interface Section {
+    id: string;
+    from_top?: Size;
+    from_bottom?: Size;
+    until?: {
+        section: string;
+    };
+    anchor_regex?: string;
+    anchor_flags?: string;
+    anchor_last?: boolean;
+    anchor_context?: {
+        prev?: string;
+        prev_flags?: string;
+        next?: string;
+        next_flags?: string;
+    };
+    lines?: number;
+    until_regex?: string;
+    until_regex_flags?: string;
+}
+/** v1 condition — flat SectionRegex object. Also used in control_bar.wait_for. */
+export interface SectionRegex {
+    section?: string;
+    regex?: string;
+    flags?: string;
+    cursor_row_min?: number;
+    cursor_row_max?: number;
+    cursor_col_min?: number;
+    cursor_col_max?: number;
+    /** v1 delta detection (cursor_above_lines + changed). */
+    cursor_above_lines?: number;
+    /** Required when cursor_above_lines is set. */
+    changed?: true;
+}
+export interface ModalButtonsRule {
+    section?: string;
+    pattern?: string;
+    flags?: string;
+    patterns?: Array<{
+        pattern: string;
+        flags?: string;
+    }>;
+    key_for_index: string;
+    min_count?: number;
+    continuation_lines?: boolean;
+}
+/**
+ * v3 section definition — keyed by id in `sections{}` object.
+ * `until` is EITHER a section-id string (no `^` prefix) OR a regex string (starts with `^`).
+ */
+export interface SectionDef {
+    from_top?: Size;
+    from_bottom?: Size;
+    until?: string;
+    anchor?: string;
+    anchor_flags?: string;
+    anchor_last?: boolean;
+    anchor_context?: {
+        prev?: string;
+        next?: string;
+        prev_flags?: string;
+        next_flags?: string;
+    };
+    lines?: number;
+    until_regex?: string;
+    until_regex_flags?: string;
+}
+/** v3 regex condition against a section (or full screen). */
+export interface RegexCondition {
+    section?: string;
+    matches: string;
+    flags?: string;
+    cursor_row_min?: number;
+    cursor_row_max?: number;
+    cursor_col_min?: number;
+    cursor_col_max?: number;
+}
+/** v3 delta condition: N lines above cursor changed vs prevLines. */
+export interface ChangedCondition {
+    cursor_above: number;
+    changed: true;
+}
+/** v3 AND composite. */
+export interface AllCondition {
+    all: Condition[];
+}
+/** v3 OR composite. */
+export interface AnyCondition {
+    any: Condition[];
+}
+export type Condition = RegexCondition | ChangedCondition | AllCondition | AnyCondition;
+/** v3 extract_title config. */
+export interface ExtractTitle {
+    section?: string;
+    regex?: string;
+    flags?: string;
+    first_line?: true;
+}
+/** v3 extract_buttons config. */
+export interface ExtractButtons {
+    section?: string;
+    pattern: string;
+    flags?: string;
+    key_for_index: string;
+    min_count?: number;
+    continuation_lines?: boolean;
+}
+/** v3 state. */
+export interface SpecStateV3 {
+    id: string;
+    label: string;
+    when: AllCondition | AnyCondition;
+    extract?: {
+        title?: ExtractTitle;
+        buttons?: ExtractButtons;
+    };
+}
 export interface CliSpec {
-    $schema: 'adhdev:cli/spec@1';
+    $schema: 'adhdev:cli/spec@3';
     id: string;
     name: string;
     binary: string;
     spawn_args?: string[];
     env?: Record<string, string>;
+    cli_version_range?: string;
+    send_message: {
+        submit_key: string;
+        delay_ms_before_submit?: number;
+        delay_ms_per_char?: number;
+    };
+    sections: Record<string, SectionDef>;
+    states: SpecStateV3[];
+    default_state: string;
+    control_bar?: Control[];
+    notifications?: NotificationRule[];
+    delegate?: DelegateTrigger[];
+    native_history?: NativeHistoryConfig;
+    requiresFinalAssistantBeforeIdle?: boolean;
     /**
-     * Optional version constraint this spec is authored for, e.g. ">=2.1.0".
-     * Pure metadata used by tooling/UI — the actual version-to-spec match
-     * happens in provider-loader via the manifest's compatibility array.
-     * Spec authors include this so a misrouted spec is easy to spot.
+     * Per-spec timing knobs. Replaces v1 `debounce`.
+     * Also accessible via the legacy `debounce` alias in the driver for
+     * backward compat with older callers.
      */
+    timing?: {
+        busy_hold_ms?: number;
+        idle_hold_ms?: number;
+        startup_grace_ms?: number;
+        completion_marker?: {
+            section?: string;
+            matches: string;
+            flags?: string;
+            hold_ms: number;
+            force_after_ms?: number;
+        };
+    };
+    /**
+     * Legacy alias: v1 callers (tests, tooling) may read `debounce`.
+     * The loader populates both `timing` and `debounce` pointing at the
+     * same object so callers that use either field see consistent values.
+     * @deprecated Use `timing` instead.
+     */
+    debounce?: {
+        busy_hold_ms?: number;
+        idle_hold_ms?: number;
+        startup_grace_ms?: number;
+        /**
+         * Legacy field name for completion_marker. Populated by the loader
+         * from `timing.completion_marker` for backward compat.
+         */
+        completion_idle_after?: {
+            section?: string;
+            regex: string;
+            flags?: string;
+            hold_ms: number;
+            force_after_ms?: number;
+        };
+    };
+}
+/** Raw v1 spec shape as loaded from JSON. Used only in migrateV1toV3. */
+export interface RawCliSpecV1 {
+    $schema: 'adhdev:cli/spec@1';
+    id: string;
+    name: string;
+    binary: string;
+    spawn_args?: string[];
+    env?: Record<string, string>;
     cli_version_range?: string;
     send_message: {
         submit_key: string;
@@ -233,57 +273,30 @@ export interface CliSpec {
     layout: {
         sections: Section[];
     };
-    states: SpecState[];
+    states: Array<{
+        id: string;
+        label: string;
+        when: SectionRegex;
+        extract_title?: SectionRegex;
+        modal_buttons?: ModalButtonsRule;
+    }>;
     default_state: string;
     control_bar?: Control[];
     notifications?: NotificationRule[];
     delegate?: DelegateTrigger[];
     native_history?: NativeHistoryConfig;
-    /**
-     * When true, the daemon's completed-finalization gate will not emit
-     * `generating_completed` until native history confirms the last message
-     * role is `assistant`. Prevents PTY quiet-period from triggering idle
-     * before the CLI has finished writing its response to disk.
-     */
     requiresFinalAssistantBeforeIdle?: boolean;
-    /**
-     * Per-spec debounce knobs. Defaults are conservative (busy_hold_ms
-     * 6000) and live in SpecDriver. Spec authors override here when a
-     * particular CLI flickers slower/faster than the default.
-     */
     debounce?: {
-        /** Min time to stay in busy after the evaluator last reported it.
-         *  Absorbs per-frame flicker in TUIs that stream output through
-         *  the same region as the spinner. */
         busy_hold_ms?: number;
-        /** Min time the idle state must remain matched before it is
-         *  committed. Filters transient idle flickers that appear during
-         *  approval dismissals, layout reflows, or brief spinner gaps.
-         *  Any non-idle reading within the window cancels the transition.
-         *  When omitted the idle transition is immediate (legacy behaviour). */
         idle_hold_ms?: number;
-        /** Min time after start() before a send_message is allowed to
-         *  reach the PTY. Banner paints + auth flows + skill listings
-         *  can keep the agent unable to accept input for several seconds
-         *  even though the idle regex matches a transient empty prompt.
-         *  Send_messages issued before this window are queued and drained
-         *  once the window passes and an idle state has actually been
-         *  observed. */
         startup_grace_ms?: number;
-        /** Treat a provider-specific completion marker as idle after it has
-         *  remained visible for hold_ms. This handles TUIs that leave their
-         *  last spinner glyph next to a completed timer, causing the normal
-         *  busy regex to keep matching after the turn is done. */
         completion_idle_after?: {
             section?: string;
             regex: string;
             flags?: string;
             hold_ms: number;
-            /** If the target-state check still fails this many ms after hold_ms
-             *  expired, force an idle transition anyway. Guards against TUIs that
-             *  show transient UI (pickers, option lists) after the completion
-             *  marker appears, keeping the target-state regex from matching. */
             force_after_ms?: number;
         };
     };
 }
+export type SpecState = SpecStateV3;

package/dist/repo-mesh-types.d.ts

@@ -361,6 +361,7 @@ export interface RepoMeshNodeStatus {
     worktreeBootstrap?: LocalMeshNodeEntry['worktreeBootstrap'];
     launchBlockedReason?: string;
     launchBlockedMessage?: string;
+    recoveryHint?: string;
     lastSeenAt?: string;
     updatedAt?: string;
     connection?: RepoMeshPeerConnectionStatus;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adhdev/daemon-core",
-  "version": "0.9.82-rc.209",
+  "version": "0.9.82-rc.210",
   "description": "ADHDev daemon core — CDP, IDE detection, providers, command execution",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/boot/daemon-lifecycle.ts

@@ -353,6 +353,9 @@ export async function initDaemonComponents(config: DaemonInitConfig): Promise<Da
     // 11. Setup Mesh Event Forwarding
     setupMeshEventForwarding(components);
 
+    // 12. Resume any refine jobs that were interrupted by a previous daemon restart.
+    setImmediate(() => void router.resumePendingRefineJobsOnStartup());
+
     return components;
 }
 

package/src/cli-adapters/terminal-backends/ghostty-vt-backend.ts

@@ -83,45 +83,58 @@ function loadGhosttyVtBinding(): GhosttyVtBinding {
 export class GhosttyVtTerminalBackend implements TerminalViewportBackend {
     readonly kind = 'ghostty-vt' as const;
     private terminal: GhosttyVtTerminal;
+    private rows: number;
+    private disposed = false;
 
     constructor(options: TerminalViewportBackendOptions) {
         const binding = loadGhosttyVtBinding();
+        this.rows = Math.max(1, options.rows | 0);
         this.terminal = binding.createTerminal({
             cols: Math.max(1, options.cols | 0),
-            rows: Math.max(1, options.rows | 0),
+            rows: this.rows,
             scrollback: Math.max(0, options.scrollback | 0),
         });
     }
 
     resize(rows: number, cols: number): void {
-        this.terminal.resize(Math.max(1, cols | 0), Math.max(1, rows | 0));
+        this.rows = Math.max(1, rows | 0);
+        this.terminal.resize(Math.max(1, cols | 0), this.rows);
     }
 
     write(data: string): void {
-        if (!data) return;
+        if (!data || this.disposed) return;
         this.terminal.write(data);
     }
 
     getText(): string {
+        if (this.disposed) return '';
         // ghostty's `trim:true` collapses CUF-advanced cells — many TUIs including
         // Claude Code render spaces via cursor-forward rather than literal spaces,
         // which would break downstream regex matching. Keep per-row padding and
         // trim trailing whitespace ourselves.
+        //
+        // formatPlainText uses a .screen pin which includes scrollback history.
+        // Slice to the last `rows` lines to get only the visible viewport.
         const raw = this.terminal.formatPlainText({ trim: false }) || '';
         if (!raw) return '';
         const lines = raw.split('\n').map((row) => row.replace(/\s+$/, ''));
+        // Take only the viewport (last `rows` lines) to exclude scrollback history.
+        const viewport = lines.length > this.rows ? lines.slice(-this.rows) : lines;
         let first = 0;
-        let last = lines.length;
-        while (first < last && !lines[first]) first += 1;
-        while (last > first && !lines[last - 1]) last -= 1;
-        return lines.slice(first, last).join('\n');
+        let last = viewport.length;
+        while (first < last && !viewport[first]) first += 1;
+        while (last > first && !viewport[last - 1]) last -= 1;
+        return viewport.slice(first, last).join('\n');
     }
 
     getCursorPosition(): { col: number; row: number } {
+        if (this.disposed) return { col: 0, row: 0 };
         return this.terminal.getCursorPosition();
     }
 
     dispose(): void {
+        if (this.disposed) return;
+        this.disposed = true;
         this.terminal.dispose();
     }
 }