hale-commenting-system 3.4.0 → 3.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hale-commenting-system",
-  "version": "3.4.0",
+  "version": "3.4.1",
   "description": "A commenting system for PatternFly React applications that allows designers and developers to add comments directly on design pages, sync with GitHub Issues, and link Jira tickets.",
   "homepage": "https://www.npmjs.com/package/hale-commenting-system",
   "license": "MIT",

package/src/app/commenting-system/components/CommentOverlay.tsx

@@ -72,10 +72,13 @@ export const CommentOverlay: React.FunctionComponent = () => {
         position: 'absolute',
         top: 0,
         left: 0,
+        right: 0,
+        bottom: 0,
         width: '100%',
         height: '100%',
         pointerEvents: 'none',
         zIndex: 999,
+        overflow: 'visible', // Ensure pins can be visible even if slightly outside
       }}
     >
       {currentThreads.map((thread) => (

package/src/app/commenting-system/contexts/CommentContext.tsx

@@ -463,14 +463,18 @@ export const CommentProvider: React.FunctionComponent<{ children: React.ReactNod
             };
           });
 
+          // Track which issue numbers were returned from GitHub
+          const githubIssueNumbers = new Set(ghThreads.map(gt => gt.issueNumber).filter((n): n is number => !!n));
+
           // Keep local threads on this route/version that:
           // 1. Don't have an issueNumber yet, OR
-          // 2. Are actively syncing (prevents race condition where issue was created but GitHub API hasn't returned it yet)
+          // 2. Are actively syncing (prevents race condition where issue was created but GitHub API hasn't returned it yet), OR
+          // 3. Have an issueNumber but GitHub didn't return it (preserve existing threads even if GitHub API fails or issue is hidden)
           const localUnlinked = prev.filter(
             (t) =>
               t.route === route &&
               (t.version ?? '1') === (version ?? '1') &&
-              (!t.issueNumber || t.syncStatus === 'syncing'),
+              (!t.issueNumber || t.syncStatus === 'syncing' || !githubIssueNumbers.has(t.issueNumber)),
           );
 
           // Remove duplicates: if a thread is both in localUnlinked and merged, prefer the merged version