querysub 0.482.0 → 0.484.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "querysub",
-  "version": "0.482.0",
+  "version": "0.484.0",
   "main": "index.js",
   "license": "MIT",
   "note1": "note on node-forge fork, see https://github.com/digitalbazaar/forge/issues/744 for details",

package/src/0-path-value-core/PathValueController.ts

@@ -177,6 +177,7 @@ export class PathValueControllerBase {
                     timeIdVersion: value.time.version,
                     sourceNodeId,
                     sourceNodeThreadId: decodeNodeId(sourceNodeId)?.threadId,
+                    isValid: value.valid,
                 });
             }
         }

package/src/0-path-value-core/ValidStateComputer.ts

@@ -390,6 +390,7 @@ class ValidStateComputer {
 
             for (let pathValue of valueGroup) {
                 let prevValidState = prevValidStates.get(pathValue);
+                /* We used to log these values until 2026 June 28th 6:30 am
                 if (valid && prevValidState === false) {
                     console.info(`Accepting value that was previously rejected`, {
                         path: pathValue.path,
@@ -404,10 +405,28 @@ class ValidStateComputer {
                         timeIdFull: pathValue.time,
                     });
                 }
-
+                if (valid === false && prevValidState === true) {
+                    console.info(`Rejecting value that was valid before`, {
+                        path: pathValue.path,
+                        timeId: pathValue.time.time,
+                        timeIdFull: pathValue.time,
+                    });
+                }
+                */
                 // IMPORTANT! This means if it didn't previously exist and it's presently rejected, we count that as a change, as it this is going from undefined to false. This is actually very useful, as a lot of places want to know if a write is rejected, so they can display it in the UI for the developer.
                 if (valid === prevValidState) continue;
 
+                // NOTE: We might remove this logging later as it's pretty heavy, but right now we still have bugs somewhere here.
+                {
+                    console.info(`Changed valid state ${JSON.stringify(prevValidState)} to ${JSON.stringify(valid)}`, {
+                        path: pathValue.path,
+                        timeId: pathValue.time.time,
+                        timeIdFull: pathValue.time,
+                        valid,
+                        prevValidState,
+                    });
+                }
+
                 changed.push(pathValue);
 
                 // NOTE: This should be the only place we set it, and the read-only flag is only for us to prevent anyone from setting it.

package/src/0-path-value-core/pathValueCore.ts

@@ -743,7 +743,7 @@ class AuthorityPathValueStorage {
 
         if (isDebugLogEnabled() && !config?.doNotArchive) {
             for (let value of newValues) {
-                auditLog("INGEST VALUE", { path: value.path, timeId: value.time.time, batchTime: now });
+                auditLog("INGEST VALUE", { path: value.path, timeId: value.time.time, batchTime: now, isValid: value.valid });
             }
         }
 

package/src/library-components/animateAnyElement.tsx

@@ -42,7 +42,7 @@ export const createRefAnimation = cacheJSONArgsEqual((id: string, transitionTime
         let scrollChangeLeft = newScrollOffsetObj.left - prevScrollOffsetLeft;
         prevScrollOffsetLeft = newScrollOffsetObj.left;
 
-        if (!curPrevRect?.width || !newRect.width || curPrevRect.left === newRect.left) {
+        if (!curPrevRect?.width || !newRect.width || curPrevRect.left === newRect.left && curPrevRect.top === newRect.top) {
             return;
         }
         elem.style.transform = `translate(${curPrevRect.left - newRect.left - scrollChangeLeft}px, ${curPrevRect.top - newRect.top - scrollChangeTop}px)`;