From f11a7fef36df949a0ce6bf382589f53118a9dbba Mon Sep 17 00:00:00 2001
From: Randall Stillwell <me@randallstillwell.com>
Date: Sun, 27 Jul 2025 12:41:14 -0500
Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=B9=20Remove=20Debug=20Logging=20-=20A?=
 =?UTF-8?q?uthentication=20Issue=20Fixed?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

✅ Ownership Indicators Now Working:
- Bob's collections properly show userRole: 'owner'
- Alice's public collections show userRole: null
- Authentication headers fix resolved the issue

🧹 Cleanup:
- Removed debug console.log statements
- Cleaned up server-side logging
- Restored clean, production-ready code

The authentication issue is fully resolved! Bob now sees proper ownership
indicators (👑 Owner badges) on his collections while Alice's public
collections show as viewable without ownership indicators.
---
 pages/api/collections.js                         | 15 ---------------
 pages/api/collections/[identifier]/thumbnails.js |  4 ----
 pages/collections.js                             | 15 ---------------
 3 files changed, 34 deletions(-)

diff --git a/pages/api/collections.js b/pages/api/collections.js
index 85a73b6..c062e11 100644
--- a/pages/api/collections.js
+++ b/pages/api/collections.js
@@ -23,7 +23,6 @@ export default async function handler(req, res) {
       }
       
       const currentUserId = user.userId;
-      console.log('🔍 Debug - Collections API - Current user:', user);
 
       // Get collections based on ownership, collaboration, or public visibility
       const result = await sql`
@@ -51,14 +50,6 @@ export default async function handler(req, res) {
         ORDER BY c.updated_at DESC
       `;
 
-      console.log('🔍 Debug - Collections API - Query results:', result.rows.map(r => ({
-        name: r.name,
-        user_id: r.user_id,
-        creator_email: r.creator_email,
-        user_role: r.user_role,
-        effective_role: r.effective_role
-      })));
-
       const collections = result.rows.map(collection => ({
         id: collection.id,
         slug: collection.slug,
@@ -75,12 +66,6 @@ export default async function handler(req, res) {
         userRole: collection.effective_role
       }));
 
-      console.log('🔍 Debug - Collections API - Final response:', collections.map(c => ({
-        name: c.name,
-        userRole: c.userRole,
-        creator: c.creator
-      })));
-
       res.status(200).json(collections);
 
     } catch (error) {
diff --git a/pages/api/collections/[identifier]/thumbnails.js b/pages/api/collections/[identifier]/thumbnails.js
index 25e422a..a6d08f9 100644
--- a/pages/api/collections/[identifier]/thumbnails.js
+++ b/pages/api/collections/[identifier]/thumbnails.js
@@ -20,7 +20,6 @@ export default async function handler(req, res) {
 
   try {
     const { identifier } = req.query;
-    console.log('🔍 Debug - Thumbnails API - Identifier:', identifier);
 
     // Get authenticated user
     const user = await getUserFromRequest(req);
@@ -28,15 +27,12 @@ export default async function handler(req, res) {
       return res.status(401).json({ error: 'Authentication required' });
     }
 
-    console.log('🔍 Debug - Thumbnails API - User:', user);
-
     if (!identifier) {
       return res.status(400).json({ error: 'Collection identifier is required' });
     }
 
     // Determine if identifier is a slug or numeric ID
     const isSlug = isValidSlug(identifier) || isNaN(parseInt(identifier));
-    console.log('🔍 Debug - Thumbnails API - Is slug:', isSlug);
     
     // Verify user has access to this collection
     let collectionResult;
diff --git a/pages/collections.js b/pages/collections.js
index 4295420..7d170c7 100644
--- a/pages/collections.js
+++ b/pages/collections.js
@@ -40,16 +40,6 @@ export default function Collections() {
     }
   }, [user]);
 
-  // Debug logging
-  useEffect(() => {
-    console.log('🔍 Debug - Current user:', user);
-    console.log('🔍 Debug - Collections:', collections.map(c => ({
-      name: c.name,
-      userRole: c.userRole,
-      creator: c.creator
-    })));
-  }, [user, collections]);
-
   const fetchCollections = async () => {
     try {
       const token = localStorage.getItem('auth_token');
@@ -65,11 +55,6 @@ export default function Collections() {
       
       if (response.ok) {
         const data = await response.json();
-        console.log('🔍 Debug - API response:', data.map(c => ({
-          name: c.name,
-          userRole: c.userRole,
-          creator: c.creator
-        })));
         
         // Fetch thumbnails for each collection
         const collectionsWithThumbnails = await Promise.all(