Merge pull request #7 from ThirdKeyAI/chore/drop-node18

ci: drop Node 18 from matrix; tighten engines to >=20

Author: jaschadub

.github/workflows/ci.yml

@@ -17,7 +17,9 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node-version: ["18.x", "20.x", "22.x"]
+        # Vitest 3.x + Vite 5 (pure-ESM) require Node 20+; Node 18 breaks
+        # on `require()` of vite from vitest's CJS loader.
+        node-version: ["20.x", "22.x"]
 
     steps:
       - uses: actions/checkout@v4

package.json

@@ -47,7 +47,7 @@
     "zod": "^3.22.0"
   },
   "engines": {
-    "node": ">=16.0.0"
+    "node": ">=20.0.0"
   },
   "repository": {
     "type": "git",

packages/core/src/memory/__tests__/InMemoryStore.test.ts

@@ -60,11 +60,17 @@ describe('InMemoryStore', () => {
 
       await store.store(memory);
       const retrieved = await store.get('test-1');
-      
-      expect(retrieved).toEqual({
+
+      // `get()` calls `recordAccess()`, which bumps accessCount AND refreshes
+      // the timestamp — assert the stable fields and verify timestamp is
+      // at or after the stored time, rather than deep-equal on the whole
+      // object (which used to flake when retrieval landed in the same ms).
+      expect(retrieved).toMatchObject({
         ...memory,
-        accessCount: 1, // Access count should be incremented on retrieval
+        accessCount: 1,
+        timestamp: expect.any(Date),
       });
+      expect(retrieved!.timestamp.getTime()).toBeGreaterThanOrEqual(memory.timestamp.getTime());
     });
 
     it('should create copy of stored memory', async () => {