querysub 0.511.0 → 0.512.0

package/package.json

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

package/spec.txt

@@ -1,8 +1,3 @@
-todonext
-Make sure the AI updates the machine schema in a smart way. So we can actually have both the old and the new servers config and manage them correctly. make sure if we make multiple updates, it still remembers what the old config is. And make sure deploy all works correctly. 
-
-
-
 Trigger values
     1) validStateComputer.ingestValuesAndValidStates
     1.5) authorityStorage.ingestValues

package/src/deployManager/components/ServiceDetailPage.tsx

@@ -138,12 +138,9 @@ export class ServiceDetailPage extends qreact.Component {
             try {
                 await action();
             } catch (error) {
-                Querysub.localCommit(() => {
+                Querysub.commit(() => {
                     this.state.saveError = error instanceof Error ? error.message : String(error);
                 });
-            } finally {
-                // The write => read invalidation keys should also cover this, but explicitly re-request the config as well, so the UI always reflects what the server now has (only AFTER the action fully finishes, so we can't refetch a value from before our write committed)
-                MachineServiceController(SocketFunction.browserNodeId()).getServiceConfig.refresh(selectedServiceIdParam.value);
             }
         });
     }

package/src/library-components/SyncedController.ts

@@ -307,7 +307,8 @@ export function getSyncedController<T extends {
                             promise.then(
                                 result => {
                                     logFinished();
-                                    Querysub.commitLocal(() => {
+                                    // commit, NOT commitLocal, as this runs from a promise callback (an async context), where commitLocal writes don't reliably commit / trigger re-renders (which made the write => read invalidation flaky)
+                                    Querysub.commit(() => {
                                         obj.result = atomicObjectWriteNoFreeze({ result });
                                         obj.promise = undefined;
                                         invalidateReaders();
@@ -315,7 +316,7 @@ export function getSyncedController<T extends {
                                 },
                                 error => {
                                     logFinished();
-                                    Querysub.commitLocal(() => {
+                                    Querysub.commit(() => {
                                         obj.result = atomicObjectWriteNoFreeze({ error });
                                         obj.promise = undefined;
                                         invalidateReaders();