fixing bugs - header and message exceeding character limit

Config_files/message_system_config.json

@@ -19,8 +19,8 @@
   ],
   "AI_Jargon": ["elevate", "enhance", "ignite", "reignite", "rekindle", "rediscover","passion", "boost", "fuel", "thrill", "revive", "spark", "performing", "fresh", "tone", "enthusiasm", "illuminate"],
   "AI_phrases_singeo": ["your voice deserves more"],
-  "header_limit": 35,
-  "message_limit": 115,
+  "header_limit": 30,
+  "message_limit": 135,
   "LLM_models": ["gpt-4o-mini", "gpt-4o", "gpt-4.1-mini", "gemini-2.5-flash-lite-preview-06-17", "claude-3-5-haiku-latest", "google/gemma-3-27b-instruct/bf-16"],
   "openai_models": ["gpt-4o-mini", "gpt-4o", "gpt-4.1-nano", "gpt-3.5-turbo", "gpt-4.1-mini"],
   "reasoning": ["o1", "o4-mini", "o1-mini", "o3-mini"],

Messaging_system/LLM.py

@@ -152,7 +152,7 @@ class LLM:
                 print(e.response)
 
         print("Max retries exceeded. Returning empty response.")
-        return [], {}
+        return None
 
     # =========================================================================
     def get_message_google(self, prompt, instructions, max_retries=4):
@@ -182,6 +182,10 @@ class LLM:
                             output["message"].strip()) > self.Core.config_file["message_limit"]:
                         print(
                             f"'header' or 'message' is more than specified characters in response on attempt {attempt + 1}. Retrying...")
+                        headchar= len(output["header"].strip())
+                        messagechar = len(output["message"].strip())
+                        print(
+                            f"'header' has {headchar} chars and 'message' has {messagechar} ...")
                         continue
                 return output
 
@@ -191,7 +195,7 @@ class LLM:
                 print(f"Error in attempt {attempt}: {e}")
 
         print("Max retries exceeded. Returning empty response.")
-        return [], {}
+        return None
 
     # =========================================================================
 
@@ -273,7 +277,7 @@ class LLM:
                 print(e.response)
 
         print("Max retries exceeded. Returning empty response.")
-        return [], {}
+        return None
 
     # ======================================================================
 
@@ -331,7 +335,7 @@ class LLM:
                 print("Error processing output:", parse_error)
 
         print("Max retries exceeded. Returning empty response.")
-        return [], {}
+        return None
 
     def get_message_claude(self, prompt, instructions, max_retries=4):
         """
@@ -387,7 +391,7 @@ class LLM:
                 print("Error processing output:", parse_error)
 
         print("Max retries exceeded. Returning empty response.")
-        return [], {}
+        return None
 
     # ======================================================================
 

Messaging_system/Message_generator.py

@@ -186,7 +186,7 @@ Common instructor phrases:
 
 ABSOLUTE RULE – OVERRIDES EVERYTHING ELSE:  
 the header and the message **MUST NOT** contain any banned word or phrases(case-insensitive; singular, plural, verb forms, or their derivatives)
-- **important Note:** header must be less than {self.Core.config_file["header_limit"]} and message must be less than {self.Core.config_file["message_limit"]} characters.
+- **important Note:** header **must be** less than {self.Core.config_file["header_limit"]} characters and message **must be less** than {self.Core.config_file["message_limit"]} characters.
 
 
 Banned word:

Messaging_system/MultiMessage.py

@@ -34,7 +34,7 @@ class MultiMessage:
         Subsequent messages are generated by referencing all previously generated ones.
         """
         first_message_str = user.get("message", None)
-        if not first_message_str:
+        if first_message_str is None:
             print("No initial message found; cannot build a multi-message sequence.")
             return None
 
@@ -79,7 +79,7 @@ class MultiMessage:
 
             # ---- (B) actually generate the next message; hand it the UPDATED user ----
             next_msg_raw = self.generate_next_messages(message_sequence, msg_number, user)
-            if not next_msg_raw:
+            if next_msg_raw is None:
                 print(f"Could not generate the message for step {msg_number}. Stopping.")
                 break
 
@@ -167,7 +167,7 @@ Mimic the example style as much as possible and make it personalized using provi
         for i, m in enumerate(recent_messages, start=1):
             header = m.get("header", "").strip()
             body   = m.get("message", "").strip()
-            previous_text.append(f"Message {i}: (Header) {header}\n           (Body) {body}")
+            previous_text.append(f"Message {i}:\n header: {header}\n           message: {body}")
 
         # Combine into a single string
         previous_text_str = "\n\n".join(previous_text)
@@ -373,7 +373,7 @@ Common instructor phrases:
 
 ABSOLUTE RULE – OVERRIDES EVERYTHING ELSE:  
 the header and the message **MUST NOT** contain any banned word or phrases(case-insensitive; singular, plural, verb forms, or their derivatives)
-- **important Note:** header must be less than {self.Core.config_file["header_limit"]} and message must be less than {self.Core.config_file["message_limit"]} characters.
+- **important Note:** header **must be** less than {self.Core.config_file["header_limit"]} characters and message **must be less** than {self.Core.config_file["message_limit"]} characters.
 
 Banned word:
 {jargon_list}

Messaging_system/PromptGenerator.py

@@ -51,7 +51,7 @@ Use these info only to *flavour* the header and message if they add value. Never
 - {self.safe_get(self.Core.segment_info)}
 - first name: {self.safe_get(user.get("first_name"))} --> Only use if is available and the first name is a **valid name**
 - Experience level (0-9):  {self.safe_get(user.get("difficulty"))}
-- User profile --> use **indirectly** if it can improve personalization:
+- User profile --> use **indirectly** if it can improve personalization and **do not** use their preferences keywords ("genre, styles") directly in the message:
 {self.safe_get(user.get("user_info"))}
 """
             return user_info
@@ -197,8 +197,9 @@ Goal: Make the recommendation feel personalized and casually relevant — not ge
 
         general_instructions = f"""
 - Ensure that the output is a valid JSON following above structure.
-- header < {self.Core.config_file["header_limit"]} and message < {self.Core.config_file["message_limit"]} characters.
 - Preserve special characters and emojis in the message, you are **ONLY ALLOWED**allowed to use {self.Core.get_emoji()} emoji if needed, ONLY ONCE, and ONLY at the end of header or message).
+- **important Note:** header **must be** less than {self.Core.config_file["header_limit"]} characters and message **must be less** than {self.Core.config_file["message_limit"]} characters.
+
         """
 
         instructions = f"""

messaging_main_test.py

@@ -129,7 +129,7 @@ if __name__ == "__main__":
     recsys_contents = ["workout", "course", "quick_tips"]
 
     # number_of_samples = users.shape[0]
-    number_of_samples = 10
+    number_of_samples = 20
 
     # number of messages to generate
     number_of_messages = 3