@pepys/ingest 0.1.0 → 0.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pepys/ingest",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "description": "pepys ingest: the receive routes, the fact log and the blob store",
   "license": "MIT",
   "repository": {
@@ -22,8 +22,8 @@
   },
   "dependencies": {
     "@impz/client": "6.12.0",
-    "@pepys/kit": "0.1.0",
-    "@pepys/schema": "0.1.0",
+    "@pepys/kit": "0.2.0",
+    "@pepys/schema": "0.2.0",
     "@s2-dev/streamstore": "0.30.0",
     "@zgeoff/std": "0.4.3",
     "tiny-invariant": "1.3.3",

package/src/pull/make-pull-runner.ts

@@ -32,9 +32,9 @@ interface PullRunnerDependencies {
   resyncMarks: ResyncMarks;
 }
 
-// One run of one pull path: the pull, the envelope check, the appends, and then, for a resync pull,
-// the removal of every mark it was given. A pull's fact is noted in no mark: a resynced fact ties
-// with the body it replaces, and would mark its entity again on every run.
+// For a resync pull, removes each mark it was given that no tie set again since. A pull's fact is
+// noted in no mark: a resynced fact ties with the body it replaces, and would mark its entity again
+// on every run.
 export function makePullRunner(dependencies: PullRunnerDependencies): PullRunner {
   return async (sourceName, pathName, cursor) => {
     const source = dependencies.deployment.sources.find(
@@ -85,7 +85,7 @@ export function makePullRunner(dependencies: PullRunnerDependencies): PullRunner
     }
 
     for (const mark of marks) {
-      dependencies.resyncMarks.remove(source.name, mark.entity);
+      dependencies.resyncMarks.remove(source.name, mark);
     }
 
     return {

package/src/receive/resync-marks/create-resync-marks.ts

@@ -8,7 +8,9 @@ import type { LastBody, StoredFactTime } from './collect-tied-entities.ts';
 // it. In memory only, so a restart loses them.
 export interface ResyncMarks {
   list: (source: string) => readonly ResyncMark[];
-  remove: (source: string, entity: EntityID) => void;
+
+  // A mark set again while the pull ran is a new object, so it stays for the next pull.
+  remove: (source: string, mark: ResyncMark) => void;
 }
 
 // What the receive route of one source tells the marks.
@@ -20,8 +22,7 @@ export interface SourceMarks {
 interface SourceState {
   lastBodies: Map<EntityID, LastBody>;
 
-  // each marked entity, with the locator of the last fact stored for it
-  marks: Map<EntityID, string | undefined>;
+  marks: Map<EntityID, ResyncMark>;
 }
 
 export interface ResyncMarkStore {
@@ -53,10 +54,13 @@ export function createResyncMarks(options: ResyncMarksOptions): ResyncMarkStore
 
   return {
     marks: {
-      list: (source) =>
-        [...(sources.get(source)?.marks ?? [])].map(([entity, locator]) => ({ entity, locator })),
-      remove: (source, entity) => {
-        sources.get(source)?.marks.delete(entity);
+      list: (source) => [...(sources.get(source)?.marks.values() ?? [])],
+      remove: (source, mark) => {
+        const marks = sources.get(source)?.marks;
+
+        if (marks?.get(mark.entity) === mark) {
+          marks.delete(mark.entity);
+        }
       },
     },
     createSourceMarks: (source) => {
@@ -82,15 +86,19 @@ export function createResyncMarks(options: ResyncMarksOptions): ResyncMarkStore
   };
 }
 
-// The locator comes from the last body stored for the entity, which the caller has noted first.
+// The locator and digest come from the last body stored for the entity, which the caller has noted
+// first.
+// Each call makes a new mark object, which `remove` tells apart from the one a pull was given.
 function addMarks(
   state: SourceState,
   entities: readonly EntityID[],
   options: ResyncMarksOptions,
 ): void {
   for (const entity of entities) {
+    const last = state.lastBodies.get(entity);
+
     state.marks.delete(entity);
-    state.marks.set(entity, state.lastBodies.get(entity)?.locator);
+    state.marks.set(entity, { entity, locator: last?.locator, lastBodyDigest: last?.digest });
   }
 
   trimToCapacity(state.marks, options.capacity);
@@ -98,7 +106,7 @@ function addMarks(
 
 // drops the oldest entries first: a Map and a Set keep insertion order
 function trimToCapacity(
-  entries: Map<EntityID, LastBody> | Map<EntityID, string | undefined>,
+  entries: Map<EntityID, LastBody> | Map<EntityID, ResyncMark>,
   capacity: number,
 ): void {
   for (const key of entries.keys()) {