@quenty/boundlinkutils 14.37.0 → 14.38.0

package/CHANGELOG.md

@@ -3,6 +3,16 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [14.38.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/boundlinkutils@14.37.1...@quenty/boundlinkutils@14.38.0) (2026-07-14)
+
+### Features
+
+- **boundlinkutils:** convert package to --!strict ([6cd3e7f](https://github.com/Quenty/NevermoreEngine/commit/6cd3e7f518ffb74c0ba5266bee786b91f879a99d))
+
+## [14.37.1](https://github.com/Quenty/NevermoreEngine/compare/@quenty/boundlinkutils@14.37.0...@quenty/boundlinkutils@14.37.1) (2026-05-30)
+
+**Note:** Version bump only for package @quenty/boundlinkutils
+
 # [14.37.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/boundlinkutils@14.36.0...@quenty/boundlinkutils@14.37.0) (2026-05-29)
 
 **Note:** Version bump only for package @quenty/boundlinkutils

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quenty/boundlinkutils",
-  "version": "14.37.0",
+  "version": "14.38.0",
   "description": "Utility functions involving binders and links",
   "keywords": [
     "Roblox",
@@ -28,15 +28,15 @@
     "Quenty"
   ],
   "dependencies": {
-    "@quenty/binder": "14.36.0",
-    "@quenty/linkutils": "13.30.0",
+    "@quenty/binder": "14.37.0",
+    "@quenty/linkutils": "13.31.0",
     "@quenty/loader": "10.11.0",
     "@quenty/maid": "3.9.0",
-    "@quenty/promise": "10.18.0",
-    "@quenty/signal": "7.13.0"
+    "@quenty/promise": "10.18.1",
+    "@quenty/signal": "7.13.1"
   },
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "f4a374a0a294ee8900aa5cb68ab138b0acf3e0ae"
+  "gitHead": "13f0162f4971a77378e55e9b7236aea94f0dd3a8"
 }

package/src/Shared/BoundLinkCollection.lua

@@ -1,4 +1,4 @@
---!nonstrict
+--!strict
 --[=[
 	Handles searching for bound objects following links (object values) under a parent
 	with a specific name.
@@ -8,6 +8,7 @@
 
 local require = require(script.Parent.loader).load(script)
 
+local Binder = require("Binder")
 local Maid = require("Maid")
 local Signal = require("Signal")
 
@@ -15,8 +16,23 @@ local BoundLinkCollection = {}
 BoundLinkCollection.ClassName = "BoundLinkCollection"
 BoundLinkCollection.__index = BoundLinkCollection
 
-function BoundLinkCollection.new(binder, linkName, parent)
-	local self = setmetatable({}, BoundLinkCollection)
+export type BoundLinkCollection = typeof(setmetatable(
+	{} :: {
+		_maid: Maid.Maid,
+		_linkName: string,
+		_binder: Binder.Binder<any>,
+		ClassAdded: Signal.Signal<any>,
+		ClassRemoved: Signal.Signal<any>,
+		_classes: { [any]: boolean },
+		_canidates: { [Instance]: { [Instance]: boolean } },
+		_linkCanidate: { [Instance]: Instance },
+		_linkValues: { [Instance]: boolean },
+	},
+	{} :: typeof({ __index = BoundLinkCollection })
+))
+
+function BoundLinkCollection.new(binder: Binder.Binder<any>, linkName: string, parent: Instance): BoundLinkCollection
+	local self: BoundLinkCollection = setmetatable({} :: any, BoundLinkCollection)
 
 	self._maid = Maid.new()
 
@@ -43,7 +59,7 @@ function BoundLinkCollection.new(binder, linkName, parent)
 	return self
 end
 
-function BoundLinkCollection:GetClasses()
+function BoundLinkCollection.GetClasses(self: BoundLinkCollection): { any }
 	local list = {}
 	for class, _ in self._classes do
 		table.insert(list, class)
@@ -51,11 +67,11 @@ function BoundLinkCollection:GetClasses()
 	return list
 end
 
-function BoundLinkCollection:HasClass(class)
+function BoundLinkCollection.HasClass(self: BoundLinkCollection, class: any): boolean
 	return self._classes[class] ~= nil
 end
 
-function BoundLinkCollection:TrackParent(parent: Instance)
+function BoundLinkCollection.TrackParent(self: BoundLinkCollection, parent: Instance): ()
 	assert(parent, "Bad parent")
 
 	self._maid:GiveTask(parent.ChildAdded:Connect(function(child)
@@ -75,11 +91,11 @@ function BoundLinkCollection:TrackParent(parent: Instance)
 	end
 end
 
-function BoundLinkCollection:_removeLink(objValue)
+function BoundLinkCollection._removeLink(self: BoundLinkCollection, objValue: Instance): ()
 	self._maid[objValue] = nil
 end
 
-function BoundLinkCollection:_handleNewLink(objValue)
+function BoundLinkCollection._handleNewLink(self: BoundLinkCollection, objValue: ObjectValue): ()
 	local maid = Maid.new()
 
 	maid:GiveTask(objValue.Changed:Connect(function()
@@ -95,7 +111,7 @@ function BoundLinkCollection:_handleNewLink(objValue)
 	self:_handleLinkChanged(objValue)
 end
 
-function BoundLinkCollection:_handleLinkChanged(objValue)
+function BoundLinkCollection._handleLinkChanged(self: BoundLinkCollection, objValue: ObjectValue): ()
 	self:_removeLinkCanidates(objValue)
 
 	if objValue.Value then
@@ -103,7 +119,7 @@ function BoundLinkCollection:_handleLinkChanged(objValue)
 	end
 end
 
-function BoundLinkCollection:_removeLinkCanidates(objValue)
+function BoundLinkCollection._removeLinkCanidates(self: BoundLinkCollection, objValue: Instance): ()
 	local canidate = self._linkCanidate[objValue]
 	if not canidate then
 		return
@@ -113,7 +129,6 @@ function BoundLinkCollection:_removeLinkCanidates(objValue)
 
 	if not self._canidates[canidate] then
 		error("[BoundLinkCollection] - Got link canidate that isn''t real. This shouldn't happen.")
-		return
 	end
 
 	local canidateLinks = self._canidates[canidate]
@@ -124,7 +139,7 @@ function BoundLinkCollection:_removeLinkCanidates(objValue)
 	end
 end
 
-function BoundLinkCollection:_removeCanidate(canidate)
+function BoundLinkCollection._removeCanidate(self: BoundLinkCollection, canidate: Instance): ()
 	self._canidates[canidate] = nil
 
 	local class = self._binder:Get(canidate)
@@ -135,7 +150,7 @@ function BoundLinkCollection:_removeCanidate(canidate)
 	self:_removeClass(class)
 end
 
-function BoundLinkCollection:_addCanidate(objValue, canidate)
+function BoundLinkCollection._addCanidate(self: BoundLinkCollection, objValue: Instance, canidate: Instance): ()
 	assert(not self._linkCanidate[objValue], "Should not have existing canidate set for link")
 
 	self._linkCanidate[objValue] = canidate
@@ -152,7 +167,7 @@ function BoundLinkCollection:_addCanidate(objValue, canidate)
 	self:_addClass(class)
 end
 
-function BoundLinkCollection:_removeClass(class)
+function BoundLinkCollection._removeClass(self: BoundLinkCollection, class: any): ()
 	if not self._classes[class] then
 		return
 	end
@@ -161,7 +176,7 @@ function BoundLinkCollection:_removeClass(class)
 	self.ClassRemoved:Fire(class)
 end
 
-function BoundLinkCollection:_addClass(class)
+function BoundLinkCollection._addClass(self: BoundLinkCollection, class: any): ()
 	if self._classes[class] then
 		return
 	end
@@ -170,7 +185,7 @@ function BoundLinkCollection:_addClass(class)
 	self.ClassAdded:Fire(class)
 end
 
-function BoundLinkCollection:_handleNewClassBound(class, inst)
+function BoundLinkCollection._handleNewClassBound(self: BoundLinkCollection, class: any, inst: Instance): ()
 	if not self._canidates[inst] then
 		return
 	end
@@ -178,9 +193,9 @@ function BoundLinkCollection:_handleNewClassBound(class, inst)
 	self:_addClass(class)
 end
 
-function BoundLinkCollection:Destroy()
+function BoundLinkCollection.Destroy(self: BoundLinkCollection): ()
 	self._maid:Destroy()
-	setmetatable(self, nil)
+	setmetatable(self :: any, nil)
 end
 
 return BoundLinkCollection

package/src/Shared/BoundLinkUtils.lua

@@ -1,4 +1,4 @@
---!nonstrict
+--!strict
 --[=[
 	Utility functions involving binders and links. It's a common pattern to link
 	back to a bound class. This allows you to quickly retrieve these objects.
@@ -10,6 +10,7 @@ local require = require(script.Parent.loader).load(script)
 
 local CollectionService = game:GetService("CollectionService")
 
+local Binder = require("Binder")
 local BinderUtils = require("BinderUtils")
 local LinkUtils = require("LinkUtils")
 
@@ -23,7 +24,7 @@ local BoundLinkUtils = {}
 	@param from Instance
 	@return T?
 ]=]
-function BoundLinkUtils.getLinkClass(binder, linkName, from)
+function BoundLinkUtils.getLinkClass<T>(binder: Binder.Binder<T>, linkName: string, from: Instance): T?
 	assert(type(binder) == "table", "Bad binder")
 	assert(type(linkName) == "string", "Bad linkName")
 	assert(typeof(from) == "Instance", "Bad froM")
@@ -44,12 +45,12 @@ end
 	@param from Instance
 	@return { T }
 ]=]
-function BoundLinkUtils.getLinkClasses(binder, linkName, from)
+function BoundLinkUtils.getLinkClasses<T>(binder: Binder.Binder<T>, linkName: string, from: Instance): { T }
 	assert(type(binder) == "table", "Bad binder")
 	assert(type(linkName) == "string", "Bad linkName")
 	assert(typeof(from) == "Instance", "Bad from")
 
-	local classes = {}
+	local classes: { T } = {}
 	for _, value in LinkUtils.getAllLinkValues(linkName, from) do
 		local class = binder:Get(value)
 		if class then
@@ -67,7 +68,7 @@ end
 	@param from Instance
 	@return { T }
 ]=]
-function BoundLinkUtils.getClassesForLinkValues(binders, linkName, from)
+function BoundLinkUtils.getClassesForLinkValues<T>(binders: { Binder.Binder<T> }, linkName: string, from: Instance): { T }
 	assert(type(binders) == "table", "Bad binders")
 	assert(type(linkName) == "string", "Bad linkName")
 	assert(typeof(from) == "Instance", "Bad from")
@@ -77,7 +78,7 @@ function BoundLinkUtils.getClassesForLinkValues(binders, linkName, from)
 	end
 
 	local tags = BinderUtils.mapBinderListToTable(binders)
-	local classes = {}
+	local classes: { T } = {}
 
 	for _, instance in LinkUtils.getAllLinkValues(linkName, from) do
 		for _, tag in CollectionService:GetTags(instance) do
@@ -103,7 +104,13 @@ end
 	@param methodName string
 	@param args {}
 ]=]
-function BoundLinkUtils.callMethodOnLinkedClasses(binders, linkName, from, methodName, args)
+function BoundLinkUtils.callMethodOnLinkedClasses(
+	binders: { Binder.Binder<any> },
+	linkName: string,
+	from: Instance,
+	methodName: string,
+	args: { any }
+)
 	assert(type(binders) == "table", "Bad arg 'binders'")
 	assert(type(linkName) == "string", "Bad arg 'linkName'")
 	assert(typeof(from) == "Instance", "Bad arg 'from'")
@@ -116,9 +123,9 @@ function BoundLinkUtils.callMethodOnLinkedClasses(binders, linkName, from, metho
 
 	local tags = BinderUtils.mapBinderListToTable(binders)
 
-	local called = {}
+	local called: { [any]: boolean } = {}
 
-	local function callForTag(value, tag)
+	local function callForTag(value: Instance, tag: string)
 		local binder = tags[tag]
 		if not binder then
 			return

package/src/Shared/promiseBoundLinkedClass.lua

@@ -1,4 +1,4 @@
---!nonstrict
+--!strict
 --[=[
 	Promise that returns an objectValue's value that has a bound
 	value to it.
@@ -8,10 +8,11 @@
 
 local require = require(script.Parent.loader).load(script)
 
+local Binder = require("Binder")
 local Maid = require("Maid")
 local Promise = require("Promise")
 
-return function(binder, objValue)
+return function<T>(binder: Binder.Binder<T>, objValue: ObjectValue): Promise.Promise<T>
 	if objValue.Value then
 		local class = binder:Get(objValue.Value)
 		if class then
@@ -20,12 +21,15 @@ return function(binder, objValue)
 	end
 
 	local maid = Maid.new()
-	local promise = Promise.new()
+	local promise: Promise.Promise<T> = Promise.new() :: any
 
 	maid:GiveTask(objValue.Changed:Connect(function()
-		local class = binder:Get(objValue.Value)
-		if class then
-			promise:Resolve(class)
+		local value = objValue.Value
+		if value then
+			local class = binder:Get(value)
+			if class then
+				promise:Resolve(class)
+			end
 		end
 	end))