@remnic/core 9.69.56 → 9.69.57

package/dist/coding/export-okf-codegraph.js

@@ -24,18 +24,18 @@ import "../chunk-FEHIOYML.js";
 import "../chunk-P32IDSHD.js";
 import {
   StorageManager
-} from "../chunk-QWQB7TOB.js";
-import "../chunk-UHVXE24Y.js";
+} from "../chunk-S2OBXHMN.js";
 import "../chunk-46WM6JX2.js";
+import "../chunk-UHVXE24Y.js";
 import "../chunk-D5F7TKMH.js";
-import "../chunk-AIVRD66I.js";
-import "../chunk-PLJ2MQFT.js";
-import "../chunk-KVO2BTA4.js";
-import "../chunk-CYEPCZN5.js";
-import "../chunk-G6NQ3L4T.js";
+import "../chunk-W6YK672C.js";
+import "../chunk-IAZPYX6W.js";
 import "../chunk-WPGR3RW4.js";
 import "../chunk-6KTXE6A3.js";
 import "../chunk-D7MAYFQP.js";
+import "../chunk-KVO2BTA4.js";
+import "../chunk-CYEPCZN5.js";
+import "../chunk-G6NQ3L4T.js";
 import "../chunk-4O6WRBGE.js";
 import "../chunk-GUMZC2BO.js";
 import "../chunk-TFA6HPOA.js";
@@ -43,15 +43,15 @@ import "../chunk-F4TWWUMP.js";
 import "../chunk-72A4L5JJ.js";
 import "../chunk-A7Q43EUZ.js";
 import "../chunk-S2V7RUCL.js";
-import "../chunk-4UVJ6EIT.js";
 import "../chunk-MRPOZBJC.js";
 import "../chunk-S2HLHAZ4.js";
 import "../chunk-DM2T26WE.js";
 import "../chunk-LDXUBPMO.js";
-import "../chunk-S45INSV3.js";
+import "../chunk-FF6N7BXC.js";
 import "../chunk-J3IWHCMF.js";
 import "../chunk-AAD2ZS6P.js";
 import "../chunk-DUJYRVUD.js";
+import "../chunk-YXZG5TVN.js";
 import "../chunk-ZIKENA2C.js";
 import {
   OKF_ARCHITECTURE_CARD_TYPE,
@@ -60,7 +60,6 @@ import {
 } from "../chunk-PCEYGOSL.js";
 import "../chunk-SCU65EZI.js";
 import "../chunk-3ONXXHQO.js";
-import "../chunk-YXZG5TVN.js";
 import "../chunk-5GINB5LC.js";
 import "../chunk-3WU7XOFN.js";
 import "../chunk-G7D6GZ5J.js";
@@ -84,12 +83,11 @@ import "../chunk-45GF7Y3V.js";
 import "../chunk-EP6BA7VV.js";
 import "../chunk-E2AWJ7KQ.js";
 import "../chunk-YCVB7X56.js";
+import "../chunk-PQVCH2AE.js";
 import "../chunk-PJKTFXE2.js";
 import "../chunk-W5BIQOAJ.js";
 import "../chunk-Z7L72ZHL.js";
 import "../chunk-4O4TKIXQ.js";
-import "../chunk-RGMVMVMF.js";
-import "../chunk-PQVCH2AE.js";
 import "../chunk-GDYU6VF4.js";
 import "../chunk-BT7NVCML.js";
 import "../chunk-VYCOJKSX.js";
@@ -104,11 +102,13 @@ import "../chunk-AL3UYZ5N.js";
 import {
   openBetterSqlite3
 } from "../chunk-RL6NEVXT.js";
-import "../chunk-PWPIERBM.js";
 import "../chunk-C7HJ3NEI.js";
 import "../chunk-NB3LRY4T.js";
 import "../chunk-5FVVY2EB.js";
 import "../chunk-FFAZ2XFU.js";
+import "../chunk-FNQEXI3D.js";
+import "../chunk-RGMVMVMF.js";
+import "../chunk-PWPIERBM.js";
 import {
   stripAttributesSuffix
 } from "../chunk-JDSVLVCG.js";

package/dist/coding/git-context.d.ts

@@ -0,0 +1,92 @@
+import { expandTildePath } from "../utils/path.js";
+export { expandTildePath };
+export interface GitContext {
+    /**
+     * Stable identifier for the project. Derived from `git remote get-url origin`
+     * when an origin remote is configured, otherwise from the repo root path.
+     *
+     * Formatted as `origin:<hex>` or `root:<hex>` so that the source is visible
+     * to operators (see `remnic doctor`, issue #569 acceptance criteria).
+     */
+    projectId: string;
+    /**
+     * Current branch, e.g. `main`, `feat/foo`. `null` only in detached-HEAD
+     * state (e.g. rebase in progress). Callers should treat `null` as "no
+     * branch-scope overlay applies" without erroring.
+     */
+    branch: string | null;
+    /**
+     * Absolute path to the repository root (the directory containing `.git`).
+     * Tilde-expanded per CLAUDE.md #17.
+     */
+    rootPath: string;
+    /**
+     * Best-effort default branch (usually `main` or `master`). Derived from the
+     * `refs/remotes/origin/HEAD` symbolic ref. `null` when not available (e.g.
+     * fresh clone without a default branch symref, or no origin remote).
+     */
+    defaultBranch: string | null;
+}
+export interface GitInvokeOptions {
+    abortSignal?: AbortSignal;
+    timeoutMs?: number;
+}
+export interface GitInvokeResult {
+    stdout: string;
+    exitCode: number;
+}
+/**
+ * Run `git <args>` with `cwd` as the working directory. Implementations
+ * should return non-zero exit codes rather than throwing for command failure.
+ */
+export interface GitInvoker {
+    (cwd: string, args: string[], options?: GitInvokeOptions): GitInvokeResult | Promise<GitInvokeResult>;
+}
+export interface SyncGitInvoker {
+    (cwd: string, args: string[]): GitInvokeResult;
+}
+export declare function defaultGitInvokerSync(): SyncGitInvoker;
+export declare function defaultGitInvoker(): GitInvoker;
+/**
+ * Non-cryptographic stable hash. Used only to derive a deterministic
+ * `projectId` from either the origin URL or the root path. The hash does not
+ * need to be collision-resistant against adversarial input — it is purely a
+ * namespace discriminator.
+ *
+ * Uses FNV-1a 32-bit so we don't pull in `node:crypto` for a simple bucket
+ * key. Output is lowercase hex, zero-padded to 8 characters.
+ */
+export declare function stableHash(input: string): string;
+/**
+ * Normalize a git remote URL so that equivalent SSH / HTTPS forms of the
+ * same repo produce the same `projectId`. Handles:
+ *   - `git@github.com:foo/bar.git`  → `github.com/foo/bar`
+ *   - `https://github.com/foo/bar`  → `github.com/foo/bar`
+ *   - `https://github.com/foo/bar.git` → `github.com/foo/bar`
+ *   - `ssh://git@github.com/foo/bar` → `github.com/foo/bar`
+ *   - `ssh://git@github.com:2222/foo/bar` → `github.com/foo/bar` (port stripped)
+ *
+ * Case-insensitive (remote hostnames and most repo paths on major forges are
+ * case-insensitive in practice).
+ */
+export declare function normalizeOriginUrl(rawUrl: string): string;
+export interface ResolveGitContextOptions {
+    /** Inject a git invoker (tests). Defaults to spawning real `git`. */
+    invoker?: GitInvoker;
+    abortSignal?: AbortSignal;
+    /** Absolute epoch deadline shared with the extraction lifecycle. */
+    deadlineMs?: number;
+    /** Injectable clock for deadline tests. */
+    now?: () => number;
+}
+/**
+ * Detect the git project + branch for `cwd`.
+ *
+ * Returns `null` when `cwd` is invalid, outside a worktree, or git is
+ * unavailable. Unexpected command failures also resolve to `null`.
+ *
+ * Caller cancellation and an exhausted deadline reject so lifecycle callers
+ * can stop promptly instead of treating an interrupted probe as "not a repo".
+ */
+export declare function resolveGitContext(cwd: string, options?: ResolveGitContextOptions): Promise<GitContext | null>;
+//# sourceMappingURL=git-context.d.ts.map

package/dist/coding/git-context.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"git-context.d.ts","sourceRoot":"","sources":["../../src/coding/git-context.ts"],"names":[],"mappings":"AA2BA,OAAO,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAC;AAMnD,OAAO,EAAE,eAAe,EAAE,CAAC;AAM3B,MAAM,WAAW,UAAU;IACzB;;;;;;OAMG;IACH,SAAS,EAAE,MAAM,CAAC;IAClB;;;;OAIG;IACH,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB;;;OAGG;IACH,QAAQ,EAAE,MAAM,CAAC;IACjB;;;;OAIG;IACH,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B;AAED,MAAM,WAAW,gBAAgB;IAC/B,WAAW,CAAC,EAAE,WAAW,CAAC;IAC1B,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED;;;GAGG;AACH,MAAM,WAAW,UAAU;IACzB,CACE,GAAG,EAAE,MAAM,EACX,IAAI,EAAE,MAAM,EAAE,EACd,OAAO,CAAC,EAAE,gBAAgB,GACzB,eAAe,GAAG,OAAO,CAAC,eAAe,CAAC,CAAC;CAC/C;AAED,MAAM,WAAW,cAAc;IAC7B,CAAC,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,GAAG,eAAe,CAAC;CAChD;AAQD,wBAAgB,qBAAqB,IAAI,cAAc,CActD;AAED,wBAAgB,iBAAiB,IAAI,UAAU,CAiE9C;AAMD;;;;;;;;GAQG;AACH,wBAAgB,UAAU,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAOhD;AAMD;;;;;;;;;;;GAWG;AACH,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAwFzD;AAMD,MAAM,WAAW,wBAAwB;IACvC,qEAAqE;IACrE,OAAO,CAAC,EAAE,UAAU,CAAC;IACrB,WAAW,CAAC,EAAE,WAAW,CAAC;IAC1B,oEAAoE;IACpE,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,2CAA2C;IAC3C,GAAG,CAAC,EAAE,MAAM,MAAM,CAAC;CACpB;AAED;;;;;;;;GAQG;AACH,wBAAsB,iBAAiB,CACrC,GAAG,EAAE,MAAM,EACX,OAAO,GAAE,wBAA6B,GACrC,OAAO,CAAC,UAAU,GAAG,IAAI,CAAC,CAqG5B"}

package/dist/coding/optional-coding-graph.d.ts

@@ -1,6 +1,4 @@
-import { CreateCodingGraphEngineOptions, CodingGraphEngine, CodingGraphLanguage, CodingGraphErrorCode } from './coding-graph-types.js';
-export { CODING_GRAPH_ENGINE_VERSION, FileIR, ParseFileInput, ParseResult, SymbolIR, TIER_1_LANGUAGES } from './coding-graph-types.js';
-
+import { CODING_GRAPH_ENGINE_VERSION, TIER_1_LANGUAGES, type CodingGraphEngine, type CodingGraphErrorCode, type CodingGraphLanguage, type CreateCodingGraphEngineOptions, type FileIR, type ParseFileInput, type ParseResult, type SymbolIR } from "./coding-graph-types.js";
 /**
  * Structural minimal shape of `@remnic/coding-graph`'s public surface,
  * as seen by core after a successful dynamic import. We narrow to
@@ -17,7 +15,7 @@ interface LoadedCodingGraphModule {
  * Build the user-facing install hint message. Exported so tests can assert
  * on the exact text (CLAUDE.md rule 51 / issue #1551 prove-fail-before).
  */
-declare function buildCodingGraphInstallHint(): string;
+export declare function buildCodingGraphInstallHint(): string;
 /**
  * Return true when `err` is a module-not-found failure for exactly the
  * `@remnic/coding-graph` specifier. Same boundary-aware regex as
@@ -28,7 +26,7 @@ declare function buildCodingGraphInstallHint(): string;
  * Exported for tests; the production loaders call the internal alias
  * below.
  */
-declare function isSpecifierNotFoundErrorForCodingGraph(err: unknown, specifier?: string): boolean;
+export declare function isSpecifierNotFoundErrorForCodingGraph(err: unknown, specifier?: string): boolean;
 /**
  * Load `@remnic/coding-graph` if installed and return its
  * `createCodingGraphEngine` factory. Throws a user-facing diagnostic
@@ -42,13 +40,13 @@ declare function isSpecifierNotFoundErrorForCodingGraph(err: unknown, specifier?
  * and shared with the try* helpers; a failure never poisons the cache
  * so each call attempts a fresh import.
  */
-declare function loadCodingGraphEngineFactory(): Promise<(options?: CreateCodingGraphEngineOptions) => CodingGraphEngine>;
+export declare function loadCodingGraphEngineFactory(): Promise<(options?: CreateCodingGraphEngineOptions) => CodingGraphEngine>;
 /**
  * Return `true` only when `@remnic/coding-graph` is installed AND
  * importable in a usable shape. Returns `false` for missing, broken,
  * or incompatible installs. Never throws.
  */
-declare function isCodingGraphInstalled(): Promise<boolean>;
+export declare function isCodingGraphInstalled(): Promise<boolean>;
 /**
  * Return the engine factory module if `@remnic/coding-graph` is
  * installed AND importable in a usable shape, or `null` otherwise.
@@ -58,6 +56,7 @@ declare function isCodingGraphInstalled(): Promise<boolean>;
  * import attempt, then caches the successful result so subsequent
  * loadFactory calls skip the import.
  */
-declare function tryLoadCodingGraphModule(): Promise<LoadedCodingGraphModule | null>;
-
-export { CodingGraphEngine, CodingGraphErrorCode, CodingGraphLanguage, CreateCodingGraphEngineOptions, buildCodingGraphInstallHint, isCodingGraphInstalled, isSpecifierNotFoundErrorForCodingGraph, loadCodingGraphEngineFactory, tryLoadCodingGraphModule };
+export declare function tryLoadCodingGraphModule(): Promise<LoadedCodingGraphModule | null>;
+export { CODING_GRAPH_ENGINE_VERSION, TIER_1_LANGUAGES, };
+export type { CodingGraphEngine, CodingGraphErrorCode, CodingGraphLanguage, CreateCodingGraphEngineOptions, FileIR, ParseFileInput, ParseResult, SymbolIR, };
+//# sourceMappingURL=optional-coding-graph.d.ts.map

package/dist/coding/optional-coding-graph.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"optional-coding-graph.d.ts","sourceRoot":"","sources":["../../src/coding/optional-coding-graph.ts"],"names":[],"mappings":"AA0DA,OAAO,EACL,2BAA2B,EAC3B,gBAAgB,EAChB,KAAK,iBAAiB,EACtB,KAAK,oBAAoB,EACzB,KAAK,mBAAmB,EACxB,KAAK,8BAA8B,EACnC,KAAK,MAAM,EACX,KAAK,cAAc,EACnB,KAAK,WAAW,EAChB,KAAK,QAAQ,EACd,MAAM,yBAAyB,CAAC;AAIjC;;;;;GAKG;AACH,UAAU,uBAAuB;IAC/B,cAAc,EAAE,MAAM,CAAC;IACvB,gBAAgB,EAAE,SAAS,mBAAmB,EAAE,CAAC;IACjD,gBAAgB,EAAE,KAChB,IAAI,EAAE,oBAAoB,EAC1B,OAAO,EAAE,MAAM,EACf,aAAa,CAAC,EAAE,MAAM,KACnB,KAAK,CAAC;IACX,uBAAuB,EAAE,CACvB,OAAO,CAAC,EAAE,8BAA8B,KACrC,iBAAiB,CAAC;CACxB;AAcD;;;GAGG;AACH,wBAAgB,2BAA2B,IAAI,MAAM,CAWpD;AA0BD;;;;;;;;;GASG;AACH,wBAAgB,sCAAsC,CACpD,GAAG,EAAE,OAAO,EACZ,SAAS,GAAE,MAAkB,GAC5B,OAAO,CAqBT;AAwDD;;;;;;;;;;;;GAYG;AACH,wBAAsB,4BAA4B,IAAI,OAAO,CAC3D,CAAC,OAAO,CAAC,EAAE,8BAA8B,KAAK,iBAAiB,CAChE,CA2BA;AAgDD;;;;GAIG;AACH,wBAAsB,sBAAsB,IAAI,OAAO,CAAC,OAAO,CAAC,CAE/D;AAED;;;;;;;;GAQG;AACH,wBAAsB,wBAAwB,IAAI,OAAO,CAAC,uBAAuB,GAAG,IAAI,CAAC,CAMxF;AAMD,OAAO,EACL,2BAA2B,EAC3B,gBAAgB,GACjB,CAAC;AAEF,YAAY,EACV,iBAAiB,EACjB,oBAAoB,EACpB,mBAAmB,EACnB,8BAA8B,EAC9B,MAAM,EACN,cAAc,EACd,WAAW,EACX,QAAQ,GACT,CAAC"}

package/dist/coding/pre-action-gate.d.ts

@@ -1,18 +1,58 @@
-import '../causal-trajectory.js';
-export { cB as NormalizedActionIntent, cC as PRE_ACTION_FINGERPRINT_VERSION, cD as PRE_ACTION_GATE_DEFAULT_TIMEOUT_MS, cE as PRE_ACTION_WARNING_VERSION, cF as PreActionFailureGate, cG as PreActionGateDependencies, cH as PreActionGateRequest, cI as PreActionGateResult, cJ as PreActionGateStatus, d1 as normalizeActionIntent, db as sanitizePayloadString } from '../types-Du6VdoAg.js';
-import '../objective-state.js';
-import '../message-parts/index.js';
-import '../bounded-jsonl-state.js';
-import '../operator-doctor-types.js';
-import '../external-wiki-config.js';
-import '../recall-state-view.js';
-import '../contradiction-config.js';
-import '../graph-path-scoring-config.js';
-import '../background-generation-config.js';
-import '../deep-recall-config.js';
-import '../recall-navigation-config.js';
-import '../recall-recognition-tier.js';
-import '../active-context-config.js';
-import '../local-llm-config.js';
-import '../semantic-merge-config.js';
-import '../types-continuity.js';
+import { type ActionIntent, type ActionStrategyId, type CausalTrajectoryStrictReadResult } from "../causal-trajectory.js";
+import type { CodingContext } from "../types.js";
+export declare const PRE_ACTION_FINGERPRINT_VERSION = 1;
+export declare const PRE_ACTION_WARNING_VERSION = 1;
+export declare const PRE_ACTION_GATE_DEFAULT_TIMEOUT_MS = 50;
+export type PreActionGateStatus = "NO_MATCH" | "MATCH_WARN" | "ERROR_FAIL_OPEN";
+export interface PreActionGateRequest {
+    sessionKey: string;
+    strategyId: ActionStrategyId;
+    intent: ActionIntent;
+    codingContext: CodingContext;
+    memoryDir?: string;
+    causalTrajectoryStoreDir?: string;
+    signal?: AbortSignal;
+}
+export interface PreActionGateResult {
+    status: PreActionGateStatus;
+    reason?: string;
+    advisoryText?: string;
+    fingerprint?: string;
+    matchedTrajectoryId?: string;
+    revision?: string;
+}
+export interface NormalizedActionIntent {
+    kind: "command" | "edit";
+    strategyId: ActionStrategyId;
+    fingerprint: string;
+}
+export interface PreActionGateDependencies {
+    readStrict?: (options: {
+        memoryDir: string;
+        causalTrajectoryStoreDir?: string;
+        signal?: AbortSignal;
+    }) => Promise<CausalTrajectoryStrictReadResult>;
+    getRevision?: (options: {
+        memoryDir: string;
+        causalTrajectoryStoreDir?: string;
+    }) => Promise<string>;
+    clock?: () => number;
+    maxCacheSize?: number;
+    timeoutMs?: number;
+}
+export declare function sanitizePayloadString(input: string): string;
+export declare function normalizeActionIntent(intent: ActionIntent, strategyId: ActionStrategyId, codingContext: CodingContext): NormalizedActionIntent;
+export declare class PreActionFailureGate {
+    private readonly cache;
+    private readonly maxCacheSize;
+    private readonly readStrict;
+    private readonly getRevision;
+    private readonly clock;
+    private readonly timeoutMs;
+    constructor(deps?: PreActionGateDependencies);
+    clearCache(): void;
+    evaluate(request: PreActionGateRequest): Promise<PreActionGateResult>;
+    private evaluateInternal;
+    private setCache;
+}
+//# sourceMappingURL=pre-action-gate.d.ts.map

package/dist/coding/pre-action-gate.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"pre-action-gate.d.ts","sourceRoot":"","sources":["../../src/coding/pre-action-gate.ts"],"names":[],"mappings":"AAEA,OAAO,EAIL,KAAK,YAAY,EACjB,KAAK,gBAAgB,EAErB,KAAK,gCAAgC,EAEtC,MAAM,yBAAyB,CAAC;AACjC,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAEjD,eAAO,MAAM,8BAA8B,IAAI,CAAC;AAChD,eAAO,MAAM,0BAA0B,IAAI,CAAC;AAC5C,eAAO,MAAM,kCAAkC,KAAK,CAAC;AAErD,MAAM,MAAM,mBAAmB,GAAG,UAAU,GAAG,YAAY,GAAG,iBAAiB,CAAC;AAEhF,MAAM,WAAW,oBAAoB;IACnC,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,gBAAgB,CAAC;IAC7B,MAAM,EAAE,YAAY,CAAC;IACrB,aAAa,EAAE,aAAa,CAAC;IAC7B,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,wBAAwB,CAAC,EAAE,MAAM,CAAC;IAClC,MAAM,CAAC,EAAE,WAAW,CAAC;CACtB;AAED,MAAM,WAAW,mBAAmB;IAClC,MAAM,EAAE,mBAAmB,CAAC;IAC5B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,sBAAsB;IACrC,IAAI,EAAE,SAAS,GAAG,MAAM,CAAC;IACzB,UAAU,EAAE,gBAAgB,CAAC;IAC7B,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,MAAM,WAAW,yBAAyB;IACxC,UAAU,CAAC,EAAE,CAAC,OAAO,EAAE;QACrB,SAAS,EAAE,MAAM,CAAC;QAClB,wBAAwB,CAAC,EAAE,MAAM,CAAC;QAClC,MAAM,CAAC,EAAE,WAAW,CAAC;KACtB,KAAK,OAAO,CAAC,gCAAgC,CAAC,CAAC;IAChD,WAAW,CAAC,EAAE,CAAC,OAAO,EAAE;QACtB,SAAS,EAAE,MAAM,CAAC;QAClB,wBAAwB,CAAC,EAAE,MAAM,CAAC;KACnC,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IACtB,KAAK,CAAC,EAAE,MAAM,MAAM,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AA0BD,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAM3D;AA2CD,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,YAAY,EACpB,UAAU,EAAE,gBAAgB,EAC5B,aAAa,EAAE,aAAa,GAC3B,sBAAsB,CAoCxB;AAoBD,qBAAa,oBAAoB;IAC/B,OAAO,CAAC,QAAQ,CAAC,KAAK,CAA0C;IAChE,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAS;IACtC,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAuD;IAClF,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAwD;IACpF,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAe;IACrC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAS;gBAEvB,IAAI,GAAE,yBAA8B;IAQzC,UAAU,IAAI,IAAI;IAInB,QAAQ,CAAC,OAAO,EAAE,oBAAoB,GAAG,OAAO,CAAC,mBAAmB,CAAC;YA6B7D,gBAAgB;IAsE9B,OAAO,CAAC,QAAQ;CAOjB"}

package/dist/coding/pre-action-gate.js

@@ -5,8 +5,8 @@ import {
   PreActionFailureGate,
   normalizeActionIntent,
   sanitizePayloadString
-} from "../chunk-IWO6QXWJ.js";
-import "../chunk-L63GFNKL.js";
+} from "../chunk-BICNRVEC.js";
+import "../chunk-LMMQKZK3.js";
 import "../chunk-YCVB7X56.js";
 import "../chunk-A2IYSXDQ.js";
 import "../chunk-B6O2YLUR.js";

package/dist/coding/review-context.d.ts

@@ -0,0 +1,147 @@
+/**
+ * Diff-aware review-context packer (issue #569 PR 4).
+ *
+ * When an agent is asked "review this PR" / "what changed in this diff" /
+ * "look at this diff", the prompt that reaches recall is short and generic
+ * — the real signal is the diff itself. This module:
+ *
+ *   1. Detects review-intent prompts via `isReviewPrompt`.
+ *   2. Extracts the touched file list from a unified diff via
+ *      `parseTouchedFiles`.
+ *   3. Re-ranks a set of candidate memories so that memories whose
+ *      `entityRefs` mention a touched path float to the top. The boost is
+ *      additive and bounded so it doesn't obliterate the original ranking —
+ *      it's a bias, not a filter.
+ *
+ * Pure — no orchestrator, no storage. Callers inject the candidate memories
+ * they already have from their normal recall pipeline. This keeps the
+ * module easy to test and integrates cleanly with the existing tiered-recall
+ * code in `orchestrator.ts` (the tier itself can be wired later; the pure
+ * surface is what PRs 5/6/7 will call).
+ */
+import type { StructuralContextDegradation, StructuralContextProvider } from "./structural-context.js";
+/**
+ * A memory candidate as fed into review-context ranking. The shape is a
+ * deliberate subset of the core `MemorySummary` / recall result — only the
+ * fields we actually need — so this module stays decoupled from the rest of
+ * the codebase and can be reused by CLI tools, bench fixtures, etc.
+ */
+export interface ReviewCandidate {
+    /** Opaque identifier. Echoed unchanged in the output. */
+    id: string;
+    /**
+     * Pre-review relevance score from the upstream recall pipeline. Higher is
+     * better. `0` is treated as "no prior signal" and gets the full review
+     * boost when a path match is found.
+     */
+    score: number;
+    /**
+     * References the memory mentions (file paths, entity names, etc.). Used
+     * to decide whether any touched file appears in the memory's scope.
+     *
+     * Accepts `undefined`/missing so callers can pass sparse records from
+     * legacy storage without pre-filling.
+     */
+    entityRefs?: string[];
+}
+export interface ReviewContext {
+    /**
+     * Normalized file paths touched by the diff. Each entry is forward-slashed
+     * and relative to the repo root when possible.
+     */
+    touchedFiles: string[];
+    /**
+     * Candidates re-sorted so memories whose `entityRefs` mention a touched
+     * path are boosted. Shape matches the input `ReviewCandidate[]` — the
+     * boost is recorded on each entry as `boost` for observability.
+     */
+    rankedRecall: Array<ReviewCandidate & {
+        boost: number;
+    }>;
+    /**
+     * Structural-context provider degradation observed while expanding the
+     * diff to touched symbols (issue #1548 Track A PR 5, #1536 pattern).
+     * Present only when a provider was consulted AND it failed — a silent
+     * provider failure must never look like "no structural matches"
+     * (CLAUDE.md rule 34). Undefined when no provider was consulted (the
+     * pure {@link packReviewContext} path) or when the provider succeeded.
+     */
+    structuralDegradation?: StructuralContextDegradation;
+}
+/**
+ * `true` when the prompt looks like a review / diff-explanation request.
+ *
+ * Empty / non-string input → `false` (the caller shouldn't branch on an
+ * invalid prompt).
+ */
+export declare function isReviewPrompt(prompt: string | null | undefined): boolean;
+/**
+ * Parse a unified diff and return the set of files touched. Accepts both the
+ * `diff --git` form (`diff --git a/foo b/bar`) and the `--- / +++` form
+ * (`--- a/foo\n+++ b/bar`). Returns deduplicated, repo-root-relative paths
+ * (with the conventional `a/` / `b/` prefixes stripped).
+ *
+ * Path entries of `/dev/null` (used in adds/deletes) are excluded.
+ */
+export declare function parseTouchedFiles(diff: string | null | undefined): string[];
+/**
+ * Build a review-context ranking for a set of candidate memories.
+ *
+ * Contract:
+ *   - `touchedFiles` is the parsed diff file list.
+ *   - `candidates` is passed through unchanged when no boost applies.
+ *   - When a boost applies, the result is sorted by `(score + boost)` desc,
+ *     with a stable secondary sort on the original `id` for determinism
+ *     (CLAUDE.md #19 — comparators must return 0 for equal items).
+ */
+export declare function rankReviewCandidates(candidates: ReviewCandidate[], touchedFiles: string[]): Array<ReviewCandidate & {
+    boost: number;
+}>;
+export interface PackReviewContextInput {
+    /** Unified diff, as produced by `git diff`. */
+    diff: string | null | undefined;
+    /** Candidate memories from the upstream recall pipeline. */
+    candidates: ReviewCandidate[];
+}
+/**
+ * Top-level entry point used by the orchestrator (and CLI / bench) when a
+ * review-intent prompt is detected.
+ *
+ * Parses the diff, re-ranks the candidates, and returns both artefacts so
+ * the caller can surface `touchedFiles` as context and `rankedRecall` as
+ * the recall result.
+ */
+export declare function packReviewContext(input: PackReviewContextInput): ReviewContext;
+/**
+ * Input for the structural-context-aware review-context packer. The pure
+ * {@link packReviewContext} stays unchanged (gate-off parity); this async
+ * variant consults a {@link StructuralContextProvider} to expand the diff's
+ * touched FILE paths into touched SYMBOL names, widening the match set so
+ * memories that mention a changed symbol — not just the file path — float
+ * up. The gate (`structuralProvider !== "none"`) is checked ONCE by the
+ * caller, never inside this function (rule 39).
+ */
+export interface PackReviewContextStructuralInput extends PackReviewContextInput {
+    /** A probed structural-context provider. Never undefined here. */
+    provider: StructuralContextProvider;
+    /** Cancellation forwarded to the provider subprocess (rule 40). */
+    signal?: AbortSignal;
+}
+/**
+ * Structural-context-aware packer.
+ *
+ * Contract (issue #1548 Track A PR 5, prove-fail-before characterization):
+ *   - Provider returns symbols (`ok: true`) → symbol names are ADDED to the
+ *     match set alongside touched files; memories mentioning a symbol get the
+ *     same bounded additive boost as a file-path match.
+ *   - Provider fails (`ok: false`, any code) → ranking falls back to
+ *     FILE-PATH-ONLY boosting, BYTE-IDENTICAL to {@link packReviewContext}
+ *     (the match set is exactly the touched files), AND `structuralDegradation`
+ *     is populated so xray/doctor can surface the failure (rule 34).
+ *
+ * The underlying ranker ({@link rankReviewCandidates}) is reused unchanged,
+ * so the deterministic ordering, boost cap, and tie-break are identical to
+ * the pure path.
+ */
+export declare function packReviewContextStructural(input: PackReviewContextStructuralInput): Promise<ReviewContext>;
+//# sourceMappingURL=review-context.d.ts.map

package/dist/coding/review-context.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-context.d.ts","sourceRoot":"","sources":["../../src/coding/review-context.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;GAoBG;AAEH,OAAO,KAAK,EACV,4BAA4B,EAC5B,yBAAyB,EAC1B,MAAM,yBAAyB,CAAC;AAMjC;;;;;GAKG;AACH,MAAM,WAAW,eAAe;IAC9B,yDAAyD;IACzD,EAAE,EAAE,MAAM,CAAC;IACX;;;;OAIG;IACH,KAAK,EAAE,MAAM,CAAC;IACd;;;;;;OAMG;IACH,UAAU,CAAC,EAAE,MAAM,EAAE,CAAC;CACvB;AAED,MAAM,WAAW,aAAa;IAC5B;;;OAGG;IACH,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB;;;;OAIG;IACH,YAAY,EAAE,KAAK,CAAC,eAAe,GAAG;QAAE,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IACzD;;;;;;;OAOG;IACH,qBAAqB,CAAC,EAAE,4BAA4B,CAAC;CACtD;AAoBD;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAAG,OAAO,CAKzE;AAMD;;;;;;;GAOG;AACH,wBAAgB,iBAAiB,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAAG,MAAM,EAAE,CAoD3E;AAgND;;;;;;;;;GASG;AACH,wBAAgB,oBAAoB,CAClC,UAAU,EAAE,eAAe,EAAE,EAC7B,YAAY,EAAE,MAAM,EAAE,GACrB,KAAK,CAAC,eAAe,GAAG;IAAE,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC,CAkB5C;AAMD,MAAM,WAAW,sBAAsB;IACrC,+CAA+C;IAC/C,IAAI,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,CAAC;IAChC,4DAA4D;IAC5D,UAAU,EAAE,eAAe,EAAE,CAAC;CAC/B;AAED;;;;;;;GAOG;AACH,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,sBAAsB,GAAG,aAAa,CAI9E;AAMD;;;;;;;;GAQG;AACH,MAAM,WAAW,gCAAiC,SAAQ,sBAAsB;IAC9E,kEAAkE;IAClE,QAAQ,EAAE,yBAAyB,CAAC;IACpC,mEAAmE;IACnE,MAAM,CAAC,EAAE,WAAW,CAAC;CACtB;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAsB,2BAA2B,CAC/C,KAAK,EAAE,gCAAgC,GACtC,OAAO,CAAC,aAAa,CAAC,CAyBxB"}

package/dist/coding/session-delta-surfaces.d.ts

@@ -0,0 +1,138 @@
+/**
+ * Session-delta surface contract + handler (issue #1548 Track A PR 4).
+ *
+ * The single shared implementation behind the MCP, HTTP, and CLI
+ * session-delta surfaces. All three transports dispatch through the
+ * `coding_delta` boundary operation, which calls {@link handleCodingDelta}
+ * via the service delegate.
+ *
+ * Gate (rule 39): `codingKnowledge.enabled + sessionDelta + coding
+ * context` — one predicate, identical on every surface. The tools/list
+ * visibility gate checks conditions 1–2 only (config-level).
+ *
+ * Persistence (rule 25): the new last-seen-head marker is written AFTER
+ * the delta is computed from the old one. State writes use temp-file-then-
+ * rename (rule 54) via the pure module's `writeLastSeenState`.
+ *
+ * Subcommand: `get` — computes the delta against the persisted last-seen
+ * head and updates the marker in the same call (so the NEXT session sees
+ * this one as the baseline). Read-only semantically, but it mutates the
+ * state file: it is therefore NOT counted against the write-quota (the
+ * state file is operator-side bookkeeping, not user memory content — the
+ * same way `calibration.ts` writes are uncounted).
+ */
+import type { CodingKnowledgeConfig, CodingContext } from "../types.js";
+import { sessionDeltaStatePath, type GitLogSlice, type LastSeenState, type SessionDeltaGitInvoker } from "./session-delta.js";
+export declare const DELTA_SUBCOMMANDS: readonly ["get"];
+export type DeltaSubcommand = (typeof DELTA_SUBCOMMANDS)[number];
+/** Type guard — narrows an unknown subcommand string. */
+export declare function isDeltaSubcommand(value: unknown): value is DeltaSubcommand;
+/** Human-readable subcommand list for error messages (rule 51). */
+export declare function formatDeltaSubcommands(): string;
+/** Canonical surface request — one shape for all three transports. */
+export interface DeltaSurfaceRequest {
+    subcommand: DeltaSubcommand;
+    sessionKey?: string;
+    namespace?: string;
+}
+/**
+ * Surface response. Mirrors {@link SessionDeltaResult} but flattened for
+ * transport: every outcome carries `nextState` so callers can observe the
+ * new marker even on failure (the head still advanced).
+ */
+export type DeltaSurfaceResponse = {
+    subcommand: "get";
+    ok: true;
+    /** A real delta was computed. */
+    kind: "changed";
+    delta: {
+        commits: ReadonlyArray<{
+            sha: string;
+            subject: string;
+        }>;
+        touchedFiles: readonly string[];
+        /**
+         * Uncapped total commit count (the {@link commits} slice is capped
+         * for transport). Issue #1630 fix 1.
+         */
+        totalCommits: number;
+        /**
+         * Uncapped total touched-file count (the {@link touchedFiles} slice
+         * is capped for transport). Issue #1630 fix 1.
+         */
+        totalTouchedFiles: number;
+        summaryLine: string;
+    };
+    nextState: LastSeenState;
+} | {
+    subcommand: "get";
+    ok: true;
+    /** Prior head equals current — no changes. */
+    kind: "unchanged";
+    nextState: LastSeenState;
+} | {
+    subcommand: "get";
+    ok: true;
+    /** No prior state — first session for this namespace. */
+    kind: "first_run";
+    nextState: LastSeenState;
+} | {
+    subcommand: "get";
+    /**
+     * Delta unavailable. `code` is one of `unreachable_head` (force-push/
+     * rebase erased the prior head) or `git_failed` (invoker error). The
+     * state file is still advanced when `nextState` is present.
+     */
+    ok: false;
+    code: "unreachable_head" | "git_failed";
+    detail: string;
+    nextState?: LastSeenState;
+};
+/**
+ * The resolved namespace context the handler needs. The service produces
+ * this through the SAME coding-scoped read path as the other coding surfaces.
+ */
+export interface DeltaSurfaceStorage {
+    /** The orchestrator's memoryDir — base for the state file path. */
+    readonly memoryDir: string;
+    /** The resolved coding-scoped namespace — basis for the state filename. */
+    readonly namespace: string;
+    /**
+     * Whether the calling principal may WRITE the namespace — i.e. advance the
+     * last-seen-head marker. Read-only callers (e.g. a principal with read-but-
+     * not-write on the shared namespace) receive the computed delta but the
+     * state file is NOT advanced, so they cannot move another principal's
+     * baseline (issue #1630 fix 2). Defaults to `true` when omitted so existing
+     * callers (and the surface contract tests) keep their pre-fix behavior.
+     */
+    readonly canAdvanceState?: boolean;
+}
+/**
+ * Dependencies the handler borrows from the service. The service constructs
+ * this context per call; the handler never touches the orchestrator directly.
+ */
+export interface DeltaSurfaceContext {
+    readonly codingKnowledge: CodingKnowledgeConfig;
+    getCodingContext(sessionKey: string): CodingContext | null;
+    /** Resolve storage (memoryDir + namespace) via the coding-scoped read path. */
+    resolveStorage(request: DeltaSurfaceRequest): Promise<DeltaSurfaceStorage>;
+    /** Git invoker — drives `rev-parse` + `log`. Defaults to a real invoker. */
+    gitInvoker: SessionDeltaGitInvoker;
+    /** Throw the surface-appropriate input-validation error. */
+    throwInputError(message: string): never;
+}
+/**
+ * The single shared implementation behind the MCP, HTTP, and CLI
+ * session-delta surfaces.
+ *
+ * Order of operations (rule 25 — do not destroy the old marker before the
+ * new state is useful):
+ *   1. Read the persisted last-seen-head.
+ *   2. Resolve the current repo slice via the git invoker.
+ *   3. Compute the delta from the OLD state + current slice.
+ *   4. Persist the NEW state (temp-file-then-rename).
+ *   5. Return the result.
+ */
+export declare function handleCodingDelta(request: DeltaSurfaceRequest, ctx: DeltaSurfaceContext): Promise<DeltaSurfaceResponse>;
+export { sessionDeltaStatePath, type GitLogSlice };
+//# sourceMappingURL=session-delta-surfaces.d.ts.map

package/dist/coding/session-delta-surfaces.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"session-delta-surfaces.d.ts","sourceRoot":"","sources":["../../src/coding/session-delta-surfaces.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;GAsBG;AACH,OAAO,KAAK,EACV,qBAAqB,EACrB,aAAa,EACd,MAAM,aAAa,CAAC;AACrB,OAAO,EAKL,qBAAqB,EACrB,KAAK,WAAW,EAChB,KAAK,aAAa,EAElB,KAAK,sBAAsB,EAE5B,MAAM,oBAAoB,CAAC;AAQ5B,eAAO,MAAM,iBAAiB,kBAAmB,CAAC;AAElD,MAAM,MAAM,eAAe,GAAG,CAAC,OAAO,iBAAiB,CAAC,CAAC,MAAM,CAAC,CAAC;AAIjE,yDAAyD;AACzD,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,eAAe,CAE1E;AAED,mEAAmE;AACnE,wBAAgB,sBAAsB,IAAI,MAAM,CAE/C;AAOD,sEAAsE;AACtE,MAAM,WAAW,mBAAmB;IAClC,UAAU,EAAE,eAAe,CAAC;IAC5B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED;;;;GAIG;AACH,MAAM,MAAM,oBAAoB,GAC5B;IACE,UAAU,EAAE,KAAK,CAAC;IAClB,EAAE,EAAE,IAAI,CAAC;IACT,iCAAiC;IACjC,IAAI,EAAE,SAAS,CAAC;IAChB,KAAK,EAAE;QACL,OAAO,EAAE,aAAa,CAAC;YAAE,GAAG,EAAE,MAAM,CAAC;YAAC,OAAO,EAAE,MAAM,CAAA;SAAE,CAAC,CAAC;QACzD,YAAY,EAAE,SAAS,MAAM,EAAE,CAAC;QAChC;;;WAGG;QACH,YAAY,EAAE,MAAM,CAAC;QACrB;;;WAGG;QACH,iBAAiB,EAAE,MAAM,CAAC;QAC1B,WAAW,EAAE,MAAM,CAAC;KACrB,CAAC;IACF,SAAS,EAAE,aAAa,CAAC;CAC1B,GACD;IACE,UAAU,EAAE,KAAK,CAAC;IAClB,EAAE,EAAE,IAAI,CAAC;IACT,8CAA8C;IAC9C,IAAI,EAAE,WAAW,CAAC;IAClB,SAAS,EAAE,aAAa,CAAC;CAC1B,GACD;IACE,UAAU,EAAE,KAAK,CAAC;IAClB,EAAE,EAAE,IAAI,CAAC;IACT,yDAAyD;IACzD,IAAI,EAAE,WAAW,CAAC;IAClB,SAAS,EAAE,aAAa,CAAC;CAC1B,GACD;IACE,UAAU,EAAE,KAAK,CAAC;IAClB;;;;OAIG;IACH,EAAE,EAAE,KAAK,CAAC;IACV,IAAI,EAAE,kBAAkB,GAAG,YAAY,CAAC;IACxC,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,CAAC,EAAE,aAAa,CAAC;CAC3B,CAAC;AAMN;;;GAGG;AACH,MAAM,WAAW,mBAAmB;IAClC,mEAAmE;IACnE,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,2EAA2E;IAC3E,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B;;;;;;;OAOG;IACH,QAAQ,CAAC,eAAe,CAAC,EAAE,OAAO,CAAC;CACpC;AAED;;;GAGG;AACH,MAAM,WAAW,mBAAmB;IAClC,QAAQ,CAAC,eAAe,EAAE,qBAAqB,CAAC;IAChD,gBAAgB,CAAC,UAAU,EAAE,MAAM,GAAG,aAAa,GAAG,IAAI,CAAC;IAC3D,+EAA+E;IAC/E,cAAc,CAAC,OAAO,EAAE,mBAAmB,GAAG,OAAO,CAAC,mBAAmB,CAAC,CAAC;IAC3E,4EAA4E;IAC5E,UAAU,EAAE,sBAAsB,CAAC;IACnC,4DAA4D;IAC5D,eAAe,CAAC,OAAO,EAAE,MAAM,GAAG,KAAK,CAAC;CACzC;AAMD;;;;;;;;;;;GAWG;AACH,wBAAsB,iBAAiB,CACrC,OAAO,EAAE,mBAAmB,EAC5B,GAAG,EAAE,mBAAmB,GACvB,OAAO,CAAC,oBAAoB,CAAC,CAkB/B;AAqHD,OAAO,EAAE,qBAAqB,EAAE,KAAK,WAAW,EAAE,CAAC"}