@quenty/valueobject 13.13.0 → 13.13.1-canary.522.1d2faf4.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.
 
+## [13.13.1-canary.522.1d2faf4.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/valueobject@13.13.0...@quenty/valueobject@13.13.1-canary.522.1d2faf4.0) (2024-12-03)
+
+
+### Features
+
+* Add :SetValue() behavior to cleanup past mounted sub and return a function to unset the value ([03fd05a](https://github.com/Quenty/NevermoreEngine/commit/03fd05aaefbeccc6cc26359239876e1a39569bc1))
+
+
+
+
+
 # [13.13.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/valueobject@13.12.0...@quenty/valueobject@13.13.0) (2024-11-13)
 
 **Note:** Version bump only for package @quenty/valueobject

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quenty/valueobject",
-  "version": "13.13.0",
+  "version": "13.13.1-canary.522.1d2faf4.0",
   "description": "To work like value objects in Roblox and track a single item with .Changed events",
   "keywords": [
     "Roblox",
@@ -26,16 +26,16 @@
     "Quenty"
   ],
   "dependencies": {
-    "@quenty/brio": "^14.13.0",
-    "@quenty/ducktype": "^5.7.1",
-    "@quenty/loader": "^10.7.1",
-    "@quenty/maid": "^3.4.0",
-    "@quenty/rx": "^13.13.0",
-    "@quenty/signal": "^7.9.0",
-    "@quenty/valuebaseutils": "^13.13.0"
+    "@quenty/brio": "14.13.1-canary.522.1d2faf4.0",
+    "@quenty/ducktype": "5.7.1",
+    "@quenty/loader": "10.7.1",
+    "@quenty/maid": "3.4.0",
+    "@quenty/rx": "13.13.1-canary.522.1d2faf4.0",
+    "@quenty/signal": "7.9.0",
+    "@quenty/valuebaseutils": "13.13.1-canary.522.1d2faf4.0"
   },
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "451334becfa36a10b55fa53cbdd88091cdf9ddbf"
+  "gitHead": "1d2faf48e778ada6f57aecb8000c7c6c76ac33cd"
 }

package/src/Shared/ValueObject.lua

@@ -29,6 +29,7 @@ ValueObject.ClassName = "ValueObject"
 function ValueObject.new(baseValue, checkType)
 	local self = setmetatable({
 		_value = baseValue;
+		_default = baseValue;
 		_checkType = checkType;
 	}, ValueObject)
 
@@ -111,7 +112,7 @@ function ValueObject:Mount(value)
 		self:_cleanupLastMountedSub()
 
 		local sub = observable:Subscribe(function(...)
-			self:SetValue(...)
+			ValueObject._applyValue(self, ...)
 		end)
 
 		rawset(self, "_lastMountedSub", sub)
@@ -124,7 +125,7 @@ function ValueObject:Mount(value)
 	else
 		self:_cleanupLastMountedSub()
 
-		self:SetValue(value)
+		ValueObject._applyValue(self, value)
 
 		return EMPTY_FUNCTION
 	end
@@ -234,8 +235,21 @@ end
 
 	@param value T
 	@param ... any -- Additional args. Can be used to pass event changing state args with value
+	@return () -> () -- Cleanup
 ]=]
 function ValueObject:SetValue(value, ...)
+	self:_cleanupLastMountedSub()
+
+	ValueObject._applyValue(self, value, ...)
+
+	return function()
+		if rawget(self, "_value") == value then
+			ValueObject._applyValue(self, rawget(self, "_default"))
+		end
+	end
+end
+
+function ValueObject:_applyValue(value, ...)
 	local previous = rawget(self, "_value")
 	local checkType = rawget(self, "_checkType")
 
@@ -298,7 +312,7 @@ end
 function ValueObject:__newindex(index, value)
 	if index == "Value" then
 		-- Avoid deoptimization
-		ValueObject.SetValue(self, value)
+		ValueObject._applyValue(self, value)
 	elseif index == "LastEventContext" or ValueObject[index] then
 		error(string.format("%q cannot be set in ValueObject", tostring(index)))
 	else