fix: strip redundant speaker labels, add timestamps, protect delete_env

- Strip "**Parent (Adam):**" etc. from message text (already labeled)
- Add HH:MM timestamps to chatlog entries, display in frontend
- delete_env now requires actual variable/secret collision before deleting
(prevents agents from blindly deleting legitimate variables)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

frontend/electron-standalone.html

@@ -468,6 +468,7 @@
         }
         #chatlog-content .chat-msg .chat-speaker.adam { color: #f87171; }
         #chatlog-content .chat-msg .chat-speaker.eve { color: #a78bfa; }
+        #chatlog-content .chat-msg .chat-time { color: #6b7280; font-size: 0.85em; }
         #game-container {
             position: relative;
             border: 0;
@@ -5647,7 +5648,8 @@ function toggleBrokerPanel() {
             el.innerHTML = chatlogCache.slice().reverse().map(m => {
                 const cls = (m.speaker || '').toLowerCase();
                 const text = chatLang === 'zh' ? (m.text_zh || m.text || '') : (m.text || '');
-                return `<div class="chat-msg"><span class="chat-speaker ${cls}">${m.speaker}:</span> ${text}</div>`;
+                const timeStr = m.time ? `<span class="chat-time">${m.time}</span> ` : '';
+                return `<div class="chat-msg">${timeStr}<span class="chat-speaker ${cls}">${m.speaker}:</span> ${text}</div>`;
             }).join('');
             el.scrollTop = 0;
         }

scripts/conversation-loop.py

@@ -237,10 +237,20 @@ def action_restart():
 
 
 def action_delete_env(key):
-    """Delete an environment variable from the child's Space (fixes CONFIG_ERROR collisions)."""
+    """Delete an environment variable — ONLY if it collides with a secret (safety check)."""
     try:
+        # Safety: only allow deleting variables that collide with secrets
+        vars_dict = hf_api.get_space_variables(CHILD_SPACE_ID)
+        if key not in (vars_dict or {}):
+            return f"BLOCKED: Variable '{key}' does not exist. Nothing to delete."
+        info = hf_api.space_info(CHILD_SPACE_ID)
+        secret_names = set()
+        if hasattr(info, 'runtime') and info.runtime and hasattr(info.runtime, 'secrets'):
+            secret_names = set(info.runtime.secrets or [])
+        if key not in secret_names:
+            return f"BLOCKED: Variable '{key}' does NOT collide with a secret. Refusing to delete a non-colliding variable."
         hf_api.delete_space_variable(CHILD_SPACE_ID, key)
-        return f"Deleted variable {key} from {CHILD_NAME}'s Space. Use [ACTION: restart] to apply."
+        return f"Deleted colliding variable '{key}' from {CHILD_NAME}'s Space. Use [ACTION: restart] to apply."
     except Exception as e:
         return f"Error deleting variable {key}: {e}"
 
@@ -504,6 +514,15 @@ def call_llm(system_prompt, user_prompt):
 def _has_chinese(s):
     return bool(re.search(r'[\u4e00-\u9fff]', s))
 
+def _strip_speaker_labels(text):
+    """Remove redundant speaker self-references like **Parent (Adam):** or **Eve:** etc."""
+    # Patterns: **Parent (Adam):**, **Adam:**, **父亲 (Adam):**, **Eve:**, **母亲:**, etc.
+    text = re.sub(r'\*\*(?:Parent|Father|Mother|Dad|Mom|父亲|母亲|父级|亲爱的|伴侣)?\s*\(?(?:Adam|Eve|亚当|夏娃)?\)?\s*[:：]\*\*\s*', '', text)
+    # Also: "Adam:" or "Eve:" at the very start of text
+    text = re.sub(r'^(?:Adam|Eve|亚当|夏娃)\s*[:：]\s*', '', text.strip())
+    return text.strip()
+
+
 def parse_bilingual(text):
     """Parse bilingual response into (en, zh)."""
     display = re.sub(r'\[TASK\].*?\[/TASK\]', '', text, flags=re.DOTALL)
@@ -831,6 +850,7 @@ if reply:
     clean, actions = parse_and_execute_turn(reply, ctx)
     last_action_results = actions
     en, zh = parse_bilingual(clean)
+    en, zh = _strip_speaker_labels(en), _strip_speaker_labels(zh)
     print(f"[Adam/EN] {en}")
     if zh != en:
         print(f"[Adam/ZH] {zh}")
@@ -839,7 +859,9 @@ if reply:
         if ar['action'] == 'claude_code':
             result_preview = ar['result'][:800].replace('\n', '\n  ')
             print(f"  [CC-RESULT] {result_preview}")
-    entry = {"speaker": "Adam", "text": en, "text_zh": zh}
+    import datetime
+    ts = datetime.datetime.utcnow().strftime("%H:%M")
+    entry = {"speaker": "Adam", "time": ts, "text": en, "text_zh": zh}
     if actions:
         labels = " ".join(f"🔧{ar['action'].split(':')[0]}" for ar in actions)
         entry["text"] = f"{en} {labels}"
@@ -888,6 +910,7 @@ def do_turn(speaker, other, space_url):
     last_action_results = action_results
 
     en, zh = parse_bilingual(clean_text)
+    en, zh = _strip_speaker_labels(en), _strip_speaker_labels(zh)
     print(f"[{speaker}/EN] {en}")
     if zh != en:
         print(f"[{speaker}/ZH] {zh}")
@@ -902,12 +925,16 @@ def do_turn(speaker, other, space_url):
     else:
         print(f"[{speaker}] Turn #{turn_count}: no task assigned ({elapsed:.1f}s)")
 
-    # Add to history
+    # Add to history with timestamp
+    import datetime
+    ts = datetime.datetime.utcnow().strftime("%H:%M")
+    entry = {"speaker": speaker, "time": ts}
     if action_results:
         labels = " ".join(f"🔧{ar['action'].split(':')[0]}" for ar in action_results)
-        history.append({"speaker": speaker, "text": f"{en} {labels}", "text_zh": f"{zh} {labels}"})
+        entry.update({"text": f"{en} {labels}", "text_zh": f"{zh} {labels}"})
     else:
-        history.append({"speaker": speaker, "text": en, "text_zh": zh})
+        entry.update({"text": en, "text_zh": zh})
+    history.append(entry)
 
     set_bubble(space_url, en, zh)
     post_chatlog(history)