From 01385d798d7c767c638fa62ff1f3bcfb6e7e3328 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E4=BE=9D=E7=91=AA=E8=B2=93?= <imacat@mail.imacat.idv.tw>
Date: Mon, 9 Mar 2026 16:31:29 +0800
Subject: [PATCH] Return null for unknown type in getStateData to prevent empty
 URL request

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 src/stores/compare.ts        | 2 ++
 tests/stores/compare.test.js | 7 +++++++
 2 files changed, 9 insertions(+)

diff --git a/src/stores/compare.ts b/src/stores/compare.ts
index c29a557..4acc59f 100644
--- a/src/stores/compare.ts
+++ b/src/stores/compare.ts
@@ -55,6 +55,8 @@ export const useCompareStore = defineStore("compareStore", {
         case "filter":
           api = `/api/filters/${id}/discover`;
           break;
+        default:
+          return null;
       }
       try {
         const response = await apiClient.get(api);
diff --git a/tests/stores/compare.test.js b/tests/stores/compare.test.js
index 33cdc86..00d016c 100644
--- a/tests/stores/compare.test.js
+++ b/tests/stores/compare.test.js
@@ -77,6 +77,13 @@ describe("compareStore", () => {
       expect(mockGet).toHaveBeenCalledWith("/api/filters/3/discover");
       expect(result).toEqual({ cases: 50 });
     });
+
+    it("returns null for unknown type", async () => {
+      const result = await store.getStateData("unknown", 1);
+
+      expect(mockGet).not.toHaveBeenCalled();
+      expect(result).toBeNull();
+    });
   });
 
   describe("getFileName", () => {