@aitne/daemon 0.1.6 → 0.1.8

package/dist/core/scheduler.d.ts

@@ -65,6 +65,16 @@ export declare function buildUserProfileSweepMorningCronExpr(dayBoundaryHour: nu
  * track the same config knob in lockstep.
  */
 export declare const USER_PROFILE_SWEEP_EVENING_CRON_EXPR = "50 17 * * *";
+/**
+ * Cron expression for the daemon-side roadmap mechanical maintenance
+ * pass: fixed at 17:45 local — 15 min before Evening Review's fixed
+ * 18:00 and 5 min before the evening user-profile sweep at 17:50.
+ * The ordering matters: this job must release its lock before the
+ * surviving evening_review Step 2 (Long-term Plans promotion + Review:
+ * fire) acquires `roadmap_write_lock`. See
+ * `docs/design/appendices/evening-review-slimdown.md` §2.2.
+ */
+export declare const ROADMAP_MAINTENANCE_CRON_EXPR = "45 17 * * *";
 /**
  * AgentScheduler — manages recurring cron jobs and the DB-driven ScheduleWatcher.
  *
@@ -112,6 +122,17 @@ export declare class AgentScheduler {
      * morning flow reads it. The observer owns the run-once guard.
      */
     private onContextIndexReconcile;
+    /**
+     * Evening-review slimdown §2.2 — daily mechanical roadmap.md
+     * maintenance pass at 17:45 local, 15 min before evening_review.
+     * Wraps `runRoadmapMechanicalMaintenance` so substeps 2a / 2b / 2d
+     * run in-process inside the daemon rather than inside a Sonnet
+     * routine that would otherwise re-derive the same date math. The
+     * callback is fire-and-forget (no return value contract): the
+     * implementation owns its own lock acquisition, audit emission, and
+     * agent/journal.md append.
+     */
+    private onRoadmapMaintenance;
     /**
      * Setup gate — returns a skip reason when autonomous work should be
      * paused (initial setup incomplete, or setup conversation active).
@@ -122,26 +143,32 @@ export declare class AgentScheduler {
     private autonomousGate;
     private lastGateBlockLoggedAt;
     /**
-     * Aborted by `stop()` to interrupt the ScheduleWatcher's between-poll sleep
-     * immediately. Without this, SIGTERM has to wait up to
-     * `schedulePollIntervalSeconds` before the loop notices `this.shutdown`.
+     * Single source of truth for the ScheduleWatcher's between-poll wake
+     * signaling, unifying what was previously a (pollAbort: AbortController
+     * + hasPendingNudge: boolean) pair (v4.14 audit, Task #3).
      *
-     * Also aborted by `nudgeWatcher()` so a freshly-INSERTed `agent_schedule`
-     * row with `scheduled_for = now` (e.g. the post-setup morning-routine wake)
-     * is picked up on the next event-loop tick instead of waiting up to a
-     * full poll interval. `sleepInterruptible` replaces the controller after
-     * consuming an abort so subsequent sleeps start fresh.
-     */
-    private pollAbort;
-    /**
-     * Pending-nudge flag — set by `nudgeWatcher()` and consumed at the top of
-     * `sleepInterruptible`. Covers the race where `nudgeWatcher()` fires
-     * between two sleeps (after the previous sleep's abort listener cleared
-     * the flag, but before the next sleep is entered). The AbortController
-     * alone cannot carry this signal across sleeps because a fresh controller
-     * is created per sleep.
+     * Mechanics:
+     *   - `nudgeSeq` is monotonically incremented by every `nudgeWatcher()`
+     *     call (and `stop()` reuses it to break the sleep too).
+     *   - `observedSeq` is advanced when a nudge has been consumed —
+     *     either at the top of `sleepInterruptible` (between-sleep case)
+     *     or by the resolver path inside `nudgeWatcher()` / `stop()` when
+     *     they wake an in-flight sleep.
+     *   - `sleepWaiter` holds the resolver + timer for the currently-pending
+     *     sleep, if any. The ScheduleWatcher loop awaits sequentially, so
+     *     at most one waiter exists at a time.
+     *
+     * Why this replaces the AbortController pair: with a monotonic counter
+     * we cannot lose a nudge across a sleep boundary — a `nudgeWatcher()`
+     * call that arrives between two sleeps simply leaves `nudgeSeq >
+     * observedSeq`, and the next sleep entry consumes it. The two-stage
+     * (flag + signal) scheme was correct on paper but required readers to
+     * trace the precise synchronous ordering of three branches in
+     * `sleepInterruptible`; the counter compresses that to one branch.
      */
-    private hasPendingNudge;
+    private nudgeSeq;
+    private observedSeq;
+    private sleepWaiter;
     constructor(eventBus: EventBus, db: Database.Database, config: AgentConfig);
     /**
      * Register a callback that runs at the day boundary (4 AM) BEFORE
@@ -167,6 +194,14 @@ export declare class AgentScheduler {
      * and enqueue a reconcile via the observer's run-once guard.
      */
     setContextIndexReconcilerCallback(fn: () => void): void;
+    /**
+     * Register the daily roadmap mechanical maintenance callback. Called
+     * at 17:45 local; the callback is fire-and-forget — it owns its own
+     * lock acquisition, validation, audit emission, and journal append.
+     * Failures are logged but do NOT cascade into the 18:00
+     * evening_review.
+     */
+    setRoadmapMaintenanceCallback(fn: () => void): void;
     /**
      * Register the autonomous-work gate. Returning null allows autonomous work;
      * any non-null string is treated as a skip reason and surfaced in logs.
@@ -174,6 +209,45 @@ export declare class AgentScheduler {
     setAutonomousGate(fn: () => null | string): void;
     /** Log gate blocks at most once every 5 minutes to avoid spam. */
     private logGateBlock;
+    /**
+     * In-process re-entrance mutex for the stall watchdog. The hourly cron
+     * fires the watchdog as a fire-and-forget promise; a slow `sendDm`
+     * (DM hub latency, owner channel reconnect) could overlap with the
+     * next hourly tick. Without serialization both invocations would pass
+     * the dedup-marker read (the marker isn't written until AFTER the DM
+     * succeeds, see below) and emit duplicate alerts. The mutex is
+     * process-local — fine, since a single scheduler instance is the only
+     * watchdog firing path.
+     */
+    private morningStallWatchdogRunning;
+    /**
+     * Watchdog for the silent-stall pattern documented in the v4.14 audit
+     * (CLAUDE.md "morning_routine wake stall"). When the morning routine
+     * never writes an `agent_actions.result='success'` row, the dedup
+     * inside `queueMorningRoutineWake` keeps the stuck wake row pinned in
+     * `pending`/`running` and the hourly-check pre-routine gate silently
+     * skips every subsequent autonomous tick. The user gets no morning
+     * brief, no evening review, no hourly check, and no error — the
+     * system is functionally dead until the wake row clears.
+     *
+     * Detection: oldest `task_type='wake'` row tied to
+     * `routine='morning_routine'` older than the configured threshold (see
+     * `readMorningRoutineStallThresholdMinutes`) with no matching
+     * `agent_actions.result='success'` row in the current agent-day window.
+     *
+     * Delivery & dedup: DM-then-mark, NOT mark-then-DM. A failed DM (DM
+     * hub error, missing owner channel) leaves the marker empty so the
+     * next hourly tick retries. Repeat alerts on a chronically-broken DM
+     * channel are accepted as the lesser evil vs. a silent miss. The
+     * `morningStallWatchdogRunning` mutex serialises overlapping
+     * invocations so a slow DM cannot produce duplicate alerts.
+     *
+     * Fire-and-forget: returns nothing, surfaces all failures via logger.
+     * The owner DM is best-effort — a sendDm failure logs and does NOT
+     * re-throw, so a missing or broken message hub cannot cascade into a
+     * cron-callback exception.
+     */
+    private checkMorningRoutineStall;
     start(): void;
     stop(): void;
     /**
@@ -183,15 +257,23 @@ export declare class AgentScheduler {
      *
      * Safe to call any time after `start()` (and a no-op before): the public
      * surface is idempotent — multiple rapid calls coalesce into a single
-     * extra wake at the next sleep boundary. The pending-nudge flag covers
-     * the race where this method is invoked between sleeps; `sleepInterruptible`
-     * consumes it at the top of the next sleep cycle.
+     * extra wake at the next sleep boundary. Between-sleep nudges land on
+     * the monotonic `nudgeSeq` counter and are consumed at the top of the
+     * next `sleepInterruptible`; in-flight nudges resolve the active
+     * waiter directly. Both paths advance `observedSeq` so a single nudge
+     * skips exactly one sleep, never two.
      *
      * Wired today from `queueMorningRoutineWake` (post-setup, post-Google-auth,
      * catchup); other `INSERT INTO agent_schedule` sites use future-dated rows
      * and do not need a nudge.
      */
     nudgeWatcher(): void;
+    /**
+     * Resolve and clear the active `sleepWaiter`, if any. Used by both
+     * `stop()` and `nudgeWatcher()` to break a running sleep. Safe to call
+     * when no waiter exists — it's a no-op.
+     */
+    private consumeWaiter;
     /** Hot-reload cron schedules (e.g. after dayBoundaryHour changes) */
     reloadCrons(): void;
     private stopCronJobs;
@@ -220,25 +302,27 @@ export declare class AgentScheduler {
      */
     private startScheduleWatcher;
     /**
-     * Sleep for `ms` milliseconds, but resolve early if `pollAbort` aborts.
-     * Used by the ScheduleWatcher between polls so `stop()` and
-     * `nudgeWatcher()` can interrupt it.
+     * Sleep for `ms` milliseconds, but resolve early when `stop()` or
+     * `nudgeWatcher()` fire. Used by the ScheduleWatcher between polls so
+     * shutdown / "wake now" signals interrupt the wait.
      *
-     * Resolution order:
+     * Resolution order (single linear sequence — no branch fan-out):
      *   1. `shutdown` → immediate resolve.
-     *   2. pending nudge flag (set by `nudgeWatcher()` while we were processing
-     *      the previous batch) → consume and immediate resolve.
-     *   3. existing controller already aborted (legacy `stop()` contract, plus
-     *      the in-flight `nudgeWatcher()` path where `onAbort` already
-     *      consumed the flag) → replace controller and immediate resolve so
-     *      the next sleep starts fresh.
-     *   4. otherwise wait for either the timer or a fresh abort.
+     *   2. `nudgeSeq > observedSeq` → consume + immediate resolve. Covers
+     *      every nudge that arrived while no sleep was active (before the
+     *      first sleep ever ran, or between two consecutive sleeps).
+     *   3. otherwise install a `sleepWaiter` with a setTimeout and wait;
+     *      the resolver is invoked either by the timer or by a subsequent
+     *      `stop()` / `nudgeWatcher()` via `consumeWaiter()`.
      *
-     * Controller lifecycle: a new `AbortController` is installed on entry to
-     * step 4 so a previous abort cannot make this sleep return instantly.
-     * `nudgeWatcher()` always sets `hasPendingNudge` BEFORE calling `abort()`
-     * — the synchronous order is what closes the race against this method
-     * reading `pollAbort` and installing listeners.
+     * Invariant: `observedSeq` only advances on consumption, so a `nudgeSeq`
+     * bump that arrives during the active sleep is observed by the caller
+     * (`nudgeWatcher`) which advances `observedSeq` AND resolves the waiter
+     * in the same synchronous block. A bump that arrives between sleeps is
+     * still visible (`nudgeSeq > observedSeq`) on the next entry and gets
+     * consumed there. The counter therefore cannot lose a nudge across the
+     * sleep boundary that the previous AbortController scheme had to guard
+     * against with a separate `hasPendingNudge` flag.
      */
     private sleepInterruptible;
     /**

package/dist/core/scheduler.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"scheduler.d.ts","sourceRoot":"","sources":["../../src/core/scheduler.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAY3C,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAChD,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAK/C,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAelE;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH,wBAAgB,mBAAmB,CACjC,eAAe,EAAE,MAAM,EACvB,SAAS,EAAE,MAAM,EACjB,gBAAgB,EAAE,MAAM,GACvB,MAAM,CAaR;AAED;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,sBAAsB,CACpC,SAAS,EAAE,MAAM,EACjB,WAAW,EAAE,MAAM,EACnB,eAAe,EAAE,MAAM,EACvB,eAAe,EAAE,MAAM,GACtB,OAAO,CAWT;AAED;;;;;;GAMG;AACH,wBAAgB,oCAAoC,CAClD,eAAe,EAAE,MAAM,GACtB,MAAM,CAGR;AAED;;;;;GAKG;AACH,eAAO,MAAM,oCAAoC,gBAAgB,CAAC;AAgBlE;;;;;;;;;;;;;GAaG;AACH,qBAAa,cAAc;IACzB,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAW;IACpC,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAoB;IACvC,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAc;IACrC,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAuB;IAChD,OAAO,CAAC,mBAAmB,CAAS;IACpC,OAAO,CAAC,aAAa,CAAsC;IAC3D,OAAO,CAAC,MAAM,CAAwF;IACtG,OAAO,CAAC,aAAa,CAAuD;IAC5E;;;;;;;;;;;;;;OAcG;IACH,OAAO,CAAC,WAAW,CAAyC;IAC5D;;;;;OAKG;IACH,OAAO,CAAC,uBAAuB,CAA6B;IAC5D;;;;;;OAMG;IACH,OAAO,CAAC,cAAc,CAAqC;IAC3D,OAAO,CAAC,qBAAqB,CAAK;IAClC;;;;;;;;;;OAUG;IACH,OAAO,CAAC,SAAS,CAAgC;IACjD;;;;;;;OAOG;IACH,OAAO,CAAC,eAAe,CAAS;gBAEpB,QAAQ,EAAE,QAAQ,EAAE,EAAE,EAAE,QAAQ,CAAC,QAAQ,EAAE,MAAM,EAAE,WAAW;IAM1E;;;OAGG;IACH,sBAAsB,CAAC,EAAE,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,GAAG,IAAI;IAIrD;;;OAGG;IACH,iBAAiB,CAAC,EAAE,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,SAAS,CAAC,EAAE,MAAM,EAAE,KAAK,OAAO,CAAC,eAAe,EAAE,CAAC,GAAG,IAAI;IAIlG,sBAAsB,CAAC,EAAE,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,GAAG,IAAI;IAItE;;;;;OAKG;IACH,oBAAoB,CAAC,EAAE,EAAE,MAAM,OAAO,CAAC,OAAO,CAAC,GAAG,IAAI;IAItD;;;;OAIG;IACH,iCAAiC,CAAC,EAAE,EAAE,MAAM,IAAI,GAAG,IAAI;IAIvD;;;OAGG;IACH,iBAAiB,CAAC,EAAE,EAAE,MAAM,IAAI,GAAG,MAAM,GAAG,IAAI;IAIhD,kEAAkE;IAClE,OAAO,CAAC,YAAY;IAOpB,KAAK,IAAI,IAAI;IAMb,IAAI,IAAI,IAAI;IASZ;;;;;;;;;;;;;;OAcG;IACH,YAAY,IAAI,IAAI;IAKpB,qEAAqE;IACrE,WAAW,IAAI,IAAI;IAMnB,OAAO,CAAC,YAAY;IAOpB,OAAO,CAAC,kBAAkB;IA4P1B,OAAO,CAAC,WAAW;IAkBnB;;;;;;OAMG;IACH,uBAAuB,CACrB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE;QAAE,mBAAmB,CAAC,EAAE,MAAM,EAAE,CAAC;QAAC,sBAAsB,CAAC,EAAE,OAAO,CAAA;KAAE,GAC7E;QAAE,QAAQ,EAAE,OAAO,CAAC;QAAC,UAAU,CAAC,EAAE,MAAM,CAAA;KAAE;IAkG7C;;;;;;OAMG;IACH,OAAO,CAAC,oBAAoB;IA+M5B;;;;;;;;;;;;;;;;;;;;OAoBG;IACH,OAAO,CAAC,kBAAkB;IA0B1B;;;;;;;OAOG;YACW,cAAc;IA8D5B,oDAAoD;IACpD,OAAO,CAAC,YAAY;CA2FrB"}
+{"version":3,"file":"scheduler.d.ts","sourceRoot":"","sources":["../../src/core/scheduler.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAe3C,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAChD,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAK/C,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAC;AA6BlE;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH,wBAAgB,mBAAmB,CACjC,eAAe,EAAE,MAAM,EACvB,SAAS,EAAE,MAAM,EACjB,gBAAgB,EAAE,MAAM,GACvB,MAAM,CAaR;AAED;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,sBAAsB,CACpC,SAAS,EAAE,MAAM,EACjB,WAAW,EAAE,MAAM,EACnB,eAAe,EAAE,MAAM,EACvB,eAAe,EAAE,MAAM,GACtB,OAAO,CAWT;AAED;;;;;;GAMG;AACH,wBAAgB,oCAAoC,CAClD,eAAe,EAAE,MAAM,GACtB,MAAM,CAGR;AAED;;;;;GAKG;AACH,eAAO,MAAM,oCAAoC,gBAAgB,CAAC;AAElE;;;;;;;;GAQG;AACH,eAAO,MAAM,6BAA6B,gBAAgB,CAAC;AA8B3D;;;;;;;;;;;;;GAaG;AACH,qBAAa,cAAc;IACzB,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAW;IACpC,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAoB;IACvC,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAc;IACrC,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAuB;IAChD,OAAO,CAAC,mBAAmB,CAAS;IACpC,OAAO,CAAC,aAAa,CAAsC;IAC3D,OAAO,CAAC,MAAM,CAAwF;IACtG,OAAO,CAAC,aAAa,CAAuD;IAC5E;;;;;;;;;;;;;;OAcG;IACH,OAAO,CAAC,WAAW,CAAyC;IAC5D;;;;;OAKG;IACH,OAAO,CAAC,uBAAuB,CAA6B;IAC5D;;;;;;;;;OASG;IACH,OAAO,CAAC,oBAAoB,CAA6B;IACzD;;;;;;OAMG;IACH,OAAO,CAAC,cAAc,CAAqC;IAC3D,OAAO,CAAC,qBAAqB,CAAK;IAClC;;;;;;;;;;;;;;;;;;;;;;;OAuBG;IACH,OAAO,CAAC,QAAQ,CAAK;IACrB,OAAO,CAAC,WAAW,CAAK;IACxB,OAAO,CAAC,WAAW,CAGH;gBAEJ,QAAQ,EAAE,QAAQ,EAAE,EAAE,EAAE,QAAQ,CAAC,QAAQ,EAAE,MAAM,EAAE,WAAW;IAM1E;;;OAGG;IACH,sBAAsB,CAAC,EAAE,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,GAAG,IAAI;IAIrD;;;OAGG;IACH,iBAAiB,CAAC,EAAE,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,SAAS,CAAC,EAAE,MAAM,EAAE,KAAK,OAAO,CAAC,eAAe,EAAE,CAAC,GAAG,IAAI;IAIlG,sBAAsB,CAAC,EAAE,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,GAAG,IAAI;IAItE;;;;;OAKG;IACH,oBAAoB,CAAC,EAAE,EAAE,MAAM,OAAO,CAAC,OAAO,CAAC,GAAG,IAAI;IAItD;;;;OAIG;IACH,iCAAiC,CAAC,EAAE,EAAE,MAAM,IAAI,GAAG,IAAI;IAIvD;;;;;;OAMG;IACH,6BAA6B,CAAC,EAAE,EAAE,MAAM,IAAI,GAAG,IAAI;IAInD;;;OAGG;IACH,iBAAiB,CAAC,EAAE,EAAE,MAAM,IAAI,GAAG,MAAM,GAAG,IAAI;IAIhD,kEAAkE;IAClE,OAAO,CAAC,YAAY;IAOpB;;;;;;;;;OASG;IACH,OAAO,CAAC,2BAA2B,CAAS;IAE5C;;;;;;;;;;;;;;;;;;;;;;;;;;OA0BG;YACW,wBAAwB;IAiGtC,KAAK,IAAI,IAAI;IAMb,IAAI,IAAI,IAAI;IAYZ;;;;;;;;;;;;;;;;OAgBG;IACH,YAAY,IAAI,IAAI;IAQpB;;;;OAIG;IACH,OAAO,CAAC,aAAa;IAQrB,qEAAqE;IACrE,WAAW,IAAI,IAAI;IAMnB,OAAO,CAAC,YAAY;IAOpB,OAAO,CAAC,kBAAkB;IAyS1B,OAAO,CAAC,WAAW;IAkBnB;;;;;;OAMG;IACH,uBAAuB,CACrB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE;QAAE,mBAAmB,CAAC,EAAE,MAAM,EAAE,CAAC;QAAC,sBAAsB,CAAC,EAAE,OAAO,CAAA;KAAE,GAC7E;QAAE,QAAQ,EAAE,OAAO,CAAC;QAAC,UAAU,CAAC,EAAE,MAAM,CAAA;KAAE;IAyH7C;;;;;;OAMG;IACH,OAAO,CAAC,oBAAoB;IAkO5B;;;;;;;;;;;;;;;;;;;;;;OAsBG;IACH,OAAO,CAAC,kBAAkB;IAqB1B;;;;;;;OAOG;YACW,cAAc;IAyF5B,oDAAoD;IACpD,OAAO,CAAC,YAAY;CA2FrB"}