@rbxts/replion 1.0.10 → 1.0.12

package/README.md

@@ -2,4 +2,4 @@
 
 ## Install
 Install with [wally](https://wally.run/):\
-`Replion = "shouxtech/replion@1.0.10"`
+`Replion = "shouxtech/replion@1.0.12"`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rbxts/replion",
-  "version": "1.0.10",
+  "version": "1.0.12",
   "description": "",
   "main": "src/init.lua",
   "scripts": {},

package/src/Client.lua

@@ -58,29 +58,45 @@ function Client.new(channel: string, data: Shared.GenericDataTable)
 	return self;
 end;
 
-function Client:get(key: (string | {string})?)
-	if not key then return self.data; end;
-	if typeof(key) == 'table' then
-		return Shared.getNestedValue(self.data, key);
-	end;
-
-	return self.data[key];
+function Client:get(path: { string }?)
+	if not path then return self.data; end;
+	return Shared.getNestedValue(self.data, path);
 end;
 
-function Client:observe(key: string?, callback: Observer)
-	if not key then
+function Client:observe(path: { string }?, callback: Observer)
+	if not path then
 		task.spawn(callback, self.data, nil);
-
 		return self._allSignal:Connect(callback);
 	end;
 
-	if not self._signals[key] then
-		self._signals[key] = Signal.new();
+	local signalKey = Shared.getSignalKey(path);
+	if not self._signals[signalKey] then
+		self._signals[signalKey] = Signal.new();
 	end;
 
-	task.spawn(callback, self.data[key], nil);
+	local initialValue = Shared.getNestedValue(self.data, path);
 
-	return self._signals[key]:Connect(callback);
+	task.spawn(callback, initialValue, nil);
+	return self._signals[signalKey]:Connect(callback);
+end;
+
+function Client:_recursiveSignalCheck(currentPath: { string }, updateTree: any)
+	for k, v in updateTree do
+		local nextPath = table.clone(currentPath);
+		table.insert(nextPath, k);
+
+		local signalKey = Shared.getSignalKey(nextPath);
+		local signal = self._signals[signalKey];
+
+		if signal then
+			local newValue = Shared.getNestedValue(self.data, nextPath);
+			signal:Fire(newValue, newValue); -- Warning: Sends improper 'old' argument due to shallow copy optimization.
+		end;
+
+		if typeof(v) == 'table' then
+			self:_recursiveSignalCheck(nextPath, v);
+		end;
+	end;
 end;
 
 function Client:_applyUpdates(updates: Shared.GenericDataTable)
@@ -93,13 +109,15 @@ function Client:_applyUpdates(updates: Shared.GenericDataTable)
 	for key, newValue in updates do
 		local oldValue = oldData[key];
 
-		-- If it's a table, we assume it changed if it's in the update list.
+		-- If it's a table, we assume it changed if it's in the updates list.
 		if typeof(newValue) == 'table' then
 			anyChanged = true
 			local signal = self._signals[key];
 			if signal then
 				signal:Fire(self.data[key], self.data[key]); -- Warning: Sends improper 'old' argument due to shallow copy optimization.
 			end;
+
+			self:_recursiveSignalCheck({key}, newValue);
 		elseif self.data[key] ~= oldValue then
 			anyChanged = true;
 			local signal = self._signals[key];
@@ -126,7 +144,7 @@ remote.OnClientEvent:Connect(function(type, channel, payload)
 	if type == 'I' then
 		local newReplion = Client.new(channel, payload);
 		Client._activeReplions[channel] = newReplion;
-		
+
 		local initializedSignal = Client._replionInitializedSignals[channel];
 		if not initializedSignal then return; end;
 

package/src/Server.lua

@@ -99,24 +99,26 @@ function Server.new(config: ServerConfig)
 	return self;
 end;
 
-function Server:observe(key: string?, callback: Observer)
-	if not key then
+function Server:observe(path: { string }?, callback: Observer)
+	if not path then
 		task.spawn(callback, self.data, nil);
 		return self._allSignal:Connect(callback);
 	end;
 
-	if not self._signals[key] then
-		self._signals[key] = Signal.new();
+	local signalKey = Shared.getSignalKey(path);
+	if not self._signals[signalKey] then
+		self._signals[signalKey] = Signal.new();
 	end;
 
-	task.spawn(callback, self.data[key], nil);
-	return self._signals[key]:Connect(callback);
+	local initialValue = Shared.getNestedValue(self.data, path);
+
+	task.spawn(callback, initialValue, nil);
+	return self._signals[signalKey]:Connect(callback);
 end;
 
-function Server:set(key: string | {string}, value: any)
+function Server:set(path: { string }, value: any)
 	if self._destroyed then return; end;
 
-	local path = if typeof(key) == 'table' then key else {key};
 	local topKey = path[1];
 
 	local oldValue = Shared.getNestedValue(self.data, path);
@@ -130,9 +132,17 @@ function Server:set(key: string | {string}, value: any)
 	self:_scheduleUpdatesFlush();
 
 	do
-		local signal = self._signals[topKey];
-		if signal then
-			signal:Fire(self.data[topKey], oldTop);
+		local topSignal = self._signals[topKey];
+		if topSignal then
+			topSignal:Fire(self.data[topKey], oldTop);
+		end;
+
+		local signalKey = Shared.getSignalKey(path);
+		if signalKey ~= topKey then
+			local pathSignal = self._signals[signalKey];
+			if pathSignal then
+				pathSignal:Fire(newValue, oldValue)
+			end;
 		end;
 
 		local thisUpdate = {};
@@ -141,13 +151,9 @@ function Server:set(key: string | {string}, value: any)
 	end;
 end;
 
-function Server:get(key: (string | {string})?)
-	if not key then return self.data; end;
-	if typeof(key) == 'table' then
-		return Shared.getNestedValue(self.data, key);
-	end;
-
-	return self.data[key];
+function Server:get(path: { string }?)
+	if not path then return self.data; end;
+	return Shared.getNestedValue(self.data, path);
 end;
 
 function Server:destroy()
@@ -230,4 +236,4 @@ Players.PlayerRemoving:Connect(function(player)
 	Server._activeReplions[player] = nil;
 end);
 
-return Server;
+return Server;

package/src/Shared.lua

@@ -15,4 +15,9 @@ function Shared.getNestedValue(root: any, path: { string })
 	return current;
 end;
 
+function Shared.getSignalKey(key: string | { string })
+    if typeof(key) == 'string' then return key; end;
+    return table.concat(key, '\0');
+end;
+
 return Shared;

package/src/index.d.ts

@@ -1,6 +1,25 @@
 import { Connection } from '@rbxts/sleitnick-signal';
 
 declare namespace Replion {
+	/**
+	 * Recursively builds a union of all valid path tuples for object T.
+	 * E.g., { a: { b: 1 } } -> ['a'] | ['a', 'b']
+	 */
+	type Path<T> = T extends object
+		? { [K in keyof T & string]: [K] | [K, ...Path<T[K]>] }[keyof T & string]
+		: never;
+
+	/**
+	 * Resolves the type of the value at a specific path P within object T.
+	 */
+	type PathValue<T, P extends any[]> = P extends [infer K, ...infer R]
+		? K extends keyof T
+			? R extends []
+				? T[K]
+				: PathValue<T[K], R>
+			: never
+		: never;
+
 	interface ServerConfig<T extends object> {
 		channel: string;
 		replicateTo?: Player;
@@ -10,17 +29,18 @@ declare namespace Replion {
 	class Server<T extends object> {
 		constructor(config: ServerConfig<T>);
 
-		set<K extends keyof T>(key: K, value: T[K] | ((oldValue: T[K]) => T[K])): void;
-		set(path: string[], value: any | ((oldValue: any) => any)): void;
+		set<P extends Path<T>>(
+			path: P,
+			value: PathValue<T, P> | ((oldValue: PathValue<T, P>) => PathValue<T, P>),
+		): void;
 
 		get(): T;
-		get<K extends keyof T>(key: K): T[K];
-		get(path: string[]): any;
+		get<P extends Path<T>>(path: P): PathValue<T, P>;
 
 		observe(key: undefined, callback: (newValue: T, oldValue: Partial<T>) => void): Connection;
-		observe<K extends keyof T>(
-			key: K,
-			callback: (newValue: T[K], oldValue: T[K] | undefined) => void,
+		observe<P extends Path<T>>(
+			path: P,
+			callback: (newValue: PathValue<T, P>, oldValue: PathValue<T, P> | undefined) => void,
 		): Connection;
 
 		destroy(): void;
@@ -30,13 +50,12 @@ declare namespace Replion {
 		static waitForReplion: <T extends object>(channel: string) => Client<T>;
 
 		get(): T;
-		get<K extends keyof T>(key: K): T[K];
-		get(path: string[]): any;
+		get<P extends Path<T>>(path: P): PathValue<T, P>;
 
 		observe(key: undefined, callback: (newValue: T, oldValue: Partial<T>) => void): Connection;
-		observe<K extends keyof T>(
-			key: K,
-			callback: (newValue: T[K], oldValue: T[K] | undefined) => void,
+		observe<P extends Path<T>>(
+			path: P,
+			callback: (newValue: PathValue<T, P>, oldValue: PathValue<T, P> | undefined) => void,
 		): Connection;
 
 		destroy(): void;