@remnic/core 9.3.685 → 9.3.687

package/dist/index.js

@@ -98,7 +98,7 @@ import {
   registerTrainingExportAdapter,
   runBulkImportCliCommand,
   runWearablesCliCommand
-} from "./chunk-JPCKLFWK.js";
+} from "./chunk-QVMXQGT7.js";
 import "./chunk-MC4FJXPA.js";
 import "./chunk-LQHDIS7L.js";
 import "./chunk-7F7Z6MOS.js";
@@ -500,7 +500,7 @@ import "./chunk-LBJBNWS2.js";
 import "./chunk-OADWQ5CR.js";
 import {
   EngramAccessHttpServer
-} from "./chunk-BZISAF67.js";
+} from "./chunk-S2OU5DZY.js";
 import "./chunk-SEDEKFYQ.js";
 import "./chunk-RKNJBZ55.js";
 import "./chunk-J64TK33U.js";
@@ -514,7 +514,7 @@ import {
 } from "./chunk-7WV3F5DQ.js";
 import {
   EngramMcpServer
-} from "./chunk-OFUULUSY.js";
+} from "./chunk-473JIN2U.js";
 import {
   REMNIC_CHATGPT_MEMORY_INSPECTOR_CANONICAL_TOOL,
   REMNIC_CHATGPT_MEMORY_INSPECTOR_MIME_TYPE,
@@ -530,7 +530,7 @@ import {
   parseOaiMemCitation,
   sanitizeNoteForCitation
 } from "./chunk-D24OXEPB.js";
-import "./chunk-IIDSFFE5.js";
+import "./chunk-KFBOZYME.js";
 import {
   formatZodError,
   memoryStoreRequestSchema,
@@ -539,13 +539,13 @@ import {
   suggestionSubmitRequestSchema,
   validateRequest
 } from "./chunk-UDJLF3BO.js";
-import "./chunk-PK6RGRSD.js";
+import "./chunk-NN7QYW5W.js";
 import {
   EngramAccessInputError,
   EngramAccessService,
   computeProcedureStats,
   formatProcedureStatsText
-} from "./chunk-SQGPGC76.js";
+} from "./chunk-FUCUR2OZ.js";
 import "./chunk-GDASG7NC.js";
 import "./chunk-GDB4J2H3.js";
 import {

package/dist/mcp-memory-inspector-app.d.ts

@@ -1,4 +1,4 @@
-import { c as EngramAccessRecallResponse } from './access-service-DeKrlYU_.js';
+import { d as EngramAccessRecallResponse } from './access-service-DmCHJ4cH.js';
 import { ActionConfidenceRequest } from './access-schema.js';
 import { RecallXraySnapshot } from './recall-xray.js';
 import { ActionConfidenceResult } from './action-confidence.js';

package/dist/schemas.d.ts

@@ -310,12 +310,12 @@ declare const ExtractedQuestionSchema: z.ZodObject<{
     priority: z.ZodNumber;
 }, "strip", z.ZodTypeAny, {
     question: string;
-    priority: number;
     context: string;
+    priority: number;
 }, {
     question: string;
-    priority: number;
     context: string;
+    priority: number;
 }>;
 declare const ProactiveQuestionsResultSchema: z.ZodObject<{
     questions: z.ZodArray<z.ZodObject<{
@@ -324,24 +324,24 @@ declare const ProactiveQuestionsResultSchema: z.ZodObject<{
         priority: z.ZodNumber;
     }, "strip", z.ZodTypeAny, {
         question: string;
-        priority: number;
         context: string;
+        priority: number;
     }, {
         question: string;
-        priority: number;
         context: string;
+        priority: number;
     }>, "many">;
 }, "strip", z.ZodTypeAny, {
     questions: {
         question: string;
-        priority: number;
         context: string;
+        priority: number;
     }[];
 }, {
     questions: {
         question: string;
-        priority: number;
         context: string;
+        priority: number;
     }[];
 }>;
 declare const ExtractedRelationshipSchema: z.ZodObject<{
@@ -987,12 +987,12 @@ declare const ExtractionResultSchema: z.ZodObject<{
         priority: z.ZodNumber;
     }, "strip", z.ZodTypeAny, {
         question: string;
-        priority: number;
         context: string;
+        priority: number;
     }, {
         question: string;
-        priority: number;
         context: string;
+        priority: number;
     }>, "many">;
     identityReflection: z.ZodNullable<z.ZodOptional<z.ZodString>>;
     relationships: z.ZodNullable<z.ZodOptional<z.ZodArray<z.ZodObject<{
@@ -1014,8 +1014,8 @@ declare const ExtractionResultSchema: z.ZodObject<{
 }, "strip", z.ZodTypeAny, {
     questions: {
         question: string;
-        priority: number;
         context: string;
+        priority: number;
     }[];
     facts: {
         tags: string[];
@@ -1069,8 +1069,8 @@ declare const ExtractionResultSchema: z.ZodObject<{
 }, {
     questions: {
         question: string;
-        priority: number;
         context: string;
+        priority: number;
     }[];
     facts: {
         tags: string[];
@@ -1386,8 +1386,8 @@ declare const BehaviorLoopAdjustmentSchema: z.ZodObject<{
     reason: z.ZodString;
     appliedAt: z.ZodString;
 }, "strip", z.ZodTypeAny, {
-    reason: string;
     confidence: number;
+    reason: string;
     parameter: string;
     previousValue: number;
     nextValue: number;
@@ -1395,8 +1395,8 @@ declare const BehaviorLoopAdjustmentSchema: z.ZodObject<{
     evidenceCount: number;
     appliedAt: string;
 }, {
-    reason: string;
     confidence: number;
+    reason: string;
     parameter: string;
     previousValue: number;
     nextValue: number;
@@ -1420,8 +1420,8 @@ declare const BehaviorLoopPolicyStateSchema: z.ZodObject<{
         reason: z.ZodString;
         appliedAt: z.ZodString;
     }, "strip", z.ZodTypeAny, {
-        reason: string;
         confidence: number;
+        reason: string;
         parameter: string;
         previousValue: number;
         nextValue: number;
@@ -1429,8 +1429,8 @@ declare const BehaviorLoopPolicyStateSchema: z.ZodObject<{
         evidenceCount: number;
         appliedAt: string;
     }, {
-        reason: string;
         confidence: number;
+        reason: string;
         parameter: string;
         previousValue: number;
         nextValue: number;
@@ -1447,8 +1447,8 @@ declare const BehaviorLoopPolicyStateSchema: z.ZodObject<{
     maxDeltaPerCycle: number;
     protectedParams: string[];
     adjustments: {
-        reason: string;
         confidence: number;
+        reason: string;
         parameter: string;
         previousValue: number;
         nextValue: number;
@@ -1464,8 +1464,8 @@ declare const BehaviorLoopPolicyStateSchema: z.ZodObject<{
     maxDeltaPerCycle: number;
     protectedParams: string[];
     adjustments: {
-        reason: string;
         confidence: number;
+        reason: string;
         parameter: string;
         previousValue: number;
         nextValue: number;

package/dist/transfer/types.d.ts

@@ -108,6 +108,10 @@ declare const ExportBundleV1Schema: z.ZodObject<{
         path: string;
     }>, "many">;
 }, "strip", z.ZodTypeAny, {
+    records: {
+        content: string;
+        path: string;
+    }[];
     manifest: {
         schemaVersion: 1;
         createdAt: string;
@@ -120,11 +124,11 @@ declare const ExportBundleV1Schema: z.ZodObject<{
         pluginVersion: string;
         includesTranscripts: boolean;
     };
+}, {
     records: {
         content: string;
         path: string;
     }[];
-}, {
     manifest: {
         schemaVersion: 1;
         createdAt: string;
@@ -137,10 +141,6 @@ declare const ExportBundleV1Schema: z.ZodObject<{
         pluginVersion: string;
         includesTranscripts: boolean;
     };
-    records: {
-        content: string;
-        path: string;
-    }[];
 }>;
 type ExportBundleV1 = z.infer<typeof ExportBundleV1Schema>;
 /**
@@ -803,6 +803,10 @@ declare const ExportBundleV2Schema: z.ZodObject<{
         path: string;
     }>, "many">;
 }, "strip", z.ZodTypeAny, {
+    records: {
+        content: string;
+        path: string;
+    }[];
     manifest: {
         schemaVersion: 2;
         createdAt: string;
@@ -837,11 +841,11 @@ declare const ExportBundleV2Schema: z.ZodObject<{
             };
         };
     };
+}, {
     records: {
         content: string;
         path: string;
     }[];
-}, {
     manifest: {
         schemaVersion: 2;
         createdAt: string;
@@ -876,10 +880,6 @@ declare const ExportBundleV2Schema: z.ZodObject<{
             } | null | undefined;
         };
     };
-    records: {
-        content: string;
-        path: string;
-    }[];
 }>;
 type ExportBundleV2 = z.infer<typeof ExportBundleV2Schema>;
 type AnyExportManifest = ExportManifestV1 | ExportManifestV2;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remnic/core",
-  "version": "9.3.685",
+  "version": "9.3.687",
   "description": "Framework-agnostic Remnic memory engine — orchestrator, storage, extraction, search, trust zones",
   "type": "module",
   "main": "dist/index.js",
@@ -2931,7 +2931,7 @@
     "core"
   ],
   "peerDependencies": {
-    "@remnic/coding-graph": "^9.3.685"
+    "@remnic/coding-graph": "^9.3.687"
   },
   "peerDependenciesMeta": {
     "@remnic/coding-graph": {

package/src/access-boundary.ts

@@ -33,7 +33,8 @@ import { expandTildePath } from "./utils/path.js";
 export type OperationName =
   | "memory_get"
   | "memory_search"
-  | "memory_store";
+  | "memory_store"
+  | "coding_decision";
 
 // ---------------------------------------------------------------------------
 // Operation context — what every handler receives

package/src/access-cli.ts

@@ -11,10 +11,11 @@ import { getOperation } from "./access-boundary.js";
 // Importing access-operations registers the pilot boundary operations as a
 // side effect; the store command dispatches through the registry (issue #1525).
 import "./access-operations.js";
+import { projectTagProjectId } from "./coding/coding-namespace.js";
 
 const OPENCLAW_REMNIC_PLUGIN_IDS = ["openclaw-remnic", "openclaw-engram"] as const;
 
-type CommandName = "browse" | "store";
+type CommandName = "browse" | "store" | "decision";
 
 type ParsedArgs = {
   command: CommandName;
@@ -133,9 +134,9 @@ function writeCliOutput(text: string = ""): void {
 
 function usage(): string {
   return [
-    "Usage:",
     "  engram-access browse [options]",
     "  engram-access store [options]",
+    "  engram-access decision [options]",
     "",
     "Browse options:",
     "  --namespace <name>",
@@ -160,6 +161,21 @@ function usage(): string {
     "  --source-reason <text>",
     "  --idempotency-key <key>",
     "  --dry-run",
+    "",
+    "Decision options:",
+    "  --subcommand <list|get|record|supersede>",
+    "  --namespace <name>",
+    "  --session-key <key>",
+    "  --principal <principal>",
+    "  --id <id> (get/supersede)",
+    "  --title <title> (record/supersede)",
+    "  --status <proposed|accepted|superseded|rejected> (record)",
+    "  --context <text> (record/supersede)",
+    "  --decision <text> (record/supersede)",
+    "  --consequences <text> (record/supersede)",
+    "  --entity-ref <ref> (repeatable)",
+    "  --project-tag <tag> (attach coding context for this invocation)",
+    "  --supersedes-id <id> (alias for --id on supersede)",
   ].join("\n");
 }
 
@@ -194,8 +210,25 @@ const COMMAND_SPECS: Record<CommandName, CommandSpec> = {
     ]),
     flagOptions: new Set(["dry-run"]),
   },
+  decision: {
+    valueOptions: new Set([
+      "subcommand",
+      "namespace",
+      "session-key",
+      "principal",
+      "id",
+      "title",
+      "status",
+      "context",
+      "decision",
+      "consequences",
+      "entity-ref",
+      "project-tag",
+      "supersedes-id",
+    ]),
+    flagOptions: new Set(),
+  },
 };
-
 const BROWSE_SORT_VALUES = Object.freeze([
   "updated_desc",
   "updated_asc",
@@ -207,7 +240,7 @@ type BrowseSort = (typeof BROWSE_SORT_VALUES)[number];
 
 function parseArgs(argv: string[]): ParsedArgs {
   const [commandRaw, ...rest] = argv;
-  if (commandRaw !== "browse" && commandRaw !== "store") {
+  if (commandRaw !== "browse" && commandRaw !== "store" && commandRaw !== "decision") {
     throw new UsageError("unsupported-command");
   }
   const spec = COMMAND_SPECS[commandRaw];
@@ -466,6 +499,59 @@ function expandOptionalPath(value: string | undefined): string | undefined {
   return value === undefined ? undefined : expandTildePath(value);
 }
 
+/**
+ * Decision-record surface (issue #1548 Track A PR 2). Dispatches through the
+ * same `coding_decision` operation as the MCP tool and HTTP route — one
+ * validation boundary, three transports.
+ */
+async function runDecision(args: ParsedArgs, preferredId?: string): Promise<void> {
+  const subcommand = requireOption(args, "subcommand");
+  const { config, service } = buildRuntime(preferredId);
+  // The CLI creates a fresh Orchestrator per invocation, so the session
+  // coding-context map is empty. If --project-tag + --session-key are
+  // provided, attach a coding context BEFORE dispatching so the gate
+  // passes and project-scoped writes resolve to the right namespace
+  // (review P2).
+  const projectTag = getLastOption(args, "project-tag");
+  const sessionKey = getLastOption(args, "session-key");
+  if (projectTag && projectTag.trim().length > 0 && sessionKey && sessionKey.trim().length > 0) {
+    const projectId = projectTagProjectId(projectTag.trim());
+    service.setCodingContext({
+      sessionKey,
+      codingContext: {
+        projectId,
+        branch: null,
+        rootPath: projectId,
+        defaultBranch: null,
+      },
+    });
+  }
+  const op = getOperation("coding_decision");
+  if (!op) {
+    throw new Error("access-boundary: operation not registered: coding_decision");
+  }
+  const output = (await op.run(
+    {
+      subcommand,
+      namespace: getLastOption(args, "namespace"),
+      sessionKey,
+      id: getLastOption(args, "id"),
+      supersedesId: getLastOption(args, "supersedes-id"),
+      title: getLastOption(args, "title"),
+      status: getLastOption(args, "status"),
+      context: getLastOption(args, "context"),
+      decision: getLastOption(args, "decision"),
+      consequences: getLastOption(args, "consequences"),
+      entityRefs: getAllOptions(args, "entity-ref"),
+    },
+    {
+      service,
+      authenticatedPrincipal: getLastOption(args, "principal") ?? config.agentAccessHttp.principal,
+    },
+  )) as { result: unknown };
+  console.log(JSON.stringify(output.result, null, 2));
+}
+
 export async function main(
   argv: string[] = process.argv.slice(2),
   options: AccessCliOptions = {},
@@ -475,6 +561,10 @@ export async function main(
     await runBrowse(args, options.preferredId);
     return;
   }
+  if (args.command === "decision") {
+    await runDecision(args, options.preferredId);
+    return;
+  }
   await runStore(args, options.preferredId);
 }
 

package/src/access-http.ts

@@ -315,6 +315,7 @@ export class EngramAccessHttpServer {
       citationsEnabled: options.citationsEnabled,
       citationsAutoDetect: options.citationsAutoDetect,
       emitLegacyTools: options.emitLegacyTools,
+      codingDecisionVisible: this.service.decisionRecordSurfaceVisible,
     });
   }
 
@@ -1331,6 +1332,35 @@ export class EngramAccessHttpServer {
       return;
     }
 
+    if (req.method === "POST" && pathname === "/engram/v1/coding/decisions") {
+      // Migrated through the access boundary (issue #1525/#1548): the
+      // registry entry owns schema validation and service dispatch. HTTP
+      // resolves the request principal; the boundary re-validates the
+      // cleaned envelope. record/supersede persist decision memories, so they
+      // are gated by the same 30/min write quota as /engram/v1/memories,
+      // suggestions, and observe; list/get are pure reads and stay uncounted
+      // (review P2: apply write quotas to decision writes).
+      const body = await this.readJsonBody(req);
+      const isWriteSubcommand =
+        body.subcommand === "record" || body.subcommand === "supersede";
+      if (isWriteSubcommand) {
+        this.ensureWriteRateLimitAvailable();
+      }
+      const op = getOperation("coding_decision");
+      if (!op) {
+        throw new EngramAccessInputError("access-boundary: operation not registered: coding_decision");
+      }
+      const output = (await op.run(body, {
+        service: this.service,
+        authenticatedPrincipal: this.resolveRequestPrincipal(req),
+      })) as { result: unknown };
+      if (isWriteSubcommand) {
+        this.recordWriteRateLimitHit();
+      }
+      this.respondJson(res, 200, output.result);
+      return;
+    }
+
     if (req.method === "POST" && pathname === "/engram/v1/suggestions") {
       const body = await this.readValidatedBody(req, "suggestionSubmit");
       const request = {
@@ -2183,6 +2213,13 @@ export class EngramAccessHttpServer {
       typeof toolArgs === "object" &&
       !Array.isArray(toolArgs) &&
       (toolArgs as { dryRun?: unknown }).dryRun === true;
+    const codingDecisionWrite =
+      (toolName === "engram.coding_decision" || toolName === "remnic.coding_decision") &&
+      toolArgs !== null &&
+      typeof toolArgs === "object" &&
+      !Array.isArray(toolArgs) &&
+      ((toolArgs as { subcommand?: unknown }).subcommand === "record" ||
+        (toolArgs as { subcommand?: unknown }).subcommand === "supersede");
     const isMcpWrite =
       request.method === "tools/call" &&
       (
@@ -2210,7 +2247,8 @@ export class EngramAccessHttpServer {
             toolName === "engram.memory_action_apply" ||
             toolName === "remnic.memory_action_apply"
           )
-        )
+        ) ||
+        codingDecisionWrite
       );
     if (isMcpWrite) {
       this.ensureWriteRateLimitAvailable();

package/src/access-mcp.ts

@@ -109,6 +109,7 @@ const MCP_MIGRATED_OPERATIONS: Readonly<Record<string, OperationName>> = {
   "engram.memory_get": "memory_get",
   "engram.memory_search": "memory_search",
   "engram.memory_store": "memory_store",
+  "engram.coding_decision": "coding_decision",
 };
 
 function resolveChatGptInspectorRecallSessionKey(
@@ -304,7 +305,9 @@ export class EngramMcpServer {
    */
   private initSessionIds = new Map<string, string>();
 
-  /** Whether oai-mem-citation guidance is explicitly enabled via config. */
+  /**
+   * Whether oai-mem-citation guidance is explicitly enabled via config.
+   */
   private readonly citationsEnabled: boolean;
   /** Whether to auto-enable citations for Codex adapter connections. */
   private readonly citationsAutoDetect: boolean;
@@ -314,6 +317,13 @@ export class EngramMcpServer {
    * set false to halve the advertised `tools/list` surface.
    */
   private readonly emitLegacyTools: boolean;
+  /**
+   * Whether the `coding_decision` tool should appear in `tools/list`. Gated on
+   * `codingKnowledge.enabled && codingKnowledge.decisionRecords` (issue #1548
+   * Track A PR 2, rule 39). When false the tools array is byte-identical to
+   * pre-feature.
+   */
+  private readonly codingDecisionVisible: boolean;
 
   constructor(
     private readonly service: EngramAccessService,
@@ -322,11 +332,13 @@ export class EngramMcpServer {
       citationsEnabled?: boolean;
       citationsAutoDetect?: boolean;
       emitLegacyTools?: boolean;
+      codingDecisionVisible?: boolean;
     } = {},
   ) {
     this.citationsEnabled = options.citationsEnabled === true;
     this.citationsAutoDetect = options.citationsAutoDetect !== false;
     this.emitLegacyTools = options.emitLegacyTools !== false;
+    this.codingDecisionVisible = options.codingDecisionVisible === true;
     this.authenticatedPrincipal =
       options.principal?.trim() ||
       readEnvVar("OPENCLAW_ENGRAM_ACCESS_PRINCIPAL")?.trim() ||
@@ -1946,6 +1958,47 @@ export class EngramMcpServer {
         },
       },
     ].flatMap((tool) => withToolAliases(tool, this.emitLegacyTools));
+    if (this.codingDecisionVisible) {
+      const codingDecisionTools = withToolAliases(
+        {
+          name: "engram.coding_decision",
+          description:
+            "List, get, record, or supersede decision records in the session's coding namespace (issue #1548 Track A). Subcommands: list, get, record, supersede.",
+          inputSchema: {
+            type: "object",
+            properties: {
+              subcommand: {
+                type: "string",
+                enum: ["list", "get", "record", "supersede"],
+                description: "Which decision-record operation to run.",
+              },
+              sessionKey: { type: "string", description: "Session identifier whose coding context scopes the operation." },
+              namespace: { type: "string", description: "Optional explicit namespace (overrides coding-context overlay)." },
+              id: { type: "string", description: "Decision record id (required for get and supersede)." },
+              title: { type: "string", description: "Decision title (required for record and supersede)." },
+              status: {
+                type: "string",
+                enum: ["proposed", "accepted", "superseded", "rejected"],
+                description: "Decision status (record only; defaults to proposed).",
+              },
+              context: { type: "string", description: "Context/background for the decision." },
+              decision: { type: "string", description: "The decision itself (required for record and supersede)." },
+              consequences: { type: "string", description: "Consequences of the decision." },
+              entityRefs: {
+                type: "array",
+                items: { type: "string" },
+                description: "Entity references the decision relates to.",
+              },
+              supersedesId: { type: "string", description: "Id of the record this decision supersedes (supersede only)." },
+            },
+            required: ["subcommand"],
+            additionalProperties: false,
+          },
+        },
+        this.emitLegacyTools,
+      );
+      this.tools = [...this.tools, ...codingDecisionTools];
+    }
   }
 
   /** Get clientInfo for a specific MCP session. Returns undefined for non-MCP requests. */