From 7caff35ba3d3cb9081b1c5af3bad42732e9e7856 Mon Sep 17 00:00:00 2001
From: Konstantin Wohlwend <n2d4xc@gmail.com>
Date: Tue, 14 Apr 2026 09:19:18 -0700
Subject: [PATCH] Fix lint

---
 .../projects/[projectId]/widget-playground/page-client.tsx  | 6 +++---
 claude/CLAUDE-KNOWLEDGE.md                                  | 3 +++
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/widget-playground/page-client.tsx b/apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/widget-playground/page-client.tsx
index 39f388efd..bbedefbd7 100644
--- a/apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/widget-playground/page-client.tsx
+++ b/apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/widget-playground/page-client.tsx
@@ -1630,9 +1630,9 @@ function Draggable(props: {
                 <div className="text-red-500 text-sm p-2 bg-red-500/10 font-mono whitespace-pre-wrap">
                   A runtime error occured while rendering this widget.<br />
                   <br />
-                  {props.reset && <button className="text-blue-500 hover:underline" onClick={() => {
-                    props.reset!();
-                  }}>Reload widget</button>}<br />
+                  <button className="text-blue-500 hover:underline" onClick={() => {
+                    props.reset();
+                  }}>Reload widget</button><br />
                   <br />
                   {errorToNiceString(props.error)}
                 </div>
diff --git a/claude/CLAUDE-KNOWLEDGE.md b/claude/CLAUDE-KNOWLEDGE.md
index d4b642a2c..943980b48 100644
--- a/claude/CLAUDE-KNOWLEDGE.md
+++ b/claude/CLAUDE-KNOWLEDGE.md
@@ -261,3 +261,6 @@ A: In `docs-mintlify/snippets/docs-apps-home-grid.jsx`, track visible rows while
 
 Q: Why did internal feedback E2E tests expect 1 Inbucket message but get 2?
 A: Inbucket persists mail across runs. `Mailbox.waitForMessagesWithSubject` waits until at least one match then returns **all** messages whose subject includes the string. Fixed subjects like `[Support] devtool-user@example.com` accumulate, so assertions should use a unique subject per run (e.g. `randomUUID()` in the sender email) or a baseline count before/after.
+
+Q: Why does `@typescript-eslint/no-unnecessary-condition` fire on `props.reset` in Next.js `ErrorBoundary` `errorComponent`?
+A: Next’s typings treat `reset` as always present on the error component props, so `props.reset &&` is redundant; render the reload control unconditionally and call `props.reset()` directly.