Nicholai/nicholai-work-2026
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove debugging logs from Hubert chat component and API endpoints

Browse Source 
- Stripped console.log statements from client and server

Hubert The Eunuch
This commit is contained in:
Nicholai Vogel 2026-01-03 20:09:31 -07:00
parent b49b5d1df4
commit db3ba306c8
3 changed files with 3 additions and 49 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
src/components/HubertChat.tsx
View File

@ -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);

5
src/pages/api/hubert/chat.ts
View File

@ -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) {

15
src/pages/api/hubert/new-visitor.ts
View File

@ -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