fix: prevent duplicate release notes checks using ref (#2912)

## Summary - Add `hasCheckedReleaseNotes` ref to track whether release notes have already been checked in the current session - Prevent the useEffect from running multiple times due to settings/appVersion dependency changes - Refactor conditional logic to use early-return pattern for improved readability ## Test plan - Open the app and navigate to Home page - Verify release notes dialog only shows once per session when there are new release notes - Confirm no duplicate API calls to `doesReleaseNoteExist` 🤖 Generated with [Claude Code](https://claude.com/claude-code)  --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2912" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a>  --------- Co-authored-by: Will Chen <willchen90@gmail.com> Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

fix: prevent duplicate release notes checks using ref (#2912)
936cc73c · wwwillchen-bot · GitHub · d8774673 · 936cc73c
--- a/src/pages/home.tsx
+++ b/src/pages/home.tsx
@@ -45,6 +45,10 @@ import {
 import { hasDyadProKey, getEffectiveDefaultChatMode } from "@/lib/schemas";
 import { useFreeAgentQuota } from "@/hooks/useFreeAgentQuota";
+// Track whether we've already checked release notes this session (module-scoped
+// so it persists across component unmount/remount cycles).
+let hasCheckedReleaseNotes = false;
 // Adding an export for attachments
 export interface HomeSubmitOptions {
  attachments?: FileAttachment[];
@@ -86,35 +90,39 @@ export default function HomePage() {
  useEffect(() => {
    const updateLastVersionLaunched = async () => {
      if (
-        appVersion &&
+        hasCheckedReleaseNotes ||
-        settings &&
+        !appVersion ||
-        settings.lastShownReleaseNotesVersion !== appVersion
+        !settings ||
+        settings.lastShownReleaseNotesVersion === appVersion
      ) {
-        const shouldShowReleaseNotes = !!settings.lastShownReleaseNotesVersion;
+        return;
-        await updateSettings({
+      }
-          lastShownReleaseNotesVersion: appVersion,
+      hasCheckedReleaseNotes = true;
-        });
-        // It feels spammy to show release notes if it's
-        // the users very first time.
-        if (!shouldShowReleaseNotes) {
-          return;
-        }
-        try {
+      const shouldShowReleaseNotes = !!settings.lastShownReleaseNotesVersion;
-          const result = await ipc.system.doesReleaseNoteExist({
+      await updateSettings({
-            version: appVersion,
+        lastShownReleaseNotesVersion: appVersion,
-          });
+      });
+      // It feels spammy to show release notes if it's
+      // the users very first time.
+      if (!shouldShowReleaseNotes) {
+        return;
+      }
+      try {
+        const result = await ipc.system.doesReleaseNoteExist({
+          version: appVersion,
+        });
-          if (result.exists && result.url) {
+        if (result.exists && result.url) {
-            setReleaseUrl(result.url + "?hideHeader=true&theme=" + theme);
+          setReleaseUrl(result.url + "?hideHeader=true&theme=" + theme);
-            setReleaseNotesOpen(true);
+          setReleaseNotesOpen(true);
-          }
-        } catch (err) {
-          console.warn(
-            "Unable to check if release note exists for: " + appVersion,
-            err,
-          );
        }
+      } catch (err) {
+        console.warn(
+          "Unable to check if release note exists for: " + appVersion,
+          err,
+        );
      }
    };
    updateLastVersionLaunched();