From a28e7e2590a51f6dbebeccc1335d47f706250de5 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Fri, 17 Apr 2026 06:44:23 +0000
Subject: [PATCH 1/3] =?UTF-8?q?=F0=9F=A7=AA=20Add=20Edge=20Case=20Tests=20?=
 =?UTF-8?q?for=20normalizeCardLayout?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
---
 src/lib/__tests__/cardLayout.test.ts | 89 ++++++++++++++++++++++++----
 1 file changed, 78 insertions(+), 11 deletions(-)

diff --git a/src/lib/__tests__/cardLayout.test.ts b/src/lib/__tests__/cardLayout.test.ts
index 2342648..c5c91bd 100644
--- a/src/lib/__tests__/cardLayout.test.ts
+++ b/src/lib/__tests__/cardLayout.test.ts
@@ -32,20 +32,87 @@ describe("cardLayout utilities", () => {
     });
   });
 
-  it("normalizeCardLayout fills missing blocks and removes invalid ones", () => {
-    const normalized = normalizeCardLayout({
-      blocks: [
-        { id: "avatar", visible: false, column: "full" },
-        { id: "unknown", visible: true, column: "left" },
-      ],
+  describe("normalizeCardLayout", () => {
+    it("fills missing blocks and removes invalid ones", () => {
+      const normalized = normalizeCardLayout({
+        blocks: [
+          { id: "avatar", visible: false, column: "full" },
+          { id: "unknown", visible: true, column: "left" },
+        ],
+      });
+
+      expect(normalized.blocks.find((b) => b.id === "avatar")).toMatchObject({
+        visible: false,
+        column: "full",
+      });
+      expect(normalized.blocks.some((b) => (b as { id: string }).id === "unknown")).toBe(false);
+      expect(normalized.blocks).toHaveLength(DEFAULT_CARD_LAYOUT.blocks.length);
+    });
+
+    it.each([
+      null,
+      undefined,
+      {},
+      "string",
+      123,
+      { blocks: null },
+      { blocks: {} },
+      { blocks: "string" },
+    ])("returns cloned default layout for invalid input: %j", (input) => {
+      const result = normalizeCardLayout(input);
+      expect(result).toEqual(DEFAULT_CARD_LAYOUT);
+      expect(result.blocks).not.toBe(DEFAULT_CARD_LAYOUT.blocks); // check clone
+    });
+
+    it("ignores non-object items in blocks array", () => {
+      const normalized = normalizeCardLayout({
+        blocks: [
+          null,
+          "string",
+          123,
+          { id: "avatar", visible: false, column: "full" },
+        ],
+      });
+
+      expect(normalized.blocks.find((b) => b.id === "avatar")).toMatchObject({
+        visible: false,
+        column: "full",
+      });
+      expect(normalized.blocks).toHaveLength(DEFAULT_CARD_LAYOUT.blocks.length);
     });
 
-    expect(normalized.blocks.find((b) => b.id === "avatar")).toMatchObject({
-      visible: false,
-      column: "full",
+    it("ignores duplicate block IDs, using the first one encountered", () => {
+      const normalized = normalizeCardLayout({
+        blocks: [
+          { id: "avatar", visible: false, column: "full" },
+          { id: "avatar", visible: true, column: "left" },
+        ],
+      });
+
+      expect(normalized.blocks.find((b) => b.id === "avatar")).toMatchObject({
+        visible: false,
+        column: "full",
+      });
+      expect(normalized.blocks.filter((b) => b.id === "avatar")).toHaveLength(1);
+    });
+
+    it("falls back to default column and visible if provided are invalid", () => {
+      // Find a block in defaults to check fallbacks
+      const defaultAvatar = DEFAULT_CARD_LAYOUT.blocks.find((b) => b.id === "avatar");
+      expect(defaultAvatar).toBeDefined();
+
+      const normalized = normalizeCardLayout({
+        blocks: [
+          { id: "avatar", visible: "not-a-boolean", column: "invalid-column" },
+        ],
+      });
+
+      const avatar = normalized.blocks.find((b) => b.id === "avatar");
+      expect(avatar).toMatchObject({
+        visible: defaultAvatar?.visible,
+        column: defaultAvatar?.column,
+      });
     });
-    expect(normalized.blocks.some((b) => (b as { id: string }).id === "unknown")).toBe(false);
-    expect(normalized.blocks).toHaveLength(DEFAULT_CARD_LAYOUT.blocks.length);
   });
 
 describe("toggleBlockVisibility", () => {

From 3df76546d273eed56c0737b2f2c613820effadd4 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Fri, 17 Apr 2026 08:36:20 +0000
Subject: [PATCH 2/3] =?UTF-8?q?=F0=9F=A7=AA=20Add=20Edge=20Case=20Tests=20?=
 =?UTF-8?q?for=20normalizeCardLayout?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>

From 9d3767fd69fe5b1b97f34080a7172b1bb4727345 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Fri, 17 Apr 2026 08:40:55 +0000
Subject: [PATCH 3/3] =?UTF-8?q?=F0=9F=A7=AA=20Add=20Edge=20Case=20Tests=20?=
 =?UTF-8?q?for=20normalizeCardLayout?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>