Add debug logging to trace chat history auto-save and userType flow

Author: thisIsAnVariableOfACoder

backend/routes/chat.js

@@ -100,6 +100,12 @@ router.post('/message', optionalAuth, async (req, res) => {
     const userId = authUser?.user_id || null;
     const effectiveUserType = authUser?.user_type || user_type || null;
 
+    // DEBUG: Log userType flow
+    console.log('[CHAT DEBUG] /message - authUser.user_type:', authUser?.user_type);
+    console.log('[CHAT DEBUG] /message - req.body.user_type:', user_type);
+    console.log('[CHAT DEBUG] /message - effectiveUserType:', effectiveUserType);
+    console.log('[CHAT DEBUG] /message - userId:', userId);
+
     const convId = conversation_id || `conv_${Date.now()}_${Math.floor(Math.random() * 1000)}`;
 
     if (userId) {
@@ -251,9 +257,13 @@ router.post('/message', optionalAuth, async (req, res) => {
 
 router.get('/history/:userId', requireAuth, async (req, res) => {
   const { userId } = req.params;
+  console.log('[CHAT DEBUG] /history/:userId - userId:', userId, 'req.user.user_id:', req.user.user_id);
+  
   if (req.user.user_type !== 'admin' && String(req.user.user_id) !== String(userId)) {
+    console.log('[CHAT DEBUG] /history - Forbidden: user_id mismatch');
     return res.status(403).json({ success: false, error: 'Forbidden' });
   }
+  
   const query = `
     SELECT m.conversation_id,
            COALESCE(c.title, m.conversation_id) as title,
@@ -266,8 +276,13 @@ router.get('/history/:userId', requireAuth, async (req, res) => {
     GROUP BY m.conversation_id
     ORDER BY updated_at DESC
   `;
+  
   global.db.all(query, [userId], (err, rows) => {
-    if (err) return res.status(500).json({ success: false, error: err.message });
+    if (err) {
+      console.log('[CHAT DEBUG] /history ERROR:', err);
+      return res.status(500).json({ success: false, error: err.message });
+    }
+    console.log('[CHAT DEBUG] /history SUCCESS - rows:', rows?.length || 0, 'data:', rows);
     res.json({ success: true, data: rows });
   });
 });
@@ -415,7 +430,10 @@ router.delete('/conversation/:conversationId', requireAuth, (req, res) => {
 });
 
 async function saveMessage(convId, userId, sender, message, nodeId) {
+  console.log('[CHAT DEBUG] saveMessage called - convId:', convId, 'userId:', userId, 'sender:', sender, 'nodeId:', nodeId);
+  
   if (!global.db) {
+    console.log('[CHAT DEBUG] saveMessage - No database, using memory');
     MEMORY_MESSAGES.push({
       conversation_id: convId,
       user_id: userId,
@@ -431,8 +449,13 @@ async function saveMessage(convId, userId, sender, message, nodeId) {
     const query = `INSERT INTO chat_messages (conversation_id, user_id, sender, message, node_id)
                    VALUES (?, ?, ?, ?, ?)`;
     global.db.run(query, [convId, userId, sender, message, nodeId], function (err) {
-      if (err) reject(err);
-      else resolve(this.lastID);
+      if (err) {
+        console.log('[CHAT DEBUG] saveMessage ERROR:', err);
+        reject(err);
+      } else {
+        console.log('[CHAT DEBUG] saveMessage SUCCESS - messageId:', this.lastID);
+        resolve(this.lastID);
+      }
     });
   });
 }

backend/services/llmScorer.js

@@ -34,6 +34,10 @@ async function generateCareerQuestion({ userType, profile, memoryAnswers, intent
     ? memoryAnswers.slice(-8).map((a) => ({ q: a?.question || a?.q, a: a?.answer || a?.a })).filter((x) => x.q || x.a)
     : [];
 
+  console.log('[LLM DEBUG] generateCareerQuestion - userType:', safeUserType);
+  console.log('[LLM DEBUG] generateCareerQuestion - profile:', profileText);
+  console.log('[LLM DEBUG] generateCareerQuestion - memory:', JSON.stringify(memoryText));
+
   const systemPrompt = `Bạn là chuyên gia tư vấn hướng nghiệp.
 Nhiệm vụ: tạo 1 câu hỏi TIẾP THEO phù hợp với nhóm người dùng và bối cảnh hiện tại.
 
@@ -71,6 +75,11 @@ profile: ${profileText}
 memory: ${JSON.stringify(memoryText)}
 intent: ${JSON.stringify(intent || {})}
 
+TRỌNG: Hãy kiểm tra kỹ userType và chỉ tạo câu hỏi phù hợp với nhóm đó.
+Nếu userType là "professional", tuyệt đối KHÔNG hỏi về môn học hay hoạt động học đường.
+Nếu userType là "high_school", tuyệt đối KHÔNG hỏi về kinh nghiệm làm việc hay công ty.
+Nếu userType là "university", tuyệt đối KHÔNG hỏi về kinh nghiệm làm việc dài hạn hay quản lý nhân sự.
+
 Hãy tạo câu hỏi tiếp theo bằng tiếng Việt, xưng hô lịch sự, ngắn gọn.`;
 
   const payload = {

frontend/src/pages/Chat.jsx

@@ -126,8 +126,13 @@ export default function Chat() {
   }, [userId, token]);
 
   async function refreshHistory() {
-    if (!IS_OFFLINE && (!userId || !token)) return;
+    console.log('[FRONTEND DEBUG] refreshHistory called - userId:', userId, 'token:', !!token);
+    if (!IS_OFFLINE && (!userId || !token)) {
+      console.log('[FRONTEND DEBUG] refreshHistory skipped - no userId or token');
+      return;
+    }
     const jsonHistory = await api.getHistory(userId, token);
+    console.log('[FRONTEND DEBUG] refreshHistory response:', jsonHistory);
     setHistory(jsonHistory?.data || []);
   }
 
@@ -176,8 +181,10 @@ export default function Chat() {
         request_more: options.requestMore || false,
         user_id: userId || null
       };
+      console.log('[FRONTEND DEBUG] sendMessage - userType:', userType, 'userId:', userId, 'conversationId:', conversationId);
       const json = await api.sendMessage(body, token);
       const data = json?.data || {};
+      console.log('[FRONTEND DEBUG] sendMessage - response data:', data);
 
       const botMessage = { id: `${Date.now()}-b`, sender: 'bot', text: data.bot_reply };
       const applyBotMessage = () => {
@@ -188,9 +195,11 @@ export default function Chat() {
       if (data.conversation_id) {
         const prevId = conversationId;
         setConversationId(data.conversation_id);
-        // Refresh history when conversation changes OR when recommendations are saved
-        if (token && userId && (data.conversation_id !== prevId || data.completed)) {
+        // Always refresh history after every message for logged-in users
+        if (token && userId) {
+          console.log('[FRONTEND DEBUG] About to refreshHistory after message');
           await refreshHistory();
+          console.log('[FRONTEND DEBUG] refreshHistory completed');
         }
       }
       setCurrentNode(data.next_node || null);