@north-light/crouter-api 0.3.249 → 0.3.251

package/dist/api/__tests__/integration/client.test.js

@@ -1,6 +1,6 @@
 // client.test.ts — regression for issue #516: a cold-start `/healthz` timeout
 // used to throw a bare `daemon_unavailable` message that discarded the actual
-// startup failure (the tail of crtrd.err). `coldStartTimeoutMessage` is the
+// startup failure (the tail of crtrd.log). `coldStartTimeoutMessage` is the
 // pure composition `handleColdSocket` uses on timeout; this proves an injected
 // diagnostic is appended (not dropped) and that a missing/empty diagnostic
 // still falls back to the original bare message unchanged. `safeColdStartDiagnostic`
@@ -11,19 +11,19 @@
 import { test } from 'node:test';
 import assert from 'node:assert/strict';
 import { createServer } from 'node:http';
-import { mkdtempSync, rmSync } from 'node:fs';
+import { mkdirSync, mkdtempSync, rmSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { coldStartTimeoutMessage, CrtrClient, safeColdStartDiagnostic } from '../../client.js';
 import { ApiError } from '../../errors.js';
 test('coldStartTimeoutMessage appends a present diagnostic to the base message', () => {
-    const msg = coldStartTimeoutMessage('crtrd.err (tail):\nError: bind EADDRINUSE');
+    const msg = coldStartTimeoutMessage('crtrd.log (tail):\nError: bind EADDRINUSE');
     assert.match(msg, /crtrd did not start/);
-    assert.match(msg, /crtrd\.err \(tail\):\nError: bind EADDRINUSE/);
+    assert.match(msg, /crtrd\.log \(tail\):\nError: bind EADDRINUSE/);
 });
 test('coldStartTimeoutMessage falls back to the bare message when no diagnostic is available', () => {
-    assert.equal(coldStartTimeoutMessage(undefined), 'crtrd did not start; run `crtr sys daemon start` and check crtrd.err.');
-    assert.equal(coldStartTimeoutMessage(''), 'crtrd did not start; run `crtr sys daemon start` and check crtrd.err.');
+    assert.equal(coldStartTimeoutMessage(undefined), 'crtrd did not start; run `crtr sys daemon start` and check crtrd.log.');
+    assert.equal(coldStartTimeoutMessage(''), 'crtrd did not start; run `crtr sys daemon start` and check crtrd.log.');
 });
 function startDelayedHealthzServer(socketPath, delayMs) {
     const server = createServer((_req, res) => {
@@ -85,6 +85,27 @@ test('a hang-up mid-mutation reports daemon_restarting, not daemon_unavailable',
         rmSync(dir, { recursive: true, force: true });
     }
 });
+test('a non-socket cold-start path surfaces its diagnostic after autostart times out', async () => {
+    const dir = mkdtempSync(join(tmpdir(), 'crtr-client-nonsocket-'));
+    const socketPath = join(dir, 'crtrd.sock');
+    mkdirSync(socketPath);
+    const diagnostic = 'crtrd.log (tail):\napi.server.failed: EADDRINUSE';
+    const client = new CrtrClient({
+        socketPath,
+        autostart: true,
+        onColdSocket: async () => { },
+        coldStartPollWindowMs: 50,
+        coldStartDiagnostic: () => diagnostic,
+    });
+    try {
+        await assert.rejects(() => client.healthz(), (error) => error instanceof ApiError
+            && error.code === 'daemon_unavailable'
+            && error.message.includes(diagnostic));
+    }
+    finally {
+        rmSync(dir, { recursive: true, force: true });
+    }
+});
 test('cliClient-style cold start fails loud when the injected poll window is shorter than the (valid) startup delay', async () => {
     const dir = mkdtempSync(join(tmpdir(), 'crtr-client-coldstart-'));
     const socketPath = join(dir, 'crtrd.sock');
@@ -132,7 +153,7 @@ test('safeColdStartDiagnostic returns undefined for an absent hook', () => {
     assert.equal(safeColdStartDiagnostic(undefined), undefined);
 });
 test('safeColdStartDiagnostic returns the hook result when it succeeds', () => {
-    assert.equal(safeColdStartDiagnostic(() => 'crtrd.err (tail):\nboom'), 'crtrd.err (tail):\nboom');
+    assert.equal(safeColdStartDiagnostic(() => 'crtrd.log (tail):\nboom'), 'crtrd.log (tail):\nboom');
 });
 test('safeColdStartDiagnostic treats a THROWING hook as absent, not a propagated error', () => {
     assert.equal(safeColdStartDiagnostic(() => {

package/dist/api/client.d.ts

@@ -39,7 +39,7 @@ export interface CrtrClientOptions {
     onColdSocket?: () => Promise<void>;
     /** Injected cold-start diagnostic. Called ONLY when the bounded
      *  `/healthz` poll times out after `onColdSocket`, so the caller can attach
-     *  operator-useful context (e.g. a bounded tail of crtrd's stderr log) to the
+     *  operator-useful context (e.g. a bounded tail of `crtrd.log`) to the
      *  `daemon_unavailable` error instead of it staying a bare message. Must
      *  return synchronously and cheaply — it runs on the failure path, not the
      *  happy path. A thrown/undefined result is treated as "no diagnostic". */

package/dist/api/client.js

@@ -668,7 +668,7 @@ export class CrtrClient {
         if (this.socketPath === undefined)
             return false;
         const code = err?.code;
-        return code === 'ECONNREFUSED' || code === 'ENOENT';
+        return code === 'ECONNREFUSED' || code === 'ENOENT' || code === 'ENOTSOCK';
     }
     /** A connection torn down MID-request (Node's "socket hang up" / a broken
      *  pipe) — as distinct from a refused connect, which means nothing is
@@ -817,7 +817,7 @@ function sleep(ms) {
  *  design) purely so the regression test can assert the composition without
  *  waiting out the real poll window. */
 export function coldStartTimeoutMessage(diagnostic) {
-    const base = 'crtrd did not start; run `crtr sys daemon start` and check crtrd.err.';
+    const base = 'crtrd did not start; run `crtr sys daemon start` and check crtrd.log.';
     return diagnostic !== undefined && diagnostic !== '' ? `${base}\n${diagnostic}` : base;
 }
 /** Invoke the injected `coldStartDiagnostic` hook, treating a THROW the same

package/dist/api/dto/lifecycle.d.ts

@@ -47,6 +47,9 @@ export interface ReviveAllResultDTO {
  *  from edges; the flag only opts out of cascading descendants. */
 export interface CloseRequest {
     cascade?: boolean;
+    /** PID of the shell that invoked this close. The daemon excludes this process
+     *  and its descendants from broker teardown so a node can close itself. */
+    caller_pid?: number;
     /** Root close disposition: `true` finalizes the root to `done` (the browse `x`
      *  "finish" semantics); default/`false` cancels it. The cascade set (computed
      *  server-side) is torn down either way. */

package/dist/api/dto/messages.d.ts

@@ -62,6 +62,10 @@ export interface MessageResultDTO {
      *  frozen and the daemon relaunches it when one frees. The entry is durably
      *  appended either way, and delivery follows that relaunch. */
     not_revived_reason?: 'capacity_frozen';
+    /** Present only when the entry was appended ABOVE the requested tier: a
+     *  terminal target never takes the later cycle `normal`/`deferred` wait for,
+     *  so its mail is raised to the steering tier. */
+    delivered_tier?: InboxTierDTO;
     delivered_at?: IsoTime;
     /** Which channel a `delivery:'interactive'` send actually used: `'engine'`
      *  (live broker frame loop, no inbox entry) or `'inbox'` (durable fallback).

package/dist/api/dto/nodes.d.ts

@@ -36,6 +36,8 @@ export interface CreateNodeRequest {
     /** Caller-supplied short description, preserved against automatic naming. */
     description?: string;
     parent?: NodeIdDTO | null;
+    /** Node that created this request; absent for an external process. */
+    creator?: NodeIdDTO;
     root?: boolean;
     /** Lifecycle of the new node. Roots default to `resident`; managed children
      *  default to `terminal`. Use `resident` for a child that remains wakeable
@@ -165,6 +167,8 @@ export interface NodeFaultDTO {
 /** The full node view — summary ∪ identity extras ∪ edges ∪ paths. Returned by
  *  `GET /v1/nodes/{id}` and by the create/lifecycle actions that yield a node. */
 export interface NodeDetailDTO extends NodeSummaryDTO {
+    /** Node that created this node, or null when an external process did. */
+    creator: NodeIdDTO | null;
     description?: string;
     /** The namer's prose form of `description` — sentence case, punctuation intact
      *  (`NodeMeta.title`). What a surface showing this node to a person reads;

package/dist/api/dto/reports.d.ts

@@ -1,9 +1,14 @@
 import type { IsoTime, NodeIdDTO } from './common.js';
-/** Report tier — `crtr push {update,urgent,final}`. */
+/** Report kind — `crtr push {update,final}`; legacy `urgent` rows remain readable. */
 export type ReportTierDTO = 'update' | 'urgent' | 'final';
-/** `POST /v1/nodes/{id}/reports` body ({id} = the reporting node). */
+/** Delivery urgency for an update report's subscriber fan-out. */
+export type ReportDeliveryTierDTO = 'deferred' | 'normal' | 'urgent';
+/** `POST /v1/nodes/{id}/reports` body ({id} = the reporting node).
+ * `tier` is the stored report kind; `delivery_tier` is the optional inbox
+ * delivery urgency for an update's subscriber fan-out. */
 export interface PushReportRequest {
     tier: ReportTierDTO;
+    delivery_tier?: ReportDeliveryTierDTO;
     body: string;
 }
 /** Result of a push. `transitioned` is present only for `final`, which drives a

package/dist/shared/generated-context.d.ts

@@ -10,10 +10,14 @@ export declare const REVIEW_BOUNDARY_CUSTOM_TYPE = "crtr-review-boundary";
 export declare const STALL_REPROMPT: string;
 /** The daemon's parking mandate: the last turn of a conversation the unattended
  *  clock is concluding. Delivered live, wrapped in a `park` runtime card. It
- *  shares a CONTRACT with `node yield`'s pre-invocation guide (roadmap current,
- *  short and shrinking; context dir for in-progress material; memory only for
- *  gated permanent lessons) but not a string: yield is read by an agent choosing
- *  to refresh, this by an agent being told to conclude. */
+ *  governs both the node's durable inheritance and reader-facing output:
+ *  one update for subscribers and history, then a goodbye that does not narrate
+ *  the parking.
+ *
+ *  The durable inheritance shares a CONTRACT with `node yield`'s pre-invocation
+ *  guide (roadmap current, short and shrinking; context dir for in-progress
+ *  material; memory only for gated permanent lessons) but not a string: yield is
+ *  read by an agent choosing to refresh, this by an agent being told to conclude. */
 export declare const PARK_SUMMARY_PROMPT: string;
 /** Static recovery prompts shared by the broker producer and display classifier. */
 export declare const AUTH_FAULT_RECOVERY_BODY = "Provider credentials were just updated (a new login landed). Your previous turn stopped on a provider authentication failure. Continue from where you left off and retry the work that failed.";

package/dist/shared/generated-context.js

@@ -15,15 +15,21 @@ export const STALL_REPROMPT = "You've stopped but you're not waiting on anyone a
     "Pipe the result to `crtr push final` through a single-quoted heredoc if the work is done, or use `crtr human send` if you are blocked or need the user.";
 /** The daemon's parking mandate: the last turn of a conversation the unattended
  *  clock is concluding. Delivered live, wrapped in a `park` runtime card. It
- *  shares a CONTRACT with `node yield`'s pre-invocation guide (roadmap current,
- *  short and shrinking; context dir for in-progress material; memory only for
- *  gated permanent lessons) but not a string: yield is read by an agent choosing
- *  to refresh, this by an agent being told to conclude. */
-export const PARK_SUMMARY_PROMPT = 'This conversation has been idle with nothing waiting on it, so it is being concluded. This is your last turn — do these four things now, then stop.\n\n'
-    + '1. Bring your roadmap current. It lives at `$CRTR_CONTEXT_DIR/roadmap.md`, and it is the document a fresh cycle reads to know where things stand: what was done, what is still open, and what someone picking this up needs. Write one now if you never had one. Keep it short and shrinking — strategy and present state only, deleting completed items rather than marking them done.\n\n'
-    + '2. Save anything else worth keeping. In-progress task material — what has happened, identifiers, in-flight state — goes to one or more files in your context directory. A permanent lesson worth remembering indefinitely goes to node memory (`crtr memory write -h`), gated deliberately: transcript state does not belong in memory.\n\n'
-    + '3. Push one regular report to your subscribers summarizing where things stand: `crtr push update`. NOT `crtr push final` — this concludes the conversation, it does not finish a mandate.\n\n'
-    + '4. Stop. A later message reopens you on a fresh context window grounded in your goal and that roadmap, so what you leave behind now is what you get back.';
+ *  governs both the node's durable inheritance and reader-facing output:
+ *  one update for subscribers and history, then a goodbye that does not narrate
+ *  the parking.
+ *
+ *  The durable inheritance shares a CONTRACT with `node yield`'s pre-invocation
+ *  guide (roadmap current, short and shrinking; context dir for in-progress
+ *  material; memory only for gated permanent lessons) but not a string: yield is
+ *  read by an agent choosing to refresh, this by an agent being told to conclude. */
+export const PARK_SUMMARY_PROMPT = 'This conversation has been idle with nothing left to wake it, so it is being concluded. This is your last turn. Use it to leave a trustworthy inheritance, not to restart or broaden the work. Do these six things now, then stop.\n\n'
+    + '1. Establish current truth. Check only state that may have changed outside the transcript and matters to resuming—such as the working tree, a remote run, or an external decision. Do not start new work; perform only a quick check needed to avoid recording an unverified claim. If no mandate or work ever began, say that plainly and keep every artifact minimal.\n\n'
+    + '2. Rewrite `$CRTR_CONTEXT_DIR/roadmap.md` for a fresh context window. This is the only handoff document a later fresh cycle receives in full. Preserve the current goal and exit criteria when they still apply, then state the present outcome; what remains or is blocked; decisions or questions still open; exact recovery handles for in-flight state; and the first safe move on return. Keep strategy and present state, not a transcript recap. Delete stale and completed steps instead of marking them done; the roadmap should stay short and shrink. Write a minimal one now if none exists.\n\n'
+    + '3. Put supporting material in your context directory only when the roadmap would become bulky without it. Rewrite existing living documents rather than leave superseded versions. Name every supporting file the next cycle must read from the roadmap and say what it is for—the revive shows filenames but does not inject their contents. Task state, identifiers, and recovery detail belong here, not in memory.\n\n'
+    + '4. Use memory only for a non-obvious, reusable lesson that should survive this task and is not already recorded. Read `crtr memory write -h`, find before writing, and choose the narrowest scope that will reach the next agent who needs it. Do not put a conversation recap, task status, recovery handles, or facts already captured in code or docs into memory.\n\n'
+    + '5. Push exactly one regular update with `crtr push update --tier deferred`, never `crtr push final`. Write it for subscribers and history, not as a second roadmap. Its first line must stand alone as the current outcome, blocker, or decision that matters; then include only unfinished work, a needed decision, and concrete handles a subscriber may need. This concludes the conversation; it does not finish the mandate.\n\n'
+    + '6. End with a short sign-off to the reader in second person: say you are putting your notes in order and pausing here until they come back, and mention anything genuinely worth their attention, such as unfinished work or an open question. Keep all visible text this turn—including the update\'s first line—about their work, not crouter\'s machinery: do not mention roadmap or context paths, filing a report, parking, idling, residency, or node ids. Then stop. A later message reopens you on a fresh context window grounded in your goal and roadmap, so the inheritance you leave now is what you get back.';
 /** Static recovery prompts shared by the broker producer and display classifier. */
 export const AUTH_FAULT_RECOVERY_BODY = 'Provider credentials were just updated (a new login landed). Your previous turn stopped on a provider authentication failure. Continue from where you left off and retry the work that failed.';
 export const CONNECTION_FAULT_RECOVERY_BODY = 'The network connection is back online. Your previous turn stopped on a connection error (the network was down). Continue from where you left off and retry the work that failed.';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@north-light/crouter-api",
-  "version": "0.3.249",
+  "version": "0.3.251",
   "description": "Typed crtrd /v1 API contract — DTOs, route builders, the error contract, and the CrtrClient. Zero runtime dependencies.",
   "type": "module",
   "main": "./dist/api/index.js",