packages/bruno-app/src/components/ResponsePane/index.js (2)

55-65: Consider optimizing useEffect dependencies for performance.

Including the focusedTab object reference as a dependency may cause this effect to run more frequently than necessary, since the object is recreated whenever the tabs array changes.

While the guards prevent unwanted side effects, consider depending on more specific values:

🔎 Suggested optimization

-  }, [initialFormat, initialTab, persistedFormat, persistedViewTab, focusedTab, item.uid, dispatch]);
+  }, [initialFormat, initialTab, persistedFormat, persistedViewTab, activeTabUid, item.uid, dispatch]);

This avoids unnecessary effect runs when the tabs array is modified but the active tab hasn't actually changed.

---

59-64: Note: Existing tabs from prior versions will have undefined fields.

The initialization checks persistedFormat === null and persistedViewTab === null, but for tabs created before this PR, these fields will be undefined (not null). The strict equality check will skip initialization for those tabs.

The fallback chain still ensures correct behavior, so existing tabs will display properly but won't persist their format/view until users explicitly change them. This is likely acceptable, but if you want consistent initialization for existing tabs, consider:

🔎 Alternative check for both null and undefined

-    if (persistedFormat === null) {
+    if (persistedFormat == null) {  // checks both null and undefined
       dispatch(updateResponseFormat({ uid: item.uid, responseFormat: initialFormat }));
     }
-    if (persistedViewTab === null) {
+    if (persistedViewTab == null) {  // checks both null and undefined
       dispatch(updateResponseViewTab({ uid: item.uid, responseViewTab: initialTab }));
     }

Note: This uses loose equality (==) which is generally discouraged, but is idiomatic for null/undefined checks. Alternatively, use explicit checks: persistedFormat === null || persistedFormat === undefined.

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Learnt from: naman-bruno
Repo: usebruno/bruno PR: 6407
File: packages/bruno-app/src/components/Environments/ConfirmCloseEnvironment/index.js:5-41
Timestamp: 2025-12-17T21:41:24.730Z
Learning: Do not suggest PropTypes validation for React components in the Bruno codebase. The project does not use PropTypes, so reviews should avoid proposing PropTypes and rely on the existing typing/validation approach (e.g., TypeScript or alternative runtime checks) if applicable. This guideline applies broadly to all JavaScript/JSX components in the repo.

Learnt from: sid-bruno
Repo: usebruno/bruno PR: 6266
File: packages/bruno-app/src/components/ResponsePane/ResponseCopy/index.js:38-38
Timestamp: 2025-12-02T09:45:31.709Z
Learning: In the ResponseCopy component (packages/bruno-app/src/components/ResponsePane/ResponseCopy/index.js), the copy button is intentionally disabled using `!response.data` to prevent copying stream resets which result in empty strings.