@noetic-tools/core 1.2.0 → 1.2.1

package/dist/harness/agent-harness.d.ts

@@ -105,6 +105,13 @@ export declare class AgentHarness<TParams extends Record<string, unknown> = Reco
     readonly layerStateStore: LayerStateStore;
     /** Per-harness memoization cache for `recallMode: 'eventual'` layers. */
     readonly recallCache: RecallCache;
+    /**
+     * Execution ids whose layer `init` hooks have already run, so repeated/nested
+     * `run()` calls and the session turn path never re-init (which would clobber
+     * accumulated in-memory state by re-hydrating from storage). Keyed by
+     * `ctx.id` (the layer-state store's executionId).
+     */
+    private readonly initializedExecutions;
     readonly traceExporter: TraceExporter;
     /**
      * Long-lived shared cwd state. The same reference is seeded into every
@@ -143,6 +150,14 @@ export declare class AgentHarness<TParams extends Record<string, unknown> = Reco
     /** Run all layer `init` hooks before the first step executes. Per spec 11,
      *  init MUST complete before any recall fires so layer state is populated. */
     private initAndRun;
+    /**
+     * Idempotently run layer `init` hooks for `ctx` (keyed by `ctx.id`). Init
+     * MUST complete before any recall/store fires so init-bearing layers are not
+     * treated as disabled. Guarded so nested/repeated `run()` calls and the
+     * session turn path never re-init — re-init would re-hydrate from storage and
+     * clobber accumulated in-memory state.
+     */
+    private ensureLayersInit;
     detachedSpawn<I, O>(s: Step<ContextMemory, I, O>, input: I, parentCtx: Context, overrides?: DetachedSpawnOverrides): DetachedHandle<O>;
     createContext(opts?: {
         parent?: Context;

package/dist/harness/agent-harness.js

@@ -110,6 +110,13 @@ export class AgentHarness {
     layerStateStore;
     /** Per-harness memoization cache for `recallMode: 'eventual'` layers. */
     recallCache;
+    /**
+     * Execution ids whose layer `init` hooks have already run, so repeated/nested
+     * `run()` calls and the session turn path never re-init (which would clobber
+     * accumulated in-memory state by re-hydrating from storage). Keyed by
+     * `ctx.id` (the layer-state store's executionId).
+     */
+    initializedExecutions = new Set();
     traceExporter;
     /**
      * Long-lived shared cwd state. The same reference is seeded into every
@@ -356,17 +363,36 @@ export class AgentHarness {
                 ...this.harnessTools,
             ]);
         }
+        // Hydrate/recall/persist memory on the bare `run()` path too (issue #48):
+        // configuring `memory` + `storage` must "just work" without going through
+        // `execute()`/`seedSessionHistory`. Idempotent — see `ensureLayersInit`.
+        await this.ensureLayersInit(ctx);
         return execute(s, input, ctx);
     }
     /** Run all layer `init` hooks before the first step executes. Per spec 11,
      *  init MUST complete before any recall fires so layer state is populated. */
     async initAndRun(s, input, ctx) {
+        await this.ensureLayersInit(ctx);
+        return execute(s, input, ctx);
+    }
+    /**
+     * Idempotently run layer `init` hooks for `ctx` (keyed by `ctx.id`). Init
+     * MUST complete before any recall/store fires so init-bearing layers are not
+     * treated as disabled. Guarded so nested/repeated `run()` calls and the
+     * session turn path never re-init — re-init would re-hydrate from storage and
+     * clobber accumulated in-memory state.
+     */
+    async ensureLayersInit(ctx) {
         const layers = ctx.layers;
         const storage = this.config.storage;
-        if (layers && layers.length > 0 && storage) {
-            await this.initLayers(layers, ctx, storage);
+        if (!layers || layers.length === 0 || !storage) {
+            return;
         }
-        return execute(s, input, ctx);
+        if (this.initializedExecutions.has(ctx.id)) {
+            return;
+        }
+        this.initializedExecutions.add(ctx.id);
+        await this.initLayers(layers, ctx, storage);
     }
     detachedSpawn(s, input, parentCtx, overrides) {
         return dispatchStepThroughAdapter(this, s, input, parentCtx, overrides);
@@ -538,6 +564,9 @@ export class AgentHarness {
         });
     }
     async disposeLayers(layers, ctx) {
+        // Drop the init guard so a deliberate dispose→run cycle re-hydrates from
+        // storage (disposeLayers also clears the layer-state store for this id).
+        this.initializedExecutions.delete(ctx.id);
         await disposeLayers({
             layers,
             ctx: this.toExecCtx(ctx),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@noetic-tools/core",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "type": "module",
   "description": "Step primitives, interpreter, runtime, and memory layer contract for Noetic agents.",
   "license": "Apache-2.0",