querysub 0.402.0 → 0.403.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "querysub",
-  "version": "0.402.0",
+  "version": "0.403.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

@@ -15,7 +15,7 @@ import { sha256 } from "js-sha256";
 import debugbreak from "debugbreak";
 import { ClientWatcher } from "../1-path-client/pathValueClientWatcher";
 import { auditLog, isDebugLogEnabled } from "./auditLogs";
-import { debugNodeId } from "../-c-identity/IdentityController";
+import { debugNodeId, debugNodeThread } from "../-c-identity/IdentityController";
 import { logDisk } from "../diagnostics/logs/diskLogger";
 import { isDiskAudit } from "../config";
 import { decodeNodeId } from "../-a-auth/certs";
@@ -182,7 +182,7 @@ class PathValueControllerBase {
         if (isDebugLogEnabled()) {
             let sourceNodeId = debugNodeId(callerId);
             for (let value of config.paths) {
-                auditLog("WATCH PATH", { path: value, sourceNodeId });
+                auditLog("WATCH PATH", { path: value, sourceNodeId, sourceNodeThreadId: debugNodeThread(callerId) });
             }
             for (let value of config.parentPaths) {
                 auditLog("WATCH PARENT PATH", { path: value, sourceNodeId });
@@ -195,7 +195,7 @@ class PathValueControllerBase {
         if (isDebugLogEnabled()) {
             let sourceNodeId = debugNodeId(SocketFunction.getCaller().nodeId);
             for (let value of config.paths) {
-                auditLog("UNWATCH PATH", { path: value, sourceNodeId });
+                auditLog("UNWATCH PATH", { path: value, sourceNodeId, sourceNodeThreadId: debugNodeThread(callerId) });
             }
             for (let value of config.parentPaths) {
                 auditLog("UNWATCH PARENT PATH", { path: value, sourceNodeId });

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

@@ -1254,7 +1254,11 @@ class PathWatcher {
                 }
             } else {
                 for (let path of newPathsWatched) {
-                    auditLog("new non-local WATCH VALUE", { path, watcher: config.callback });
+                    auditLog("new non-local WATCH VALUE", {
+                        path,
+                        watcher: config.callback,
+                        targetNodeThreadId: debugNodeThread(config.callback),
+                    });
                 }
                 for (let path of newParentsWatched) {
                     auditLog("new non-local WATCH PARENT", { path, watcher: config.callback });

package/src/1-path-client/RemoteWatcher.ts

@@ -17,7 +17,8 @@ import { appendToPathStr, getParentPathStr, getPathDepth, getPathIndexAssert, ge
 import { isEmpty } from "../misc";
 import debugbreak from "debugbreak";
 import { isDevDebugbreak } from "../config";
-import { auditLog } from "../0-path-value-core/auditLogs";
+import { auditLog, isDebugLogEnabled } from "../0-path-value-core/auditLogs";
+import { debugNodeThread } from "../-c-identity/IdentityController";
 
 // NOTE: In some cases this can half our the PathValues sent. AND, it can reduce our sync requests by N, and as those are uploads those are event more expensive. However, it also complicates other code, requiring not just checking if a value is synced, but also the parent path. We should try to make this stable, but worst case scenario, we can set this to false, and it will make our synchronization a lot easier to verify.
 const STOP_KEYS_DOUBLE_SENDS = true;
@@ -457,6 +458,14 @@ export class RemoteWatcher {
                         paths: Array.from(paths),
                         parentPaths: Array.from(parentPaths),
                     };
+                    if (isDebugLogEnabled()) {
+                        for (let path of paths) {
+                            auditLog("Asking to watch path", { path, authorityId, targetNodeThreadId: debugNodeThread(authorityId) });
+                        }
+                        for (let path of parentPaths) {
+                            auditLog("Asking to watch parent path", { path, authorityId, targetNodeThreadId: debugNodeThread(authorityId) });
+                        }
+                    }
                     logErrors(RemoteWatcher.REMOTE_WATCH_FUNCTION(config, authorityId));
                 }
             });

package/src/diagnostics/logs/lifeCycleAnalysis/LifeCycleEntryEditor.tsx

@@ -331,7 +331,11 @@ export class LifeCycleEntryEditor extends qreact.Component<{
                                     hue={0}
                                     onClick={() => {
                                         let updatedLifeCycle = deepCloneJSON(lifeCycle);
-                                        delete updatedLifeCycle.entries[entryIndex].variables[key];
+                                        for (let e of updatedLifeCycle.entries) {
+                                            if (key in e.variables) {
+                                                delete e.variables[key];
+                                            }
+                                        }
                                         Querysub.onCommitFinished(async () => {
                                             await this.controller.setLifeCycle.promise(updatedLifeCycle);
                                         });