github fixes

app/icon.svg

···
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16">
+  <style>
+    path {
+      fill: hsl(var(--primary)); /* Default to primary theme color */
+    }
+    @media (prefers-color-scheme: dark) {
+      path {
+        fill: #FFFFFF; /* White for dark mode */
+      }
+    }
+  </style>
+  <path 
+    fillRule="evenodd" 
+    d="M1.5 9c0 0.69781 0.10996 1.3699 0.3135 2H0v1.5h2.52182c0.11848 0.1851 0.24608 0.3637 0.38217 0.5353L1 14.9393 2.06066 16l1.90399 -1.904C5.07323 14.975 6.47531 15.5 8 15.5s2.9268 -0.525 4.0353 -1.404L13.9393 16 15 14.9393l-1.904 -1.904c0.1361 -0.1716 0.2637 -0.3502 0.3822 -0.5353H16V11h-1.8135c0.2035 -0.6301 0.3135 -1.30219 0.3135 -2V0.5C12.2612 0.5 10.366 1.97145 9.7289 4H6.2711C5.63397 1.97145 3.73882 0.5 1.5 0.5V9ZM8 13l-2 -2h4l-2 2Z" 
+    clipRule="evenodd"
+  />
+</svg> 

lib/repositories/index.ts

···
 // Import and re-export all repositories for easier imports
 
-export * from './user-repository';
-export * from './organization-repository';
-export * from './repository-repository';
-export * from './category-repository';
-export * from './pr-repository'; 
+export { 
+  findUserById, 
+  findUserByEmail, 
+  createUser, 
+  updateUser, 
+  updateOrganizationRole,
+  getUserOrganizations,
+  addUserToOrganization,
+  removeUserFromOrganization,
+  getOrganizationRole,
+  findOrCreateUserByGitHubId,
+} from './user-repository';
+export { 
+  findOrCreateOrganization, 
+  findOrganizationById, 
+  updateOrganization, 
+} from './organization-repository';
+export { 
+  findRepositoryById, 
+  findRepositoryByGitHubId, 
+  findOrCreateRepository, 
+  updateRepository, 
+  getOrganizationRepositories,
+  setRepositoryTracking, 
+  getTrackedRepositories,
+  findRepositoryByFullName,
+} from './repository-repository';
+export { 
+  createPullRequest, 
+  findPullRequestById, 
+  findPullRequestByNumber, 
+  updatePullRequest, 
+  getRepositoryPullRequests,
+  createPullRequestReview,
+  findReviewByGitHubId,
+  updatePullRequestReview,
+} from './pr-repository';
+export { 
+  getDefaultCategories, 
+  getOrganizationCategories, 
+  createCategory, 
+  updateCategory, 
+  deleteCategory, 
+  findCategoryById
+} from './category-repository';
+// Commented out sections for missing files remain for user to address
+// export { 
+//   getSettings, 
+//   updateSetting, 
+//   getOrganizationSettings, 
+//   updateOrganizationSetting 
+// } from './settings-repository'; 
+// export { 
+//   createRecommendation, 
+//   getRecommendationsByOrganizationId, 
+//   updateRecommendationStatus 
+// } from './recommendation-repository'; 
+
+// export * from './schema-version-repository';

lib/repositories/user-repository.ts

···
     'UPDATE user_organizations SET role = ? WHERE user_id = ? AND organization_id = ?',
     [role, userId, organizationId]
   );
+}
+
+export async function findOrCreateUserByGitHubId(userData: {
+  id: string; // GitHub user ID
+  login: string; // GitHub login/username
+  email?: string | null;
+  avatar_url?: string | null;
+  name?: string | null; // GitHub display name
+}): Promise<User> {
+  const existingUser = await findUserById(userData.id);
+  if (existingUser) {
+    // Optionally update user details if they've changed
+    // For now, just return the existing user
+    return existingUser;
+  }
+
+  // User not found, create a new one
+  // The users table expects 'name', 'email', 'image'
+  // We'll use github login for name if display name is not available
+  // and github avatar_url for image
+  return createUser({
+    id: userData.id,
+    name: userData.name || userData.login,
+    email: userData.email || null,
+    image: userData.avatar_url || null,
+  });
 } 

lib/services/github-service.ts

···
   setRepositoryTracking,
   findRepositoryById,
   findRepositoryByGitHubId,
-  addUserToOrganization
+  addUserToOrganization,
+  findOrCreateUserByGitHubId
 } from '@/lib/repositories';
 import { GitHubRepository, GitHubPullRequest, GitHubOrganization, GitHubUser, PRReview } from '@/lib/types';
 
···
           ? 'merged' 
           : pr.state === 'closed' ? 'closed' : 'open';
         
+        // Ensure author exists in our database
+        const prAuthor = pr.user ? await findOrCreateUserByGitHubId({
+          id: pr.user.id.toString(),
+          login: pr.user.login,
+          avatar_url: pr.user.avatar_url,
+          name: pr.user.name // pr.user might not have 'name', adjust if necessary based on GitHub API response
+        }) : null;
+
         if (existingPR) {
           // Update existing PR
           await updatePullRequest(existingPR.id, {
···
             closed_at: pr.closed_at,
             merged_at: pr.merged_at,
             draft: pr.draft,
+            // author_id is not typically updated, but if it could change or be initially null, handle here
           });
         } else {
           // Create new PR
+          if (!prAuthor) {
+            console.warn(`Skipping PR #${pr.number} for repo ${owner}/${repo} due to missing author information from GitHub.`);
+            return; // Skip this PR if author couldn't be processed
+          }
           const newPR = await createPullRequest({
             github_id: pr.id,
             repository_id: repositoryId,
             number: pr.number,
             title: pr.title,
             description: pr.body || null,
-            author_id: pr.user.id.toString(), // This might require creating user records
+            author_id: prAuthor.id, // Use the ID from our users table
             state,
             created_at: pr.created_at,
             updated_at: pr.updated_at,
···
         const existingReview = await findReviewByGitHubId(review.id);
         
         if (!existingReview) {
+          // Ensure reviewer exists in our database
+          const reviewAuthor = await findOrCreateUserByGitHubId({
+            id: review.user.id.toString(),
+            login: review.user.login,
+            avatar_url: review.user.avatar_url,
+            name: review.user.name // review.user might not have 'name', adjust if necessary
+          });
+
+          if (!reviewAuthor) {
+            console.warn(`Skipping review for PR #${prNumber} in ${owner}/${repo} by ${review.user.login} due to missing author information.`);
+            return; // Skip this review if author couldn't be processed
+          }
+
           // Map GitHub review state to our enum
           const reviewState = this.mapReviewState(review.state);
           
           await createPullRequestReview({
             github_id: review.id,
             pull_request_id: pullRequestId,
-            reviewer_id: review.user.id.toString(), // This might require creating user records
+            reviewer_id: reviewAuthor.id, // Use the ID from our users table
             state: reviewState,
             submitted_at: review.submitted_at
           });