fix: add input validation for type

Author: manjindersingh98

src/commands/view/module.ts

@@ -4,17 +4,27 @@
 
 import { Configurator, IModule } from "../../core/command/module-handler";
 import { Context } from "../../core/command/cli-context";
-import { Command, OptionValues } from "commander";
+import { Command, InvalidArgumentError, OptionValues } from "commander";
 import { ViewBookmarksCommandService } from "./view-bookmarks-command.service";
 
+const VIEW_BOOKMARK_TYPES = ["USER", "SHARED", "ALL"];
+
+function parseViewBookmarkType(value: string): string {
+    const normalized = value.toUpperCase();
+    if (!VIEW_BOOKMARK_TYPES.includes(normalized)) {
+        throw new InvalidArgumentError(`Allowed values are: ${VIEW_BOOKMARK_TYPES.join(", ")}.`);
+    }
+    return normalized;
+}
+
 class Module extends IModule {
 
     public register(context: Context, configurator: Configurator): void {
         const pullCommand = configurator.command("pull");
         pullCommand
             .command("view-bookmarks")
             .description("Command to pull view bookmarks")
-            .option("--type <type>", "Type of view bookmarks to pull: USER (default), SHARED, or ALL")
+            .option("--type <type>", "Type of view bookmarks to pull: USER (default), SHARED, or ALL", parseViewBookmarkType)
             .requiredOption("--id <id>", "ID of the view (board) to pull bookmarks from")
             .action(this.pullViewBookmarks);
 

src/core/command/module-handler.ts

@@ -181,8 +181,14 @@ export class CommandConfig {
         return this;
     }
 
-    public option(flags: string, description?: string, defaultValue?: string | boolean | string[]): CommandConfig {
-        this.cmd.option(flags, description, defaultValue);
+    public option(flags: string, description?: string, defaultValue?: string | boolean | string[]): CommandConfig;
+    public option(flags: string, description: string, parseArg: (value: string, previous: unknown) => unknown, defaultValue?: string | boolean | string[]): CommandConfig;
+    public option(flags: string, description?: string, parseArgOrDefault?: ((value: string, previous: unknown) => unknown) | string | boolean | string[], defaultValue?: string | boolean | string[]): CommandConfig {
+        if (typeof parseArgOrDefault === "function") {
+            this.cmd.option(flags, description, parseArgOrDefault, defaultValue);
+        } else {
+            this.cmd.option(flags, description, parseArgOrDefault);
+        }
         return this;
     }