fix: GitHub MCP / VS Code extension registry enforcing -d (#519)

Github MCP registry links to VS Code extension that always enforce -d
param (domains), thus here treating empty for empty confirmation.

<img width="2457" height="517" alt="image"
src="https://github.com/user-attachments/assets/48c60f40-df78-4598-a699-510bc14476e0"
/>

If press enter other than the skip, it leads to null issue. This aims to
patch it.
<img width="2464" height="492" alt="image"
src="https://github.com/user-attachments/assets/209ace4b-bde0-4c02-901a-8c96c11b09a1"
/>

Author: Novaes

src/shared/domains.ts

@@ -122,8 +122,8 @@ export class DomainsManager {
    * @returns Normalized array of domain strings
    */
   public static parseDomainsInput(domainsInput?: string | string[]): string[] {
-    if (!domainsInput) {
-      return [];
+    if (!domainsInput || this.isEmptyDomainsInput(domainsInput)) {
+      return ["all"];
     }
 
     if (typeof domainsInput === "string") {
@@ -132,4 +132,10 @@ export class DomainsManager {
 
     return domainsInput.map((d) => d.trim().toLowerCase());
   }
+
+  private static isEmptyDomainsInput(domainsInput?: string | string[]): boolean {
+    if (typeof domainsInput === "string" && domainsInput.trim() === "") return true;
+    if (Array.isArray(domainsInput) && domainsInput.length === 0) return true;
+    return false;
+  }
 }

test/src/domains.test.ts

@@ -231,14 +231,14 @@ describe("DomainsManager: backward compatibility and domain enabling", () => {
   });
 
   describe("parseDomainsInput static method", () => {
-    it("returns empty array when no input is provided", () => {
+    it("returns 'all' when no input is provided", () => {
       const result = DomainsManager.parseDomainsInput();
-      expect(result).toEqual([]);
+      expect(result).toEqual(["all"]);
     });
 
-    it("returns empty array when undefined is provided", () => {
+    it("returns 'all' array when undefined is provided", () => {
       const result = DomainsManager.parseDomainsInput(undefined);
-      expect(result).toEqual([]);
+      expect(result).toEqual(["all"]);
     });
 
     it("parses comma-separated string input", () => {
@@ -324,4 +324,24 @@ describe("DomainsManager: backward compatibility and domain enabling", () => {
       expect(enabledDomains.size).toBe(9); // Should enable all because 'all' is present
     });
   });
+
+  describe("edge empty string cases for enabling all domains", () => {
+    it("when an empty array is passed", () => {
+      const manager = new DomainsManager([]);
+      const enabledDomains = manager.getEnabledDomains();
+      expect(enabledDomains.size).toBe(9);
+    });
+
+    it("when a string with only a break line is passed", () => {
+      const manager = new DomainsManager("\n");
+      const enabledDomains = manager.getEnabledDomains();
+      expect(enabledDomains.size).toBe(9);
+    });
+
+    it("when an empty string is passed", () => {
+      const manager = new DomainsManager("");
+      const enabledDomains = manager.getEnabledDomains();
+      expect(enabledDomains.size).toBe(9);
+    });
+  });
 });