@quenty/binder 14.36.0 → 14.37.0

package/CHANGELOG.md

@@ -3,6 +3,20 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [14.37.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/binder@14.36.1...@quenty/binder@14.37.0) (2026-07-14)
+
+### Bug Fixes
+
+- **lint:** resolve selene unused-self and Rx shadowing from --!strict conversions ([3f4a085](https://github.com/Quenty/NevermoreEngine/commit/3f4a085911808146f10a2435efbe9e89dff8edff))
+
+### Features
+
+- **binder:** convert package to --!strict ([78f42c2](https://github.com/Quenty/NevermoreEngine/commit/78f42c2265f2ef9313db146220cb15e87baf6747))
+
+## [14.36.1](https://github.com/Quenty/NevermoreEngine/compare/@quenty/binder@14.36.0...@quenty/binder@14.36.1) (2026-05-30)
+
+**Note:** Version bump only for package @quenty/binder
+
 # [14.36.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/binder@14.35.0...@quenty/binder@14.36.0) (2026-05-29)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quenty/binder",
-  "version": "14.36.0",
+  "version": "14.37.0",
   "description": "Utility object to Bind a class to Roblox object, and associated helper methods",
   "keywords": [
     "Roblox",
@@ -29,23 +29,23 @@
   ],
   "dependencies": {
     "@quenty/baseobject": "10.13.0",
-    "@quenty/brio": "14.30.0",
-    "@quenty/canceltoken": "11.19.0",
-    "@quenty/instanceutils": "13.30.0",
-    "@quenty/linkutils": "13.30.0",
+    "@quenty/brio": "14.30.1",
+    "@quenty/canceltoken": "11.19.1",
+    "@quenty/instanceutils": "13.30.1",
+    "@quenty/linkutils": "13.31.0",
     "@quenty/loader": "10.11.0",
     "@quenty/maid": "3.9.0",
     "@quenty/nevermore-test-runner": "1.4.0",
-    "@quenty/promise": "10.18.0",
-    "@quenty/rx": "13.28.2",
-    "@quenty/servicebag": "11.18.0",
-    "@quenty/signal": "7.13.0",
+    "@quenty/promise": "10.18.1",
+    "@quenty/rx": "13.28.3",
+    "@quenty/servicebag": "11.18.1",
+    "@quenty/signal": "7.13.1",
     "@quenty/table": "3.9.2",
-    "@quenty/valueobject": "13.31.0",
+    "@quenty/valueobject": "13.31.1",
     "@quentystudios/jest-lua": "3.10.0-quenty.2"
   },
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "f4a374a0a294ee8900aa5cb68ab138b0acf3e0ae"
+  "gitHead": "13f0162f4971a77378e55e9b7236aea94f0dd3a8"
 }

package/src/Shared/BinderGroupProvider.lua

@@ -1,4 +1,4 @@
---!nonstrict
+--!strict
 --[=[
 	Provides a basis for binderGroups that can be retrieved anywhere
 	@class BinderGroupProvider
@@ -13,13 +13,23 @@ BinderGroupProvider.ClassName = "BinderGroupProvider"
 BinderGroupProvider.ServiceName = "BinderGroupProvider"
 BinderGroupProvider.__index = BinderGroupProvider
 
+export type BinderGroupProvider = typeof(setmetatable(
+	{} :: {
+		_initMethod: (BinderGroupProvider, ...any) -> (),
+		_groupsAddedPromise: Promise.Promise<()>,
+		_init: boolean,
+		_binderGroups: { any },
+	},
+	{} :: typeof({ __index = BinderGroupProvider })
+))
+
 --[=[
 	Constructs a new BinderGroupProvider
 	@param initMethod (BinderGroupProvider) -> ()
 	@return BinderGroupProvider
 ]=]
-function BinderGroupProvider.new(initMethod)
-	local self = setmetatable({}, BinderGroupProvider)
+function BinderGroupProvider.new(initMethod: (BinderGroupProvider, ...any) -> ()): BinderGroupProvider
+	local self: BinderGroupProvider = setmetatable({} :: any, BinderGroupProvider)
 
 	self._initMethod = initMethod or error("No initMethod")
 	self._groupsAddedPromise = Promise.new()
@@ -34,7 +44,7 @@ end
 	Returns a promise that will resolve once groups are added.
 	@return Promise
 ]=]
-function BinderGroupProvider:PromiseGroupsAdded()
+function BinderGroupProvider.PromiseGroupsAdded(self: BinderGroupProvider): Promise.Promise<()>
 	return self._groupsAddedPromise
 end
 
@@ -42,7 +52,7 @@ end
 	Starts the binder provider. Should be called via ServiceBag.
 	@param ... ServiceBag | any
 ]=]
-function BinderGroupProvider:Init(...)
+function BinderGroupProvider.Init(self: BinderGroupProvider, ...: any): ()
 	assert(not self._init, "Already initialized")
 
 	self._initMethod(self, ...)
@@ -54,11 +64,11 @@ end
 --[=[
 	Starts the binder provider. Should be called via ServiceBag.
 ]=]
-function BinderGroupProvider:Start()
+function BinderGroupProvider.Start(_self: BinderGroupProvider): ()
 	-- Do nothing
 end
 
-function BinderGroupProvider:__index(index)
+(BinderGroupProvider :: any).__index = function(_self, index)
 	if BinderGroupProvider[index] then
 		return BinderGroupProvider[index]
 	end
@@ -72,9 +82,9 @@ end
 	@param groupName string
 	@return BinderGroup?
 ]=]
-function BinderGroupProvider:Get(groupName: string)
+function BinderGroupProvider.Get(self: BinderGroupProvider, groupName: string): any
 	assert(type(groupName) == "string", "Bad groupName")
-	return rawget(self, groupName)
+	return rawget(self :: any, groupName)
 end
 
 --[=[
@@ -83,17 +93,17 @@ end
 	@param groupName string
 	@param binderGroup BinderGroup
 ]=]
-function BinderGroupProvider:Add(groupName, binderGroup)
+function BinderGroupProvider.Add(self: BinderGroupProvider, groupName: string, binderGroup: any): ()
 	assert(type(groupName) == "string", "Bad groupName")
 	assert(type(binderGroup) == "table", "Bad binderGroup")
 	assert(not self._init, "Already initialized")
 	assert(not self:Get(groupName), "Duplicate groupName")
 
-	table.insert(self._binderGroups, binderGroup)
-	self[groupName] = binderGroup
+	table.insert(self._binderGroups, binderGroup);
+	(self :: any)[groupName] = binderGroup
 end
 
-function BinderGroupProvider:Destroy()
+function BinderGroupProvider.Destroy(_self: BinderGroupProvider): ()
 	-- Do nothing
 end
 

package/src/Shared/BinderProvider.lua

@@ -1,4 +1,4 @@
---!nonstrict
+--!strict
 --[=[
 	Provides a basis for binders that can be retrieved anywhere
 	@class BinderProvider
@@ -6,6 +6,7 @@
 
 local require = require(script.Parent.loader).load(script)
 
+local Binder = require("Binder")
 local Maid = require("Maid")
 local Promise = require("Promise")
 
@@ -14,6 +15,21 @@ BinderProvider.ClassName = "BinderProvider"
 BinderProvider.ServiceName = "BinderProvider"
 BinderProvider.__index = BinderProvider
 
+export type BinderProvider = typeof(setmetatable(
+	{} :: {
+		ServiceName: string,
+		_initMethod: (self: BinderProvider, ...any) -> (),
+		_initialized: boolean,
+		_destroyed: boolean,
+		_started: boolean,
+		_maid: Maid.Maid,
+		_binders: { any },
+		_bindersAddedPromise: Promise.Promise<()>,
+		_startPromise: Promise.Promise<()>,
+	},
+	{} :: typeof({ __index = BinderProvider })
+))
+
 --[=[
 	Constructs a new BinderProvider.
 
@@ -43,8 +59,8 @@ BinderProvider.__index = BinderProvider
 	@param initMethod (self, serviceBag: ServiceBag)
 	@return BinderProvider
 ]=]
-function BinderProvider.new(serviceName: string, initMethod)
-	local self = setmetatable({}, BinderProvider)
+function BinderProvider.new(serviceName: string, initMethod: ((self: BinderProvider, ...any) -> ())?): BinderProvider
+	local self: BinderProvider = setmetatable({} :: any, BinderProvider)
 
 	if type(serviceName) == "string" then
 		self.ServiceName = serviceName
@@ -83,7 +99,7 @@ end
 	@param binderName string
 	@return Promise<Binder<T>>
 ]=]
-function BinderProvider:PromiseBinder(binderName)
+function BinderProvider.PromiseBinder(self: BinderProvider, binderName: string): Promise.Promise<Binder.Binder<any>>
 	if self._bindersAddedPromise:IsFulfilled() then
 		local binder = self:Get(binderName)
 		if binder then
@@ -100,7 +116,7 @@ function BinderProvider:PromiseBinder(binderName)
 		else
 			return Promise.rejected()
 		end
-	end)
+	end) :: any
 end
 
 --[=[
@@ -108,7 +124,7 @@ end
 
 	@param ... ServiceBag | any
 ]=]
-function BinderProvider:Init(...)
+function BinderProvider.Init(self: BinderProvider, ...: any): ()
 	assert(not self._initialized, "Already initialized")
 
 	self._maid = Maid.new()
@@ -134,7 +150,7 @@ end
 
 	@return Promise
 ]=]
-function BinderProvider:PromiseBindersAdded()
+function BinderProvider.PromiseBindersAdded(self: BinderProvider): Promise.Promise<()>
 	return assert(self._bindersAddedPromise, "Be sure to require via serviceBag")
 end
 
@@ -143,14 +159,14 @@ end
 
 	@return Promise
 ]=]
-function BinderProvider:PromiseBindersStarted()
+function BinderProvider.PromiseBindersStarted(self: BinderProvider): Promise.Promise<()>
 	return assert(self._startPromise, "Be sure to require via serviceBag")
 end
 
 --[=[
 	Starts all of the binders.
 ]=]
-function BinderProvider:Start()
+function BinderProvider.Start(self: BinderProvider): ()
 	assert(self._initialized, "Not initialized")
 	assert(not self._started, "Already started")
 
@@ -161,8 +177,7 @@ function BinderProvider:Start()
 
 	self._startPromise:Resolve()
 end
-
-function BinderProvider:__index(index)
+(BinderProvider :: any).__index = function(self, index)
 	if BinderProvider[index] then
 		return BinderProvider[index]
 	end
@@ -180,9 +195,9 @@ end
 	@param tagName string
 	@return Binder<T>?
 ]=]
-function BinderProvider:Get(tagName: string)
+function BinderProvider.Get(self: BinderProvider, tagName: string): Binder.Binder<any>?
 	assert(type(tagName) == "string", "Bad tagName")
-	return rawget(self, tagName)
+	return rawget(self :: any, tagName)
 end
 
 --[=[
@@ -190,30 +205,30 @@ end
 
 	@param binder Binder<T>
 ]=]
-function BinderProvider:Add(binder)
+function BinderProvider.Add(self: BinderProvider, binder: Binder.Binder<any>): ()
 	assert(not self._started, "Already inited")
 	assert(not self:Get(binder:GetTag()), "Binder already exists")
 
 	self._maid:GiveTask(binder)
 
-	table.insert(self._binders, binder)
-	self[binder:GetTag()] = binder
+	table.insert(self._binders, binder);
+	(self :: any)[binder:GetTag()] = binder
 end
 
-function BinderProvider:Destroy()
+function BinderProvider.Destroy(self: BinderProvider): ()
 	self._destroyed = true
 
-	local binders = rawget(self, "_binders")
-	rawset(self, "_binders", nil)
+	local binders = rawget(self :: any, "_binders")
+	rawset(self :: any, "_binders", nil)
 
 	if binders then
 		for _, item in binders do
-			rawset(self, item:GetTag(), nil)
+			rawset(self :: any, item:GetTag(), nil)
 		end
 	end
 
-	local maid = rawget(self, "_maid")
-	rawset(self, "_maid", nil)
+	local maid = rawget(self :: any, "_maid")
+	rawset(self :: any, "_maid", nil)
 
 	if maid then
 		maid:DoCleaning()

package/src/Shared/Trackers/BoundAncestorTracker.lua

@@ -1,4 +1,4 @@
---!nonstrict
+--!strict
 --[=[
 	Tracks a parent bound to a specific binder
 	@class BoundAncestorTracker
@@ -7,6 +7,7 @@
 local require = require(script.Parent.loader).load(script)
 
 local BaseObject = require("BaseObject")
+local Binder = require("Binder")
 local BinderUtils = require("BinderUtils")
 local ValueObject = require("ValueObject")
 
@@ -14,6 +15,17 @@ local BoundAncestorTracker = setmetatable({}, BaseObject)
 BoundAncestorTracker.ClassName = "BoundAncestorTracker"
 BoundAncestorTracker.__index = BoundAncestorTracker
 
+export type BoundAncestorTracker<T> =
+	typeof(setmetatable(
+		{} :: {
+			_child: Instance,
+			_binder: Binder.Binder<T>,
+			Class: ValueObject.ValueObject<T?>,
+		},
+		{} :: typeof({ __index = BoundAncestorTracker })
+	))
+	& BaseObject.BaseObject
+
 --[=[
 Constructs a new BoundAncestorTracker
 
@@ -21,8 +33,8 @@ Constructs a new BoundAncestorTracker
 @param child Instance
 @return BoundAncestorTracker
 ]=]
-function BoundAncestorTracker.new(binder, child)
-	local self = setmetatable(BaseObject.new(), BoundAncestorTracker)
+function BoundAncestorTracker.new<T>(binder: Binder.Binder<T>, child: Instance): BoundAncestorTracker<T>
+	local self: BoundAncestorTracker<T> = setmetatable(BaseObject.new() :: any, BoundAncestorTracker)
 
 	self._child = child or error("No child")
 	self._binder = binder or error("No binder")
@@ -33,7 +45,7 @@ function BoundAncestorTracker.new(binder, child)
 	@within BoundAncestorTracker
 	Bound value
 ]=]
-	self.Class = ValueObject.new()
+	self.Class = ValueObject.new() :: ValueObject.ValueObject<T?>
 	self._maid:GiveTask(self.Class)
 
 	-- Handle instance removing
@@ -58,7 +70,7 @@ function BoundAncestorTracker.new(binder, child)
 	return self
 end
 
-function BoundAncestorTracker:_update()
+function BoundAncestorTracker._update<T>(self: BoundAncestorTracker<T>): ()
 	local parent = self._child.Parent
 	if not parent then
 		self.Class.Value = nil

package/src/Shared/Trackers/BoundParentTracker.lua

@@ -1,4 +1,4 @@
---!nonstrict
+--!strict
 --[=[
 	Tracks a parent bound to a specific binder
 	@class BoundParentTracker
@@ -7,20 +7,32 @@
 local require = require(script.Parent.loader).load(script)
 
 local BaseObject = require("BaseObject")
+local Binder = require("Binder")
 local ValueObject = require("ValueObject")
 
 local BoundParentTracker = setmetatable({}, BaseObject)
 BoundParentTracker.ClassName = "BoundParentTracker"
 BoundParentTracker.__index = BoundParentTracker
 
-function BoundParentTracker.new(binder, child)
-	local self = setmetatable(BaseObject.new(), BoundParentTracker)
+export type BoundParentTracker<T> =
+	typeof(setmetatable(
+		{} :: {
+			_child: Instance,
+			_binder: Binder.Binder<T>,
+			Class: ValueObject.ValueObject<T?>,
+		},
+		{} :: typeof({ __index = BoundParentTracker })
+	))
+	& BaseObject.BaseObject
+
+function BoundParentTracker.new<T>(binder: Binder.Binder<T>, child: Instance): BoundParentTracker<T>
+	local self: BoundParentTracker<T> = setmetatable(BaseObject.new() :: any, BoundParentTracker)
 
 	self._child = child or error("No child")
 	self._binder = binder or error("No binder")
 
 	-- Bound value
-	self.Class = ValueObject.new()
+	self.Class = ValueObject.new(nil :: T?)
 	self._maid:GiveTask(self.Class)
 
 	-- Handle instance removing
@@ -39,7 +51,7 @@ function BoundParentTracker.new(binder, child)
 	return self
 end
 
-function BoundParentTracker:_update()
+function BoundParentTracker._update<T>(self: BoundParentTracker<T>): ()
 	local parent = self._child.Parent
 	if not parent then
 		self.Class.Value = nil