@yuneta/gobj-ui 7.25.14 → 7.25.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yuneta/gobj-ui",
-  "version": "7.25.14",
+  "version": "7.25.16",
   "type": "module",
   "main": "dist/gobj-ui.cjs.js",
   "module": "dist/gobj-ui.es.js",
@@ -30,7 +30,7 @@
   "devDependencies": {
     "@rollup/plugin-terser": "^1.0.0",
     "@vitest/coverage-v8": "^4.1.10",
-    "@yuneta/gobj-js": "^7.22.2",
+    "@yuneta/gobj-js": "^7.25.2",
     "vite": "^8.2.1",
     "vitest": "^4.1.10"
   },

package/src/c_g6_nodes_tree.js

@@ -94,6 +94,7 @@ import {
 
 import {
     apply_graphs_echo,
+    forget_refused_graphs_write,
     plan_graph_saves,
     topic_arrangement_changed,
 } from "./graph_save_plan.js";
@@ -492,6 +493,7 @@ let PRIVATE_DATA = {
     __graphs__:         [],         // Rows of __graphs__
     _graph_properties:  {},         // topic_name → {nodes: {node_id: {x,y,size,...}}}
     _saved_graph_properties: {},    // the same, as the BACKEND has it: what a Save compares against
+    _graphs_writes_owed: {},        // topic_name → true: a __graphs__ write the host said did not land
     yet_showed:         false,
     edit_mode:          false,
     operation_mode:     null,
@@ -3275,13 +3277,16 @@ function save_topic_graph_properties(gobj, topic_name)
      *  the publish and as a copy, because the object below travels by
      *  reference and goes on being arranged after the write.
      *
-     *  Nothing answers an `EV_UPDATE_NODE` of `__graphs__` -- the view
-     *  writes that topic itself and is deliberately not told about it
-     *  (see the EV_RECORD_WRITTEN of C_YUI_TREEDB_GRAPH) -- so this is
-     *  the write taken for granted. A refused one leaves the view
-     *  believing the backend holds an arrangement it does not, until
-     *  the next load reads the records again.  */
+     *  Nothing answers an `EV_UPDATE_NODE` of `__graphs__` when it
+     *  lands -- the view writes that topic itself and is deliberately
+     *  not told about it (see the EV_RECORD_WRITTEN of
+     *  C_YUI_TREEDB_GRAPH) -- so this is the write taken for granted.
+     *  The host says when one does NOT land (EV_GRAPHS_WRITE_REFUSED),
+     *  and the next Save writes the topic again.  */
     priv._saved_graph_properties[topic_name] = json_deep_copy(properties);
+    /*  Paid: this is the write that was owed, if one was. Refused
+     *  again, EV_GRAPHS_WRITE_REFUSED owes it again.  */
+    delete priv._graphs_writes_owed[topic_name];
 
     let kw_update = {
         treedb_name: priv.treedb_name,
@@ -3387,9 +3392,16 @@ function update_history_buttons(gobj)
                 disableElements($container, ".EV_HISTORY_UNDO");
                 set_active_state($container, ".EV_HISTORY_UNDO", false);
 
-                // No more undos: graph is back to original state, disable save
-                disableElements($container, ".EV_SAVE_GRAPH");
-                set_submit_state($container, ".EV_SAVE_GRAPH", false);
+                /*  No more undos: the graph is back to what was loaded
+                 *  or saved -- unless a write of it did not land. That
+                 *  one is still owed, and only a Save pays it.  */
+                if(graphs_write_is_owed(gobj)) {
+                    enableElements($container, ".EV_SAVE_GRAPH");
+                    set_submit_state($container, ".EV_SAVE_GRAPH", true);
+                } else {
+                    disableElements($container, ".EV_SAVE_GRAPH");
+                    set_submit_state($container, ".EV_SAVE_GRAPH", false);
+                }
             }
         } else {
             disableElements($container, ".EV_HISTORY_REDO");
@@ -4762,6 +4774,17 @@ function run_preview_undo(gobj, key)
     }
 }
 
+/************************************************************
+ *  Is a `__graphs__` write owed? The host said one did not land
+ *  (EV_GRAPHS_WRITE_REFUSED), and no Save has written that topic
+ *  since. The Save button stays lit while it is: the history
+ *  knows nothing of it, so `canUndo()` alone would put it out.
+ ************************************************************/
+function graphs_write_is_owed(gobj)
+{
+    return Object.keys(gobj.priv._graphs_writes_owed).length > 0;
+}
+
 function mark_graph_dirty(gobj)
 {
     let $container = gobj_read_attr(gobj, "$container");
@@ -10345,6 +10368,11 @@ function ac_clear_data(gobj, event, kw, src)
 
     gobj_write_attr(gobj, "records", {});
 
+    /*  A write owed is owed on the arrangement being thrown away: the
+     *  reload brings what the backend holds, and that is what the view
+     *  will hold.  */
+    priv._graphs_writes_owed = {};
+
     /*  The node counters belong to the data that is going away: a refresh
      *  that kept them would decide the layout on the previous treedb. */
     priv._nodes_total = 0;
@@ -10494,15 +10522,49 @@ function ac_save_graph(gobj, event, kw, src)
         let history = graph_get_plugin(gobj, "history");
         if(history) {
             history.clear();
-            update_history_buttons(gobj);
         }
 
         save_geometry(gobj); // Publish an EV_UPDATE_NODE of each node
+
+        /*  After the writes: they pay a write owed, and the buttons
+         *  are repainted from what is still owed.  */
+        update_history_buttons(gobj);
     }
 
     return 0;
 }
 
+/************************************************************
+ *  The host says a write of `__graphs__` did not land: the
+ *  backend refused it, the transport refused it on the way out,
+ *  or there was no session to send it on. kw: {topic}
+ *
+ *  save_topic_graph_properties() recorded it as saved before it
+ *  left. Forgotten now, and the Save is lit again: the next Save
+ *  writes the topic again. Before, the view believed the backend
+ *  held the arrangement, and the next Save had nothing to write.
+ ************************************************************/
+function ac_graphs_write_refused(gobj, event, kw, src)
+{
+    let priv = gobj.priv;
+    let topic_name = kw && kw.topic;
+
+    if(!topic_name) {
+        log_error(`${gobj_short_name(gobj)}: ${event} names no topic`);
+        return -1;
+    }
+    log_warning(`${gobj_short_name(gobj)}: the arrangement of '${topic_name}' ` +
+        `was not saved: the next Save writes it again`);
+    forget_refused_graphs_write(priv._saved_graph_properties, topic_name);
+    /*  Owed until a Save writes it: update_history_buttons() keeps
+     *  the Save lit while it is, in edition and on entering it.  */
+    priv._graphs_writes_owed[topic_name] = true;
+    if(priv.edit_mode) {
+        mark_graph_dirty(gobj);
+    }
+    return 0;
+}
+
 /************************************************************
  *  Node created, from subscription
  ************************************************************/
@@ -10582,8 +10644,8 @@ function ac_node_updated(gobj, event, kw, src)
             priv.__graphs__.push(node);
         }
         /*  Only THIS record's topic: a rebuild of every topic took their
-         *  saved copies from the live objects (M32 of the 2026-09-21
-         *  review). build_graph_properties() is for the load.  */
+         *  saved copies from the live objects.
+         *  build_graph_properties() is for the load.  */
         apply_graphs_echo(
             priv._graph_properties, priv._saved_graph_properties, priv.__graphs__, node
         );
@@ -10637,7 +10699,7 @@ function ac_node_updated(gobj, event, kw, src)
     update_local_node(gobj, topic_name, node);
     /*  `priv.graph`: this read an undeclared `graph`, the ReferenceError
      *  fell into the catch, and a card on screen kept its old record after
-     *  every UPDATED (M33 of the 2026-09-21 review). The catch is for G6,
+     *  every UPDATED. The catch is for G6,
      *  which throws on an id it does not hold -- a folded card.  */
     let on_screen = false;
     try {
@@ -11778,8 +11840,8 @@ function ac_confirmed(gobj, event, kw, src)
         }
         if(missing.length) {
             /*  Gone while the question was open (another writer deleted
-             *  them): said, to the log and to the person (a low of the
-             *  2026-09-22 review); the rest go on.  */
+             *  them): said, to the log and to the person; the rest go
+             *  on.  */
             log_error(`${gobj_short_name(gobj)}: ${event}: cards gone while ` +
                 `confirming the delete, not deleted: ${missing.join(", ")}`);
             yui_shell_show_error(yui_shell_of(gobj), "some records were gone before the delete", {t: t});
@@ -12022,6 +12084,7 @@ function create_gclass(gclass_name)
             ["EV_NODE_CREATED",             ac_node_created,        null],
             ["EV_NODE_UPDATED",             ac_node_updated,        null],
             ["EV_NODE_DELETED",             ac_node_deleted,        null],
+            ["EV_GRAPHS_WRITE_REFUSED",     ac_graphs_write_refused, null],
 
             /*--- Graph interaction events ---*/
             ["EV_NODE_CLICK",               ac_node_click,          null],
@@ -12091,6 +12154,7 @@ function create_gclass(gclass_name)
         ["EV_NODE_CREATED",             0],
         ["EV_NODE_UPDATED",             0],
         ["EV_NODE_DELETED",             0],
+        ["EV_GRAPHS_WRITE_REFUSED",     0],
 
         /*--- Graph interaction (internal) ---*/
         ["EV_NODE_CLICK",               0],