Merge pull request #37 from alexpolo1/release/v0.10.0

release: v0.10.0

.coveragerc

@@ -1,8 +1,21 @@
 # .coveragerc to control coverage.py
 [run]
 branch = True
+source = src
+# Exclude conda/xonsh internal shims that leak into coverage data. The phantom
+# is an absolute path like D:\a\...\config-3.py, so a bare "config-3.py" omit
+# never matches — use a glob that matches any path ending in config-<n>.py.
+omit =
+    *config-*.py
+    */site-packages/*
+    */conda-meta/*
+
+[xml]
+output = coverage.xml
 
 [report]
+# Ignore missing source files (conda shims reference files not in tree)
+ignore_errors = True
 # Regexes for lines to exclude from consideration
 exclude_lines =
     # Have to re-enable the standard pragma

.env

@@ -1 +0,0 @@
-PYTHONPATH=./src

.env.example

@@ -0,0 +1,25 @@
+# Botty personal environment overrides.
+# Copy this file to ".env" in the repo root and edit your values there.
+# ".env" is ignored by git and should never be committed.
+#
+# Format: BOTTY_<CONFIG_KEY>=<value>
+# These override values from config/params.ini and config/custom.ini at runtime.
+#
+# Common personal settings:
+
+# Bot display/account name used in logs/Discord usernames.
+# BOTTY_NAME=zapzap
+
+# Main Discord webhook (status/death/chicken/general messages).
+# BOTTY_CUSTOM_MESSAGE_HOOK=https://discord.com/api/webhooks/...
+
+# Optional dedicated loot webhook for item drops.
+# BOTTY_CUSTOM_LOOT_MESSAGE_HOOK=https://discord.com/api/webhooks/...
+
+# Optional auto-login credentials (keep private).
+# BOTTY_BNET_NAME=your-battlenet-email-or-name
+# BOTTY_BNET_PASS=your-battlenet-password
+# BOTTY_CHAR_NAME=your-character-name
+
+# Optional path override if you reference it in params.ini:
+# BOTTY_SAVED_GAMES_FOLDER=C:\Users\you\Saved Games\Diablo II Resurrected

.github/workflows/ci.yml

@@ -1,91 +1,213 @@
-name: Botty - CI
+name: CI
+
 on:
-  pull_request:
   push:
-    branches: [main, mine]
-  release:
-    types: [published]
+    branches: [main, stable]
+  pull_request:
+    branches: [main, stable]
+
+# Cancel in-progress runs on new push
+concurrency:
+  group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+  cancel-in-progress: true
 
 jobs:
-  test:
+  install-and-test:
+    name: Install & Test (Windows)
     runs-on: windows-latest
     steps:
-      - uses: actions/checkout@v4
+      - name: Checkout
+        uses: actions/checkout@v4
 
-      - name: Setup Miniconda Python 3.10
-        uses: conda-incubator/setup-miniconda@v3
+      - name: Set up Python
+        uses: actions/setup-python@v5
         with:
-          python-version: '3.10'
-          activate-environment: botty
-          channel-priority: strict
-          environment-file: environment.yml
-          use-only-tar-bz2: true
+          python-version: "3.10"
+          cache: "pip"
 
-      - name: Download test assets
-        shell: powershell
+      - name: Install dependencies
         run: |
-          C:\Miniconda\condabin\conda.bat activate botty
-          python -c "import utils.download_test_assets"
-
-      - name: Pytest
-        shell: powershell
-        run: |
-          C:\Miniconda\condabin\conda.bat activate botty
-          python -c "import sys; print(sys.version)"
-          python -m pytest test/smoke_test.py test/nip/ test/game_stats_test.py test/closest_non_hud_test.py test/transmute/ -v -s
-
-      - name: Coverage
-        shell: powershell
-        run: |
-          C:\Miniconda\condabin\conda.bat activate botty
-          coverage xml
-
-  build:
-    needs: test
-    runs-on: windows-latest
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: Setup Miniconda Python 3.10
-        uses: conda-incubator/setup-miniconda@v3
-        with:
-          python-version: '3.10'
-          activate-environment: botty
-          channel-priority: strict
-          environment-file: environment.yml
-          use-only-tar-bz2: true
-
-      - name: Build exe
-        shell: powershell
-        env:
-          BOTTY_NO_RENAME: '1'
-        run: |
-          C:\Miniconda\condabin\conda.bat activate botty
-          python build.py --conda_path C:\Miniconda
-
-      - name: Prepare release zip
-        shell: powershell
-        run: |
-          $BOTTY_DIR = Get-ChildItem -Directory -Name | Where-Object { $_ -match '^botty_v' } | Select-Object -First 1
-          Write-Host "Botty dir: $BOTTY_DIR"
-          Get-ChildItem -Path $BOTTY_DIR -Recurse | Select-Object -Property FullName, Length
-          $ZIP = "${BOTTY_DIR}.zip"
-          Compress-Archive -Path "${BOTTY_DIR}\*" -DestinationPath $ZIP -Force
-          Write-Host "Release zip: $ZIP"
-          Get-Item $ZIP | Select-Object -Property FullName, Length
-
-      - name: Upload build artifacts
-        uses: actions/upload-artifact@v4
-        with:
-          name: botty-build
-          path: botty_v*/
-          retention-days: 7
-
-      - name: Upload to Release
-        if: github.event_name == 'release'
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          python -m pip install --upgrade pip
+          pip install -r requirements-win11.txt
         shell: bash
+
+      - name: Install Tesseract OCR
+        run: choco install tesseract --no-progress -y
+
+      - name: Python version
+        run: python -c "import sys; print(sys.version)"
+
+      - name: Syntax check
+        env:
+          PYTHONPATH: ./src
+        run: python -m compileall -q src tools test scripts
+
+      - name: Verify core imports
+        env:
+          PYTHONPATH: ./src
         run: |
-          BOTTY_DIR=$(ls -d botty_v* | head -1)
-          gh release upload "${{ github.event.release.tag_name }}" "${BOTTY_DIR}.zip" --clobber
+          python -c "
+          import sys
+          sys.path.insert(0, 'src')
+          modules = [
+              'config',
+              'logger',
+              'screen',
+              'pather',
+              'template_finder',
+              'game_stats',
+              'health_manager',
+              'death_manager',
+              'd2r_image',
+              'item',
+              'item.pickit',
+              'transmute',
+              'shop',
+              'char',
+              'utils',
+              'utils.os_detect',
+              'messages',
+          ]
+          for mod in modules:
+              try:
+                  __import__(mod)
+                  print(f'  {mod}: OK')
+              except ImportError as e:
+                  print(f'  {mod}: FAILED - {e}')
+                  sys.exit(1)
+          print('All core imports successful.')
+          "
+
+      - name: Verify botty runs (import main modules)
+        env:
+          PYTHONPATH: ./src
+        run: |
+          python -c "
+          import sys, os
+          sys.path.insert(0, 'src')
+          import ssl
+          ssl.SSLContext.load_default_certs = lambda *a, **k: None
+
+          from version import __version__
+          print(f'  Version: {__version__}')
+
+          from config import Config
+          print('  Config: OK')
+
+          from game_controller import GameController
+          print('  GameController: OK')
+
+          from bot import Bot
+          print('  Bot: OK')
+
+          from run.diablo import Diablo
+          print('  Diablo run: OK')
+
+          from run.pindle import Pindle
+          print('  Pindle run: OK')
+
+          from run.arcane import Arcane
+          print('  Arcane run: OK')
+
+          from run.vizier import Vizier
+          print('  Vizier run: OK')
+
+          print('All botty entry modules import successfully.')
+          "
+
+      - name: Verify OCR (pytesseract)
+        env:
+          PYTHONPATH: ./src
+        run: |
+          python -c "
+          import sys, os, tempfile
+          sys.path.insert(0, 'src')
+
+          import pytesseract
+          pytesseract.pytesseract.tesseract_cmd = r'C:\Program Files\Tesseract-OCR\tesseract.exe'
+
+          import cv2
+          import numpy as np
+
+          version = pytesseract.get_tesseract_version()
+          print(f'  Tesseract version: {version}')
+
+          # Create a simple test image with text
+          img = np.full((50, 200), 255, dtype=np.uint8)
+          cv2.putText(img, 'Hello Botty', (20, 35),
+              cv2.FONT_HERSHEY_SIMPLEX, 0.8, (0, 0, 0), 1)
+
+          with tempfile.NamedTemporaryFile(suffix='.png', delete=False) as f:
+              cv2.imwrite(f.name, img)
+              text = pytesseract.image_to_string(f.name, lang='eng').strip()
+              print(f'  OCR result: {text}')
+
+          print('OCR pytesseract: OK')
+          "
+
+      - name: Verify OCR (botty ocr module)
+        env:
+          PYTHONPATH: ./src
+          PYTESSERACT_TESSERACT_CMD: C:\Program Files\Tesseract-OCR\tesseract.exe
+        run: |
+          python -c "
+          import sys, os
+          sys.path.insert(0, 'src')
+
+          # Import the botty OCR module - it reads PYTESSERACT_TESSERACT_CMD from env
+          from d2r_image.ocr import image_to_text, pytesseract
+          import cv2
+          import numpy as np
+          import tempfile
+
+          if pytesseract is None:
+              print('  pytesseract not available — skipping')
+              sys.exit(0)
+
+          # Create test image (3-channel for invert)
+          img = np.full((50, 200, 3), 255, dtype=np.uint8)
+          cv2.putText(img, '123', (80, 35),
+              cv2.FONT_HERSHEY_SIMPLEX, 1.0, (0, 0, 0), 2)
+
+          # Run through botty's image_to_text
+          results = image_to_text(
+              [img],
+              model='hover-eng_inconsolata_inv_th_fast',
+              psm=7,
+              crop_pad=False,
+              invert=True,
+              threshold=25,
+          )
+          result_text = results[0].text if results else 'empty'
+          print(f'  OCR result: {result_text}')
+          print('OCR botty module: OK')
+          "
+
+      - name: Log analyzer tests (self-healing framework)
+        env:
+          PYTHONPATH: ./src:.
+          RUN_ENV: test
+        run: |
+          python -m pytest test/auto/test_log_analyzer.py -v --tb=short
+
+      - name: Tests with coverage
+        env:
+          PYTHONPATH: ./src:.
+          RUN_ENV: test
+        run: |
+          python -m coverage run -m pytest -v --tb=short --ignore=test/auto/test_self_healing.py --ignore=test/smoke_test.py --ignore=test/test_version_consistency.py
+          python -m coverage xml --ignore-errors || true
+
+      - name: Upload coverage
+        uses: actions/upload-artifact@v4
+        if: always()
+        # Non-blocking: this is a convenience artifact, not a gate. When the account's
+        # artifact storage quota is full the upload fails with
+        # "Failed to CreateArtifact: Artifact storage quota has been hit" and marked the
+        # whole run red even though every test step passed. A quota problem is not a
+        # broken build.
+        continue-on-error: true
+        with:
+          name: coverage-report
+          path: coverage.xml
+          retention-days: 7

.gitignore

@@ -15,13 +15,75 @@ botty_v*/
 custom.ini
 config/custom.ini
 config/custom.*.ini
+config/*.local.ini
 .vscode
 .vs/
-test/assets/
+# Logs and run reports
+log/
+log/archive/
+log/runs/
+log/stats/
+*.log
+
+# Scraped data (cached web pages, URLs)
+data/d2jsp_pages/
+data/d2jsp_topic_urls.txt
+
+# Debug files
+debug_*.png
+debug_forum.html
+
+# Dev tools (moved to tools/)
+asset_extractor.py
+asset_manager.py
+build.py
+desktop_snap.py
+development.md
+quest_debug.py
+quest_plan.md
+quest_screenshot_tool.py
+references/improvements.md
+run_asset_extractor.bat
+run_chipped_gems.bat
+run_gems_all.bat
+run_quest_debug.bat
+screenshot_tool.py
+start_bot_detached.bat
+start_botty.ps1
+test_plan.md
+update.bat
+
+# Generated/runtime price data (updated automatically by bot)
+config/fg_daily_estimates.json
+config/fg_prices.json
+config/fg_prices.db
+config/daily_prices.json
+config/daily_prices_history.json
+config/traderie_prices.json
+
+# Screenshots (debug/test captures)
+screenshots/
+
+# Tools dev cache
+tools/__pycache__/
+
+# Build artifacts
+botty_v*/
+botty_v*.zip
+
 .venv
+.env
+.env.*
+!.env.example
 *.bak
 .coverage
 htmlcov/
+
+# Secrets / account-specific — never commit
+cookies.txt
+cookies_temp*
+*.cookies
+config/custom.ini
 coverage.xml
 utils/live-view/
 config/bnip/*
@@ -33,5 +95,32 @@ loot_screenshots/
 pickit_screenshots/
 *info_log_parsed.txt
 *info_*.png
-*.log
-log/
+blizzpw.txt
+
+# Local user/workspace files
+.claude/
+GEMINI.md
+test_config_load.py
+
+# Debug directory
+botty_debug/
+
+# per-user character profiles (survive git pulls)
+config/profiles/
+config/active_profile.txt
+
+# Docker / dev-only files (not production)
+.dockerignore
+Dockerfile
+docker-compose.yml
+docker-*.bat
+DOCKER.md
+botty_next/
+src/bridge_server.py
+src/input_layer/bridge_input.py
+.hermes/
+fixtures/
+test/run/
+
+# Installer output (generated by run_install_capture.bat)
+install_log.txt

.hermes/plans/botty_improvements.md

@@ -1,87 +0,0 @@
-# Botty Improvements Implementation Plan
-
-## Status Legend
-- [ ] Not started
-- [~] In progress
-- [x] Done
-- [-] Cancelled / low priority
-
----
-
-## Phase 1: Key Auto-Detection (Issue #940)
-Read D2R .key file and auto-fill hotkeys.
-
-- [x] Create src/utils/key_detector.py module
-- [x] VK code mapping (partial — needs review for accuracy)
-- [x] Parse .key file (text format: VK action_type param)
-- [x] Auto-fill empty [char] hotkeys from detected bindings
-- [x] Auto-fill build-specific skill hotkeys (fohdin, hammerdin, etc.)
-- [x] Wire into config.py load_data()
-- [ ] REVIEW: Verify VK_MAP accuracy (D2R uses its own VK offset scheme)
-- [ ] REVIEW: Skill slot-to-config matching is heuristic — may misassign
-- [ ] TEST: Verify against actual D2R .key file on user's machine
-
-## Phase 2: Target Detection False Positives (Issues #959/#964)
-Health bars and "immune to X" text mistaken for targets.
-
-- [ ] Analyze current get_visible_targets() in target_detect.py
-- [ ] Add shape/size filtering: health bars are thin horizontal strips, immune text is small
-- [ ] Add aspect ratio check: real targets (poison/freeze auras) are roughly circular/elliptical
-- [ ] Add minimum bounding box height constraint (filter out thin text)
-- [ ] Optionally: add color temperature check (immune text is yellow/gold, not blue/green)
-- [ ] Test with screenshots of edge cases
-
-## Phase 3: Pickit Timing Fix (Issue #939)
-Items skipped because bot teleports away before grabbing.
-
-- [ ] Review pickit.py _yoink_item() for timing issues
-- [ ] Add configurable pickup_delay parameter (current: fixed timing)
-- [ ] Add retry logic: if item still visible after pickup attempt, re-try
-- [ ] Add "slow mode" for large/heavy items (framed/magic items may animate longer)
-- [ ] Ensure bot doesn't teleport until pickup animation completes
-- [ ] Test: verify no "Attempt to pick xyz" warnings followed by teleport
-
-## Phase 4: Parallel Template Search (Issue #848)
-Speed up template_finder.search() with threading.
-
-- [ ] Add ThreadPoolExecutor-based search_all_parallel() 
-- [ ] Keep existing search() for single-template (no overhead)
-- [ ] Only parallelize when searching >3 templates simultaneously
-- [ ] Benchmark: measure speedup on typical 1280x720 grab
-
-## Phase 5: Async Mouse Moves (Issue #955)
-Non-blocking mouse movement.
-
-- [ ] Add async_move() to utils/custom_mouse.py
-- [ ] Run movement in background thread
-- [ ] Add is_moving() / wait_for_move() synchronization
-- [ ] Integrate into game_controller.py for smoother action chains
-
-## Phase 6: Hardcore Chicken Loop Fix (Issue #942)
-Prevent infinite death loops on Hardcore characters.
-
-- [ ] Review death_manager.py chicken logic
-- [ ] Add max_chicken_count config parameter (default: 3)
-- [ ] If max chicken count exceeded on HC, exit gracefully instead of re-entering
-- [ ] Add defensive chicken config option (chicken to TP instead of full chicken)
-- [ ] Test: verify HC character exits cleanly after N deaths
-
-## Phase 7: Auto-Label NPCs (Issue #950)
-Learn vendor identities automatically during gameplay.
-
-- [ ] During town states, detect NPC name plates via OCR
-- [ ] Cross-reference detected names with known NPC list
-- [ ] Auto-capture NPC templates when confidence is high
-- [ ] Store learned templates in assets/npc/
-- [ ] This is a long-term feature — lower priority
-
----
-
-## Priority Order (implement in this order)
-1. **Phase 1** - Key auto-detection (already partially done, needs review + test)
-2. **Phase 3** - Pickit timing (high impact on loot collection)
-3. **Phase 2** - Target detection (high impact on kill reliability)
-4. **Phase 6** - Hardcore chicken fix (safety critical)
-5. **Phase 4** - Parallel template search (performance)
-6. **Phase 5** - Async mouse moves (quality of life)
-7. **Phase 7** - Auto-label NPCs (long-term feature)

.hermes/plans/quest-framework-doe.md

@@ -1,561 +0,0 @@
-# Quest Framework + Den of Evil Plan
-
-## Goal
-Build a quest automation framework in botty that can interact with D2R NPCs, handle dialogue,
-track quest progress, and run Den of Evil as the first quest -- all usable by a low-level FoHdin.
-
----
-
-## Architecture
-
-The quest framework is a new subsystem that plugs into the existing botty state machine.
-It follows the same patterns as existing runs (approach -> battle -> return to town) but adds
-NPC dialogue interaction and quest state persistence.
-
-### New files
-
-```
-src/quest/
-    __init__.py                  # Exports
-    quest_manager.py             # Quest state machine + persistence (JSON)
-    quest_dialogue.py            # OCR-based NPC dialogue interaction
-    quest_items.py               # Quest item detection/pickup
-    quest_combat.py              # Lightweight combat wrapper (killing trash)
-    a1/
-        __init__.py
-        q_den_of_evil.py         # Den of Evil run
-```
-
-### Modified files
-
-```
-src/npc_manager.py               # Add TOWN_MAIDEN NPC constant + templates
-src/pather.py                    # Add A1_ROARING_CANYON + DoE entrance locations
-src/bot.py                       # Add quest state, transitions, handler
-src/run/__init__.py              # Export DenOfEvil
-src/town/a1.py                   # (optional) Add can_do_den_of_evil method
-config/params.ini                # Add run_doe to [routes]
-config/bnip/                     # Add town_maiden.png template
-```
-
----
-
-## Phase 1: Foundation
-
-### 1.1 `src/quest/quest_manager.py`
-
-Purpose: Track which quests are done, persist between sessions, dispatch to quest modules.
-
-```python
-class QuestManager:
-    """Manages quest state: tracks done/available quests per act, persists to JSON."""
-    
-    # Quest definitions per act
-    QUESTS = {
-        "a1": ["den_of_evil"],
-        "a2": [],  # future: radament, horadric_staff, etc.
-        ...
-    }
-    
-    def __init__(self):
-        self._state_file = "config/quest_state.json"
-        self._state = self._load()
-    
-    def is_done(self, quest_name: str) -> bool:
-        return self._state.get(quest_name, False)
-    
-    def mark_done(self, quest_name: str):
-        self._state[quest_name] = True
-        self._save()
-    
-    def mark_all_done(self, act: str):
-        for q in self.QUESTS.get(act, []):
-            self._state[q] = True
-        self._save()
-    
-    def next_pending(self, act: str) -> str | None:
-        for q in self.QUESTS.get(act, []):
-            if not self.is_done(q):
-                return q
-        return None
-    
-    def all_done(self, act: str) -> bool:
-        return all(self._state.get(q, False) for q in self.QUESTS.get(act, []))
-    
-    def _load(self) -> dict:
-        if os.path.exists(self._state_file):
-            with open(self._state_file) as f:
-                return json.load(f)
-        return {}
-    
-    def _save(self):
-        with open(self._state_file, "w") as f:
-            json.dump(self._state, f, indent=2)
-```
-
-JSON format (config/quest_state.json):
-```json
-{
-    "den_of_evil": true,
-    "search_for_smith": true,
-    ...
-}
-```
-
-### 1.2 `src/quest/quest_dialogue.py`
-
-Purpose: Talk to NPCs, read dialogue options via OCR, click the right branch.
-This is the core of quest automation -- it makes the bot "converse" with NPCs.
-
-```python
-class QuestDialogue:
-    """OCR-based NPC dialogue interaction for quest conversations."""
-    
-    # ROI at 1280x720
-    DIALOGUE_TEXT_ROI = (200, 470, 680, 100)   # NPC speech text
-    DIALOGUE_OPTIONS_ROI = (200, 560, 680, 140)  # Player response buttons
-    DIALOGUE_CLOSE_Y = 670                       # Close button area
-    
-    @staticmethod
-    def open_dialogue(npc_name: str) -> bool:
-        """Walk to NPC and open their dialogue menu."""
-        from npc_manager import Npc, open_npc_menu
-        return open_npc_menu(getattr(Npc, npc_name.upper()))
-    
-    @staticmethod
-    def read_dialogue() -> dict:
-        """OCR the current dialogue box. Returns:
-        {
-            'npc_text': str,       # What the NPC said
-            'options': [str, ...],  # Response options (may be empty if no choice)
-            'has_continue': bool   # True if just need to click continue
-        }
-        """
-        img = grab()
-        npc_text = ocr_roi(img, self.DIALOGUE_TEXT_ROI)
-        options_text = ocr_roi(img, self.DIALOGUE_OPTIONS_ROI)
-        # Parse options: split by line, filter out empty, return list
-        options = [line.strip() for line in options_text.split('\n') if line.strip()]
-        has_continue = len(options) == 0 or "continue" in options_text.lower()
-        return {
-            'npc_text': npc_text.strip(),
-            'options': options,
-            'has_continue': has_continue
-        }
-    
-    @staticmethod
-    def click_option(option_text: str) -> bool:
-        """Find and click a specific dialogue option by matching text via OCR.
-        Searches the options ROI for a template match of the option text."""
-        img = grab()
-        options_img = cut_roi(img, self.DIALOGUE_OPTIONS_ROI)
-        # Use template_finder or OCR to locate which button matches
-        # Then click at that position
-        ...
-    
-    @staticmethod
-    def continue_dialogue() -> bool:
-        """Click the close/continue button to advance dialogue."""
-        # Click in the close button area
-        x, y, w, h = self.DIALOGUE_CLOSE_Y
-        mouse.click at center of close area
-        ...
-    
-    @staticmethod
-    def follow_conversation(expected_options: list[str]) -> bool:
-        """Follow a multi-step conversation:
-        - Read NPC text
-        - If options present, click the expected one
-        - If no options, click continue
-        - Repeat until dialogue closes or unexpected text appears
-        """
-        max_steps = 20  # Safety limit
-        for i in range(max_steps):
-            dialogue = self.read_dialogue()
-            if not dialogue['has_continue'] and dialogue['options']:
-                # We have a choice - click the expected option
-                for opt in expected_options:
-                    if opt.lower() in ' '.join(dialogue['options']).lower():
-                        if not self.click_option(opt):
-                            return False
-                        break
-                else:
-                    Logger.warning(f"Unexpected dialogue options: {dialogue['options']}")
-                    return False
-            else:
-                # Just continue
-                if not self.continue_dialogue():
-                    return False
-            wait(1.0, 1.5)
-            # Check if dialogue box is still visible
-            if not is_visible(ScreenObjects.NPCDialogue):
-                return True  # Done
-        return False  # Hit max steps
-```
-
-Key design: `follow_conversation()` takes a list of expected response text. It will
-match against whatever options the NPC presents and click the right one. This handles
-multi-branch dialogues without hardcoding step-by-step clicks.
-
-### 1.3 `src/quest/quest_combat.py`
-
-Purpose: Lightweight combat for clearing trash during quests. Reuses existing char methods.
-
-```python
-class QuestCombat:
-    """Combat helpers for quest areas -- reuses existing character combat logic."""
-    
-    @staticmethod
-    def clear_area(pather: Pather, char: IChar, path_nodes: list[int], 
-                   timeout: float = 60) -> bool:
-        """Walk a path while killing monsters until timeout or all nodes cleared.
-        This is the core of DoE: walk down, kill, walk back."""
-        return pather.traverse_nodes(path_nodes, char, timeout=timeout, do_combat=True)
-    
-    @staticmethod
-    def wait_for_clear(char: IChar, timeout: float = 15) -> bool:
-        """Wait until no monsters are visible (area is clear)."""
-        start = time.time()
-        while time.time() - start < timeout:
-            targets = get_visible_targets()
-            if not targets or len(targets) == 0:
-                return True
-            # Attack if enemies present
-            char.attack()
-            wait(0.5)
-        return False
-```
-
-### 1.4 `src/quest/quest_items.py`
-
-Purpose: Detect and pick up quest items (gold glow detection).
-
-```python
-class QuestItems:
-    """Quest item detection and management."""
-    
-    @staticmethod
-    def detect_quest_items(img: np.ndarray) -> list[tuple[float, float]]:
-        """Detect gold-glowing items on screen (quest items).
-        Returns list of (x, y) positions in monitor coords."""
-        quest_item_mask, _ = color_filter(img, Config().colors.get("gold_glow", [
-            (180, 140, 0), (255, 220, 80)
-        ]))
-        # Find contours, return centers
-        ...
-    
-    @staticmethod
-    def pick_up_quest_items(char: IChar, img: np.ndarray = None) -> bool:
-        """Find and pick up any quest items currently visible."""
-        if img is None:
-            img = grab()
-        items = self.detect_quest_items(img)
-        for pos in items:
-            char.pick_up_item(pos, item_name="Quest Item")
-            wait(0.5)
-        return len(items) > 0
-```
-
----
-
-## Phase 2: NPC & Location additions
-
-### 2.1 Add Town_Maiden to `src/npc_manager.py`
-
-```python
-# In class Npc:
-TOWN_MAIDEN = "town_maiden"  # Act 1, Roaring Canyon
-
-# In _build_npcs():
-Npc.TOWN_MAIDEN: {
-    "head": "town_maiden.png",  # Need to capture template
-    "actions": {}  # No trade/identify - just dialogue
-}
-```
-
-The Town Maiden sits in Roaring Canyon (eastern part of Act 1 town). She has a simple
-dialogue: you talk to her to "unlock" the Den of Evil entrance, then you talk to her
-again after clearing it to get the XP reward and reset it for another run.
-
-### 2.2 Add locations to `src/pather.py`
-
-```python
-class Location:
-    # ... existing locations ...
-    
-    # Act 1 Roaring Canyon / Den of Evil
-    A1_ROARING_CANYON = "a1_roaring_canyon"       # Town area where Maiden is
-    A1_DEN_OF_EVIL_ENTRANCE = "a1_doe_entrance"   # Stairs down to DoE
-    A1_DEN_LEVEL_1 = "a1_doe_level_1"
-    A1_DEN_LEVEL_2 = "a1_doe_level_2"
-    A1_DEN_LEVEL_3 = "a1_doe_level_3"
-    A1_DEN_LEVEL_4 = "a1_doe_level_4"
-    # (DoE has 3-5 levels depending on game version - need to confirm)
-```
-
-Path nodes will need to be added for the Roaring Canyon area and each DoE level.
-These are captured via quest_debug.py by walking the path and recording waypoints.
-
----
-
-## Phase 3: Den of Evil run module
-
-### 3.1 `src/quest/a1/q_den_of_evil.py`
-
-```python
-class DenOfEvil:
-    """Den of Evil run - Act 1 repeatable quest for XP.
-    
-    Flow:
-    1. Ensure character is in Act 1
-    2. Walk to Roaring Canyon (Town Maiden)
-    3. Talk to Town Maiden (unlock entrance if needed)
-    4. Enter Den of Evil
-    5. Pre-buff (FoH + Conviction for FoHdin)
-    6. Walk through each level, killing trash
-    7. Exit back to Roaring Canyon
-    8. Talk to Town Maiden again for reward
-    9. Return to town center
-    """
-    
-    name = "run_doe"
-    
-    # Path nodes per level (to be filled in via quest_debug.py)
-    LEVEL_PATHS = {
-        1: [],  # Entrance to level 1 stairs
-        2: [],  # Level 1 to level 2
-        3: [],  # Level 2 to level 3
-        4: [],  # Level 3 to level 4 (or final area)
-    }
-    
-    def __init__(self, pather, town_manager, char, pickit, runs):
-        self._pather = pather
-        self._town_manager = town_manager
-        self._char = char
-        self._pickit = pickit
-        self._runs = runs
-        self._quest_manager = QuestManager()
-        self._dialogue = QuestDialogue()
-    
-    def approach(self, curr_loc: Location, do_buff: bool) -> Location | bool:
-        """Get to Roaring Canyon and talk to Town Maiden."""
-        Logger.info("Run Den of Evil")
-        
-        # Ensure we're in Act 1
-        if TownManager.get_act_from_location(curr_loc) != Location.A1_TOWN_START:
-            curr_loc = self._town_manager.go_to_act(1, curr_loc)
-            if not curr_loc:
-                return False
-        
-        # Walk to Roaring Canyon (Town Maiden area)
-        if not self._pather.traverse_nodes(
-            (curr_loc, Location.A1_ROARING_CANYON), self._char, force_move=True
-        ):
-            return False
-        
-        # Talk to Town Maiden to unlock/open the Den
-        if not self._dialogue.open_dialogue("town_maiden"):
-            return False
-        
-        # Follow the conversation (expect "Oh no, not again" or similar)
-        if not self._dialogue.follow_conversation(["Tell me more", "I'll help you"]):
-            return False
-        
-        # Enter the Den
-        if not self._pather.traverse_nodes(
-            (Location.A1_ROARING_CANYON, Location.A1_DEN_OF_EVIL_ENTRANCE),
-            self._char, force_move=True
-        ):
-            return False
-        
-        return Location.A1_DEN_OF_EVIL_ENTRANCE
-    
-    def battle(self, do_pre_buff: bool) -> bool | tuple[Location, bool]:
-        """Fight through the Den of Evil."""
-        # Pre-buff
-        if do_pre_buff:
-            if not self._char.pre_buff():
-                return False
-        
-        # Clear each level
-        for level in sorted(self.LEVEL_PATHS.keys()):
-            Logger.info(f"Clearing Den of Evil level {level}")
-            if not self._pather.traverse_nodes(
-                self.LEVEL_PATHS[level], self._char, timeout=120, do_combat=True
-            ):
-                Logger.error(f"Failed to clear DoE level {level}")
-                return False
-            
-            # Pick up any quest items / loot
-            self._pickit.pick_up_items(self._char)
-            QuestItems.pick_up_quest_items(self._char)
-        
-        # Walk back to Roaring Canyon
-        if not self._pather.traverse_nodes(
-            (Location.A1_DEN_OF_EVIL_ENTRANCE, Location.A1_ROARING_CANYON),
-            self._char, force_move=True
-        ):
-            return False
-        
-        # Talk to Town Maiden for reward
-        if not self._dialogue.open_dialogue("town_maiden"):
-            return False
-        if not self._dialogue.follow_conversation(["Yes", "Thank you"]):
-            Logger.warning("Failed to collect DoE reward from Town Maiden")
-        
-        # Mark as done (for non-repeatable quests) or just return success
-        # Note: DoE is repeatable once per real-day, so we DON'T mark permanently done
-        # self._quest_manager.mark_done("den_of_evil")  # Only if non-repeatable
-        
-        return (Location.A1_ROARING_CANYON, True)
-```
-
----
-
-## Phase 4: Bot integration
-
-### 4.1 `src/bot.py` changes
-
-```python
-# Add import
-from quest.a1.q_den_of_evil import DenOfEvil
-
-# In __init__:
-self._do_runs["run_doe"] = Config().routes.get("run_doe")
-
-self._doe = DenOfEvil(self._pather, self._town_manager, self._char, self._pickit, self._do_runs)
-
-# In _states list:
-# (No new state needed - DoE uses the existing pattern: town -> doe -> end_run -> town)
-
-# In _transitions list (add):
-{ 'trigger': 'run_doe', 'source': 'town', 'dest': 'doe', 'before': "on_run_doe" },
-
-# Add 'doe' to end_run source list:
-{ 'trigger': 'end_run', 'source': [..., 'doe'], 'dest': 'town', 'before': "on_end_run" },
-
-# Add end_game source:
-{ 'trigger': 'end_game', 'source': [..., 'doe'], 'dest': 'initialization', 'before': "on_end_game" },
-
-# Add handler method:
-def on_run_doe(self):
-    res = False
-    self._do_runs["run_doe"] = False
-    self._game_stats.update_location("DoE")
-    self._curr_loc = self._doe.approach(self._curr_loc, not self._pre_buffed)
-    if self._curr_loc:
-        set_pause_state(False)
-        res = self._doe.battle(not self._pre_buffed)
-    self._ending_run_helper(res)
-```
-
-### 4.2 `src/run/__init__.py` changes
-
-```python
-# No change needed if DoE lives in src/quest/ (not src/run/)
-# But if we want consistency, add:
-from quest.a1.q_den_of_evil import DenOfEvil
-```
-
-### 4.3 `config/params.ini` changes
-
-```ini
-[routes]
-; ... existing runs ...
-; run_doe              (Act 1 Den of Evil - repeatable daily XP)
-order=run_doe
-```
-
-### 4.4 `config/params.ini` FoHdin config
-
-For a lvl 1 Paladin running DoE, the params.ini needs:
-
-```ini
-[char]
-type=fohdin
-...
-
-[fohdin]
-; FoHdin-specific config for low-level DoE runs
-teleport=
-; No teleport at lvl 1-9, so pathing is on foot
-```
-
----
-
-## Phase 5: Testing workflow
-
-### What needs user input (I cannot see D2R):
-
-1. **Capture Town_Maiden template:**
-   - Go to Roaring Canyon in Act 1
-   - Stand near the Town Maiden
-   - Run `quest_debug.py`, press F4 (NPC detection)
-   - Paste output so I can save the template
-
-2. **Capture DoE path nodes:**
-   - Enter the Den of Evil
-   - Run `quest_debug.py`, press F1 at each waypoint
-   - Walk from entrance through each level
-   - Paste outputs so I can build the path arrays
-
-3. **Capture dialogue:**
-   - Talk to Town Maiden (both before and after clearing)
-   - Run `quest_debug.py`, press F2 (dialogue OCR)
-   - Paste output so I can code the conversation flow
-
-4. **Test run:**
-   - After I write the code, you run botty with `run_doe` in the route order
-   - Report what happens / paste terminal output
-   - I iterate based on results
-
-### Lvl 1 Paladin specifics:
-
-- **FoHdin requires FOH skill lvl 6 for Feign of Life passive** -- this needs 3 skill points
-  in FoH, meaning character level 9 minimum (or level 4 with a +1 skill weapon)
-- Before reaching lvl 9, the bot can still run DoE but will be much more fragile
-- Recommended: manually level Paladin to ~lvl 4-5 (short runs in Area 1 or 2) before
-  letting the bot solo DoE with FoH
-- The bot pathing should handle the walk-through at low speed with heavy FoH spam
-
----
-
-## Implementation order
-
-1. Write `quest_manager.py` (simple JSON state tracker)
-2. Write `quest_dialogue.py` (OCR-based NPC interaction)
-3. Write `quest_items.py` + `quest_combat.py` (lightweight helpers)
-4. Add Town_Maiden NPC to npc_manager.py
-5. Write `q_den_of_evil.py` (skeleton with placeholder paths)
-6. Integrate into bot.py (state, transitions, handler)
-7. Update params.ini
-8. **USER TESTS** -- captures templates, paths, dialogue
-9. I fill in the actual path nodes and dialogue based on your captures
-10. Full test run and iterate
-
----
-
-## File tree after implementation
-
-```
-my-botty/
-├── config/
-│   ├── params.ini              # Modified: +run_doe in routes
-│   ├── quest_state.json        # New: auto-created by QuestManager
-│   └── bnip/
-│       └── town_maiden.png     # New: captured template
-├── src/
-│   ├── quest/                  # New directory
-│   │   ├── __init__.py
-│   │   ├── quest_manager.py
-│   │   ├── quest_dialogue.py
-│   │   ├── quest_items.py
-│   │   ├── quest_combat.py
-│   │   └── a1/
-│   │       ├── __init__.py
-│   │       └── q_den_of_evil.py
-│   ├── npc_manager.py          # Modified: +TOWN_MAIDEN
-│   ├── pather.py               # Modified: +A1_ROARING_CANYON, +A1_DEN_* locations
-│   ├── bot.py                  # Modified: +doe state, transitions, handler
-│   └── run/__init__.py         # Modified: +DenOfEvil export
-```

ARCHITECTURE.md

@@ -34,14 +34,14 @@ main.py (main thread)
 
 ### Input Layer (`src/input_layer/`)
 
-Native Windows API input replacement (no kernel drivers):
+Native Windows API input replacement designed to evade kernel-level anti-cheat (Warden). It bypasses common Python libraries like `pyautogui` or `pynput` which can be easily detected.
 
 | File | Purpose |
 |---|---|
-| `win_input.py` | ctypes wrappers for `SendInput`, `GetAsyncKeyState`, `GetCursorPos` |
-| `mouse_impl.py` | Humanized mouse with Bezier curves, Gaussian distortion, endpoint wobble |
-| `hotkey.py` | Polling-based hotkey manager (replaces `keyboard.add_hotkey`) |
-| `__init__.py` | Drop-in API: `from input_layer import keyboard, mouse` |
+| `win_input.py` | Low-level `ctypes` wrappers for `SendInput`, `GetAsyncKeyState`, `GetCursorPos`. Uses standard Windows user-mode APIs. |
+| `mouse_impl.py` | Humanized mouse controller. Features include: Bezier curve trajectories, Gaussian noise (hand tremor), endpoint wobble, and randomized arrival-to-click delays. |
+| `hotkey.py` | Polling-based hotkey manager that avoids global hooks. All polling intervals include micro-jitter. |
+| `__init__.py` | Drop-in API that shims standard input calls with stealth timing and variable duration automatically. |
 
 All key presses include stealth micro-pauses and variable press duration automatically.
 
@@ -53,6 +53,27 @@ Handles D2R window detection and screenshot capture via MSS library. Converts be
 
 Template matching via OpenCV (`cv2.matchTemplate`). Searches against pre-captured asset templates in `assets/templates/`. Returns match position and validity.
 
+**Asset conventions — read before adding or recapturing any template:**
+
+- Every `.png` under the `TEMPLATE_PATHS` roots is loaded recursively and keyed by its
+  **uppercased filename** (`a5_red_portal.png` → `A5_RED_PORTAL`). Keys are one flat
+  namespace across all roots, so filenames must be globally unique — and **never leave a
+  backup or scratch `.png` anywhere under `assets/`**, or it silently becomes a live
+  template.
+- **Keep templates fully opaque (3-channel, or 4-channel with no zero-alpha pixel).**
+  `alpha_to_mask` only produces a mask when the image has 4 channels *and* contains a
+  fully transparent pixel; that mask is then passed to
+  `cv2.matchTemplate(..., TM_CCOEFF_NORMED, mask=...)`. **OpenCV only properly supports
+  masks for `TM_SQDIFF` and `TM_CCORR_NORMED`** — masked `TM_CCOEFF_NORMED` returns
+  unreliable scores and wandering match positions. A masked template will appear to
+  "work" in isolation and then fail at random. See CLAUDE.md Bug 23.
+- Crop something **structurally stable and unoccluded**. Animated or partly hidden
+  features (a swirling portal's interior, a ring occluded by scenery) make poor anchors.
+- **Validate on held-out frames**: build the crop from one capture, score it against
+  *other* captures, and — critically — against frames where the subject is **absent**.
+  A template that scores high on both is matching background, not the subject. Aim for a
+  clear gap straddling the 0.68 default threshold.
+
 ### UI Detection (`src/ui/`)
 
 | File | Detects |
@@ -162,12 +183,15 @@ Singleton that merges config files in priority order:
 
 Supports variable substitution via `[variables]` sections.
 
-### Stealth (`src/utils/stealth.py`)
+### Stealth System (`src/utils/stealth.py`)
 
-Three-tier stealth system:
-- **Tier 1 (Input)**: Micro-pauses, click variance, key press duration, endpoint wobble
-- **Tier 2 (Behavior)**: Wrong waypoint chance, skill mistake chance, skill hesitation
-- **Tier 3 (Session)**: AFK breaks, run skipping, personality seed per character
+A multi-tiered approach to mimicking human behavior and evading detection:
+
+- **Tier 1: Input Stealth**: Automatic micro-pauses (20-120ms), variable key press durations (20-200ms), and non-linear mouse paths via `input_layer`.
+- **Tier 2: Behavioral Stealth**: Probabilistic "mistakes" such as clicking the wrong waypoint (2.5% chance) or skill hesitation (80-300ms) before casting.
+- **Tier 3: Session Stealth**: Randomized run durations (+/-15%), AFK breaks (2-12 mins), and shuffling of farming routes between rotations.
+
+All timing across the bot is routed through `utils.misc.wait()`, which applies Gaussian jitter to every sleep call.
 
 ### Utilities (`src/utils/`)
 
@@ -181,6 +205,17 @@ Three-tier stealth system:
 | `node_recorder.py` | Record new path nodes |
 | `stealth.py` | Stealth behavior randomization |
 
+### Auxiliary Tools
+
+Standalone tools located in the root directory for project maintenance and development:
+
+- `asset_manager.py`: Unified interface for auditing, searching, and optimizing template assets.
+- `asset_extractor.py`: Screenshot capture and AI-assisted entity cropping workflow.
+- `build.py`: PyInstaller wrapper for building production executables.
+- `desktop_snap.py`: Lightweight tool for capturing full desktop screenshots.
+- `quest_debug.py`: Debugging interface for the questing system.
+- `screenshot_tool.py`: Simple utility for taking D2R client area screenshots.
+
 ## Data Flow
 
 ```
@@ -219,6 +254,16 @@ D2R Game Window
 
 At startup, `key_detector.py` reads the character's `.key`/`.keyo` file from `Saved Games/Diablo II Resurrected/` to auto-detect skill bindings and non-skill keys (inventory, show items, etc.). This eliminates manual key configuration.
 
+Slot-to-key mapping (`CHAR_BINDING_SLOTS`):
+| Slot | Config key |
+|---|---|
+| 41 | `show_belt` |
+| 36 | `stand_still` |
+| 44 | `weapon_switch` |
+| 43 | `force_move` |
+
+If params.ini and the `.keyo` file disagree, params.ini wins and a `"Keeping configured key binding"` line is logged. No disagreement = no log line.
+
 ## Coordinate Systems
 
 | System | Origin | Used By |
@@ -229,3 +274,72 @@ At startup, `key_detector.py` reads the character's `.key`/`.keyo` file from `Sa
 | Relative | Template match position | Inventory grid, NPC interaction |
 
 Conversion functions in `screen.py`: `convert_monitor_to_screen()`, `convert_screen_to_abs()`, etc.
+
+## State Machine
+
+`bot.py` uses the `transitions` library. States and transitions are defined at `Bot.__init__` time.
+
+**States**: `initialization`, `hero_selection`, `town`, `level`, `pindle`, `shenk`, `trav`, `nihlathak`, `arcane`, `diablo`, `vizier`, `baal`, `mephisto`, `andariel`, `countess`
+
+**Key transitions**:
+
+| Trigger | Source | Dest | Notes |
+|---|---|---|---|
+| `init` | initialization | initialization | Screen detection, routes to create_game or start_from_town |
+| `select_character` | initialization | hero_selection | |
+| `start_from_town` | initialization/hero_selection | town | |
+| `maintenance` | town | town | Heal, buy pots, stash, repair, resurrect merc |
+| `run_pindle` | town | pindle | |
+| `run_arcane` | town | arcane | |
+| `end_run` | any run state | town | TP back; calls `on_end_run` which TPs to town |
+| `end_game` | town/any run state | initialization | Save & exit; use when no TP scrolls or unrecoverable |
+
+`end_run` requires working TP scrolls. If charges = 0, trigger `end_game` instead — otherwise the bot loops trying to TP back indefinitely.
+
+## HealthManager — Internal Timing
+
+The background health monitor (`src/health_manager.py`) polls at:
+
+```
+interval = max(0, (3/25 - fn_elapsed) * jitter(0.8–1.2))
+```
+
+That's approximately every 3 game frames (96–144ms at 25 FPS). The jitter prevents perfectly regular polling patterns from being detectable.
+
+**Rejuv logic**:
+1. Minimum 0.60s between rejuv drinks (hit recovery guard).
+2. Drinks rejuv if `health ≤ take_rejuv_potion_health` OR `mana ≤ take_rejuv_potion_mana`.
+3. "Double rejuv" chicken fires only if `last_drink < 8s` **AND** `health ≤ take_rejuv_potion_health`.
+   - The HP check is critical: mana-triggered rejuvs can legitimately fire back-to-back at full HP (Hammerdin spending mana fast). Without the HP check, false chickens occur at 99.9% HP.
+
+## PickIt — Item Identity
+
+`GroundItem` has two identity fields:
+
+| Field | Formula | Purpose |
+|---|---|---|
+| `ID` | `slugify(Name + all as_dict() values including Amount)` | Pickit cache key; different gold amounts = different IDs |
+| `UID` | `ID + screen center position` | Deduplication within a single items list; same pile at same coords = same UID |
+
+The fail-detection in `_pick_up_item` uses `item.ID == prev.ID` for gold and `item.UID == prev.UID` for everything else. Two nearby gold piles with different amounts bypass the ID check since they produce different IDs.
+
+`_yoink_item()` always returns `PickedUpResult.PickedUp` regardless of actual success. True pickup failures surface only through `_pick_up_item`'s same-ID/UID repeat detection. On confirmed failure, the item's `ID` is blacklisted in `_cached_pickit_items` so it isn't retried in the same session.
+
+## Configuration Priority
+
+Merge order (highest priority first):
+
+```
+custom.ini > params.ini > game.ini > shop.ini > transmute.ini
+```
+
+`Config` is a singleton (`__new__` + `data_loaded` class variable). First instantiation loads all files; subsequent calls return the same instance. Key detection runs during `__init__` after `self.char` is populated.
+
+## Known Architectural Issues
+
+See `IMPROVEMENTS.md` for the full list. Highest-priority unresolved items:
+
+- **C10**: `kill_thread()` uses `PyThreadState_SetAsyncExc` (CPython private API). Can corrupt locks/GIL. Replace with `threading.Event` cooperative shutdown.
+- **H12/H14**: Health/death managers use module-level globals for state. HealthManager now has a `_state_lock`; death manager does not yet.
+- **M14**: `PickedUpResult` enum has a gap (values 0,1,3,4,5 — missing 2).
+- **H10/H11**: `pather.py` (750 lines) and `config.py` are oversized and should be split.

DEPENDENCIES.md

@@ -0,0 +1,44 @@
+# Botty working-state dependencies (verified 2026-06-11)
+
+Everything required for the bot to run as well as it did during the verified
+full Diablo runs on 2026-06-11. If a future setup misbehaves, diff against this.
+
+## Runtime stack
+| Layer | Requirement | Verified value |
+|---|---|---|
+| Python env | conda env `botty` | `C:\ProgramData\miniforge3\envs\botty` (python 3.10.14) — NOTE: `C:\Users\alex\miniforge3\envs\botty` is a broken leftover (no python.exe); `find_python.bat` skips it correctly |
+| OCR binary | Tesseract 5.5.0 (winget) | `C:\Program Files\Tesseract-OCR\tesseract.exe` — wired in `src/d2r_image/ocr.py` (env var `PYTESSERACT_TESSERACT_CMD` → PATH → this default). Conda tesseract must stay UNINSTALLED (access violations) |
+| OCR backend | pytesseract fallback | tesserocr wheel is DLL-broken (needs Tesseract 4.x libs) — pytesseract is the working path; startup logs `OCR backend: pytesseract (fallback)` |
+| Launcher | `run_botty.bat` | sets conda-like PATH, UTF-8, TESSDATA. No exe build exists — bot always runs current source |
+
+## Key python packages (installed, working)
+```
+opencv-python==4.5.5.64   numpy==1.26.4        pytesseract==0.3.13
+mss==7.0.1                beautifultable==1.1.0 colorama==0.4.6
+discord.py==2.7.1         aiohttp==3.14.1       certifi==2026.5.20
+pillow==12.2.0            rapidfuzz==2.15.1     pywin32==312
+psutil==7.2.2             cryptography==48.0.1
+```
+
+## D2R requirements (template matching breaks without these)
+- Settings must match `assets/d2r_settings.json` — startup warns loudly if not.
+  Verified in-game 2026-06-11: 1280x720 windowed, resolution scale 100, DLSS OFF,
+  AA OFF, AO OFF, texture HIGH, character/environment/transparency/shadow LOW.
+  (DLSS ON was the root cause of the CS template failures earlier that day.)
+- Window: bot enforces client area at (5, 98) size 1280x720 (`enforce_d2r_window`).
+- Keybinds file: `C:\Users\alex\Saved Games\Diablo II Resurrected\Fistman*.keyo`
+  (auto-parsed at startup). Skill hotkeys in `config/params.ini` must match the
+  in-game skill assignments: blessed_hammer=f1, holy_shield=f2, redemption=f3,
+  vigor=f4, conviction=f5, concentration=f8, teleport=b, BO=7, BC=8.
+  Use `tools/capture_skill_hotkeys.py` to verify/capture them from the live game.
+- Char: hammerdin "fistman", Hell, CTA swap on weapon slot 2.
+
+## Host specifics
+- Windows 11 build 26200, display 1920x1200 physical at 125% scaling (1536x960
+  logical). D2R renders 1:1 physical. Any DPI-unaware automation (PowerShell
+  SetCursorPos/mouse_event) lands 1.25x off — use the bot's input_layer from the
+  conda env with `SetProcessDPIAware()` instead.
+- Mouse mode: relative (Win11) with SetCursorPos retry fallback (`win_input.py`).
+- `max_consecutive_fails=5`, `max_game_length_s=900`, maintenance timeout ~280s,
+  per-game WP failure budget = 2 (town_manager) are the safety nets that keep a
+  bad game cheap.

FIRST-RUN.md

@@ -0,0 +1,171 @@
+# Botty First Run Guide
+
+After running `install.bat` and seeing "Installation complete!", follow these steps.
+
+---
+
+## Step 1 — Create your character profile
+
+Botty uses **profiles** to store your character setup. Each profile is a single file in `config/profiles/`.
+
+### Find your character type
+
+Open `config/params.ini` in Notepad, scroll to `[char]`, find `type=`:
+
+| If you play... | Set `type=` to |
+|---|---|
+| Hammerdin Paladin | `hammerdin` |
+| FoH Paladin | `fohdin` |
+| Blizzard Sorceress | `sorceress` |
+| Bone Necromancer | `necro` |
+| Traps Assassin | `trapsin` |
+| Amazon | `amazon` |
+| Barbarian | `barbarian` |
+| Warlock Druid | `warlock` |
+
+### Set your keybinds
+
+In `config/params.ini`, scroll to `[char]` and match these to your D2R key bindings:
+
+```ini
+[char]
+type=hammerdin              ; your build (see table above)
+tp_key=b                    ; Teleport key
+stand_still=capslock        ; Hold to stand still (capslock is common)
+show_belt=k                 ; Open belt
+weapon_switch=w             ; Swap weapons (for CTA builds)
+force_move=e                ; Force movement through mobs
+```
+
+**Important:** `stand_still` cannot be `shift` (conflicts with merc healing). `capslock` is the recommended default.
+
+### Pick your runs
+
+Scroll to `[routes]` in `config/params.ini`:
+
+```ini
+[routes]
+; Enable the runs you want (1 = on, 0 = off)
+run_pindle=1                ; Act 5 Nihlathak temple red portal boss
+run_countess=0              ; Act 1 Forgotten Tower
+run_andariel=0              ; Act 1 Catacombs
+run_arcane=0                ; Act 2 Arcane Sanctuary / Summoner (teleport recommended)
+run_trav=0                  ; Act 3 Travincal council
+run_mephisto=0              ; Act 3 Durance of Hate
+run_diablo=0                ; Act 4 Chaos Sanctuary (teleport recommended)
+run_vizier=0                ; Act 4 Chaos, Vizier only (faster than full Diablo)
+run_nihlathak=0             ; Act 5 Halls of Vaught (teleport recommended)
+run_eldritch=0              ; Act 5 Frigid Highlands, Eldritch only
+run_eldritch_shenk=0        ; Act 5 Eldritch then Shenk in the same game
+
+; Execution order (only includes enabled runs)
+order=run_pindle
+```
+
+> There is no `run_shenk` option — Shenk is Act 5 and is farmed via
+> `run_eldritch_shenk`. The commented route list at the top of the `[routes]`
+> section in `params.ini` is always the authoritative set of names.
+
+Common combinations:
+- **Pindle only:** `run_pindle=1`, `order=run_pindle`
+- **Pindle + Arcane:** `run_pindle=1`, `run_arcane=1`, `order=run_pindle, run_arcane`
+- **Full farm:** Enable all you want, set `order=` to your preferred sequence
+
+### Set difficulty
+
+In `[general]`:
+
+```ini
+difficulty=hell     ; normal, nightmare, or hell
+```
+
+Start with `nightmare` if you're testing. Use `hell` for serious farming.
+
+---
+
+## Step 2 — Save as a profile (optional but recommended)
+
+Profiles survive git pulls. To save your setup:
+
+1. Create a folder: `config/profiles/mychar/`
+2. Create `config/profiles/mychar/profile.ini` with:
+
+```ini
+[general]
+name=mychar
+difficulty=hell
+
+[char]
+type=hammerdin
+tp_key=b
+stand_still=capslock
+show_belt=k
+weapon_switch=w
+force_move=e
+
+[routes]
+run_pindle=1
+order=run_pindle
+```
+
+3. Select it. The profile is chosen by `config/active_profile.txt`, **not** by
+   `name=` in `params.ini` (that field is only a display/label value). Either:
+   - start the bot and press **`end`** to cycle to your profile, or
+   - put the profile folder name on the first line of `config/active_profile.txt`
+
+   At startup the bot logs which one it loaded:
+   `Profile active: mychar (config\profiles\mychar\profile.ini)`
+
+---
+
+## Step 3 — Launch
+
+1. **Start D2R** — set to 1280x720 windowed mode
+2. **Log in** and select your character
+3. **Double-click `run_botty.bat`**
+4. The bot shows a hotkey menu — press **`f11`** to start or pause it. Press **`f12`** to stop.
+
+> The hotkey menu printed at startup is always the source of truth — it reflects
+> your actual `config/params.ini` keys. Do **not** press `insert`: that is
+> `restore_settings_from_backup_key`, which overwrites your current D2R settings
+> with a previously saved backup.
+
+---
+
+## Step 4 — Optional: Discord notifications
+
+To get bot status in Discord:
+
+1. Create a Discord webhook: Server Settings → Integrations → Webhooks → New Webhook → Copy Webhook URL
+2. In `config/params.ini`, under `[general]`:
+
+```ini
+custom_message_hook=https://discord.com/api/webhooks/YOUR/WEBHOOK_URL
+discord_log_chicken=1
+discord_log_errors=1
+```
+
+---
+
+## Troubleshooting
+
+**"Could not find botty conda environment"** — Run `install.bat` first.
+
+**"D2R is not running"** — Launch D2R before running `run_botty.bat`, or set `auto_login=1` in params.ini with your Battle.net credentials.
+
+**Bot can't see templates / everything fails** — Make sure D2R is 1280x720 windowed, not fullscreen or borderless.
+
+**OCR not working** — `install.bat` sets up OCR automatically. Check the install output for "tesserocr: OK" or "pytesseract: OK". At least one must work.
+
+**Check logs:** Open `log/log.txt` in Notepad for detailed output.
+
+---
+
+## Quick reference: what to edit
+
+| File | What to change |
+|---|---|
+| `config/params.ini` | `type=`, keybinds, `difficulty=`, routes, Discord webhook |
+| `config/profiles/*/profile.ini` | Per-character overrides (optional) |
+| `config/game.ini` | **Don't edit** — templates and UI coordinates |
+| `config/shop.ini` | **Don't edit** — vendor buy/sell lists |

IMPROVEMENTS.md

@@ -24,20 +24,9 @@ Character: FOH Paladin | Priority: Anti-cheat stealth > everything else
 
 ## CRITICAL - DO FIRST
 
-### C1. Stealth: Consolidate all timing through centralized wait()
-
-Many places use bare `time.sleep()` instead of `utils.misc.wait()` (which has Gaussian jitter).
-Every direct sleep creates a predictable timing signature detectable by anti-cheat.
-
-Files with bare `time.sleep()`:
-- health_manager.py line 14
-- chest.py
-- pather.py
-- game_recovery.py
-- npc_manager.py
-- bot.py
-
-Fix: Replace all bare `time.sleep(n)` with `wait(n, n*1.2)` for human-like jitter.
+- [x] **C1. Stealth: Consolidate all timing through centralized wait()**
+    - [x] All 47 bare `time.sleep()` replaced with `wait()` (which has Gaussian jitter)
+    - [x] Verified no remaining bare `time.sleep()` in `src/` (except inside `utils.misc.wait`)
 
 ### C2. Stealth: Add variable typing rhythm
 
@@ -83,11 +72,12 @@ Fix: Add per-segment timing variation in `HumanCurve` execution loop.
 
 ### C9. ~~Bug: Fix FoHdin missing PickIt (bot.py line 72)~~ ~~(DONE)~~
 
-### C10. ~~Bug: Replace thread killing with cooperative shutdown~~
-~~`utils.misc.kill_thread()` uses `PyThreadState_SetAsyncExc` (CPython private API).~~
-~~This can leave locks in inconsistent state, cause GIL issues, or corrupt numpy arrays.~~
-~~Fix: Replace with threading.Event flags for cooperative shutdown.~~
-(Still needs doing - this is the most dangerous remaining bug.)
+- [x] **C10. Bug: Replace thread killing with cooperative shutdown**
+    - [x] `utils.misc.kill_thread()` now prefers `cooperative_shutdown()`
+    - [x] Added `register_stop_condition` to `utils.misc`
+    - [x] Centralized `wait()` and `search_and_wait()` now check for shutdown signals
+    - [x] `Bot`, `HealthManager`, and `DeathManager` register their stop conditions
+
 
 ---
 

README.md

@@ -4,13 +4,49 @@
 
 Pixelbot for Diablo 2 Resurrected. This project is for informational and educational purposes only.
 
+## Installation (first time)
+
+**Step 1 — Install Miniforge** (only needed once, skip if you already have conda/Miniconda)
+Download and run the installer from: https://github.com/conda-forge/miniforge/releases/latest
+Pick the Windows x86_64 `.exe`. Keep defaults; tick "Add to PATH" if asked.
+
+**Step 2 — Download Botty**
+Click the green **Code** button on this GitHub page → **Download ZIP**. Extract the ZIP anywhere (e.g. `C:\botty`).
+
+**Step 3 — Install dependencies**
+Double-click **`install.bat`** inside the extracted folder. It will create the `botty` conda environment and install everything. This takes a few minutes the first time.
+
+`install.bat` detects Windows 10 vs Windows 11 automatically:
+- Windows 10 uses `environment-win10.yml`, `requirements-win10.txt`, and absolute mouse input.
+- Windows 11 uses `environment-win11.yml`, `requirements-win11.txt`, and relative mouse input.
+
+Botty also runs OS detection at startup and logs the selected requirements profile and mouse mode.
+
+**Step 4 — Configure**
+Open `config\params.ini` in Notepad and set at minimum:
+- `[char] type=` — your build (`sorceress`, `hammerdin`, `paladin`, `trapsin`, …)
+- `[routes] order=` — which bosses to farm (e.g. `run_pindle`)
+- Hotkeys under your build's section to match your D2R keybinds
+
+**Step 5 — Start**
+Double-click **`run_botty.bat`**. Switch to D2R, go to the hero selection screen, then press **F11** to start. Press **F12** to stop.
+
+> D2R must be in **English** and running at **720p** window mode.
+
+Optional personal setup:
+- Copy `.env.example` to `.env` in the repo root and set personal values there.
+- `.env` is git-ignored so multiple testers can use different local values without git conflicts.
+
+---
+
 ## Getting started & Prerequisites
+
 - D2R needs to be in English Language,
 - Botty currently works with 720p D2R window (will be adjusted automatically on auto settings)
 
 ### 1) Graphics and Gameplay Settings
 
-All settings will automatically be set when you execute `main.exe` and press the hotkey for "Adjust D2R settings" (default f9). It is not a 100% thing, in rare cases you might still have to fiddle around with your brightness. I suggest using the "Graphic Debugger" to verify your settings.
+All settings will automatically be set when you execute `main.exe` and press the hotkey for "Adjust D2R settings" (default **ctrl+f9**). It is not a 100% thing, in rare cases you might still have to fiddle around with your brightness. I suggest using the "Graphic Debugger" to verify your settings.
 **Note**: Make sure that no other external programs adapt your graphics settings (HDR, Geforce Experience, etc.)
 
 ### 2) Supported builds
@@ -23,13 +59,20 @@ Open up D2R and wait till you are at the hero selection screen. Make sure the ch
 
 ### 4) Start Botty
 
-Refer to [development.md](development.md) for setup instructions. Once the conda environment is created:
-
 - **Quick start**: Double-click `run_botty.bat` (auto-detects your conda env)
 - **Manual**: `conda activate botty` then `python src\main.py`
 
 After starting, focus your D2R window and press the start key (default f11). You can always force stop botty with f12. Note: Botty will use the /nopickup command in the first game to avoid pickup up trash while traversing. This command will only allow item pickup when "show items" is active.
 
+### Stability and safety updates
+
+- XP OCR parsing now tolerates common OCR mistakes (`I/l/| -> 1`, `O/o -> 0`, mixed-case "experience").
+- XP status math no longer throws on early-session edge cases; unavailable projections show as `n/a`.
+- Routine repair is best-effort for native-teleport builds to reduce fail spirals when NPC detection is flaky.
+- Repair fallback from A5 now attempts A4 from Larzuk location for better path reliability.
+- Discord message sending now guards invalid embed payloads and has a plain-text fallback.
+- Selling now logs item names (not just positions) and shields are protected by default.
+
 ## Development
 
 Check out the [development.md](development.md) docu for infos on how to build from source and details of the project structure and code.
@@ -99,7 +142,10 @@ order=run_pindle, run_eldritch
 
 | [routes]     | Descriptions                                                             |
 | ------------ | ------------------------------------------------------------------------ |
-| order        | List of runs botty should do. These will be run in the the order listed unless `randomize_runs` is set to 1. Possible runs: </br> run_trav, run_pindle, run_eldritch, run_eldritch_shenk, run_nihlathak (requires teleport), run_arcane (requires teleport), run_diablo (requires teleport, only hammardin)
+| order        | Comma-delimited run list. If `randomize_runs=0`, Botty executes left-to-right. If `randomize_runs=1`, enabled runs are shuffled each game. Possible runs: </br> run_trav, run_pindle, run_eldritch, run_eldritch_shenk, run_nihlathak (teleport strongly recommended), run_arcane (teleport strongly recommended), run_diablo (teleport recommended), run_vizier, run_andariel, run_countess, run_mephisto, run_baal |
+
+Hammerdin keyrun example (stable-first):
+`order=run_countess, run_arcane, run_nihlathak`
 
 | [char]             | Descriptions |
 | ------------------ | -------------------------------------------------------------------------------------------------|
@@ -198,6 +244,32 @@ python tools/click_recorder.py playback  # replay with human-like timing
 
 Records mouse clicks with timestamps and replays them with configurable speed and repeat count.
 
+### Asset Manager
+
+A comprehensive tool for managing bot assets (templates). It helps with inventorying, auditing, searching, and optimizing template images.
+
+```bash
+python asset_manager.py inventory  # List all templates
+python asset_manager.py audit      # Check for missing/low-quality templates
+python asset_manager.py search <name>  # Search for a specific template
+python asset_manager.py quality    # Analyze template quality (SNR, contrast)
+python asset_manager.py batch resize 64x64  # Batch process templates
+```
+
+Features include similarity checking (to find duplicates), automatic cropping, and validation of template paths.
+
+### Asset Extractor
+
+A workflow tool for capturing and cropping new templates from D2R screenshots. Designed to work alongside an AI agent for rapid asset generation.
+
+```bash
+python asset_extractor.py
+```
+
+- **F1**: Capture D2R screen to `screenshots/debug/latest.png`.
+- **F2**: Crop entities using an AI-generated `latest_annotations.json` file.
+- **F3**: List all currently extracted assets.
+
 ### Builds
 || [sorceress]   | Descriptions                                                                  |
 || ------------- | ----------------------------------------------------------------------------- |

assets/d2r_settings.json

@@ -8,14 +8,14 @@
     "Screen Resolution (Windowed)": "1280x720",
     "Resolution Scale": 100,
     "Sharpening": 6,
-    "Game Resolution": 1,
+    "Game Resolution": 2,
     "Light Quality": 2,
     "Blended Shadows": 0,
     "Perspective": 0,
     "VSync": 1,
     "Framerate Cap": 60,
     "Framerate Target": 0,
-    "Window Mode": 0,
+    "Window Mode": 1,
     "Graphic Presets": 4,
     "Texture Quality": 4,
     "Texture Anisotropy": 0,

blizzpw.txt

@@ -1 +0,0 @@
-3LONE*dkbcy^v0I8

build.py

@@ -1,109 +0,0 @@
-import os
-import shutil
-from pathlib import Path
-from src.version import __version__
-import argparse
-import getpass
-import random
-from cryptography.fernet import Fernet
-import string
-
-
-parser = argparse.ArgumentParser(description="Build Botty")
-parser.add_argument(
-    "-v" , "--version",
-    type=str,
-    help="New release version e.g. 0.4.2",
-    default=""
-)
-parser.add_argument(
-    "-c", "--conda_path",
-    type=str,
-    help="Path to local conda e.g. C:\\Users\\USER\\miniconda3",
-    default=f"C:\\Users\\{getpass.getuser()}\\miniconda3")
-parser.add_argument(
-    "-r", "--random_name",
-    action='store_true',
-    help="Will generate a random name for the botty exe")
-parser.add_argument(
-    "-k", "--use_key",
-    action='store_true',
-    help="Will build with encryption key")
-args = parser.parse_args()
-
-
-# clean up
-def clean_up():
-    # pyinstaller
-    if os.path.exists("build"):
-        shutil.rmtree("build")
-    if os.path.exists("main.spec"):
-        os.remove("main.spec")
-    if os.path.exists("health_manager.spec"):
-        os.remove("health_manager.spec")
-    if os.path.exists("shopper.spec"):
-        os.remove("shopper.spec")
-
-if __name__ == "__main__":
-    new_version_code = None
-    if args.version != "":
-        print(f"Releasing new version: {args.version}")
-        os.system(f"git checkout -b new-release-v{args.version}")
-        botty_dir = f"botty_v{args.version}"
-        version_code = ""
-        with open('src/version.py', 'r') as f:
-            version_code = f.read()
-        version_code = version_code.split("=")
-        new_version_code = f"{version_code[0]}= '{args.version}'"
-        with open('src/version.py', 'w') as f:
-            f.write(new_version_code)
-    else:
-        botty_dir = f"botty_v{__version__}"
-        print(f"Building version: {__version__}")
-
-    clean_up()
-
-    if os.path.exists(botty_dir):
-        for path in Path(botty_dir).glob("**/*"):
-            if path.is_file():
-                os.remove(path)
-            elif path.is_dir():
-                shutil.rmtree(path)
-        shutil.rmtree(botty_dir)
-
-    for exe in ["main.py", "shopper.py"]:
-        key_cmd = " "
-        if args.use_key:
-            key = Fernet.generate_key().decode("utf-8")
-            key_cmd = " --key " + key
-        installer_cmd = f"pyinstaller --onefile --noconsole --distpath {botty_dir}{key_cmd} --exclude-module graphviz --exclude-module keyboard --exclude-module mouse --exclude-module pyclick --exclude-module mouseinfo --paths .\\src --paths {args.conda_path}\\envs\\botty\\Lib\\site-packages src\\{exe}"
-        os.system(installer_cmd)
-
-    os.system(f"cd {botty_dir} && mkdir config && cd ..")
-
-    with open(f"{botty_dir}/config/custom.ini", "w") as f:
-        f.write("; Add parameters you want to overwrite from param.ini here")
-    shutil.copy("config/game.ini", f"{botty_dir}/config/")
-    shutil.copy("config/params.ini", f"{botty_dir}/config/")
-    shutil.copy("config/shop.ini", f"{botty_dir}/config/")
-    shutil.copy("config/default.bnip", f"{botty_dir}/config/")
-    os.makedirs(f"{botty_dir}/config/bnip", exist_ok=True)
-    shutil.copy("README.md", f"{botty_dir}/")
-    shutil.copytree("assets", f"{botty_dir}/assets")
-    clean_up()
-
-    if args.random_name:
-        print("Generate random names")
-        new_name = ''.join(random.choices(string.ascii_letters, k=random.randint(6, 14)))
-        os.rename(f'{botty_dir}/main.exe', f'{botty_dir}/{new_name}.exe')
-
-    # Rename main.exe to avoid Warden flagging the obvious name
-    # In CI/production builds (env BOTTY_NO_RENAME=1) keep main.exe as-is
-    if not args.random_name and not os.environ.get("BOTTY_NO_RENAME"):
-        new_name = ''.join(random.choices(string.ascii_lowercase + string.digits, k=8))
-        os.rename(f'{botty_dir}/main.exe', f'{botty_dir}/{new_name}.exe')
-        print(f"Renamed main.exe -> {new_name}.exe")
-
-    if new_version_code is not None:
-        os.system(f'git add .')
-        os.system(f'git commit -m "Bump version to v{args.version}"')

check_dependencies.bat

@@ -0,0 +1,15 @@
+@echo off
+setlocal
+
+set "BOTTY_DIR=%~dp0"
+powershell -NoProfile -ExecutionPolicy Bypass -File "%BOTTY_DIR%tools\check_dependencies.ps1"
+set "RC=%ERRORLEVEL%"
+
+if not "%RC%"=="0" (
+  echo.
+  echo Dependency check reported issues.
+)
+
+pause
+exit /b %RC%
+

config/custom.ini.example

@@ -0,0 +1,23 @@
+; Personal overrides for your local machine/user.
+; This file is tracked as an example.
+; Copy to config/custom.ini and edit values there.
+;
+; Botty auto-loads config/custom.ini when present.
+
+[general]
+; name=botty
+; custom_message_hook=
+; custom_loot_message_hook=
+; discord_status_runs=10
+
+[char]
+; type=blizz_sorc
+; atk_len_pindle=3.0
+; show_items=alt
+
+[sorceress]
+; teleport=t
+
+[blizz_sorc]
+; blizzard=f1
+; ice_blast=f2

config/fg_daily_estimates.json

@@ -0,0 +1,800 @@
+{
+  "generated_at": "2026-06-17T07:19:33.572415+00:00",
+  "mode": "offline",
+  "offline_dir": "data\\d2jsp_pages",
+  "ladder_start_date": "2026-05-12",
+  "days": 60,
+  "topics_scanned": 853,
+  "estimates": {
+    "day_1": {},
+    "day_2": {},
+    "day_3": {},
+    "day_4": {},
+    "day_5": {},
+    "day_6": {},
+    "day_7": {},
+    "day_8": {},
+    "day_9": {},
+    "day_10": {},
+    "day_11": {},
+    "day_12": {},
+    "day_13": {},
+    "day_14": {},
+    "day_15": {},
+    "day_16": {},
+    "day_17": {},
+    "day_18": {},
+    "day_19": {},
+    "day_20": {},
+    "day_21": {},
+    "day_22": {},
+    "day_23": {},
+    "day_24": {},
+    "day_25": {},
+    "day_26": {},
+    "day_27": {},
+    "day_28": {},
+    "day_29": {},
+    "day_30": {},
+    "day_31": {
+      "CTA": {
+        "median_fg": 100.0,
+        "avg_fg": 358.0,
+        "min_fg": 1.0,
+        "max_fg": 1199.0,
+        "samples": 5
+      },
+      "Cham Rune": {
+        "median_fg": 30.0,
+        "avg_fg": 121.0,
+        "min_fg": 5.0,
+        "max_fg": 450.0,
+        "samples": 5
+      },
+      "Lo Rune": {
+        "median_fg": 25.0,
+        "avg_fg": 38.8,
+        "min_fg": 5.0,
+        "max_fg": 100.0,
+        "samples": 4
+      },
+      "Um Rune": {
+        "median_fg": 75.0,
+        "avg_fg": 99.6,
+        "min_fg": 1.0,
+        "max_fg": 400.0,
+        "samples": 10
+      },
+      "Lem Rune": {
+        "median_fg": 60.0,
+        "avg_fg": 72.1,
+        "min_fg": 5.0,
+        "max_fg": 190.0,
+        "samples": 7
+      },
+      "Sorc Torch": {
+        "median_fg": 100.0,
+        "avg_fg": 100.0,
+        "min_fg": 100.0,
+        "max_fg": 100.0,
+        "samples": 1
+      },
+      "Aldur's Advance": {
+        "median_fg": 27.5,
+        "avg_fg": 43.8,
+        "min_fg": 20.0,
+        "max_fg": 100.0,
+        "samples": 4
+      },
+      "Stealth RW": {
+        "median_fg": 55.0,
+        "avg_fg": 55.0,
+        "min_fg": 10.0,
+        "max_fg": 100.0,
+        "samples": 2
+      },
+      "Ist Rune": {
+        "median_fg": 95.0,
+        "avg_fg": 112.2,
+        "min_fg": 1.0,
+        "max_fg": 400.0,
+        "samples": 12
+      },
+      "Mal Rune": {
+        "median_fg": 100.0,
+        "avg_fg": 112.0,
+        "min_fg": 1.0,
+        "max_fg": 400.0,
+        "samples": 13
+      },
+      "Pul Rune": {
+        "median_fg": 60.0,
+        "avg_fg": 72.1,
+        "min_fg": 5.0,
+        "max_fg": 190.0,
+        "samples": 7
+      },
+      "Unid Anni": {
+        "median_fg": 550.0,
+        "avg_fg": 416.7,
+        "min_fg": 100.0,
+        "max_fg": 600.0,
+        "samples": 3
+      },
+      "Dual Leech Ring": {
+        "median_fg": 85.0,
+        "avg_fg": 85.0,
+        "min_fg": 70.0,
+        "max_fg": 100.0,
+        "samples": 2
+      },
+      "Gul Rune": {
+        "median_fg": 25.0,
+        "avg_fg": 63.1,
+        "min_fg": 5.0,
+        "max_fg": 200.0,
+        "samples": 8
+      },
+      "Key": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Insight": {
+        "median_fg": 25.0,
+        "avg_fg": 91.0,
+        "min_fg": 1.0,
+        "max_fg": 400.0,
+        "samples": 6
+      },
+      "Ohm Rune": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Vampire Gaze": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "White": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Shako": {
+        "median_fg": 25.0,
+        "avg_fg": 91.0,
+        "min_fg": 1.0,
+        "max_fg": 400.0,
+        "samples": 6
+      },
+      "Black": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Monarch": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Vex Rune": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Java SK": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "BK Ring": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Skin": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Oculus": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Spirit": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Bone": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Jah Rune": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Moser": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Ber Rune": {
+        "median_fg": 20.0,
+        "avg_fg": 18.3,
+        "min_fg": 5.0,
+        "max_fg": 30.0,
+        "samples": 3
+      },
+      "Arach": {
+        "median_fg": 90.0,
+        "avg_fg": 163.7,
+        "min_fg": 1.0,
+        "max_fg": 400.0,
+        "samples": 3
+      },
+      "5/5 Facet": {
+        "median_fg": 90.0,
+        "avg_fg": 163.7,
+        "min_fg": 1.0,
+        "max_fg": 400.0,
+        "samples": 3
+      },
+      "Mara": {
+        "median_fg": 90.0,
+        "avg_fg": 163.7,
+        "min_fg": 1.0,
+        "max_fg": 400.0,
+        "samples": 3
+      }
+    },
+    "day_32": {},
+    "day_33": {},
+    "day_34": {},
+    "day_35": {},
+    "day_36": {},
+    "day_37": {
+      "Thresher": {
+        "median_fg": 32.5,
+        "avg_fg": 133.3,
+        "min_fg": 0.0,
+        "max_fg": 3500.0,
+        "samples": 210
+      },
+      "Monarch": {
+        "median_fg": 20.0,
+        "avg_fg": 71.5,
+        "min_fg": 0.0,
+        "max_fg": 1300.0,
+        "samples": 318
+      },
+      "Spirit": {
+        "median_fg": 25.0,
+        "avg_fg": 78.3,
+        "min_fg": 0.0,
+        "max_fg": 1300.0,
+        "samples": 237
+      },
+      "Wind": {
+        "median_fg": 30.0,
+        "avg_fg": 175.0,
+        "min_fg": 0.0,
+        "max_fg": 1200.0,
+        "samples": 60
+      },
+      "Arach": {
+        "median_fg": 40.0,
+        "avg_fg": 184.2,
+        "min_fg": 5.0,
+        "max_fg": 3500.0,
+        "samples": 210
+      },
+      "Sorc Torch": {
+        "median_fg": 50.0,
+        "avg_fg": 166.2,
+        "min_fg": 10.0,
+        "max_fg": 2500.0,
+        "samples": 64
+      },
+      "Griffon": {
+        "median_fg": 60.0,
+        "avg_fg": 254.7,
+        "min_fg": 5.0,
+        "max_fg": 3500.0,
+        "samples": 111
+      },
+      "Guardian's Light": {
+        "median_fg": 50.0,
+        "avg_fg": 342.6,
+        "min_fg": 0.0,
+        "max_fg": 3500.0,
+        "samples": 236
+      },
+      "Skin": {
+        "median_fg": 30.0,
+        "avg_fg": 206.8,
+        "min_fg": 0.0,
+        "max_fg": 3000.0,
+        "samples": 242
+      },
+      "Viper Gorge": {
+        "median_fg": 30.0,
+        "avg_fg": 75.9,
+        "min_fg": 0.0,
+        "max_fg": 1200.0,
+        "samples": 231
+      },
+      "Lo Rune": {
+        "median_fg": 30.0,
+        "avg_fg": 113.6,
+        "min_fg": 5.0,
+        "max_fg": 1500.0,
+        "samples": 105
+      },
+      "Enigma": {
+        "median_fg": 20.0,
+        "avg_fg": 182.4,
+        "min_fg": 0.0,
+        "max_fg": 1500.0,
+        "samples": 49
+      },
+      "Bone": {
+        "median_fg": 40.0,
+        "avg_fg": 244.3,
+        "min_fg": 0.0,
+        "max_fg": 3500.0,
+        "samples": 264
+      },
+      "Um Rune": {
+        "median_fg": 45.0,
+        "avg_fg": 187.3,
+        "min_fg": 0.0,
+        "max_fg": 3500.0,
+        "samples": 191
+      },
+      "Lem Rune": {
+        "median_fg": 20.0,
+        "avg_fg": 48.7,
+        "min_fg": 0.0,
+        "max_fg": 750.0,
+        "samples": 126
+      },
+      "Crescent": {
+        "median_fg": 10.0,
+        "avg_fg": 63.0,
+        "min_fg": 5.0,
+        "max_fg": 600.0,
+        "samples": 70
+      },
+      "Ist Rune": {
+        "median_fg": 45.0,
+        "avg_fg": 74.9,
+        "min_fg": 3.0,
+        "max_fg": 750.0,
+        "samples": 173
+      },
+      "Highlord": {
+        "median_fg": 25.0,
+        "avg_fg": 113.0,
+        "min_fg": 5.0,
+        "max_fg": 1500.0,
+        "samples": 70
+      },
+      "BK Ring": {
+        "median_fg": 20.0,
+        "avg_fg": 70.0,
+        "min_fg": 0.0,
+        "max_fg": 1200.0,
+        "samples": 161
+      },
+      "Mal Rune": {
+        "median_fg": 30.0,
+        "avg_fg": 67.0,
+        "min_fg": 3.0,
+        "max_fg": 750.0,
+        "samples": 166
+      },
+      "Insight": {
+        "median_fg": 40.0,
+        "avg_fg": 189.9,
+        "min_fg": 5.0,
+        "max_fg": 3500.0,
+        "samples": 155
+      },
+      "Pul Rune": {
+        "median_fg": 30.0,
+        "avg_fg": 69.6,
+        "min_fg": 5.0,
+        "max_fg": 750.0,
+        "samples": 113
+      },
+      "CTA": {
+        "median_fg": 50.0,
+        "avg_fg": 251.5,
+        "min_fg": 5.0,
+        "max_fg": 3500.0,
+        "samples": 171
+      },
+      "Blessed": {
+        "median_fg": 30.0,
+        "avg_fg": 180.3,
+        "min_fg": 7.0,
+        "max_fg": 1200.0,
+        "samples": 36
+      },
+      "Ohm Rune": {
+        "median_fg": 25.0,
+        "avg_fg": 39.8,
+        "min_fg": 3.0,
+        "max_fg": 400.0,
+        "samples": 119
+      },
+      "Java SK": {
+        "median_fg": 20.0,
+        "avg_fg": 47.0,
+        "min_fg": 1.0,
+        "max_fg": 450.0,
+        "samples": 58
+      },
+      "Vampire Gaze": {
+        "median_fg": 10.0,
+        "avg_fg": 110.6,
+        "min_fg": 0.0,
+        "max_fg": 1500.0,
+        "samples": 80
+      },
+      "Giant": {
+        "median_fg": 20.0,
+        "avg_fg": 96.1,
+        "min_fg": 0.0,
+        "max_fg": 600.0,
+        "samples": 70
+      },
+      "Ber Rune": {
+        "median_fg": 60.0,
+        "avg_fg": 253.1,
+        "min_fg": 5.0,
+        "max_fg": 1200.0,
+        "samples": 58
+      },
+      "Key": {
+        "median_fg": 20.0,
+        "avg_fg": 83.1,
+        "min_fg": 1.0,
+        "max_fg": 1200.0,
+        "samples": 114
+      },
+      "Shako": {
+        "median_fg": 30.0,
+        "avg_fg": 113.6,
+        "min_fg": 0.0,
+        "max_fg": 1500.0,
+        "samples": 208
+      },
+      "Pcomb SK": {
+        "median_fg": 160.0,
+        "avg_fg": 685.0,
+        "min_fg": 5.0,
+        "max_fg": 3000.0,
+        "samples": 48
+      },
+      "Sacred": {
+        "median_fg": 35.0,
+        "avg_fg": 166.1,
+        "min_fg": 0.0,
+        "max_fg": 3500.0,
+        "samples": 244
+      },
+      "Light Jewel": {
+        "median_fg": 80.0,
+        "avg_fg": 234.0,
+        "min_fg": 20.0,
+        "max_fg": 1500.0,
+        "samples": 39
+      },
+      "Fist": {
+        "median_fg": 20.0,
+        "avg_fg": 51.7,
+        "min_fg": 15.0,
+        "max_fg": 120.0,
+        "samples": 9
+      },
+      "Vex Rune": {
+        "median_fg": 30.0,
+        "avg_fg": 97.2,
+        "min_fg": 5.0,
+        "max_fg": 1500.0,
+        "samples": 132
+      },
+      "Unid Anni": {
+        "median_fg": 35.0,
+        "avg_fg": 45.0,
+        "min_fg": 3.0,
+        "max_fg": 150.0,
+        "samples": 96
+      },
+      "Light SK": {
+        "median_fg": 40.0,
+        "avg_fg": 383.2,
+        "min_fg": 5.0,
+        "max_fg": 3500.0,
+        "samples": 41
+      },
+      "Mara": {
+        "median_fg": 50.0,
+        "avg_fg": 146.1,
+        "min_fg": 5.0,
+        "max_fg": 1500.0,
+        "samples": 132
+      },
+      "Cold SK": {
+        "median_fg": 50.0,
+        "avg_fg": 162.9,
+        "min_fg": 7.0,
+        "max_fg": 1200.0,
+        "samples": 45
+      },
+      "Black": {
+        "median_fg": 30.0,
+        "avg_fg": 106.7,
+        "min_fg": 0.0,
+        "max_fg": 2400.0,
+        "samples": 199
+      },
+      "Jah Rune": {
+        "median_fg": 60.0,
+        "avg_fg": 267.5,
+        "min_fg": 0.0,
+        "max_fg": 1200.0,
+        "samples": 87
+      },
+      "Troll": {
+        "median_fg": 30.0,
+        "avg_fg": 32.1,
+        "min_fg": 1.0,
+        "max_fg": 75.0,
+        "samples": 66
+      },
+      "Ward": {
+        "median_fg": 10.0,
+        "avg_fg": 21.8,
+        "min_fg": 1.0,
+        "max_fg": 50.0,
+        "samples": 45
+      },
+      "White": {
+        "median_fg": 20.0,
+        "avg_fg": 37.6,
+        "min_fg": 3.0,
+        "max_fg": 300.0,
+        "samples": 126
+      },
+      "Stealth RW": {
+        "median_fg": 32.5,
+        "avg_fg": 31.7,
+        "min_fg": 5.0,
+        "max_fg": 50.0,
+        "samples": 24
+      },
+      "Doom": {
+        "median_fg": 50.0,
+        "avg_fg": 285.3,
+        "min_fg": 45.0,
+        "max_fg": 1500.0,
+        "samples": 30
+      },
+      "Gul Rune": {
+        "median_fg": 30.0,
+        "avg_fg": 50.4,
+        "min_fg": 0.0,
+        "max_fg": 280.0,
+        "samples": 136
+      },
+      "Death's Fathom": {
+        "median_fg": 110.0,
+        "avg_fg": 96.7,
+        "min_fg": 60.0,
+        "max_fg": 120.0,
+        "samples": 12
+      },
+      "Unid Torch": {
+        "median_fg": 60.0,
+        "avg_fg": 49.5,
+        "min_fg": 5.0,
+        "max_fg": 120.0,
+        "samples": 39
+      },
+      "Sanctuary": {
+        "median_fg": 85.0,
+        "avg_fg": 82.5,
+        "min_fg": 5.0,
+        "max_fg": 170.0,
+        "samples": 24
+      },
+      "Grief": {
+        "median_fg": 50.0,
+        "avg_fg": 170.2,
+        "min_fg": 5.0,
+        "max_fg": 1500.0,
+        "samples": 56
+      },
+      "Night": {
+        "median_fg": 250.0,
+        "avg_fg": 209.0,
+        "min_fg": 20.0,
+        "max_fg": 700.0,
+        "samples": 15
+      },
+      "5/5 Facet": {
+        "median_fg": 50.0,
+        "avg_fg": 313.3,
+        "min_fg": 10.0,
+        "max_fg": 1300.0,
+        "samples": 15
+      },
+      "Treachery": {
+        "median_fg": 20.0,
+        "avg_fg": 157.0,
+        "min_fg": 5.0,
+        "max_fg": 1300.0,
+        "samples": 30
+      },
+      "Pala Torch": {
+        "median_fg": 50.0,
+        "avg_fg": 215.6,
+        "min_fg": 20.0,
+        "max_fg": 1500.0,
+        "samples": 43
+      },
+      "Bulwark": {
+        "median_fg": 50.0,
+        "avg_fg": 440.0,
+        "min_fg": 50.0,
+        "max_fg": 1300.0,
+        "samples": 10
+      },
+      "Fortitude": {
+        "median_fg": 70.0,
+        "avg_fg": 915.0,
+        "min_fg": 20.0,
+        "max_fg": 3500.0,
+        "samples": 16
+      },
+      "Cham Rune": {
+        "median_fg": 20.0,
+        "avg_fg": 22.5,
+        "min_fg": 5.0,
+        "max_fg": 60.0,
+        "samples": 53
+      },
+      "War Traveler": {
+        "median_fg": 20.0,
+        "avg_fg": 33.0,
+        "min_fg": 5.0,
+        "max_fg": 80.0,
+        "samples": 33
+      },
+      "Oculus": {
+        "median_fg": 20.0,
+        "avg_fg": 29.3,
+        "min_fg": 5.0,
+        "max_fg": 90.0,
+        "samples": 63
+      },
+      "Moser": {
+        "median_fg": 12.5,
+        "avg_fg": 12.5,
+        "min_fg": 5.0,
+        "max_fg": 20.0,
+        "samples": 16
+      },
+      "Hellfire": {
+        "median_fg": 60.0,
+        "avg_fg": 53.3,
+        "min_fg": 20.0,
+        "max_fg": 80.0,
+        "samples": 12
+      },
+      "Conviction": {
+        "median_fg": 20.0,
+        "avg_fg": 13.7,
+        "min_fg": 1.0,
+        "max_fg": 20.0,
+        "samples": 6
+      },
+      "Terror Key": {
+        "median_fg": 5.0,
+        "avg_fg": 6.7,
+        "min_fg": 5.0,
+        "max_fg": 10.0,
+        "samples": 12
+      },
+      "Hate Key": {
+        "median_fg": 5.0,
+        "avg_fg": 6.7,
+        "min_fg": 5.0,
+        "max_fg": 10.0,
+        "samples": 12
+      },
+      "Dual Leech Ring": {
+        "median_fg": 50.0,
+        "avg_fg": 198.8,
+        "min_fg": 20.0,
+        "max_fg": 1500.0,
+        "samples": 16
+      },
+      "Cyclone": {
+        "median_fg": 300.0,
+        "avg_fg": 306.7,
+        "min_fg": 20.0,
+        "max_fg": 600.0,
+        "samples": 12
+      },
+      "Fury": {
+        "median_fg": 30.0,
+        "avg_fg": 76.9,
+        "min_fg": 5.0,
+        "max_fg": 250.0,
+        "samples": 32
+      }
+    },
+    "day_38": {},
+    "day_39": {},
+    "day_40": {},
+    "day_41": {},
+    "day_42": {},
+    "day_43": {},
+    "day_44": {},
+    "day_45": {},
+    "day_46": {},
+    "day_47": {},
+    "day_48": {},
+    "day_49": {},
+    "day_50": {},
+    "day_51": {},
+    "day_52": {},
+    "day_53": {},
+    "day_54": {},
+    "day_55": {},
+    "day_56": {},
+    "day_57": {},
+    "day_58": {},
+    "day_59": {},
+    "day_60": {}
+  }
+}

config/fg_day_estimates.json

@@ -0,0 +1,50 @@
+{
+  "notes": "Estimated Day 1-14 ladder prices derived from Day 3 snapshot using decay multipliers. Use as planning guidance, not exact market truth.",
+  "source_day": 3,
+  "day_multipliers": {
+    "day_1": 1.45,
+    "day_2": 1.2,
+    "day_3": 1.0,
+    "day_4": 0.93,
+    "day_5": 0.88,
+    "day_6": 0.84,
+    "day_7": 0.8,
+    "day_8": 0.77,
+    "day_9": 0.74,
+    "day_10": 0.72,
+    "day_11": 0.7,
+    "day_12": 0.68,
+    "day_13": 0.66,
+    "day_14": 0.64
+  },
+  "base_day_3_fg": {
+    "Cham Rune": 800,
+    "Lo Rune": 900,
+    "Ohm Rune": 600,
+    "Vex Rune": 400,
+    "Gul Rune": 200,
+    "Ist Rune": 200,
+    "Mal Rune": 120,
+    "Um Rune": 90,
+    "Pul Rune": 50,
+    "Lem Rune": 70,
+    "Unid Anni": 400,
+    "Unid Torch": 1500,
+    "Unid Griffon": 2000,
+    "Unid Eth Andy": 2000,
+    "Shako": 400,
+    "Mara 30": 1200,
+    "Mara Mid": 750,
+    "BK 5": 600,
+    "War Traveler": 500,
+    "Death's Fathom": 800,
+    "5/5 Facet": 500,
+    "5@res SC": 300,
+    "20life SC": 100,
+    "7mf SC": 100,
+    "Pcomb SK": 200,
+    "Cold SK": 250,
+    "Java SK": 200,
+    "Light SK": 200
+  }
+}

config/game.ini

@@ -18,7 +18,7 @@ rejuv_potion=140,50,40,160,255,255
 skill_charges=70,30,25,150,163,255
 health_globe_red=178,110,20,183,255,255
 health_globe_green=47,90,20,54,255,255
-mana_globe=117,120,20,121,255,255
+mana_globe=110,50,15,125,255,255
 blue_slot=102,194,18,138,230,54
 green_slot=33,181,18,87,258,69
 red_slot=161,204,28,197,240,64
@@ -51,8 +51,8 @@ potion1_y=695
 potion_width=30
 potion_height=30
 potion_next=41
-merc_health_top=14
-merc_health_left=15
+merc_health_top=675
+merc_health_left=305
 merc_health_width=40
 ; skills
 skill_y=693
@@ -88,7 +88,7 @@ play_btn=426,616,320,71
 difficulty_select=536,236,210,320
 gold_btn=997,521,20,18
 inventory_gold=980,510,150,40
-gold_btn_stash=160,520,25,25
+gold_btn_stash=158,518,30,30
 vendor_gold_digits=186,509,97,16
 stash_gold_digits=184,527,100,16
 inventory_gold_digits=1017,523,100,16
@@ -97,16 +97,16 @@ health_globe=160,580,240,140
 mana_globe=887,580,240,140
 health_slice=309,610,7,101
 mana_slice=961,610,7,101
-cut_skill_bar=0,0,1280,653
+cut_skill_bar=0,0,1284,653
 reduce_to_center=120,60,1040,540
-search_npcs=120,0,1040,620
-merc_icon=0,0,100,100
+search_npcs=120,0,1042,620
+merc_icon=10,9,56,56
 loading_left_black=0,0,350,720
 death=444,198,397,71
 tp_search=353,120,547,400
 repair_btn=318,473,90,80
-left_inventory=35,86,378,378
-right_inventory=866,348,379,152
+left_inventory=33,84,382,382
+right_inventory=868,348,379,152
 transmute_third_slot=242,342,38,38
 skill_right=664,673,41,41
 skill_right_expanded=655,375,385,255
@@ -128,12 +128,19 @@ cube_btn_roi=160,368,125,57
 xp_bar_text=369,630,554,34
 corpse=459,195,414,213
 chat_icon=7,555,43,41
-left_panel_header=0,0,450,54
-right_panel_header=830,0,450,54
-npc_dialogue=458,4,21,140
+left_panel_header=0,0,455,56
+right_panel_header=830,0,455,56
+; A CENTRED panel (Chronicle, and anything else that opens mid-screen) puts its
+; close button outside both header ROIs above — the Chronicle's X sits at
+; (952, 56), inside right_panel_header's x-range but exactly at its 56px height
+; boundary, so the match centre falls out. Nothing then closes it, and a
+; centred panel blanks every later template search: measured as a 66s
+; click_red_portal failure on 2026-08-28 (Bug 31's residual).
+center_panel_header=400,0,620,92
+npc_dialogue=456,0,30,150
 bind_skill=516,619,251,29
 quest_skill_btn=284,455,710,121
-left_inventory_tabs=31,62,385,28
+left_inventory_tabs=29,60,389,30
 tab_indicator=31,71,385,14
 stash_page_select_left=155,475,20,20
 stash_page_select_right=270,475,20,20

config/params.ini

@@ -1,58 +1,166 @@
 ; There is detailed documentation for each parameter in the README.md
 [general]
-difficulty=normal
-name=bigfont
+; Personal values can reference env keys from repo-root .env:
+; Example syntax:
+;   custom_message_hook=${BOTTY_CUSTOM_MESSAGE_HOOK}
+;   custom_loot_message_hook=${ENV:BOTTY_CUSTOM_LOOT_MESSAGE_HOOK}
+;   bnet_name=${BOTTY_BNET_NAME}
+;   bnet_pass=${BOTTY_BNET_PASS}
+;   char_name=${BOTTY_CHAR_NAME}
+; difficulty: game difficulty to create ("normal", "nightmare", "hell")
+;
+; Hammerdin difficulty guide (CTA build with Conviction aura):
+;   NORMAL     - Very easy. 1-2h Hammer kills most packs. No CTA needed.
+;   NIGHTMARE  - Recommended starting point. Bosses hit ~100-200 dmg.
+;                CTA + Conviction drops resists. Need decent armor (ED/HR).
+;                Gear targets: ~200 AR, ~15 ED, ~30% HR, ~50% FCR on hammer.
+;   HELL       - Bosses hit 400-800+ dmg per swing. Conviction is mandatory.
+;                Gear targets: ~350+ AR, ~20+ ED, ~50%+ HR, ~60%+ FCR,
+;                ~150% IAS on weapon. Full Rejuv belt recommended.
+;                If you chicken/die repeatedly, drop to Nightmare first.
+difficulty=hell
+; name: bot profile name used in logs/messages and mod launch option replacement
+name=profile1
+; randomize_runs: 0 = run in listed order, 1 = shuffle run order
 randomize_runs=0
+; target_tz: target Terror Zone id (leave as default unless you know the mapping)
 target_tz=1
-saved_games_folder=C:\Users\alex\Saved Games\Diablo II Resurrected
+; saved_games_folder: optional override path to D2R Saved Games folder (blank = auto)
+saved_games_folder=
+; level_max_steps: max pathing steps for leveling-style routines
 level_max_steps=20
 
-; Battle.net credentials (for auto-login at launch)
-; Leave empty to log in manually
-bnet_name=
-bnet_pass=
+; Set to 1 to enable auto-login and auto-launch of D2R on startup.
+; Credentials below are ONLY used when auto_login=1.
+auto_login=0
+bnet_name=${BOTTY_BNET_NAME}
+bnet_pass=${BOTTY_BNET_PASS}
 
 ; Character name to auto-select from the character selection screen
 ; If empty, bot relies on the saved character template from previous sessions
-char_name=Burr
+char_name=${BOTTY_CHAR_NAME}
 
 ; messaging
-custom_loot_message_hook=
-custom_message_hook=
+; custom_loot_message_hook: optional separate webhook for loot notifications
+custom_loot_message_hook=${BOTTY_CUSTOM_LOOT_MESSAGE_HOOK}
+; custom_message_hook: main webhook for status/death/chicken messages
+custom_message_hook=${BOTTY_CUSTOM_MESSAGE_HOOK}
+; discord_log_chicken: 1 = send chicken/death style notifications
 discord_log_chicken=1
+; discord_log_errors: 1 = send a Discord message + error screenshot every time a
+; run fails (approach/battle/exception). Set to 0 to keep error screenshots on
+; disk only. Can also be toggled via [discord_events] error=0.
+discord_log_errors=1
+; discord_status_runs: send periodic status every X completed runs (blank/0 disables)
+discord_status_runs=10
+; discord_status_count: legacy fallback, send periodic status every X games (blank/0 disables)
 discord_status_count=20
+; discord_timing_report_h: post a timing + failure digest to Discord every N hours
+; (0 disables). Aggregated from the same TL> timeline the log uses, so the report and the
+; log can never disagree. The window resets on every send.
+discord_timing_report_h=2
+; pickup_drought_window: warn/alert after this many consecutive games with zero
+; item pickups. Raise this if you run a strict pickit and 0-pickup streaks are
+; expected/normal for you (a fast boss-only rush route with a tight filter can
+; easily go 10 games without a keep-worthy drop).
+pickup_drought_window=10
+; message_api_type: "" disables messaging, "discord" or "generic_api"
 message_api_type=discord
 
 ; breaks
-break_length_m=0
-max_runtime_before_break_m=0
+; break_length_m: scheduled break duration in minutes (0 = disabled)
+break_length_m=15
+; max_runtime_before_break_m: runtime before taking scheduled break (0 = disabled)
+max_runtime_before_break_m=120
 
 ; timers / fail handling
+; d2r_path: Diablo II: Resurrected install path
 d2r_path=C:\Program Files (x86)\Diablo II Resurrected
+; max_consecutive_fails: stop bot after this many failed runs in a row
 max_consecutive_fails=5
-max_game_length_s=160
+; max_game_length_s: emergency timeout per run/game
+; 900: a full Chaos Sanctuary clear (3 seals + bosses + loot) takes ~10 min;
+; 600 force-quit a game while literally waiting for Diablo to spawn (2026-06-10).
+; Genuinely stuck games are caught much earlier by max_maintenance_time_s and
+; approach step timeouts.
+max_game_length_s=900
+; max_maintenance_time_s: if the town maintenance loop (heal/buy/stash/repair) takes
+; longer than this many seconds, save-and-exit and rejoin a fresh game.
+; Prevents the bot staying stuck in A5 town forever when NPCs or pathing fail.
+max_maintenance_time_s=240
+; auto_downgrade_threshold: if combined chickens+deaths exceed this number within 1 hour,
+; bot automatically lowers difficulty by one tier (hell->nightmare->normal) and restarts.
+; Set to 0 to disable. (NOTE: feature is parsed but not yet active in bot logic)
+auto_downgrade_threshold=0
 
-; if you set this field to 1, botty will attempt to restart d2 after a crash or failure
-restart_d2r_when_stuck=0
+restart_d2r_when_stuck=1
+; hardcore: 1 enables hardcore-safe assumptions in some routines
 hardcore=0
 
 ; screenshots
+; info_screenshots: save screenshots for info/chicken/death events
 info_screenshots=1
+; error_screenshots: save a screenshot to log/screenshots/error/ every time a run
+; fails (approach failure, battle failure, or an exception) so logs and visuals can
+; be reviewed side by side. Falls back to info_screenshots if unset.
+error_screenshots=1
+; loot_screenshots: save screenshots for picked loot
 loot_screenshots=0
-pickit_screenshots=0
+; pickit_screenshots: save screenshots for pickit debugging
+pickit_screenshots=1
+
+; recovery
+; disable_run_after_failures: after this many CONSECUTIVE failures of the same run
+; (e.g. run_vizier), the bot disables just that run for the rest of the session and
+; keeps doing the other runs instead of stopping. A single success resets the count.
+; If every run gets disabled the bot stops for investigation. Set high to effectively
+; disable this behaviour.
+disable_run_after_failures=5
+; stash_scan_interval: scan all stash tabs and export stash_list.csv every X runs (0 disables)
+stash_scan_interval=0
+
+[discord_events]
+; Fine-grained Discord event toggles (1=send, 0=disable)
+; status: periodic status + generic bot messages (breaks, shop notifications, etc.)
+status=1
+; item_keep: send kept item notifications (loot webhooks/embeds)
+item_keep=1
+; death: send death notifications
+death=1
+; chicken: send chicken (emergency leave) notifications
+chicken=1
+; stash_full: send stash full notifications
+stash_full=1
+; gold_full: send gold full notifications
+gold_full=1
+; error: send run-failure notifications (message + error screenshot)
+error=1
 
 [stealth]
 ; Multiplies all wait() calls by a random value in this range each call
 ; 1.0 = no change. Set range wider for more human-like timing variation.
-wait_jitter_min = 0.85
+; wait_jitter_* multiplies EVERY wait() in the codebase, not just mouse moves.
+; The floor is clamped to jitter_min*0.8, so 0.85 allowed waits to come out 32%
+; SHORT — the one place jitter stole time from an action instead of adding it
+; between actions. 0.95 keeps waits at-or-longer (effective floor 0.76 vs the
+; old 0.68, and the Gaussian centre sits above 1.0), so a wait can no longer
+; expire before the UI it was waiting on has settled.
+wait_jitter_min = 0.95
 wait_jitter_max = 1.20
 
 ; Extra pixel variance added to every mouse click (on top of existing randomize=5)
 ; 0 = off, 10 = +/-10px extra random offset per click
-click_variance = 8
+; NOTE: click position randomization lives at the CALL SITES (npc_manager,
+; ui/waypoint) where it is tuned to real button geometry — 2-3px for NPC
+; hover targets, +/-9px inside a 47px waypoint button. There is deliberately
+; no global click-offset knob: stacking one on top of those is what starts
+; missing NPCs (see CLAUDE.md Bugs 3/4/6/7).
 
 ; Re-shuffle run order after completing a full rotation (vs only at session start)
-reshuffle_each_rotation = 1
+; 0: keep [pindle, diablo] fixed so every game ENDS in A4 town (Diablo run TPs
+; there) — the next game then spawns at A4 where Jamella/Cain/Tyrael work,
+; avoiding the A5 Malah vendor entirely (stale templates in current patch).
+reshuffle_each_rotation = 0
 
 ; Probability (0-100) of randomly skipping a run each game
 ; 0 = never skip, 20 = skip ~1 in 5 runs
@@ -62,7 +170,14 @@ skip_run_chance = 10
 afk_break_chance = 5
 ; Break duration range in minutes
 afk_break_min_m = 2
-afk_break_max_m = 12
+; 12 -> 7 (2026-08-28). The configured value is NOT the real ceiling:
+; maybe_afk_break does wait(minutes*60, minutes*60*1.5) and wait() then applies
+; its own jitter (up to 1.44x), so a "12 minute" break can idle for ~26 minutes.
+; Measured: an 11.9m break ran 1167.7s = 19.5m, and a 20:56 scheduled break ran
+; 25.5m. A ~25m idle is what left D2R at character select once, unable to
+; re-enter, with the bot spawning a new process every ~20s.
+; 7 caps the real-world worst case near 15m, inside the range proven to resume.
+afk_break_max_m = 7
 
 ; Vary run/battle duration by a Gaussian factor (default +/-15%)
 ; 0.0 = no variation, 0.3 = up to +/-30% variation
@@ -73,6 +188,8 @@ micro_pause_min_ms = 20
 micro_pause_max_ms = 120
 
 ; Vary kill time to avoid perfectly consistent boss fight durations
+; Varies boss attack duration. Only ever LENGTHENS the fight (1.0-1.4x) —
+; shortening it risks leaving a boss alive, which is a failed run, not stealth.
 vary_kill_time = 1
 
 ; Human mouse curve complexity (1.0 = default, 0.5 = more direct, 1.5 = more winding)
@@ -80,8 +197,13 @@ human_curve_complexity = 1.0
 
 ; Arrival-to-click delay: human-like pause between mouse arriving and clicking (milliseconds)
 ; 50-800ms range simulates "is this the right thing?" hesitation
+; Hesitation between arriving at a target and clicking it. OFF by default:
+; it applies to EVERY click, so the old 800ms ceiling could add minutes per
+; run. Enable with click_delay_enabled=1 if you want it; 250 is a realistic
+; ceiling that stays affordable.
+click_delay_enabled = 0
 click_delay_min_ms = 50
-click_delay_max_ms = 800
+click_delay_max_ms = 250
 
 ; Key press duration variance: how long a key is held (milliseconds)
 ; Most presses are short (20-100ms), some linger (up to 200ms)
@@ -95,38 +217,84 @@ skill_hesitation_max_ms = 300
 
 ; Wrong waypoint chance: 2-3% of selecting wrong TP portal then correcting
 ; Humans occasionally misclick waypoint targets
+; ── session rhythm ───────────────────────────────────────────────────────────
+; The strongest remaining signal is not per-click timing: averaged over a
+; session jitter converges, but a player who starts at the same hour, plays the
+; same length and never does anything unproductive does not.
+;
+; session_budget_h: stop the bot after roughly this many hours (0 = unlimited).
+; Actual length is rolled per run at 0.65-1.35x, so consecutive days differ.
+; Raised to 10 for an overnight run: the budget is rolled at 0.65-1.35x, so 6
+; could have stopped the bot after 3.9h and left it idle. 10 guarantees at
+; least ~6.5h while keeping the day-to-day variation.
+; Set to 8 for a >= 5h run (2026-08-28). The value is rolled at 0.65-1.35x, so
+; session_budget_h=5 would average 5h but could stop after 3.25h. 8 gives a
+; 5.2-10.8h window, guaranteeing the 5 hours while keeping day-to-day variation.
+session_budget_h = 8
+; Small cursor movement during long idles. Between actions the cursor otherwise
+; sits exactly where the last click left it.
+idle_drift_enabled = 1
+; Per-game chicken threshold spread. Only ever RAISES the threshold (safer) —
+; a fixed 0.40 every game is a precise tell, but lowering it would cost deaths.
+chicken_variance = 0.08
+; Chance of an unproductive town action (open inventory, close it).
+town_browse_chance = 0.06
+; Chance of walking past an item the filter wanted. Costs real loot — keep low.
+pickup_skip_chance = 0.02
 wrong_waypoint_chance = 0.025
 
 ; Skill mistake chance: 1-2% chance of miscasting and correcting
 ; Simulates human error during combat
-skill_mistake_chance = 0.015
+; Presses a DIFFERENT bound skill before the intended one. Disabled: on a
+; character mid-fight this swaps the active skill or aura, and the attack
+; sequences rely on ending in a known skill state. The stealth value is small
+; next to the risk of a stray keypress landing on something that is not a skill
+; at all — which is exactly how the loot filter ended up being toggled all
+; session (params.ini [paladin] vigor=f4, unbound on this char, F4 = filter).
+; Set back to 0.015 to re-enable.
+skill_mistake_chance = 0
 
 [routes]
-; Add these possible routes to "order" as a comma delimited list to run them:
-; run_trav
-; run_pindle
-; run_eldritch
-; run_eldritch_shenk
-; run_nihlathak
-; run_arcane
-; run_diablo
-; run_vizier
+; Controls which farm runs Botty performs each game.
+; "order" is a comma-delimited list and runs left-to-right when randomize_runs=0.
+; If randomize_runs=1 (in [general]), Botty shuffles enabled runs each game.
+;
+; Hammerdin keyrun recommendation (stable-first):
+; order=run_countess, run_arcane, run_nihlathak
+;
+; Route quick notes:
+; run_trav           (Act 3 Travincal council farm; short/high-density run)
+; run_pindle         (Act 5 Nihlathak temple red portal boss farm)
+; run_eldritch       (Act 5 Frigid Highlands Eldritch-only run)
+; run_eldritch_shenk (Act 5 Eldritch then Shenk in same game)
+; run_nihlathak      (Act 5 Halls of Vaught, teleport strongly recommended)
+; run_arcane         (Act 2 Arcane Sanctuary / Summoner, teleport strongly recommended)
+; run_diablo         (Act 4 Chaos Sanctuary, teleport recommended)
+; run_vizier         (Act 4 Chaos Vizier-only route; lighter/faster than full Diablo run)
 ; run_andariel       (Act 1 Catacombs)
 ; run_countess       (Act 1 Forgotten Tower)
 ; run_mephisto       (Act 3 Durance of Hate)
 ; run_baal           (Act 5 Throne of Destruction)
-order=run_pindle
+; run_baal_xp        (Join public Baal games, hide, collect XP, leave — see [baal_xp])
+order=run_pindle, run_diablo
 
 [char]
 ; ==========================
 ; ==== Mandatory Fields ====
 ; ==========================
 ; These configs have to be alligned with your d2r settings and char build
-type=fohdin
+; type: character build profile to use (must match one section below)
+; examples: blizz_sorc, hammerdin, fohdin
+type=hammerdin
+; belt_rows: in-game belt size (2/3/4)
 belt_rows=4
+; casting_frames: your char cast breakpoint (affects action timing)
 casting_frames=8
+; cta_casting_frames: cast breakpoint on CTA swap (if used)
 cta_casting_frames=8
+; attack_frames: base attack animation timing for non-cast attacks
 attack_frames=15
+; cta_available: 1 if Call to Arms swap exists, else 0
 cta_available=1
 ;Do we want to cast non-cta buffs (ex energy shield) with cta
 buff_with_cta=1
@@ -136,9 +304,22 @@ safer_routines=1
 
 ; num_loot_columns: Number of empty columns from left to right of inventory to be used for looting.
 ; Store charms, etc. to the right of the inventory.
+;
+; This drives ui_roi[restricted_inventory_area] in config.py: the columns to the RIGHT of
+; this count are protected — mouse._is_clicking_safe() cancels any click landing there, so
+; the bot never sells, drops or moves anything in them. Both tomes MUST live in that
+; reserved area: common.tome_state() only searches restricted_inventory_area for them.
+;   3 columns = 12 loot slots, 28 reserved
+;   4 columns = 16 loot slots, 24 reserved   <- now
+;   6 columns = 24 loot slots, 16 reserved   (tried 2026-08-27, reverted — shrinking the
+;                                             reserve is the wrong direction when the
+;                                             books and the charms both live there)
+; Lower this if the tomes or charms are running out of room; raise it only if the reserved
+; area is provably empty.
 num_loot_columns=4
 
 ; game hotkeys:
+; NOTE: each key must match your in-game binding exactly
 force_move=e
 inventory_screen=i
 potion1=1
@@ -151,9 +332,10 @@ show_items=alt
 ; stand_still cannot be the default "shift" as it would interfere with merc healing
 stand_still=capslock
 ; teleport: leave empty if you can't use
-teleport=5
+teleport=b
 town_portal=6
 ; call to arms settings:
+; weapon_switch/battle_orders/battle_command only used when cta_available=1
 weapon_switch=w
 battle_orders=7
 battle_command=8
@@ -165,13 +347,43 @@ stash_gold=1
 use_merc=1
 
 ; Attack length for barbarians should be as high as 8-10 and even 10-12 for trav/shenk
+;
+; Hammerdin attack lengths (seconds of hammer spam per boss):
+;   atk_len_trav        = 4.0  (Council of 3 - 3 council members, easy)
+;   atk_len_pindle      = 8.0  (Pindle — Hell: 13094-16070 HP, 75% fire, 100% poison)
+;     Pindle stats by difficulty:
+;       Normal:    1064-1588 HP, 75% fire, 70% poison
+;       Nightmare: 4773-5859 HP, 100% poison
+;       Hell:      13094-16070 HP, 75% fire, 50% cold, 33% light, 100% poison
+;     With Conviction (-33% res), Hell Pindle = 50% fire, 17% cold, 0% light, 67% poison
+;     CTA adds +100% dmg to hammer. Need ~60%+ FCR and good IAS to kill in 8s.
+;   atk_len_nihlathak   = 4.0  (Nihlathak - single boss)
+;   atk_len_eldritch    = 3.0  (Eldritch only - single boss)
+;   atk_len_shenk       = 4.0  (Shenk only - single boss)
+;   atk_len_arc         = 2.5  (Summoner in Arcane - single boss)
+;   atk_len_diablo      = 10.0 (Diablo in CS - longest single boss)
+;   atk_len_countess    = 3.0  (Countess - single boss, easy)
+;   atk_len_andariel    = 4.0  (Andariel - single boss)
+;   atk_len_mephisto    = 12.0 (Mephisto - single boss, high HP)
+;   atk_len_baal        = 10.0 (Baal - single boss)
+;   atk_len_baal_waves  = 30.0 (Baal's spawn waves before boss)
+;
+; Chaos Sanctuary (Diablo run) individual fights:
+;   atk_len_cs_trashmobs   = 2.0  (Trash packs in CS)
+;   atk_len_diablo_vizier  = 2.0  (Vizier of Chaos - seal boss A)
+;   atk_len_diablo_infector = 4.0 (Infector of Souls - seal boss C)
+;   atk_len_diablo_deseis  = 5.0  (Lord De Seis - seal boss B, hardest seal)
+;
+; Increase these if your hammer kills slower (low IAS/FCR).
+; Decrease if your hammer kills faster (high IAS/FCR, good gear).
+; If you die during a fight, the attack length is too long for your defense.
 atk_len_arc=2.5
 atk_len_eldritch=3.0
 atk_len_nihlathak=4.0
-atk_len_pindle=9.0
+atk_len_pindle=8.0
 atk_len_shenk=4.0
 atk_len_diablo=3.0
-atk_len_trav=3.0
+atk_len_trav=4.0
 
 ; Boss run attack lengths (per-character defaults in kill_* methods)
 ; Adjust these if your build is faster/slower against these bosses
@@ -197,42 +409,107 @@ belt_mp_columns=1
 belt_rejuv_columns=2
 
 ; Potion/chicken settings
-take_health_potion=0.90
-take_mana_potion=0.80
-take_rejuv_potion_health=0.75
-take_rejuv_potion_mana=0.50
-heal_merc=0.7
-heal_rejuv_merc=0.25
-chicken=0.75
-merc_chicken=0.25
+take_health_potion=0.60
+take_mana_potion=0.40
+take_rejuv_potion_health=0.45
+take_rejuv_potion_mana=0.10
+heal_merc=0.70
+heal_rejuv_merc=0.45
+chicken=0.40
+merc_chicken=0.20
 
 ; Misc.
 ; helps reduce accidental pickups when enabled especially on walking characters
-enable_no_pickup=1
+enable_no_pickup=0
+; fill_shared_stash_first: 1 = prefer shared stash tabs before personal stash
 fill_shared_stash_first=0
 ; to gamble, add any/all of the following: circlet, ring, coronet, talon, amulet
 gamble_items=
+; open_chests: 1 = open clickable chests along path when possible
 open_chests=1
-pre_buff_every_run=0
-runs_per_repair=0
+; pre_buff_every_run: 1 = always recast buffs at run start
+pre_buff_every_run=1
+; runs_per_repair: visit repair vendor every X runs (blank/0 disables)
+runs_per_repair=5
+; repair_npc: preferred repair vendor strategy.
+; - a5_larzuk (recommended: stays in A5, no cross-act trip; falls back to Halbu)
+; - a4_halbu (requires WP trip to A4 every repair — act desync risk if it fails)
+; 2026-06-10: switched to a5_larzuk — session logs showed Halbu detection failing
+; 100% (body score ~0.39) and each failed A4 trip desynced the bot's act state.
+repair_npc=a5_larzuk
+; resurrect_npc: which NPC revives the mercenary. Blank = whichever act the character is
+; already in. Set to a4_tyrael to always use Act 4 Tyrael.
+; Measured 2026-08-27 on this setup:
+;   A4 Tyrael     ok 8.2s, ok 24.9s   -  0 errors, ever
+;   A5 Qual-Kehk  fail 113.6s / 52.0s / 163.1s / 72.6s  -  43 errors
+; Tyrael stands on a fixed spot by the A4 waypoint; Qual-Kehk is the least reliable NPC in
+; the route. A failed hunt costs more than the whole A5->A4 trip, and one 163s hunt pushed
+; the town visit past max_maintenance_time_s and killed the game.
+resurrect_npc=a4_tyrael
+; runs_per_stash: stash/sell every X runs (blank/0 disables)
 runs_per_stash=4
-sell_junk=0
+; sell_junk: 1 = vendor non-keep items automatically
+sell_junk=1
+; protect_shields_from_sell: 1 = never vendor items with "shield" in detected name.
+; DISABLED 2026-08-27. This is redundant: the EQUIPPED shield is already protected
+; positionally by mouse._is_clicking_safe(), which cancels any click landing in
+; ui_roi[equipped_inventory_area] while the inventory is open. The name rule only ever
+; hit shields sitting in the INVENTORY grid, which the pickit had already judged.
+; Cost of leaving it on: 130 blocked sells in one day (FIEND SHIELD 64, AERIN SHIELD 41,
+; HERALDIC SHIELD 23...). The same shields were re-evaluated and re-blocked every single
+; game — "Discarding FIEND SHIELD." then "Blocked sell for protected item: FIEND SHIELD" —
+; so they could never leave the pack and permanently occupied slots.
+protect_shields_from_sell=0
+; protect_charms_from_sell: 1 = never drop/vendor charms regardless of pickit verdict
+; (safety net — a misread charm can't be un-sold). Set to 0 to let your pickit rules
+; decide keep/discard for charms same as any other item.
+;
+; KEEP THIS AT 1. Charms only give their bonus while they sit in the INVENTORY, and the
+; resistances matter in nightmare. Briefly set to 0 on 2026-08-27 to reclaim slots and
+; that was wrong — it vendored LARGE CHARM OF FIRE, STOUT SMALL CHARM, SMALL CHARM OF
+; FLAME, STOUT SMALL CHARM OF STRENGTH and LAPIS SMALL CHARM OF VITA (+20 life,
+; cold resist +7%) before it was reverted.
+;
+; The "673 blocked sells" in the log are NOT a bug to fix by selling — they are the guard
+; doing its job, repeatedly, on charms the pickit does not have a keep rule for. If those
+; slots are needed, the answer is a pickit rule that keeps the good charms and stops
+; picking up junk ones, NOT disabling this guard.
+protect_charms_from_sell=1
+; pick_rares_for_gold: 1 = pick all yellow (rare) ground items; non-keep rares will be sold
+; Requires sell_junk=1 to convert extra pickups into gold.
+pick_rares_for_gold=0
+; pick_gold: 1 = pick up ground gold piles, 0 = ignore all ground gold
+pick_gold=1
 
 [transmute]
 ;stash tabs by priority where to put transmuted gems
-stash_destination=3,2,1,0
+stash_destination=0,1,2,3
 ; Add these possible gems to "transmute" to transmute them:
 ; chipped, flawed, standard, flawless
-transmute=flawless
+; DISABLED 2026-08-26: gems are stashed to the GEMS tab and left alone, no cube/convert
+; runs. An empty value makes run_transmutes() bail at "No gem tiers configured", which
+; also holds for force=True (tools/gem_transmute.py), unlike transmute_every_x_game=0.
+; Re-enable by listing tiers again, e.g. transmute=flawless
+transmute=
 ;how often we want to run transmute routine(e.g. every 100 games)
-transmute_every_x_game=2000
+transmute_every_x_game=60
+; number of stash tabs — drives tab click geometry in inventory/common.tab_properties().
+; MUST match the tab bar on screen. This client shows 5: PERSONAL SHARED GEMS MATERIALS RUNES.
+; It was 6, which made tab_properties compute centres of 63/127/192/256/320/384 while the
+; real label centres are 68/144/220/295/370 — tabs 2, 3 and 4 clicked the gaps between tabs.
+stash_tabs=5
+; potion transmute settings
+; convert_rejuv: 1 = convert regular Rejuv Potions to Full Rejuv via cube (3 -> 1)
+convert_rejuv=1
+; min_rejuv_to_convert: minimum regular rejuv potions in inventory before starting conversion
+min_rejuv_to_convert=6
 
 ; ===========================
 ; ==== Builds: Sorceress ====
 ; ===========================
 [sorceress]
-energy_shield=f3
-frozen_armor=f4
+energy_shield=f4
+frozen_armor=f7
 static_field=f5
 telekinesis=f6
 thunder_storm=
@@ -249,14 +526,14 @@ frozen_orb=
 ; blizzard must be right skill, hotkey required
 blizzard=f1
 ; ice_blast must be left skill (hotkey optional as it shouldnt change)
-ice_blast=
+ice_blast=f2
 
 [blizzorb_sorc]
 ; frozen orb must be left skill and preselected (no hotkey required)
 
 ;All others must be right skill and hotkey required!
 blizzard=f1
-glacial_spike=f2 
+glacial_spike=f7
 
 [nova_sorc]
 ; nova must be right skill, hotkey required
@@ -273,22 +550,23 @@ hydra=f1
 ; ==== Builds: Paladin ====
 ; =========================
 [paladin]
-cleansing=
+cleansing=f9
 holy_shield=f2
 redemption=f3
 vigor=f4
 
 [fohdin]
 ; foh must be left skill, hotkey required
-blessed_hammer=
-concentration=
+blessed_hammer=f1
+concentration=f8
 conviction=f5
 foh=f6
 holy_bolt=f7
 
 [hammerdin]
-blessed_hammer=f4
-concentration=f5
+blessed_hammer=f1
+concentration=f8
+conviction=f5
 
 ; =========================
 ; ==== Builds: Warlock ====
@@ -441,27 +719,127 @@ buff_1=
 buff_2=
 
 
+; ==== Run: Cold Plains clear ====
+; Roams an outdoor area and kills with ONE configured skill. Works with any build
+; and from clvl 1, so it is the levelling counterpart to the boss runs.
+; Enable by adding run_cold_plains to [routes] order above.
+;
+; PREREQUISITE: the character must already have the destination waypoint.
+; A fresh character has none - walk there once by hand first.
+[cold_plains]
+; Any Act 1 waypoint label from src/ui/waypoint.py _WAYPOINTS, e.g.
+; "Cold Plains", "Stony Field", "Dark Wood", "Black Marsh".
+area=Cold Plains
+; Key holding the attack skill. NOTE Fireball is clvl 12 - on a fresh character
+; put Fire Bolt (clvl 1) here and re-bind to Fireball later; nothing else changes.
+attack_hotkey=
+; Most sorc/caster skills cast on right click. Melee builds want "left".
+attack_button=right
+; How many roam-or-fight steps before heading back to town.
+max_steps=12
+; Casts per engagement before re-scanning for targets.
+casts_per_target=4
+; Ignore anything further than this many px from the character.
+target_radius=600
+; Delay between casts. Raise if you out-run your cast animation.
+cast_delay=0.25
+; Hard runtime cap per run, seconds - stops a bad area pinning the bot.
+max_runtime_s=180
+; Engagements per step, so an unkillable/misdetected target cannot stall the run.
+max_engagements=8
+
+[baal_xp]
+; Baal XP farm: join public Baal games, hide, collect XP, leave. Repeat.
+; Enable by adding run_baal_xp to [routes] order above.
+enabled=1
+; Substring to match against game names (case-insensitive). Empty = join first game.
+game_name_filter=
+; Max seconds to wait in a game before leaving (XP keeps ticking while you hide).
+max_wait_s=900
+; Leave early if XP gained in this game reaches this value.
+xp_threshold=50000000
+; Leave immediately if HP drops below this percentage (chicken).
+min_hp_pct=40
+; Screen position (client coords) to walk to and stand still while hiding.
+hide_x=640
+hide_y=360
+; Max seconds to wait for a matching game to appear in the browser.
+join_timeout_s=60
+
 [advanced_options]
 ; startup hotkeys
-restore_settings_from_backup_key=f7
-settings_backup_key=f8
-auto_settings_key=f9
-graphic_debugger_key=f10
+; select_runs_key: open run selector UI
+select_runs_key=pagedown
+; restore_settings_from_backup_key: restore backed-up D2R settings
+restore_settings_from_backup_key=insert
+; settings_backup_key: create backup of current D2R settings
+settings_backup_key=pause
+; auto_settings_key: auto-apply required D2R settings
+auto_settings_key=pageup
+; graphic_debugger_key: toggle on-screen debug layers
+graphic_debugger_key=delete
+; resume_key: start/pause bot loop
 resume_key=f11
+; exit_key: hard stop bot
 exit_key=f12
+; cycle_pickit_profile_key: cycle through pickit profiles in config/pickit_profiles/
+cycle_pickit_profile_key=f10
 
 ; etc.
+; graphic_debugger_layer_creator: 1 = enable interactive layer creator tooling
 graphic_debugger_layer_creator=0
+; hwnd_window_process: process regex used to locate D2R window handle
 hwnd_window_process=D2R\.exe
+; hwnd_window_title: optional title regex override for window lookup
 hwnd_window_title=
 ; launch_options: will replace <name> with setting for [general] "name" above
 launch_options=-mod <name> -txt
+; logg_lvl: logging verbosity (debug/info/warning/error)
 logg_lvl=debug
+; message_body_template: payload template for generic_api mode
 message_body_template={{"content": "{msg}"}}
+; message_headers: optional JSON headers for generic_api mode
 message_headers=
+; ocr_during_pickit: 1 = run OCR while looting (slower, more diagnostics)
 ocr_during_pickit=0
 ;use "can_teleport_natively" or "can_teleport_with_charges" if you want to force certain behavior in case autodetection isn't working properly
-override_capabilities=
+override_capabilities=can_teleport_natively
+; pathing_delay_factor: movement/click delay multiplier (1 fast .. 10 slow)
 pathing_delay_factor=2
 ; If you want to control Hyper-V window from host use 0,51 here
+; window_client_area_offset: x,y pixel offset for captured game client area
 window_client_area_offset=0,0
+
+[log_rotation]
+; Log rotation prevents screenshot directories from filling the disk.
+; Each managed directory has a max file count and max total size.
+; When limits are exceeded, oldest files are deleted automatically.
+; Checked every 60 seconds (to avoid I/O overhead during runs).
+;
+; The text log (log/log.txt) is size-capped separately: it rotates at 50 MB,
+; keeps 5 zipped backups in log/archive/ (max 30 archives). Override the per-file
+; cap with the BOTTY_LOG_MAX_MB environment variable (e.g. set 100 for 100 MB).
+
+; pickit directory (log/screenshots/pickit/):
+; Can grow very fast — every item scan writes a PNG + JSON.
+; pickit_max_files=500        ; max files before rotation kicks in
+; pickit_max_mb=500           ; max total size in MB
+pickit_max_files=300
+pickit_max_mb=200
+
+; info directory (log/screenshots/info/):
+; Debug screenshots for deaths, chickens, errors, etc.
+; info_max_files=200          ; max files before rotation kicks in
+; info_max_mb=500             ; max total size in MB
+info_max_files=100
+info_max_mb=200
+
+; items directory (log/screenshots/items/):
+; Loot screenshots sent to Discord.
+; items_max_files=100
+; items_max_mb=100
+items_max_files=50
+items_max_mb=50
+
+; discord_notify_rotation: 1 = send Discord message when log rotation deletes files
+discord_notify_rotation=0

config/pickit_profiles/README.md

@@ -0,0 +1,26 @@
+# Shared pickit profiles (git-tracked)
+
+One folder per pickit set, e.g. for season phases:
+
+```
+config/pickit_profiles/
+  season_start/   *.bnip   (leveling: keep bases, gems, chipped...)
+  mid_season/     *.bnip
+  endgame/        *.bnip   (GG-only filter)
+```
+
+Drop `.bnip` (or `.nip`) files in a folder; a `.nipignore` works like in
+`config/bnip`. These folders ARE committed — build them once, everyone
+gets them via git pull.
+
+## Selecting a set
+Per user, in your gitignored `config/profiles/<you>/profile.ini`:
+
+```ini
+[general]
+pickit_profile=season_start
+```
+
+Priority: `config/profiles/<you>/pickit/` (personal set, gitignored)
+> `config/pickit_profiles/<pickit_profile>/` (shared, from this folder)
+> `config/bnip/` > `config/default.bnip`.

config/shop.ini

@@ -1,4 +1,5 @@
 [claws]
+; shop_trap_claws: 1 = enable claw-shopping routine for trap claws from Anya/Drognan flow
 ; Current scoring for trap claws
 ; 3 traps: +12
 ; 2/1 traps: +8
@@ -7,17 +8,22 @@
 ; x weapon block: +1
 ; x death sentrey: +4
 shop_trap_claws=0
+; trap_min_score: minimum combined score required to keep a trap claw
 trap_min_score=13
 
+; shop_melee_claws: 1 = enable shopping melee claws (venom/block focused)
 ; Current scoring for melee claws
 ; 2 assa: +10
 ; x venom: +6
 ; x weapon block: +2
 shop_melee_claws=0
+; melee_min_score: minimum combined score required to keep a melee claw
 melee_min_score=13
 
 [gloves]
+; shop_3_skills_ias_gloves: 1 = shop +3 skill tree + IAS gloves
 shop_3_skills_ias_gloves=1
+; shop_2_skills_ias_gloves: 1 = shop +2 skill tree + IAS gloves
 shop_2_skills_ias_gloves=0
 
 ;
@@ -29,6 +35,9 @@ shop_2_skills_ias_gloves=0
 ; apply_pather_adjustment - alternative option that applies an adjustment to movements.
 ;                           Should not need this. Try it if you have trouble when it is off.
 [scepters]
+; shop_hammerdin_scepters: 1 = enable Hammerdin scepter shopping route
 shop_hammerdin_scepters=1
+; speed_factor: movement compensation for FRW and path timing during shopping loop
 speed_factor=0.25
-apply_pather_adjustment=0
+; apply_pather_adjustment: optional alternate node adjustment if default route misses NPC/shop spots
+apply_pather_adjustment=0

docs/auto_build_plan.md

@@ -0,0 +1,141 @@
+# Auto Skill + Attribute Allocation Plan
+
+## Goal
+Add an optional system that automatically assigns:
+- skill points
+- attribute points
+
+based on:
+- active character profile (`blizz_sorc`, `fohdin`, `hammerdin`, etc.)
+- current character level
+
+without breaking existing manual setups.
+
+## Scope
+- Planning and architecture for Botty repo.
+- No forced behavior changes: feature must be opt-in.
+
+## Requirements
+1. Determine current level reliably at runtime.
+2. Select a build template by character profile.
+3. Apply points safely only when unspent points exist.
+4. Record every allocation in logs/events for audit/replay.
+5. Abort safely on uncertainty (wrong UI state, OCR mismatch, missing templates).
+
+## Current Level Detection Strategy
+
+### Primary path
+Use `player_bar.get_experience()` (already used in `game_stats.log_exp`) to derive level from XP table.
+
+### Secondary fallback
+Open character panel (`C`) and OCR level/name line directly from upper-left panel region.
+
+### Tertiary fallback
+If OCR fails repeatedly:
+- keep previous known good level for session,
+- do **not** allocate points until confidence is restored.
+
+### Confidence rules
+- Require two consistent reads before first allocation in a session.
+- Reject impossible jumps (e.g., +5 levels at once).
+- Persist `last_known_level` in session stats snapshot.
+
+## Build Template Model
+
+Add config-backed build templates, e.g.:
+- `config/auto_builds/blizz_sorc.ini`
+- `config/auto_builds/hammerdin.ini`
+- `config/auto_builds/fohdin.ini`
+
+Each template defines per-level targets:
+- desired skill totals by level milestone
+- desired attribute distribution (str/dex/vit/ene)
+
+Example concept:
+- Level 1-17: early progression targets
+- Level 18-29: mid-game unlock path
+- Level 30+: core skill maxing order
+
+## Runtime Flow
+1. Enter town and open character/skill UI.
+2. Detect level and unspent points.
+3. Load template for `Config().char["type"]`.
+4. Compute delta between current allocation and target-at-level.
+5. Apply points stepwise:
+   - attributes first (optional toggle),
+   - skills second.
+6. Verify post-apply state.
+7. Log allocation summary and persist snapshot.
+
+## Safety Guards
+- Only run in town.
+- Require stash/vendor windows closed.
+- Hard cap per cycle (e.g., max 10 clicks per stat/skill group).
+- On mismatch/timeout:
+  - stop allocation immediately,
+  - screenshot + structured error event,
+  - continue bot without crashing.
+
+## Config Additions (Planned)
+In `[char]` or new `[auto_build]` section:
+- `auto_assign_skills=0/1`
+- `auto_assign_attributes=0/1`
+- `auto_build_profile=` (defaults to `char.type`)
+- `auto_build_check_every_x_games=`
+- `auto_build_safe_mode=1` (extra verification)
+
+## Logging / Telemetry
+Add structured events:
+- `auto_build_check_started`
+- `auto_build_level_detected`
+- `auto_build_points_detected`
+- `auto_build_applied`
+- `auto_build_skipped`
+- `auto_build_error`
+
+Include:
+- profile
+- level
+- points spent
+- before/after snapshots
+
+## UI / Input Dependencies
+Need stable template references for:
+- character panel level region
+- unspent attribute points indicator
+- unspent skill points indicator
+- individual plus-buttons for stats/skills
+
+## Test Plan
+1. Unit tests:
+   - level-to-target mapping
+   - delta computation
+   - guard conditions
+2. Integration dry-run mode:
+   - compute and log planned actions without clicking.
+3. Live smoke tests per profile:
+   - `blizz_sorc`, `hammerdin`, `fohdin`
+4. Regression:
+   - ensure normal runs unaffected with feature disabled.
+
+## Inputs Needed From You
+1. Screenshots for each supported class at:
+   - character panel open,
+   - skill tree open,
+   - visible unspent points.
+2. Preferred leveling templates:
+   - exact skill priority order by level range.
+   - attribute rules (e.g., str to gear breakpoint, then vit).
+3. Whether respec-aware logic is needed in v1.
+
+## Rollout Phases
+1. Phase 1: Level detection + dry-run planner only.
+2. Phase 2: Attribute auto-assign (safer, fewer UI branches).
+3. Phase 3: Skill auto-assign with full verification.
+4. Phase 4: Expanded profile templates + docs.
+
+## Definition of Done
+- Feature is opt-in and stable for `blizz_sorc`, `hammerdin`, `fohdin`.
+- Level detection is reliable with fallback behavior.
+- No crash on detection/allocation failure.
+- Full logs available for every auto-allocation decision.

docs/bnip_guide.md

@@ -0,0 +1,74 @@
+# BNIP Guide
+
+This guide explains how to edit `config/default.bnip` safely and predictably.
+
+## What BNIP Does
+
+BNIP rules decide which items Botty keeps.
+Each line is a filter expression evaluated against detected item data.
+
+## Rule Shape
+
+Typical rule format:
+
+```text
+[Name] == Ring && [Quality] == Rare # [Fcr] >= 10 && [Allres] >= 15
+```
+
+- Left side (`[Name]`, `[Type]`, `[Quality]`, etc.) narrows item identity.
+- Right side after `#` checks stats/rolls.
+
+## Enable or Disable Rules
+
+- Enabled: line starts with `[...`
+- Disabled: line starts with `//`
+
+Example:
+
+```text
+//[Name] == Lemrune
+[Name] == Pulrune
+```
+
+## Safe Editing Workflow
+
+1. Copy an existing nearby rule.
+2. Keep your new rule commented out initially (`//`).
+3. Enable one new rule at a time.
+4. Run a few games and verify behavior before adding more.
+
+## Rule Ordering
+
+BNIP files are easier to maintain when ordered from specific to broad.
+
+- Put strict high-value rules first.
+- Put broad catch-all rules later.
+- Avoid duplicated broad rules in multiple sections.
+
+## Common Fields You Will Use
+
+- `[Name]`
+- `[Type]`
+- `[Quality]`
+- `[Flag]` (for ethereal/sockets behavior)
+- Stat aliases like `[Fcr]`, `[Allres]`, `[Enhanceddefense]`, `[Enhanceddamage]`
+
+## Troubleshooting
+
+If a desired item is not kept:
+
+1. Confirm the rule is enabled (no `//`).
+2. Relax one condition at a time.
+3. Check for typos in stat aliases.
+4. Ensure no local BNIP file in `config/bnip/` is overriding expectations.
+
+If too much junk is kept:
+
+1. Tighten broad rules.
+2. Disable catch-all rules first.
+3. Add stricter stat thresholds.
+
+## Recommended Local Customization
+
+Keep `config/default.bnip` as the team baseline.
+Put personal experiments in separate local `.bnip` files under `config/bnip/` and test there first.

docs/broken_runs_plan.md

@@ -0,0 +1,142 @@
+# Broken Runs - Setup Guide
+
+## Current Status
+
+All 4 boss runs (Countess, Andariel, Mephisto, Baal) have **code structure in place** but require **path recording** before they can run.
+
+| Run | Code | Guards | Path Coords | Templates | Walking Fallback |
+|---|---|---|---|---|---|
+| Countess | OK | (0,0) check | 0,0 placeholder | Empty dir | Nodes 1000-1004 |
+| Andariel | OK | (0,0) check | 0,0 placeholder | Empty dir | Nodes 1010-1012 |
+| Mephisto | OK | (0,0) check | 0,0 placeholder | Empty dir | Nodes 1020-1021 |
+| Baal | OK | (0,0) check | 0,0 placeholder | Empty dir | Nodes 1030-1031 |
+
+## What's Done
+
+- **Run classes**: `countess.py`, `andariel.py`, `mephisto.py`, `baal.py` - all wired in `bot.py`
+- **Kill methods**: `kill_countess()`, `kill_andariel()`, `kill_mephisto()`, `kill_baal()`, `kill_baal_waves()` - implemented in Hammerdin/FoHdin/Warlock
+- **Path guards**: All 4 runs check for `(0,0)` paths and refuse to run with a clear error message
+- **Walking fallback**: `pather.py` has node definitions (1000-1031) and path routes for walking fallback
+- **Template dirs**: `assets/templates/countess/`, `andariel/`, `mephisto/`, `baal/` created (empty)
+
+## What's Needed to Enable Each Run
+
+### Option A: Teleport (faster, recommended)
+
+Record path coordinates in `config/game.ini` using `node_recorder.py`:
+
+```bash
+cd C:\Users\alex\Downloads\my-botty
+python src/utils/node_recorder.py
+```
+
+Then follow the on-screen instructions:
+- **F8**: Capture template ROI (top-left, then bottom-right) -> saves PNG
+- **F9**: Record node position at cursor -> generates path coordinates
+- **F10**: Update all nodes with visible templates
+
+### Option B: Walking (slower, needs templates + nodes)
+
+Same as above, but also requires creating template PNGs for each landmark.
+
+---
+
+## 1. COUNTESS (Act 1 Forgotten Tower)
+
+**game.ini keys to record (5):**
+
+| Key | What to Record |
+|---|---|
+| `a1_tower_level2_enter` | First click inside tower after entering from Black Marsh |
+| `a1_tower_level3_enter` | Top of stairs from L2 to L3 |
+| `a1_tower_level4_enter` | Top of stairs from L3 to L4 |
+| `a1_tower_level5_enter` | Top of stairs from L4 to L5 |
+| `a1_countess_safe_dist` | Position near Countess at safe hammer range |
+
+**Templates for walking fallback (in `assets/templates/countess/`):**
+
+| Template File | Template Key | Purpose |
+|---|---|---|
+| `countess_tower_l2.png` | COUNTESS_TOWER_L2 | Tower L2 entrance |
+| `countess_tower_l3.png` | COUNTESS_TOWER_L3 | Stairs L2->L3 |
+| `countess_tower_l4.png` | COUNTESS_TOWER_L4 | Stairs L3->L4 |
+| `countess_tower_l5.png` | COUNTESS_TOWER_L5 | Stairs L4->L5 |
+| `countess_boss.png` | COUNTESS_BOSS | Countess boss area |
+
+**Nodes in pather.py:** 1000-1004 (placeholder coords `(0,0)` - update after recording)
+
+---
+
+## 2. ANDARIEL (Act 1 Catacombs)
+
+**game.ini keys to record (3):**
+
+| Key | What to Record |
+|---|---|
+| `a1_andy_level3_enter` | Catacombs L3 entrance (from L2) |
+| `a1_andy_level4_enter` | Catacombs L4 entrance (from L3) |
+| `a1_andy_safe_dist` | Position near Andariel's cage at safe hammer range |
+
+**Templates for walking fallback (in `assets/templates/andariel/`):**
+
+| Template File | Template Key | Purpose |
+|---|---|---|
+| `andy_l3_stairs.png` | ANDY_L3_STAIRS | Stairs from L2 to L3 |
+| `andy_l4_stairs.png` | ANDY_L4_STAIRS | Stairs from L3 to L4 |
+| `andy_cage.png` | ANDY_CAGE | Andariel cage area |
+
+**Nodes in pather.py:** 1010-1012 (placeholder coords `(0,0)` - update after recording)
+
+---
+
+## 3. MEPHISTO (Act 3 Durance of Hate)
+
+**game.ini keys to record (2):**
+
+| Key | What to Record |
+|---|---|
+| `a3_meph_level3_enter` | Durance of Hate L3 entrance (from L2) |
+| `a3_meph_safe_dist` | Position near Mephisto's cage at safe hammer range |
+
+**Templates for walking fallback (in `assets/templates/mephisto/`):**
+
+| Template File | Template Key | Purpose |
+|---|---|---|
+| `meph_l3_stairs.png` | MEPH_L3_STAIRS | Stairs from L2 to L3 |
+| `meph_cage.png` | MEPH_CAGE | Mephisto cage area |
+
+**Nodes in pather.py:** 1020-1021 (placeholder coords `(0,0)` - update after recording)
+
+---
+
+## 4. BAAL (Act 5 Throne of Destruction)
+
+**game.ini keys to record (2):**
+
+| Key | What to Record |
+|---|---|
+| `a5_baal_throne_entry` | Throne of Destruction entrance (from Worldstone Keep L2) |
+| `a5_baal_safe_dist` | Position near Baal at safe hammer range (after wave clear) |
+
+**Templates for walking fallback (in `assets/templates/baal/`):**
+
+| Template File | Template Key | Purpose |
+|---|---|---|
+| `baal_throne_entry.png` | BAAL_THRONE_ENTRY | Entrance to Throne area |
+| `baal_arena.png` | BAAL_ARENA | Baal arena / boss position |
+
+**Nodes in pather.py:** 1030-1031 (placeholder coords `(0,0)` - update after recording)
+
+---
+
+## Quick Start: Record Paths for One Run
+
+1. Launch D2R and navigate to the boss area
+2. Run `python src/utils/node_recorder.py` from the botty directory
+3. Enter the run name when prompted (e.g., `countess`)
+4. Position your cursor at the target location in-game
+5. Press F8 to capture a template, F9 to record a node
+6. Press F10 to update all nodes with visible templates
+7. Copy the generated path coordinates to `config/game.ini`
+8. Copy the generated template PNGs to `assets/templates/<run>/`
+9. Update `pather.py` node definitions with real coordinates from the recorder output

docs/codex_analysis.md

@@ -0,0 +1,539 @@
+# Codex Fix Analysis
+
+Source files read:
+
+- `docs/fix_plan.md`
+- `src/run/nihlathak.py`
+- `src/town/town_manager.py`
+- `src/inventory/vendor.py`
+- `src/char/i_char.py` (`src/char.py` does not exist in this repo; CTA is implemented here)
+- `config/game.ini`
+- Supporting files needed to trace the failures: `src/town/a1.py`, `src/town/a5.py`, `src/town/a4.py`, `src/npc_manager.py`, `src/pather.py`, and the CTA key section in `config/params.ini`
+
+## Priority 1: Nihlathak approach fails
+
+### Finding
+
+The Nihlathak route has three brittle points:
+
+1. `approach()` returns success immediately after clicking the waypoint and never verifies that the Halls of Pain actually loaded.
+2. Level 1 layout detection has no fallback if `NI1_A`, `NI1_B`, or `NI1_C` is stale.
+3. `traverse_nodes_fixed()` always returns `True`, so a bad static path in `config/game.ini` cannot be detected until the stairs click times out.
+
+The `config/game.ini` Nihlathak path keys are present:
+
+```ini
+ni1_a=871,472, 1205,600, 1162,600, 1169,584, 1169,584, 1232,213, 1221,237, 1164,228, 1145,572, 1146,547, 1223,185
+ni1_b=23,187, 23,187, 23,187, 23,187, 12,192, 10,192, 10,190, 123,70, 378,120
+ni1_c=118,500, 158,602, 187,577, 217,563, 184,551, 70,413, 127,240, 154,493, 197,504, 218,545, 83,246, 45,526, 300,380
+```
+
+So the immediate code fix is not "add missing keys"; it is to verify waypoint/area entry and reduce the failure blast radius when stale templates or coordinates are encountered.
+
+### Exact code that needs to change
+
+`src/run/nihlathak.py`:
+
+```python
+        wait(0.4)
+        if waypoint.use_wp("Halls of Pain"): # use Halls of Pain Waypoint (5th in A5)
+            return Location.A5_NIHLATHAK_START
+        return False
+```
+
+```python
+        template_match = template_finder.search_and_wait(["NI1_A", "NI1_B", "NI1_C"], threshold=0.65, timeout=20)
+        if not template_match.valid:
+            return False
+```
+
+```python
+        self._pather.traverse_nodes_fixed(template_match.name.lower(), self._char)
+```
+
+### Proposed fix
+
+Replace the waypoint block with a verified load:
+
+```python
+        wait(0.4)
+        if not waypoint.use_wp("Halls of Pain"):
+            return False
+        if not template_finder.search_and_wait(["NI1_A", "NI1_B", "NI1_C"], threshold=0.55, timeout=8).valid:
+            Logger.error("Nihlathak approach: waypoint click did not land in Halls of Pain")
+            return False
+        return Location.A5_NIHLATHAK_START
+```
+
+Replace layout detection and static path traversal with a lower-threshold retry and an explicit path result check:
+
+```python
+        template_match = template_finder.search_and_wait(["NI1_A", "NI1_B", "NI1_C"], threshold=0.65, timeout=20)
+        if not template_match.valid:
+            Logger.warning("Nihlathak: strict NI1 layout detection failed, retrying with grayscale/lower threshold")
+            template_match = template_finder.search_and_wait(
+                ["NI1_A", "NI1_B", "NI1_C"],
+                threshold=0.55,
+                best_match=True,
+                timeout=6,
+                use_grayscale=True,
+            )
+        if not template_match.valid:
+            return False
+```
+
+```python
+        if not self._pather.traverse_nodes_fixed(template_match.name.lower(), self._char):
+            Logger.error(f"Nihlathak: failed static route {template_match.name.lower()}")
+            return False
+```
+
+If `search_and_wait()` does not support `use_grayscale` in this repo version, use this compatible form instead:
+
+```python
+        if not template_match.valid:
+            start = time.time()
+            while time.time() - start < 6:
+                template_match = template_finder.search(
+                    ["NI1_A", "NI1_B", "NI1_C"],
+                    grab(),
+                    threshold=0.55,
+                    best_match=True,
+                    use_grayscale=True,
+                )
+                if template_match.valid:
+                    break
+                wait(0.2)
+```
+
+That compatible form also needs imports:
+
+```python
+import time
+from screen import grab, convert_abs_to_monitor
+```
+
+### Why it will work
+
+This turns the approach from "clicked the waypoint, assume success" into "clicked the waypoint, confirm an NI1 layout is visible." If the waypoint interaction fails or lands somewhere unexpected, the run fails immediately instead of burning 600+ seconds.
+
+The lower-threshold/grayscale retry handles the likely stale-template case without permanently weakening the first pass. The strict threshold still wins when templates are good; the fallback only runs when the current behavior would fail.
+
+The static route check makes future changes safer. `traverse_nodes_fixed()` currently returns `True`, but guarding the call is still correct because it protects this runner if path traversal later gains real validation.
+
+Fresh templates and re-recorded `ni1_*` coordinates are still required if the fallback logs low-confidence matches or reaches the wrong stairs side. The code fix limits total game loss and gives a useful failure point.
+
+## Priority 2: Vendor trade button not found
+
+### Finding
+
+The failure is not in `src/inventory/vendor.py`; that file buys items after the vendor panel is already open. The failing log comes from `src/npc_manager.py`:
+
+```python
+Logger.error(f"Could not find {action_btn_key} btn. Should not happen! Continue...")
+```
+
+The current town code also returns a vendor location even if pressing the trade button failed. For A5 Malah:
+
+```python
+    def open_trade_menu(self, curr_loc: Location) -> Location | bool:
+        if not self._pather.traverse_nodes((curr_loc, Location.A5_MALAH), self._char, force_move=True): return False
+        if open_npc_menu(Npc.MALAH):
+            press_npc_btn(Npc.MALAH, "trade")
+            return Location.A5_MALAH
+        return False
+```
+
+And `press_npc_btn()` does not return `True` on success:
+
+```python
+    if res.valid:
+        mouse.move(*res.center_monitor, randomize=3, delay_factor=[1.0, 1.5])
+        wait(0.2, 0.4)
+        mouse.click(button="left")
+        wait(0.04, 0.08)
+        center_mouse()
+    else:
+        Logger.error(f"Could not find {action_btn_key} btn. Should not happen! Continue...")
+        keyboard.send("esc")
+```
+
+The template threshold is also hard-coded very high for white and blue text:
+
+```python
+        filtered_inp_w, 0.85, roi=Config().ui_roi["cut_skill_bar"]
+```
+
+### Exact code that needs to change
+
+`src/npc_manager.py`, `press_npc_btn()` needs to return a boolean and use a retry/fallback. A5/A1/A4 trade functions need to check that boolean or verify the vendor panel.
+
+### Proposed fix
+
+Replace `press_npc_btn()` with:
+
+```python
+def press_npc_btn(npc_key: Npc, action_btn_key: str) -> bool:
+    global npcs
+    for threshold in (0.85, 0.78):
+        img = grab()
+        img = escape_dialogue(img)
+        _, filtered_inp_w = color_filter(img, Config().colors["white"])
+        res = template_finder.search(
+            npcs[npc_key]["action_btns"][action_btn_key]["white"],
+            filtered_inp_w,
+            threshold,
+            roi=Config().ui_roi["cut_skill_bar"],
+        )
+        if not res.valid and "blue" in npcs[npc_key]["action_btns"][action_btn_key]:
+            _, filtered_inp_b = color_filter(img, Config().colors["blue"])
+            res = template_finder.search(
+                npcs[npc_key]["action_btns"][action_btn_key]["blue"],
+                filtered_inp_b,
+                threshold,
+                roi=Config().ui_roi["cut_skill_bar"],
+            )
+        if not res.valid:
+            res = template_finder.search(
+                npcs[npc_key]["action_btns"][action_btn_key]["white"],
+                img,
+                threshold,
+                roi=Config().ui_roi["cut_skill_bar"],
+                use_grayscale=True,
+            )
+        if res.valid:
+            mouse.move(*res.center_monitor, randomize=3, delay_factor=[1.0, 1.5])
+            wait(0.2, 0.4)
+            mouse.click(button="left")
+            wait(0.2, 0.3)
+            center_mouse()
+            return True
+
+    if "red" in npcs[npc_key]["action_btns"][action_btn_key]:
+        img = grab()
+        _, filtered_inp_r = color_filter(img, Config().colors["red"])
+        res = template_finder.search(
+            npcs[npc_key]["action_btns"][action_btn_key]["red"],
+            filtered_inp_r,
+            0.78,
+            roi=Config().ui_roi["cut_skill_bar"],
+        )
+        if res.valid:
+            Logger.warning(f"Cannot afford {action_btn_key} (red button detected). Skipping...")
+            keyboard.send("esc")
+            wait(0.3)
+            return False
+
+    Logger.error(f"Could not find {action_btn_key} btn. Should not happen! Continue...")
+    keyboard.send("esc")
+    return False
+```
+
+Then change A5 Malah trade from:
+
+```python
+        if open_npc_menu(Npc.MALAH):
+            press_npc_btn(Npc.MALAH, "trade")
+            return Location.A5_MALAH
+        return False
+```
+
+to:
+
+```python
+        if open_npc_menu(Npc.MALAH):
+            if press_npc_btn(Npc.MALAH, "trade") and is_visible(ScreenObjects.GoldBtnVendor):
+                return Location.A5_MALAH
+        return False
+```
+
+Make the same pattern in `src/town/a1.py`:
+
+```python
+        if open_npc_menu(Npc.AKARA):
+            if press_npc_btn(Npc.AKARA, "trade") and is_visible(ScreenObjects.GoldBtnVendor):
+                return Location.A1_AKARA
+        return False
+```
+
+and in `src/town/a4.py`:
+
+```python
+        if open_npc_menu(Npc.JAMELLA):
+            if press_npc_btn(Npc.JAMELLA, "trade") and is_visible(ScreenObjects.GoldBtnVendor):
+                return Location.A4_JAMELLA
+        return False
+```
+
+### Why it will work
+
+The bot currently proceeds as if trade opened even when the button was not clicked. Returning `False` stops `TownManager.buy_consumables()` at the correct point:
+
+```python
+new_loc = self._acts[curr_act].open_trade_menu(curr_loc)
+if not (new_loc and common.wait_for_left_inventory()): return False, items
+```
+
+The fallback search keeps the current exact template behavior first, then retries with a slightly lower threshold and grayscale. That covers text color/anti-aliasing differences without making every match permissive.
+
+Verifying `ScreenObjects.GoldBtnVendor` makes the action result state-based. Even if the template click returns true, the caller only continues when the vendor panel is actually open.
+
+Fresh `TRADE` / `TRADE_BLUE` templates are still recommended, but this code fix prevents false success and reduces sensitivity to minor UI rendering differences.
+
+## Priority 3: Stash detection fails
+
+### Finding
+
+The stash failures are in act-specific methods, not in `TownManager.stash()` itself. A1 and A5 use default `IChar.select_by_template()` threshold `0.68`:
+
+```python
+if not self._char.select_by_template(["A1_TOWN_0"], stash_is_open_func):
+    return False
+```
+
+```python
+if not self._char.select_by_template(["A5_STASH", "A5_STASH_2"], stash_is_open_func, telekinesis=True):
+    return False
+```
+
+The common selector only closes the waypoint menu for A5 stash templates:
+
+```python
+if type(template_type) == list and "A5_STASH" in template_type:
+    # sometimes waypoint is opened and stash not found because of that, check for that
+    if is_visible(ScreenObjects.WaypointLabel):
+        keyboard.send("esc")
+```
+
+So A1 stash can fail when a waypoint/dialog is left open, and both A1/A5 have no lower-threshold retry.
+
+### Exact code that needs to change
+
+`src/town/a1.py`:
+
+```python
+        if not self._char.select_by_template(["A1_TOWN_0"], stash_is_open_func):
+            return False
+```
+
+`src/town/a5.py`:
+
+```python
+        if not self._char.select_by_template(["A5_STASH", "A5_STASH_2"], stash_is_open_func, telekinesis=True):
+            return False
+```
+
+`src/char/i_char.py`:
+
+```python
+        if type(template_type) == list and "A5_STASH" in template_type:
+            # sometimes waypoint is opened and stash not found because of that, check for that
+            if is_visible(ScreenObjects.WaypointLabel):
+                keyboard.send("esc")
+```
+
+### Proposed fix
+
+Change the selector guard in `src/char/i_char.py` to handle all stash templates:
+
+```python
+        templates = template_type if isinstance(template_type, list) else [template_type]
+        if any(template in ["A1_TOWN_0", "A5_STASH", "A5_STASH_2"] for template in templates):
+            # sometimes waypoint is opened and stash not found because of that, check for that
+            if is_visible(ScreenObjects.WaypointLabel):
+                keyboard.send("esc")
+                wait(0.2, 0.3)
+```
+
+Change A1 stash to retry lower after the default threshold fails:
+
+```python
+        if not self._char.select_by_template(["A1_TOWN_0"], stash_is_open_func, threshold=0.68, timeout=4.0):
+            Logger.warning("A1 stash: default threshold failed, retrying with lower threshold")
+            if not self._char.select_by_template(["A1_TOWN_0"], stash_is_open_func, threshold=0.58, timeout=4.0):
+                return False
+```
+
+Change A5 stash similarly:
+
+```python
+        if not self._char.select_by_template(["A5_STASH", "A5_STASH_2"], stash_is_open_func, threshold=0.68, timeout=4.0, telekinesis=True):
+            Logger.warning("A5 stash: default threshold failed, retrying with lower threshold")
+            if not self._char.select_by_template(["A5_STASH", "A5_STASH_2"], stash_is_open_func, threshold=0.58, timeout=4.0, telekinesis=True):
+                return False
+```
+
+### Why it will work
+
+The default threshold remains unchanged for normal cases. The lower threshold only runs after a specific stash attempt fails, which narrows the risk of false-positive clicks.
+
+Closing the waypoint menu for A1 and A5 prevents stale UI overlays from blocking the stash templates. This directly addresses the logged sequence where town/stash templates are not found after other town interactions.
+
+The success function already checks for stash/inventory gold buttons:
+
+```python
+found = is_visible(ScreenObjects.GoldBtnInventory, img)
+found |= is_visible(ScreenObjects.GoldBtnStash, img)
+```
+
+That means a lower-threshold click must still produce the actual stash UI to count as success.
+
+Fresh `A1_TOWN_0`, `A5_STASH`, and `A5_STASH_2` templates should still be captured if logs continue to show low match confidence. The code change makes the current templates less brittle and prevents open UI overlays from causing avoidable failures.
+
+## Priority 4: CTA weapon switch fails
+
+### Finding
+
+The CTA code is in `src/char/i_char.py`. It depends on `Config().char["weapon_switch"]`, which comes from `config/params.ini`, not `config/game.ini`:
+
+```ini
+weapon_switch=w
+battle_orders=f6
+battle_command=f5
+```
+
+The current CTA routine has two reliability problems:
+
+1. It invalidates active-skill cache implicitly by switching weapons but does not reset `_active_skill`.
+2. It verifies the switch back by comparing a screenshot of the previous right-skill icon. That can fail when the same skill exists on both swaps, when the icon is visually similar, or when the UI updates slightly late.
+
+Current code:
+
+```python
+        while time.time() - start < 4:
+            keyboard.send(Config().char["weapon_switch"])
+            wait(0.4, 0.45)
+            keyboard.send(Config().char["battle_command"])
+            wait(0.2, 0.3)
+            if skills.is_right_skill_selected(["BC", "BO"]):
+                switch_sucess = True
+                break
+            else:
+                Logger.warning("Failed to find Battle Command, swapping weapons again.")
+```
+
+```python
+        while time.time() - start < 4:
+            keyboard.send(Config().char["weapon_switch"])
+            wait(0.4, 0.45)
+            skill_after = cut_roi(grab(), Config().ui_roi["skill_right"])
+            _, max_val, _, _ = cv2.minMaxLoc(cv2.matchTemplate(skill_after, skill_before, cv2.TM_CCOEFF_NORMED))
+            if max_val > 0.8:
+                switch_sucess = True
+                break
+            else:
+                Logger.warning("Failed to switch weapon, try again")
+                wait(0.5)
+        return switch_sucess
+```
+
+### Exact code that needs to change
+
+Add a helper inside `IChar` and use it whenever the weapon switch key is sent in `_pre_buff_cta()`.
+
+### Proposed fix
+
+Add this method to `IChar`:
+
+```python
+    def _weapon_switch(self):
+        keyboard.send(Config().char["weapon_switch"])
+        self._set_active_skill("left", "")
+        self._set_active_skill("right", "")
+        wait(0.55, 0.65)
+```
+
+Change the first CTA-side check from:
+
+```python
+        if skills.is_right_skill_selected(["BC", "BO"]):
+            keyboard.send(Config().char["weapon_switch"])
+            wait(0.4, 0.45)
+```
+
+to:
+
+```python
+        if skills.is_right_skill_selected(["BC", "BO"]):
+            self._weapon_switch()
+```
+
+Change the switch-to-CTA loop from:
+
+```python
+            keyboard.send(Config().char["weapon_switch"])
+            wait(0.4, 0.45)
+            keyboard.send(Config().char["battle_command"])
+```
+
+to:
+
+```python
+            self._weapon_switch()
+            keyboard.send(Config().char["battle_command"])
+```
+
+Change the switch-back loop from:
+
+```python
+            keyboard.send(Config().char["weapon_switch"])
+            wait(0.4, 0.45)
+            skill_after = cut_roi(grab(), Config().ui_roi["skill_right"])
+            _, max_val, _, _ = cv2.minMaxLoc(cv2.matchTemplate(skill_after, skill_before, cv2.TM_CCOEFF_NORMED))
+            if max_val > 0.8:
+                switch_sucess = True
+                break
+            else:
+                Logger.warning("Failed to switch weapon, try again")
+                wait(0.5)
+```
+
+to:
+
+```python
+            self._weapon_switch()
+            if not skills.is_right_skill_selected(["BC", "BO"]):
+                switch_sucess = True
+                break
+            skill_after = cut_roi(grab(), Config().ui_roi["skill_right"])
+            _, max_val, _, _ = cv2.minMaxLoc(cv2.matchTemplate(skill_after, skill_before, cv2.TM_CCOEFF_NORMED))
+            if max_val > 0.8:
+                switch_sucess = True
+                break
+            Logger.warning("Failed to switch weapon, try again")
+            wait(0.5)
+```
+
+Also fix the spelling while touching this code:
+
+```python
+switch_success = False
+```
+
+instead of:
+
+```python
+switch_sucess = False
+```
+
+### Why it will work
+
+Resetting `_active_skill` after weapon swap prevents `_select_skill()` from skipping a hotkey press because it thinks the old right skill is still selected. Weapon swap changes the available skill bar, so the cache is no longer trustworthy.
+
+The longer wait gives D2R more time to update the skill icon and weapon state before validation. The current 0.4 second delay is close to the UI transition timing and is likely why the failure is intermittent.
+
+The switch-back validation now accepts the most direct state: the right skill is no longer Battle Command/Battle Orders. The old image comparison remains as a fallback, so this still handles cases where BC/BO detection briefly lags.
+
+The configured key should be checked in `config/params.ini`, not `config/game.ini`. If the user's actual D2R weapon switch key is Caps Lock, then this line must change:
+
+```ini
+weapon_switch=w
+```
+
+to:
+
+```ini
+weapon_switch=capslock
+```
+
+Only do that if the in-game key binding is actually Caps Lock; otherwise leave it as `w`.

docs/d2jsp_scraping_guide.md

@@ -0,0 +1,57 @@
+# d2jsp Semi-Automated Scraping Guide
+
+Due to Cloudflare's aggressive bot protection, fully automated scraping is currently restricted. This project uses a **Semi-Automated (Offline) Workflow** that leverages your authenticated browser session to safely collect market data.
+
+## Prerequisites
+
+1.  **Python Dependencies:** Ensure `keyboard` and `requests` are installed (included in `requirements.txt`).
+2.  **Browser:** Google Chrome is recommended.
+3.  **Authentication:** Be logged into [forums.d2jsp.org](https://forums.d2jsp.org/) in your browser.
+
+---
+
+## The Workflow
+
+### 1. Initial Directory Setup
+Before running the automation, you must set the default "Save As" path in your browser:
+1.  Open any topic on d2jsp.
+2.  Press `Ctrl + S`.
+3.  Navigate to your bot folder: `data/d2jsp_pages/`.
+4.  Save the file. Your browser will now remember this location as the default.
+
+### 2. Collect Topic URLs
+If you don't have a fresh list of URLs, run the collector on a saved forum listing page:
+```powershell
+python tools/d2jsp_topic_collector.py --offline-dir data/d2jsp_pages --out data/d2jsp_topic_urls.txt
+```
+
+### 3. Run Browser Automation
+This script will open the first 20 topics from your list, wait for Cloudflare to pass, and simulate the save command.
+```powershell
+python tools/browser_auto_save.py
+```
+**Important:** 
+*   Keep your browser as the active window.
+*   Do not move the mouse or type while the script is running.
+*   It will automatically `Ctrl + S` -> `Enter` -> `Ctrl + W` for each tab.
+
+### 4. Generate Price Estimates
+Once the HTML files are saved in `data/d2jsp_pages/`, run the offline scraper to update your bot's configuration:
+```powershell
+python tools/fg_market_scraper.py --ladder-start-date 2026-05-20 --offline-dir data/d2jsp_pages --out config/fg_daily_estimates.json
+```
+
+---
+
+## Troubleshooting
+
+### Cloudflare "Just a Moment" Loop
+If the automation is too fast and hits a "Just a Moment" screen that doesn't resolve:
+1.  Increase the `time.sleep(8)` value in `tools/browser_auto_save.py`.
+2.  Manually solve one challenge in the browser to "warm up" the IP clearance.
+
+### Files Not Saving to Correct Folder
+If files are saving to your "Downloads" folder instead of `data/d2jsp_pages`, the browser's default path was reset. Repeat **Step 1** to fix it.
+
+### Date Parsing Errors
+If the scraper reports 0 topics scanned or dates are missing, ensure your browser language isn't translating the page, as the scraper expects English month names (Jan, Feb, Mar, etc.).

docs/d2r_window_input_troubleshooting.md

@@ -0,0 +1,18 @@
+# D2R Window and Input Troubleshooting
+
+Botty expects Diablo II: Resurrected to expose a stable 1280x720 client area. If
+the live D2R window reports a slightly different client size, template matching
+can still succeed while mouse clicks land at the wrong monitor coordinate.
+
+At startup and before creating a game, Botty now resizes and positions the D2R
+client area to the configured `config/game.ini` dimensions. The expected size is
+`1280x720`, matching `assets/d2r_settings.json` and the template assets.
+
+If a UI click is detected but the cursor does not visibly move, the native input
+layer first tries `SendInput` and then verifies the cursor position. When
+`SendInput` misses the target coordinate, Botty falls back to `SetCursorPos` and
+logs the fallback.
+
+If movement or clicks still do not reach D2R, check that D2R and the bot process
+are running at the same privilege level. Windows can block input from a
+non-admin process into an elevated game window.

docs/diablo_wp_recovery.md

@@ -0,0 +1,41 @@
+# Diablo Waypoint Recovery Notes
+
+Date: 2026-06-07
+
+## What changed
+
+- Diablo is now first in the configured run order, followed by Pindle only.
+- After using the Act 4 River of Flame waypoint, the Diablo approach now closes the waypoint panel, resets stale health-manager panel detections, and verifies that River of Flame templates are visible.
+- If the first River of Flame waypoint attempt leaves the bot in town, the approach retries from the Act 4 town start before failing the run.
+- River of Flame and Pentagram traversal now include template-based verification before continuing into Chaos Sanctuary logic.
+- Diablo now has a reusable `_search_and_log()` wrapper for template searches that can report match confidence during route debugging.
+- Health manager reset now clears `_count_panel_detects` so detections from a previous game do not immediately chicken the next game.
+- `config/fg_daily_estimates.json` was regenerated with the improved offline estimator output, including skipped-topic counts and trimmed price statistics.
+
+## Why
+
+Same-act waypoint use can leave the waypoint panel open without a loading screen. That stale panel could combine with the CTA weapon-swap panel during pre-buffing and trigger the health manager panel-detection chicken path. The bot could also silently remain in town after a missed waypoint click and continue as if it had reached River of Flame.
+
+The new checks make Diablo startup state-based: the bot confirms River of Flame and Pentagram markers before continuing. Failed waypoint transitions are retried once, then reported as approach failures instead of drifting into later pathing.
+
+## Validation
+
+Run a focused route order:
+
+```ini
+order=run_diablo, run_pindle
+```
+
+Expected log behavior:
+
+- `_verify_in_rof: confirmed in River of Flame ...` after the Act 4 waypoint.
+- `CS: Calibrated at PENTAGRAM` after Pentagram traversal.
+- No immediate chicken caused by the waypoint panel plus CTA weapon-swap panel sequence.
+
+If River of Flame or Pentagram verification fails repeatedly, refresh the affected templates:
+
+- `DIABLO_ROF_WP_0`
+- `DIABLO_ROF_WP_1`
+- `DIABLO_ENTRANCE_50` through `DIABLO_ENTRANCE_55`
+- `DIA_NEW_PENT_TP`
+- `DIA_NEW_PENT_0` through `DIA_NEW_PENT_2`

docs/fg_market_scraper.md

@@ -0,0 +1,44 @@
+# FG Market Scraper (Day 1-14)
+
+This tool estimates FG prices per ladder day by scraping public trade topics from:
+- https://forums.d2jsp.org/forum.php?f=271
+
+## Script
+- `tools/fg_market_scraper.py`
+
+## What it does
+1. Scans forum listing pages for topic links.
+2. Fetches topic pages (rate-limited).
+3. Detects known item/rune keywords.
+4. Extracts `fg` prices from post text.
+5. Buckets prices by ladder day index (Day 1..Day N).
+6. Writes median estimates + sample counts.
+
+## Usage
+From repo root:
+
+```powershell
+python tools/fg_market_scraper.py --ladder-start-date 2026-05-23 --days 14
+```
+
+Output:
+- `config/fg_daily_estimates.json`
+
+## Tuning
+- `--max-forum-pages`: listing pages to scan (default 40)
+- `--max-topics`: hard cap on fetched topics (default 800)
+- `--delay-s`: delay between requests (default 0.35s)
+
+Example heavier run:
+
+```powershell
+python tools/fg_market_scraper.py --ladder-start-date 2026-05-23 --days 14 --max-forum-pages 120 --max-topics 2400 --delay-s 0.5
+```
+
+## Notes
+- This is a heuristic estimator, not a full market engine.
+- Accuracy depends on post format quality and keyword matches.
+- Keep request rate polite to avoid stressing the forum.
+- Some environments/IPs will receive HTTP `403` from d2jsp. In that case use:
+  - `config/fg_day_estimates.json` (Day 1-14 estimated multiplier model from Day 3 snapshot),
+  - and update `config/fg_prices.json` manually from your current market sample.

docs/fix_plan.md

@@ -0,0 +1,91 @@
+# Botty Fix Plan — 2026-06-05
+
+## Symptoms (from today's logs)
+
+- 4 sessions today: 31 + 1 + 9 + 6 games = 47 games total
+- 51 deaths across all sessions
+- 0 valuable items found (only charms, jewels, gold)
+- Nihlathak run fails every time, ending entire games
+- Vendor trade button never found (can't buy potions)
+- Stash never opens (can't stash items)
+- CTA weapon switch fails occasionally
+
+---
+
+## Priority 1: Nihlathak approach fails (game-ending)
+
+**Error:** `Approach failed for run_nihlathak` — ends game after 600-650s
+**Impact:** 3+ games ended per session
+
+**Root cause:** Teleport activation times out, bot can't reach Nihlathak
+**Fix:**
+- Check `config/game.ini` for Nihlathak coordinates (`a5_nihlathak_*`)
+- Likely missing or stale path nodes
+- Re-record path with `node_recorder.py`
+- May need fresh template images for Nihlathak area
+
+**Files to check:**
+- `config/game.ini` (Nihlathak section)
+- `src/run/nihlathak.py`
+- `assets/templates/nihlathak/`
+
+---
+
+## Priority 2: Vendor trade button not found
+
+**Error:** `Could not find trade btn. Should not happen!`
+**Impact:** Can't buy potions every town return
+
+**Root cause:** Trade button template outdated or offset wrong for Hell difficulty UI
+**Fix:**
+- Take fresh screenshot of vendor trade window
+- Update trade button template in `assets/templates/`
+- Check if trade button position shifted between Normal/Nightmare/Hell
+
+**Files to check:**
+- `assets/templates/` (trade button template)
+- `src/` (vendor interaction code — search for "trade btn")
+
+---
+
+## Priority 3: Stash detection fails
+
+**Error:** `select_by_template: could not find ['A1_TOWN_0']` then `['A5_STASH', 'A5_STASH_2']`
+**Impact:** Can't stash items, inventory fills with junk
+
+**Root cause:** Stash template confidence threshold too high or template outdated
+**Fix:**
+- Take fresh screenshot of stash UI in Hell difficulty
+- Update A1_TOWN_0, A5_STASH, A5_STASH_2 templates
+- Consider lowering confidence threshold (currently 0.68)
+
+**Files to check:**
+- `assets/templates/a1_town/`
+- `assets/templates/a5_stash/`
+- Template matching threshold in code
+
+---
+
+## Priority 4: CTA weapon switch fails
+
+**Error:** `_pre_buff_cta: switch to CTA slot failed — retrying`
+**Impact:** Occasional, bot retries
+
+**Root cause:** Weapon switch key or template unreliable
+**Fix:**
+- Verify weapon switch key binding (capslock per memory)
+- Check CTA slot detection template
+- May need more robust retry logic
+
+**Files to check:**
+- `src/` (search for `_pre_buff_cta`)
+- `config/` (weapon_switch key)
+
+---
+
+## Approach
+
+All four issues are template/coordinate problems — stale screenshots or wrong positions.
+Fix pattern: screenshot current game UI at those locations, update templates.
+
+Do in order: Nihlathak > vendor > stash > CTA.

docs/gems_transmute_flow.md

@@ -0,0 +1,87 @@
+# GEMS-tab transmute flow (rewritten in commit 48d8445)
+
+`src/transmute/transmute.py` — `Transmute.convert_all_gems()` and helpers.
+
+## What changed and why
+
+The old flow drove gem upgrades through the **Horadric Cube UI**: it required the cube to live in
+the PERSONAL stash tab, opened the cube, ctrl+shift+right-clicked 3 gems into it, clicked the cube's
+Transmute button, then pulled the result back out. This was fragile — it depended on cube placement,
+cube-open detection (`ScreenObjects.CubeOpened` via `wait_until_visible`), and a pre-flight
+"empty the cube" pass.
+
+The rewrite uses D2R's **native GEMS-tab convert panel** instead of the cube. Gems are moved
+directly between the GEMS stack grid and the convert panel, and conversion uses the GEMS tab's own
+convert button. The cube is no longer required to be in the stash and, if present in the character
+inventory, is left untouched.
+
+## Fixed stack-coordinate grid
+
+`GEMS_TAB_STACK_COORDS` is a new module-level dict mapping every gem template
+(`INVENTORY_<FAMILY>_<TIER>`) to a fixed `(x, y)` **screen** coordinate at 1280×720. The grid is:
+
+- **Columns by family** (x): diamond 85, emerald 132, ruby 180, topaz 227, amethyst 275,
+  sapphire 322, skull 370.
+- **Rows by tier** (y): chipped 154, flawed 195, standard 234, flawless 273, perfect 311.
+
+Because gem stacks always render at these fixed slots on the GEMS tab, the flow can click them by
+coordinate instead of template-searching for them each time. `_gems_stack_monitor_for(template)`
+converts the screen coord to monitor coords (returns `None` if the template isn't in the grid, so
+the caller can fall back to a template search).
+
+## Panel constants (1280×720)
+
+| Constant | Value | Meaning |
+|----------|-------|---------|
+| `GEMS_TAB_X` | `223` | GEMS tab click X (was `200`) |
+| `GEMS_CONVERT_BUTTON` | `(225, 500)` | native convert button in the GEMS panel |
+| `GEMS_CONVERT_PANEL_ROI` | `(160, 296, 128, 160)` | ROI to search for the converted result |
+| `GEMS_CONVERT_FIRST_SLOT` | `(181, 318)` | fallback slot to grab the result if the search misses |
+
+## Per-transmute flow (new)
+
+For each gem type/tier with enough gems to convert:
+
+1. **Switch to GEMS tab.**
+2. **Load 3 gems** — `_ctrl_shift_left_click_monitor` the stack slot ×3 to move 3 gems into the
+   convert panel. If exactly 3 weren't loaded, abort that batch.
+3. **Convert** — `_click_gems_tab_convert_button()` clicks the panel's native convert button.
+4. **Recover the result** — template-search `gems_out` within `GEMS_CONVERT_PANEL_ROI`; if found,
+   ctrl+shift+left-click it back onto the GEMS tab. If not found, fall back to ctrl+shift+left-click
+   on `GEMS_CONVERT_FIRST_SLOT` (instead of the old behavior of clearing the cube and bailing).
+5. **Leave the inventory cube untouched** and return to the GEMS tab for the next iteration.
+
+The old pre-flight "open cube → empty 12 cube slots → reopen stash" pass and the per-iteration
+"close cube / reopen stash" steps are **removed**.
+
+## Helper changes
+
+- `_open_cube_from_stash()` / `_open_cube_from_gems_tab()` / `_open_available_cube_for_gems_tab()`
+  now operate from the **GEMS** tab rather than PERSONAL, and can open the cube from character
+  inventory (`_open_cube_from_inventory`) when the GEMS tab is active. These remain for older
+  inventory flows; the new gem flow doesn't need them.
+- `_locate_cube()` can now return `'opened'` (cube UI already open on the GEMS tab) in addition to
+  `'stash'` / `'inventory'`.
+- `_ensure_cube_available()` now treats `inventory` as usable ("it will stay there") instead of an
+  error — the cube no longer has to be moved to PERSONAL.
+- `_ensure_cube_in_stash()` dropped the loot-column safety guard: it now ctrl+clicks any positive
+  `CubeInventory` match, so the cube can be moved from reserved inventory columns too.
+- **Keyboard modifier handling** switched from the low-level
+  `win_input.key_down/key_up(_get_vk(...))` to `keyboard.send(..., do_release=False)` /
+  `keyboard.release(...)` wrapped in `try/finally`, so ctrl+shift are always released even if the
+  click raises. Applies to both `_ctrl_shift_click_monitor` (right) and
+  `_ctrl_shift_left_click_monitor` (left).
+- `_empty_cube_to_gems_tab()` now uses `_ctrl_click_monitor` (plain ctrl) rather than ctrl+shift.
+
+## Gem counting (`_count_gems_by_ocr`)
+
+Counting no longer template-searches the left-inventory ROI for each gem. It now reads each fixed
+slot from `GEMS_TAB_STACK_COORDS`: crop the slot icon, and **skip empty slots** by checking the
+95th-percentile grayscale brightness (`< 45` ⇒ no gem present). Occupied slots still OCR the count
+badge in the bottom-right quadrant. This is faster and immune to template-match drift, at the cost
+of depending on the fixed 1280×720 grid.
+
+## Resolution assumption
+
+The whole flow is hard-coded to **1280×720**. All coordinates above are screen coordinates at that
+resolution, converted to monitor coordinates at click time via `convert_screen_to_monitor`.

docs/hermes_qwen_takeover.md

@@ -0,0 +1,360 @@
+# Hermes Takeover Guide
+
+Practical handoff for continuing Botty development. Read every section before touching code — the trap patterns section alone will save you hours.
+
+---
+
+## 1) Working model
+
+- Treat Botty as a **state machine app** with side effects across D2R UI automation, OCR parsing, inventory/sell/stash routines, and messaging (Discord/webhooks).
+- Prefer **small, production-safe patches** over broad refactors.
+- Prioritize runtime stability: avoid failing full runs because one subsystem is flaky.
+- **Always reproduce from logs first** — most bugs leave a clear trail.
+
+---
+
+## 2) Daily workflow
+
+1. Get fresh logs:
+   - `log/log.txt`
+   - `log/stats/events_*.jsonl`
+   - `log/stats/stats_*.log`
+2. Confirm failure path in code with `rg`.
+3. Patch narrowly with targeted edits.
+4. Validate:
+   - `python -m compileall <changed files>`
+   - targeted pytest where available
+5. Commit with a clear, single-purpose message.
+
+---
+
+## 3) Core debug commands
+
+From repo root (PowerShell):
+
+```powershell
+# Tail errors/warnings
+rg -n "ERROR|WARNING|Failed|chicken|exception" log/log.txt | tail -50
+
+# Last 200 log lines
+Get-Content log/log.txt | Select-Object -Last 200
+
+# Search source code
+rg -n "<keyword>" src/
+
+# Compile check after edits
+python -m compileall src/health_manager.py src/item/pickit.py
+
+# Config smoke test (confirm a key's resolved value)
+python -c "import sys; sys.path.insert(0,'src'); from config import Config; print(Config().char['show_belt'])"
+
+# Run test suite
+python -m pytest -q test/
+```
+
+Use `rg` first. It's the fastest way to localize faults.
+
+---
+
+## 4) High-risk areas
+
+### 4.1 Repair/vendor flow
+
+- A5 Larzuk detection is noisy and can fail.
+- Current behavior: A5 normal flow → Larzuk direct template click → Act 4 Halbu fallback.
+- `repair_npc=a4_halbu` config key skips Larzuk entirely.
+
+Files: `src/town/a5.py`, `src/town/town_manager.py`, `src/bot.py`
+
+### 4.2 Discord messaging
+
+Known issue: `'NoneType' object has no attribute 'to_dict'`
+
+Current fix: `src/messages/discord_embeds.py` `_send_embed` only passes `file=` when attachment exists. Plain text fallback on embed failure.
+
+Config: `config/params.ini` → `[discord_events]`
+
+### 4.3 Selling safety
+
+Current protections:
+- Sell logs include item names (not coordinates only).
+- `protect_shields_from_sell=1` blocks selling items whose detected name includes "shield".
+
+Files: `src/inventory/personal.py`, `config/params.ini`, `src/config.py`
+
+### 4.4 XP logging
+
+Two separate concerns:
+1. OCR extraction: `src/ui/player_bar.py` — parser handles `I/l/|→1`, `O/o→0`.
+2. XP status projection math: `src/game_stats.py` `_create_msg()` — zero denominators guarded.
+
+### 4.5 HealthManager (rejuv / chicken logic) ⚠️
+
+The most subtle source of false positives. See Section 10.2 for full details.
+
+Key thresholds (params.ini `[char]`):
+- `take_rejuv_potion_health = 0.80` — drink rejuv if HP ≤ 80%
+- `take_rejuv_potion_mana = 0.55` — drink rejuv if mana ≤ 55%
+- `chicken = 0.50` — flee if HP ≤ 50%
+
+The "two juvs in 8s → chicken" check **must also verify HP was the trigger**, not just mana. Hammerdins burn mana fast; mana can legitimately trigger back-to-back rejuvs at full HP.
+
+### 4.6 PickIt gold loop ⚠️
+
+`_yoink_item()` **always returns `PickedUpResult.PickedUp`** regardless of actual success (line 129). This means pickup failures are only detectable through `_pick_up_item`'s same-ID or same-UID checks — and different gold pile amounts produce different IDs, bypassing those checks entirely.
+
+Fix in place: `pick_up_items()` now blacklists `item.ID` in `_cached_pickit_items` on `PickedUpFailed`. Do not remove this case.
+
+---
+
+## 5) State machine (bot.py)
+
+States: `initialization → hero_selection → town → [run state] → town → ...`
+
+Full state list: `initialization`, `hero_selection`, `town`, `level`, `pindle`, `shenk`, `trav`, `nihlathak`, `arcane`, `diablo`, `vizier`, `baal`, `mephisto`, `andariel`, `countess`
+
+Key transitions:
+| Trigger | Source | Dest | Handler |
+|---|---|---|---|
+| `init` | initialization | initialization | `on_init` |
+| `select_character` | initialization | hero_selection | `on_select_character` |
+| `start_from_town` | initialization/hero_selection | town | `on_start_from_town` |
+| `maintenance` | town | town | `on_maintenance` |
+| `run_pindle` | town | pindle | `on_run_pindle` |
+| `run_arcane` | town | arcane | `on_run_arcane` |
+| `end_run` | any run state | town | `on_end_run` |
+| `end_game` | town/any run state | initialization | `on_end_game` |
+
+**`on_maintenance` guard**: If `_curr_loc` is None (e.g. after a chicken recovery), defaults to `A1_TOWN_START` with a warning log. Do not remove this guard.
+
+**`end_game` vs `end_run`**: When TP charges run out, trigger `end_game` so the bot restarts and restocks in the next game — not `end_run` which tries to TP back and loops.
+
+---
+
+## 6) Config system
+
+Priority order (highest first):
+```
+custom.ini > params.ini > game.ini > shop.ini > transmute.ini
+```
+
+`custom.ini` is gitignored — user-only overrides. Never edit `game.ini` for user settings.
+
+### Key detection flow
+
+On every `Config()` instantiation:
+1. Reads `Saved Games/Diablo II Resurrected/<charname>.keyo` binary file.
+2. Parses slot-to-key mapping (`CHAR_BINDING_SLOTS` in `key_detector.py`).
+3. Slot 41 = `show_belt`, slot 36 = `stand_still`, slot 44 = `weapon_switch`, etc.
+4. Compares each detected key against params.ini value.
+5. If they match → use detected key silently.
+6. If they differ → logs `"Keeping configured key binding for X: 'params_val' (detected 'keyo_val')"` and keeps the params.ini value.
+
+**What "Keeping..." means**: params.ini disagrees with the in-game binding. Usually indicates a config typo or stale params.ini. If you see `Keeping show_belt: 'k' (detected 'n')`, the fix is `show_belt=n` in params.ini `[char]`.
+
+### Config singleton pattern
+
+`Config` uses `__new__` with a `data_loaded` class variable. Calling `Config()` multiple times in the same process returns the same loaded instance. After editing params.ini at runtime, `Config().reload()` is needed (or restart the bot).
+
+---
+
+## 7) Current bot profile (Fistman — as of 2026-06-05)
+
+- **Character**: Hammerdin (`src/char/paladin/hammerdin.py`)
+- **Runs**: Pindle + Arcane Sanctuary
+- **Key bindings** (from .keyo + params.ini):
+  - `show_belt = n` (belt hotkey)
+  - `stand_still = capslock` (overrides detected 'shift')
+  - `weapon_switch = w`
+  - `show_items = alt`
+  - `potion1..4 = 1,2,3,4`
+- **Capabilities**: `can_teleport_natively = True` (set via `override_capabilities` in `[advanced_options]`)
+- **Thresholds**: `take_rejuv_potion_health=0.80`, `take_rejuv_potion_mana=0.55`, `chicken=0.50`
+
+---
+
+## 8) Testing strategy by change type
+
+### Messaging changes
+```powershell
+python -m pytest -q test/test_discord_embeds.py
+```
+
+### Config parsing changes
+```powershell
+python -m compileall src/config.py
+python -c "import sys; sys.path.insert(0,'src'); from config import Config; c=Config(); print(c.char['show_belt'], c.char['stand_still'])"
+```
+
+### Inventory/sell logic
+- Validate no exceptions in `inspect_items` / `transfer_items`.
+- Log outputs should include item names (not just coordinates).
+- Prefer dry functional checks from logs before gameplay runs.
+
+### HealthManager changes
+- Check that chicken thresholds still fire at the right HP%.
+- Verify the two-rejuv check only triggers when `health_percentage <= take_rejuv_potion_health`.
+
+### PickIt changes
+- Confirm `_cached_pickit_items` is populated on both `PickedUp` (cached True) and `PickedUpFailed` (cached False).
+- Confirm `_yoink_item` return value is not relied on for success detection.
+
+---
+
+## 9) Git hygiene
+
+- Keep untracked: `.env`, `config/custom.ini`, `log/`, `log/screenshots/`
+- Do not revert unrelated user changes.
+- Commit frequently with single-purpose messages.
+- `python -m compileall src/` must pass cleanly before committing.
+
+---
+
+## 10) Module internals and trap patterns
+
+### 10.1 Config / key detection traps
+
+**Trap**: `apply_key_bindings` runs *after* `self.char` is built in `config.py`. If you add a new key to `self.char` dict and it doesn't exist in `CHAR_BINDING_SLOTS`, the keyo detector won't touch it — but the user still needs to have it in params.ini.
+
+**Trap**: "Keeping configured key binding" is NOT an error. It means the user explicitly configured something different from the game default. It becomes a problem only if the params.ini value is wrong (e.g., 'k' instead of 'n' for show_belt).
+
+**Trap**: `Config()` is a singleton via `__new__`. The first call loads everything. Subsequent calls within the same process return the cached instance. Do NOT expect param changes at runtime to be visible without `Config().reload()`.
+
+### 10.2 HealthManager rejuv traps
+
+The rejuv logic in `start_monitor()`:
+```python
+if last_drink > 0.60:  # minimum between rejuvs
+    if health <= take_rejuv_potion_health or mana <= take_rejuv_potion_mana:
+        drink_rejuv()
+        self._last_rejuv = time.time()
+        # Two juvs in 8 seconds → chicken ONLY if HP was the trigger
+        if last_drink < 8 and health_percentage <= Config().char["take_rejuv_potion_health"]:
+            self._do_chicken(img)
+```
+
+**Critical**: The `last_drink < 8` chicken check MUST also check `health_percentage`. Without it, any mana-triggered second rejuv (common for Hammerdins) will false-chicken at 99.9% HP. The fix is already in place — do not revert it.
+
+**Timing**: The monitor polls every `3/25s * jitter(±20%)` ≈ 96–144ms. At 25 FPS that's every 3 frames.
+
+**Thread safety**: `_pause_state` and `_panel_check_paused` are protected by `_state_lock`. Module-level `get_pause_state()` / `set_pause_state()` functions delegate to the singleton. Always use these functions from external code.
+
+### 10.3 PickIt ID/UID system
+
+`GroundItem` has two identifiers:
+
+```python
+ID  = slugify(f"{Name}_{'_'.join([str(v) for _,v in as_dict().items()])}")
+# Includes Amount in the string. Two gold piles with different amounts = different IDs.
+
+UID = f"{ID}_{'_'.join([str(v) for v in center])}"
+# ID + screen position. Same pile at same coordinates = same UID.
+```
+
+**Trap**: `_pick_up_item`'s gold-fail detection uses `item.ID == prev.ID`. If two nearby gold piles have different amounts (e.g., 338g and 157g), they alternate as the "next" item and each one's ID never matches the previous, so the same-ID fail check never fires. The loop runs until timeout (20s).
+
+**Trap**: `_yoink_item` ALWAYS returns `PickedUpResult.PickedUp`. It never returns `PickedUpFailed`. Pickup failures for teleport builds are silently swallowed.
+
+**Fix in place**: `pick_up_items()` match block now has:
+```python
+case PickedUpResult.PickedUpFailed:
+    self._cached_pickit_items[item.ID] = False  # blacklist this session
+```
+This stops the alternating-gold loop by blacklisting the item after the first confirmed failure.
+
+### 10.4 Belt system open() key chain
+
+`belt.open()` tries keys in this order:
+```python
+[config_val, "n", "k", "`", "~"]  # deduplicated
+```
+
+If `show_belt = n` in params.ini, the first key tried is 'n'. If it works, no fallback keys appear in logs. If you see `"Trying to open belt with key: k"` it means 'n' failed — check if `show_belt` is actually set to 'n' and if the D2R window is focused.
+
+### 10.5 TownManager location routing
+
+`get_act_from_location(loc)` returns `None` for non-string inputs (e.g., `True`, `False`). The isinstance guard at line 36 (`if not isinstance(loc, str): return None`) prevents `AttributeError: 'bool' object has no attribute 'upper'`. Do not remove it.
+
+All town methods that receive a `Location` return `False` (not `None`) on failure. Callers should check `if not new_loc` not `if new_loc is None`.
+
+### 10.6 State machine: `end_game` vs `end_run`
+
+`end_run` sends a TP, waits in town, does maintenance, then starts another run. If something prevents getting back to town (no TP scrolls, merc dead with no body, disconnected), `end_run` loops.
+
+`end_game` saves and exits, restarts the game fresh. Use it when:
+- TP charges = 0 (bot will restock on next game start)
+- Unrecoverable in-game state
+- Max consecutive failed runs reached
+
+Triggering `end_run` when TP is gone causes an infinite "No TP charges left, trying to walk back" loop (pre-fix behavior).
+
+### 10.7 distance calculation (processing_helpers.py)
+
+The y-center of the screen for distance math is `screen_height / 2`, NOT `screen_width / 2`. Using the wrong dimension skews distance sorting for items on the top/bottom half of the screen. Fix is already applied.
+
+---
+
+## 11) Bugs fixed in 2026-06-04/05 session
+
+All fixes were applied and verified by Python compile/config tests:
+
+| Bug | File | Symptom in logs | Fix |
+|---|---|---|---|
+| `show_belt` wrong key (`n` instead of `k`) | `config/params.ini` | "Recovered belt hotkey using 'k'" on first game, then silent in-memory mutation | `show_belt=n` → `show_belt=k` |
+| `AttributeError: 'bool' object has no attribute 'upper'` | `src/town/town_manager.py:36` | Crash in `get_act_from_location` when `True`/`False` passed as loc | Added `isinstance(loc, str)` guard |
+| No-TP → infinite loop | `src/bot.py` | "No TP charges left, trying to walk back" repeated forever | Trigger `end_game` instead of `end_run` on zero TP |
+| Distance y-axis wrong | `src/d2r_image/processing_helpers.py` | Items sorted by wrong distance; far items picked first | `screen_width/2` → `screen_height/2` for y |
+| `on_maintenance` crash with no location | `src/bot.py` | Crash after chicken recovery when `_curr_loc=None` | Guard: default to `A1_TOWN_START` if None |
+| False-positive chicken on mana rejuv | `src/health_manager.py:133` | "Two juvs drank within 0.63s. Chicken, HP 99.9%!" | Added `and health_percentage <= take_rejuv_potion_health` |
+| Gold pickup infinite loop | `src/item/pickit.py:241` | 338g/157g alternating in logs for 20s | Added `PickedUpFailed` case to blacklist `item.ID` |
+| Health pots sold when needed | `src/inventory/personal.py:351` | "Discarding SUPER HEALING POTION." + "Confirmed sell SUPER HEALING POTION" despite health needs | Check `get_needs()` before dropping consumable; `continue` to skip sell/drop when pot is needed |
+| No fill_from_inventory after failed buy | `src/bot.py` (after line 438) | Belt empty all game despite pots sitting in inventory; "Out of gold" then nothing fills belt | After buy_consumables block, call `fill_up_belt_from_inventory` + `update_pot_needs` when needs > 0 |
+| Wrong weapon in combat after chicken mid-buff | `src/char/i_char.py` `_pre_buff_cta` | Character dies immediately; dies with CTA flail/shield instead of main weapon | Added BC skill-bar template verification after each `weapon_switch`; corrects slot if wrong at game start; retries once on failure |
+
+---
+
+## 12) Known pending issues (as of 2026-06-05)
+
+- **C10** (IMPROVEMENTS.md): `kill_thread()` uses `PyThreadState_SetAsyncExc` — can leave locks inconsistent. Replace with `threading.Event` cooperative shutdown. High risk.
+- **optipng pass on assets/**: Pending. Run `asset_manager.py batch` or `optipng -o7` on all PNGs.
+- **Thread safety** (H14 in IMPROVEMENTS.md): `health_manager` and `death_manager` shared state — Lock is now present in HealthManager but verify all paths use it.
+- **PickedUpResult enum gap** (M14): Values are 0,1,3,4,5. Value 2 is missing. Non-critical but confusing.
+- **Gold vicious cycle**: Low gold → can't buy pots → health empty → more chickens/deaths → less gold. Monitor runs after the personal.py + bot.py fix — if the cycle still triggers, also check that `inspect_items` isn't being called with vendor_open=True before `fill_up_belt_from_inventory`.
+
+---
+
+## 13) Fast triage mapping
+
+| Symptom in logs | Where to look | Likely cause |
+|---|---|---|
+| "Recovered belt hotkey using 'k'" on game 1, then silent | `config/params.ini` | `show_belt=n` should be `show_belt=k` |
+| "Two juvs drank... Chicken" at HP > 80% | `src/health_manager.py:133` | Missing HP check on two-rejuv condition |
+| Gold pile (XYZg) repeating 5+ times | `src/item/pickit.py:241` | `PickedUpFailed` case missing; item not blacklisted |
+| "Failed to pick up X" then same X again immediately | `_yoink_item` / `_cached_pickit_items` | Blacklist not being set on failure |
+| `AttributeError: 'bool' object has no attribute 'upper'` | `src/town/town_manager.py:36` | isinstance guard removed or bypassed |
+| "No TP charges left, trying to walk back" (repeating) | `src/bot.py` around `end_run` | `end_run` triggered when should be `end_game` |
+| "No current location set" | `src/bot.py on_maintenance` | `_curr_loc` was None after chicken/recovery |
+| Discord embed errors | `src/messages/discord_embeds.py` | `file=` kwarg passed when attachment is None |
+| Repair fail loops | `src/town/a5.py` + `town_manager.py` | Larzuk template noise; check A4 fallback path |
+| "Failed to log exp" | `src/ui/player_bar.py` | OCR misread; check for `I/l` → `1` ambiguity |
+| Sell includes wrong items | `src/inventory/personal.py` | `protect_shields_from_sell` or item filter issue |
+| "Discarding SUPER HEALING POTION" + "Confirmed sell..." | `src/inventory/personal.py:351` | Consumable sold despite belt need — fixed by get_needs() guard |
+| Belt needs stay health=3/mana=3 game after game; pots never drunk | `src/bot.py` after buy_consumables + `personal.py:351` | Health pots sold during inspect; no fill_from_inventory fallback |
+| "started on CTA slot" in logs; dies in first seconds of run | `src/char/i_char.py _pre_buff_cta` | Game saved with CTA slot active (interrupted buff). Use `BC` template check at startup to detect and correct |
+| Character enters run at partial HP (e.g. 40% after chicken) | `src/bot.py on_maintenance` | Health manager paused in town; no town-heal loop. Check `meters.get_health` and drink belt pots in maintenance before `update_pot_needs` |
+
+---
+
+## 14) "Done" checklist for a fix
+
+- [ ] Reproduced from logs
+- [ ] Root cause identified in source
+- [ ] Patch applied in smallest reasonable scope
+- [ ] `python -m compileall <changed_files>` passes
+- [ ] Target tests pass (or explicitly explain why unavailable)
+- [ ] Python smoke test confirms the fix (e.g., `Config().char['show_belt']`)
+- [ ] Behavior documented here or in README/params if user-visible
+
+---
+
+If you need to continue immediately: start from latest `main`, run a short bot session, then inspect only the newest 200–300 log lines before changing anything.

docs/legacy-go/ANTI_DETECTION.md

@@ -1,258 +0,0 @@
-# Anti-Detection Framework for Botty-Go
-
-## Overview
-
-This document outlines the multi-layered anti-detection system built into botty-go.
-Each layer addresses a specific detection vector that Blizzard and modern anti-cheat
-systems use to identify bots.
-
----
-
-## 1. Server-Side Behavior Analysis Countermeasures
-
-### Detection: Session length, timing consistency, pathing patterns, repetition
-
-### Countermeasures:
-
-#### 1a. Variable Session Scheduling
-- **Implementation:** `internal/schedule/scheduler.go`
-- Randomized session start times using a circadian model
-- Simulated human sleep patterns: 6-10 hour breaks between sessions
-- Weekend/weekday behavior variance (humans play differently on weekends)
-- Random session lengths: 20min to 6hours with exponential distribution
-- Occasional "just 5 more minutes" overtime and "I'm tired" early stops
-
-#### 1b. Stochastic Pathing
-- **Implementation:** `internal/pather/stochastic.go`
-- Add deliberate pathing imperfection: 5-15% deviation from optimal route
-- Occasional wrong-way teleports followed by course correction
-- Non-optimal waypoint selections (humants don't always take shortest path)
-- Variable route ordering with cooldown-dependent choices
-- 2-3% chance of "getting lost" and using wrong waypoint first
-
-#### 1c. Skill Rotation Variance
-- **Implementation:** `internal/char/behavior.go`
-- Variable pre-buff timing (humans rush sometimes, sometimes take time)
-- Occasional wrong skill selection followed by correction
-- Potion usage with human-like hesitation (check multiple times before drinking)
-- Merc healing variance: sometimes forget, sometimes over-heal
-
-#### 1d. Route Randomization with Context
-- **Implementation:** `internal/bot/route_planner.go`
-- Dynamic route selection based on:
-  - Time since last run of each type
-  - Current TP scroll count (humans adapt)
-  - Gem/transmute urgency
-  - Occasional "feels like it" switches
-- Never perfect round-robin; use weighted probability with drift
-
-#### 1e. Farming Repetition Masking
-- Never run the same route more than 8 times consecutively
-- Insert "town breaks": stash visit, shrine check, repair, gamble
-- 1-2% chance of "I'm bored, switching to different run" mid-session
-- Vary kill strategies: sometimes rush, sometimes methodical
-
----
-
-## 2. Warden / Client Integrity Countermeasures
-
-### Detection: Loaded modules, injected DLLs, memory signatures, debuggers
-
-### Countermeasures:
-
-#### 2a. Pixel-Only Architecture (No Memory Access)
-- **Implementation:** entire bot reads game state ONLY via screenshots
-- NO memory reading, NO DLL injection, NO process hooking
-- Same attack surface as a human with a camera pointed at the screen
-- This is the #1 defense: if you only use screen capture + input simulation,
-  there's nothing to scan in process memory
-
-#### 2b. Clean Process Environment
-- **Implementation:** `internal/runtime/clean_env.go`
-- Standard Go binary with no suspicious imports
-- No debuggers, no memory readers, no process manipulation
-- Run as a normal application, not injected
-
-#### 2c. Overlay Avoidance
-- Never draw on top of game window
-- No window hooking or injection
-- Screenshot from a separate thread, not an overlay
-
----
-
-## 3. Input Pattern Analysis Countermeasures
-
-### Detection: Synthetic inputs, smooth cursor paths, periodic inputs, no micro-corrections
-
-### Countermeasures:
-
-#### 3a. Human Motor Model
-- **Implementation:** `internal/mouse/human_model.go`
-- Full biomechanical mouse model based on Fitts' Law and human motion studies
-- Real human mouse data characteristics:
-  - Multi-segment movement with micro-pauses (1-3 segments per motion)
-  - Acceleration curve: start slow, peak in middle, decelerate into target
-  - Endpoint micro-adjustments: 2-5 pixel wobble before click
-  - Inter-trial variability: each movement is unique even to same target
-  - Asymmetric error distribution: overshoot more right/down (human bias)
-
-#### 3b. Click Timing Model
-- **Implementation:** `internal/mouse/click_model.go`
-- Variable time between "arriving" at target and clicking: 50ms-800ms
-- Pressure curve: humans don't click at exact same speed
-- Double-click rate varies naturally
-- Occasional misses: 0.5-1% of clicks land slightly off (1-3px)
-
-#### 3c. Keyboard Behavior Model
-- **Implementation:** `internal/keyboard/human_model.go`
-- Key press duration variance: not all keypresses are identical
-- Typing rhythm for skill hotkeys: natural cadence with micro-pauses
-- Occasional key repeat (holding too long = rapid fire)
-- Realistic key-up/key-down timing ratios
-
-#### 3d. Statistical Indistinguishability
-- **Implementation:** `internal/input/stats.go`
-- All input streams modeled from real human motion capture data
-- Entropy analysis of output matches human baselines
-- Auto-calibration: measure user's own input if they do manual play
-- Periodically inject "manual-looking" variance spikes
-
----
-
-## 4. Economy and Item-Flow Countermeasures
-
-### Detection: Gold accumulation, rune farming, item transfer networks, mule behavior
-
-### Countermeasures:
-
-#### 4a. Natural Accumulation Rate
-- **Implementation:** `internal/inventory/economy.go`
-- Vary farming intensity: some sessions heavy, some light
-- Match accumulation to stated playtime (more sessions = more loot)
-- Occasionally "waste" items on gambling/repairs like a real player
-
-#### 4b. Realistic Trading Patterns
-- No mass item funneling
-- If trading, do it in human-sized batches with natural pauses
-- Vary trade partners and timing
-
-#### 4c. Rune Farming Variance
-- Don't farm the same runes every session
-- Match rune acquisition to character progression
-- Occasionally skip rune picks when "full"
-
----
-
-## 5. Ban Wave Defense
-
-### Detection: Delayed batch bans
-
-### Countermeasures:
-
-#### 5a. Graceful Degradation
-- **Implementation:** `internal/runtime/safe_mode.go`
-- If one account gets banned, immediately reduce intensity across all
-- Auto-pause farming for 48-72 hours (simulating "taking a break")
-- Gradual return with reduced session lengths
-- Change behavior patterns after any ban event
-
-#### 5b. Account Diversity
-- Each account has distinct "personality":
-  - Different session timing preferences
-  - Different route preferences
-  - Different response timing distributions
-  - Different play styles (rusher vs methodical)
-
----
-
-## 6. Server Authority Countermeasures
-
-### Detection: Server-side validation of movement, drops, combat, inventory
-
-### Countermeasures:
-
-#### 6a. Server-Authoritative Behavior
-- **Implementation:** `internal/bot/server_aware.go`
-- Only interact with what the server actually shows
-- Wait for server confirmation before acting (e.g., confirm item picked up)
-- Respect server-enforced movement limits (no speed hacks)
-- Process drops in game-authorized order
-
-#### 6b. No Client Manipulation
-- Never try to spoof packets, modify client, or exploit desync
-- Purely reactive: see screen -> decide -> act -> wait for response
-
----
-
-## 7. Social/Reporting System Countermeasures
-
-### Detection: Player reports + telemetry correlation
-
-### Countermeasures:
-
-#### 7a. Social Stealth
-- **Implementation:** `internal/social/stealth.go`
-- Play during off-peak hours less suspiciously
-- Avoid solo-public routes that attract attention
-- Occasionally join other players' games (with reduced automation)
-- Inherit human-like chat behavior if configured
-
----
-
-## 8. Hardware/Identity Correlation Countermeasures
-
-### Detection: IP patterns, hardware fingerprints, VMs, account clusters
-
-### Countermeasures:
-
-#### 8a. Clean Deployment
-- **Implementation:** `internal/deploy/clean.go`
-- Run on real hardware, not VMs
-- Use residential IP, not datacenter
-- One account per hardware profile
-- No VPN/proxy during play sessions
-
----
-
-## Implementation Architecture
-
-```
-internal/
-├── input/              # Human-like input generation
-│   ├── mouse_model.go   # Fitts' Law mouse movement
-│   ├── click_model.go   # Human click timing
-│   ├── keyboard_model.go # Keyboard behavior
-│   └── stats.go         # Statistical verification
-├── behavior/           # High-level human behavior simulation
-│   ├── scheduler.go     # Session scheduling
-│   ├── route_planner.go  # Dynamic route selection
-│   ├── fatigue.go       # Simulated fatigue/boredom
-│   └── personality.go   # Per-account personality
-├── economy/            # Economic behavior masking
-│   ├── accumulation.go  # Natural loot accumulation
-│   └── trading.go       # Human-like trading patterns
-├── safe_mode/          # Graceful degradation
-│   ├── detection.go     # Ban wave detection
-│   └── cooldown.go      # Auto-pause and return
-└── deploy/             # Clean deployment helpers
-    └── check.go         # Pre-flight integrity checks
-```
-
-## Key Design Principles
-
-1. **Statistical indistinguishability:** Output must be statistically
-   indistinguishable from real human input. We use actual human motion
-   capture data distributions, not made-up random numbers.
-
-2. **Controlled imperfection:** A human is inefficient, forgetful, and
-   inconsistent. The bot should be too — but in a way that matches
-   real human distributions.
-
-3. **No single fingerprint:** Every instance should have unique enough
-   characteristics that correlating two accounts is hard.
-
-4. **Adaptability:** If behavior changes are detected, the system should
-   be able to recalibrate based on new data.
-
-5. **Defense in depth:** No single countermeasure is sufficient. The
-   combination across all layers is what provides real protection.

docs/legacy-go/GO_REWRITE_README.md

@@ -1,33 +0,0 @@
-# Botty-Go
-
-D2R Pixel Bot rewritten in Go for cross-platform support (Linux + Windows).
-
-Based on the Python Botty project (johannes-do/botty), this is a ground-up rewrite
-in Go that maintains compatibility with the same config files, templates, and run
-logic while adding native Linux support.
-
-## Features
-
-- Cross-platform: Linux (X11/Wayland) and Windows
-- Same config format as original Botty (params.ini, game.ini, shop.ini)
-- Template matching with OpenCV Go bindings
-- Tesseract OCR for item identification
-- Human-like mouse movement (Bezier curves)
-- BNIP pickit language
-- All original character builds (Sorc, Paladin, Necro, Barbarian, etc.)
-- All original runs (Pindle, Eldritch, Shenk, Trav, Nihlathak, Arcane, Diablo)
-
-## Building
-
-```bash
-# Linux
-go build -o botty ./cmd/botty
-
-# Windows (from Linux with cross-compile)
-GOOS=windows GOARCH=amd64 go build -o botty.exe ./cmd/botty
-```
-
-## Configuration
-
-Copy `config/` from the original Botty project. Params, routes, and character
-config work identically.

docs/legacy-go/INDEX.md

@@ -1,19 +0,0 @@
-# Legacy: Go Rewrite Design Notes
-
-These docs are archived from an abandoned `~/git/botty-go` directory (May 2026).
-That project was a planned ground-up Go rewrite of `johannes-do/botty` for
-cross-platform (Linux + Windows) support. Only design docs existed — no `.go`
-source was ever written.
-
-The Python `my-botty` project (this repo) is the active path. These docs are
-kept here as **reference material**, primarily for Milestone 2 (anti-detection /
-stealth) of `~/.claude/plans/continue-the-make-up-sunny-honey.md`.
-
-## Files
-
-- **`ANTI_DETECTION.md`** — Multi-layer anti-detection framework. Covers
-  server-side behavior analysis countermeasures (session scheduling, stochastic
-  pathing, skill rotation variance) and more. Directly applicable as the design
-  basis for the Python stealth layer.
-- **`GO_REWRITE_README.md`** — Original README of the abandoned Go project.
-  Context only — explains feature scope and what the rewrite was aiming for.

docs/linux_port_plan.md

@@ -0,0 +1,83 @@
+# Linux Port Plan (Botty)
+
+## Goal
+Make Botty runnable on Linux in phased steps, with clear checkpoints and minimal regressions for current Windows users.
+
+## Current status
+Botty is currently Windows-first. Full gameplay flow does not run on Linux due to:
+- Windows input stack (`win_input`, Win32 hotkey polling).
+- Windows process/window management (`taskkill`, Win32 window APIs, `os.startfile`).
+- Windows dependency assumptions (`pywin32`, Windows tesserocr wheel guidance).
+- Windows path/env assumptions (`APPDATA`, `C:\...`, `D2R.exe`, `.bat` scripts).
+
+## Principles
+- Keep Windows behavior unchanged while adding Linux support.
+- Introduce platform abstractions before replacing implementations.
+- Land small, testable phases.
+- Prefer graceful `NotImplemented` behavior over hard crashes on unsupported paths.
+
+## Phase 1: Platform abstraction layer
+1. Add a `platform_adapter` module with interfaces for:
+   - Input (keyboard/mouse send + hotkeys)
+   - Window management (find game window, set top-most, geometry)
+   - Process control (start/stop/check D2R/Battle.net)
+2. Route existing Windows calls through adapters.
+3. Add Linux stub implementations that fail gracefully with actionable logs.
+4. Add unit tests for adapter selection and fallback behavior.
+
+## Phase 2: Linux-safe startup and tooling
+1. Add Linux entry script (`run_botty.sh`) and dependency checker shell script.
+2. Update startup to avoid Windows-only calls unless platform is Windows.
+3. Normalize path handling to `pathlib` where feasible.
+4. Ensure `main.py` can start on Linux without immediate import/runtime crashes.
+
+## Phase 3: Linux input backend
+1. Implement Linux input backend (X11/Wayland-compatible strategy):
+   - Candidate libs: `pynput`, `python-xlib`, or tool-backed approach (`xdotool` for X11).
+2. Match required Botty features:
+   - Key press/hold/release
+   - Mouse move/click with jitter and timing controls
+   - Hotkey registration/polling
+3. Add integration tests/mocks for input primitives.
+
+## Phase 4: Linux screen/window backend
+1. Validate capture compatibility for `mss` under target Linux desktop/session.
+2. Implement Linux window discovery/focus/geometry handling.
+3. Rework DPI/coordinate normalization independent of Win32 APIs.
+4. Add diagnostics tool to verify coordinates, capture ROI, and template matching on Linux.
+
+## Phase 5: Process and launcher integration
+1. Linux-compatible process management (replace `taskkill` paths).
+2. Replace `os.startfile` launcher logic with cross-platform process spawning.
+3. Add platform-specific config defaults for game executable path conventions.
+
+## Phase 6: Dependency and OCR strategy
+1. Split dependencies by platform (base + windows extras + linux extras).
+2. Document Linux OCR setup (tesseract/leptonica packages + python bindings).
+3. Add CI matrix entries:
+   - Windows: full current pipeline
+   - Linux: import/startup + unit/integration subset first, expand later
+
+## Phase 7: Feature parity validation
+1. Verify end-to-end flows:
+   - Start game, run cycle, maintenance, save/exit, restart handling
+2. Validate pickit, stash/sell, discord messaging, stats logging.
+3. Benchmark timing-sensitive routines and tune Linux defaults.
+
+## Risk register
+- Wayland restrictions can block synthetic input/screen capture depending on compositor.
+- Template matching thresholds may differ due to capture pipeline differences.
+- Hotkey handling behavior can differ across desktop environments.
+- OCR reliability can vary based on font rendering stack.
+
+## Suggested delivery milestones
+1. **M1**: Linux no-crash startup + stubs + docs.
+2. **M2**: Linux input backend functional in sandbox diagnostics.
+3. **M3**: Linux screen/window backend and maintenance loop stable.
+4. **M4**: End-to-end run support in supported Linux environments.
+
+## Acceptance criteria
+- Botty starts on Linux and logs clear capability status.
+- No Windows-only hard failures on Linux code paths.
+- Core run loop can execute in a supported Linux environment.
+- Windows behavior remains stable and covered by existing tests/CI.

docs/melee_hunt_test_log.md

@@ -0,0 +1,154 @@
+# Paladalla Melee Hunt Test Log
+
+Date: 2026-08-23  
+Workspace: `C:\Users\alex\my-botty`  
+Character/profile: `paladalla`  
+Script: `tools/melee_hunt.py`
+
+## Current State
+
+- Testing was stopped at the user's request.
+- No `melee_hunt` Python process was running when checked.
+- Paladalla was alive in the Rogue Encampment at the end of the session.
+- The four visible belt potions were manually consumed. The last screenshot shows
+  partial life and an empty belt, so refill and fully heal before the next run.
+- The game must not be controlled automatically until the user asks to resume.
+
+Final-state screenshot: [healed_from_belt.png](../log/screenshots/manual/healed_from_belt.png)
+
+## Commands Used
+
+The successful short run used:
+
+```powershell
+C:\Users\alex\miniforge3\envs\botty\python.exe tools\melee_hunt.py --debug --button left
+```
+
+The later long tests used:
+
+```powershell
+C:\Users\alex\miniforge3\envs\botty\python.exe tools\melee_hunt.py --debug --button left --minutes 1440
+```
+
+`F12` is intended to stop the script. It worked during one run but was not
+reliable while an in-game overlay was open. `Ctrl+C` in the script terminal was
+the reliable emergency stop.
+
+## Verified Results
+
+### Initial ten-minute run
+
+- Started outside town and completed its configured timer normally.
+- Result: `engaged 107, roamed 15`.
+- XP changed from `261/500` to `357/500`.
+- Final reported life was `69%`.
+- The process then exited because the default duration is ten minutes. Paladalla
+  later died because nothing was controlling or protecting the character.
+
+### Corpse recovery and level gain
+
+- The death prompt was cleared and the game was cycled through Save and Exit.
+- The corpse appeared beside Paladalla in town and was recovered successfully.
+- Weapon and attack slots were restored.
+- Paladalla was manually routed across the bridge into the Blood Moor.
+- Manual movement/attack clicks later produced a level-up. The next script start
+  read XP as `564/1500`, confirming level 2 and working melee damage.
+
+Evidence:
+
+- [corpse_recovered.png](../log/screenshots/manual/corpse_recovered.png)
+- [blood_moor_ready2.png](../log/screenshots/manual/blood_moor_ready2.png)
+- [deep_moor_restart.png](../log/screenshots/manual/deep_moor_restart.png)
+
+## Failures Found
+
+### 1. Ten-minute default looked like a crash
+
+The first hunt was not dead or hung. It reached its default ten-minute deadline
+and exited cleanly. Long unattended tests must pass an explicit `--minutes`
+value or implement a true continuous mode.
+
+### 2. Friendly bridge NPC was treated as motion target
+
+The first 24-hour attempt remained near the town bridge. At the two-minute
+checkpoint it showed repeated attacks, full life, and unchanged XP
+(`357/500`). The screenshot showed it attacking the friendly rogue near the
+bridge. Motion alone cannot distinguish that NPC from a monster.
+
+Evidence: [hunt_2min_checkpoint.png](../log/screenshots/manual/hunt_2min_checkpoint.png)
+
+Workaround used: stop the script and move several screens deeper into the Blood
+Moor before restarting. A permanent NPC rejection/stall detector is still
+needed.
+
+### 3. Animated HUD controls were treated as targets
+
+After reaching level 2, the pulsing new-stat/new-skill controls appeared above
+the original HUD mask. The motion detector could target that region. During the
+second long attempt an in-game Loot Filter overlay opened and the script kept
+issuing clicks behind it.
+
+Evidence: [restart_live_40s.png](../log/screenshots/manual/restart_live_40s.png)
+
+### 4. No-potion condition was unsafe
+
+Life fell through `18%`, `14%`, `10%`, `8%`, and eventually `0%`. The script
+logged `no healing potion in belt!` but continued its loop. Belt detection also
+reported no potion even though red potions were visibly present, so belt
+detection or its profile assumptions need separate investigation.
+
+### 5. XP OCR became invalid after level-up/UI changes
+
+The log reported XP OCR values such as `'-'` and `''`, then incorrectly printed
+`*** LEVEL UP *** now 0/0`. Invalid OCR should not be interpreted as a level-up,
+and `xp_last` should retain the last valid sample.
+
+## Code Changes Made
+
+`tools/melee_hunt.py` was changed in two ways:
+
+1. The target scan bottom boundary was raised from `640` to `510`, excluding the
+   animated level-up/stat controls and the interactive HUD from motion targets.
+2. Failure to detect or use a healing potion at the configured health threshold
+   now raises `SafetyStop`, exits the hunt loop, and prints a safety-stop reason.
+
+The edited script passed:
+
+```powershell
+C:\Users\alex\miniforge3\envs\botty\python.exe -m py_compile tools\melee_hunt.py
+```
+
+These changes were not runtime-verified after editing because testing was
+stopped. The narrower scan may ignore monsters near the bottom of the viewport;
+that tradeoff must be checked visually.
+
+## Next Test Checklist
+
+1. Confirm no old hunt process is running.
+2. Refill the belt with healing potions and fully heal at Akara.
+3. Save and Exit, re-enter, and recover any corpse if one appears.
+4. Spend or dismiss the new stat/skill notifications if practical.
+5. Walk across the bridge and several screens into the Blood Moor before launch.
+6. Start with a short five-minute test, not a 24-hour run:
+
+```powershell
+C:\Users\alex\miniforge3\envs\botty\python.exe tools\melee_hunt.py --debug --button left --minutes 5
+```
+
+7. Capture and inspect screenshots at launch, two minutes, and five minutes.
+8. At each checkpoint verify: outside town, no modal open, attacking a hostile,
+   XP increasing, life stable, movement occurring, and no repeated target lock.
+9. Force a controlled low-health/no-potion condition only after normal combat is
+   proven, and verify `SAFETY STOP` exits immediately.
+10. Use `Ctrl+C` if any modal opens, XP remains unchanged across two checkpoints,
+    life drops rapidly, or the character approaches the town bridge NPC.
+
+## Follow-up Engineering Work
+
+- Add a stall detector based on unchanged XP plus repeated target coordinates.
+- Temporarily blacklist a target position after excessive swings without XP.
+- Reject invalid XP samples instead of converting them into `0/0` level-ups.
+- Investigate why `belt.drink_potion("health", ...)` missed visible red potions.
+- Detect death and modal overlays explicitly and stop before sending more input.
+- Consider an explicit `--continuous` option instead of using a large minute
+  count.

docs/postmortem_pindle_2026-08-27.md

@@ -0,0 +1,145 @@
+# Post-mortem — run_pindle total failure (26–27 Aug 2026)
+
+**Outcome:** failure rate 100% → ~8%. Character 32 → 69. Cycle time 185–250s → ~52s.
+18 commits, `98e12c0` … `402c768`.
+(Hashes changed on 27 Aug: history was rewritten before the PR to purge a Discord
+webhook that had been committed into `config/params.ini` by mistake.)
+
+Published rendering: `docs/postmortems/pindle-2026-08-27.html`
+(artifact: https://claude.ai/code/artifact/00668e28-dbb8-40e8-b585-db4c8b790d0a)
+
+Per-bug detail with code references lives in `CLAUDE.md` under **Known Bugs and Fixes**
+(Bugs 23–30). This document is the narrative: what happened, in what order, and what to do
+differently.
+
+---
+
+## 1. The expensive part: a green log is not evidence
+
+The original symptom was loud and honest — every run died on
+`Wanted to select A5_RED_PORTAL, but could not find it`.
+
+After the portal template was fixed, the logs went clean **and stayed wrong**. Five
+consecutive games produced:
+
+```
+INFO  Pindle approach: confirmed temple entry (PINDLE_7 @ 75.7%)
+INFO  Loot from run_pindle: nothing picked up
+INFO  End game. Elapsed time: 50.98s
+      "runs_failed_total": 0
+```
+
+Every line green. The character was standing in the middle of Harrogath swinging at nothing.
+
+**What gave it away was not the log — it was the XP counter.** 6,779,050 before five
+"successful" runs, 6,779,050 after. Pindle always drops something; five runs of
+`nothing picked up` is not variance, it is a run that never happened.
+
+> A boss run's success is only provable by things the bot cannot fake: an item in the loot
+> line, or a jump in `current_exp`. `failed:false` is the bot's *opinion*, derived from the
+> very check that was lying.
+
+---
+
+## 2. The causal chain
+
+Four of the first five causes were one failure propagating. Fixing only the visible end
+would have left the rest armed.
+
+```
+Portal template is masked (scores 0.50-0.60, needs 0.68, matches the wrong place)
+  -> portal click fails, approach re-paths from a hardcoded town start it no longer occupies
+    -> landmarks miss, the pather's 0.55 fallback invents a node position from scenery
+      -> char walks at the phantom and wedges on the town wall, or onto the waypoint stone
+        -> waypoint panel opens, health manager reads it as a threat, chickens at full HP
+```
+
+---
+
+## 3. Root causes, in order of cost
+
+| Bug | Cause | Severity |
+|---|---|---|
+| 24 | "Already in Pindle area?" shortcut fired **in town** (0.76–0.79 on `PINDLE_7` vs its 0.62 bar), returned success without clicking the portal | **Silent** |
+| 28 | `detect_current_act` committed on a **1.7pp margin** (`A5_TOWN_1` 0.636 vs `A4_TOWN_5` 0.619) → A5 pathing ran inside Act 4. **25% of games** | **Silent** |
+| 30 | NPC grid sweep `SWEEP_TAG_THRESHOLD=0.4` settled for noise (real hits 0.98–1.00, false 0.42–0.50) | Loud |
+| 23 | `a5_red_portal.png` was the only **masked** template in its folder → unsupported masked `TM_CCOEFF_NORMED` | Loud |
+| 25 | Pather 0.55 **first-match** fallback fabricated node positions → walked into the town wall | Loud |
+| 29 | Undetectable resurrect NPC re-hunted every game (~80s), per-game flag reset by `log_start_game` | Loud |
+| 26 | Waypoint panel counted toward a chicken — 3 of 6 games died at full HP | Loud |
+| 27 | `GEMS_TAB_Y=100` missed the tab row (labels at y 63–78, grid starts y≈87) → 800+ silent warnings | Loud |
+| 9b | Pindle retry re-pathed from a hardcoded `A5_TOWN_START` | Loud |
+
+The two marked **Silent** cost the most, because they reported success.
+
+---
+
+## 4. Four corrections — times the obvious answer was wrong
+
+These are recorded because in each case the wrong answer looked correct.
+
+**1. The red-portal guard.** To stop the false "already in Pindle area" I added: *if the red
+portal is on screen we must still be in town.* The portal is visible from **both sides** —
+the return portal renders inside the temple. The guard blocked every genuine entry. It
+traded a silent false-success for an honest false-failure: an improvement, still a bug.
+
+**2. XP as proof of a boss kill.** An XP gain proves *something* died — the minion pack and
+the mercenary's kills both count. Confirmation needed a loot line naming an actual drop.
+
+**3. "The template is degenerate."** I blamed the Qual-Kehk asset for a repeated identical
+0.424 score. Walking the character to the NPC and scoring the stored template against a
+frame where the tag renders gave **1.000** (0.997 through the real `color_filter` path). The
+template was perfect; the *threshold* was wrong.
+
+> A repeated identical score means the search is settling for **noise**. Check the threshold
+> before blaming the asset.
+
+**4. Selling the charms.** I read 673 blocked-sell lines as a clog to clear and disabled
+`protect_charms_from_sell`. It vendored five resistance charms before it was caught. The
+blocked lines were **the guard working**, and the pickit's `Discarding` verdict was never a
+judgement that they were junk — `LAPIS SMALL CHARM OF VITA` failed its rule only because the
+rule wanted `coldresist >= 11` and it had 7.
+
+---
+
+## 5. What actually made this expensive
+
+Every bug above cost hours because the log recorded **events, not structure**. There was no
+way to ask "where did this game spend its time?" without reading thousands of lines.
+
+That is now fixed — see `CLAUDE.md` → *The run timeline* and *Diagnosing a failure*:
+
+- **`grep "TL>"`** — one fixed-width line per step, spawn to loot, every step timed.
+- **`grep "FAIL>"`** — a self-contained failure record: reason, location, screenshot, the
+  three slowest steps, and a breadcrumb trail of the game with `!` marking failures.
+- **A Discord digest every 2h** (`general.discord_timing_report_h`), aggregated from the same
+  timeline so report and log cannot drift.
+
+The first thing the timeline surfaced, unprompted:
+
+```
+TL> g2 r1 | town  | resurrect_merc    | fail  | took=113.6s | NPC not reachable
+TL> g2 r1 | town  | maintenance       | ok    | took=137.6s | at a5_larzuk
+TL> g2 r1 | run   | approach          | fail  | took=71.9s  | step=click_red_portal
+```
+
+That failed game spent **113.6s of its 137.6s town visit** on a merc resurrect that achieved
+nothing — 83% of the time, on a step that was not the one that ended the game.
+
+---
+
+## 6. Rules carried forward
+
+1. **Verify boss runs with XP delta and loot lines**, never `failed:false`.
+2. **Never mask part of a template** to hide an artifact — masked `TM_CCOEFF_NORMED` is
+   unsupported in OpenCV and fails unpredictably. Recapture it clean.
+3. **Prove a template fails on a frame where the subject is definitely visible** before
+   recapturing anything.
+4. **Confirm any click that changes UI mode** — tab, page, panel. An unverified click that
+   "should" have worked fails silently forever.
+5. **Never let a detector guess on a thin margin.** Refusing to answer is usually safe; a
+   wrong confident answer skips the recovery path entirely.
+6. **Read repeated warnings as a system working**, not a clog to clear.
+7. **Restart the bot after editing `src/` or config** — both load at process start.
+8. **A "we might already be there" fast path is dangerous** when its test can false-positive
+   at the origin. It converts a loud failure into a silent one.

docs/postmortems/pindle-2026-08-27.html

@@ -0,0 +1,726 @@
+<title>Pindle Run Post-Mortem</title>
+<link rel="preconnect" href="https://fonts.googleapis.com">
+<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
+<link rel="stylesheet" href="https://fonts.googleapis.com/css2?family=Fraunces:opsz,[email protected],500;9..144,600;9..144,700&family=IBM+Plex+Mono:wght@400;500;600&family=IBM+Plex+Sans:wght@400;500;600&display=swap">
+
+<style>
+:root{
+  --ground:#EDF0F4;
+  --surface:#FFFFFF;
+  --surface-2:#E3E8EF;
+  --ink:#15181E;
+  --ink-2:#3D4652;
+  --slate:#5A6675;
+  --hair:#CBD3DE;
+  --accent:#B01B2E;
+  --accent-soft:#F6E4E7;
+  --good:#1B6B52;
+  --good-soft:#DFEFE8;
+  --warn:#8A5A0B;
+  --warn-soft:#F7EBD6;
+  --shadow:0 1px 2px rgba(21,24,30,.06),0 8px 24px -12px rgba(21,24,30,.18);
+}
+@media (prefers-color-scheme:dark){
+  :root:not([data-theme="light"]){
+    --ground:#101317;
+    --surface:#171B21;
+    --surface-2:#1F242C;
+    --ink:#E6EAF0;
+    --ink-2:#BAC3CF;
+    --slate:#8B97A6;
+    --hair:#2A313A;
+    --accent:#FF5F70;
+    --accent-soft:#2C1A1F;
+    --good:#4FC79B;
+    --good-soft:#14261F;
+    --warn:#E0A44A;
+    --warn-soft:#2A2114;
+    --shadow:0 1px 2px rgba(0,0,0,.4),0 10px 28px -14px rgba(0,0,0,.7);
+  }
+}
+:root[data-theme="dark"]{
+  --ground:#101317;
+  --surface:#171B21;
+  --surface-2:#1F242C;
+  --ink:#E6EAF0;
+  --ink-2:#BAC3CF;
+  --slate:#8B97A6;
+  --hair:#2A313A;
+  --accent:#FF5F70;
+  --accent-soft:#2C1A1F;
+  --good:#4FC79B;
+  --good-soft:#14261F;
+  --warn:#E0A44A;
+  --warn-soft:#2A2114;
+  --shadow:0 1px 2px rgba(0,0,0,.4),0 10px 28px -14px rgba(0,0,0,.7);
+}
+
+*{box-sizing:border-box}
+body{
+  margin:0;
+  background:var(--ground);
+  color:var(--ink);
+  font-family:"IBM Plex Sans",-apple-system,BlinkMacSystemFont,"Segoe UI",sans-serif;
+  font-size:16.5px;
+  line-height:1.65;
+  -webkit-font-smoothing:antialiased;
+}
+.wrap{max-width:1080px;margin:0 auto;padding:0 24px 96px}
+.col{max-width:672px;margin-inline:auto}
+
+h1,h2,h3{font-family:Fraunces,Georgia,serif;text-wrap:balance;margin:0}
+h1{font-size:clamp(2.1rem,5.2vw,3.15rem);font-weight:600;line-height:1.08;letter-spacing:-.015em}
+h2{font-size:clamp(1.4rem,3vw,1.75rem);font-weight:600;line-height:1.2}
+h3{font-size:1.06rem;font-weight:600;line-height:1.3}
+p{margin:0}
+a{color:var(--accent)}
+code,.mono{font-family:"IBM Plex Mono",ui-monospace,Menlo,monospace}
+
+.eyebrow{
+  font-family:"IBM Plex Mono",monospace;font-size:.7rem;font-weight:500;
+  letter-spacing:.16em;text-transform:uppercase;color:var(--slate);
+}
+
+/* ---- masthead ---- */
+header{padding:72px 0 40px;border-bottom:1px solid var(--hair)}
+.head-inner{display:flex;flex-direction:column;gap:20px}
+.lede{font-size:1.12rem;color:var(--ink-2);max-width:64ch}
+.verdict{
+  display:inline-flex;align-items:center;gap:10px;align-self:flex-start;
+  background:var(--good-soft);color:var(--good);
+  border:1px solid color-mix(in srgb,var(--good) 30%,transparent);
+  border-radius:2px;padding:7px 13px;
+  font-family:"IBM Plex Mono",monospace;font-size:.76rem;font-weight:600;letter-spacing:.06em;
+}
+.verdict::before{content:"";width:7px;height:7px;border-radius:50%;background:var(--good)}
+
+/* ---- metric band ---- */
+.metrics{
+  display:grid;grid-template-columns:repeat(auto-fit,minmax(158px,1fr));
+  gap:1px;background:var(--hair);border:1px solid var(--hair);
+  margin:40px 0 0;
+}
+.metric{background:var(--surface);padding:18px 20px;display:flex;flex-direction:column;gap:5px}
+.metric .k{font-size:.72rem;letter-spacing:.09em;text-transform:uppercase;color:var(--slate);font-family:"IBM Plex Mono",monospace}
+.metric .v{font-family:Fraunces,serif;font-size:1.6rem;font-weight:600;font-variant-numeric:tabular-nums;line-height:1.1}
+.metric .n{font-size:.8rem;color:var(--slate)}
+.metric.hi .v{color:var(--good)}
+.metric.lo .v{color:var(--accent)}
+
+section{padding-top:64px}
+.section-head{display:flex;flex-direction:column;gap:8px;margin-bottom:26px}
+.body-flow{display:flex;flex-direction:column;gap:18px}
+
+/* ---- the trap callout ---- */
+.trap{
+  background:var(--accent-soft);
+  border-left:3px solid var(--accent);
+  padding:22px 24px;display:flex;flex-direction:column;gap:12px;
+}
+.trap h3{color:var(--accent)}
+
+/* ---- log evidence ---- */
+.log{
+  background:var(--surface);border:1px solid var(--hair);
+  padding:16px 18px;overflow-x:auto;
+  font-family:"IBM Plex Mono",monospace;font-size:.79rem;line-height:1.75;
+  color:var(--ink-2);white-space:pre;
+}
+.log .ok{color:var(--good);font-weight:600}
+.log .bad{color:var(--accent);font-weight:600}
+.log .dim{color:var(--slate)}
+figure{margin:0;display:flex;flex-direction:column;gap:9px}
+figcaption{font-size:.82rem;color:var(--slate)}
+
+/* ---- causal chain ---- */
+.chain{display:flex;flex-direction:column;gap:0;margin-top:8px}
+.node{
+  background:var(--surface);border:1px solid var(--hair);
+  padding:14px 18px;display:flex;gap:14px;align-items:baseline;
+}
+.node .tag{
+  font-family:"IBM Plex Mono",monospace;font-size:.7rem;font-weight:600;
+  color:var(--accent);letter-spacing:.05em;flex-shrink:0;min-width:52px;
+}
+.node .txt{font-size:.93rem;color:var(--ink-2)}
+.node strong{color:var(--ink);font-weight:600}
+.arrow{
+  height:22px;width:1px;background:var(--hair);margin-left:34px;position:relative;flex-shrink:0;
+}
+.arrow::after{
+  content:"";position:absolute;bottom:0;left:-3px;
+  border-left:3.5px solid transparent;border-right:3.5px solid transparent;
+  border-top:5px solid var(--hair);
+}
+.node.terminal{border-color:color-mix(in srgb,var(--accent) 45%,var(--hair));background:var(--accent-soft)}
+.node.terminal .txt strong{color:var(--accent)}
+
+/* ---- cause cards ---- */
+.causes{display:flex;flex-direction:column;gap:14px}
+.cause{
+  background:var(--surface);border:1px solid var(--hair);box-shadow:var(--shadow);
+  display:flex;flex-direction:column;
+}
+.cause-head{
+  display:flex;gap:14px;align-items:flex-start;
+  padding:20px 24px 16px;border-bottom:1px solid var(--hair);
+}
+.bugid{
+  font-family:"IBM Plex Mono",monospace;font-size:.7rem;font-weight:600;letter-spacing:.06em;
+  color:var(--surface);background:var(--ink);padding:4px 8px;border-radius:2px;flex-shrink:0;
+  margin-top:3px;
+}
+.cause-head .titles{display:flex;flex-direction:column;gap:3px}
+.cause-head .file{font-family:"IBM Plex Mono",monospace;font-size:.76rem;color:var(--slate)}
+.cause-body{padding:20px 24px 24px;display:flex;flex-direction:column;gap:16px}
+.field{display:flex;flex-direction:column;gap:5px}
+.field .lbl{
+  font-family:"IBM Plex Mono",monospace;font-size:.67rem;letter-spacing:.13em;
+  text-transform:uppercase;color:var(--slate);
+}
+.field p{font-size:.94rem;color:var(--ink-2)}
+.field.fix .lbl{color:var(--good)}
+.sev{
+  margin-left:auto;flex-shrink:0;font-family:"IBM Plex Mono",monospace;
+  font-size:.66rem;font-weight:600;letter-spacing:.08em;text-transform:uppercase;
+  padding:4px 8px;border-radius:2px;white-space:nowrap;margin-top:3px;
+}
+.sev.silent{background:var(--accent-soft);color:var(--accent);border:1px solid color-mix(in srgb,var(--accent) 30%,transparent)}
+.sev.loud{background:var(--warn-soft);color:var(--warn);border:1px solid color-mix(in srgb,var(--warn) 30%,transparent)}
+
+/* ---- tables ---- */
+.tablewrap{overflow-x:auto;border:1px solid var(--hair);background:var(--surface)}
+table{border-collapse:collapse;width:100%;font-size:.86rem}
+th,td{text-align:left;padding:10px 16px;border-bottom:1px solid var(--hair)}
+th{
+  font-family:"IBM Plex Mono",monospace;font-size:.68rem;letter-spacing:.1em;
+  text-transform:uppercase;color:var(--slate);font-weight:500;background:var(--surface-2);
+}
+tbody tr:last-child td{border-bottom:none}
+td.num{font-family:"IBM Plex Mono",monospace;font-variant-numeric:tabular-nums;white-space:nowrap}
+td .yes{color:var(--good);font-weight:600}
+td .no{color:var(--accent);font-weight:600}
+
+/* ---- checklist ---- */
+.checks{display:flex;flex-direction:column;gap:1px;background:var(--hair);border:1px solid var(--hair)}
+.check{background:var(--surface);padding:15px 20px;display:flex;gap:13px;align-items:flex-start}
+.check .mark{color:var(--good);font-weight:700;flex-shrink:0;line-height:1.6}
+.check.dont .mark{color:var(--accent)}
+.check .ct{font-size:.92rem;color:var(--ink-2)}
+.check .ct strong{color:var(--ink)}
+
+/* ---- wrong turns ---- */
+.turn{
+  border:1px dashed var(--hair);background:transparent;
+  padding:18px 20px;display:flex;flex-direction:column;gap:8px;
+}
+.turn .lbl{font-family:"IBM Plex Mono",monospace;font-size:.68rem;letter-spacing:.12em;text-transform:uppercase;color:var(--warn)}
+.turn p{font-size:.92rem;color:var(--ink-2)}
+
+footer{
+  margin-top:80px;padding-top:26px;border-top:1px solid var(--hair);
+  display:flex;flex-wrap:wrap;gap:10px 20px;justify-content:space-between;
+  font-family:"IBM Plex Mono",monospace;font-size:.76rem;color:var(--slate);
+}
+@media (max-width:640px){
+  header{padding-top:48px}
+  .cause-head{flex-wrap:wrap}
+  .sev{margin-left:0}
+}
+</style>
+<div class="wrap">
+
+<header>
+  <div class="col head-inner">
+    <span class="eyebrow">Post-mortem · run_pindle · 26–27 Aug 2026</span>
+    <h1>The bot reported five perfect runs and killed nothing</h1>
+    <p class="lede">
+      <code>run_pindle</code> was failing 100% of the time. Then a fix made it worse in the
+      most expensive way possible: it started reporting success. Nine root causes, four wrong
+      turns of my own, and the instrumentation built so the next one takes minutes.
+    </p>
+    <span class="verdict">Resolved &middot; 17 commits</span>
+  </div>
+
+  <div class="col">
+    <div class="metrics">
+      <div class="metric lo">
+        <span class="k">Failure rate</span>
+        <span class="v">100%</span>
+        <span class="n">at the start</span>
+      </div>
+      <div class="metric hi">
+        <span class="k">Now</span>
+        <span class="v">~8%</span>
+        <span class="n">1 in 28 approaches</span>
+      </div>
+      <div class="metric hi">
+        <span class="k">Character</span>
+        <span class="v">32 → 69</span>
+        <span class="n">37 levels</span>
+      </div>
+      <div class="metric hi">
+        <span class="k">Cycle time</span>
+        <span class="v">~52s</span>
+        <span class="n">was 185–250s</span>
+      </div>
+    </div>
+  </div>
+</header>
+
+<section>
+  <div class="col section-head">
+    <span class="eyebrow">The expensive part</span>
+    <h2>A green log is not evidence</h2>
+  </div>
+  <div class="col body-flow">
+    <p>
+      The original symptom was loud and honest: every run died on
+      <code>Wanted to select A5_RED_PORTAL, but could not find it</code>. After the template
+      was fixed, the logs went clean — and stayed wrong. Five consecutive games produced this:
+    </p>
+
+    <figure>
+      <div class="log"><span class="ok">INFO  Pindle approach: confirmed temple entry (PINDLE_7 @ 75.7%)</span>
+<span class="ok">INFO  Loot from run_pindle: nothing picked up</span>
+<span class="ok">INFO  End game. Elapsed time: 50.98s</span>
+<span class="dim">      "runs_failed_total": 0</span></div>
+      <figcaption>Every line green. The character was standing in the middle of town, swinging at nothing.</figcaption>
+    </figure>
+
+    <div class="trap">
+      <h3>What gave it away</h3>
+      <p>
+        Not the logs — the <strong>XP counter never moved</strong>. 6,779,050 before five
+        "successful" runs, 6,779,050 after. Pindle always drops something; five runs of
+        <code>nothing picked up</code> is not variance, it is a run that never happened.
+      </p>
+    </div>
+
+    <p>
+      A boss run's success is only provable by things the bot cannot fake: an item in the loot
+      line, or a jump in <code>current_exp</code>. The <code>failed:false</code> flag is the
+      bot's <em>opinion</em>, derived from the very template check that was lying.
+    </p>
+  </div>
+</section>
+
+<section>
+  <div class="col section-head">
+    <span class="eyebrow">How one bad asset cascaded</span>
+    <h2>The causal chain</h2>
+  </div>
+  <div class="col body-flow">
+    <p>
+      Four of the first five causes are one failure propagating. This is a real dependency
+      chain, so fixing only the visible end would have left the rest armed.
+    </p>
+    <div class="chain">
+      <div class="node">
+        <span class="tag">CAUSE</span>
+        <span class="txt"><strong>Portal template is masked</strong> — scores 0.50–0.60, below the 0.68 it needs, matching the wrong place entirely.</span>
+      </div>
+      <div class="arrow"></div>
+      <div class="node">
+        <span class="tag">THEN</span>
+        <span class="txt">Portal click fails, so the approach <strong>re-paths from a hardcoded town start</strong> the character no longer occupies.</span>
+      </div>
+      <div class="arrow"></div>
+      <div class="node">
+        <span class="tag">THEN</span>
+        <span class="txt">Landmarks miss, so the pather's <strong>0.55 fallback invents a node position</strong> from unrelated scenery.</span>
+      </div>
+      <div class="arrow"></div>
+      <div class="node">
+        <span class="tag">THEN</span>
+        <span class="txt">The character walks toward the phantom and <strong>wedges against the town wall</strong> — or onto the waypoint stone.</span>
+      </div>
+      <div class="arrow"></div>
+      <div class="node terminal">
+        <span class="tag">RESULT</span>
+        <span class="txt">The waypoint panel opens, the health manager reads it as a threat, and <strong>chickens a game at full HP</strong>.</span>
+      </div>
+    </div>
+  </div>
+</section>
+
+<section>
+  <div class="col section-head">
+    <span class="eyebrow">Root causes</span>
+    <h2>Nine bugs, in order of cost</h2>
+  </div>
+  <div class="col causes">
+
+    <article class="cause">
+      <div class="cause-head">
+        <span class="bugid">BUG 24</span>
+        <div class="titles">
+          <h3>An optimistic shortcut faked every kill</h3>
+          <span class="file">src/run/pindle.py — approach()</span>
+        </div>
+        <span class="sev silent">Silent</span>
+      </div>
+      <div class="cause-body">
+        <div class="field">
+          <span class="lbl">Root cause</span>
+          <p>
+            <code>approach()</code> opened with an "are we already there?" shortcut that ran
+            before clicking the portal. Harrogath scenery near the portal scores
+            <strong>0.76–0.79 on <code>PINDLE_7</code></strong> — above its 0.62 bar — so the
+            shortcut fired <em>in town</em>, returned <code>A5_PINDLE_START</code> without ever
+            clicking the portal, and the bot ran <code>kill_pindle()</code> in the middle of
+            Harrogath.
+          </p>
+        </div>
+        <div class="field fix">
+          <span class="lbl">Fix</span>
+          <p>Shortcut deleted. Entry is proven only by the portal click's loading screen. Real temple entries score <strong>85.8–88.7%</strong>, clear of the town false positives.</p>
+        </div>
+        <div class="field">
+          <span class="lbl">Generalise</span>
+          <p>A "we might already be there" fast path is dangerous whenever its test can false-positive at the origin. It converts a loud failure into a silent one — the worst trade in an unattended system.</p>
+        </div>
+      </div>
+    </article>
+
+    <article class="cause">
+      <div class="cause-head">
+        <span class="bugid">BUG 28</span>
+        <div class="titles">
+          <h3>The act was decided by a 1.7-point margin</h3>
+          <span class="file">src/town/town_manager.py — detect_current_act()</span>
+        </div>
+        <span class="sev silent">Silent</span>
+      </div>
+      <div class="cause-body">
+        <div class="field">
+          <span class="lbl">Root cause</span>
+          <p>
+            The biggest cause of ordinary failures: <strong>25% of games</strong>. A consumables
+            trip sends the character A5→A4, and <code>go_to_act(5)</code> then asked
+            <code>detect_current_act()</code> to verify. It committed to whatever cleared 0.68
+            first:
+          </p>
+        </div>
+        <figure>
+          <div class="log"><span class="bad">A5_TOWN_1   0.636</span>   <span class="dim">&lt;- phantom, always at (1046, 40), the top-right corner</span>
+<span class="ok">A4_TOWN_5   0.619</span>   <span class="dim">&lt;- the genuine marker for the act actually occupied</span></div>
+          <figcaption>A 1.7 percentage point gap decided which act the bot believed it was in — so it skipped the travel and ran A5 pathing inside Act 4.</figcaption>
+        </figure>
+        <div class="field fix">
+          <span class="lbl">Fix</span>
+          <p>
+            The winner must now beat the best marker <strong>from any other act</strong> by 5pp.
+            Below that it refuses to answer. Refusing is the safe outcome: callers treat
+            <code>None</code> as "keep the assumed act", so <code>go_to_act</code> actually
+            travels. <strong>Failure rate 25% → 9%.</strong>
+          </p>
+        </div>
+      </div>
+    </article>
+
+    <article class="cause">
+      <div class="cause-head">
+        <span class="bugid">BUG 30</span>
+        <div class="titles">
+          <h3>The NPC search settled for noise</h3>
+          <span class="file">src/npc_manager.py — grid sweep</span>
+        </div>
+        <span class="sev loud">Loud</span>
+      </div>
+      <div class="cause-body">
+        <div class="field">
+          <span class="lbl">Root cause</span>
+          <p>
+            <code>SWEEP_TAG_THRESHOLD = 0.4</code>. A rendered name tag matches almost perfectly,
+            so anything mediocre is noise — and at 0.4 the noise won. Every sweep stopped at the
+            first match over threshold, clicked empty ground, and gave up.
+          </p>
+        </div>
+        <div class="tablewrap">
+          <table>
+            <thead><tr><th>NPC</th><th>Score</th><th>Outcome</th></tr></thead>
+            <tbody>
+              <tr><td>akara</td><td class="num">0.980</td><td><span class="yes">real — dialogue opened</span></td></tr>
+              <tr><td>halbu</td><td class="num">0.995</td><td><span class="yes">real</span></td></tr>
+              <tr><td>larzuk</td><td class="num">0.996</td><td><span class="yes">real</span></td></tr>
+              <tr><td>qual_kehk</td><td class="num">0.424</td><td><span class="no">false — clicked nothing</span></td></tr>
+              <tr><td>malah</td><td class="num">0.501</td><td><span class="no">false</span></td></tr>
+              <tr><td>larzuk</td><td class="num">0.494</td><td><span class="no">false</span></td></tr>
+            </tbody>
+          </table>
+        </div>
+        <div class="field fix">
+          <span class="lbl">Fix</span>
+          <p>Raised to <strong>0.7</strong>, in the gap with margin either side. This is why qual_kehk failed 100% (5 timeouts in 5) while akara succeeded 177 times — nothing was wrong with the Qual-Kehk asset.</p>
+        </div>
+      </div>
+    </article>
+
+    <article class="cause">
+      <div class="cause-head">
+        <span class="bugid">BUG 23</span>
+        <div class="titles">
+          <h3>The only masked template in the folder</h3>
+          <span class="file">assets/templates/a5_town/a5_red_portal.png</span>
+        </div>
+        <span class="sev loud">Loud</span>
+      </div>
+      <div class="cause-body">
+        <div class="field">
+          <span class="lbl">Root cause</span>
+          <p>
+            The capture had the portal's hover tooltip baked in, and alpha was used to hide it.
+            That made <code>alpha_to_mask</code> return a mask, routing the search to
+            <code>cv2.matchTemplate(…, TM_CCOEFF_NORMED, mask=…)</code> — a combination
+            <strong>OpenCV only supports for <code>TM_SQDIFF</code> and
+            <code>TM_CCORR_NORMED</code></strong>.
+          </p>
+        </div>
+        <figure>
+          <div class="log">a5_red_portal.png   4 chan   <span class="bad"> 60.9% opaque   MASKED</span>   <span class="dim">&lt;- the only one that failed</span>
+a5_town_1..12.png   3 chan   <span class="ok">100.0% opaque   no mask</span>  <span class="dim">&lt;- every one that works</span></div>
+          <figcaption>Found by auditing the folder rather than the file: every working template is opaque; the failing one was the sole outlier.</figcaption>
+        </figure>
+        <div class="field fix">
+          <span class="lbl">Fix</span>
+          <p>Recaptured as a plain 3-channel opaque crop of the portal's <strong>upper arch</strong> — the lower ring is occluded by branches. Present 0.949–1.000, absent 0.398–0.514.</p>
+        </div>
+      </div>
+    </article>
+
+    <article class="cause">
+      <div class="cause-head">
+        <span class="bugid">BUG 25</span>
+        <div class="titles">
+          <h3>The pather invented positions and walked into a wall</h3>
+          <span class="file">src/pather.py — find_abs_node_pos()</span>
+        </div>
+        <span class="sev loud">Loud</span>
+      </div>
+      <div class="cause-body">
+        <div class="field">
+          <span class="lbl">Root cause</span>
+          <p>
+            When the primary 0.68 search missed, the fallback dropped to a <strong>0.55
+            first-match</strong> full-image search. <code>A5_TOWN_1</code> scored 0.60–0.62 on
+            unrelated scenery at three different coordinates across three consecutive frames.
+            The landmarks were healthy where genuinely visible (<code>A5_TOWN_6 @ 0.93</code>) —
+            only the fallback was lying.
+          </p>
+        </div>
+        <div class="field fix">
+          <span class="lbl">Fix — three parts</span>
+          <p>
+            Threshold to <strong>0.62</strong>; fallback forced to <code>best_match=True</code>
+            (it was <code>False</code>, so ambiguous frames were decided by <em>dict ordering</em>);
+            and a heading gate rejecting any <em>low-confidence</em> match implying a &gt;90°
+            reversal.
+          </p>
+        </div>
+      </div>
+    </article>
+
+    <article class="cause">
+      <div class="cause-head">
+        <span class="bugid">BUG 29</span>
+        <div class="titles">
+          <h3>An undetectable NPC cost ~80s every game</h3>
+          <span class="file">src/bot.py · src/game_stats.py</span>
+        </div>
+        <span class="sev loud">Loud</span>
+      </div>
+      <div class="cause-body">
+        <div class="field">
+          <span class="lbl">Root cause</span>
+          <p>
+            In nightmare the merc dies most games, so resurrect ran constantly — and it retried
+            once. <code>_merc_resurrect_failed</code> is reset by <code>log_start_game</code>, so
+            it only ever suppressed a second attempt <em>within one game</em>. Game length blew
+            out to 185–250s against a normal ~60s.
+          </p>
+        </div>
+        <div class="field fix">
+          <span class="lbl">Fix</span>
+          <p>
+            A cross-game circuit breaker that <code>log_start_game</code> deliberately does
+            <em>not</em> reset. Over 30 games with an undetectable NPC: <strong>60 hunts → 4</strong>.
+            Measured live, game times went 250s / 185s → 14s, 14s, 43s, 71s.
+          </p>
+        </div>
+      </div>
+    </article>
+
+    <article class="cause">
+      <div class="cause-head">
+        <span class="bugid">BUG 26·27</span>
+        <div class="titles">
+          <h3>Two clicks that missed by ~30 pixels</h3>
+          <span class="file">src/health_manager.py · src/transmute/transmute.py</span>
+        </div>
+        <span class="sev loud">Loud</span>
+      </div>
+      <div class="cause-body">
+        <div class="field">
+          <span class="lbl">Waypoint panel counted as a chicken</span>
+          <p>A mis-steered pather walks the char over the waypoint stone, opening its panel. The panel check counted it twice and killed the game — <strong>3 of 6 games died this way at full health</strong>. Now escaped without counting, bounded at six attempts.</p>
+        </div>
+        <div class="field">
+          <span class="lbl">Gem conversion ran in the wrong tab</span>
+          <p>
+            <code>GEMS_TAB_Y = 100</code>, but the stash tab labels occupy <strong>y 63–78</strong>
+            and the grid starts at y≈87 — so every tab-switch click landed on a stash slot. The
+            whole routine ran in PERSONAL, logging the same warning <strong>800+ times</strong>
+            without one error. Tab switches are now verified, not fire-and-forget.
+          </p>
+        </div>
+      </div>
+    </article>
+
+  </div>
+</section>
+
+<section>
+  <div class="col section-head">
+    <span class="eyebrow">Course corrections</span>
+    <h2>Four times I was wrong</h2>
+  </div>
+  <div class="col body-flow">
+    <p>
+      Each looked correct in isolation, and each was caught by the user or by checking rather
+      than by reasoning. They are recorded because the wrong answer is often the obvious one.
+    </p>
+    <div class="turn">
+      <span class="lbl">1 — the red-portal guard</span>
+      <p>
+        To stop the false "already in Pindle area", I added: if the red portal is on screen we
+        must still be in town. <strong>The portal is visible from both sides</strong> — the
+        return portal renders inside the temple. The guard blocked every genuine entry. It
+        traded a silent false-success for an honest false-failure, which is an improvement and
+        still a bug.
+      </p>
+    </div>
+    <div class="turn">
+      <span class="lbl">2 — XP as proof of a boss kill</span>
+      <p>An XP gain proves <em>something</em> died — the minion pack and the mercenary's kills both count. Confirmation required a loot line naming an actual drop.</p>
+    </div>
+    <div class="turn">
+      <span class="lbl">3 — "the template is degenerate"</span>
+      <p>
+        I blamed the Qual-Kehk asset for a repeated identical 0.424 score. Walking the character
+        to the NPC and scoring the stored template against a frame where the tag renders gave
+        <strong>1.000</strong>. The template was perfect; the <em>threshold</em> was wrong. A
+        repeated identical score means the search is settling for noise — check the threshold
+        before blaming the asset.
+      </p>
+    </div>
+    <div class="turn">
+      <span class="lbl">4 — selling the charms</span>
+      <p>
+        I read 673 blocked-sell log lines as a clog to clear and disabled the guard. It vendored
+        five resistance charms before it was caught. <strong>The blocked lines were the guard
+        working</strong>, and the pickit's "Discarding" verdict was never a judgement that they
+        were junk — <code>LAPIS SMALL CHARM OF VITA</code> failed its rule only because the rule
+        wanted <code>coldresist &gt;= 11</code> and it had 7.
+      </p>
+    </div>
+  </div>
+</section>
+
+<section>
+  <div class="col section-head">
+    <span class="eyebrow">The durable part</span>
+    <h2>Instrumentation, so the next one takes minutes</h2>
+  </div>
+  <div class="col body-flow">
+    <p>
+      Every bug above cost hours because the log recorded events, not <em>structure</em>. The
+      last commits fix that: one fixed-width, machine-readable line per step, spawn to loot,
+      every step timed.
+    </p>
+    <figure>
+      <div class="log"><span class="dim">grep "TL&gt;" log/log.txt</span>
+
+TL&gt; g2 r1 | game  | spawn             | ok    | at a5_town_start
+TL&gt; g2 r1 | town  | inspect_inventory | ok    | took=3.0s   | in pack=2 keep=0 sell=2
+TL&gt; g2 r1 | town  | repair            | ok    | took=19.3s  | at a5_larzuk
+TL&gt; g2 r1 | town  | item_sell         | ok    | SOUL IMPALER @ (928, 465)
+<span class="bad">TL&gt; g2 r1 | town  | resurrect_merc    | fail  | took=113.6s | NPC not reachable</span>
+TL&gt; g2 r1 | town  | maintenance       | ok    | took=137.6s | at a5_larzuk
+<span class="bad">TL&gt; g2 r1 | run   | approach          | fail  | took=71.9s  | step=click_red_portal</span></div>
+      <figcaption>
+        That failed game spent <strong>113.6s of its 137.6s town visit</strong> on a merc
+        resurrect that achieved nothing — 83% of the time, on a step that was not the one that
+        ended the game. That proportion was invisible before.
+      </figcaption>
+    </figure>
+    <p>
+      Failures get a self-contained record (<code>grep "FAIL&gt;"</code>) carrying the reason,
+      location, screenshot, the three slowest steps, and a breadcrumb trail of the whole game
+      with <code>!</code> marking failures. <strong>The step named in the reason is frequently
+      not the one that caused the problem</strong> — the trail is what shows that.
+    </p>
+    <p>
+      A digest posts to Discord every two hours, aggregated from the same timeline so the report
+      and the log cannot drift apart.
+    </p>
+    <figure>
+      <div class="log">**Timing report** - last 2.0h
+Games: 12 (11 ok, 1 failed - 8.3%)
+Avg town 20s | approach 14s | battle 19s | cycle ~52s
+
+__Slowest steps (avg)__
+`town.resurrect_merc     `    33s  x6  (2 fail)
+`run.approach            `    14s  x28  (1 fail)</div>
+      <figcaption>Real measured output. Rankings exclude umbrella steps and deliberate stealth pauses, which would otherwise always come first and say nothing.</figcaption>
+    </figure>
+  </div>
+</section>
+
+<section>
+  <div class="col section-head">
+    <span class="eyebrow">Prevention</span>
+    <h2>Rules carried into the docs</h2>
+  </div>
+  <div class="col body-flow">
+    <div class="checks">
+      <div class="check">
+        <span class="mark">✓</span>
+        <span class="ct"><strong>Verify boss runs with XP delta and loot lines.</strong> Never with <code>failed:false</code> — that is the bot agreeing with itself.</span>
+      </div>
+      <div class="check dont">
+        <span class="mark">✕</span>
+        <span class="ct"><strong>Never mask part of a template to hide an artifact.</strong> Masked <code>TM_CCOEFF_NORMED</code> is unsupported in OpenCV and fails unpredictably.</span>
+      </div>
+      <div class="check">
+        <span class="mark">✓</span>
+        <span class="ct"><strong>Prove a template fails on a frame where the subject is definitely visible</strong> before recapturing it. A repeated identical score means the threshold is too low, not that the asset is bad.</span>
+      </div>
+      <div class="check">
+        <span class="mark">✓</span>
+        <span class="ct"><strong>Confirm any click that changes UI mode</strong> — a tab, a page, a panel. An unverified click that "should" have worked will fail silently forever.</span>
+      </div>
+      <div class="check dont">
+        <span class="mark">✕</span>
+        <span class="ct"><strong>Never let a detector guess on a thin margin.</strong> Refusing to answer is usually the safe outcome; a wrong confident answer skips the recovery path entirely.</span>
+      </div>
+      <div class="check">
+        <span class="mark">✓</span>
+        <span class="ct"><strong>Read repeated warnings as a system working, not a clog to clear.</strong> 673 blocked sells were a guard doing its job.</span>
+      </div>
+      <div class="check">
+        <span class="mark">✓</span>
+        <span class="ct"><strong>Restart the bot after editing <code>src/</code> or config.</strong> Python loads modules and config at process start; a running bot keeps executing the old code.</span>
+      </div>
+    </div>
+  </div>
+</section>
+
+<footer>
+  <div class="col" style="display:flex;flex-wrap:wrap;gap:10px 20px;justify-content:space-between;width:100%">
+    <span>my-botty · branch <strong>stable</strong> · 17 commits</span>
+    <span>9 root causes · 4 corrections · verified live</span>
+  </div>
+</footer>
+
+</div>

docs/recovery_and_error_logging.md

@@ -0,0 +1,112 @@
+# Recovery & Error Logging
+
+This document describes the failure-recovery and diagnostics features that let the
+bot survive broken runs and makes every failure reviewable after the fact.
+
+## 1. Comprehensive failure logging
+
+Every town action (`src/town/a1.py` … `a5.py`) and every run `approach()` method
+(`src/run/*.py`) now logs the **specific step that failed** via `Logger.error`
+instead of silently returning `False`. Examples of what now gets logged:
+
+- `A1 resurrect: open_npc_menu(KASHYA) failed — NPC not found or menu did not open`
+- `A4 open_wp: waypoint template A4_WP/A4_WP_2 not found — WP may be obscured or template mismatch`
+- `Vizier approach: use_wp('River of Flame') failed`
+- `Mephisto approach: go_to_act(3) failed`
+- `gamble: go_to_act(4) failed from A1_TOWN_START — cannot reach Jamella`
+
+Town-manager fallbacks (`resurrect`, `identify`, `open_stash`, `gamble`, `stash`,
+`heal`) also log when their `go_to_act` recovery hop fails, so you can tell whether
+the problem was reaching the NPC, opening the menu, pressing the button, or the
+panel never opening.
+
+This means that "if Cain is missing, if the WP is not enabled, if some other game
+factor is not in place" the reason ends up in the log.
+
+## 2. Error screenshots
+
+When a run fails, the bot saves a screenshot so logs and visuals can be compared.
+
+- **Where:** `log/screenshots/error/`
+- **Filename:**
+  `error_<run_name>_<reason>_g<game#>_r<run#>_<YYYYMMDD_HHMMSS>.png`
+  e.g. `error_run_vizier_approach_failed_g12_r47_20260607_143501.png`
+- **When:** captured on three failure types inside `Bot._run_wrapper`:
+  - `approach_failed` — couldn't reach the boss
+  - `battle_failed` — boss fight didn't complete
+  - `exception_<ExceptionType>` — an uncaught exception during battle
+- The `g<game#>_r<run#>` in the filename matches the `game`/`run` fields in the
+  structured event log (`log/stats/events_*.jsonl`) and the `Logger` output, so a
+  screenshot can be lined up with its log lines.
+
+### Discord delivery
+
+When a run fails the bot also sends the **error message and the screenshot** to
+Discord (as a red embed with the screenshot attached). For the generic webhook API
+a text-only message is sent (the screenshot still lands on disk). This is on by
+default and can be turned off two ways:
+
+- `[general] discord_log_errors=0` — master switch for error → Discord
+- `[discord_events] error=0` — fine-grained event toggle (Discord API only)
+
+### Config
+
+`config/params.ini` → `[general]`:
+
+```ini
+; save a screenshot every time a run fails (falls back to info_screenshots if unset)
+error_screenshots=1
+; also send the error message + screenshot to Discord on each run failure
+discord_log_errors=1
+```
+
+`config/params.ini` → `[discord_events]`:
+
+```ini
+; send run-failure notifications (message + error screenshot)
+error=1
+```
+
+Set `error_screenshots=0` to turn off error screenshots entirely. Old screenshots
+are size-rotated using the `info_*` limits in `[log_rotation]` (or optional
+`error_max_files` / `error_max_mb` if you add them).
+
+## 3. Auto-disable broken runs (recovery)
+
+Previously, repeated failures of one run could stall the whole bot. Now each run
+has its own **consecutive failure counter**:
+
+- Every failed attempt of a run increments its counter; a single success resets it.
+- When a run reaches the threshold of consecutive failures, it is **disabled for
+  the rest of the session**. The bot keeps doing the other runs instead of stopping.
+- Disabling is logged and sent to Discord (if enabled):
+  `DISABLING run_vizier for this session: it failed 5 consecutive times. …`
+- The disable is **session-only / in-memory** — it does not edit `params.ini`.
+  Restarting the bot re-enables the run. This is intentional so a transient problem
+  (e.g. a game hiccup) doesn't permanently change your config.
+- If **all** runs end up disabled there are no routes left, so the bot saves a
+  session report and stops cleanly (kills the game and exits) rather than looping
+  over empty games or pointlessly restarting D2R.
+
+### Config
+
+`config/params.ini` → `[general]`:
+
+```ini
+; after this many CONSECUTIVE failures of the same run, disable just that run
+; for the rest of the session and keep doing the others. A success resets the count.
+disable_run_after_failures=5
+```
+
+Set this high (e.g. `9999`) to effectively disable the auto-disable behaviour.
+
+## Implementation notes
+
+- `src/bot.py`
+  - `__init__`: `_run_failure_counts`, `_disabled_runs`, `_max_run_failures`
+  - `_record_run_result(run_name, failed)`: counts failures, disables runs, resets on success
+  - `_save_error_screenshot(run_name, reason)`: writes the diagnostic screenshot
+  - `_run_wrapper(...)`: calls both on each failure path (approach / battle / exception)
+- `src/config.py`: parses `error_screenshots` and `disable_run_after_failures`
+  (both via `_select_optional`, so existing configs keep working).
+- `src/utils/log_rotation.py`: routes the `error/` directory through rotation.

docs/run_countess_plan.md

@@ -0,0 +1,71 @@
+# Run Countess Route Plan
+
+## Goal
+Record path data and templates for the Countess run (Act 1 Forgotten Tower) so it executes reliably with Hammerdin.
+
+## Current State
+- `src/run/countess.py` exists and is wired into bot.py
+- `Location` constants defined in `pather.py`: A1_TOWER_LEVEL2_ENTER through A1_TOWER_LEVEL5_ENTER, A1_COUNTESS_SAFE_DIST
+- `config/game.ini` has 5 path entries — all set to `0,0` (placeholders)
+- Run is disabled in `params.ini` (`order=run_arcane, run_pindle`)
+
+## Paths Needed (5 entries in game.ini)
+
+| Config Key | Description | How to Record |
+|---|---|---|
+| `a1_tower_level2_enter` | First click point inside tower after entering from Black Marsh | Walk into tower, record first position on L2 |
+| `a1_tower_level3_enter` | Click point at top of stairs to L3 | Walk up stairs, record position at L3 entrance |
+| `a1_tower_level4_enter` | Click point at top of stairs to L4 | Walk up stairs, record position at L4 entrance |
+| `a1_tower_level5_enter` | Click point at top of stairs to L5 | Walk up stairs, record position at L5 entrance |
+| `a1_countess_safe_dist` | Click point near Countess (safe hammer range) | Walk to Countess, step back to hammer range |
+
+## Recording Steps
+
+### 1. Record path coordinates
+1. Start botty, let it waypoint to Black Marsh
+2. Enter the tower manually (walk through the door)
+3. Run `static_run_recorder.py`:
+   ```
+   conda activate botty
+   cd src/utils
+   python static_run_recorder.py
+   ```
+4. Right-click each target position in game order (L2 enter → L3 enter → L4 enter → L5 enter → Countess safe distance)
+5. Copy the printed coordinate list
+
+### 2. Update game.ini
+Replace the `0,0` values with recorded coordinates. Format: `x,y` per line, or comma-separated for multi-point paths.
+
+Example:
+```ini
+a1_tower_level2_enter=400,300
+a1_tower_level3_enter=410,280
+a1_tower_level4_enter=420,260
+a1_tower_level5_enter=430,240
+a1_countess_safe_dist=450,200
+```
+
+### 3. Test the run
+1. Add `run_countess` back to `order=` in `params.ini`
+2. Start botty and verify it navigates through all tower levels
+3. Adjust coordinates if it misses stairs or clicks in wrong spots
+
+## Hammerdin-Specific Notes
+- `atk_len_countess=3.0` is already set in `params.ini`
+- Hammerdin uses `kill_countess()` method — ensure conviction aura is active before engaging
+- Safe distance should account for hammer swing range (~3-4 game units)
+
+## Path Recorder Alternative
+If `static_run_recorder.py` is inconvenient, use `node_recorder.py`:
+```
+cd src/utils
+python node_recorder.py
+```
+Follow on-screen prompts to record nodes, then copy the generated path coordinates.
+
+## Acceptance Criteria
+- Botty enters Black Marsh via waypoint
+- Navigates L2 → L3 → L4 → L5 without getting stuck
+- Reaches Countess and engages
+- Completes kill and returns to town
+- No false template matches on tower stairs

docs/run_cows_plan.md

@@ -0,0 +1,225 @@
+# Run Cows Route Plan
+
+## Goal
+Add a stable `run_cows` route that can:
+- create/open Secret Cow Level portal in Act 1,
+- clear cows with safe movement and loot handling,
+- exit cleanly and continue normal run rotation.
+
+## Scope
+This plan is for implementation and testing in this repo only.  
+No behavior changes to existing routes unless needed for shared helpers.
+
+## Constraints
+- Keep it compatible with current config style (`[routes]`, `[char]`, per-build sections).
+- Prioritize stability over speed for first release.
+- Avoid forcing hard dependencies on OCR for core route progression.
+
+## Route Design
+1. Town prep in Act 1:
+   - go to stash/cube workflow position.
+   - ensure cube available.
+2. Cow portal creation:
+   - verify Wirt's Leg + Tome of Town Portal availability.
+   - transmute in Act 1 town.
+   - detect red portal reliably.
+3. Entry and combat:
+   - enter portal.
+   - execute deterministic clear pattern (clockwise + center sweep).
+   - run character-specific attack loop with timeout guard.
+4. Loot and exit:
+   - reuse existing pickit flow.
+   - return through TP or save/exit based on safety state.
+
+## Code Changes (Planned)
+1. Add new run module:
+   - `src/run/cows.py` (`name = "run_cows"`)
+   - methods: `approach()`, `battle()`
+2. Wire route registration:
+   - include `run_cows` in route loader/dispatcher used by `bot.py`.
+3. Town/cube helpers:
+   - add minimal helper(s) for cube transmute flow if missing.
+4. Pathing/templates:
+   - add cow portal/town waypoint templates only as needed.
+   - add robust fallback checks for portal detect/select.
+5. Config/docs:
+   - document `run_cows` in `config/params.ini` route comments.
+   - add usage notes to `README.md`.
+
+## Config Additions (Planned)
+In `[char]` or dedicated route section:
+- `cows_clear_timeout_s` (default safe value, e.g. 90-150)
+- `cows_use_tp_exit` (1/0)
+- `cows_repair_before_run` (1/0, default 1)
+
+Keep defaults conservative.
+
+## Safety & Recovery
+- If portal creation fails: log once, abort current cow attempt, continue next route.
+- If portal click fails: retry from known anchor position once.
+- If in-combat timeout reached: force exit sequence and mark run failed.
+- If inventory full during cows: run pickit stop logic and exit route early.
+
+## Character Compatibility Strategy
+Phase 1 support:
+- Blizz Sorc
+- Hammerdin
+- FoHdin
+
+Phase 2 support:
+- remaining builds with route-specific combat tuning.
+
+## Telemetry / Stats
+Track in `game_stats`:
+- `run_cows` attempts
+- success/fail count
+- elapsed duration
+- rune/valuable item outcomes (already covered by global item tracking)
+
+## Test Plan
+1. Unit-ish tests:
+   - route registration includes `run_cows`
+   - config parsing accepts route key without crash
+2. Integration smoke (manual):
+   - portal creation success path
+   - missing Wirt’s Leg failure path
+   - missing tome failure path
+   - portal click retry path
+3. Stability run:
+   - 25+ games mixed with existing routes
+   - verify no regression in Pindle/Trav/Diablo flows
+
+## Assets / Inputs Needed
+- Screenshot set for:
+  - Act 1 stash/cube interaction points
+  - successful cow portal in town
+  - entry location inside cows
+- Optional:
+  - map movement recording for an efficient clear loop.
+
+## Screenshot + Coordinates SOP (Detailed)
+
+### Why this matters
+Cow route reliability depends on visual matching and deterministic interaction points.  
+Bad captures (cursor overlap, wrong UI state, mismatched lighting) are the #1 reason for unstable routes.
+
+### Environment lock before capture
+1. Use the same D2R setup that bot runtime uses:
+   - same resolution,
+   - same UI scale,
+   - same graphics preset,
+   - same window mode.
+2. Apply Botty auto settings before capture.
+3. Keep panel state consistent:
+   - inventory closed unless the step explicitly needs it,
+   - item labels in known state,
+   - no obstructing chat/quest panels.
+
+### Capture tools in this repo
+- `screenshot_tool.py`
+- `quest_screenshot_tool.py`
+- `quest_debug.py`
+- `desktop_snap.py`
+- Graphic debugger hotkey from `config/params.ini` (`advanced_options.graphic_debugger_key`)
+
+### Capture types required per target
+For every actionable object, collect both:
+1. Full-context screenshot (for debugging and reproducibility)
+2. Tight template crop (for matching)
+
+Template crop rules:
+- include only the unique visual feature + small margin,
+- avoid cursor overlap,
+- avoid tooltip overlap unless tooltip itself is the detection target,
+- avoid over-cropping edges that change with animation.
+
+### Coordinate types to record
+For each target interaction, record:
+1. Screen/monitor click point used when interaction succeeds.
+2. Whether template-center click worked.
+3. Optional fallback click point if template-center is unreliable.
+4. ROI used (if narrowed in code/tests).
+5. Confidence range seen in quick manual validation.
+
+### Mandatory cows capture checklist (in order)
+1. Act 1 town anchor (post-spawn)
+   - full screenshot
+   - possible anchor templates
+2. Stash interaction
+   - full screenshot near stash
+   - stash template crop(s)
+   - successful click coordinate(s)
+3. Cube interaction/transmute context
+   - inventory+cube state screenshot
+   - transmute UI screenshot references
+4. Cow portal in town (opened)
+   - full screenshot with portal visible
+   - portal template crops (2-3 variants)
+   - portal click success coordinates
+5. Cow level entry area
+   - immediate post-load screenshot(s)
+   - safe first-move anchor capture
+6. Exit path reference
+   - screenshot where exit action is normally triggered
+   - any portal/escape interaction target needed
+
+### Variants to capture for robustness
+Per critical template (stash, portal, entry anchor), capture:
+- 3-5 samples with slight camera/position variation,
+- at least one “busy background” sample,
+- one sample with nearby NPC/player clutter if possible.
+
+### Validation pass before route coding
+For each template candidate:
+1. Verify it matches across the sample set at intended threshold.
+2. Verify it does not false-match to nearby unrelated objects.
+3. Verify click-at-center triggers the intended interaction.
+4. Record accepted threshold and failure notes.
+
+Reject templates that:
+- only match at very low confidence,
+- require exact pixel-perfect camera alignment,
+- frequently collide with non-target objects.
+
+### File naming convention
+Use deterministic names aligned with route semantics, e.g.:
+- `A1_COW_PORTAL_0`
+- `A1_COW_PORTAL_1`
+- `A1_STASH_REF_0`
+- `COW_ENTRY_ANCHOR_0`
+
+Keep source full screenshots in a dated debug folder and final template crops in the templates location used by the project.
+
+### Capture log format (what testers should write down)
+For each target:
+- `template_name`
+- capture timestamp
+- location/context
+- successful click point(s)
+- confidence range observed
+- notes on failed attempts
+
+### Inputs needed from testers (you + friend)
+Provide:
+1. ZIP/folder of full screenshots + template crops.
+2. Short markdown/text capture log using format above.
+3. Character/build used during capture (Blizz Sorc / FoH / Hammerdin).
+4. Any observed instability notes (miss-clicks, wrong target matches, etc.).
+
+### Acceptance criteria for capture package
+Capture package is ready for implementation when:
+- all mandatory checklist targets are present,
+- each critical target has multi-sample variants,
+- at least one tester validated successful interaction per target,
+- naming is consistent and unambiguous.
+
+## Rollout
+1. Land Phase 1 behind route opt-in (`order=... ,run_cows`).
+2. Keep disabled by default in examples until 10+ stable sessions.
+3. Promote to documented standard route after stability threshold.
+
+## Definition of Done
+- `run_cows` can run repeatedly without manual intervention.
+- Failure modes recover without crashing whole bot loop.
+- Route is documented in `params.ini` and `README.md`.
+- CI/lint/tests pass for changed files.