address comment: fix typo and add unicode error validation

Author: blva

src/tools/args.ts

@@ -1,12 +1,12 @@
 import { z, type ZodString } from "zod";
 
 const NO_UNICODE_REGEX = /^[\x20-\x7E]*$/;
-const NO_UNICODE_ERROR = "String cannot contain special characters or Unicode symbols";
+export const NO_UNICODE_ERROR = "String cannot contain special characters or Unicode symbols";
 
 const ALLOWED_NAME_CHARACTERS_REGEX = /^[a-zA-Z0-9._-]+$/;
 export const ALLOWED_CHARACTERS_ERROR = " can only contain letters, numbers, dots, hyphens, and underscores";
 
-const ALLLOWED_REGION_CHARACTERS_REGEX = /^[a-zA-Z0-9_-]+$/;
+const ALLOWED_REGION_CHARACTERS_REGEX = /^[a-zA-Z0-9_-]+$/;
 export const ALLOWED_REGION_CHARACTERS_ERROR = "Region can only contain letters, numbers, hyphens, and underscores";
 
 const ALLOWED_CLUSTER_NAME_CHARACTERS_REGEX = /^[a-zA-Z0-9_-]+$/;
@@ -62,7 +62,7 @@ export const AtlasArgs = {
             .string()
             .min(1, "Region is required")
             .max(50, "Region must be 50 characters or less")
-            .regex(ALLLOWED_REGION_CHARACTERS_REGEX, ALLOWED_REGION_CHARACTERS_ERROR),
+            .regex(ALLOWED_REGION_CHARACTERS_REGEX, ALLOWED_REGION_CHARACTERS_ERROR),
 
     password: (): z.ZodString =>
         z.string().min(1, "Password is required").max(100, "Password must be 100 characters or less"),

tests/unit/args.test.ts

@@ -6,6 +6,7 @@ import {
     ALLOWED_CHARACTERS_ERROR,
     ALLOWED_REGION_CHARACTERS_ERROR,
     ALLOWED_CLUSTER_NAME_CHARACTERS_ERROR,
+    NO_UNICODE_ERROR,
 } from "../../src/tools/args.js";
 
 describe("Tool args", () => {
@@ -28,22 +29,22 @@ describe("Tool args", () => {
                 const schema = CommonArgs.string();
 
                 // Unicode characters
-                expect(() => schema.parse("héllo")).toThrow();
-                expect(() => schema.parse("测试")).toThrow();
-                expect(() => schema.parse("café")).toThrow();
+                expect(() => schema.parse("héllo")).toThrow(NO_UNICODE_ERROR);
+                expect(() => schema.parse("测试")).toThrow(NO_UNICODE_ERROR);
+                expect(() => schema.parse("café")).toThrow(NO_UNICODE_ERROR);
 
                 // Emojis
-                expect(() => schema.parse("🚀")).toThrow();
-                expect(() => schema.parse("hello😀")).toThrow();
+                expect(() => schema.parse("🚀")).toThrow(NO_UNICODE_ERROR);
+                expect(() => schema.parse("hello😀")).toThrow(NO_UNICODE_ERROR);
 
                 // Control characters (below ASCII 32)
-                expect(() => schema.parse("hello\nworld")).toThrow();
-                expect(() => schema.parse("hello\tworld")).toThrow();
-                expect(() => schema.parse("hello\0world")).toThrow();
+                expect(() => schema.parse("hello\nworld")).toThrow(NO_UNICODE_ERROR);
+                expect(() => schema.parse("hello\tworld")).toThrow(NO_UNICODE_ERROR);
+                expect(() => schema.parse("hello\0world")).toThrow(NO_UNICODE_ERROR);
 
                 // Extended ASCII characters (above ASCII 126)
-                expect(() => schema.parse("hello\x80")).toThrow();
-                expect(() => schema.parse("hello\xFF")).toThrow();
+                expect(() => schema.parse("hello\x80")).toThrow(NO_UNICODE_ERROR);
+                expect(() => schema.parse("hello\xFF")).toThrow(NO_UNICODE_ERROR);
             });
 
             it("should reject non-string values", () => {