ctx-core 4.9.0 → 4.9.1

package/all/rmemo/index.d.ts

@@ -10,6 +10,7 @@ export declare function memosig_<val_T>(
 	def:rmemo_def_T<val_T>,
 	...subscriber_a:rmemo_subscriber_T<val_T>[]
 ):sig_T<val_T>
+// TODO: lock_memosig_
 export type rmemo_T<val_T> = memo_T<val_T>|sig_T<val_T>
 export type memo_T<val_T> = (()=>val_T)&{
 	readonly _:val_T

package/all/rmemo/index.js

@@ -1,7 +1,4 @@
-/** @typedef {import('./index.d.ts').rmemo_def_T} */
-/** @typedef {import('./index.d.ts').memo_T} */
-/** @typedef {import('./index.d.ts').rmemo_subscriber_T} */
-/** @typedef {import('./index.d.ts').sig_T} */
+/// <reference types="./index.d.ts" />
 /** @type {WeakRef<memo_T>} */
 let cur_memo
 /** @type {Set<()=>unknown>} */
@@ -30,30 +27,36 @@ export function memo_(rmemo_def, ...subscriber_a) {
 	Object.defineProperty(memo, '_', {
 		get: memo,
 		set: val=>{
-			let i = memo.val
-			memo.val = val
-			if (i !== val) {
-				// val is available for other purposes
-				let run_queue = !queue.size
+			let run_queue
+			if (memo.val !== val) {
+				run_queue = !queue.size
 				memo.memor = memo.memor.filter(r=>{
-					// val is no longer used...saving bytes
-					val = r.deref() // val is no longer used...saving bytes
-					if (val && ~val.s.indexOf(memo)) { // if conditional r refresh calls this r_memo, add to queue
-						queue.add(val)
+					r = r.deref()
+					if (r && ~r.s.indexOf(memo)) { // if conditional r refresh calls this r_memo, add to queue
+						queue.add(r)
 					}
-					return val
+					return r
 				})
-				if (run_queue) {
-					cur_refresh_loop:for (let cur_refresh of queue) {
-						queue.delete(cur_refresh)
-						for (let queue_refresh of queue) {
-							if (cur_refresh.l > queue_refresh.l) {
-								queue.add(cur_refresh)
-								continue cur_refresh_loop
-							}
+			}
+			memo.val = val
+			if (!memo.b) {
+				// add reference to subscribers to prevent GC
+				memo.b = subscriber_a.map(subscriber=>
+					memo_(()=>subscriber(memo)))
+				for (let s of memo.b) {
+					s()
+				}
+			}
+			if (run_queue) {
+				cur_refresh_loop:for (let cur_refresh of queue) {
+					queue.delete(cur_refresh)
+					for (let queue_refresh of queue) {
+						if (cur_refresh.l > queue_refresh.l) {
+							queue.add(cur_refresh)
+							continue cur_refresh_loop
 						}
-						cur_refresh()
 					}
+					cur_refresh()
 				}
 			}
 		},
@@ -68,13 +71,6 @@ export function memo_(rmemo_def, ...subscriber_a) {
 			console.error(err)
 		}
 		cur_memo = prev_memo // finally is not necessary...catch does not throw
-		// add reference to subscribers to prevent GC
-		memo.b ||=
-			subscriber_a.map(subscriber=>
-				memo_(subscriber$=>(
-					subscriber(memo),
-					subscriber$
-				))())
 	}
 	memo.f.l = 0
 	memo.f.s = []
@@ -97,3 +93,4 @@ export function sig_(init_val, ...subscriber_a) {
 	},
 	...subscriber_a)
 }
+// TODO: lock_memosig_

package/all/rmemo/index.test.ts

@@ -153,12 +153,30 @@ test('sig_|undefined', ()=>{
 	equal(sig(), undefined)
 	equal(memo(), undefined)
 })
+test('sig_|subscriber|notified if sig is set before read', ()=>{
+	let count = 0
+	let subscriber__num:number|undefined = undefined
+	const num$ = sig_<number|undefined>(
+		undefined,
+		num$=>{
+			count++
+			subscriber__num = num$()
+		})
+	equal(count, 0)
+	equal(subscriber__num, undefined)
+	num$._ = 1
+	equal(count, 1)
+	equal(subscriber__num, 1)
+	num$()
+	equal(count, 1)
+	equal(subscriber__num, 1)
+})
 test('sig_|subscriber|sets sig', ()=>{
 	const base$ = sig_(0)
 	let count = 0
 	const num$ = sig_(
 		0,
-		async (num$)=>{
+		async num$=>{
 			count++
 			num$._ = base$() + 1
 		})

package/all/weak_r/index.js

@@ -3,6 +3,7 @@ import { assign } from '../assign/index.js'
 const { get } = Reflect
 const add_strong_sym = Symbol('add_strong')
 const delete_strong_sym = Symbol('delete_strong')
+let _undefined
 /**
  * @param {object}[back_o]
  * @returns {object}
@@ -24,12 +25,16 @@ export function weak_r_(back_o = {}) {
 			if ((value === null || value === void 0 ? void 0 : value.deref) && !strong_set.has(prop)) {
 				value = value.deref()
 			}
-			return value === undefined ? undefined : {
-				configurable: true,
-				enumerable: true,
-				value,
-				writable: true
-			}
+			return (
+				value === _undefined
+					? _undefined
+					: {
+						configurable: true,
+						enumerable: true,
+						value,
+						writable: true
+					}
+			)
 		},
 		get(back_ctx, prop) {
 			const uw_val = get(back_ctx, prop)
@@ -61,7 +66,7 @@ export { strong__call as ref_strong }
  */
 export function strong__assign(obj, prop, val) {
 	strong__call(obj, prop)
-	if (val !== undefined) {
+	if (val !== _undefined) {
 		assign(obj, {
 			[prop]: val
 		})

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ctx-core",
-  "version": "4.9.0",
+  "version": "4.9.1",
   "description": "ctx-core core library",
   "keywords": [
     "ctx-core",
@@ -107,7 +107,7 @@
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.13.2",
     "@size-limit/preset-small-lib": "^11.0.0",
-    "@types/node": "^20.10.2",
+    "@types/node": "^20.10.3",
     "@types/sinon": "^17.0.2",
     "c8": "^8.0.1",
     "check-dts": "^0.7.2",
@@ -150,28 +150,28 @@
       "import": {
         "./rmemo": "{ memo_ }"
       },
-      "limit": "342 B"
+      "limit": "338 B"
     },
     {
       "name": "memo_ sig_",
       "import": {
         "./rmemo": "{ sig_, memo_ }"
       },
-      "limit": "357 B"
+      "limit": "354 B"
     },
     {
       "name": "memo_ sig_ be_ ctx_",
       "import": {
         "./rmemo": "{ sig_, memo_, be_, ctx_ }"
       },
-      "limit": "511 B"
+      "limit": "506 B"
     },
     {
       "name": "memo_ sig_ be_ ctx_ be_memo_pair_ be_sig_triple_",
       "import": {
         "./rmemo": "{ sig_, memo_, be_, ctx_, be_memo_pair_, be_sig_triple_ }"
       },
-      "limit": "599 B"
+      "limit": "602 B"
     }
   ],
   "scripts": {