querysub 0.483.0 → 0.484.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "querysub",
-  "version": "0.483.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/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,
@@ -411,10 +412,21 @@ class ValidStateComputer {
                         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.