@quenty/steputils 3.7.2 → 3.9.0

package/CHANGELOG.md

@@ -3,6 +3,14 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [3.9.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/steputils@3.8.0...@quenty/steputils@3.9.0) (2026-08-28)
+
+**Note:** Version bump only for package @quenty/steputils
+
+# [3.8.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/steputils@3.7.2...@quenty/steputils@3.8.0) (2026-08-28)
+
+**Note:** Version bump only for package @quenty/steputils
+
 ## [3.7.2](https://github.com/Quenty/NevermoreEngine/compare/@quenty/steputils@3.7.1...@quenty/steputils@3.7.2) (2026-08-14)
 
 **Note:** Version bump only for package @quenty/steputils

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quenty/steputils",
-  "version": "3.7.2",
+  "version": "3.9.0",
   "description": "Binds animations into step, where the animation only runs as needed",
   "keywords": [
     "Roblox",
@@ -28,12 +28,14 @@
     "Quenty"
   ],
   "dependencies": {
+    "@quenty/jestutils": "1.1.0",
     "@quenty/loader": "10.11.1",
-    "@quenty/nevermore-test-runner": "1.5.1",
+    "@quenty/maid": "3.12.0",
+    "@quenty/nevermore-test-runner": "1.6.0",
     "@quentystudios/jest-lua": "3.10.0-quenty.2"
   },
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "f084360c3729ea424d49d9bc42f8f0d1c128fa82"
+  "gitHead": "346201e192e467cb71160f43029863853be855b5"
 }

package/src/Shared/StepUtils.spec.lua

@@ -8,6 +8,8 @@ local require = require(script.Parent.loader).load(script)
 local RunService = game:GetService("RunService")
 
 local Jest = require("Jest")
+local JestUtils = require("JestUtils")
+local Maid = require("Maid")
 local StepUtils = require("StepUtils")
 
 local describe = Jest.Globals.describe
@@ -21,13 +23,23 @@ type Controller = {
 	bindStepped: ((...any) -> boolean) -> ((...any) -> (), () -> ()),
 	track: (() -> ()) -> () -> (),
 	stepFrames: (number) -> (),
-	destroy: () -> (),
+	Destroy: (self: Controller) -> (),
 }
 
 local function setup(): Controller
+	local maid = Maid.new()
 	local instances: { Instance } = {}
 	local cleanups: { () -> () } = {}
 
+	maid:GiveTask(function()
+		for _, cleanup in cleanups do
+			cleanup()
+		end
+		for _, inst in instances do
+			inst:Destroy()
+		end
+	end)
+
 	local function track(cleanup: () -> ()): () -> ()
 		table.insert(cleanups, cleanup)
 		return cleanup
@@ -66,16 +78,13 @@ local function setup(): Controller
 			end
 		end,
 
-		destroy = function()
-			for _, cleanup in cleanups do
-				cleanup()
-			end
-			for _, inst in instances do
-				inst:Destroy()
-			end
+		Destroy = function(_self)
+			maid:DoCleaning()
 		end,
 	}
 
+	maid:GiveTask(JestUtils.afterThis(controller))
+
 	return controller
 end
 
@@ -85,7 +94,7 @@ describe("StepUtils.getAnimationStepSignal", function()
 
 		expect(typeof(StepUtils.getAnimationStepSignal())).toBe("RBXScriptSignal")
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("falls back to Heartbeat off a running client", function()
@@ -93,7 +102,7 @@ describe("StepUtils.getAnimationStepSignal", function()
 
 		expect(StepUtils.getAnimationStepSignal()).toBe(RunService.Heartbeat)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("returns a signal that actually fires here", function()
@@ -111,7 +120,7 @@ describe("StepUtils.getAnimationStepSignal", function()
 
 		expect(fired).toBe(true)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)
 
@@ -121,7 +130,7 @@ describe("StepUtils.getSteppedSignal", function()
 
 		expect(typeof(StepUtils.getSteppedSignal())).toBe("RBXScriptSignal")
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("falls back to Heartbeat on a non-running DataModel", function()
@@ -130,7 +139,7 @@ describe("StepUtils.getSteppedSignal", function()
 		expect(StepUtils.getSteppedSignal()).toBe(RunService.Heartbeat)
 		expect(StepUtils.getSteppedSignal()).never.toBe(RunService.Stepped)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("returns a signal that actually fires here", function()
@@ -148,7 +157,7 @@ describe("StepUtils.getSteppedSignal", function()
 
 		expect(fired).toBe(true)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)
 
@@ -167,7 +176,7 @@ describe("StepUtils.bindToSignal", function()
 
 		expect(calls).toBe(1)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("does not connect when the first update returns false", function()
@@ -186,7 +195,7 @@ describe("StepUtils.bindToSignal", function()
 
 		expect(calls).toBe(1)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("keeps invoking the update while it returns true", function()
@@ -205,7 +214,7 @@ describe("StepUtils.bindToSignal", function()
 
 		expect(calls).toBe(3)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("stops invoking the update once it returns false", function()
@@ -225,7 +234,7 @@ describe("StepUtils.bindToSignal", function()
 
 		expect(calls).toBe(2)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("forwards the connect arguments to every later update", function()
@@ -244,7 +253,7 @@ describe("StepUtils.bindToSignal", function()
 
 		expect(seen).toEqual({ "self", "self", "self" })
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("ignores the signal's own arguments", function()
@@ -262,7 +271,7 @@ describe("StepUtils.bindToSignal", function()
 
 		expect(seen).toEqual({ { value = nil }, { value = nil } })
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("ignores a repeated connect while already connected", function()
@@ -280,7 +289,7 @@ describe("StepUtils.bindToSignal", function()
 
 		expect(calls).toBe(1)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("reconnects after an explicit disconnect", function()
@@ -303,7 +312,7 @@ describe("StepUtils.bindToSignal", function()
 
 		expect(calls).toBe(3)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("tolerates disconnect before connect and repeated disconnects", function()
@@ -319,7 +328,7 @@ describe("StepUtils.bindToSignal", function()
 			disconnect()
 		end).never.toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("throws on something that is not a signal", function()
@@ -331,7 +340,7 @@ describe("StepUtils.bindToSignal", function()
 			end)
 		end).toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("throws on a non-function update", function()
@@ -342,7 +351,7 @@ describe("StepUtils.bindToSignal", function()
 			(StepUtils :: any).bindToSignal(bindable.Event, "update")
 		end).toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)
 
@@ -361,7 +370,7 @@ describe("StepUtils.bindToRenderStep", function()
 
 		expect(calls).toBe(3)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("does not bind when the first update returns false", function()
@@ -378,7 +387,7 @@ describe("StepUtils.bindToRenderStep", function()
 
 		expect(calls).toBe(1)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("stops driving the update after disconnect", function()
@@ -399,7 +408,7 @@ describe("StepUtils.bindToRenderStep", function()
 
 		expect(calls).toBe(afterDisconnect)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)
 
@@ -417,7 +426,7 @@ describe("StepUtils.bindToStepped", function()
 
 		expect(calls).toBe(1)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("tolerates disconnect before connect", function()
@@ -431,7 +440,7 @@ describe("StepUtils.bindToStepped", function()
 			disconnect()
 		end).never.toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)
 
@@ -449,7 +458,7 @@ describe("StepUtils.deferWait", function()
 
 		expect(order).toEqual({ "deferred", "resumed" })
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)
 
@@ -467,7 +476,7 @@ describe("StepUtils.onceAtEvent", function()
 
 		expect(calls).toBe(1)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("passes the event arguments through", function()
@@ -483,7 +492,7 @@ describe("StepUtils.onceAtEvent", function()
 
 		expect(seen).toBe("payload")
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("only invokes the function once", function()
@@ -500,7 +509,7 @@ describe("StepUtils.onceAtEvent", function()
 
 		expect(calls).toBe(1)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("does not invoke the function once cancelled", function()
@@ -517,7 +526,7 @@ describe("StepUtils.onceAtEvent", function()
 
 		expect(calls).toBe(0)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("tolerates cancelling after the function ran", function()
@@ -532,7 +541,7 @@ describe("StepUtils.onceAtEvent", function()
 			cancel()
 		end).never.toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("throws on a non-function", function()
@@ -543,7 +552,7 @@ describe("StepUtils.onceAtEvent", function()
 			(StepUtils :: any).onceAtEvent(bindable.Event, "func")
 		end).toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)
 
@@ -560,7 +569,7 @@ describe("StepUtils.onceAtRenderStepped", function()
 
 		expect(calls).toBe(1)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("does not invoke the function once cancelled", function()
@@ -576,7 +585,7 @@ describe("StepUtils.onceAtRenderStepped", function()
 
 		expect(calls).toBe(0)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)
 
@@ -595,7 +604,7 @@ describe("StepUtils.onceAtStepped", function()
 		end).never.toThrow()
 		expect(calls).toBe(0)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)
 
@@ -610,7 +619,7 @@ describe("StepUtils.onceAtRenderPriority", function()
 			cancel()
 		end).never.toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("throws on a non-number priority", function()
@@ -620,7 +629,7 @@ describe("StepUtils.onceAtRenderPriority", function()
 			(StepUtils :: any).onceAtRenderPriority("high", function() end)
 		end).toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("throws on a non-function", function()
@@ -630,6 +639,6 @@ describe("StepUtils.onceAtRenderPriority", function()
 			(StepUtils :: any).onceAtRenderPriority(100, "func")
 		end).toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)

package/src/Shared/onRenderStepFrame.spec.lua

@@ -6,6 +6,8 @@
 local require = require(script.Parent.loader).load(script)
 
 local Jest = require("Jest")
+local JestUtils = require("JestUtils")
+local Maid = require("Maid")
 local onRenderStepFrame = require("onRenderStepFrame")
 
 local describe = Jest.Globals.describe
@@ -14,12 +16,19 @@ local it = Jest.Globals.it
 
 type Controller = {
 	bind: (number, () -> ()) -> () -> (),
-	destroy: () -> (),
+	Destroy: (self: Controller) -> (),
 }
 
 local function setup(): Controller
+	local maid = Maid.new()
 	local unbinds: { () -> () } = {}
 
+	maid:GiveTask(function()
+		for _, unbind in unbinds do
+			unbind()
+		end
+	end)
+
 	local controller: Controller = {
 		bind = function(priority: number, callback: () -> ())
 			local unbind = onRenderStepFrame(priority, callback)
@@ -27,13 +36,13 @@ local function setup(): Controller
 			return unbind
 		end,
 
-		destroy = function()
-			for _, unbind in unbinds do
-				unbind()
-			end
+		Destroy = function(_self)
+			maid:DoCleaning()
 		end,
 	}
 
+	maid:GiveTask(JestUtils.afterThis(controller))
+
 	return controller
 end
 
@@ -45,7 +54,7 @@ describe("onRenderStepFrame", function()
 
 		expect(type(unbind)).toBe("function")
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("tolerates a repeated unbind", function()
@@ -58,7 +67,7 @@ describe("onRenderStepFrame", function()
 			unbind()
 		end).never.toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("binds each call under its own key", function()
@@ -73,7 +82,7 @@ describe("onRenderStepFrame", function()
 			second()
 		end).never.toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("throws on a non-number priority", function()
@@ -83,7 +92,7 @@ describe("onRenderStepFrame", function()
 			(onRenderStepFrame :: any)("high", function() end)
 		end).toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("throws on a non-function callback", function()
@@ -93,6 +102,6 @@ describe("onRenderStepFrame", function()
 			(onRenderStepFrame :: any)(100, "callback")
 		end).toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)

package/src/Shared/onSteppedFrame.spec.lua

@@ -6,6 +6,8 @@
 local require = require(script.Parent.loader).load(script)
 
 local Jest = require("Jest")
+local JestUtils = require("JestUtils")
+local Maid = require("Maid")
 local onSteppedFrame = require("onSteppedFrame")
 
 local describe = Jest.Globals.describe
@@ -14,12 +16,19 @@ local it = Jest.Globals.it
 
 type Controller = {
 	bind: (() -> ()) -> RBXScriptConnection,
-	destroy: () -> (),
+	Destroy: (self: Controller) -> (),
 }
 
 local function setup(): Controller
+	local maid = Maid.new()
 	local connections: { RBXScriptConnection } = {}
 
+	maid:GiveTask(function()
+		for _, conn in connections do
+			conn:Disconnect()
+		end
+	end)
+
 	local controller: Controller = {
 		bind = function(callback)
 			local conn = onSteppedFrame(callback)
@@ -27,13 +36,13 @@ local function setup(): Controller
 			return conn
 		end,
 
-		destroy = function()
-			for _, conn in connections do
-				conn:Disconnect()
-			end
+		Destroy = function(_self)
+			maid:DoCleaning()
 		end,
 	}
 
+	maid:GiveTask(JestUtils.afterThis(controller))
+
 	return controller
 end
 
@@ -46,7 +55,7 @@ describe("onSteppedFrame", function()
 		expect(typeof(conn)).toBe("RBXScriptConnection")
 		expect(conn.Connected).toBe(true)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("goes dead once disconnected", function()
@@ -57,7 +66,7 @@ describe("onSteppedFrame", function()
 
 		expect(conn.Connected).toBe(false)
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("tolerates a repeated disconnect", function()
@@ -70,7 +79,7 @@ describe("onSteppedFrame", function()
 			conn:Disconnect()
 		end).never.toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 
 	it("throws on a non-function", function()
@@ -80,6 +89,6 @@ describe("onSteppedFrame", function()
 			(onSteppedFrame :: any)("func")
 		end).toThrow()
 
-		controller.destroy()
+		controller:Destroy()
 	end)
 end)