From db3ba306c8b9a515ee8b389c1b63c9020f144886 Mon Sep 17 00:00:00 2001 From: Nicholai Date: Sat, 3 Jan 2026 20:09:31 -0700 Subject: [PATCH] Remove debugging logs from Hubert chat component and API endpoints - Stripped console.log statements from client and server Hubert The Eunuch --- src/components/HubertChat.tsx | 32 ----------------------------- src/pages/api/hubert/chat.ts | 5 ----- src/pages/api/hubert/new-visitor.ts | 15 +++----------- 3 files changed, 3 insertions(+), 49 deletions(-) diff --git a/src/components/HubertChat.tsx b/src/components/HubertChat.tsx index 58d9328..2c16232 100644 --- a/src/components/HubertChat.tsx +++ b/src/components/HubertChat.tsx @@ -38,24 +38,19 @@ export default function HubertChat() { useEffect(() => { const initVisitor = async () => { try { - console.log('[Hubert] Starting initialization...'); setIsInitializing(true); setInitError(null); - console.log('[Hubert] Fetching /api/hubert/new-visitor...'); const response = await fetchWithTimeout('/api/hubert/new-visitor', { method: 'POST', headers: { 'Content-Type': 'application/json' } }, 8000); // 8 second timeout - console.log('[Hubert] Response status:', response.status); - if (!response.ok) { throw new Error(`HTTP ${response.status}: ${response.statusText}`); } const data = await response.json(); - console.log('[Hubert] Initialization successful:', data); setVisitorId(data.visitor_id); setConversationId(data.conversation_id); @@ -74,10 +69,8 @@ export default function HubertChat() { if (error instanceof Error) { if (error.name === 'AbortError') { errorMessage = '/// ERROR: TIMEOUT - API_UNRESPONSIVE'; - console.error('[Hubert] Request timed out after 8 seconds'); } else if (error.message.includes('Failed to fetch')) { errorMessage = '/// ERROR: NETWORK_FAILURE - CHECK_API_ROUTE'; - console.error('[Hubert] Network error - API route may not exist'); } else { errorMessage = `/// ERROR: ${error.message}`; } @@ -90,7 +83,6 @@ export default function HubertChat() { timestamp: new Date().toISOString(), }]); } finally { - console.log('[Hubert] Initialization complete, setting isInitializing to false'); setIsInitializing(false); } }; @@ -99,7 +91,6 @@ export default function HubertChat() { // Retry initialization const retryInit = () => { - console.log('[Hubert] Retrying initialization...'); setIsInitializing(true); setInitError(null); setMessages([]); @@ -107,24 +98,19 @@ export default function HubertChat() { // Re-trigger initialization const initVisitor = async () => { try { - console.log('[Hubert] Starting initialization...'); setIsInitializing(true); setInitError(null); - console.log('[Hubert] Fetching /api/hubert/new-visitor...'); const response = await fetchWithTimeout('/api/hubert/new-visitor', { method: 'POST', headers: { 'Content-Type': 'application/json' } }, 8000); - console.log('[Hubert] Response status:', response.status); - if (!response.ok) { throw new Error(`HTTP ${response.status}: ${response.statusText}`); } const data = await response.json(); - console.log('[Hubert] Initialization successful:', data); setVisitorId(data.visitor_id); setConversationId(data.conversation_id); @@ -142,10 +128,8 @@ export default function HubertChat() { if (error instanceof Error) { if (error.name === 'AbortError') { errorMessage = '/// ERROR: TIMEOUT - API_UNRESPONSIVE'; - console.error('[Hubert] Request timed out after 8 seconds'); } else if (error.message.includes('Failed to fetch')) { errorMessage = '/// ERROR: NETWORK_FAILURE - CHECK_API_ROUTE'; - console.error('[Hubert] Network error - API route may not exist'); } else { errorMessage = `/// ERROR: ${error.message}`; } @@ -158,7 +142,6 @@ export default function HubertChat() { timestamp: new Date().toISOString(), }]); } finally { - console.log('[Hubert] Initialization complete, setting isInitializing to false'); setIsInitializing(false); } }; @@ -176,19 +159,9 @@ export default function HubertChat() { }, [messages]); const sendMessage = async () => { - console.log('[Hubert] sendMessage called', { input, isTyping, visitorId, conversationId }); - if (!input.trim() || isTyping || !visitorId || !conversationId) { - console.log('[Hubert] sendMessage blocked:', { - noInput: !input.trim(), - isTyping, - noVisitorId: !visitorId, - noConversationId: !conversationId - }); return; } - - console.log('[Hubert] Sending message:', input); const userMessage: Message = { role: 'user', content: input, @@ -200,7 +173,6 @@ export default function HubertChat() { setIsTyping(true); try { - console.log('[Hubert] Fetching /api/hubert/chat...'); const response = await fetch('/api/hubert/chat', { method: 'POST', headers: { 'Content-Type': 'application/json' }, @@ -214,14 +186,11 @@ export default function HubertChat() { }), }); - console.log('[Hubert] Response status:', response.status); - if (!response.ok) { throw new Error(`HTTP ${response.status}: ${response.statusText}`); } const data = await response.json(); - console.log('[Hubert] Response data:', data); if (data.error) { throw new Error(data.error); @@ -233,7 +202,6 @@ export default function HubertChat() { timestamp: new Date().toISOString(), }; - console.log('[Hubert] Adding assistant message:', assistantMessage.content); setMessages(prev => [...prev, assistantMessage]); } catch (error) { console.error('[Hubert] Chat error:', error); diff --git a/src/pages/api/hubert/chat.ts b/src/pages/api/hubert/chat.ts index 3c6993d..acec8b4 100644 --- a/src/pages/api/hubert/chat.ts +++ b/src/pages/api/hubert/chat.ts @@ -75,11 +75,6 @@ export const POST = async (context) => { // In Astro with Cloudflare adapter, env is at locals.runtime.env const env = locals?.runtime?.env; - console.log('[Hubert API] Chat endpoint called'); - console.log('[Hubert API] env object:', env); - console.log('[Hubert API] env keys:', env ? Object.keys(env) : 'no env'); - console.log('[Hubert API] OPENROUTER_API_KEY present:', !!(env?.OPENROUTER_API_KEY)); - const { messages, conversation_id, visitor_id } = await request.json(); if (!messages || !conversation_id || !visitor_id) { diff --git a/src/pages/api/hubert/new-visitor.ts b/src/pages/api/hubert/new-visitor.ts index 1bc93e2..c7cd272 100644 --- a/src/pages/api/hubert/new-visitor.ts +++ b/src/pages/api/hubert/new-visitor.ts @@ -9,41 +9,32 @@ export const prerender = false; * Used when Hubert interface first loads */ export const POST = async (context) => { - console.log('[Hubert API] /api/hubert/new-visitor endpoint called'); - try { const { request, locals } = context; // In Astro with Cloudflare adapter, env is at locals.runtime.env const env = locals?.runtime?.env; - console.log('[Hubert API] Request received, env binding available:', !!env); - console.log('[Hubert API] HUBERT_DB binding available:', !!(env && env.HUBERT_DB)); - const userAgent = request.headers.get('user-agent') || 'unknown'; const ip = request.headers.get('cf-connecting-ip') || 'unknown'; const visitorId = randomUUID(); - console.log('[Hubert API] Generated visitor ID:', visitorId); // Only insert into database if HUBERT_DB binding exists (production) // In dev mode, this allows the chatbot to work without Cloudflare bindings if (env && env.HUBERT_DB) { - console.log('[Hubert API] Attempting database insert...'); try { await env.HUBERT_DB.prepare(` INSERT INTO visitors (visitor_id, first_seen_at, last_seen_at, ip_address, user_agent) VALUES (?, datetime('now'), datetime('now'), ?, ?) `).bind(visitorId, ip, userAgent).run(); - console.log(`[Hubert API] Database insert successful for visitor: ${visitorId}`); + console.log(`[Hubert] New visitor initialized: ${visitorId}`); } catch (dbError) { - console.error('[Hubert API] Database insert failed (continuing anyway):', dbError); + console.error('[Hubert] Database insert failed (continuing anyway):', dbError); // Continue anyway - don't fail initialization if DB is misconfigured } } else { - console.log(`[Hubert API] Dev mode: Skipping database insert for visitor: ${visitorId}`); + console.log(`[Hubert] Dev mode: Skipping database insert for visitor: ${visitorId}`); } - - console.log('[Hubert API] Returning success response'); return Response.json({ visitor_id: visitorId, conversation_id: visitorId, // Use visitor_id as initial conversation_id