@ibgib/core-gib 0.1.60 → 0.1.61

package/src/sync/sync-saga-coordinator.mts

@@ -466,154 +466,158 @@ export class SyncSagaCoordinator {
             }));
             // #endregion set up peer observable for any domainPayloadsMap
 
-            // ...create/compose the Request Context itself...
-            const requestCtx = await this.createSyncSagaContext({
-                sagaFrame: currentFrame,
-                /**
-                 * init frame: empty
-                 * ack frame: possible push offers
-                 * delta frame: requested ibgibs or commit offer
-                 * commit frame: empty
-                 */
-                payloadIbGibsDomain: nextDomainIbGibs,
-                localSpace,
-                metaspace,
-                sessionIdentityAddr: peer.currentSessionIdentityAddr,
-                peer,
-            });
+            try {
+                // ...create/compose the Request Context itself...
+                const requestCtx = await this.createSyncSagaContext({
+                    sagaFrame: currentFrame,
+                    /**
+                     * init frame: empty
+                     * ack frame: possible push offers
+                     * delta frame: requested ibgibs or commit offer
+                     * commit frame: empty
+                     */
+                    payloadIbGibsDomain: nextDomainIbGibs,
+                    localSpace,
+                    metaspace,
+                    sessionIdentityAddr: peer.currentSessionIdentityAddr,
+                    peer,
+                });
 
-            // #region Log what we're sending
-            if (logalotControlDomain) {
-                const domainAddrs = nextDomainIbGibs.map(p => getIbGibAddr({ ibGib: p }));
-                console.log(`${lc}${lcControlDomain} SENDER TRANSMIT -> peer.witness (I: 5b0081803698770f0bf64992220b312)`);
-                console.log(`${lc}${lcControlDomain}   Context: ${getIbGibAddr({ ibGib: requestCtx })}`);
-                console.log(`${lc}${lcControlDomain}   Frame: ${getIbGibAddr({ ibGib: currentFrame })}`);
-                console.log(`${lc}${lcControlDomain}   DOMAIN Payloads (${domainAddrs.length}): ${domainAddrs.join(', ') || '(none)'}`);
-            }
-            // #endregion Log what we're sending
-
-            // update our saga listeners...
-            // if (logalot) { console.log(`${lc} transmitting... requestCtx: ${pretty(requestCtx)} (I: 8cf20817c66899abdb1e76df50356826)`); }
-            updates$.next(requestCtx); // spins off for saga UI updates
-
-            // ...And send the context.
-            peer.setOptionalOpts({ localSpace, localTempSpace: tempSpace, });
-            const responseCtx = await peer.witness(requestCtx);
-
-            // the send returned, but a peer can return a falsy responseCtx, if
-            // we just sent a commit to them and they're done. If so, there will
-            // necessarily be no payload to wait to receive
-            if (!responseCtx) {
-                if (currentFrame) {
-                    // Check for Commit (Peer silence expected)
-                    const msg = await getSyncSagaMessageFromFrame({ frameIbGib: currentFrame, space: localSpace });
-                    if (msg?.data?.stage === SyncStage.commit) {
-                        if (logalot) { console.log(`${lc} Sender sent Commit. Peer returned no response. Saga Complete. (I: 26f9ee073858ca78b8284753368b5226)`); }
-                        currentFrame = null;
-                        break;
+                // #region Log what we're sending
+                if (logalotControlDomain) {
+                    const domainAddrs = nextDomainIbGibs.map(p => getIbGibAddr({ ibGib: p }));
+                    console.log(`${lc}${lcControlDomain} SENDER TRANSMIT -> peer.witness (I: 5b0081803698770f0bf64992220b312)`);
+                    console.log(`${lc}${lcControlDomain}   Context: ${getIbGibAddr({ ibGib: requestCtx })}`);
+                    console.log(`${lc}${lcControlDomain}   Frame: ${getIbGibAddr({ ibGib: currentFrame })}`);
+                    console.log(`${lc}${lcControlDomain}   DOMAIN Payloads (${domainAddrs.length}): ${domainAddrs.join(', ') || '(none)'}`);
+                }
+                // #endregion Log what we're sending
+
+                // update our saga listeners...
+                // if (logalot) { console.log(`${lc} transmitting... requestCtx: ${pretty(requestCtx)} (I: 8cf20817c66899abdb1e76df50356826)`); }
+                updates$.next(requestCtx); // spins off for saga UI updates
+
+                // ...And send the context.
+                peer.setOptionalOpts({ localSpace, localTempSpace: tempSpace, });
+                const responseCtx = await peer.witness(requestCtx);
+
+                // the send returned, but a peer can return a falsy responseCtx, if
+                // we just sent a commit to them and they're done. If so, there will
+                // necessarily be no payload to wait to receive
+                if (!responseCtx) {
+                    if (currentFrame) {
+                        // Check for Commit (Peer silence expected)
+                        const msg = await getSyncSagaMessageFromFrame({ frameIbGib: currentFrame, space: localSpace });
+                        if (msg?.data?.stage === SyncStage.commit) {
+                            if (logalot) { console.log(`${lc} Sender sent Commit. Peer returned no response. Saga Complete. (I: 26f9ee073858ca78b8284753368b5226)`); }
+                            currentFrame = null;
+                            break;
+                        } else {
+                            throw new Error(`(UNEXPECTED) responseCtx falsy and currentFrame truthy, but we're not in the commit stage? This may be expected ultimately, but atow I am not seeing this as being expected. (E: cc34498962bd370deeff351fac939f26)`);
+                        }
                     } else {
-                        throw new Error(`(UNEXPECTED) responseCtx falsy and currentFrame truthy, but we're not in the commit stage? This may be expected ultimately, but atow I am not seeing this as being expected. (E: cc34498962bd370deeff351fac939f26)`);
+                        throw new Error(`(UNEXPECTED) no response and currentFrame falsy? (E: 8d1085ea2f28cfc3f9c922649864a826)`);
                     }
-                } else {
-                    throw new Error(`(UNEXPECTED) no response and currentFrame falsy? (E: 8d1085ea2f28cfc3f9c922649864a826)`);
                 }
-            }
 
-            // at this point, we did indeed receive a response to analyze. BUT!
-            // we have only necessarily received the response context ibgib.
-            // if there were payloads expected to be transferred, they may not
-            // be done yet.
+                // at this point, we did indeed receive a response to analyze. BUT!
+                // we have only necessarily received the response context ibgib.
+                // if there were payloads expected to be transferred, they may not
+                // be done yet.
 
-            if (!responseCtx.data) { throw new Error(`(UNEXPECTED) responseCtx.data falsy? (E: a969992bae53ab18a827ec58aec15826)`); }
-            updates$.next(responseCtx); // spins off for saga UI updating
+                if (!responseCtx.data) { throw new Error(`(UNEXPECTED) responseCtx.data falsy? (E: a969992bae53ab18a827ec58aec15826)`); }
+                updates$.next(responseCtx); // spins off for saga UI updating
 
-            // immediately validate context/saga frame (but not payloads because
-            // we may not have those yet)
-            const contextAndSagaFrameValidationErrors =
-                await validateContextAndSagaFrame({ context: responseCtx });
-            if (contextAndSagaFrameValidationErrors.length > 0) { throw new Error(`contextAndSagaFrameValidationErrors: ${contextAndSagaFrameValidationErrors} (E: 6eebe8e7fa437c00a8cde3ada3c66826)`); }
-
-            // Turn-by-turn continuation & identity checks
-            const returnContextErrors = await this.validateReturnContext({
-                requestCtx,
-                responseCtx,
-                localSpace,
-            });
-            if (returnContextErrors.length > 0) {
-                throw new Error(`validateReturnContext errors: ${returnContextErrors.join(', ')} (E: cb8a023b9d0728cceb09fa3da0bb8226)`);
-            }
+                // immediately validate context/saga frame (but not payloads because
+                // we may not have those yet)
+                const contextAndSagaFrameValidationErrors =
+                    await validateContextAndSagaFrame({ context: responseCtx });
+                if (contextAndSagaFrameValidationErrors.length > 0) { throw new Error(`contextAndSagaFrameValidationErrors: ${contextAndSagaFrameValidationErrors} (E: 6eebe8e7fa437c00a8cde3ada3c66826)`); }
 
-            // Extract expected domain addresses from response context
-            const responsePayloadAddrsDomain = responseCtx.data[SYNC_SAGA_PAYLOAD_ADDRS_DOMAIN] as string[] || [];
-
-            // Poll for them if needed. see above jsdocs for domainPayloadsMap
-            if (responsePayloadAddrsDomain.length > 0) {
-                responseCtx.payloadIbGibsDomain = await this.pollForDomainPayloads({
-                    expectedAddrs: responsePayloadAddrsDomain,
-                    pollIntervalMs: 20, // relatively arbitrary right now
-                    domainPayloadsMap,
-                    tempSpace,
+                // Turn-by-turn continuation & identity checks
+                const returnContextErrors = await this.validateReturnContext({
+                    requestCtx,
+                    responseCtx,
+                    localSpace,
                 });
-            }
-
-            // #region Log what we received back
-            if (!responseCtx.sagaFrame) { throw new Error(`(UNEXPECTED) responseCtx.sagaFrame falsy? the Peer should have set this when it got the response back from the remote. (E: e650adadf9a2063ec6764a1e31d3d826)`); }
-            if (logalotControlDomain) {
-                const responseControlAddrs = responseCtx.data?.['@payloadAddrsControl'] as string[] || [];
-                console.log(`${lc}${lcControlDomain} SENDER RECEIVED <- peer.witness (I: 3dc76a9744d89a4fc3e2f076c2be4826)`);
-                console.log(`${lc}${lcControlDomain}   Response Context: ${getIbGibAddr({ ibGib: responseCtx })}`);
-                console.log(`${lc}${lcControlDomain}   Response Saga Frame: ${getIbGibAddr({ ibGib: responseCtx.sagaFrame })}`);
-                console.log(`${lc}${lcControlDomain}   CONTROL Payloads (${responseControlAddrs.length}): ${responseControlAddrs.join(', ') || '(none)'}`);
-                console.log(`${lc}${lcControlDomain}   DOMAIN Payloads (${responsePayloadAddrsDomain.length}): ${responsePayloadAddrsDomain.join(', ') || '(none)'}`);
-            }
-            // #endregion Log what we received back
+                if (returnContextErrors.length > 0) {
+                    throw new Error(`validateReturnContext errors: ${returnContextErrors.join(', ')} (E: cb8a023b9d0728cceb09fa3da0bb8226)`);
+                }
 
-            // at this point, we have received the context AND **all** of the
-            // domain payloads (if applicable), so we are ready to do the next
-            // iteration in the saga loop
+                // Extract expected domain addresses from response context
+                const responsePayloadAddrsDomain = responseCtx.data[SYNC_SAGA_PAYLOAD_ADDRS_DOMAIN] as string[] || [];
 
-            // this is the part that drives the FSM forward, i.e., when we
-            // evolve the sync saga ibgib itself to the next frame if we aren't
-            // finished/errored out.
-            const contextResult = await this.handleResponseSagaContext({
-                sagaContext: responseCtx,
-                initDomainGraph,
-                mySpace: localSpace,
-                myTempSpace: tempSpace,
-                metaspace,
-            });
+                // Poll for them if needed. see above jsdocs for domainPayloadsMap
+                if (responsePayloadAddrsDomain.length > 0) {
+                    responseCtx.payloadIbGibsDomain = await this.pollForDomainPayloads({
+                        expectedAddrs: responsePayloadAddrsDomain,
+                        pollIntervalMs: 20, // relatively arbitrary right now
+                        domainPayloadsMap,
+                        tempSpace,
+                    });
+                }
 
-            if (!contextResult) {
-                // should this ever hit?
-                console.error(`${lc} NAG ERROR (DOES NOT THROW): does this ever hit now? (E: e04d02efc2a8e72a88b79f1f0f95ca26)`);
-                break;
-            } else if (contextResult.nextFrameInfo?.sagaComplete) {
-                if (logalot) { console.log(`${lc} Handler returned null (Saga End). (I: 123bf9e7dca8886de72553a8d4f29e26)`); }
-                break;
-            }
+                // #region Log what we received back
+                if (!responseCtx.sagaFrame) { throw new Error(`(UNEXPECTED) responseCtx.sagaFrame falsy? the Peer should have set this when it got the response back from the remote. (E: e650adadf9a2063ec6764a1e31d3d826)`); }
+                if (logalotControlDomain) {
+                    const responseControlAddrs = responseCtx.data?.['@payloadAddrsControl'] as string[] || [];
+                    console.log(`${lc}${lcControlDomain} SENDER RECEIVED <- peer.witness (I: 3dc76a9744d89a4fc3e2f076c2be4826)`);
+                    console.log(`${lc}${lcControlDomain}   Response Context: ${getIbGibAddr({ ibGib: responseCtx })}`);
+                    console.log(`${lc}${lcControlDomain}   Response Saga Frame: ${getIbGibAddr({ ibGib: responseCtx.sagaFrame })}`);
+                    console.log(`${lc}${lcControlDomain}   CONTROL Payloads (${responseControlAddrs.length}): ${responseControlAddrs.join(', ') || '(none)'}`);
+                    console.log(`${lc}${lcControlDomain}   DOMAIN Payloads (${responsePayloadAddrsDomain.length}): ${responsePayloadAddrsDomain.join(', ') || '(none)'}`);
+                }
+                // #endregion Log what we received back
+
+                // at this point, we have received the context AND **all** of the
+                // domain payloads (if applicable), so we are ready to do the next
+                // iteration in the saga loop
+
+                // this is the part that drives the FSM forward, i.e., when we
+                // evolve the sync saga ibgib itself to the next frame if we aren't
+                // finished/errored out.
+                const contextResult = await this.handleResponseSagaContext({
+                    sagaContext: responseCtx,
+                    initDomainGraph,
+                    mySpace: localSpace,
+                    myTempSpace: tempSpace,
+                    metaspace,
+                });
 
-            // #region error conditions throw
-            if (contextResult.errorMsg) {
-                throw new Error(`Couldn't handle response saga context. errorMsg: ${contextResult.errorMsg} (E: c948e81d513b2a0eb8b8afa878edc626)`);
-            } else if (!contextResult.nextFrameInfo) {
-                throw new Error(`(UNEXPECTED) contextResult.nextFrameInfo falsy? (E: c287a82e823e662a77923278e2418826)`);
-            }
-            // #endregion error conditions throw
+                if (!contextResult) {
+                    // should this ever hit?
+                    console.error(`${lc} NAG ERROR (DOES NOT THROW): does this ever hit now? (E: e04d02efc2a8e72a88b79f1f0f95ca26)`);
+                    break;
+                } else if (contextResult.nextFrameInfo?.sagaComplete) {
+                    if (logalot) { console.log(`${lc} Handler returned null (Saga End). (I: 123bf9e7dca8886de72553a8d4f29e26)`); }
+                    break;
+                }
 
-            // we have another frame to process and send out, with possibly
-            // payload domain ibgibs as well
-            const { frame, payloadIbGibsDomain, } = contextResult.nextFrameInfo;
-            currentFrame = frame;
-            nextDomainIbGibs = [...(payloadIbGibsDomain || [])];
-
-            // #region Log handler output for next iteration
-            if (logalotControlDomain) {
-                const handlerDomainAddrs = nextDomainIbGibs.map(p => getIbGibAddr({ ibGib: p }));
-                console.log(`${lc}${lcControlDomain} HANDLER RESULT -> next iteration (I: 6b0d88c4c28857ccd812381515bd7826)`);
-                console.log(`${lc}${lcControlDomain}   Next Frame: ${getIbGibAddr({ ibGib: currentFrame })}`);
-                console.log(`${lc}${lcControlDomain}   DOMAIN for next (${handlerDomainAddrs.length}): ${handlerDomainAddrs.join(', ') || '(none)'}`);
+                // #region error conditions throw
+                if (contextResult.errorMsg) {
+                    throw new Error(`Couldn't handle response saga context. errorMsg: ${contextResult.errorMsg} (E: c948e81d513b2a0eb8b8afa878edc626)`);
+                } else if (!contextResult.nextFrameInfo) {
+                    throw new Error(`(UNEXPECTED) contextResult.nextFrameInfo falsy? (E: c287a82e823e662a77923278e2418826)`);
+                }
+                // #endregion error conditions throw
+
+                // we have another frame to process and send out, with possibly
+                // payload domain ibgibs as well
+                const { frame, payloadIbGibsDomain, } = contextResult.nextFrameInfo;
+                currentFrame = frame;
+                nextDomainIbGibs = [...(payloadIbGibsDomain || [])];
+
+                // #region Log handler output for next iteration
+                if (logalotControlDomain) {
+                    const handlerDomainAddrs = nextDomainIbGibs.map(p => getIbGibAddr({ ibGib: p }));
+                    console.log(`${lc}${lcControlDomain} HANDLER RESULT -> next iteration (I: 6b0d88c4c28857ccd812381515bd7826)`);
+                    console.log(`${lc}${lcControlDomain}   Next Frame: ${getIbGibAddr({ ibGib: currentFrame })}`);
+                    console.log(`${lc}${lcControlDomain}   DOMAIN for next (${handlerDomainAddrs.length}): ${handlerDomainAddrs.join(', ') || '(none)'}`);
+                }
+                // #endregion Log handler output for next iteration
+            } finally {
+                await subscription.unsubscribe();
             }
-            // #endregion Log handler output for next iteration
         }
     }
 
@@ -1033,7 +1037,8 @@ export class SyncSagaCoordinator {
 
             let resultDomainPayloads: IbGib_V1[] = [];
 
-            let pending = [...expectedAddrs];
+            const uniqueExpectedAddrs = [...new Set(expectedAddrs)];
+            let pending = [...uniqueExpectedAddrs];
             const start = Date.now();
             /**
              * This needs
@@ -1067,8 +1072,8 @@ export class SyncSagaCoordinator {
                 if (pending.length > 0) { await delay(pollIntervalMs); }
             }
 
-            if (expectedAddrs.length !== resultDomainPayloads.length) {
-                throw new Error(`(UNEXPECTED) expectedAddrs.length !== resultDomainPayloads.length? at this point, we expect all of the payload ibgibs to have been received. (E: 03749a7478c4b8b28bfc86951887a826)`);
+            if (uniqueExpectedAddrs.length !== resultDomainPayloads.length) {
+                throw new Error(`(UNEXPECTED) uniqueExpectedAddrs.length !== resultDomainPayloads.length? at this point, we expect all of the payload ibgibs to have been received. (E: 03749a7478c4b8b28bfc86951887a826)`);
             }
 
             return resultDomainPayloads;
@@ -1662,12 +1667,10 @@ export class SyncSagaCoordinator {
              */
             const outgoingPayloadIbGibsDomain_conflicts: IbGib_V1[] = [];
 
+            const activeConflicts: SyncSagaConflictInfo[] = [];
             if (conflicts.length > 0) {
                 if (logalot) { console.log(`${lc} [CONFLICT DEBUG] Processing ${conflicts.length} non-terminal conflicts`); }
 
-                // we will need to resolve conflicts, so no commit yet
-                proposeCommit = false;
-
                 for (const conflict of conflicts) {
                     const {
                         receiverTipAddr,
@@ -1739,11 +1742,24 @@ export class SyncSagaCoordinator {
                         const deltaPayloadIbGib = deltaGraph[deltaPayloadAddr];
                         outgoingPayloadIbGibsDomain_conflicts.push(deltaPayloadIbGib);
                     });
-                    if (conflict.accretivePayloadAddrs) { throw new Error(`(UNEXPECTED) conflict.accretivePayloadAddrs already truthy? This is expected to be falsy at this point in the sync. (E: 6b6b1846779867849cbde208f4374326)`); }
-                    conflict.accretivePayloadAddrs = accretivePayloadAddrs;
+
+                    const isFastForward = latestCommonFrameAddr === receiverTipAddr;
+                    if (isFastForward) {
+                        if (logalot) { console.log(`${lc} [CONFLICT DEBUG] TJP ${tjpAddr}: Fast-forward detected (receiver behind sender). Excluding from active conflicts and sending delta payload.`); }
+                    } else {
+                        if (conflict.accretivePayloadAddrs) { throw new Error(`(UNEXPECTED) conflict.accretivePayloadAddrs already truthy? This is expected to be falsy at this point in the sync. (E: 6b6b1846779867849cbde208f4374326)`); }
+                        conflict.accretivePayloadAddrs = accretivePayloadAddrs;
+                        activeConflicts.push(conflict);
+                    }
+                }
+
+                if (activeConflicts.length > 0) {
+                    proposeCommit = false;
+                } else {
+                    proposeCommit = true;
                 }
 
-                if (logalot) { console.log(`${lc} [CONFLICT DEBUG] Finished processing ${conflicts.length} conflicts. outgoingPayloadIbGibsDomain_conflicts.length (total outgoing payload ibgibs for ALL conflicts): ${outgoingPayloadIbGibsDomain_conflicts.length}`); }
+                if (logalot) { console.log(`${lc} [CONFLICT DEBUG] Finished processing ${conflicts.length} conflicts. Active (divergent) conflicts: ${activeConflicts.length}. outgoingPayloadIbGibsDomain_conflicts.length (total outgoing payload ibgibs for ALL conflicts): ${outgoingPayloadIbGibsDomain_conflicts.length}`); }
             } else {
                 if (logalot) { console.log(`${lc} [CONFLICT DEBUG] No optimistic conflicts to process`); }
             }
@@ -1783,7 +1799,7 @@ export class SyncSagaCoordinator {
                 /**
                  * these are the modified conflicts with additional information.
                  */
-                conflicts,
+                conflicts: activeConflicts,
                 /**
                  * we're sending these domain ibgibs as payload (they were
                  * requested by receiver and/or they are being sent as delta

package/src/sync/sync-withid.pingpong.respec.mts

@@ -9,7 +9,7 @@
  */
 
 import {
-    respecfully, ifWeMight, iReckon,
+    respecfully, ifWeMight, iReckon, ifWe,
 } from '@ibgib/helper-gib/dist/respec-gib/respec-gib.mjs';
 const maam = `[${import.meta.url}]`, sir = maam;
 import { clone, delay, extractErrorMsg } from '@ibgib/helper-gib/dist/helpers/utils-helper.mjs';
@@ -154,12 +154,12 @@ await respecfully(sir, `Test Phase 3A: Ping Pong Sync with Identity`, async () =
         syncError = error;
     }
 
-    await ifWeMight(sir, 'completes the entire sync saga successfully', async () => {
+    await ifWe(sir, 'completes the entire sync saga successfully', async () => {
         const errorMsg = syncError ? extractErrorMsg(syncError) : '';
         iReckon(sir, syncError).asTo(`syncError: ${errorMsg}`).isGonnaBeFalsy();
     });
 
-    await ifWeMight(sir, 'alpha dep graph matches on source and dest', async () => {
+    await ifWe(sir, 'alpha dep graph matches on source and dest', async () => {
         const resGetDest = await getFromSpace({ space: destSpace, addr: xStoneAddr });
         iReckon(sir, resGetDest.success && resGetDest.ibGibs?.length === 1).asTo('xStone exists in destSpace').isGonnaBeTrue();
         if (resGetDest.success && resGetDest.ibGibs?.[0]) {
@@ -171,7 +171,7 @@ await respecfully(sir, `Test Phase 3A: Ping Pong Sync with Identity`, async () =
         }
     });
 
-    await ifWeMight(sir, 'sessionIdentity evolved the expected number of times', async () => {
+    await ifWe(sir, 'sessionIdentity evolved the expected number of times', async () => {
         const latestS = peer.currentSessionIdentity;
         iReckon(sir, latestS).asTo('latest session identity exists on peer').isGonnaBeTruthy();
         if (latestS) {
@@ -179,7 +179,7 @@ await respecfully(sir, `Test Phase 3A: Ping Pong Sync with Identity`, async () =
         }
     });
 
-    await ifWeMight(sir, 'sender durable space has I (evolved frame)', async () => {
+    await ifWe(sir, 'sender durable space has I (evolved frame)', async () => {
         const latestSenderIdentityAddr = await metaspace.getLatestAddr({
             addr: getIbGibAddr({ ibGib: senderIdentity }),
             space: sourceSpace,
@@ -195,7 +195,7 @@ await respecfully(sir, `Test Phase 3A: Ping Pong Sync with Identity`, async () =
         }
     });
 
-    await ifWeMight(sir, 'receiver durable space has I (evolved frame)', async () => {
+    await ifWe(sir, 'receiver durable space has I (evolved frame)', async () => {
         const latestSenderIdentityAddr = await metaspace.getLatestAddr({
             addr: getIbGibAddr({ ibGib: senderIdentity }),
             space: sourceSpace,
@@ -211,7 +211,7 @@ await respecfully(sir, `Test Phase 3A: Ping Pong Sync with Identity`, async () =
         }
     });
 
-    await ifWeMight(sir, 'sender durable space has full S dep graph', async () => {
+    await ifWe(sir, 'sender durable space has full S dep graph', async () => {
         const latestS = peer.currentSessionIdentity;
         iReckon(sir, latestS).asTo('latestS exists').isGonnaBeTruthy();
         if (latestS) {
@@ -221,7 +221,7 @@ await respecfully(sir, `Test Phase 3A: Ping Pong Sync with Identity`, async () =
         }
     });
 
-    await ifWeMight(sir, 'receiver durable space has full S dep graph', async () => {
+    await ifWe(sir, 'receiver durable space has full S dep graph', async () => {
         const latestS = peer.currentSessionIdentity;
         iReckon(sir, latestS).asTo('latestS exists').isGonnaBeTruthy();
         if (latestS) {

package/src/timeline/timeline-api.mts

@@ -293,11 +293,7 @@ export async function appendToTimeline({
                 });
             }
 
-            // should we publish within the lock or not within the lock?
-            // should we await the publish or spin it off
-            // right now I have this spin off, inside the lock
-            // await metaspace.registerNewIbGib({ ibGib: newTimelineIbGib, space });
-            metaspace.registerNewIbGib({ ibGib: newTimelineIbGib, space }); // spin off
+            await metaspace.registerNewIbGib({ ibGib: newTimelineIbGib, space }); // already spins off
 
             return newTimelineIbGib;
         };
@@ -314,12 +310,6 @@ export async function appendToTimeline({
                 fn,
             });
 
-        // should we publish within the lock or not within the lock?
-        // should we await the publish or spin it off
-        // right now I have this spin off, inside the lock
-        // await metaspace.registerNewIbGib({ ibGib: newTimelineIbGib, space });
-        // metaspace.registerNewIbGib({ ibGib: newTimelineIbGib, space }); // spin off
-
         return newTimelineIbGib;
     } catch (error) {
         console.error(`${lc} ${extractErrorMsg(error)}`);
@@ -691,10 +681,6 @@ export async function mut8Timeline<TData extends IbGibData_V1 = any>({
                 });
             }
 
-            // should we publish within the lock or not within the lock?
-            // should we await the publish or spin it off
-            // right now I have this spin off, inside the lock
-            // await metaspace.registerNewIbGib({ ibGib: newTimelineIbGib, space });
             // we await the register call because this spins off the publish events (we are assuming implementation details here)
             await metaspace.registerNewIbGib({ ibGib: newTimelineIbGib, space }); // already spins off
 

package/src/timeline/timeline-api.respec.mts

@@ -186,14 +186,14 @@ respecfully(sir, `[timeline-api]`, async () => {
              * I'm getting output (warnings?) that the transform ibgib addr
              * already exists. This stems from the fact that dna/transform
              * ibgibs do not have timestamps or anything, they simply are
-             * memoized args to the transform functions. So we need something 
+             * memoized args to the transform functions. So we need something
              * unique _in the arg itself_, which in this case means in the
              * `dataToAddOrPatch` field.
-             * 
+             *
              * Note, however, that it is expected for the original `fork`
              * transform that created tInitial to be common and for it to
              * already have this fork dna ibgib. But this shouldn't happen for
-             * each mut8 transform. 
+             * each mut8 transform.
              */
             const uniqueIdsForUniqueTransforms: string[] = [];
             for (let i = 0; i < N; i++) {

package/src/witness/space/reconciliation-space/reconciliation-space-base.mts

@@ -689,52 +689,6 @@ export abstract class ReconciliationSpaceBase<
         }
     }
 
-    // protected async applyTransforms({
-    //     src,
-    //     createdIbGibs_Running,
-    //     dnaAddrsToApplyToStoreVersion,
-    //     allLocalIbGibs,
-    // }: {
-    //     src: TIbGib,
-    //     createdIbGibs_Running: TIbGib[],
-    //     dnaAddrsToApplyToStoreVersion: IbGibAddr[],
-    //     allLocalIbGibs: TIbGib[],
-    // }): Promise<TIbGib> {
-    //     const lc = `${this.lc}[${this.applyTransforms.name}]`;
-    //     try {
-    //         if (logalot) { console.log(`${lc} starting...`); }
-
-    //         let currentSrc = src;
-    //         // find the dna ibgibs from the local ibgibs that were passed in.
-    //         const allLocalIbGibsMap = new Map<IbGibAddr, TIbGib>();
-    //         allLocalIbGibs.forEach(x => allLocalIbGibsMap.set(getIbGibAddr({ ibGib: x }), x));
-
-    //         for (const dnaAddr of dnaAddrsToApplyToStoreVersion) {
-    //             const dnaIbGib = allLocalIbGibsMap.get(dnaAddr);
-    //             if (!dnaIbGib) { throw new Error(`(UNEXPECTED) dnaIbGib not found in allLocalIbGibs. addr: ${dnaAddr} (E: 106402377a644e5989710f606e9323ff)`); }
-    //             const resMut8 = await mut8({
-    //                 src: currentSrc,
-    //                 mut8Ib: dnaIbGib.ib,
-    //                 dataToAddOrPatch: { ...dnaIbGib.data }, // this is a transform, so we just care about data and ib
-    //                 noTimestamp: true,
-    //             });
-
-    //             currentSrc = resMut8.newIbGib as TIbGib;
-    //             if (resMut8.intermediateIbGibs) {
-    //                 resMut8.intermediateIbGibs.forEach(x => createdIbGibs_Running.push(x as TIbGib));
-    //             }
-    //             createdIbGibs_Running.push(currentSrc);
-    //         }
-
-    //         return currentSrc;
-    //     } catch (error) {
-    //         const emsg = `${lc} ${extractErrorMsg(error)}`;
-    //         console.error(emsg);
-    //         throw error;
-    //     }
-    // }
-
-
     protected async routeAndDoCommand<TCmdModifier extends SyncSpaceOptionsCmdModifier = SyncSpaceOptionsCmdModifier>({
         cmd,
         cmdModifiers,

package/src/witness/space/reconciliation-space/reconciliation-space-helper.mts

@@ -135,13 +135,14 @@ export async function applyTransforms({
             const dnaIbGib_IntermediateArray =
                 allLocalIbGibs.filter(x => getIbGibAddr({ ibGib: x }) === dnaAddrToApply);
             let dnaIbGib: IbGib_V1;
-            if (dnaIbGib_IntermediateArray.length === 1) {
+            if (dnaIbGib_IntermediateArray.length > 0) {
                 dnaIbGib = dnaIbGib_IntermediateArray[0];
-            } else if (dnaIbGib_IntermediateArray.length === 0) {
-                throw new Error(`dna ibGib not found in supplied allLocalIbGibs. dnaAddr: ${dnaAddrToApply}. (E: 7f56826852cf48a79ab8af16bf27e284)`);
+                // NOTE: length > 1 is NOT an error condition because mut8/rel8
+                // transforms do not have uuids or timestamps. So if the same
+                // mut8/rel8 call occurs, e.g., changing the same field on the
+                // same source, then we could have a duplicate dna transform.
             } else {
-                // more than one ibgib with the dna address?
-                throw new Error(`More than one ibGib in allLocalIbGibs with the dna address of ${dnaAddrToApply}? (E: a726134f4cc14a4fb2ed2d39d22af17c)(UNEXPECTED)`);
+                throw new Error(`dna ibGib not found in supplied allLocalIbGibs. dnaAddr: ${dnaAddrToApply}. (E: 7f56826852cf48a79ab8af16bf27e284)`);
             }
 
             // we have our dna to apply, so do so against the incoming src

package/tsconfig.json

@@ -9,9 +9,14 @@
     "outDir": "./dist", /* Redirect output structure to the directory. */
     "module": "ESNext", /* https://www.typescriptlang.org/docs/handbook/esm-node.html */
   },
+  "include": [
+    "src/**/*.ts",
+    "src/**/*.mts",
+    "src/keystone/policy/profiles/*.json"
+  ],
   "exclude": [
     "./node_modules",
     "**/*.spec.mts",
     "./dist/**/*"
   ]
-}
+}