Optimize code

app/error.tsx

@@ -2,7 +2,7 @@
 
 import { useEffect } from "react";
 import { Button } from "@heroui/react";
-import { AlertCircle, RefreshCcw } from "lucide-react";
+import { AlertCircle, RefreshCcw, RotateCcw } from "lucide-react";
 import { reportError } from "@/lib/error-reporting";
 
 export default function Error({
@@ -34,6 +34,7 @@ export default function Error({
           onClick={() => window.location.reload()}
           className="focus-ring rounded-xl px-8"
         >
+          <RotateCcw className="h-4 w-4 mr-2" aria-hidden="true" />
           Reload Page
         </Button>
         <Button

app/lib/error-reporting.ts

@@ -24,8 +24,12 @@ function sanitizeError(data: Error, seen: WeakSet<object>): Error {
 
 function sanitizeObject(data: Record<string, unknown>, seen: WeakSet<object>): Record<string, unknown> {
   const sanitizedObj: Record<string, unknown> = {};
-  for (const [key, value] of Object.entries(data)) {
-    sanitizedObj[key] = SENSITIVE_KEY_PATTERN.test(key) ? "[REDACTED]" : sanitize(value, seen);
+  for (const key in data) {
+    if (Object.hasOwn(data, key)) {
+      sanitizedObj[key] = SENSITIVE_KEY_PATTERN.test(key)
+        ? "[REDACTED]"
+        : sanitize(data[key], seen);
+    }
   }
   return sanitizedObj;
 }

app/page.tsx

@@ -17,9 +17,7 @@ const JSON_LD_STRING = safeJsonLdStringify(JSON_LD);
 export default function Page() {
   return (
     <div className="flex flex-col gap-20 pb-20">
-      <script type="application/ld+json">
-        {JSON_LD_STRING}
-      </script>
+      <script type="application/ld+json" dangerouslySetInnerHTML={{ __html: JSON_LD_STRING }} />
       <Hero />
       <Features />
     </div>

tests/unit/app/error.test.tsx

@@ -40,6 +40,17 @@ describe("Error component", () => {
     expect(mockReset).toHaveBeenCalledTimes(1);
   });
 
+  it("ensures decorative icons are hidden from screen readers", () => {
+    const { container } = render(
+      <ErrorComponent error={mockError} reset={mockReset} />
+    );
+    const svgs = container.querySelectorAll("svg");
+    expect(svgs.length).toBeGreaterThan(0);
+    svgs.forEach((svg) => {
+      expect(svg).toHaveAttribute("aria-hidden", "true");
+    });
+  });
+
   it("reloads the page when 'Reload Page' button is clicked", () => {
     const reloadMock = vi.fn();