@gaia-ai/conductor 0.11.0 → 0.11.2

package/dist/src/core/conductor.d.ts

@@ -71,23 +71,25 @@ export declare class Conductor {
      */
     private finalizeDoneRuns;
     /**
-     * Reconcile finished tickets against their herdr worktrees and tear down any
+     * Reconcile closed tickets against their herdr worktrees and tear down any
      * whose workspace still lingers (GAIA-89). Driven by the durable `cleaned_up`
-     * flag via {@link GaiaRemote.fetchUncleanedTickets}: every ticket that is
-     * finished (state=done OR closed) but not yet cleaned. The SAME reconciliation
-     * runs on every tick AND standalone as `gaia conductor reap`, and the query is
+     * flag via {@link GaiaRemote.fetchUncleanedTickets}: every ticket the server
+     * has already closed but not yet cleaned. The SAME reconciliation runs on
+     * every tick AND standalone as `gaia conductor reap`, and the query is
      * scoped to THIS conductor (GAIA-121) — so it catches the orphans a
      * live-tick-only path structurally cannot (the conductor was down when the
-     * ticket hit `done`, or crashed mid-cleanup) for its OWN tickets. A foreign
+     * ticket closed, or crashed mid-cleanup) for its OWN tickets. A foreign
      * (non-gaia) worktree has no gaia ticket, so it is never on the work list; a
      * ticket assigned to another conductor is filtered out at the query — both are
      * left untouched with no host enumeration.
      *
-     * Per ticket, close is DECOUPLED from teardown (no withholding): a done ticket
-     * is closed at once as a lifecycle step, independent of whether its worktree
-     * teardown then succeeds. `cleaned_up` is withheld unless the teardown is
-     * ACCOUNTED FOR (GAIA-141 RC-2), which `removeWorktree` reports as one of
-     * three outcomes (GAIA-293):
+     * The conductor is structurally UNABLE to close a ticket (GAIA-451): closing
+     * is a server-side projection of terminality written by
+     * `GaiaTicket::preSave()` on any save of a terminal, unclosed ticket, so it
+     * holds for every write path and cannot be withheld by a teardown hiccup —
+     * the reap reasons about `cleaned_up` alone. `cleaned_up` is withheld unless
+     * the teardown is ACCOUNTED FOR (GAIA-141 RC-2), which `removeWorktree`
+     * reports as one of three outcomes (GAIA-293):
      *   - `removed` — gone because this host tore it down → flag cleaned;
      *   - `already_absent` — nothing was there to tear down. A FINISHED teardown,
      *     not a failure: it is recorded in this log AND on the run record, then

package/dist/src/core/conductor.js

@@ -330,23 +330,25 @@ export class Conductor {
         }
     }
     /**
-     * Reconcile finished tickets against their herdr worktrees and tear down any
+     * Reconcile closed tickets against their herdr worktrees and tear down any
      * whose workspace still lingers (GAIA-89). Driven by the durable `cleaned_up`
-     * flag via {@link GaiaRemote.fetchUncleanedTickets}: every ticket that is
-     * finished (state=done OR closed) but not yet cleaned. The SAME reconciliation
-     * runs on every tick AND standalone as `gaia conductor reap`, and the query is
+     * flag via {@link GaiaRemote.fetchUncleanedTickets}: every ticket the server
+     * has already closed but not yet cleaned. The SAME reconciliation runs on
+     * every tick AND standalone as `gaia conductor reap`, and the query is
      * scoped to THIS conductor (GAIA-121) — so it catches the orphans a
      * live-tick-only path structurally cannot (the conductor was down when the
-     * ticket hit `done`, or crashed mid-cleanup) for its OWN tickets. A foreign
+     * ticket closed, or crashed mid-cleanup) for its OWN tickets. A foreign
      * (non-gaia) worktree has no gaia ticket, so it is never on the work list; a
      * ticket assigned to another conductor is filtered out at the query — both are
      * left untouched with no host enumeration.
      *
-     * Per ticket, close is DECOUPLED from teardown (no withholding): a done ticket
-     * is closed at once as a lifecycle step, independent of whether its worktree
-     * teardown then succeeds. `cleaned_up` is withheld unless the teardown is
-     * ACCOUNTED FOR (GAIA-141 RC-2), which `removeWorktree` reports as one of
-     * three outcomes (GAIA-293):
+     * The conductor is structurally UNABLE to close a ticket (GAIA-451): closing
+     * is a server-side projection of terminality written by
+     * `GaiaTicket::preSave()` on any save of a terminal, unclosed ticket, so it
+     * holds for every write path and cannot be withheld by a teardown hiccup —
+     * the reap reasons about `cleaned_up` alone. `cleaned_up` is withheld unless
+     * the teardown is ACCOUNTED FOR (GAIA-141 RC-2), which `removeWorktree`
+     * reports as one of three outcomes (GAIA-293):
      *   - `removed` — gone because this host tore it down → flag cleaned;
      *   - `already_absent` — nothing was there to tear down. A FINISHED teardown,
      *     not a failure: it is recorded in this log AND on the run record, then
@@ -371,13 +373,6 @@ export class Conductor {
         const tickets = await this.remote.fetchUncleanedTickets(this.id);
         for (const t of tickets) {
             try {
-                // Lifecycle: close a done+unclosed ticket at once, decoupled from the
-                // infra teardown below — a done ticket whose conductor was down was
-                // never closed, and withholding the close on a teardown hiccup would
-                // make a finished ticket look unfinished (the pre-GAIA-89 defect).
-                if (t.state === 'done' && !t.closed) {
-                    await this.remote.closeTicket(t.ticketUuid);
-                }
                 // Infra teardown. The `after_done` hook runs in the worktree regardless
                 // of executor kind (a non-persistent gitWorkspace still has an on-disk
                 // worktree); only the workspace removal is gated on the persistent

package/dist/src/plugins/remote.d.ts

@@ -116,12 +116,14 @@ export interface RunMetrics {
     user_prompt_words: number;
 }
 /**
- * A finished ticket whose worktree has not yet been torn down — the
+ * A closed ticket whose worktree has not yet been torn down — the
  * conductor's teardown work list, driven by the durable `cleaned_up` flag
- * (GAIA-89). "Finished" = reached `done` OR already `closed`; `cleaned_up=0`
- * means the herdr worktree still needs reclaiming. The reconciliation is NOT
- * scoped to a conductor's machine_id, so it also surfaces tickets whose
- * conductor was down at `done` or whose `conductor_id` was reassigned.
+ * (GAIA-89). Since GAIA-451 the server closes every terminal ticket on its own
+ * save, so "finished" collapses to `closed = 1`; `cleaned_up=0` (or never
+ * written at all) means the herdr worktree still needs reclaiming. The
+ * reconciliation is NOT scoped to a conductor's machine_id, so it also
+ * surfaces tickets whose conductor was down at close or whose `conductor_id`
+ * was reassigned.
  */
 export interface UncleanTicket {
     ticketUuid: string;
@@ -132,10 +134,6 @@ export interface UncleanTicket {
      * `worktree_path`), or '' when unresolved. The cwd the teardown runs in.
      */
     worktreePath: string;
-    /** Workflow state (e.g. `coding`, `done`). */
-    state: string;
-    /** Whether the ticket's lifecycle has already been closed out. */
-    closed: boolean;
     /**
      * UUID of the ticket's latest run — the one {@link worktreePath} came from —
      * or '' when the ticket has no run with a path. The record teardown notes are
@@ -215,22 +213,19 @@ export interface GaiaRemote {
      */
     finalizeRun(uuid: string, log: string, metrics?: RunMetrics): Promise<void>;
     /**
-     * Finished tickets (state=done OR closed) whose worktree is not yet torn
-     * down (cleaned_up=0) — the conductor's teardown work list (GAIA-89). Driven
-     * by the durable `cleaned_up` flag AND scoped to the reaping conductor
-     * (GAIA-121): only tickets assigned to `conductorId` are loaded, so every
-     * ticket on the list is unambiguously this conductor's — a missing worktree
-     * means "already torn down on this host", not "belongs to another host". The
-     * same query backs the live tick and the standalone `gaia conductor reap`.
-     * Empty once every finished ticket this conductor owns is cleaned.
+     * Closed tickets (closed=1) whose worktree is not yet torn down
+     * (cleaned_up=0, or never written at all) — the conductor's teardown work
+     * list (GAIA-89). Since GAIA-451 the server closes every terminal ticket on
+     * its own save, so this asks one question — closed but not cleaned up —
+     * with no workflow-state reasoning left in it. Driven by the durable
+     * `cleaned_up` flag AND scoped to the reaping conductor (GAIA-121): only
+     * tickets assigned to `conductorId` are loaded, so every ticket on the list
+     * is unambiguously this conductor's — a missing worktree means "already torn
+     * down on this host", not "belongs to another host". The same query backs
+     * the live tick and the standalone `gaia conductor reap`. Empty once every
+     * closed ticket this conductor owns is cleaned.
      */
     fetchUncleanedTickets(conductorId: string): Promise<UncleanTicket[]>;
-    /**
-     * Close a ticket: set closed=true + closed_date. Ticket-lifecycle only, and
-     * decoupled from teardown — a done ticket is closed even if its worktree
-     * teardown later fails. No state change (the ticket is already done).
-     */
-    closeTicket(uuid: string): Promise<void>;
     /**
      * Mark a ticket's worktree torn down: set cleaned_up=true so it drops off the
      * {@link fetchUncleanedTickets} work list. Written ONLY after a verified

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gaia-ai/conductor",
-  "version": "0.11.0",
+  "version": "0.11.2",
   "description": "GAIA conductor engine + CLI: registers, claims tickets via JSON:API, dispatches agents.",
   "type": "module",
   "license": "MIT",
@@ -28,7 +28,7 @@
     "directory": "gaia-cli/conductor"
   },
   "dependencies": {
-    "@gaia-ai/core": "^0.11.0",
+    "@gaia-ai/core": "^0.11.2",
     "@dropsh/plugin-oauth2": "^0.6.1",
     "@dropsh/plugin-jsonapi-schema": "^0.6.1",
     "commander": "^12.1.0",