From d1f8d27d1c2726ca8df7544a703bd49ec558f2e7 Mon Sep 17 00:00:00 2001 From: Chris Farhood Date: Wed, 20 May 2026 10:55:24 +0000 Subject: [PATCH 1/2] fix: remove unused X import from lucide-react Resolves ESLint error: 'X' is defined but never used GRO-1347 Co-Authored-By: Paperclip --- src/portal/sections/PetProfiles.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/portal/sections/PetProfiles.tsx b/src/portal/sections/PetProfiles.tsx index 787eeb3..6f0fdb6 100644 --- a/src/portal/sections/PetProfiles.tsx +++ b/src/portal/sections/PetProfiles.tsx @@ -1,5 +1,5 @@ import { useState, useEffect } from "react"; -import { PawPrint, Heart, Scissors, Clock, Edit3, Loader2, Star, X } from "lucide-react"; +import { PawPrint, Heart, Scissors, Clock, Edit3, Loader2, Star } from "lucide-react"; import { PetForm } from "./PetForm.js"; import type { Pet } from "@groombook/types"; -- 2.52.0 From f414d2589f44a18500e7d69706cb0984feee3ae0 Mon Sep 17 00:00:00 2001 From: Chris Farhood Date: Wed, 20 May 2026 14:22:02 +0000 Subject: [PATCH 2/2] fix(GRO-1366): add non-null assertion to removeButtons[0] Fix TypeScript error on line 114: HTMLElement | undefined is not assignable to Element. Added ! assertion since length guard already excludes the empty-array case. Co-Authored-By: Paperclip --- src/__tests__/PetForm.test.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/__tests__/PetForm.test.tsx b/src/__tests__/PetForm.test.tsx index 8c5f3b1..f49e135 100644 --- a/src/__tests__/PetForm.test.tsx +++ b/src/__tests__/PetForm.test.tsx @@ -111,7 +111,7 @@ describe("PetForm", () => { render(); const removeButtons = screen.getAllByRole("button", { name: "" }); if (removeButtons.length === 0) return; - const removeButton = removeButtons[0]; + const removeButton = removeButtons[0]!; if (!removeButton) return; fireEvent.click(removeButton); expect(screen.queryByText("Allergic to chicken")).toBeNull(); -- 2.52.0