Fix duel opponent resolution and blessing UX

Author: web3blind

app/js/i18n/en.js

@@ -210,6 +210,8 @@ var LangEN = {
     chronicle_placeholder: 'Write your words into the World...',
     chronicle_send: 'Inscribe',
     chronicle_empty: 'Chronicle is empty. Be the first to write!',
+    chronicle_bless_posts_only_hint: 'Blessings are available only through Chronicle posts. There are no suitable posts from other mages in the feed yet.',
+    chronicle_no_bless_targets_hint: 'There are no other mages\' posts to bless right now. Blessings work only on other players\' Chronicle posts.',
     chronicle_narrative_hunt: '{name} set out on a hunt.',
     chronicle_narrative_hunt_win: '{name} defeated {creature}!',
     chronicle_narrative_hunt_lose: '{name} fell in battle with {creature}.',

app/js/i18n/ru.js

@@ -210,6 +210,8 @@ var LangRU = {
     chronicle_placeholder: 'Впиши слова в Мир...',
     chronicle_send: 'Вписать',
     chronicle_empty: 'Хроника пуста. Будь первым, кто впишет слова!',
+    chronicle_bless_posts_only_hint: 'Благословения доступны только через записи в Хронике. Пока в ленте нет подходящих постов других магов.',
+    chronicle_no_bless_targets_hint: 'Сейчас в ленте нет чужих записей для благословения. Благословлять можно только посты других магов в Хронике.',
     chronicle_narrative_hunt: '{name} отправился на охоту.',
     chronicle_narrative_hunt_win: '{name} одолел {creature}!',
     chronicle_narrative_hunt_lose: '{name} пал в бою с {creature}.',

app/js/ui/screens/chronicle.js

@@ -238,12 +238,22 @@ var ChronicleScreen = (function() {
             return (b.blockNum || 0) - (a.blockNum || 0);
         });
 
+        var user = VizAccount.getCurrentUser();
+        var blessableCount = 0;
+        for (var b = 0; b < entries.length; b++) {
+            if (entries[b].account && entries[b].account !== user) blessableCount++;
+        }
+
         if (entries.length === 0) {
-            feed.innerHTML = '<p class="empty-state">' + Helpers.t('chronicle_empty') + '</p>';
+            feed.innerHTML = '<p class="empty-state">' + Helpers.t('chronicle_empty') + '</p>' +
+                '<p class="chronicle-hint">' + Helpers.t('chronicle_bless_posts_only_hint') + '</p>';
             return;
         }
 
         var html = '';
+        if (blessableCount === 0) {
+            html += '<p class="chronicle-hint">' + Helpers.t('chronicle_no_bless_targets_hint') + '</p>';
+        }
         for (var k = 0; k < entries.length; k++) {
             html += _renderEntry(entries[k]);
         }
@@ -289,19 +299,27 @@ var ChronicleScreen = (function() {
         var icon = _getActionIcon(entry.actionType);
         var timeStr = entry.timestamp ? Helpers.timeAgo(entry.timestamp) : '';
 
+        var user = VizAccount.getCurrentUser();
+        var canBless = entry.account && user && entry.account !== user;
+        var actionsHtml = '';
+
+        if (canBless) {
+            actionsHtml = '<div class="chronicle-actions">' +
+                '<button class="bless-button" data-account="' + (entry.account || '') + '" ' +
+                    'aria-label="' + Helpers.escapeHtml(t('chronicle_bless_cost', { cost: Helpers.manaCost(BLESS_ENERGY), name: charName })) + '">' +
+                    '✨ ' + t('chronicle_bless') + ' · ' + Helpers.manaCost(BLESS_ENERGY) +
+                '</button>' +
+            '</div>';
+        }
+
         return '<article class="chronicle-entry" role="article" aria-label="' + Helpers.escapeHtml(charName) + '">' +
             '<div class="chronicle-entry-header">' +
                 '<span class="chronicle-icon" aria-hidden="true">' + icon + '</span>' +
                 '<strong class="chronicle-author">' + Helpers.escapeHtml(charName) + '</strong>' +
                 (timeStr ? '<span class="chronicle-time">' + timeStr + '</span>' : '') +
             '</div>' +
             '<p class="chronicle-text">' + Helpers.escapeHtml(entry.text) + '</p>' +
-            '<div class="chronicle-actions">' +
-                '<button class="bless-button" data-account="' + (entry.account || '') + '" ' +
-                    'aria-label="' + Helpers.escapeHtml(t('chronicle_bless_cost', { cost: Helpers.manaCost(BLESS_ENERGY), name: charName })) + '">' +
-                    '✨ ' + t('chronicle_bless') + ' · ' + Helpers.manaCost(BLESS_ENERGY) +
-                '</button>' +
-            '</div>' +
+            actionsHtml +
             '</article>';
     }
 

app/js/ui/screens/duel.js

@@ -10,6 +10,7 @@ var DuelScreen = (function() {
     var STATE_POLL_INTERVAL_MS = 3000;
     var registeredListeners = false;
     var _statePollTimer = null;
+    var _hydratingAccounts = {};
 
     var duelState = _createInitialState();
 
@@ -47,6 +48,7 @@ var DuelScreen = (function() {
         if (opts.combatRef) duelState.combatRef = String(opts.combatRef);
         if (opts.challengeData) duelState.challengeData = opts.challengeData;
 
+        _ensureCharacterHydrated(duelState.opponent);
         _syncFromState();
         _ensureEventListeners();
 
@@ -129,10 +131,14 @@ var DuelScreen = (function() {
     function _renderPreDuel(el) {
         var t = Helpers.t;
         var user = VizAccount.getCurrentUser();
-        var myChar = StateEngine.getCharacter(user) || {};
-        var oppChar = StateEngine.getCharacter(duelState.opponent) || {};
-        var myDuels = _getPlayerStats(user);
-        var oppDuels = _getPlayerStats(duelState.opponent);
+        var participants = _getResolvedParticipants();
+        var myAccount = participants.me || user;
+        var oppAccount = participants.opponent || duelState.opponent;
+        var myChar = StateEngine.getCharacter(myAccount) || {};
+        var oppChar = StateEngine.getCharacter(oppAccount) || {};
+        duelState.opponent = oppAccount || duelState.opponent;
+        var myDuels = _getPlayerStats(myAccount);
+        var oppDuels = _getPlayerStats(oppAccount);
         var pendingNotice = '';
         var beginLabel = duelState.combatRef ? t('duel_begin') : t('duel_begin_real');
 
@@ -149,14 +155,14 @@ var DuelScreen = (function() {
             '<div class="duel-matchup">' +
                 '<div class="duel-mage duel-mage-left">' +
                     '<span class="duel-mage-icon" aria-hidden="true">' + Helpers.classIcon(myChar.className) + '</span>' +
-                    '<strong>' + Helpers.escapeHtml(myChar.name || user || t('duel_you')) + '</strong>' +
+                    '<strong>' + Helpers.escapeHtml(myChar.name || myAccount || t('duel_you')) + '</strong>' +
                     '<span class="duel-mage-info">' + t('home_level') + ' ' + (myChar.level || 1) + '</span>' +
                     '<span class="duel-mage-info">' + t('duel_wins') + ': ' + myDuels.wins + '</span>' +
                 '</div>' +
                 '<span class="duel-vs" aria-hidden="true">VS</span>' +
                 '<div class="duel-mage duel-mage-right">' +
                     '<span class="duel-mage-icon" aria-hidden="true">' + Helpers.classIcon(oppChar.className) + '</span>' +
-                    '<strong>' + Helpers.escapeHtml(oppChar.name || duelState.opponent || '???') + '</strong>' +
+                    '<strong>' + Helpers.escapeHtml(oppChar.name || oppAccount || '???') + '</strong>' +
                     '<span class="duel-mage-info">' + t('home_level') + ' ' + (oppChar.level || 1) + '</span>' +
                     '<span class="duel-mage-info">' + t('duel_wins') + ': ' + oppDuels.wins + '</span>' +
                 '</div>' +
@@ -190,7 +196,7 @@ var DuelScreen = (function() {
 
         if (typeof BattleNarrator !== 'undefined') {
             BattleNarrator.announce(t('duel_narrator_pre', {
-                opponent: oppChar.name || duelState.opponent
+                opponent: oppChar.name || oppAccount
             }));
         }
     }
@@ -724,6 +730,9 @@ var DuelScreen = (function() {
         duelState.currentRound = duel.currentRound || duelState.currentRound;
 
         var user = VizAccount.getCurrentUser();
+        duelState.opponent = _resolveOpponentFromDuel(duel, user, duelState.opponent);
+        _ensureCharacterHydrated(duel.challenger);
+        _ensureCharacterHydrated(duel.target);
         duelState.opponentCommitted = _hasOpponentCommitted(duel, user, duelState.currentRound);
 
         if (duel.roundResults && duel.roundResults.length) {
@@ -756,6 +765,64 @@ var DuelScreen = (function() {
         }
     }
 
+    function _getResolvedParticipants() {
+        var user = VizAccount.getCurrentUser();
+        var duel = _getCurrentDuel();
+        return {
+            me: user,
+            opponent: _resolveOpponentFromDuel(duel, user, duelState.opponent)
+        };
+    }
+
+    function _resolveOpponentFromDuel(duel, user, fallbackOpponent) {
+        if (!duel || !user) return fallbackOpponent || '';
+        if (duel.challenger === user) return duel.target || fallbackOpponent || '';
+        if (duel.target === user) return duel.challenger || fallbackOpponent || '';
+        return fallbackOpponent || duel.target || duel.challenger || '';
+    }
+
+    function _ensureCharacterHydrated(account) {
+        if (!account || StateEngine.getCharacter(account) || _hydratingAccounts[account]) return;
+        if (typeof VizAccount === 'undefined' || typeof VizAccount.getAccount !== 'function') return;
+        if (typeof CharacterSystem === 'undefined' || typeof CharacterSystem.createCharacter !== 'function') return;
+
+        _hydratingAccounts[account] = true;
+        VizAccount.getAccount(account, function(err, accountData) {
+            delete _hydratingAccounts[account];
+            if (err || !accountData || StateEngine.getCharacter(account)) return;
+
+            var grimoire = VizAccount.parseGrimoire(accountData);
+            if (!grimoire || !grimoire.class) return;
+
+            var state = StateEngine.getState();
+            var character = CharacterSystem.createCharacter(account, grimoire.name || account, grimoire.class);
+            if (!character) return;
+
+            if (grimoire.level && grimoire.level > 1) {
+                character.level = grimoire.level;
+                character.xp = grimoire.xp || 0;
+                if (typeof GameFormulas !== 'undefined' && GameFormulas.calculateMaxHp && CharacterSystem.getTotalStat) {
+                    character.hp = GameFormulas.calculateMaxHp(character.className, character.level, CharacterSystem.getTotalStat(character, 'res'));
+                    character.maxHp = character.hp;
+                }
+            }
+
+            if (VizAccount.getEffectiveShares && CharacterSystem.updateCoreBonus) {
+                var effectiveShares = VizAccount.getEffectiveShares(accountData);
+                var cappedShares = Math.min(effectiveShares, 1000000000000);
+                CharacterSystem.updateCoreBonus(character, cappedShares);
+            }
+
+            state.characters[account] = character;
+            state.inventories[account] = state.inventories[account] || [];
+            state.quests[account] = state.quests[account] || (typeof QuestSystem !== 'undefined' && QuestSystem.createPlayerQuestState
+                ? QuestSystem.createPlayerQuestState()
+                : {});
+
+            if (App.getCurrentScreen() === 'duel') render();
+        });
+    }
+
     function _findRelatedDuel(state, opponent) {
         var user = VizAccount.getCurrentUser();
         var duel;

plan.md

@@ -1,39 +1,47 @@
-# Viz-magic — chronicle posts-only cleanup plan
+# Viz-magic — duel opponent resolution + blessing UX plan
 
 ## Scope
-- Вернуть хронику к исходной модели: лента постов пользователей, а не mixed activity feed.
-- Убрать игровые события и bless/award narrative из основной ленты хроники.
-- Сохранить кнопку благословения как действие под постом.
+- Восстановить и зафиксировать корректный источник истины для участников дуэли во всех UI-переходах.
+- Убрать stale/local-state путаницу в duel screen и подтягивать реальные данные противника, когда они есть на chain/grimoire.
+- Проверить blessing flow на предмет реального level restriction и, если ограничения нет, сделать UX хроники понятным для низких уровней.
+- Сохранить все уже внесённые изменения по guild / inventory / chronicle feed cleanup.
 
 ## Non-goals
-- Не трогать VoiceProtocol и storage формат пользовательских постов.
-- Не проектировать отдельную activity-вкладку в этом проходе.
-- Не ломать compose/send flow и bless action под постом.
+- Не откатывать текущий `main`.
+- Не менять on-chain duel/blessing protocol без необходимости.
+- Не возвращать mixed activity feed в Chronicle.
 
 ## Milestones
-1. Зафиксировать источник истины для хроники: посты из voice/publication + локально отправленные post entries.
-2. Убрать сбор recentActions/duel history в основной feed.
-3. Оставить табы working поверх post-only набора.
-4. Прогнать локальные проверки JS.
+1. Проследить duel flow: `challenge -> accept -> state -> render pre-duel/active duel`.
+2. Привязать duel UI к canonical duel participants из chain state, а не к устаревшему локальному экранному state.
+3. Добавить безопасную догрузку character/grimoire данных для opponent, если локальный state их ещё не знает.
+4. Проверить blessing flow и явно отразить в UX, что blessing доступен через Chronicle posts других магов.
+5. Прогнать `node --check` для изменённых файлов, затем commit + push в `main`.
 
 ## Expected files or subsystems
+- `app/js/ui/screens/duel.js`
 - `app/js/ui/screens/chronicle.js`
+- `app/js/i18n/ru.js`
+- `app/js/i18n/en.js`
 
 ## Implementation tasks
-- Оставить в ленте только `chronicle_post` и voice/publication entries с required tag.
-- Не рендерить blessing/hunt/rest/duel как feed entries.
-- Сохранить локальный optimistic post injection после отправки пользователем.
+- В duel screen вычислять opponent из `duel.challenger/duel.target` при наличии state-дуэли.
+- При отсутствии локального character state для участника пробовать восстановить его из chain grimoire и перерендерить экран.
+- Убрать misleading blessing CTA там, где благословить нельзя (например, свой же пост).
+- Добавить понятное объяснение в Chronicle, когда в ленте нет чужих постов для blessing.
 
 ## Validation
-- В хронике больше не появляются bless/award/hunt/duel/rest записи.
-- Пользовательские посты по-прежнему видны.
-- Кнопка благословения под постом остаётся рабочей.
-- `node --check` проходит для изменённых JS файлов.
+- Duel pre-screen и active duel показывают корректного opponent и его реальные name/class/level, если данные есть на chain.
+- Экран дуэли больше не зависит от stale `opponent` fallback как от основного источника истины.
+- В коде нет level-based restriction для blessing.
+- Chronicle явно объясняет, что blessing работает через posts, и не предлагает bless на собственных записях.
+- `node --check` проходит для изменённых файлов.
 
 ## Risks or assumptions
-- Временный optimistic local post живёт в `recentActions`, но фильтруется только как `chronicle_post`.
-- Если потом понадобится activity feed, его лучше делать отдельным экраном/вкладкой, не в основной хронике.
+- Если у opponent нет grimoire на chain или node не вернёт account metadata, UI останется на account fallback, но это будет уже честный degradation path.
+- Chronicle по-прежнему зависит от наличия реальных posts; если никто не пишет, blessable targets не появятся.
 
 ## Definition of done
-- Хроника снова воспринимается как лента постов.
-- Игровые события больше не засоряют social feed.
+- Duel bug исправлен на уровне источника данных, а не замаскирован.
+- Blessing UX не создаёт ложное ощущение level cap и явно объясняет модель через Chronicle posts.
+- Изменения проверены синтаксически и отправлены в `main`.