brainclaw 1.19.0 → 1.19.1

package/dist/commands/claim.js

@@ -1,7 +1,7 @@
 import { buildOperationalIdentity } from '../core/identity.js';
 import { memoryExists } from '../core/io.js';
 import { mutate } from '../core/mutation-pipeline.js';
-import { saveClaim, generateClaimId, listClaims } from '../core/claims.js';
+import { saveClaim, generateClaimId, listClaims, claimBaselineFields } from '../core/claims.js';
 import { rebuildProjectMd } from '../core/markdown.js';
 import { loadState, saveState } from '../core/state.js';
 import { nowISO } from '../core/ids.js';
@@ -64,6 +64,10 @@ export function runClaim(description, options) {
         status: 'active',
         expires_at: options.ttl ? parseTtl(options.ttl) : undefined,
         model: resolveCurrentModel(options.cwd),
+        // pln#636 C0-b / trp#1292 — resolved here, OUTSIDE the mutate() below, so the
+        // git subprocess never widens the critical section that serializes writes on
+        // the claims store.
+        ...claimBaselineFields(options.cwd),
     };
     try {
         mutate({ cwd: options.cwd }, () => {

package/dist/commands/mcp-write-claims.js

@@ -18,7 +18,7 @@ import { getTriggeredItems, renderTriggeredItems } from '../core/lifecycle.js';
 import { buildContext } from '../core/context.js';
 import { checkBrainclawInstallableUpdate, getInstalledBrainclawVersion, renderBrainclawInstallableUpdateNotice } from '../core/brainclaw-version.js';
 import { loadConfig } from '../core/config.js';
-import { generateClaimId, loadClaim, saveClaim, adoptClaimSession, releaseClaimWithCascade } from '../core/claims.js';
+import { generateClaimId, loadClaim, saveClaim, adoptClaimSession, releaseClaimWithCascade, claimBaselineFields } from '../core/claims.js';
 import { releaseClaimNextActions } from '../core/next-actions.js';
 import { reconcileClaimConformity } from '../core/claim-conformity.js';
 import { pushStructuredWarning } from '../core/warnings.js';
@@ -128,6 +128,11 @@ export async function handleBclawClaim(payload, ctx) {
         worktree_path: worktreePath,
         expires_at: claimExpiresAt,
         handoff_mode: handoffMode,
+        // pln#636 C0-b / trp#1292 — this handler builds its claim literal instead of
+        // going through acquireClaimScope, which is why the baseline was missing on
+        // every MCP-created claim and the conformity reconcile never had anything to
+        // compare against.
+        ...claimBaselineFields(claimCwd),
     }, claimCwd);
     appendAuditEntry({ actor: resolvedIdentity.agent_name, actor_id: resolvedIdentity.agent_id, action: 'claim', item_id: claimId, item_type: 'claim', scope: claimScope, session_id: identity.session_id, host_id: identity.host_id }, claimCwd);
     // Post-claim policy check: surface constraints/traps as warnings

package/dist/commands/mcp.js

@@ -13,7 +13,7 @@ import { collectLoadValidationWarnings, findLoadValidationWarning, loadState } f
 import { memoryExists, MEMORY_DIR } from '../core/io.js';
 import { getEntity, listEntities, boundListResult, DEFAULT_FIND_CHAR_BUDGET, GRAMMAR_FILTER_CONTRACT, } from '../core/entity-operations.js';
 import { handoffDiffPreviewNote } from '../core/handoff-snapshot.js';
-import { generateClaimId, listClaims, loadClaim, saveClaim, adoptClaimSession } from '../core/claims.js';
+import { generateClaimId, listClaims, loadClaim, saveClaim, adoptClaimSession, claimBaselineFields } from '../core/claims.js';
 import { assertCrossProjectBoundary, checkPolicy } from '../core/policy.js';
 import { startSession } from './session-start.js';
 import { AgentIdentityResolutionError, AgentTrustError, resolveCurrentModel, } from '../core/agent-registry.js';
@@ -1302,6 +1302,11 @@ async function _executeMcpToolCallInner(payload) {
                         status: 'active',
                         plan_id: workReq.planId,
                         model: currentModel,
+                        // pln#636 C0-b / trp#1292 — bclaw_work(execute) is the entry point the
+                        // session protocol tells every agent to start with, so a missing
+                        // baseline here made the conformity reconcile inert for nearly all
+                        // real claims.
+                        ...claimBaselineFields(targetCwd),
                     }, targetCwd);
                     appendAuditEntry({ actor: sessionResult.agent, actor_id: sessionResult.agent_id, action: 'claim', item_id: claimId, item_type: 'claim', scope: workReq.scope, session_id: sessionResult.session_id }, targetCwd);
                     claimStatus = 'created';

package/dist/core/claims.js

@@ -150,6 +150,34 @@ export function resolveClaimBaseSha(cwd) {
         return undefined;
     }
 }
+/**
+ * The baseline fields every NEW claim must carry, ready to spread into a claim
+ * literal: `{ ...claimBaselineFields(cwd) }`.
+ *
+ * WHY A HELPER AND NOT AN INLINE CALL. `base_sha` shipped in 1.19.0 stamped from
+ * exactly ONE place — inside `acquireClaimScope` — and it turned out nothing
+ * user-facing calls that function. All four real creation paths
+ * (`bclaw_work(execute)`, `bclaw_claim`, the CLI `claim create`, and
+ * `createCoordinatorClaim`) build their claim literal inline and call `saveClaim`
+ * directly, so NO real claim ever got a baseline and the whole conformity
+ * reconcile that depends on it was inert (trp#1292). A named, greppable helper is
+ * what makes "did this creation path stamp the baseline?" answerable, and it is
+ * asserted per surface rather than only on the core function — testing
+ * `acquireClaimScope` directly is exactly what hid the gap.
+ *
+ * Returns an EMPTY object rather than `{ base_sha: undefined }` so a claim created
+ * outside a git repo has no such key at all, matching the "optional, never
+ * backfilled" contract the schema documents.
+ *
+ * Deliberately NOT applied inside `saveClaim`: that function also persists
+ * UPDATES (release, patch, adoption), and the baseline must be a fixed point —
+ * re-stamping it on every save is precisely the moving-ground failure that made
+ * `git diff HEAD` unusable in the first place.
+ */
+export function claimBaselineFields(cwd) {
+    const base_sha = resolveClaimBaseSha(cwd);
+    return base_sha ? { base_sha } : {};
+}
 /**
  * Atomically check for an active claim on `scope` and save a new one if absent.
  *
@@ -160,7 +188,7 @@ export function acquireClaimScope(input, cwd) {
     // Resolved OUTSIDE the mutate callback: one git call per acquisition, and it
     // stays off the critical section (mutate serializes filesystem writes on the
     // claims store, so a subprocess spawn inside it would widen the lock window).
-    const baseSha = resolveClaimBaseSha(cwd);
+    const baseline = claimBaselineFields(cwd);
     return mutate({ cwd }, () => {
         const conflictingClaim = listClaims(cwd).find((claim) => claim.status === 'active' && claim.scope === input.scope);
         if (conflictingClaim) {
@@ -180,7 +208,7 @@ export function acquireClaimScope(input, cwd) {
             model: input.model,
             // pln#636 C0-b — capture the baseline while we know it. Absent when the
             // project is not a git repo; downstream treats that as unverifiable.
-            ...(baseSha ? { base_sha: baseSha } : {}),
+            ...baseline,
             ...(input.paths?.length ? { paths: input.paths } : {}),
         };
         saveClaimUnlocked(claim, cwd);
@@ -854,6 +882,10 @@ export function createCoordinatorClaim(options) {
         };
     }
     const claimId = generateClaimId();
+    // Resolved OUTSIDE the lock below, for the same reason acquireClaimScope does
+    // it: a subprocess spawn inside the mutate callback would widen the critical
+    // section that serializes writes on the claims store.
+    const baseline = claimBaselineFields(options.cwd);
     let worktreePath;
     let worktreeWarning;
     // Create isolated worktree (matching bclaw_claim MCP handler behavior).
@@ -905,6 +937,7 @@ export function createCoordinatorClaim(options) {
             created_at: nowISO(),
             status: 'active',
             worktree_path: worktreePath,
+            ...baseline, // pln#636 C0-b / trp#1292 — dispatched lanes need it most
         }, options.cwd);
         return { claimId, worktreePath, worktreeWarning, reusedExisting: false };
     });

package/dist/facts.js

@@ -1,8 +1,8 @@
 // Generated by scripts/emit-site-facts.mjs at build time. Do not edit manually.
-// Source: brainclaw v1.19.0 on 2026-08-01T21:36:53.526Z
+// Source: brainclaw v1.19.1 on 2026-08-02T07:16:39.143Z
 export const FACTS = {
-  "version": "1.19.0",
-  "generated_at": "2026-08-01T21:36:53.526Z",
+  "version": "1.19.1",
+  "generated_at": "2026-08-02T07:16:39.143Z",
   "tools": {
     "count": 67,
     "published_count": 65,
@@ -474,7 +474,7 @@ export const FACTS = {
   },
   "bench": {
     "schema": "brainclaw.bench.v1",
-    "generated_at": "2026-08-01T21:36:51.459Z",
+    "generated_at": "2026-08-02T07:16:37.042Z",
     "node_version": "v24.18.0",
     "platform": "linux-x64",
     "repeats": 3,
@@ -483,7 +483,7 @@ export const FACTS = {
         "name": "cold_onboard",
         "volume": "empty",
         "description": "fresh machine → init → first useful context. Baseline for time-to-first-value.",
-        "duration_ms_median": 74,
+        "duration_ms_median": 81,
         "payload_chars_median": 1640,
         "payload_tokens_est_median": 410
       },
@@ -491,7 +491,7 @@ export const FACTS = {
         "name": "warm_work",
         "volume": "medium",
         "description": "bclaw_work consult over a real-shaped store (~200 plans / 500 handoffs / 450 claims).",
-        "duration_ms_median": 124,
+        "duration_ms_median": 123,
         "payload_chars_median": 2626,
         "payload_tokens_est_median": 657
       },
@@ -499,7 +499,7 @@ export const FACTS = {
         "name": "first_edit",
         "volume": "medium",
         "description": "code_find + code_brief on the fresh-agent path (missing index, first touch).",
-        "duration_ms_median": 14,
+        "duration_ms_median": 12,
         "payload_chars_median": 499,
         "payload_tokens_est_median": 125
       }

package/dist/facts.json

@@ -1,6 +1,6 @@
 {
-  "version": "1.19.0",
-  "generated_at": "2026-08-01T21:36:53.526Z",
+  "version": "1.19.1",
+  "generated_at": "2026-08-02T07:16:39.143Z",
   "tools": {
     "count": 67,
     "published_count": 65,
@@ -472,7 +472,7 @@
   },
   "bench": {
     "schema": "brainclaw.bench.v1",
-    "generated_at": "2026-08-01T21:36:51.459Z",
+    "generated_at": "2026-08-02T07:16:37.042Z",
     "node_version": "v24.18.0",
     "platform": "linux-x64",
     "repeats": 3,
@@ -481,7 +481,7 @@
         "name": "cold_onboard",
         "volume": "empty",
         "description": "fresh machine → init → first useful context. Baseline for time-to-first-value.",
-        "duration_ms_median": 74,
+        "duration_ms_median": 81,
         "payload_chars_median": 1640,
         "payload_tokens_est_median": 410
       },
@@ -489,7 +489,7 @@
         "name": "warm_work",
         "volume": "medium",
         "description": "bclaw_work consult over a real-shaped store (~200 plans / 500 handoffs / 450 claims).",
-        "duration_ms_median": 124,
+        "duration_ms_median": 123,
         "payload_chars_median": 2626,
         "payload_tokens_est_median": 657
       },
@@ -497,7 +497,7 @@
         "name": "first_edit",
         "volume": "medium",
         "description": "code_find + code_brief on the fresh-agent path (missing index, first touch).",
-        "duration_ms_median": 14,
+        "duration_ms_median": 12,
         "payload_chars_median": 499,
         "payload_tokens_est_median": 125
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "brainclaw",
-  "version": "1.19.0",
+  "version": "1.19.1",
   "description": "Shared project memory for humans and coding agents.",
   "type": "module",
   "repository": {