harnery 0.34.1 → 0.35.0

package/src/core/agents/state/names.ts

@@ -5,6 +5,14 @@
  * Layout invariant: 260 entries (10 per starting letter A..Z). Counter N
  * picks COORD_NAMES[N % 260]; wraps to A at N=260.
  *
+ * Gender invariant: within each 26-name pass the names alternate female/male
+ * letter by letter, so every pass is 13 female and 13 male. Five passes start
+ * female at A (1, 3, 5, 7, 9) and five start male (2, 4, 6, 8, 10), which gives
+ * each letter exactly five names of each gender and the pool 130/130 overall.
+ * This exists so a fixed set of 13 female and 13 male portraits can dress one
+ * whole pass. Adding or reordering a name must preserve it; the expected
+ * pattern is pinned in tests/unit/names-pool.test.ts.
+ *
  * Durable persistence: `.harnery/.name-history` (JSONL, one row per assignment)
  * + `.harnery/.name-counter` (current counter, atomic temp+rename).
  *
@@ -24,274 +32,274 @@ import { dirname, join } from "node:path";
 
 /** All 260 names. */
 export const COORD_NAMES = [
-  // Pass 1
+  // Pass 1 — female-first (A=female), alternating to Z
   "Anna",
   "Bob",
-  "Carlos",
-  "Dalia",
+  "Carmen",
+  "Dorian",
   "Erika",
   "Fred",
   "Gemma",
   "Hank",
   "Imelda",
   "Jerry",
-  "Kirk",
-  "Lee",
-  "Milton",
+  "Karen",
+  "Lorenzo",
+  "Mindy",
   "Nate",
-  "Oscar",
-  "Patty",
-  "Quentin",
+  "Odette",
+  "Peter",
+  "Quinn",
   "Rafael",
   "Sara",
   "Tony",
-  "Ulrik",
-  "Valerie",
+  "Ursula",
+  "Vinson",
   "Whitney",
   "Xander",
   "Yara",
-  "Zara",
-  // Pass 2
-  "Adelaide",
+  "Zane",
+  // Pass 2 — male-first (A=male), alternating to Z
+  "Alex",
   "Bertha",
-  "Carmen",
-  "Dorian",
-  "Edna",
+  "Carlos",
+  "Dalia",
+  "Ernesto",
   "Francine",
-  "Greta",
+  "Gordon",
   "Helene",
-  "Ines",
+  "Isaias",
   "Joyce",
-  "Karen",
-  "Lorenzo",
-  "Mindy",
+  "Kirk",
+  "Lucia",
+  "Milton",
   "Nadine",
-  "Odette",
-  "Paulette",
-  "Quinn",
+  "Oscar",
+  "Patty",
+  "Quentin",
   "Rebekah",
   "Sean",
   "Tammy",
-  "Ursula",
-  "Vinson",
+  "Ulrik",
+  "Valerie",
   "Wallace",
   "Xanthe",
   "Yusuf",
-  "Zoe",
-  // Pass 3
-  "Alex",
+  "Zara",
+  // Pass 3 — female-first (A=female), alternating to Z
+  "Adelaide",
+  "Bruno",
+  "Cora",
+  "Damon",
+  "Edna",
+  "Felix",
+  "Greta",
+  "Hugo",
+  "Ines",
+  "Jasper",
+  "Kaia",
+  "Larry",
+  "Maxine",
+  "Nicholas",
+  "Olive",
+  "Paxton",
+  "Querida",
+  "Roman",
+  "Stella",
+  "Tobias",
+  "Una",
+  "Virgil",
+  "Willow",
+  "Xavier",
+  "Yolanda",
+  "Zephyr",
+  // Pass 4 — male-first (A=male), alternating to Z
+  "Aaron",
   "Beatrice",
   "Cyrus",
   "Delia",
-  "Ernesto",
+  "Elias",
   "Florence",
-  "Gordon",
+  "Galileo",
   "Hester",
-  "Isaias",
+  "Ian",
   "Jenna",
   "Klaus",
-  "Larry",
-  "Maxine",
-  "Nicholas",
+  "Linda",
+  "Marco",
+  "Nora",
   "Otto",
-  "Peter",
+  "Paulette",
   "Quill",
   "Renee",
-  "Stella",
+  "Sebastian",
   "Theresa",
   "Umberto",
-  "Virgil",
-  "Willow",
-  "Xavier",
-  "Yolanda",
-  "Zane",
-  // Pass 4
+  "Vera",
+  "Walter",
+  "Xena",
+  "Yves",
+  "Zoe",
+  // Pass 5 — female-first (A=female), alternating to Z
   "Anita",
-  "Bruno",
-  "Cora",
-  "Damon",
+  "Beau",
+  "Celeste",
+  "Dexter",
   "Edith",
-  "Felix",
+  "Fitz",
   "Gloria",
-  "Holly",
+  "Hadley",
   "Iris",
-  "Jasper",
-  "Kaia",
-  "Linda",
-  "Marco",
-  "Nora",
-  "Olive",
-  "Petra",
-  "Querida",
-  "Roman",
-  "Sebastian",
-  "Tobias",
-  "Una",
-  "Vera",
-  "Walter",
-  "Xena",
-  "Yusra",
-  "Zelda",
-  // Pass 5
-  "Aaron",
-  "Bonnie",
-  "Caleb",
-  "Daphne",
-  "Elias",
-  "Fiona",
-  "Galileo",
-  "Hugo",
-  "Ian",
   "Jude",
-  "Knox",
+  "Kira",
   "Leo",
   "Maya",
   "Nash",
-  "Owen",
-  "Paxton",
+  "Olga",
+  "Phillip",
   "Quetzal",
-  "Rosa",
+  "Royce",
   "Sage",
-  "Talia",
-  "Uri",
+  "Trent",
+  "Uma",
   "Vincent",
-  "Wesley",
-  "Ximena",
+  "Wynne",
+  "Xerxes",
   "Yvette",
-  "Zephyr",
-  // Pass 6
-  "Astrid",
-  "Beau",
-  "Celeste",
-  "Dexter",
+  "Zoltan",
+  // Pass 6 — male-first (A=male), alternating to Z
+  "Andre",
+  "Bonnie",
+  "Caleb",
+  "Daphne",
   "Ezra",
-  "Fitz",
-  "Genevieve",
-  "Hadley",
-  "Imogen",
-  "Joaquin",
-  "Kira",
+  "Fiona",
+  "Gibson",
+  "Holly",
+  "Ivan",
+  "Juno",
+  "Knox",
   "Lila",
   "Mason",
   "Nila",
-  "Olga",
-  "Pearl",
+  "Owen",
+  "Petra",
   "Quincy",
-  "Royce",
+  "Rosa",
   "Sterling",
-  "Trent",
-  "Ulysses",
+  "Talia",
+  "Uri",
   "Violet",
-  "Wynne",
-  "Xerxes",
-  "Yael",
-  "Zinnia",
-  // Pass 7
-  "Amelia",
-  "Bianca",
-  "Cody",
-  "Dahlia",
+  "Wesley",
+  "Ximena",
+  "Yuri",
+  "Zelda",
+  // Pass 7 — female-first (A=female), alternating to Z
+  "Astrid",
+  "Boris",
+  "Cassidy",
+  "Davis",
   "Esme",
-  "Fern",
-  "Greer",
-  "Harriet",
-  "Ivan",
-  "Juno",
-  "Kendrick",
+  "Forrest",
+  "Genevieve",
+  "Henry",
+  "Imogen",
+  "Joaquin",
+  "Kestrel",
   "Luther",
   "Margot",
   "Nigel",
-  "Orion",
-  "Phillip",
+  "Ophelia",
+  "Percy",
   "Quenby",
   "Reagan",
   "Sienna",
-  "Tessa",
-  "Uma",
-  "Vesper",
-  "Watson",
-  "Xiomara",
-  "Yuri",
-  "Zoltan",
-  // Pass 8
-  "Andre",
-  "Boris",
-  "Cassidy",
-  "Davis",
+  "Truman",
+  "Undine",
+  "Voss",
+  "Wren",
+  "Xan",
+  "Yael",
+  "Zia",
+  // Pass 8 — male-first (A=male), alternating to Z
+  "Arthur",
+  "Bianca",
+  "Cody",
+  "Dahlia",
   "Evander",
-  "Forrest",
-  "Gibson",
-  "Henry",
-  "Irene",
-  "Justus",
-  "Kasper",
+  "Fern",
+  "Gustavo",
+  "Harriet",
+  "Ira",
+  "Josephine",
+  "Kendrick",
   "Lainey",
   "Miles",
   "Naomi",
-  "Ophelia",
-  "Phoebe",
+  "Orion",
+  "Pearl",
   "Quark",
   "Rylie",
   "Saul",
-  "Truman",
-  "Ulrich",
-  "Voss",
-  "Winifred",
-  "Xuxa",
-  "Yvonne",
-  "Zia",
-  // Pass 9
-  "Aria",
-  "Brenda",
-  "Crispin",
-  "Dolores",
+  "Tessa",
+  "Ulysses",
+  "Vesper",
+  "Wyatt",
+  "Xiomara",
+  "Yancy",
+  "Zinnia",
+  // Pass 9 — female-first (A=female), alternating to Z
+  "Amelia",
+  "Barnaby",
+  "Calliope",
+  "Drake",
   "Estelle",
-  "Felicity",
-  "Goldie",
-  "Hazel",
-  "Ira",
-  "Jovi",
-  "Kestrel",
-  "Lyric",
+  "Foster",
+  "Greer",
+  "Hollis",
+  "Irene",
+  "Jericho",
+  "Klara",
+  "Logan",
   "Mavis",
   "Nico",
-  "Otis",
-  "Percy",
-  "Querubin",
+  "Oakley",
+  "Pascal",
+  "Quito",
   "Rhett",
   "Scout",
-  "Tatum",
+  "Theron",
   "Unity",
-  "Vivian",
-  "Wyatt",
-  "Xan",
-  "Yancy",
+  "Vance",
+  "Winifred",
+  "Xola",
+  "Yvonne",
   "Zev",
-  // Pass 10
+  // Pass 10 — male-first (A=male), alternating to Z
   "Atticus",
-  "Barnaby",
-  "Calliope",
-  "Drake",
+  "Brenda",
+  "Crispin",
+  "Dolores",
   "Eustace",
-  "Foster",
+  "Felicity",
   "Granger",
-  "Hollis",
-  "Indira",
-  "Jericho",
-  "Kaleb",
-  "Logan",
+  "Hazel",
+  "Idris",
+  "Jovi",
+  "Kasper",
+  "Lyric",
   "Magnus",
   "Noor",
-  "Oakley",
-  "Posy",
-  "Quito",
+  "Otis",
+  "Phoebe",
+  "Querubin",
   "Rosalind",
-  "Sloane",
-  "Theron",
+  "Silas",
+  "Tatum",
   "Upton",
-  "Vance",
+  "Vivian",
   "Wendell",
-  "Xola",
+  "Xuxa",
   "Yann",
   "Zora",
 ] as const;

package/src/core/agents/state/stale-sweep.ts

@@ -13,6 +13,7 @@
 import { existsSync, readdirSync, readFileSync, statSync, unlinkSync } from "node:fs";
 import { join } from "node:path";
 import { coordFreshnessSeconds } from "../../config.ts";
+import { writeProducerDiagnosticV3 } from "../../events/v3/producers/intake.ts";
 import { recordLiveSweepObservationV3 } from "../live-lifecycle-v3.ts";
 
 /** platform → adapter, for the swept-event envelope (mirrors heartbeat-writer's adapterOf). */
@@ -22,7 +23,15 @@ function adapterFromPlatform(platform: unknown): "claude-code" | "cursor" | "cod
   return "claude-code";
 }
 
-/** Record the sweep before deleting authority state. V3 ambiguity fails closed. */
+/**
+ * Record the sweep before deleting disposable cache state.
+ *
+ * A joinable generation owns the canonical provisional observation. Once that
+ * generation is terminal or unavailable, it must not be reopened merely to
+ * authorize cache housekeeping. Preserve the failed observation in the V3
+ * diagnostics spool instead. If neither durable record can be written, fail
+ * closed and keep the file.
+ */
 function emitSwept(
   coordRoot: string,
   instanceId: string,
@@ -46,8 +55,18 @@ function emitSwept(
       ageMs: Math.max(0, (ageSecs ?? 0) * 1_000),
     });
     return true;
-  } catch {
-    return false;
+  } catch (error) {
+    return Boolean(
+      writeProducerDiagnosticV3(coordRoot, "heartbeat_sweep_unrecorded", {
+        adapter,
+        instance_id: instanceId,
+        signal: "lifecycle.sweep_observed",
+        reason: error instanceof Error ? error.message : String(error),
+        state: "cache_cleanup_pending",
+        observation: reason,
+        age_ms: Math.max(0, (ageSecs ?? 0) * 1_000),
+      }),
+    );
   }
 }
 
@@ -85,8 +104,9 @@ export function staleSweep(coordRoot: string): {
   //     a transient (mid-write / partial read), and deleting it would nuke a
   //     LIVE agent's heartbeat, the worst possible outcome. So: never reap a
   //     fresh file on a content failure.
-  // Every deletion now emits health.heartbeat_swept so the lifecycle is
-  // auditable (sweeps used to be silent; see the swept-event schema doc).
+  // Every deletion leaves durable audit evidence. A joinable generation gets
+  // lifecycle.sweep_observed; a terminal or unavailable generation gets a
+  // heartbeat_sweep_unrecorded producer diagnostic without being resurrected.
   const liveInstanceIds = new Set<string>();
   const d = join(coordRoot, ".harnery", "active");
   if (existsSync(d)) {

package/src/core/artifacts/constants.ts

@@ -0,0 +1,4 @@
+/** Read-only artifact layout constants safe for browser-side server modules. */
+export const ARTIFACTS_DIR = ".harnery/artifacts";
+export const ARTIFACT_MANIFEST = ".harnery-artifact.json";
+export const ARTIFACT_SCHEMA_VERSION = 1 as const;

package/src/core/artifacts/index.ts

@@ -24,10 +24,9 @@ import {
 import { basename, dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
 import { readLiveCoordinationRow } from "../agents/state/live-coordination-view.ts";
 import { artifactDefaultRetentionDays } from "../config.ts";
+import { ARTIFACT_MANIFEST, ARTIFACT_SCHEMA_VERSION, ARTIFACTS_DIR } from "./constants.ts";
 
-export const ARTIFACTS_DIR = join(".harnery", "artifacts");
-export const ARTIFACT_MANIFEST = ".harnery-artifact.json";
-export const ARTIFACT_SCHEMA_VERSION = 1 as const;
+export { ARTIFACT_MANIFEST, ARTIFACT_SCHEMA_VERSION, ARTIFACTS_DIR } from "./constants.ts";
 
 export interface ArtifactActor {
   instance_id: string;
@@ -72,6 +71,7 @@ export interface ArtifactInventoryEntry {
   artifact_id: string | null;
   slug: string | null;
   created_at: string | null;
+  last_modified_at: string | null;
   expires_at: string | null;
   owner_instance_id: string | null;
 }
@@ -328,16 +328,22 @@ function classifyArtifactPath(
   }
   const manifest = parsed.manifest;
   const bytes = safeTreeSize(path);
+  const lastModifiedMs = safeTreeLastModified(path, now);
+  const retentionAnchorMs = Date.parse(manifest.retention.renewed_at ?? manifest.created_at);
+  const retentionWindowMs = Date.parse(manifest.retention.expires_at) - retentionAnchorMs;
+  const effectiveLastModifiedMs = Math.max(retentionAnchorMs, lastModifiedMs ?? 0);
+  const effectiveExpiresAt = new Date(effectiveLastModifiedMs + retentionWindowMs).toISOString();
   const base = rowFor(path, name, repoRoot, "managed-current", "retention has not expired", bytes);
   Object.assign(base, {
     artifact_id: manifest.artifact_id,
     slug: manifest.slug,
     created_at: manifest.created_at,
-    expires_at: manifest.retention.expires_at,
+    last_modified_at: new Date(effectiveLastModifiedMs).toISOString(),
+    expires_at: effectiveExpiresAt,
     owner_instance_id: manifest.created_by?.instance_id ?? null,
   });
 
-  if (base.bytes === null) {
+  if (base.bytes === null || lastModifiedMs === null) {
     return {
       ...base,
       classification: "unknown",
@@ -372,11 +378,11 @@ function classifyArtifactPath(
       };
     }
   }
-  if (Date.parse(manifest.retention.expires_at) > now.getTime()) return base;
+  if (Date.parse(effectiveExpiresAt) > now.getTime()) return base;
   return {
     ...base,
     classification: "managed-expired",
-    reason: `retention expired at ${manifest.retention.expires_at}`,
+    reason: `retention expired at ${effectiveExpiresAt}`,
     action: "would-delete",
   };
 }
@@ -410,6 +416,13 @@ function readManifest(path: string): ParsedManifest | ManifestError {
   if (!m.retention || typeof m.retention !== "object" || !validIso(m.retention.expires_at)) {
     return { ok: false, reason: "invalid retention.expires_at" };
   }
+  if (m.retention.renewed_at !== undefined && !validIso(m.retention.renewed_at)) {
+    return { ok: false, reason: "invalid retention.renewed_at" };
+  }
+  const retentionAnchor = Date.parse(m.retention.renewed_at ?? m.created_at);
+  if (Date.parse(m.retention.expires_at) <= retentionAnchor) {
+    return { ok: false, reason: "retention.expires_at must follow its retention anchor" };
+  }
   if (m.released_at !== undefined && !validIso(m.released_at)) {
     return { ok: false, reason: "invalid released_at" };
   }
@@ -478,6 +491,34 @@ function safeTreeSize(path: string): number | null {
   }
 }
 
+/**
+ * Return the newest filesystem change in a managed tree without following
+ * symlinks. A small future tolerance protects a write racing the inventory
+ * scan; timestamps farther ahead are ignored as clock-skewed metadata.
+ */
+function safeTreeLastModified(path: string, now: Date): number | null {
+  const nowMs = now.getTime();
+  const futureToleranceMs = 5 * 60 * 1000;
+  try {
+    const st = lstatSync(path);
+    let latest = 0;
+    for (const timestamp of [st.mtimeMs, st.ctimeMs]) {
+      if (Number.isFinite(timestamp) && timestamp <= nowMs + futureToleranceMs) {
+        latest = Math.max(latest, Math.min(timestamp, nowMs));
+      }
+    }
+    if (st.isSymbolicLink() || !st.isDirectory()) return latest;
+    for (const child of readdirSync(path)) {
+      const childLatest = safeTreeLastModified(join(path, child), now);
+      if (childLatest === null) return null;
+      latest = Math.max(latest, childLatest);
+    }
+    return latest;
+  } catch {
+    return null;
+  }
+}
+
 function rowFor(
   path: string,
   name: string,
@@ -497,6 +538,7 @@ function rowFor(
     artifact_id: null,
     slug: null,
     created_at: null,
+    last_modified_at: null,
     expires_at: null,
     owner_instance_id: null,
   };