Merge pull request #69 from ArjinAlbay/main

The bugs in the sidebar have been resolved and a feature has been added that starts capturing data when logged into the system.

Author: ArjinAlbay

src/app/api/auth/[...nextauth]/route.ts

@@ -1,56 +1,27 @@
 import NextAuth from "next-auth/next"
 import GitHubProvider from "next-auth/providers/github"
 
+const NEXTAUTH_SECRET = process.env.NEXTAUTH_SECRET
+if (process.env.NODE_ENV === 'production' && !NEXTAUTH_SECRET) {
+  throw new Error('NEXTAUTH_SECRET is required in production.')
+}
+const GITHUB_CLIENT_ID = process.env.GITHUB_CLIENT_ID
+const GITHUB_CLIENT_SECRET = process.env.GITHUB_CLIENT_SECRET
+if (
+  process.env.NODE_ENV === 'production' &&
+  (!GITHUB_CLIENT_ID || !GITHUB_CLIENT_SECRET)
+) {
+  throw new Error('GITHUB_CLIENT_ID and GITHUB_CLIENT_SECRET are required in production.')
+}
+
 const authOptions = {
+  secret: NEXTAUTH_SECRET,
   providers: [
     GitHubProvider({
-      clientId: process.env.GITHUB_CLIENT_ID!,
-      clientSecret: process.env.GITHUB_CLIENT_SECRET!,
-      authorization: {
-        params: {
-          scope: 'read:user user:email public_repo'
-        }
-      }
-    })
+      clientId: GITHUB_CLIENT_ID!,
+      clientSecret: GITHUB_CLIENT_SECRET!,
+    }),
   ],
-  callbacks: {
-    async jwt({ token, account, profile }: { 
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      token: any; 
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      account: any; 
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      profile?: any 
-    }) {
-      if (account?.access_token) {
-        token.accessToken = account.access_token
-        token.login = profile?.login
-      }
-      return token
-    },
-    async session({ session, token }: { 
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      session: any; 
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      token: any 
-    }) {
-      if (token.accessToken) {
-        session.accessToken = token.accessToken
-      }
-      if (token.login && session.user) {
-        session.user.login = token.login
-      }
-      return session
-    },
-    async redirect({ url, baseUrl }: { 
-      url: string; 
-      baseUrl: string 
-    }) {
-      if (url.startsWith('/')) return `${baseUrl}${url}`
-      if (new URL(url).origin === baseUrl) return url
-      return `${baseUrl}/auth/callback`
-    }
-  },
   pages: {
     signIn: '/login',
     error: '/login'

src/app/api/user/[username]/route.ts

@@ -0,0 +1,48 @@
+import { NextRequest, NextResponse } from 'next/server'
+import { githubAPIClient } from '@/lib/api/github-api-client'
+
+export async function GET(
+  request: NextRequest,
+  { params }: { params: Promise<{ username: string }> }
+) {
+  try {
+    // Next.js 15'te params bir Promise döndürüyor
+    const { username } = await params
+
+    if (!username) {
+      return NextResponse.json(
+        { error: 'Username parameter is required' },
+        { status: 400 }
+      )
+    }
+
+    const userAnalytics = await githubAPIClient.getUserAnalytics(username)
+
+    if (!userAnalytics) {
+      return NextResponse.json(
+        { error: 'User not found or data could not be retrieved' },
+        { status: 404 }
+      )
+    }
+
+    return NextResponse.json({
+      username,
+      data: userAnalytics,
+      timestamp: new Date().toISOString()
+    })
+  } catch (error) {
+    console.error('User API error:', error)
+    
+    if (error instanceof Error && error.message.includes('not found')) {
+      return NextResponse.json(
+        { error: 'User not found' },
+        { status: 404 }
+      )
+    }
+
+    return NextResponse.json(
+      { error: 'Internal server error' },
+      { status: 500 }
+    )
+  }
+}

src/app/layout.tsx

@@ -7,6 +7,7 @@ import { OAuthSessionSync } from '@/components/providers/OAuthSessionSync'
 
 import { HydrationBoundary } from "@/components/providers/HydrationBoundary";
 import { NotificationProvider } from "@/components/common/NotificationProvider";
+import { DataInitializer } from "@/components/providers/DataInitializer";
 
 
 const geistSans = Geist({
@@ -39,6 +40,7 @@ export default function RootLayout({
           <ThemeProvider defaultTheme="system" storageKey="githubmon-theme">
             <HydrationBoundary>
               <OAuthSessionSync />
+              <DataInitializer />
               {children}
               <NotificationProvider />
             </HydrationBoundary>

src/app/quick-wins/page.tsx

@@ -14,6 +14,7 @@ import {
 } from 'lucide-react'
 import { Tabs, TabsContent, TabsList, TabsTrigger } from '@/components/ui/tabs'
 import { useState, useEffect } from 'react'
+import { RateLimitWarning } from '@/components/common/RateLimitWarning'
 
 const VALID_TABS = ['good-issues', 'easy-fixes'] as const
 type ValidTab = typeof VALID_TABS[number]

src/app/search/page.tsx

@@ -115,18 +115,17 @@ export default function SearchPage() {
     try {
       const analytics = await githubAPIClient.getUserAnalytics(userParam);
       if (analytics) {
-        // Convert GitHubUserDetailed to the expected profile format
         const convertedAnalytics: UserAnalytics = {
           profile: analytics.profile ? {
             avatar_url: analytics.profile.avatar_url,
             login: analytics.profile.login,
             type: analytics.profile.type,
-            bio: analytics.profile.bio,
+            bio: analytics.profile.bio ?? null,
             public_repos: analytics.profile.public_repos,
             followers: analytics.profile.followers,
             following: analytics.profile.following,
-            location: analytics.profile.location,
-            company: analytics.profile.company,
+            location: analytics.profile.location ?? null,
+            company: analytics.profile.company ?? null,
             html_url: analytics.profile.html_url
           } : undefined,
           overview: analytics.overview,
@@ -174,7 +173,6 @@ export default function SearchPage() {
     []
   );
 
-  // Scroll spy effect - ONLY h2 titles matter
   useEffect(() => {
     const handleScroll = throttle(() => {
       const sectionIds = ['overview', 'behavior', 'star', 'code', 'code-review', 'issue', 'monthly-stats', 'contribution-activities'];
@@ -216,33 +214,10 @@ export default function SearchPage() {
     return () => window.removeEventListener("scroll", handleScroll);
   }, [activeSection, throttle]);
 
-  const loadUserAnalytics = useCallback(async () => {
-    if (!userParam) return;
-    const requestedUser = userParam;
-
-    setLoadingAnalytics(true);
-    try {
-      const analytics = await githubAPIClient.getUserAnalytics(requestedUser);
-      // Ignore if param changed while awaiting
-      if (requestedUser !== userParam) return;
-      setUserAnalytics(analytics);
-    } catch {
-      // Fallback to null if API fails
-      if (requestedUser !== userParam) return;
-      setUserAnalytics(null);
-    } finally {
-      if (requestedUser === userParam) {
-        setLoadingAnalytics(false);
-      }
-    }
-  }, [userParam]);
-
   useEffect(() => {
     if (userParam || repoParam) {
       const query = userParam || repoParam || "";
       const type = userParam ? "users" : "repos";
-
-      // Clear previous state when switching users
       setUserAnalytics(null);
       setSearchResults({
         repos: [],
@@ -256,7 +231,6 @@ export default function SearchPage() {
       performSearch(query, type);
       setHasSearched(true);
 
-      // If searching for a user, load analytics
       if (userParam) {
         loadUserAnalytics();
       }

src/app/settings/page.tsx

@@ -8,7 +8,7 @@ import { Input } from '@/components/ui/input'
 import { usePreferencesStore } from '@/stores'
 import { useRequireAuth } from '@/hooks/useAuth'
 import { ThemeSelector } from '@/components/theme/ThemeToggle'
-import { GitHubTokenSetup } from '@/components/common/GitHubTokenSetup'
+// import { GitHubTokenSetup } from '@/components/common/GitHubTokenSetup'
 import { cookieUtils } from '@/lib/cookies'
 
 export default function SettingsPage() {
@@ -48,7 +48,7 @@ export default function SettingsPage() {
           {/* GitHub API Token Setup */}
           <section>
             <h3 className="text-lg font-medium mb-4">🔑 GitHub API Token</h3>
-            <GitHubTokenSetup />
+            {/* <GitHubTokenSetup /> */}
           </section>
 
           <Card>

src/components/layout/Sidebar.tsx

@@ -2,7 +2,7 @@
 
 import Link from 'next/link'
 import { useSearchParams, usePathname } from 'next/navigation'
-import { useState } from 'react'
+import { useState, useEffect } from 'react'
 import { Button } from '@/components/ui/button'
 import { useSidebarState, useAuthStore, useStoreHydration, useActionItemsStore } from '@/stores'
 import { useQuickWinsStore } from '@/stores/quickWins'
@@ -22,8 +22,16 @@ export function Sidebar() {
 
   const { goodIssues, easyFixes, loading: quickWinsLoading } = useQuickWinsStore()
 
-  const [actionRequiredOpen, setActionRequiredOpen] = useState(true)
-  const [quickWinsOpen, setQuickWinsOpen] = useState(true)
+  const [actionRequiredOpen, setActionRequiredOpen] = useState(false)
+  const [quickWinsOpen, setQuickWinsOpen] = useState(false)
+
+  // Close accordions when navigating to dashboard
+  useEffect(() => {
+    if (pathname === '/dashboard') {
+      setActionRequiredOpen(false)
+      setQuickWinsOpen(false)
+    }
+  }, [pathname])
 
   const currentTab = searchParams?.get('tab') || 'assigned'
 
@@ -207,9 +215,9 @@ export function Sidebar() {
                 {/* Quick Wins sub-items */}
                 <CollapsibleContent className="pl-8 space-y-1 mt-1">
                   <Link
-                    href="/quick-wins?tab=good-first-issues"
+                    href="/quick-wins?tab=good-issues"
                     className={`flex items-center gap-2 px-3 py-1.5 text-sm rounded transition-colors
-                        ${(pathname === '/quick-wins' && currentTab === 'good-first-issues')
+                        ${(pathname === '/quick-wins' && currentTab === 'good-issues')
                         ? 'bg-blue-50 text-blue-600 dark:bg-blue-900/20 dark:text-blue-400'
                         : 'text-gray-600 hover:text-blue-600 hover:bg-gray-50 dark:hover:bg-gray-800'
                       }`}>
@@ -221,7 +229,7 @@ export function Sidebar() {
                       {getBadgeContent('goodFirstIssues')}
                     </Badge>
                   </Link>
-                  <Link
+                    <Link
                     href="/quick-wins?tab=easy-fixes"
                     className={`flex items-center gap-2 px-3 py-1.5 text-sm rounded transition-colors
                         ${(pathname === '/quick-wins' && currentTab === 'easy-fixes')

src/components/providers/DataInitializer.tsx

@@ -0,0 +1,24 @@
+'use client'
+
+import { useEffect , useRef} from 'react'
+import { useAuthStore, useActionItemsStore } from '@/stores'
+import { useQuickWinsStore } from '@/stores/quickWins'
+
+export function DataInitializer() {
+ const { isConnected, orgData, isTokenValid } = useAuthStore()
+ const { refreshData } = useActionItemsStore()
+ const { fetchGoodIssues, fetchEasyFixes } = useQuickWinsStore()
+
+ const initializedRef = useRef(false)
+ const tokenValid = isTokenValid()
+  useEffect(() => {
+  if (!initializedRef.current && isConnected && orgData && tokenValid) {
+      initializedRef.current = true
+      refreshData()
+      fetchGoodIssues()
+      fetchEasyFixes()
+    }
+  }, [isConnected, orgData, tokenValid, refreshData, fetchGoodIssues, fetchEasyFixes])
+
+  return null
+}

src/components/quick-wins/hooks/useQuickWins.ts

@@ -1,5 +1,5 @@
 
-import { useEffect, useCallback, useRef } from 'react'
+import { useEffect } from 'react'
 import { useQuickWinsStore } from '@/stores/quickWins'
 import { useDataCacheStore } from '@/stores/cache'
 import { useActionItemsStore } from '@/stores'
@@ -36,21 +36,8 @@ export function useQuickWins() {
     const { isQuickWinsCacheExpired } = useDataCacheStore()
     const { setGoodFirstIssues, setEasyFixes } = useActionItemsStore()
 
-    const isInitialized = useRef(false)
 
-    const initializeData = useCallback(async () => {
-        if (isInitialized.current) return
-        loadFromCache()
-        const { goodIssues: gi, easyFixes: ef } = useQuickWinsStore.getState()
-        const needsFetch = isQuickWinsCacheExpired() || (gi.length === 0 && ef.length === 0)
-        if (needsFetch) {
-            await Promise.all([
-                fetchGoodIssues(false),
-                fetchEasyFixes(false)
-            ])
-        }
-        isInitialized.current = true
-    }, [loadFromCache, isQuickWinsCacheExpired, fetchGoodIssues, fetchEasyFixes])
+    
 
     useEffect(() => {
         if (goodIssues.length > 0) {

src/lib/api/github-api-client.ts

@@ -436,7 +436,6 @@ class GitHubAPIClient {
  try {
     const oneMonthAgo = new Date()
     oneMonthAgo.setMonth(oneMonthAgo.getMonth() - 1)
-    const dateString = oneMonthAgo.toISOString().split('T')[0]
 
     const repoEndpoint = `/search/repositories?q=stars:>${minStars}&sort=stars&order=desc&per_page=50`
     const repoResponse = await this.fetchWithCache<GitHubSearchResponse<GitHubRepositoryResponse>>(repoEndpoint, true)