From f2231d7dada3f253de526fbe1459a6e9da733ab2 Mon Sep 17 00:00:00 2001
From: Mike <s.sardorb3k@gmail.com>
Date: Sun, 3 May 2026 14:42:23 +0500
Subject: [PATCH] fix(perf): fix remaining lazy-loading and dead toggle issues
 from PR review

- Gate TeamListView dialogs with showCreateDialog/launchDialogOpen
- Gate SchedulesView and ScheduleSection lazy dialogs with dialogOpen
- Fix import order in SchedulesView (import type before lazy constant)
- Add Show raw toggle button to MarkdownViewer rendered view
---
 .../chat/viewers/MarkdownViewer.tsx           | 36 +++++++++++++++----
 .../components/schedules/SchedulesView.tsx    | 30 +++++++++-------
 src/renderer/components/team/TeamListView.tsx |  4 +--
 .../team/schedule/ScheduleSection.tsx         | 20 ++++++-----
 4 files changed, 60 insertions(+), 30 deletions(-)

diff --git a/src/renderer/components/chat/viewers/MarkdownViewer.tsx b/src/renderer/components/chat/viewers/MarkdownViewer.tsx
index d5412610..6adeacfb 100644
--- a/src/renderer/components/chat/viewers/MarkdownViewer.tsx
+++ b/src/renderer/components/chat/viewers/MarkdownViewer.tsx
@@ -1169,12 +1169,36 @@ export const MarkdownViewer: React.FC<MarkdownViewerProps> = React.memo(function
           <span className="text-sm font-medium" style={{ color: COLOR_TEXT_SECONDARY }}>
             {label}
           </span>
-          {copyable && (
-            <>
-              <span className="flex-1" />
-              <CopyButton text={content} inline />
-            </>
-          )}
+          <span className="flex-1" />
+          <button
+            type="button"
+            className="text-xs underline"
+            style={{ color: PROSE_LINK }}
+            onClick={() => setShowRaw(true)}
+            title="Show raw"
+          >
+            Show raw
+          </button>
+          {copyable && <CopyButton text={content} inline />}
+        </div>
+      )}
+
+      {/* Show raw toggle for no-label path */}
+      {!label && (
+        <div
+          className="flex items-center justify-between px-3 py-1 text-xs"
+          style={{ color: COLOR_TEXT_MUTED }}
+        >
+          <span />
+          <button
+            type="button"
+            className="underline"
+            style={{ color: PROSE_LINK }}
+            onClick={() => setShowRaw(true)}
+            title="Show raw"
+          >
+            Show raw
+          </button>
         </div>
       )}
 
diff --git a/src/renderer/components/schedules/SchedulesView.tsx b/src/renderer/components/schedules/SchedulesView.tsx
index 96d72051..c8a2b2f0 100644
--- a/src/renderer/components/schedules/SchedulesView.tsx
+++ b/src/renderer/components/schedules/SchedulesView.tsx
@@ -28,12 +28,14 @@ import { ScheduleRunLogDialog } from '../team/schedule/ScheduleRunLogDialog';
 import { ScheduleRunRow } from '../team/schedule/ScheduleRunRow';
 import { ScheduleStatusBadge } from '../team/schedule/ScheduleStatusBadge';
 
-const LaunchTeamDialog = lazy(() =>
-  import('../team/dialogs/LaunchTeamDialog').then((m) => ({ default: m.LaunchTeamDialog }))
-);
-
 import type { Schedule, ScheduleRun, ScheduleStatus } from '@shared/types';
 
+const LaunchTeamDialog = lazy(() =>
+  import('@renderer/components/team/dialogs/LaunchTeamDialog').then((m) => ({
+    default: m.LaunchTeamDialog,
+  }))
+);
+
 // =============================================================================
 // Constants
 // =============================================================================
@@ -565,15 +567,17 @@ export const SchedulesView = (): React.JSX.Element => {
       </div>
 
       {/* Create/Edit Dialog */}
-      <Suspense fallback={null}>
-        <LaunchTeamDialog
-          mode="schedule"
-          open={dialogOpen}
-          teamName={editingSchedule?.teamName}
-          schedule={editingSchedule}
-          onClose={handleClose}
-        />
-      </Suspense>
+      {dialogOpen && (
+        <Suspense fallback={null}>
+          <LaunchTeamDialog
+            mode="schedule"
+            open={dialogOpen}
+            teamName={editingSchedule?.teamName}
+            schedule={editingSchedule}
+            onClose={handleClose}
+          />
+        </Suspense>
+      )}
     </div>
   );
 };
diff --git a/src/renderer/components/team/TeamListView.tsx b/src/renderer/components/team/TeamListView.tsx
index 5e597278..b3ab27f7 100644
--- a/src/renderer/components/team/TeamListView.tsx
+++ b/src/renderer/components/team/TeamListView.tsx
@@ -736,7 +736,7 @@ export const TeamListView = memo(function TeamListView(): React.JSX.Element {
     );
   }
 
-  const createDialogElement = (
+  const createDialogElement = showCreateDialog && (
     <Suspense fallback={null}>
       <CreateTeamDialog
         open={showCreateDialog}
@@ -755,7 +755,7 @@ export const TeamListView = memo(function TeamListView(): React.JSX.Element {
     </Suspense>
   );
 
-  const launchDialogElement = (
+  const launchDialogElement = launchDialogOpen && (
     <Suspense fallback={null}>
       <LaunchTeamDialog
         mode="launch"
diff --git a/src/renderer/components/team/schedule/ScheduleSection.tsx b/src/renderer/components/team/schedule/ScheduleSection.tsx
index db7dc5dd..2a755123 100644
--- a/src/renderer/components/team/schedule/ScheduleSection.tsx
+++ b/src/renderer/components/team/schedule/ScheduleSection.tsx
@@ -307,15 +307,17 @@ export const ScheduleSection = ({ teamName }: ScheduleSectionProps): React.JSX.E
       )}
 
       {/* Create/Edit Dialog */}
-      <Suspense fallback={null}>
-        <LaunchTeamDialog
-          mode="schedule"
-          open={dialogOpen}
-          teamName={teamName}
-          schedule={editingSchedule}
-          onClose={handleClose}
-        />
-      </Suspense>
+      {dialogOpen && (
+        <Suspense fallback={null}>
+          <LaunchTeamDialog
+            mode="schedule"
+            open={dialogOpen}
+            teamName={teamName}
+            schedule={editingSchedule}
+            onClose={handleClose}
+          />
+        </Suspense>
+      )}
     </div>
   );
 };