opencode-swarm 7.166.1 → 7.166.2

package/dist/observability/catalog.d.ts

@@ -2,7 +2,7 @@
  * The event catalog (issue #2029).
  *
  * Exactly 61 entries, matching the `TelemetryEvent` union at
- * `src/telemetry.ts:16-172`. Thirty-eight predate the #2029 contract; the 39th is
+ * `src/telemetry.ts:20-172`. Thirty-eight predate the #2029 contract; the 39th is
  * `agent_conflict_detected` (previously emitted through a force-cast past the
  * type system), the 40th is `close_archive_result` (issue #2030 — the
  * structured close/archive result event), the 41st is the bounded diagnostic

package/dist/session/snapshot-writer.d.ts

@@ -104,6 +104,48 @@ export interface SerializedAgentSession {
     /** Provider id paired with lastObservedModel (#1896). Omitted when undefined. */
     lastObservedProviderID?: string;
 }
+/**
+ * Compile-time snapshot-field parity map (issue #2472 W6 / AC-7; frozen check C7).
+ *
+ * WHY THIS MAP EXISTS — the serializer reads FROM `AgentSessionState` and
+ * writes TO `SerializedAgentSession`, so the drift-prone direction is a field
+ * that exists on the live state but silently never reaches the snapshot:
+ * nothing fails, the field is just lost on every restart. The
+ * `Record<Exclude<keyof AgentSessionState, keyof SerializedAgentSession>, string>`
+ * key type makes that direction exhaustive at compile time — EVERY state field
+ * that is not a serialized field MUST appear here with a one-line rationale,
+ * and a key that is NOT such a field is an excess-property type error. Adding
+ * a field to either interface without reconciling this map fails
+ * `bun run typecheck` (CI gate).
+ *
+ * The serialized-only direction (a `SerializedAgentSession` field with no
+ * `AgentSessionState` counterpart) is guarded separately by the
+ * `_serializedFieldsExistOnState` assertion below. The asymmetry is
+ * deliberate: the serializer can only *drop* state fields, while the reader
+ * could otherwise *invent* fields the live state does not have.
+ *
+ * This is NOT the same list as `TRANSIENT_SESSION_FIELDS` in
+ * `src/session/snapshot-reader.ts` (fields RESET on rehydration — a subset of
+ * the *serialized* fields). This map lists fields that are NEVER serialized.
+ * Most are genuinely process-local; `workspaceDirectory` is deliberately
+ * unserializable for trust reasons (see its TRUST BOUNDARY comment in
+ * `src/state.ts`), which is exactly why it must carry an entry here rather
+ * than fall out of the type silently.
+ *
+ * The runtime mirror of this guard lives at
+ * `tests/unit/session/snapshot-field-parity-guard.test.ts` (what frozen
+ * check C7 runs).
+ */
+export declare const SESSION_TRANSIENT_FIELDS: Readonly<Record<Exclude<keyof AgentSessionState, keyof SerializedAgentSession>, string>>;
+/**
+ * Reverse-direction parity guard: every `SerializedAgentSession` field must
+ * exist on `AgentSessionState`. The const assignment is what makes tsc
+ * actually enforce the conditional type — when a serialized field has no
+ * state counterpart, the type degrades to the error tuple below and assigning
+ * `true` fails `bun run typecheck` with that message.
+ */
+export type SerializedFieldsAllExistOnState = Exclude<keyof SerializedAgentSession, keyof AgentSessionState> extends never ? true : ['SerializedAgentSession field missing from AgentSessionState'];
+export declare const _serializedFieldsExistOnState: SerializedFieldsAllExistOnState;
 /**
  * Minimal interface for serialized InvocationWindow
  */

package/dist/telemetry.d.ts

@@ -66,7 +66,37 @@ export declare function resetTelemetryForTesting(): void;
 /**
  * Initialize telemetry with the project directory.
  * Creates `.swarm/` if it doesn't exist and opens `telemetry.jsonl` for appending.
- * Idempotent — calling multiple times has no effect after the first successful call.
+ * Idempotent — calling multiple times with the SAME directory has no effect
+ * after the first successful call.
+ *
+ * Re-home (issue #2472 W9): when called with a projectDirectory DIFFERENT
+ * from the latched `_projectDirectory` while a stream exists (a second
+ * server() instance initializing in this process for another project root),
+ * telemetry re-initializes for the new directory exactly as a fresh init
+ * does — the previous behavior latched the first directory forever, so the
+ * second project's `.swarm/telemetry.jsonl` was never created. A disabled
+ * telemetry state stays disabled.
+ *
+ * No-drop ordering + fail-open ownership retention (PR #2588, review finding
+ * 6 / PRR-019): the NEW stream is created and published FIRST, and only then
+ * is the old stream flushed/closed best-effort. The previous
+ * close-then-create order nulled `_writeStream` before the replacement
+ * existed, so any emit racing the re-home was dropped by the
+ * `_writeStream === null` guard, and a new-stream creation FAILURE took the
+ * whole process's telemetry down (`_disabled = true`) after the old stream
+ * was already gone. On creation failure the OLD stream is now retained
+ * (logged non-fatally, no re-home) — events keep flowing to the previous
+ * directory until a later successful init.
+ *
+ * OWNERSHIP CONTRACT (last-init-wins): the newest successful initTelemetry
+ * call owns THE single live stream for the whole process. In a
+ * multi-project process, a project initialized earlier SHARES the latest
+ * stream until it re-initializes; per-event project routing is out of scope
+ * — the issue #2472 obligation is directory-CHANGE detection, not per-event
+ * fan-out. Same-directory detection uses canonical project-root identity
+ * (`sameProjectRoot`), so Windows case differences and symlink/junction
+ * spellings of one root do not churn the stream.
+ *
  * @param projectDirectory - Absolute path to the project root
  */
 export declare function initTelemetry(projectDirectory: string): void;

package/dist/utils/atomic-write.d.ts

@@ -128,6 +128,38 @@ export declare function assertWellFormedTargetPath(targetPath: string): void;
 export declare function assertSwarmContainedTarget(targetPath: string): SwarmContainment;
 /** Hard bound on a single atomic write (bounded-write requirement). */
 export declare const MAX_ATOMIC_WRITE_BYTES: number;
+/**
+ * Default window within which an ENOENT read of a target this process wrote
+ * (or is writing) is treated as plausibly mid-rename rather than never-existed
+ * — see `wasRecentlyWrittenByThisProcess` and the three-way ENOENT policy in
+ * `readSwarmFileAsync` (src/hooks/utils.ts).
+ */
+export declare const RECENT_WRITE_WINDOW_MS = 500;
+/**
+ * Remove the recent-write marker for `targetPath` (PRR-008). Called from the
+ * canonical writer's failure path: a write that throws before its rename
+ * completes must not leave a stale INTENT stamp behind, or a subsequent ENOENT
+ * read of the target would be misclassified as a rename race and retried
+ * against a file this process is no longer producing. Successful writes are
+ * unaffected — they reach the WRITE COMPLETION refresh, which re-stamps the
+ * marker for the post-rename visibility window. Returns true when a stamp was
+ * present and removed.
+ */
+export declare function clearRecentWriteActivity(targetPath: string): boolean;
+/**
+ * Whether this process recorded a write INTENT or write COMPLETION for
+ * `targetPath` within the last `withinMs` milliseconds (default
+ * {@link RECENT_WRITE_WINDOW_MS}).
+ *
+ * Consumers: the ENOENT retry ladders (`readSwarmFileAsync`,
+ * `readFileOrEmpty` in src/services/context-budget-service.ts) use this as
+ * the "evidenced rename-race" arm — an ENOENT on a file THIS process just
+ * wrote (or is mid-write on: intent is recorded before the temp file write
+ * begins) retries, while an ENOENT with no evidence the file ever existed
+ * returns immediately (issue #2472 PERF-1: 4×10ms of sleeps per missing-file
+ * read, ~1.9s/turn on plan-less projects).
+ */
+export declare function wasRecentlyWrittenByThisProcess(targetPath: string, withinMs?: number): boolean;
 export interface AtomicWriteOptions {
     /** Skip the pre-rename fsync (tests / truly ephemeral data). */
     readonly skipFsync?: boolean;

package/dist/utils/swarm-artifact-cache.d.ts

@@ -15,6 +15,12 @@ export interface SwarmArtifactCacheStats {
     textEntryCount: number;
     parsedEntryCount: number;
 }
+/**
+ * Whether a previous successful stat/read inside this module observed
+ * `filePath` to exist (resolved the same way the cache keys are). Cheap Set
+ * lookup — safe on hot paths.
+ */
+export declare function wasObservedToExist(filePath: string): boolean;
 /**
  * Dependency-injection seam for testing. Tests can temporarily replace these
  * to force a specific (e.g. colliding) stat stamp without relying on

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "opencode-swarm",
-	"version": "7.166.1",
+	"version": "7.166.2",
 	"description": "Architect-centric agentic swarm plugin for OpenCode - hub-and-spoke orchestration with SME consultation, code generation, and QA review",
 	"main": "dist/index.js",
 	"types": "dist/index.d.ts",

package/dist/cli/curator-llm-factory-da5q3gg6.js

@@ -1,70 +0,0 @@
-// @bun
-import {
-  createCuratorLLMDelegate
-} from "./index-399nq780.js";
-import"./index-2sqgbmtq.js";
-import"./index-hvk3zs62.js";
-import"./index-gr8ppbq3.js";
-import"./index-8fr2wn18.js";
-import"./index-0g5v4s0x.js";
-import"./index-17yszqs5.js";
-import"./index-txdagbh7.js";
-import"./index-cw912re7.js";
-import"./index-066xxa4q.js";
-import"./index-13egyjgv.js";
-import"./index-ybcxvv17.js";
-import"./index-70ha9e09.js";
-import"./index-8hqkrd2j.js";
-import"./index-9ss2m4rs.js";
-import"./index-xezfx7ge.js";
-import"./index-zsrtt42s.js";
-import"./index-k5w284js.js";
-import"./index-6chhqq2n.js";
-import"./index-1zqf2pny.js";
-import"./index-b43rt5ph.js";
-import"./index-q7sjgr1y.js";
-import"./index-yt25kpvz.js";
-import"./index-fc1dqzjp.js";
-import"./index-w11whwn3.js";
-import"./index-zha9bdj9.js";
-import"./index-bfr392e7.js";
-import"./index-ee6ydtfe.js";
-import"./index-p612v1mv.js";
-import"./index-6rff2h8x.js";
-import"./index-c20gf089.js";
-import"./index-rtry5xyf.js";
-import"./index-nph5g4em.js";
-import"./index-3qf1vxtb.js";
-import"./index-n832052r.js";
-import"./index-xwkn5hse.js";
-import"./index-2y4kargd.js";
-import"./index-2v17wb75.js";
-import"./index-79sr3fd1.js";
-import"./index-jx9f4ke1.js";
-import"./index-aftapk6j.js";
-import"./index-ry16d01s.js";
-import"./index-n4fzmqz8.js";
-import"./index-axf65x87.js";
-import"./index-bkd3450m.js";
-import"./index-ae75rja9.js";
-import"./index-a7rw4g4s.js";
-import"./index-7qefgq4j.js";
-import"./index-edm41q4j.js";
-import"./index-gy0nyfbw.js";
-import"./index-hwj25q9r.js";
-import"./index-7g4c7s5r.js";
-import"./index-52xdezsh.js";
-import"./index-9e6fdz2h.js";
-import"./index-4rhhvd1a.js";
-import"./index-wf12wckd.js";
-import"./index-0tc0pcmh.js";
-import"./index-zpsr8rpc.js";
-import"./index-bk5tah7q.js";
-import"./index-bpmtbmy9.js";
-import"./index-q1r1cfww.js";
-import"./index-yzjaj53h.js";
-import"./index-zgwm4ryv.js";
-import"./index-a76rekgs.js";
-export {
-  createCuratorLLMDelegate
-};

package/dist/cli/guardrail-explain-eva1yrtf.js

@@ -1,71 +0,0 @@
-// @bun
-import {
-  handleGuardrailExplain
-} from "./index-3tyym20r.js";
-import"./index-399nq780.js";
-import"./index-2sqgbmtq.js";
-import"./index-hvk3zs62.js";
-import"./index-gr8ppbq3.js";
-import"./index-8fr2wn18.js";
-import"./index-0g5v4s0x.js";
-import"./index-17yszqs5.js";
-import"./index-txdagbh7.js";
-import"./index-cw912re7.js";
-import"./index-066xxa4q.js";
-import"./index-13egyjgv.js";
-import"./index-ybcxvv17.js";
-import"./index-70ha9e09.js";
-import"./index-8hqkrd2j.js";
-import"./index-9ss2m4rs.js";
-import"./index-xezfx7ge.js";
-import"./index-zsrtt42s.js";
-import"./index-k5w284js.js";
-import"./index-6chhqq2n.js";
-import"./index-1zqf2pny.js";
-import"./index-b43rt5ph.js";
-import"./index-q7sjgr1y.js";
-import"./index-yt25kpvz.js";
-import"./index-fc1dqzjp.js";
-import"./index-w11whwn3.js";
-import"./index-zha9bdj9.js";
-import"./index-bfr392e7.js";
-import"./index-ee6ydtfe.js";
-import"./index-p612v1mv.js";
-import"./index-6rff2h8x.js";
-import"./index-c20gf089.js";
-import"./index-rtry5xyf.js";
-import"./index-nph5g4em.js";
-import"./index-3qf1vxtb.js";
-import"./index-n832052r.js";
-import"./index-xwkn5hse.js";
-import"./index-2y4kargd.js";
-import"./index-2v17wb75.js";
-import"./index-79sr3fd1.js";
-import"./index-jx9f4ke1.js";
-import"./index-aftapk6j.js";
-import"./index-ry16d01s.js";
-import"./index-n4fzmqz8.js";
-import"./index-axf65x87.js";
-import"./index-bkd3450m.js";
-import"./index-ae75rja9.js";
-import"./index-a7rw4g4s.js";
-import"./index-7qefgq4j.js";
-import"./index-edm41q4j.js";
-import"./index-gy0nyfbw.js";
-import"./index-hwj25q9r.js";
-import"./index-7g4c7s5r.js";
-import"./index-52xdezsh.js";
-import"./index-9e6fdz2h.js";
-import"./index-4rhhvd1a.js";
-import"./index-wf12wckd.js";
-import"./index-0tc0pcmh.js";
-import"./index-zpsr8rpc.js";
-import"./index-bk5tah7q.js";
-import"./index-bpmtbmy9.js";
-import"./index-q1r1cfww.js";
-import"./index-yzjaj53h.js";
-import"./index-zgwm4ryv.js";
-import"./index-a76rekgs.js";
-export {
-  handleGuardrailExplain
-};

package/dist/cli/{index-2sqgbmtq.js → index-d5qf7hvh.js}

@@ -1,10 +1,10 @@
 // @bun
-import {
-  simpleGlobToRegex
-} from "./index-ee6ydtfe.js";
 import {
   bunSpawnSync
 } from "./index-p612v1mv.js";
+import {
+  simpleGlobToRegex
+} from "./index-ee6ydtfe.js";
 import {
   warn
 } from "./index-zgwm4ryv.js";