Fix typescript check for extra server check

Author: emdashcodes

packages/client/src/__tests__/api.test.ts

@@ -147,7 +147,9 @@ describe('API functions', () => {
 			};
 
 			// Use type assertion to bypass TypeScript check for testing runtime validation
-			expect(() => registerAbility(ability as unknown as ClientAbility)).toThrow(
+			expect(() =>
+				registerAbility(ability as unknown as ClientAbility)
+			).toThrow(
 				'Server abilities cannot be registered via registerAbility'
 			);
 		});
@@ -170,9 +172,9 @@ describe('API functions', () => {
 			};
 
 			// Use type assertion to bypass TypeScript check
-			expect(() => registerAbility(ability as unknown as ClientAbility)).toThrow(
-				'Client abilities must include a callback function'
-			);
+			expect(() =>
+				registerAbility(ability as unknown as ClientAbility)
+			).toThrow('Client abilities must include a callback function');
 		});
 
 		it('should throw error for ability without name', () => {

packages/client/src/__tests__/validation.test.ts

@@ -382,8 +382,18 @@ describe('validateValueFromSchema', () => {
 		it('should handle empty schema as valid', () => {
 			expect(validateValueFromSchema('anything', undefined)).toBe(true);
 			// Testing edge cases where schema is null or falsy
-			expect(validateValueFromSchema(123, null as unknown as Record<string, any>)).toBe(true);
-			expect(validateValueFromSchema(true, false as unknown as Record<string, any>)).toBe(true);
+			expect(
+				validateValueFromSchema(
+					123,
+					null as unknown as Record<string, any>
+				)
+			).toBe(true);
+			expect(
+				validateValueFromSchema(
+					true,
+					false as unknown as Record<string, any>
+				)
+			).toBe(true);
 		});
 
 		it('should handle schema compilation errors', () => {

packages/client/src/api.ts

@@ -76,12 +76,15 @@ export function registerAbility(ability: ClientAbility): void {
 		throw new Error('Ability description is required');
 	}
 
-	// Check if it's a server ability and reject it
-	if (ability.location === 'server') {
-		throw new Error('Server abilities cannot be registered via registerAbility');
+	// Runtime check for JavaScript consumers who might pass server abilities
+	// TypeScript users are protected by the ClientAbility type
+	const anyAbility = ability as any;
+	if (anyAbility.location === 'server') {
+		throw new Error(
+			'Server abilities cannot be registered via registerAbility'
+		);
 	}
 
-	// Now we know it's supposed to be a client ability
 	if (!ability.callback || typeof ability.callback !== 'function') {
 		throw new Error('Client abilities must include a callback function');
 	}