@quenty/rxbinderutils 9.2.0 → 9.3.0

package/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [9.3.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/rxbinderutils@9.2.0...@quenty/rxbinderutils@9.3.0) (2023-12-14)
+
+
+### Features
+
+* RxBinderUtils.observeAllArrayBrio ([#408](https://github.com/Quenty/NevermoreEngine/issues/408)) ([98da8c2](https://github.com/Quenty/NevermoreEngine/commit/98da8c29f61965ebf8ad516a650ccabab7ce6bdb))
+
+
+
+
+
 # [9.2.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/rxbinderutils@9.1.1...@quenty/rxbinderutils@9.2.0) (2023-12-14)
 
 **Note:** Version bump only for package @quenty/rxbinderutils

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quenty/rxbinderutils",
-  "version": "9.2.0",
+  "version": "9.3.0",
   "description": "Reactive extensions for binders",
   "keywords": [
     "Roblox",
@@ -37,5 +37,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "2c2dbbc0cb2fbb46b4f3270c559c63890fe18b26"
+  "gitHead": "d98c14a2c2c27cc066f3b188562a509e4b717d1c"
 }

package/src/Shared/RxBinderUtils.lua

@@ -229,4 +229,52 @@ function RxBinderUtils.observeAllBrio(binder)
 	end)
 end
 
+--[=[
+	Observes all instances bound to the given binder as an unordered array.
+
+	@param binder Binder
+	@return Observable<Brio<{ T }>>
+]=]
+function RxBinderUtils.observeAllArrayBrio(binder)
+	assert(Binder.isBinder(binder), "Bad binder")
+
+	return Observable.new(function(sub)
+		local maid = Maid.new()
+
+		local array = binder:GetAll()
+
+		local function emit()
+			maid._brio = Brio.new(array)
+			sub:Fire(maid._brio)
+		end
+
+		maid:GiveTask(binder:GetClassAddedSignal():Connect(function(class)
+			table.insert(array, class)
+			emit()
+		end))
+		maid:GiveTask(binder:GetClassRemovingSignal():Connect(function(class)
+			local idx: number? = table.find(array, class)
+			if not idx then
+				return
+			end
+			-- Avoid 'table.remove'; that would suck with a very large list.
+			-- We're assuming order doesn't matter. Instead, move the back element of the array over.
+			-- From earlier benchmarking, calling #arr each time is faster than caching.
+			if idx == #array then
+				-- Just truncate. Handles case where array is 1 item.
+				array[idx] = nil
+			else
+				-- Move back element forward.
+				array[idx] = array[#array]
+				array[#array] = nil
+			end
+			emit()
+		end))
+
+		emit()
+
+		return maid
+	end)
+end
+
 return RxBinderUtils