Reduce quest and travel UI delays

Author: web3blind

app/index.html

@@ -164,7 +164,7 @@
     <script src="js/engine/quest-system.js?v=20260621e"></script>
     <script src="js/engine/loci.js"></script>
     <script src="js/engine/npc-framework.js"></script>
-    <script src="js/engine/state-engine.js?v=20260621e"></script>
+    <script src="js/engine/state-engine.js?v=20260621h"></script>
     <script src="js/engine/daily-leaderboard-storage.js"></script>
     <script src="js/engine/daily-leaderboard.js"></script>
 
@@ -184,11 +184,11 @@
     <script src="js/ui/screens/character.js"></script>
     <script src="js/ui/screens/hunt.js"></script>
     <script src="js/ui/screens/inventory.js"></script>
-    <script src="js/ui/screens/chronicle.js?v=20260621g"></script>
+    <script src="js/ui/screens/chronicle.js?v=20260621h"></script>
     <script src="js/ui/screens/duel.js"></script>
     <script src="js/ui/screens/arena.js?v=20260621d"></script>
     <script src="js/ui/screens/guild.js?v=20260621d"></script>
-    <script src="js/ui/screens/map.js"></script>
+    <script src="js/ui/screens/map.js?v=20260621h"></script>
     <script src="js/ui/screens/marketplace.js"></script>
     <script src="js/ui/screens/crafting.js"></script>
     <script src="js/ui/screens/quests.js?v=20260621e"></script>

app/js/engine/state-engine.js

@@ -1212,6 +1212,20 @@ var StateEngine = (function() {
         return result;
     }
 
+    /**
+     * Process movement for live UI — same authoritative mutation path as replay.
+     * @param {string} account
+     * @param {string} zoneId
+     * @param {number} blockNum
+     * @returns {Object|null}
+     */
+    function processMoveResult(account, zoneId, blockNum) {
+        if (!zoneId) return null;
+        var events = _handleMove(account, { zone: zoneId }, blockNum || 0);
+        if (!events.length) return null;
+        return events[0];
+    }
+
     /**
      * Process Armageddon for live UI — same logic as _handleHuntArmageddon.
      * @param {string} account
@@ -1247,6 +1261,7 @@ var StateEngine = (function() {
         getCharacter: getCharacter,
         getInventory: getInventory,
         processHuntResult: processHuntResult,
+        processMoveResult: processMoveResult,
         processArmageddonResult: processArmageddonResult,
         reset: reset
     };

app/js/ui/screens/chronicle.js

@@ -346,6 +346,17 @@ var ChronicleScreen = (function() {
             events: []
         });
         cachedFeedHtml = {};
+        _updateLocalBlessingQuestProgress(account);
+    }
+
+    function _updateLocalBlessingQuestProgress(account) {
+        var state = StateEngine.getState();
+        var user = VizAccount.getCurrentUser();
+        if (!state || !user || !account || !state.quests || !state.quests[user] || typeof QuestSystem === 'undefined') return;
+        QuestSystem.updateQuestProgress(state.quests[user], 'social', { target: 'blessing', uniqueKey: account, count: 1 });
+        try {
+            CheckpointSystem.saveCheckpoint('global', state.headBlock || 0, state, function() {});
+        } catch (e) {}
     }
 
     function _renderEntry(entry) {

app/js/ui/screens/map.js

@@ -8,6 +8,7 @@ var MapScreen = (function() {
 
     var t = Helpers.t;
     var pendingTravel = null;
+    var PENDING_TRAVEL_TTL_MS = 5 * 60 * 1000;
     var TRAVEL_COST_LOW = 10;   // 0.1%
     var TRAVEL_COST_HIGH = 100; // 1%
 
@@ -35,6 +36,8 @@ var MapScreen = (function() {
         var confirmedZone = character ? character.currentZone : 'commons_first_light';
         if (pendingTravel && pendingTravel.account === user && pendingTravel.to === confirmedZone) {
             pendingTravel = null;
+        } else if (pendingTravel && pendingTravel.account === user && pendingTravel.at && (Date.now() - pendingTravel.at) > PENDING_TRAVEL_TTL_MS) {
+            pendingTravel = null;
         }
         var currentZone = confirmedZone;
         var myGuild = null;
@@ -158,7 +161,9 @@ var MapScreen = (function() {
             }
 
             // Travel button
-            if (!isCurrent && character) {
+            if (pendingTravel && pendingTravel.account === user && regionId === pendingTravel.to) {
+                html += '<div class="region-benefits" role="status">⏳ ' + t('map_pending_travel_short') + '</div>';
+            } else if (!isCurrent && character && !(pendingTravel && pendingTravel.account === user)) {
                 html += '<div class="region-travel-options">';
                 html += '<button class="btn btn-secondary btn-sm region-travel-btn" ';
                 html += 'data-region="' + regionId + '" data-cost="' + TRAVEL_COST_LOW + '" ';
@@ -171,8 +176,6 @@ var MapScreen = (function() {
                 html += '\uD83D\uDEB6 ' + Helpers.manaCost(TRAVEL_COST_HIGH);
                 html += '</button>';
                 html += '</div>';
-            } else if (pendingTravel && pendingTravel.account === user && regionId === pendingTravel.to) {
-                html += '<div class="region-benefits">⏳ ' + t('map_pending_travel_short') + '</div>';
             }
 
             html += '</section>';
@@ -226,14 +229,22 @@ var MapScreen = (function() {
             t: VizMagicConfig.ACTION_TYPES.MOVE,
             d: { zone: regionId }
         };
+        var previousZone = character ? character.currentZone : '';
 
         VizBroadcast.gameAction(moveAction, function(err) {
             if (err) {
                 Toast.error(t('error_network'));
             } else {
-                pendingTravel = { account: user, from: character ? character.currentZone : '', to: regionId, at: Date.now() };
+                var stateAfterMove = StateEngine.getState();
+                var optimisticBlock = (stateAfterMove.headBlock || 0) + 1;
+                StateEngine.processMoveResult(user, regionId, optimisticBlock);
+                stateAfterMove.headBlock = Math.max(stateAfterMove.headBlock || 0, optimisticBlock);
+                pendingTravel = { account: user, from: previousZone, to: regionId, at: Date.now() };
                 Toast.success(t('map_traveled') + ' ' + region.name);
                 SoundManager.play('transition');
+                try {
+                    CheckpointSystem.saveCheckpoint('global', stateAfterMove.headBlock || optimisticBlock, stateAfterMove, function() {});
+                } catch (e) {}
                 render();
             }
         });

app/sw.js

@@ -1,5 +1,5 @@
 // Viz Magic — Service Worker
-var CACHE_NAME = 'viz-magic-v20';
+var CACHE_NAME = 'viz-magic-v21';
 var ASSETS = [
     '/',
     '/index.html',

tests/player-bug-regressions.test.js

@@ -34,6 +34,7 @@ const inventoryJs = read('app/js/ui/screens/inventory.js');
 const huntJs = read('app/js/ui/screens/hunt.js');
 const mapJs = read('app/js/ui/screens/map.js');
 const chronicleJs = read('app/js/ui/screens/chronicle.js');
+const mapScreenJs = read('app/js/ui/screens/map.js');
 const questScreenJs = read('app/js/ui/screens/quests.js');
 const guildJs = read('app/js/ui/screens/guild.js');
 const leaderboardJs = read('app/js/ui/screens/leaderboard.js');
@@ -124,6 +125,7 @@ test('chronicle keeps loaded tabs visible and shows sent blessings immediately',
   assert.ok(/_injectLocalBlessing\(account, energy\)/.test(chronicleJs), 'blessing success path should inject visible local action');
   assert.ok(/receiver: action\.receiver/.test(chronicleJs), 'chronicle entries should preserve blessing receiver for dedupe');
   assert.ok(/\|blessing_sent\|/.test(chronicleJs), 'blessing dedupe should ignore optimistic/replay block mismatch');
+  assert.ok(/function _updateLocalBlessingQuestProgress/.test(chronicleJs), 'blessing quests should update immediately after a successful local blessing');
 });
 
 
@@ -187,12 +189,22 @@ test('high-traffic UI narration and inventory stat labels are translated', funct
 
 test('service worker updates quickly and keeps navigations network-first', function () {
   const swJs = read('app/sw.js');
-  assert.ok(/viz-magic-v20/.test(swJs), 'service worker cache version should be bumped');
+  assert.ok(/viz-magic-v21/.test(swJs), 'service worker cache version should be bumped');
   assert.ok(/self\.skipWaiting\(\)/.test(swJs), 'service worker should activate new cache without waiting for all tabs to close');
   assert.ok(/self\.clients\.claim\(\)/.test(swJs), 'service worker should claim clients after activation');
   assert.ok(/event\.request\.mode === 'navigate'[\s\S]*fetch\(event\.request\)/.test(swJs), 'navigation requests should prefer network to avoid stale cached index');
 });
 
+
+
+test('map travel updates through state-engine and blocks repeat-click loops', function () {
+  assert.ok(/function processMoveResult/.test(stateEngineJs), 'state engine should expose live movement path');
+  assert.ok(/processMoveResult: processMoveResult/.test(stateEngineJs), 'live movement path should be exported');
+  assert.ok(/StateEngine\.processMoveResult\(user, regionId, optimisticBlock\)/.test(mapScreenJs), 'map should update movement immediately through state engine');
+  assert.ok(/PENDING_TRAVEL_TTL_MS/.test(mapScreenJs), 'pending travel state should have a stale guard');
+  assert.ok(/!\(pendingTravel && pendingTravel\.account === user\)/.test(mapScreenJs), 'pending travel should suppress repeat travel buttons');
+});
+
 test('leaderboard has local character fallback while 24h scan is empty or slow', function () {
   assert.ok(/function _fallbackRowsFromState/.test(leaderboardJs), 'leaderboard should expose local fallback rows');
   assert.ok(/rows = _fallbackRowsFromState\(\)/.test(leaderboardJs), 'leaderboard should use fallback when snapshot rows are empty');