refactor: pass ExecutableCommand to sync filter

packages/core/src/server/registry.ts

@@ -1,6 +1,10 @@
 import { Players } from "@rbxts/services";
 import { CommandOptions, GroupOptions } from "../shared";
-import { BaseCommand, CommandGroup } from "../shared/core/command";
+import {
+	BaseCommand,
+	CommandGroup,
+	ExecutableCommand,
+} from "../shared/core/command";
 import { BaseRegistry } from "../shared/core/registry";
 import { SyncData } from "../shared/network";
 import { ArrayUtil, ReadonlyDeep } from "../shared/util/data";
@@ -36,7 +40,9 @@ export class ServerRegistry extends BaseRegistry<ReadonlyDeep<ServerConfig>> {
 		for (const [_, command] of this.commands) {
 			const path = command.getPath();
 			if (syncedCommands.has(path.toString())) continue;
-			if (!this.config.syncFilter(player, command)) continue;
+			if (!this.config.syncFilter(player, command as ExecutableCommand)) {
+				continue;
+			}
 
 			syncedCommands.set(path.toString(), {
 				...(command.options as CommandOptions),

packages/core/src/server/types.ts

@@ -1,10 +1,10 @@
 import { CommandContextData, SharedConfig } from "../shared";
-import { BaseCommand } from "../shared/core/command";
+import { ExecutableCommand } from "../shared/core/command";
 import { SyncData } from "../shared/network";
 
 export interface ServerConfig extends SharedConfig {
 	network: ServerNetworkConfig;
-	syncFilter: (player: Player, command: BaseCommand) => boolean;
+	syncFilter: (player: Player, command: ExecutableCommand) => boolean;
 }
 
 export interface ServerNetworkConfig {

packages/core/src/shared/core/command.ts

@@ -101,7 +101,7 @@ export abstract class BaseCommand {
 
 export class ExecutableCommand extends BaseCommand {
 	private readonly callback: CommandCallback;
-	private readonly guards: ReadonlyArray<CommandGuard>;
+	readonly guards: ReadonlyArray<CommandGuard>;
 
 	constructor(
 		config: ReadonlyDeep<SharedConfig>,