From 2b2cfc58038a3a8bb527576d820da91602d5275e Mon Sep 17 00:00:00 2001
From: Ben <b.w.donnelly1@googlemail.com>
Date: Wed, 8 Apr 2026 20:43:46 +0200
Subject: [PATCH 1/2] Add refresh button to PR drawer

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 src/renderer/src/components/PrDrawer.tsx | 27 +++++++++++++++---------
 1 file changed, 17 insertions(+), 10 deletions(-)

diff --git a/src/renderer/src/components/PrDrawer.tsx b/src/renderer/src/components/PrDrawer.tsx
index aedfd4f..7204c09 100644
--- a/src/renderer/src/components/PrDrawer.tsx
+++ b/src/renderer/src/components/PrDrawer.tsx
@@ -1,4 +1,4 @@
-import { useState, useEffect } from 'react'
+import { useState, useEffect, useCallback } from 'react'
 import Markdown from 'react-markdown'
 import remarkGfm from 'remark-gfm'
 import type { PrDetail, PrCheckRun } from '../../../shared/types'
@@ -60,13 +60,12 @@ export default function PrDrawer({
   const [loading, setLoading] = useState(true)
   const [error, setError] = useState<string | null>(null)
 
-  useEffect(() => {
-    let cancelled = false
-    // Reset handled via initial state; subsequent fetches reset in callbacks
+  const refresh = useCallback(() => {
+    setLoading(true)
+    setError(null)
     window.konductorAPI
       .getPrDetail(cwd, prNumber)
       .then((d) => {
-        if (cancelled) return
         if (d) {
           setDetail(d)
           setError(null)
@@ -75,16 +74,17 @@ export default function PrDrawer({
         }
       })
       .catch(() => {
-        if (!cancelled) setError('Failed to load PR details')
+        setError('Failed to load PR details')
       })
       .finally(() => {
-        if (!cancelled) setLoading(false)
+        setLoading(false)
       })
-    return () => {
-      cancelled = true
-    }
   }, [cwd, prNumber])
 
+  useEffect(() => {
+    refresh()
+  }, [refresh])
+
   return (
     <div className="h-full flex flex-col bg-surface-raised">
       {/* Header */}
@@ -96,6 +96,13 @@ export default function PrDrawer({
           {detail && <span className="text-xs text-gray-400 truncate">{detail.title}</span>}
         </div>
         <div className="flex items-center gap-1 shrink-0">
+          <button
+            onClick={refresh}
+            className="text-xs text-gray-500 hover:text-white px-1.5 py-0.5 transition-colors"
+            title="Refresh"
+          >
+            &#8635;
+          </button>
           <button
             onClick={() => onOpenExternal(detail?.url ?? '')}
             className="text-xs text-gray-500 hover:text-white px-1.5 py-0.5 transition-colors"

From 5fa669c96385e43575fa49dae59adb5668c878a7 Mon Sep 17 00:00:00 2001
From: Ben <b.w.donnelly1@googlemail.com>
Date: Wed, 8 Apr 2026 22:39:01 +0200
Subject: [PATCH 2/2] Add lint:fix npm script and fix set-state-in-effect lint
 error

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 package.json                             |  1 +
 src/renderer/src/components/PrDrawer.tsx | 14 +++++++++-----
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/package.json b/package.json
index b568047..bd1d12d 100644
--- a/package.json
+++ b/package.json
@@ -10,6 +10,7 @@
     "test:watch": "vitest",
     "format": "prettier --write .",
     "lint": "eslint --cache .",
+    "lint:fix": "eslint --cache --fix .",
     "typecheck:node": "tsc --noEmit -p tsconfig.node.json --composite false",
     "typecheck:web": "tsc --noEmit -p tsconfig.web.json --composite false",
     "typecheck": "npm run typecheck:node && npm run typecheck:web",
diff --git a/src/renderer/src/components/PrDrawer.tsx b/src/renderer/src/components/PrDrawer.tsx
index 7204c09..d8263b6 100644
--- a/src/renderer/src/components/PrDrawer.tsx
+++ b/src/renderer/src/components/PrDrawer.tsx
@@ -60,9 +60,7 @@ export default function PrDrawer({
   const [loading, setLoading] = useState(true)
   const [error, setError] = useState<string | null>(null)
 
-  const refresh = useCallback(() => {
-    setLoading(true)
-    setError(null)
+  const fetchPr = useCallback(() => {
     window.konductorAPI
       .getPrDetail(cwd, prNumber)
       .then((d) => {
@@ -82,8 +80,14 @@ export default function PrDrawer({
   }, [cwd, prNumber])
 
   useEffect(() => {
-    refresh()
-  }, [refresh])
+    fetchPr()
+  }, [fetchPr])
+
+  const refresh = useCallback(() => {
+    setLoading(true)
+    setError(null)
+    fetchPr()
+  }, [fetchPr])
 
   return (
     <div className="h-full flex flex-col bg-surface-raised">