@moontra/moonui-pro 2.18.0 → 2.18.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moontra/moonui-pro",
-  "version": "2.18.0",
+  "version": "2.18.2",
   "description": "Premium React components for MoonUI - Advanced UI library with 50+ pro components including performance, interactive, and gesture components",
   "type": "module",
   "main": "dist/index.mjs",

package/src/components/github-stars/hooks.ts

@@ -54,17 +54,32 @@ export function useGitHubData({
   
   const refreshTimeoutRef = useRef<NodeJS.Timeout>()
   const previousStarsRef = useRef<Map<string, number>>(new Map())
+  const errorCountRef = useRef<number>(0) // Hata sayısını takip et
+  const maxErrorCount = 2 // Maksimum hata sayısı
+
+  // checkMilestones fonksiyonunu ref olarak sakla - bu şekilde her render'da yeniden oluşturulmaz
+  const milestonesRef = useRef(milestones)
+  const onMilestoneReachedRef = useRef(onMilestoneReached)
+  
+  // Ref'leri güncelle
+  useEffect(() => {
+    milestonesRef.current = milestones
+  }, [milestones])
+  
+  useEffect(() => {
+    onMilestoneReachedRef.current = onMilestoneReached
+  }, [onMilestoneReached])
 
   const checkMilestones = useCallback((repos: GitHubRepository[]) => {
-    if (!onMilestoneReached) return
+    if (!onMilestoneReachedRef.current) return
 
     repos.forEach(repo => {
       const previousStars = previousStarsRef.current.get(repo.full_name) || 0
       const currentStars = repo.stargazers_count
 
-      milestones.forEach(milestone => {
+      milestonesRef.current.forEach(milestone => {
         if (previousStars < milestone && currentStars >= milestone) {
-          onMilestoneReached({
+          onMilestoneReachedRef.current?.({
             count: milestone,
             reached: true,
             date: new Date().toISOString(),
@@ -75,9 +90,33 @@ export function useGitHubData({
 
       previousStarsRef.current.set(repo.full_name, currentStars)
     })
-  }, [milestones, onMilestoneReached])
+  }, []) // Boş dependency array - fonksiyon asla yeniden oluşturulmaz
 
   const fetchData = useCallback(async () => {
+    // Hata limiti aşıldıysa istek yapma
+    if (errorCountRef.current >= maxErrorCount) {
+      console.warn("Maximum error count reached. Stopping requests.")
+      setLoading(false)
+      setError("Maximum retry limit exceeded. Please check your configuration.")
+      return
+    }
+
+    // Gerekli bilgiler yoksa istek yapma
+    const hasValidInput = 
+      (username && repository) || // Tek repository modu
+      (username && repositories && repositories.length > 0) || // Çoklu repository modu
+      (repositories && repositories.length > 0 && repositories.every(r => r.includes('/'))) || // Full path repositories
+      username // Kullanıcının tüm repositoryleri
+
+    if (!hasValidInput) {
+      console.warn("No valid input provided. Skipping API request.")
+      setLoading(false)
+      setError(null)
+      setRepos([])
+      setStats(null)
+      return
+    }
+
     try {
       setLoading(true)
       setError(null)
@@ -101,7 +140,15 @@ export function useGitHubData({
         const repo = await fetchRepository(username, repository, token)
         fetchedRepos = [repo]
       }
-      // Multiple specific repositories
+      // Multiple specific repositories with full paths
+      else if (repositories && repositories.length > 0 && repositories.every(r => r.includes('/'))) {
+        const repoPromises = repositories.map(fullPath => {
+          const [owner, name] = fullPath.split('/')
+          return fetchRepository(owner, name, token)
+        })
+        fetchedRepos = await Promise.all(repoPromises)
+      }
+      // Multiple specific repositories with username
       else if (repositories && repositories.length > 0 && username) {
         const repoPromises = repositories.map(repoName =>
           fetchRepository(username, repoName, token)
@@ -173,9 +220,16 @@ export function useGitHubData({
       }
       
       setLastUpdated(new Date())
+      // Başarılı olduğunda hata sayacını sıfırla
+      errorCountRef.current = 0
     } catch (err) {
       const errorMessage = err instanceof Error ? err.message : "Failed to fetch data"
       setError(errorMessage)
+      
+      // Hata sayacını artır
+      errorCountRef.current += 1
+      console.error(`GitHub API error (${errorCountRef.current}/${maxErrorCount}):`, errorMessage)
+      
       if (onError) {
         onError(err instanceof Error ? err : new Error(errorMessage))
       }
@@ -185,18 +239,29 @@ export function useGitHubData({
   }, [
     username,
     repository,
-    repositories,
+    repositories?.join(','), // Array'i string'e çevir ki referans değişmesin
     token,
     sortBy,
     maxItems,
-    checkMilestones,
+    checkMilestones, // Artık stable
     onDataUpdate,
     onError,
   ])
 
   // Initial fetch
   useEffect(() => {
-    fetchData()
+    // Sadece geçerli input varsa fetch yap
+    const hasValidInput = 
+      (username && repository) || 
+      (username && repositories && repositories.length > 0) || 
+      (repositories && repositories.length > 0 && repositories.every(r => r.includes('/'))) || 
+      username
+
+    if (hasValidInput) {
+      fetchData()
+    } else {
+      setLoading(false)
+    }
   }, [fetchData])
 
   // Auto-refresh
@@ -220,6 +285,12 @@ export function useGitHubData({
   }, [autoRefresh, refreshInterval, fetchData])
 
   const refresh = useCallback(() => {
+    // Hata limiti aşıldıysa refresh yapma
+    if (errorCountRef.current >= maxErrorCount) {
+      console.warn("Cannot refresh: maximum error count reached")
+      return Promise.resolve()
+    }
+    
     clearCache()
     return fetchData()
   }, [fetchData])

package/src/components/github-stars/index.tsx

@@ -62,6 +62,27 @@ const GitHubStarsInternal: React.FC<GitHubStarsProps> = ({
 }) => {
   const { notify } = useGitHubNotifications(enableNotifications)
 
+  // onMilestoneReached callback'ini memoize et
+  const handleMilestoneReached = React.useCallback((milestone: any) => {
+    // Handle milestone reached
+    if (celebrateAt?.includes(milestone.count)) {
+      // Trigger celebration animation
+      confetti({
+        particleCount: 100,
+        spread: 70,
+        origin: { y: 0.6 },
+      })
+      
+      // Show notification
+      notify(`🎉 Milestone Reached!`, {
+        body: `${milestone.count} stars achieved!`,
+        tag: `milestone-${milestone.count}`,
+      })
+    }
+    
+    onMilestoneReached?.(milestone)
+  }, [celebrateAt, notify, onMilestoneReached])
+
   const {
     repos,
     stats,
@@ -81,29 +102,11 @@ const GitHubStarsInternal: React.FC<GitHubStarsProps> = ({
     maxItems,
     onError,
     onDataUpdate,
-    onMilestoneReached: (milestone) => {
-      // Handle milestone reached
-      if (celebrateAt?.includes(milestone.count)) {
-        // Trigger celebration animation
-        confetti({
-          particleCount: 100,
-          spread: 70,
-          origin: { y: 0.6 },
-        })
-        
-        // Show notification
-        notify(`🎉 Milestone Reached!`, {
-          body: `${milestone.count} stars achieved!`,
-          tag: `milestone-${milestone.count}`,
-        })
-      }
-      
-      onMilestoneReached?.(milestone)
-    },
+    onMilestoneReached: handleMilestoneReached,
     milestones,
   })
 
-  const handleExport = (format: "json" | "csv") => {
+  const handleExport = React.useCallback((format: "json" | "csv") => {
     if (!enableExport) return
 
     const timestamp = new Date().toISOString().split("T")[0]
@@ -116,7 +119,47 @@ const GitHubStarsInternal: React.FC<GitHubStarsProps> = ({
     } else {
       exportAsCSV(repos, `github-stars-${username}-${timestamp}.csv`)
     }
-  }
+  }, [enableExport, repos, stats, username])
+
+  // handleDetailedExport fonksiyonunu memoize et
+  const handleDetailedExport = React.useCallback((e: React.MouseEvent) => {
+    if (!enableExport) return
+    
+    // Show export dropdown
+    const dropdown = document.createElement("div")
+    dropdown.className = "absolute z-50 mt-2 w-48 rounded-md shadow-lg bg-popover border"
+    dropdown.innerHTML = `
+      <div class="py-1">
+        <button class="w-full text-left px-4 py-2 text-sm hover:bg-accent" data-format="json">
+          Export as JSON
+        </button>
+        <button class="w-full text-left px-4 py-2 text-sm hover:bg-accent" data-format="csv">
+          Export as CSV
+        </button>
+      </div>
+    `
+    
+    dropdown.addEventListener("click", (e) => {
+      const target = e.target as HTMLElement
+      const format = target.getAttribute("data-format")
+      if (format === "json" || format === "csv") {
+        handleExport(format)
+        dropdown.remove()
+      }
+    })
+    
+    document.body.appendChild(dropdown)
+    
+    // Position dropdown
+    const rect = (e.target as HTMLElement).getBoundingClientRect()
+    dropdown.style.top = `${rect.bottom + window.scrollY}px`
+    dropdown.style.left = `${rect.left + window.scrollX}px`
+    
+    // Remove on outside click
+    setTimeout(() => {
+      document.addEventListener("click", () => dropdown.remove(), { once: true })
+    }, 0)
+  }, [enableExport, handleExport])
 
   // Loading state
   if (loading) {
@@ -201,42 +244,7 @@ const GitHubStarsInternal: React.FC<GitHubStarsProps> = ({
         return (
           <DetailedVariant 
             {...baseProps} 
-            onExport={() => {
-              // Show export dropdown
-              const dropdown = document.createElement("div")
-              dropdown.className = "absolute z-50 mt-2 w-48 rounded-md shadow-lg bg-popover border"
-              dropdown.innerHTML = `
-                <div class="py-1">
-                  <button class="w-full text-left px-4 py-2 text-sm hover:bg-accent" data-format="json">
-                    Export as JSON
-                  </button>
-                  <button class="w-full text-left px-4 py-2 text-sm hover:bg-accent" data-format="csv">
-                    Export as CSV
-                  </button>
-                </div>
-              `
-              
-              dropdown.addEventListener("click", (e) => {
-                const target = e.target as HTMLElement
-                const format = target.getAttribute("data-format")
-                if (format === "json" || format === "csv") {
-                  handleExport(format)
-                  dropdown.remove()
-                }
-              })
-              
-              document.body.appendChild(dropdown)
-              
-              // Position dropdown
-              const rect = (e.target as HTMLElement).getBoundingClientRect()
-              dropdown.style.top = `${rect.bottom + window.scrollY}px`
-              dropdown.style.left = `${rect.left + window.scrollX}px`
-              
-              // Remove on outside click
-              setTimeout(() => {
-                document.addEventListener("click", () => dropdown.remove(), { once: true })
-              }, 0)
-            }}
+            onExport={handleDetailedExport}
           />
         )
       case "card":