ctx-core 4.7.0 → 4.8.1

package/all/rmemo/index.js

@@ -13,13 +13,12 @@ let queue = new Set
  * @private
  */
 export function memo_(rmemo_def, ...subscriber_a) {
-	let memor
 	let memo = ()=>{
 		if (!('val' in memo)) {
 			memo.f()
 		}
 		if (cur_memo) {
-			if (!~memor.indexOf(cur_memo.r ||= new WeakRef(cur_memo.f))) memor.push(cur_memo.r)
+			if (!~memo.memor.indexOf(cur_memo.r ||= new WeakRef(cur_memo.f))) memo.memor.push(cur_memo.r)
 			if (cur_memo.f.l < memo.f.l + 1) cur_memo.f.l = memo.f.l + 1
 			// conditional in r calls this r_memo
 			cur_memo.f.s.push(memo)
@@ -36,23 +35,14 @@ export function memo_(rmemo_def, ...subscriber_a) {
 			if (i !== val) {
 				// val is available for other purposes
 				let run_queue = !queue.size
-				i = 0
-				for (let r of memor) {
+				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) {
-						memor.splice(i, 1)
-					} else if (~val.s.indexOf(memo)) { // if conditional r refresh calls this r_memo, add to queue
+					if (val && ~val.s.indexOf(memo)) { // if conditional r refresh calls this r_memo, add to queue
 						queue.add(val)
 					}
-					i++
-				}
-				// add reference to subscribers to prevent GC
-				memo.b ||=
-					subscriber_a.map(subscriber=>
-						memo_(subscriber$=>(
-							subscriber(memo),
-							subscriber$
-						))())
+					return val
+				})
 				if (run_queue) {
 					cur_refresh_loop:for (let cur_refresh of queue) {
 						queue.delete(cur_refresh)
@@ -78,11 +68,18 @@ 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 = []
 	memo.f.S = []
-	memo.memor = memor = []
+	memo.memor = []
 	return memo
 }
 export { memo_ as memosig_ }

package/all/rmemo/index.test.ts

@@ -153,6 +153,22 @@ test('sig_|undefined', ()=>{
 	equal(sig(), undefined)
 	equal(memo(), undefined)
 })
+test('sig_|subscriber|sets sig', ()=>{
+	const base$ = sig_(0)
+	let count = 0
+	const num$ = sig_(
+		0,
+		async (num$)=>{
+			count++
+			num$._ = base$() + 1
+		})
+	equal(count, 0)
+	equal(num$(), 1)
+	equal(count, 1)
+	base$._ = 5
+	equal(num$(), 6)
+	equal(count, 2)
+})
 test('sig_|async subsubscriber|case 1', async ()=>{
 	let resolve:(user:{ id:string })=>void
 	const user0 = { id: 'id-0' }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ctx-core",
-  "version": "4.7.0",
+  "version": "4.8.1",
   "description": "ctx-core core library",
   "keywords": [
     "ctx-core",
@@ -114,7 +114,7 @@
     "sinon": "^17.0.1",
     "size-limit": "^11.0.0",
     "ts-node": "^10.9.1",
-    "tsx": "^4.5.0",
+    "tsx": "^4.6.0",
     "typescript": "next",
     "uvu": "^0.5.6"
   },
@@ -150,28 +150,28 @@
       "import": {
         "./rmemo": "{ memo_ }"
       },
-      "limit": "344 B"
+      "limit": "342 B"
     },
     {
       "name": "memo_ sig_",
       "import": {
         "./rmemo": "{ sig_, memo_ }"
       },
-      "limit": "360 B"
+      "limit": "357 B"
     },
     {
       "name": "memo_ sig_ be_ ctx_",
       "import": {
         "./rmemo": "{ sig_, memo_, be_, ctx_ }"
       },
-      "limit": "519 B"
+      "limit": "511 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": "619 B"
+      "limit": "607 B"
     }
   ],
   "scripts": {