@milaboratories/uikit 2.15.32 → 2.15.34

package/.turbo/turbo-build.log

@@ -1,6 +1,6 @@
  WARN  Issue while reading "/home/runner/_work/platforma/platforma/.npmrc". Failed to replace env in config: ${NPMJS_TOKEN}
 
-> @milaboratories/uikit@2.15.32 build /home/runner/_work/platforma/platforma/lib/ui/uikit
+> @milaboratories/uikit@2.15.34 build /home/runner/_work/platforma/platforma/lib/ui/uikit
 > ts-builder build --target browser-lib --build-config ./build.browser-lib.config.js
 
 Building browser-lib project...
@@ -18,7 +18,7 @@ Using custom config: ./build.browser-lib.config.js
 rendering chunks...
 
 [unplugin:dts] Start generate declaration files...
-[unplugin:dts] Declaration files built in 6076ms.
+[unplugin:dts] Declaration files built in 6683ms.
 
 computing gzip size...
 dist/components/PlClipboard/PlClipboard.vue?vue&type=style&index=0&lang.css                             0.04 kB │ gzip:   0.06 kB
@@ -500,7 +500,6 @@ dist/assets/icons/icon-assets-min/24_edit.js
 dist/assets/icons/icon-assets-min/24_line-error.js                                                      0.58 kB │ gzip:   0.34 kB │ map:  0.67 kB
 dist/assets/icons/icon-assets-min/16_copy.js                                                            0.59 kB │ gzip:   0.37 kB │ map:  0.68 kB
 dist/assets/icons/icon-assets-min/24_reverse.js                                                         0.59 kB │ gzip:   0.36 kB │ map:  0.66 kB
-dist/composition/computedCached.js                                                                      0.59 kB │ gzip:   0.36 kB │ map:  2.72 kB
 dist/assets/icons/icon-assets-min/24_files-export.js                                                    0.59 kB │ gzip:   0.38 kB │ map:  0.68 kB
 dist/assets/icons/icon-assets-min/16_edit.js                                                            0.59 kB │ gzip:   0.38 kB │ map:  0.67 kB
 dist/assets/icons/icon-assets-min/24_files-import.js                                                    0.60 kB │ gzip:   0.38 kB │ map:  0.68 kB
@@ -528,12 +527,13 @@ dist/composition/useQuery.js
 dist/assets/icons/icon-assets-min/16_settings.js                                                        0.64 kB │ gzip:   0.37 kB │ map:  0.71 kB
 dist/assets/icons/icon-assets-min/24_table-upload.js                                                    0.64 kB │ gzip:   0.38 kB │ map:  0.73 kB
 dist/assets/icons/icon-assets-min/16_data-dimentions.js                                                 0.64 kB │ gzip:   0.42 kB │ map:  0.74 kB
+dist/composition/computedCached.js                                                                      0.64 kB │ gzip:   0.39 kB │ map:  3.88 kB
 [PLUGIN_TIMINGS] Warning: Your build spent significant time in plugins. Here is a breakdown:
-  - vite:asset (35%)
-  - sourcemaps (13%)
+  - vite:asset (38%)
   - vite:css-post (12%)
-  - vite:vue (9%)
-  - unplugin-dts (8%)
+  - sourcemaps (11%)
+  - unplugin-dts (10%)
+  - vite:vue (10%)
 See https://rolldown.rs/options/checks#plugintimings for more details.
 
 dist/assets/icons/icon-assets-min/16_clipboard.js                                                       0.64 kB │ gzip:   0.38 kB │ map:  0.72 kB
@@ -790,5 +790,5 @@ dist/composition/filters/metadata.js
 dist/index.js                                                                                          12.53 kB │ gzip:   2.96 kB │ map: 21.32 kB
 dist/components/PlSvg/PlSvg.vue_vue_type_script_setup_true_lang.js                                     39.41 kB │ gzip:   3.40 kB │ map:  8.66 kB
 
-✓ built in 7.98s
+✓ built in 8.95s
 Build completed successfully

package/.turbo/turbo-formatter$colon$check.log

@@ -1,6 +1,6 @@
  WARN  Issue while reading "/home/runner/_work/platforma/platforma/.npmrc". Failed to replace env in config: ${NPMJS_TOKEN}
 
-> @milaboratories/uikit@2.15.32 formatter:check /home/runner/_work/platforma/platforma/lib/ui/uikit
+> @milaboratories/uikit@2.15.34 formatter:check /home/runner/_work/platforma/platforma/lib/ui/uikit
 > ts-builder formatter --check
 
 Checking formatting...
@@ -8,5 +8,5 @@ Checking formatting...
 Checking formatting...
 
 All matched files use the correct format.
-Finished in 2322ms on 383 files using 16 threads.
+Finished in 4101ms on 383 files using 16 threads.
 Format check completed successfully

package/.turbo/turbo-linter$colon$check.log

@@ -1,10 +1,10 @@
  WARN  Issue while reading "/home/runner/_work/platforma/platforma/.npmrc". Failed to replace env in config: ${NPMJS_TOKEN}
 
-> @milaboratories/uikit@2.15.32 linter:check /home/runner/_work/platforma/platforma/lib/ui/uikit
+> @milaboratories/uikit@2.15.34 linter:check /home/runner/_work/platforma/platforma/lib/ui/uikit
 > ts-builder linter --check
 
 Linting project...
 ↳ oxlint --config /home/runner/_work/platforma/platforma/lib/ui/uikit/.oxlintrc.json --deny-warnings
 Found 0 warnings and 0 errors.
-Finished in 34ms on 310 files with 108 rules using 16 threads.
+Finished in 27ms on 310 files with 108 rules using 16 threads.
 Linting completed successfully

package/.turbo/turbo-types$colon$check.log

@@ -1,6 +1,6 @@
  WARN  Issue while reading "/home/runner/_work/platforma/platforma/.npmrc". Failed to replace env in config: ${NPMJS_TOKEN}
 
-> @milaboratories/uikit@2.15.32 types:check /home/runner/_work/platforma/platforma/lib/ui/uikit
+> @milaboratories/uikit@2.15.34 types:check /home/runner/_work/platforma/platforma/lib/ui/uikit
 > ts-builder type-check --target browser-lib
 
 ↳ vue-tsc-ts6.cjs --noEmit --project ./tsconfig.json --customConditions default

package/CHANGELOG.md

@@ -1,5 +1,34 @@
 # @milaboratories/uikit
 
+## 2.15.34
+
+### Patch Changes
+
+- @platforma-sdk/model@1.84.3
+
+## 2.15.33
+
+### Patch Changes
+
+- 43e07a3: Fix an infinite grid-remount loop in `PlAgDataTableV2` that recreated the whole
+  AG Grid around twenty times a second — strobing the column headers and filling
+  the console with AG Grid licence banners — on opening a page whose table had a
+  saved grid state, most often right after a block rebuild.
+
+  `computedCached`'s `writeThrough` made a set visible to readers immediately, but
+  the cache was still overwritten whenever the source re-emitted. A set is
+  debounced, so between the write and its arrival in the project state the source
+  still holds the value the write moved away from; a running block has that state
+  pushed back repeatedly, and each push reverted the cache. The stored grid state
+  therefore flipped between what the grid had and what the project still held, and
+  every flip made the reload watch destroy and recreate the grid — which, because
+  the flips never stopped on their own, never converged.
+
+  A write-through value now stays authoritative until the source reports it back.
+  Since `get` maps set values to themselves, that is exactly when the round trip
+  has completed; a change from elsewhere arriving mid-flight is dropped in favour
+  of the local write that is about to land.
+
 ## 2.15.32
 
 ### Patch Changes

package/dist/composition/computedCached.d.ts

@@ -6,6 +6,17 @@ import { ComputedGetter, ComputedSetter, ComputedRef, WritableComputedRef } from
  * With `writeThrough`, a set updates the cache synchronously, so a read reflects
  * the write immediately even when `set` defers its work (e.g. debounced); the
  * getter must then map set values to themselves (`get(x)` deep-equals `x`).
+ *
+ * A deferred set also leaves a window in which the source still holds the value
+ * the write moved away from, and anything that re-emits the source during that
+ * window — a running block having its project state pushed back, say — used to
+ * revert the cache to it. The next write moved the cache forward again: a
+ * ping-pong that never settled, and that whoever watches the value (the data
+ * table rebuilds its whole grid) pays for on every swing. So a write-through
+ * value stays authoritative until the source reports it back; since `get` maps
+ * set values to themselves, that is exactly when the round trip has completed.
+ * A change from elsewhere that arrives mid-flight is therefore dropped in
+ * favour of the local write, which is about to land anyway.
  */
 export declare function computedCached<T>(options: {
     get: ComputedGetter<T>;

package/dist/composition/computedCached.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"computedCached.d.ts","sourceRoot":"","sources":["../../src/composition/computedCached.ts"],"names":[],"mappings":"AACA,OAAO,EAIL,KAAK,cAAc,EACnB,KAAK,cAAc,EACnB,KAAK,WAAW,EAChB,KAAK,mBAAmB,EACzB,MAAM,KAAK,CAAC;AAEb;;;;;;;GAOG;AACH,wBAAgB,cAAc,CAAC,CAAC,EAAE,OAAO,EAAE;IACzC,GAAG,EAAE,cAAc,CAAC,CAAC,CAAC,CAAC;IACvB,GAAG,EAAE,cAAc,CAAC,CAAC,CAAC,CAAC;IACvB,YAAY,CAAC,EAAE,OAAO,CAAC;CACxB,GAAG,mBAAmB,CAAC,CAAC,CAAC,CAAC;AAC3B,wBAAgB,cAAc,CAAC,CAAC,EAAE,MAAM,EAAE,cAAc,CAAC,CAAC,CAAC,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC"}
+{"version":3,"file":"computedCached.d.ts","sourceRoot":"","sources":["../../src/composition/computedCached.ts"],"names":[],"mappings":"AACA,OAAO,EAIL,KAAK,cAAc,EACnB,KAAK,cAAc,EACnB,KAAK,WAAW,EAChB,KAAK,mBAAmB,EACzB,MAAM,KAAK,CAAC;AAEb;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,cAAc,CAAC,CAAC,EAAE,OAAO,EAAE;IACzC,GAAG,EAAE,cAAc,CAAC,CAAC,CAAC,CAAC;IACvB,GAAG,EAAE,cAAc,CAAC,CAAC,CAAC,CAAC;IACvB,YAAY,CAAC,EAAE,OAAO,CAAC;CACxB,GAAG,mBAAmB,CAAC,CAAC,CAAC,CAAC;AAC3B,wBAAgB,cAAc,CAAC,CAAC,EAAE,MAAM,EAAE,cAAc,CAAC,CAAC,CAAC,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC"}

package/dist/composition/computedCached.js

@@ -4,13 +4,17 @@ import { deepClone as r, isJsonEqual as i } from "@milaboratories/helpers";
 function a(a) {
 	let o, s, c = !1;
 	typeof a == "function" ? o = a : (o = a.get, s = a.set, c = a.writeThrough ?? !1);
-	let l = t(o());
+	let l = t(o()), u = !1;
 	return n(o, (e) => {
-		i(e, l.value) || (l.value = r(e));
+		if (i(e, l.value)) {
+			u = !1;
+			return;
+		}
+		u || (l.value = r(e));
 	}, { deep: !0 }), s ? e({
 		get: () => l.value,
 		set: c ? (e) => {
-			i(e, l.value) || (l.value = r(e)), s(e);
+			i(e, l.value) || (l.value = r(e), u = !0), s(e);
 		} : s
 	}) : l;
 }

package/dist/composition/computedCached.js.map

@@ -1 +1 @@
-{"version":3,"file":"computedCached.js","names":[],"sources":["../../src/composition/computedCached.ts"],"sourcesContent":["import { deepClone, isJsonEqual } from \"@milaboratories/helpers\";\nimport {\n  computed,\n  ref,\n  watch,\n  type ComputedGetter,\n  type ComputedSetter,\n  type ComputedRef,\n  type WritableComputedRef,\n} from \"vue\";\n\n/**\n * Alternative to `computed`, but triggering only on actual data changes.\n * Always `deep` as the plain `computed` is.\n *\n * With `writeThrough`, a set updates the cache synchronously, so a read reflects\n * the write immediately even when `set` defers its work (e.g. debounced); the\n * getter must then map set values to themselves (`get(x)` deep-equals `x`).\n */\nexport function computedCached<T>(options: {\n  get: ComputedGetter<T>;\n  set: ComputedSetter<T>;\n  writeThrough?: boolean;\n}): WritableComputedRef<T>;\nexport function computedCached<T>(getter: ComputedGetter<T>): ComputedRef<T>;\nexport function computedCached<T>(\n  arg:\n    | ComputedGetter<T>\n    | {\n        get: ComputedGetter<T>;\n        set: ComputedSetter<T>;\n        writeThrough?: boolean;\n      },\n) {\n  let getter: ComputedGetter<T>;\n  let setter: ComputedSetter<T> | undefined = undefined;\n  let writeThrough = false;\n  if (typeof arg === \"function\") {\n    getter = arg;\n  } else {\n    getter = arg.get;\n    setter = arg.set;\n    writeThrough = arg.writeThrough ?? false;\n  }\n\n  const cachedValue = ref<T>(getter());\n  watch(\n    getter,\n    (newValue) => {\n      if (!isJsonEqual(newValue, cachedValue.value)) {\n        // `deepClone` is needed because in case some fields are patched the deep would be triggered,\n        // but objects would be equal as the saved value was also patched\n        cachedValue.value = deepClone(newValue);\n      }\n    },\n    { deep: true },\n  );\n\n  if (setter) {\n    return computed({\n      get: () => cachedValue.value,\n      set: writeThrough\n        ? (newValue) => {\n            // Reflect the value in the cache now; `set` itself may defer its work.\n            if (!isJsonEqual(newValue, cachedValue.value)) cachedValue.value = deepClone(newValue);\n            setter(newValue);\n          }\n        : setter,\n    });\n  } else {\n    return cachedValue;\n  }\n}\n"],"mappings":";;;AAyBA,SAAgB,EACd,GAOA;CACA,IAAI,GACA,GACA,IAAe;AACnB,CAAI,OAAO,KAAQ,aACjB,IAAS,KAET,IAAS,EAAI,KACb,IAAS,EAAI,KACb,IAAe,EAAI,gBAAgB;CAGrC,IAAM,IAAc,EAAO,GAAQ,CAAC;AAyBlC,QAxBF,EACE,IACC,MAAa;AACZ,EAAK,EAAY,GAAU,EAAY,MAAM,KAG3C,EAAY,QAAQ,EAAU,EAAS;IAG3C,EAAE,MAAM,IAAM,CACf,EAEG,IACK,EAAS;EACd,WAAW,EAAY;EACvB,KAAK,KACA,MAAa;AAGZ,GADK,EAAY,GAAU,EAAY,MAAM,KAAE,EAAY,QAAQ,EAAU,EAAS,GACtF,EAAO,EAAS;MAElB;EACL,CAAC,GAEK"}
+{"version":3,"file":"computedCached.js","names":[],"sources":["../../src/composition/computedCached.ts"],"sourcesContent":["import { deepClone, isJsonEqual } from \"@milaboratories/helpers\";\nimport {\n  computed,\n  ref,\n  watch,\n  type ComputedGetter,\n  type ComputedSetter,\n  type ComputedRef,\n  type WritableComputedRef,\n} from \"vue\";\n\n/**\n * Alternative to `computed`, but triggering only on actual data changes.\n * Always `deep` as the plain `computed` is.\n *\n * With `writeThrough`, a set updates the cache synchronously, so a read reflects\n * the write immediately even when `set` defers its work (e.g. debounced); the\n * getter must then map set values to themselves (`get(x)` deep-equals `x`).\n *\n * A deferred set also leaves a window in which the source still holds the value\n * the write moved away from, and anything that re-emits the source during that\n * window — a running block having its project state pushed back, say — used to\n * revert the cache to it. The next write moved the cache forward again: a\n * ping-pong that never settled, and that whoever watches the value (the data\n * table rebuilds its whole grid) pays for on every swing. So a write-through\n * value stays authoritative until the source reports it back; since `get` maps\n * set values to themselves, that is exactly when the round trip has completed.\n * A change from elsewhere that arrives mid-flight is therefore dropped in\n * favour of the local write, which is about to land anyway.\n */\nexport function computedCached<T>(options: {\n  get: ComputedGetter<T>;\n  set: ComputedSetter<T>;\n  writeThrough?: boolean;\n}): WritableComputedRef<T>;\nexport function computedCached<T>(getter: ComputedGetter<T>): ComputedRef<T>;\nexport function computedCached<T>(\n  arg:\n    | ComputedGetter<T>\n    | {\n        get: ComputedGetter<T>;\n        set: ComputedSetter<T>;\n        writeThrough?: boolean;\n      },\n) {\n  let getter: ComputedGetter<T>;\n  let setter: ComputedSetter<T> | undefined = undefined;\n  let writeThrough = false;\n  if (typeof arg === \"function\") {\n    getter = arg;\n  } else {\n    getter = arg.get;\n    setter = arg.set;\n    writeThrough = arg.writeThrough ?? false;\n  }\n\n  const cachedValue = ref<T>(getter());\n  /** A write-through value the source has not reported back yet. */\n  let writePending = false;\n  watch(\n    getter,\n    (newValue) => {\n      if (isJsonEqual(newValue, cachedValue.value)) {\n        // The source has caught up with the write (or never diverged from it).\n        writePending = false;\n        return;\n      }\n      if (writePending) return;\n      // `deepClone` is needed because in case some fields are patched the deep would be triggered,\n      // but objects would be equal as the saved value was also patched\n      cachedValue.value = deepClone(newValue);\n    },\n    { deep: true },\n  );\n\n  if (setter) {\n    return computed({\n      get: () => cachedValue.value,\n      set: writeThrough\n        ? (newValue) => {\n            // Reflect the value in the cache now; `set` itself may defer its work.\n            if (!isJsonEqual(newValue, cachedValue.value)) {\n              cachedValue.value = deepClone(newValue);\n              writePending = true;\n            }\n            setter(newValue);\n          }\n        : setter,\n    });\n  } else {\n    return cachedValue;\n  }\n}\n"],"mappings":";;;AAoCA,SAAgB,EACd,GAOA;CACA,IAAI,GACA,GACA,IAAe;AACnB,CAAI,OAAO,KAAQ,aACjB,IAAS,KAET,IAAS,EAAI,KACb,IAAS,EAAI,KACb,IAAe,EAAI,gBAAgB;CAGrC,IAAM,IAAc,EAAO,GAAQ,CAAC,EAEhC,IAAe;AAgCjB,QA/BF,EACE,IACC,MAAa;AACZ,MAAI,EAAY,GAAU,EAAY,MAAM,EAAE;AAE5C,OAAe;AACf;;AAEE,QAGJ,EAAY,QAAQ,EAAU,EAAS;IAEzC,EAAE,MAAM,IAAM,CACf,EAEG,IACK,EAAS;EACd,WAAW,EAAY;EACvB,KAAK,KACA,MAAa;AAMZ,GAJK,EAAY,GAAU,EAAY,MAAM,KAC3C,EAAY,QAAQ,EAAU,EAAS,EACvC,IAAe,KAEjB,EAAO,EAAS;MAElB;EACL,CAAC,GAEK"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@milaboratories/uikit",
-  "version": "2.15.32",
+  "version": "2.15.34",
   "type": "module",
   "main": "dist/index.js",
   "exports": {
@@ -31,8 +31,8 @@
     "resize-observer-polyfill": "^1.5.1",
     "sortablejs": "^1.15.6",
     "vue": "3.5.24",
-    "@milaboratories/helpers": "1.14.5",
-    "@platforma-sdk/model": "1.84.1"
+    "@platforma-sdk/model": "1.84.3",
+    "@milaboratories/helpers": "1.14.5"
   },
   "devDependencies": {
     "@vitest/coverage-istanbul": "^4.1.3",
@@ -41,8 +41,8 @@
     "typescript": "7.0.2",
     "vitest": "^4.1.3",
     "@milaboratories/ts-builder": "1.7.2",
-    "@milaboratories/build-configs": "2.0.2",
-    "@milaboratories/ts-configs": "1.4.0"
+    "@milaboratories/ts-configs": "1.4.0",
+    "@milaboratories/build-configs": "2.0.2"
   },
   "scripts": {
     "dev": "ts-builder serve --target browser-lib --build-config ./build.browser-lib.config.js",

package/src/composition/computedCached.ts

@@ -16,6 +16,17 @@ import {
  * With `writeThrough`, a set updates the cache synchronously, so a read reflects
  * the write immediately even when `set` defers its work (e.g. debounced); the
  * getter must then map set values to themselves (`get(x)` deep-equals `x`).
+ *
+ * A deferred set also leaves a window in which the source still holds the value
+ * the write moved away from, and anything that re-emits the source during that
+ * window — a running block having its project state pushed back, say — used to
+ * revert the cache to it. The next write moved the cache forward again: a
+ * ping-pong that never settled, and that whoever watches the value (the data
+ * table rebuilds its whole grid) pays for on every swing. So a write-through
+ * value stays authoritative until the source reports it back; since `get` maps
+ * set values to themselves, that is exactly when the round trip has completed.
+ * A change from elsewhere that arrives mid-flight is therefore dropped in
+ * favour of the local write, which is about to land anyway.
  */
 export function computedCached<T>(options: {
   get: ComputedGetter<T>;
@@ -44,14 +55,20 @@ export function computedCached<T>(
   }
 
   const cachedValue = ref<T>(getter());
+  /** A write-through value the source has not reported back yet. */
+  let writePending = false;
   watch(
     getter,
     (newValue) => {
-      if (!isJsonEqual(newValue, cachedValue.value)) {
-        // `deepClone` is needed because in case some fields are patched the deep would be triggered,
-        // but objects would be equal as the saved value was also patched
-        cachedValue.value = deepClone(newValue);
+      if (isJsonEqual(newValue, cachedValue.value)) {
+        // The source has caught up with the write (or never diverged from it).
+        writePending = false;
+        return;
       }
+      if (writePending) return;
+      // `deepClone` is needed because in case some fields are patched the deep would be triggered,
+      // but objects would be equal as the saved value was also patched
+      cachedValue.value = deepClone(newValue);
     },
     { deep: true },
   );
@@ -62,7 +79,10 @@ export function computedCached<T>(
       set: writeThrough
         ? (newValue) => {
             // Reflect the value in the cache now; `set` itself may defer its work.
-            if (!isJsonEqual(newValue, cachedValue.value)) cachedValue.value = deepClone(newValue);
+            if (!isJsonEqual(newValue, cachedValue.value)) {
+              cachedValue.value = deepClone(newValue);
+              writePending = true;
+            }
             setter(newValue);
           }
         : setter,