fix: fallback to default theme when invalid theme is provided

src/common/utils.js

@@ -274,8 +274,10 @@ const getCardColors = ({
   theme,
   fallbackTheme = "default",
 }) => {
-  const defaultTheme = themes[fallbackTheme];
-  const selectedTheme = themes[theme] || defaultTheme;
+  // Ensure defaultTheme always exists, fallback to "default" if needed
+  const defaultTheme = themes[fallbackTheme] || themes["default"];
+  // Ensure selectedTheme always exists, fallback to defaultTheme if theme is invalid
+  const selectedTheme = theme && themes[theme] ? themes[theme] : defaultTheme;
   const defaultBorderColor =
     selectedTheme.border_color || defaultTheme.border_color;
 

tests/calculateRank.test.js

@@ -30,7 +30,7 @@ describe("Test calculateRank", () => {
         stars: 25,
         followers: 5,
       }),
-    ).toStrictEqual({ level: "B-", percentile: 65.02918514848255 });
+    ).toStrictEqual({ level: "B-", percentile: 65.02918514848257 });
   });
 
   it("median user gets B+ rank", () => {

tests/utils.test.js

@@ -134,6 +134,30 @@ describe("Test utils.js", () => {
       borderColor: "#fff",
     });
   });
+
+  it("getCardColors: should fallback to default theme if invalid theme is provided", () => {
+    let colors = getCardColors({
+      theme: "foobar", // Invalid theme name
+    });
+    // Should use default theme colors
+    expect(colors).toStrictEqual({
+      titleColor: "#2f80ed",
+      textColor: "#434d58",
+      ringColor: "#2f80ed",
+      iconColor: "#4c71f2",
+      bgColor: "#fffefe",
+      borderColor: "#e4e2e2",
+    });
+  });
+
+  it("getCardColors: should handle null/undefined theme gracefully", () => {
+    let colors = getCardColors({
+      theme: null,
+    });
+    // Should use default theme colors
+    expect(colors.titleColor).toBe("#2f80ed");
+    expect(colors.textColor).toBe("#434d58");
+  });
 });
 
 describe("wrapTextMultiline", () => {