@yuneta/gobj-ui 7.25.8 → 7.25.9

package/README.md

@@ -2182,12 +2182,40 @@ else.
   `ST_LOADING` declares none of a screen's actions. A **dialog** left open is
   the one thing still reachable: its Save (`EV_SAVE_COLUMN`, `EV_SAVE_TOPIC`),
   a confirmation (`EV_CONFIRMED`), an orphan delete or an import is refused
-  with `the schemas are loading: try again when they are in` (an i18n key the
-  consumer's locales carry) and the dialog stays open with what was typed, so
-  the same Save works once the load is in. A write that ends with an error --
-  a drop included -- still applies a position that waited, as a load does.
-  Out of session a load is not asked at all (no *"not in session"* errors);
-  the reconnect loads.
+  with `the schemas are loading: wait for them` (7.25.9; an i18n key the
+  consumer's locales carry). A write that ends with an error -- a drop
+  included -- still applies a position that waited, as a load does. Out of
+  session a load is not asked at all (no *"not in session"* errors); the
+  reconnect loads.
+- **A dialog is built on ONE model, and a load that replaces it closes it**
+  (7.25.9). Every load that lands bumps `model_gen`; a dialog records the one
+  it was opened on, and its Save, its import Preview/Run, an orphan delete and
+  a confirmation carry it in their kw. When a load lands under a dialog of an
+  older model, the dialog is closed and the operator told
+  `the schemas were read again: open the dialog again`; anything stamped with
+  a replaced model that arrives anyway is refused with the same words. A
+  column form sends every field it shows, so its Save after a reload wrote
+  the OLD record over the newer one -- 7.25.8 told the operator to "try
+  again", which did exactly that. The import plan is forgotten when a load
+  starts. A load that failed or never left replaced nothing, and the dialog
+  stays with what was typed.
+
+  ```js
+  gobj_send_event(editor, "EV_EDIT_COLUMN", {col: "id"}, host);   // form, model_gen N
+  gobj_send_event(editor, "EV_REFRESH", {}, host);                 // Save -> "wait for them"
+  // ... the load lands (model_gen N+1): the form closes, the operator reopens it
+  ```
+- **A load refused IN session keeps the model it was replacing** (7.25.9): a
+  routing adapter's deadline on a slow `nodes` answers -1 while the session
+  is up. The records are put back, the screen and any open dialog stay, the
+  operator is told `cannot read the schemas again: the previous ones stay`,
+  and the reload is owed. Only a first load, with no model before it, ends
+  on the empty screen. A write answered with no record reloads the model and
+  says what it did not send after it
+  (`the treedb did not describe a write back: the writes after it were not sent`).
+- The toolbar offers Diagram, Check, Export, Import, New topic and New column
+  only when the treedb (or topic) the position names is IN the model; after a
+  reload that no longer has it, Back and Refresh remain.
 - A write this view gave up on (a drop, a deadline) whose answer still says it
   was DONE makes the editor read the model again (at once, after the write in
   flight, or on the reconnect), and it is a write for the host too: it

package/dist/gobj-ui.cjs.js

@@ -3062,6 +3062,8 @@ function show_notification(shell, kind, message, opts) {
 		},
 		[["button", {
 			class: "TOAST_CLOSE delete",
+			title: (0, i18next.t)("close"),
+			"data-i18n-title": "close",
 			"aria-label": (0, i18next.t)("close"),
 			"data-i18n-aria-label": "close"
 		}], [
@@ -3185,6 +3187,8 @@ function yui_shell_show_modal(shell, content, opts) {
 						{
 							class: "MODAL_CLOSE yui-dialog-x",
 							type: "button",
+							title: (0, i18next.t)("close"),
+							"data-i18n-title": "close",
 							"aria-label": (0, i18next.t)("close"),
 							"data-i18n-aria-label": "close"
 						},
@@ -3206,6 +3210,8 @@ function yui_shell_show_modal(shell, content, opts) {
 		]];
 		if (with_close) modal_children.push(["button", {
 			class: "MODAL_CLOSE modal-close is-large",
+			title: (0, i18next.t)("close"),
+			"data-i18n-title": "close",
 			"aria-label": (0, i18next.t)("close"),
 			"data-i18n-aria-label": "close"
 		}]);
@@ -3360,6 +3366,8 @@ function build_dialog(shell, message, buttons, opts) {
 	]);
 	$card_children.push(["button", {
 		class: "CONFIRM_CLOSE delete yui-confirm-x",
+		title: (0, i18next.t)("close"),
+		"data-i18n-title": "close",
 		"aria-label": (0, i18next.t)("close"),
 		"data-i18n-aria-label": "close"
 	}], [
@@ -72173,7 +72181,9 @@ var attrs_table$1 = [
 ];
 var PRIVATE_DATA$1 = {
 	records: null,
+	records_before: null,
 	model: null,
+	model_gen: 0,
 	pending: 0,
 	load_error: "",
 	load_round: 0,
@@ -72188,6 +72198,7 @@ var PRIVATE_DATA$1 = {
 	order_undo: null,
 	diagram_gobj: null,
 	dialog: null,
+	dialog_gen: 0,
 	save_queue: null,
 	save_return: "",
 	save_wrote: false,
@@ -72348,6 +72359,29 @@ function refuse_if_readonly(gobj, event) {
 	yui_shell_show_error(yui_shell_of(gobj), "the treedb is read-only", { t: i18next.t });
 	return true;
 }
+/***************************************************************
+*  A decision taken on a model a load has since REPLACED: the Save
+*  of a form, a confirmation, an import. Each carries the
+*  `model_gen` it was built on (a number, not the model: a kw is
+*  dumped by the machine trace).
+*
+*  Refused, and said. A column form sends every field it shows, so
+*  its Save after a reload wrote the OLD record over the newer one
+*  (fourth independent review, probe A3); a form whose column the
+*  reload removed answered "names no column" for ever. end_load()
+*  closes such a dialog when the load lands; this is the same rule
+*  for whatever arrives anyway -- a stale screen, a confirmation
+*  that was up, a keyboard.
+*
+*  A kw with no stamp is not a dialog's and is not asked.
+***************************************************************/
+function built_on_a_replaced_model(gobj, event, kw) {
+	let priv = gobj.priv;
+	if (!kw || kw.model_gen === void 0 || kw.model_gen === priv.model_gen) return false;
+	(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} refused, it was decided on the schemas a reload replaced`);
+	yui_shell_show_error(yui_shell_of(gobj), "the schemas were read again: open the dialog again", { t: i18next.t });
+	return true;
+}
 /***************************
 *      The model
 ***************************/
@@ -72362,7 +72396,11 @@ function refuse_if_readonly(gobj, event) {
 *  and the toolbar stops answering: left up, a click on a card, on
 *  Back or on the drawing answered "Event NOT DEFINED in state
 *  ST_LOADING" and was lost (third independent review). A dialog
-*  left open is answered in ST_LOADING instead (ac_wait_for_the_load).
+*  left open is answered in ST_LOADING instead (ac_wait_for_the_load),
+*  and a load that LANDS closes it (end_load()): it was built on the
+*  model the load replaced, and its Save sends every field it shows.
+*  An import plan is forgotten here for the same reason: it is a list
+*  of writes computed against the model going away.
 *
 *  `keep_written`: a load the host did not ask for -- the reconnect,
 *  a write that turned out done -- keeps what this session wrote.
@@ -72372,12 +72410,16 @@ function refuse_if_readonly(gobj, event) {
 *  is not even asked): the records the model was built on are put
 *  back (emptied, the next write patched them into a model with no
 *  treedb), the round is spent so a request that did leave is
-*  stale, and the session back asks again.
+*  stale, and the session back asks again. A load that left and
+*  FAILED keeps them the same way (`records_before`,
+*  ac_mt_command_answer()).
 ***************************************************************/
 function request_model(gobj, keep_written) {
 	let priv = gobj.priv;
 	if (!(0, _yuneta_gobj_js.gobj_read_pointer_attr)(gobj, "gobj_remote_yuno")) return -1;
+	forget_import_plan(gobj);
 	let previous = priv.records;
+	priv.records_before = previous;
 	priv.records = {
 		treedbs: [],
 		topics: [],
@@ -72403,6 +72445,7 @@ function request_model(gobj, keep_written) {
 	}
 	if (left < 3) {
 		priv.records = previous;
+		priv.records_before = null;
 		priv.load_round++;
 		priv.reload_on_open = true;
 		priv.load_error = (0, i18next.t)("cannot reach the treedb");
@@ -72498,9 +72541,10 @@ function start_measuring(gobj) {
 *  until the host saves it (C_TREEDB's save-schema, which raises the
 *  versions of what changed). Two sources say a topic is one, and
 *  they are kept APART because they are forgotten differently:
-*    - `written`: what THIS session wrote since the model was loaded
-*      (a reload, the host's EV_REFRESH, forgets it -- the store says
-*      what is left);
+*    - `written`: what THIS session wrote. Only a load the HOST asked
+*      for forgets it (its EV_REFRESH, which a Save sends); the
+*      reconnect's reload, the one owed after a write and a late
+*      write's keep it (start_measuring(), request_model());
 *    - `host_ids`: what the host said last (EV_DRAFTS), replaced WHOLE
 *      by the next EV_DRAFTS. Folded into `written`, a topic the host
 *      named once stayed a draft after the Save that published it.
@@ -72668,6 +72712,8 @@ function render_toolbar(gobj) {
 	let readonly = is_readonly(gobj);
 	let has_model = !!priv.model;
 	let on_a_treedb = state !== "ST_IDLE" && state !== "ST_EMPTY";
+	let treedb_here = on_a_treedb && !!current_treedb(gobj);
+	let topic_here = treedb_here && !!current_topic(gobj);
 	let in_flight = state === "ST_SAVING" || state === "ST_LOADING";
 	let $left = [];
 	if (priv.treedb_id && on_a_treedb) $left.push([
@@ -72743,17 +72789,17 @@ function render_toolbar(gobj) {
 		$trail
 	]);
 	let $right = [];
-	if (priv.treedb_id && has_model && on_a_treedb) {
+	if (priv.treedb_id && has_model && treedb_here) {
 		$right.push(toolbar_button("SCHEMA_DIAGRAM_BTN", "yi-compass-drafting", priv.diagram ? "topics" : "diagram", "EV_TOGGLE_DIAGRAM", false));
 		$right.push(toolbar_button("SCHEMA_VALIDATE_BTN", "yi-square-check", "check", "EV_VALIDATE", false));
 		$right.push(toolbar_button("SCHEMA_EXPORT_BTN", "yi-download", "export", "EV_EXPORT", false));
 		if (!readonly) $right.push(toolbar_button("SCHEMA_IMPORT_BTN", "yi-upload", "import", "EV_IMPORT", false));
 	}
-	if (priv.topic_name && !readonly && !priv.diagram && on_a_treedb) {
+	if (priv.topic_name && !readonly && !priv.diagram && topic_here) {
 		if (undo_order_writes(gobj, current_topic(gobj)).length > 0) $right.push(toolbar_button("SCHEMA_UNDO_ORDER_BTN", "yi-arrow-rotate-left", "undo the order", "EV_UNDO_ORDER", false));
 		$right.push(toolbar_button("SCHEMA_ADD_COL_BTN", "yi-plus", "new column", "EV_ADD_COLUMN", false));
 	}
-	if (priv.treedb_id && !priv.topic_name && !readonly && !priv.diagram && on_a_treedb) $right.push(toolbar_button("SCHEMA_ADD_TOPIC_BTN", "yi-plus", "new topic", "EV_ADD_TOPIC", false));
+	if (priv.treedb_id && !priv.topic_name && !readonly && !priv.diagram && treedb_here) $right.push(toolbar_button("SCHEMA_ADD_TOPIC_BTN", "yi-plus", "new topic", "EV_ADD_TOPIC", false));
 	$right.push(toolbar_button("SCHEMA_REFRESH_BTN", "yi-arrows-rotate", "refresh", "EV_REFRESH", state === "ST_LOADING"));
 	let $bar = (0, _yuneta_gobj_js.createElement2)([
 		"div",
@@ -73573,6 +73619,7 @@ function close_dialog(gobj) {
 function open_dialog(gobj, $content, title, logical, title_prefix) {
 	let priv = gobj.priv;
 	close_dialog(gobj);
+	priv.dialog_gen = priv.model_gen;
 	priv.dialog = yui_shell_show_modal(yui_shell_of(gobj), $content, {
 		dialog: true,
 		title,
@@ -73619,11 +73666,28 @@ function name_control($spec, label) {
 			$spec[1]["aria-label"] = (0, i18next.t)(label);
 			$spec[1]["data-i18n-aria-label"] = label;
 		}
+		if (!$spec[1].title) {
+			$spec[1].title = (0, i18next.t)(label);
+			$spec[1]["data-i18n-title"] = label;
+		}
 		return true;
 	}
 	for (let child of $spec[2] || []) if (name_control(child, label)) return true;
 	return false;
 }
+/***************************************************************
+*  The four attributes that NAME a control -- a title and an
+*  aria-label, each with the key a language change re-translates
+*  it by. A visible label is not a name (CLAUDE.md).
+***************************************************************/
+function name_attrs(key, attrs) {
+	return Object.assign({
+		title: (0, i18next.t)(key),
+		"data-i18n-title": key,
+		"aria-label": (0, i18next.t)(key),
+		"data-i18n-aria-label": key
+	}, attrs);
+}
 function field(logical, name, label, $control, help) {
 	name_control($control, label);
 	return [
@@ -73800,7 +73864,7 @@ function read_flags($root) {
 *  than not offering it.
 ***************************************************************/
 function open_column_form(gobj, topic, col, prefill) {
-	gobj.priv;
+	let priv = gobj.priv;
 	let record = prefill || col && col.record || {};
 	let creating = !col;
 	let type = record.type || "string";
@@ -73810,6 +73874,7 @@ function open_column_form(gobj, topic, col, prefill) {
 	let hook_col = hook_topic ? hook[hook_topic] : "";
 	let treedb = current_treedb(gobj);
 	let sibling_topics = treedb ? treedb.topics.map((x) => x.name) : [];
+	let model_gen = priv.model_gen;
 	let $form = (0, _yuneta_gobj_js.createElement2)([
 		"div",
 		{ class: "SCHEMA_COL_FORM box" },
@@ -73872,10 +73937,10 @@ function open_column_form(gobj, topic, col, prefill) {
 				},
 				[[
 					"button",
-					{
+					name_attrs("cancel", {
 						class: "SCHEMA_COL_FORM_CANCEL button",
 						type: "button"
-					},
+					}),
 					[[
 						"span",
 						{ i18n: "cancel" },
@@ -73883,10 +73948,10 @@ function open_column_form(gobj, topic, col, prefill) {
 					]]
 				], [
 					"button",
-					{
+					name_attrs("save", {
 						class: "SCHEMA_COL_FORM_SAVE button is-primary",
 						type: "button"
-					},
+					}),
 					[[
 						"span",
 						{ class: "icon" },
@@ -73926,7 +73991,8 @@ function open_column_form(gobj, topic, col, prefill) {
 			topic: topic.name,
 			col: creating ? "" : col.name,
 			creating,
-			values
+			values,
+			model_gen
 		}, gobj);
 	});
 	open_dialog(gobj, $form, creating ? "new column" : "edit column", "SCHEMA_COL_FORM_DIALOG", creating ? topic.name : `${topic.name}.${col.name}`);
@@ -73949,6 +74015,7 @@ function open_topic_form(gobj, treedb, topic) {
 	let record = topic && topic.record || {};
 	let creating = !topic;
 	let col_names = topic ? topic.cols.map((c) => c.name) : [];
+	let model_gen = gobj.priv.model_gen;
 	let $form = (0, _yuneta_gobj_js.createElement2)([
 		"div",
 		{ class: "SCHEMA_TOPIC_FORM box" },
@@ -74007,10 +74074,10 @@ function open_topic_form(gobj, treedb, topic) {
 				},
 				[[
 					"button",
-					{
+					name_attrs("cancel", {
 						class: "SCHEMA_TOPIC_FORM_CANCEL button",
 						type: "button"
-					},
+					}),
 					[[
 						"span",
 						{ i18n: "cancel" },
@@ -74018,10 +74085,10 @@ function open_topic_form(gobj, treedb, topic) {
 					]]
 				], [
 					"button",
-					{
+					name_attrs("save", {
 						class: "SCHEMA_TOPIC_FORM_SAVE button is-primary",
 						type: "button"
-					},
+					}),
 					[[
 						"span",
 						{ class: "icon" },
@@ -74044,7 +74111,8 @@ function open_topic_form(gobj, treedb, topic) {
 		(0, _yuneta_gobj_js.gobj_send_event)(gobj, "EV_SAVE_TOPIC", {
 			topic: creating ? "" : topic.name,
 			creating,
-			values: read_form($form)
+			values: read_form($form),
+			model_gen
 		}, gobj);
 	});
 	open_dialog(gobj, $form, creating ? "new topic" : "edit topic", "SCHEMA_TOPIC_FORM_DIALOG", creating ? treedb.id : `${treedb.id}.${topic.name}`);
@@ -74324,6 +74392,7 @@ function copy_text(gobj, $text) {
 *  plan itself, so what is confirmed is what runs.
 ***************************************************************/
 function open_import(gobj, treedb) {
+	let model_gen = gobj.priv.model_gen;
 	let $content = (0, _yuneta_gobj_js.createElement2)([
 		"div",
 		{ class: "SCHEMA_IMPORT box" },
@@ -74338,12 +74407,12 @@ function open_import(gobj, treedb) {
 			],
 			[
 				"textarea",
-				{
+				name_attrs("schema", {
 					class: "SCHEMA_IMPORT_TEXT textarea is-small is-family-monospace",
 					rows: "12",
 					spellcheck: "false",
 					placeholder: "{\"id\": \"...\", \"schema_version\": \"1\", \"topics\": []}"
-				},
+				}),
 				""
 			],
 			[
@@ -74376,10 +74445,10 @@ function open_import(gobj, treedb) {
 				[
 					[
 						"button",
-						{
+						name_attrs("cancel", {
 							class: "SCHEMA_IMPORT_CANCEL button",
 							type: "button"
-						},
+						}),
 						[[
 							"span",
 							{ i18n: "cancel" },
@@ -74388,10 +74457,10 @@ function open_import(gobj, treedb) {
 					],
 					[
 						"button",
-						{
+						name_attrs("preview", {
 							class: "SCHEMA_IMPORT_PREVIEW button",
 							type: "button"
-						},
+						}),
 						[[
 							"span",
 							{ class: "icon" },
@@ -74404,11 +74473,11 @@ function open_import(gobj, treedb) {
 					],
 					[
 						"button",
-						{
+						name_attrs("import", {
 							class: "SCHEMA_IMPORT_RUN button is-warning",
 							type: "button",
 							disabled: "disabled"
-						},
+						}),
 						[[
 							"span",
 							{ class: "icon" },
@@ -74435,18 +74504,31 @@ function open_import(gobj, treedb) {
 		evt.stopPropagation();
 		(0, _yuneta_gobj_js.gobj_send_event)(gobj, "EV_PREVIEW_IMPORT", {
 			text: $text.value,
-			prune: $prune.checked
+			prune: $prune.checked,
+			model_gen
 		}, gobj);
 	});
 	$run.addEventListener("click", (evt) => {
 		evt.stopPropagation();
-		(0, _yuneta_gobj_js.gobj_send_event)(gobj, "EV_APPLY_IMPORT", {}, gobj);
+		(0, _yuneta_gobj_js.gobj_send_event)(gobj, "EV_APPLY_IMPORT", { model_gen }, gobj);
 	});
 	gobj.priv.import_pane = $plan_pane;
 	gobj.priv.import_run = $run;
 	open_dialog(gobj, $content, "import", "SCHEMA_IMPORT_DIALOG", treedb.id);
 }
 /***************************************************************
+*  The plan is a list of writes computed against the model shown:
+*  a load replaces the model, so the plan goes with it and the
+*  dialog, if it stays up (a load that failed keeps the model),
+*  asks for a Preview again.
+***************************************************************/
+function forget_import_plan(gobj) {
+	let priv = gobj.priv;
+	priv.import_plan = null;
+	clear(priv.import_pane);
+	if (priv.import_run) priv.import_run.disabled = true;
+}
+/***************************************************************
 *  The plan, drawn where the paste is: one line per write, the
 *  deletions apart and counted, because they are what the
 *  operator is being asked about.
@@ -74573,7 +74655,8 @@ function open_orphans(gobj) {
 					{ class: "SCHEMA_ORPHAN_ACTIONS has-text-right" },
 					readonly ? [] : [row_icon(gobj, "SCHEMA_ORPHAN_DELETE", "yi-trash", "delete", "EV_DELETE_ORPHAN", {
 						kind,
-						id: entry.id
+						id: entry.id,
+						model_gen: priv.model_gen
 					})]
 				]
 			]
@@ -74745,6 +74828,10 @@ function transport_dropped(gobj) {
 		priv.load_round++;
 		priv.pending = 0;
 		priv.load_error = "";
+		if (priv.records_before) {
+			priv.records = priv.records_before;
+			priv.records_before = null;
+		}
 		return end_load(gobj);
 	}
 	return 0;
@@ -74755,11 +74842,22 @@ function transport_dropped(gobj) {
 *  in flight (ac_show() kept it), or the one it had. Without a
 *  model there is no screen to go to: the position is kept for the
 *  next load, and the editor says why it is empty.
+*
+*  A dialog built on the model this load REPLACED is closed, and the
+*  operator told to open it again: it shows the old record, and its
+*  Save would write every field of it over the newer one. A load
+*  that failed or never left replaced nothing (`model_gen` did not
+*  move), and the dialog stays with what was typed.
 ***************************************************************/
 function end_load(gobj) {
 	let priv = gobj.priv;
 	let seg = priv.pending_seg;
 	priv.pending_seg = null;
+	if (priv.dialog && priv.dialog_gen !== priv.model_gen) {
+		(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: a dialog was open on the schemas the load replaced: closed`);
+		close_dialog(gobj);
+		yui_shell_show_error(yui_shell_of(gobj), "the schemas were read again: open the dialog again", { t: i18next.t });
+	}
 	if (priv.model) {
 		if (seg !== null) return apply_seg(gobj, seg);
 		return go(gobj, priv.treedb_id, priv.topic_name, priv.diagram, false);
@@ -74975,10 +75073,20 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 		else if (priv.records && Array.isArray(priv.records[topic_name])) priv.records[topic_name] = Array.isArray(data) ? data : [];
 		if (priv.pending > 0) return 0;
 		if (priv.load_error) {
+			if (priv.model) {
+				(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: the schemas could not be read again (${priv.load_error}): the previous ones stay`);
+				priv.records = priv.records_before;
+				priv.records_before = null;
+				priv.reload_on_open = true;
+				yui_shell_show_error(yui_shell_of(gobj), "cannot read the schemas again: the previous ones stay", { t: i18next.t });
+				return end_load(gobj);
+			}
 			priv.model = null;
 			return end_load(gobj);
 		}
 		priv.order_undo = {};
+		priv.records_before = null;
+		priv.model_gen++;
 		build_model(gobj);
 		start_measuring(gobj);
 		return end_load(gobj);
@@ -75000,11 +75108,14 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 			let record = (0, _yuneta_gobj_js.is_object)(data) ? data : Array.isArray(data) && (0, _yuneta_gobj_js.is_object)(data[0]) ? data[0] : null;
 			if (record) patch_record(gobj, topic_name, record);
 			else {
-				(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: '${command}' answered no record`);
+				let not_sent = priv.save_queue.length - 1;
+				(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: '${command}' answered no record` + (not_sent > 0 ? `: ${not_sent} write(s) after it were not sent` : ""));
+				if (not_sent > 0) yui_shell_show_error(yui_shell_of(gobj), "the treedb did not describe a write back: the writes after it were not sent", { t: i18next.t });
 				if (topic_name === T_TOPICS || topic_name === T_COLS) {
 					let write = priv.save_queue[0];
 					mark_written(gobj, topic_name, write ? write.record : null);
 				}
+				priv.reload_after_write = false;
 				priv.save_queue = [];
 				priv.save_return = "";
 				priv.save_wrote = false;
@@ -75062,8 +75173,14 @@ function late_write_done(gobj, command, topic_name, data) {
 /***************************************************************
 *  The topic a record written belongs to, read off the RECORD and
 *  not off the screen: a write answered late may be about a topic
-*  the operator has left. A column names its topic in its fkey
-*  (`topics^<topic id>^cols`); a topic is its own id.
+*  the operator has left. A column names its topic in its fkey; a
+*  topic is its own id.
+*
+*  The fkey is read with parse_fkey_ref(), because the store answers
+*  in list_dict -- `[{topic_name, id, hook_name}]` -- and not in the
+*  `topics^<topic id>^cols` strings this view writes. Split as a
+*  string, every production answer marked nothing (fourth independent
+*  review, probe C).
 ***************************************************************/
 function mark_written_record(gobj, topic_name, record) {
 	let priv = gobj.priv;
@@ -75073,11 +75190,7 @@ function mark_written_record(gobj, topic_name, record) {
 		return;
 	}
 	if (topic_name !== T_COLS) return;
-	let refs = Array.isArray(record.topics) ? record.topics : typeof record.topics === "string" ? [record.topics] : [];
-	for (let ref of refs) {
-		let parts = String(ref).split("^");
-		if (parts.length >= 2 && parts[1]) priv.written[parts[1]] = true;
-	}
+	for (let ref of parse_fkey_ref(record.topics)) priv.written[ref.id] = true;
 }
 /***************************************************************
 *  Remember which topic this write belongs to, which is what
@@ -75161,14 +75274,19 @@ function ac_refresh$1(gobj, event, kw, src) {
 *  operator can still reach, and what it asks is computed against
 *  a model the answers in the air are about to replace.
 *
-*  Refused, and SAID -- and the dialog stays open with what was
-*  typed, so the same Save works once the load is in. Undeclared,
-*  it answered "Event NOT DEFINED in state ST_LOADING" and the
-*  edit was gone with no word (third independent review).
+*  Refused, and SAID. Undeclared, it answered "Event NOT DEFINED
+*  in state ST_LOADING" and the edit was gone with no word (third
+*  independent review).
+*
+*  And NOT "try again": the same Save, once the load was in, sent
+*  the form built on the old model -- every field of the old record
+*  over the newer one (fourth independent review). The load that
+*  lands closes such a dialog (end_load()); one that fails leaves it
+*  up, on the model it was built on.
 ***************************************************************/
 function ac_wait_for_the_load(gobj, event, kw, src) {
 	(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} refused, the schemas are loading`);
-	yui_shell_show_error(yui_shell_of(gobj), "the schemas are loading: try again when they are in", { t: i18next.t });
+	yui_shell_show_error(yui_shell_of(gobj), "the schemas are loading: wait for them", { t: i18next.t });
 	return -1;
 }
 /***************************************************************
@@ -75234,6 +75352,7 @@ function ac_show_orphans(gobj, event, kw, src) {
 *  to feed.
 ***************************************************************/
 function confirm_then(gobj, message, detail, kw) {
+	kw = Object.assign({ model_gen: gobj.priv.model_gen }, kw);
 	yui_shell_confirm_danger(yui_shell_of(gobj), message, {
 		t: i18next.t,
 		detail
@@ -75295,6 +75414,7 @@ function ac_save_column(gobj, event, kw, src) {
 	let topic = find_topic(priv.model, priv.treedb_id, kw.topic);
 	let col = kw.creating ? null : find_col(priv.model, priv.treedb_id, kw.topic, kw.col);
 	if (refuse_if_readonly(gobj, event)) return -1;
+	if (built_on_a_replaced_model(gobj, event, kw)) return -1;
 	if (!topic || !kw.creating && !col) {
 		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} names no column: ${kw.topic}.${kw.col}`);
 		return -1;
@@ -75416,6 +75536,7 @@ function ac_save_topic(gobj, event, kw, src) {
 	let treedb = current_treedb(gobj);
 	let topic = kw.creating ? null : find_topic(priv.model, priv.treedb_id, kw.topic);
 	if (refuse_if_readonly(gobj, event)) return -1;
+	if (built_on_a_replaced_model(gobj, event, kw)) return -1;
 	if (!treedb || !kw.creating && !topic) {
 		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} names no topic: ${kw.topic}`);
 		return -1;
@@ -75458,6 +75579,7 @@ function ac_delete_topic(gobj, event, kw, src) {
 ***************************************************************/
 function ac_delete_orphan(gobj, event, kw, src) {
 	if (refuse_if_readonly(gobj, event)) return -1;
+	if (built_on_a_replaced_model(gobj, event, kw)) return -1;
 	if ((0, _yuneta_gobj_js.empty_string)(kw.id)) {
 		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} names nothing`);
 		return -1;
@@ -75514,6 +75636,7 @@ function ac_preview_import(gobj, event, kw, src) {
 	let priv = gobj.priv;
 	let treedb = current_treedb(gobj);
 	let incoming = null;
+	if (built_on_a_replaced_model(gobj, event, kw)) return -1;
 	if (!treedb) {
 		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} with no treedb open`);
 		return -1;
@@ -75541,6 +75664,7 @@ function ac_preview_import(gobj, event, kw, src) {
 function ac_apply_import(gobj, event, kw, src) {
 	let plan = gobj.priv.import_plan;
 	if (refuse_if_readonly(gobj, event)) return -1;
+	if (built_on_a_replaced_model(gobj, event, kw)) return -1;
 	if (!plan || plan.writes.length === 0) {
 		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} with no plan to run`);
 		return -1;
@@ -75583,6 +75707,7 @@ function run_import(gobj) {
 function ac_confirmed(gobj, event, kw, src) {
 	let priv = gobj.priv;
 	if (refuse_if_readonly(gobj, event)) return -1;
+	if (built_on_a_replaced_model(gobj, event, kw)) return -1;
 	if (kw.what === "import") return run_import(gobj);
 	if (kw.what === "topic") {
 		let topic = find_topic(priv.model, priv.treedb_id, kw.topic);