修复新手破冰期问候重试循环

Author: Krabbypattyl

static/app-websocket.js

@@ -3160,9 +3160,6 @@
     }
     function _isTutorialBlockingGreeting() {
         if (!_isHomeTutorialPage()) return false;
-        if (isNewUserIcebreakerBlockingGreeting()) {
-            return true;
-        }
         try {
             if (isHomeTutorialLockedForGreeting()) {
                 return true;
@@ -3211,11 +3208,24 @@
         S._greetingCheckIsSwitch = !!isSwitch;
         S._greetingCheckReason = reason || '';
     }
+    function _consumeGreetingCheckForNewUserIcebreaker() {
+        if (!isNewUserIcebreakerBlockingGreeting()) return false;
+        sendHomeTutorialState('greeting-check-consumed-by-icebreaker');
+        S._greetingCheckPending = false;
+        S._greetingCheckIsSwitch = false;
+        S._greetingCheckReason = '';
+        _resetGreetingCheckRetry(true);
+        console.log('[greeting_check] consumed by new-user icebreaker period');
+        return true;
+    }
     function _sendGreetingCheckIfReady() {
         if (!S._greetingCheckPending || !S._modelReady) {
             if (!S._greetingCheckPending) _resetGreetingCheckRetry(true);
             return;
         }
+        if (_consumeGreetingCheckForNewUserIcebreaker()) {
+            return;
+        }
         if (_isGreetingCheckBlocked()) {
             sendHomeTutorialState('greeting-check-blocked');
             _scheduleGreetingCheckRetry();

tests/unit/test_app_websocket_static.py

@@ -52,6 +52,33 @@ def test_blocked_greeting_check_reports_home_tutorial_state_before_retry():
     assert "_scheduleGreetingCheckRetry();" in blocked_branch
 
 
+def test_icebreaker_greeting_check_is_consumed_without_retry_loop():
+    source = APP_WEBSOCKET_PATH.read_text(encoding="utf-8")
+
+    send_block = source.split("function _sendGreetingCheckIfReady()", 1)[1].split(
+        "function _onModelReady()",
+        1,
+    )[0]
+    assert send_block.index("if (_consumeGreetingCheckForNewUserIcebreaker())") < send_block.index(
+        "if (_isGreetingCheckBlocked())"
+    )
+
+    consume_block = source.split("function _consumeGreetingCheckForNewUserIcebreaker()", 1)[1].split(
+        "function _sendGreetingCheckIfReady()",
+        1,
+    )[0]
+    assert "sendHomeTutorialState('greeting-check-consumed-by-icebreaker')" in consume_block
+    assert "S._greetingCheckPending = false;" in consume_block
+    assert "_resetGreetingCheckRetry(true);" in consume_block
+    assert "_scheduleGreetingCheckRetry();" not in consume_block
+
+    tutorial_block = source.split("function _isTutorialBlockingGreeting()", 1)[1].split(
+        "function _isGreetingCheckBlocked()",
+        1,
+    )[0]
+    assert "isNewUserIcebreakerBlockingGreeting()" not in tutorial_block
+
+
 def test_goodbye_blocks_stale_audio_session_started():
     source = APP_WEBSOCKET_PATH.read_text(encoding="utf-8")