fix: prevent governance audit completion when repo audits fail

README.md

@@ -2,7 +2,7 @@
 
 <div align="center" style="display: flex; align-items: center; justify-content: center; gap: 16px;">
   <img alt="AOSSIE" src="public/aossie-logo.svg" width="175">
-  <img src="public/org-explorer-logo.png" width="175" />
+  <img alt="OrgExplorer" src="public/org-explorer-logo.svg" width="175" />
 </div>
 
 <div align="center">

index.html

@@ -23,8 +23,12 @@
 
   <link rel="canonical" href="https://orgexplorer.aossie.org/" />
 
-  <link rel="icon" type="image/png" href="/org-explorer-logo.png" />
-  <link rel="apple-touch-icon" href="/org-explorer-logo.png" />
+<!-- Favicon: SVG first, PNG fallback for Safari < 18.8 -->
+<link rel="icon" type="image/svg+xml" href="/org-explorer-logo.svg" />
+<link rel="icon" type="image/png" href="/org-explorer-icon.png" sizes="64x64" />
+
+<!-- Apple touch icon: PNG only (Apple doesn't support SVG for this) -->
+<link rel="apple-touch-icon" href="/org-explorer-icon.png" />
 
   <meta property="og:type" content="website" />
 

src/context/AppContext.jsx

@@ -41,6 +41,7 @@ export function AppProvider({ children }) {
   const [advanceAnalyticsComplete, setAdvanceAnalyticsComplete] = useState(false) 
   const [isComplete, setIsComplete] = useState(false)
   const [auditComplete, setAuditComplete] = useState(false)
+  const [auditFailures, setAuditFailures] = useState([])
   const [lastOrgNames, setLastOrgNames] = useState([])
   // True until the cached analysis has been read, so routes that need a model
   // wait for the restore instead of bouncing to the picker on first paint.
@@ -220,23 +221,35 @@ export function AppProvider({ children }) {
     const repos = selectAnalysisRepos(allRepos)
 
     const map = {}
+    let hasFailures = false
+    const failedKeys = []
     for (let i = 0; i < repos.length; i += 5) {
       const batch = repos.slice(i, i + 5)
-      await Promise.allSettled(batch.map(async repo => {
+      const results = await Promise.allSettled(batch.map(async repo => {
         map[`${repo.orgLogin}/${repo.name}`] = await fetchIssues(repo.orgLogin, repo.name, pat)
       }))
+      results.forEach((r, idx) => {
+        if (r.status === 'rejected') {
+          hasFailures = true
+          failedKeys.push(`${batch[idx].orgLogin}/${batch[idx].name}`)
+        }
+      })
+    }
+    if (failedKeys.length) {
+      setAuditFailures(prev => [...new Set([...prev, ...failedKeys])])
     }
-    return map
+    return { map, hasFailures }
   }, [pat, selectAnalysisRepos])
 
   // Governance audit : used directly when repos are already complete
   const runAudit = useCallback(async () => {
     if (!model || govLoading) return
     setGovLoading(true)
-    const map = await auditRepos(model.allRepos)
+    setAuditFailures([])
+    const { map, hasFailures } = await auditRepos(model.allRepos)
     setIssuesData(map)
     setGovLoading(false)
-    setAuditComplete(!!pat)
+    setAuditComplete(!hasFailures && !!pat)
   }, [model, pat, govLoading, auditRepos])
 
   // Entry point for Governance / Analytics "Run Complete Analysis"
@@ -259,10 +272,11 @@ export function AppProvider({ children }) {
     if (!currentModel) return
 
     setGovLoading(true)
-    const map = await auditRepos(currentModel.allRepos)
+    setAuditFailures([])
+    const { map, hasFailures } = await auditRepos(currentModel.allRepos)
     setIssuesData(map)
     setGovLoading(false)
-    setAuditComplete(!!pat)
+    setAuditComplete(!hasFailures && !!pat)
   }, [isComplete, model, runFullExplore, auditRepos, pat, govLoading])
 
   // Advanced analytics — parallel batches of 5 (Section 3.2.5)
@@ -321,7 +335,11 @@ export function AppProvider({ children }) {
     setAdvanceAnalyticsLoading(true)
 
     const [issuesMap, pullsMap] = await Promise.all([
-      auditRepos(currentModel.allRepos),
+      (async () => {
+        const { map, hasFailures } = await auditRepos(currentModel.allRepos)
+        if (hasFailures) setAuditFailures(prev => prev)
+        return map
+      })(),
       (async () => {
         const repos = selectAnalysisRepos(currentModel.totalRepos)
         const map = {}
@@ -383,7 +401,7 @@ export function AppProvider({ children }) {
       rateLimit, loading, loadMsg, govLoading, error, totalRepo,
       runAdvanceAnalytics, refreshRateLimit, advanceAnalyticsLoading, advanceAnalyticsComplete,
       runFullAnalytics,
-      isComplete, auditComplete, lastOrgNames, hydrating,
+      isComplete, auditComplete, auditFailures, lastOrgNames, hydrating,
       explore, runFullExplore, runAudit, runGovernanceAnalysis, setError, staleRepoStats
     }}>
       {children}