From b8cd4fd431223bab1715af697ab919b92732b62c Mon Sep 17 00:00:00 2001 From: cliffhall Date: Sat, 11 Jul 2026 23:24:20 -0400 Subject: [PATCH] =?UTF-8?q?web:=20address=20#1671=20review=20nit=20?= =?UTF-8?q?=E2=80=94=20simplify=20redundant=20autoOpen=20guard?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit DeepLink.autoOpen is a non-optional boolean, so the `?? false` was dead once openApp was truthy. Collapse to Boolean(deepLink?.openApp && deepLink.autoOpen) — still false when there's no deep link, still gated on openApp so a malformed autoOpen-without-openApp link can't fire. Co-Authored-By: Claude Opus 4.8 (1M context) Claude-Session: https://claude.ai/code/session_01BrihGWcrM9JGRyu41nzZYw --- .../web/src/components/views/InspectorView/InspectorView.tsx | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/clients/web/src/components/views/InspectorView/InspectorView.tsx b/clients/web/src/components/views/InspectorView/InspectorView.tsx index 29d56119..cd18bded 100644 --- a/clients/web/src/components/views/InspectorView/InspectorView.tsx +++ b/clients/web/src/components/views/InspectorView/InspectorView.tsx @@ -1190,9 +1190,7 @@ export function InspectorView({ onOpenApp={onOpenApp} onCloseApp={onCloseApp} onError={onAppError} - autoOpen={ - Boolean(deepLink?.openApp) && (deepLink?.autoOpen ?? false) - } + autoOpen={Boolean(deepLink?.openApp && deepLink.autoOpen)} />