stable: clean branch for end users — removed dev-only files (botty_next, Docker, debug tools, internal docs)

Adds 41 tests covering the highest-risk production paths:

test/npc/test_npc_manager.py (14 tests):
  - _action_btns_visible: missing NPC, white found, nothing found
  - open_npc_menu: fast path (Bug 6), name-tag in ROI (Bug 3),
    ROI gate blocks outside-ROI click (Bug 7), pose-distance gate (Bug 4),
    body+pose confirmed triggers click, timeout returns False
  - press_npc_btn: white/blue/grayscale fallback chain, red=cannot-afford, nothing found

test/town/test_town_manager.py (27 tests):
  - get_act_from_location: all five acts + sub-locations, bad input
  - identify: True never returned (Bug 1), A5 fallback (Bug 5),
    A5-also-fails returns new_loc, _cain_failed_acts skip
  - open_wp: budget increment, exhaustion fast-path (Bug 9), reset, success
  - buy_consumables: unknown loc, trade-menu failure (Bug 10), success, no-need skip
  - stash: current-act delegation, A5 travel when act can't stash

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

.coveragerc

@@ -0,0 +1,29 @@
+# .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
+    pragma: no cover
+    # Don't complain if tests don't hit defensive assertion code:
+    raise AssertionError
+    raise NotImplementedError
+
+    # Don't complain if non-runnable code isn't run:
+    if 0:
+    if __name__ == "__main__":

.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/ISSUE_TEMPLATE/bug_report.md

@@ -7,6 +7,9 @@ assignees: ''
 
 ---
 
+**Is it really a Bug report?**
+If you have questions regarding setup or feature information/requests, please consider asking in the Discord channel: https://discord.gg/Jf3J8cuXWg. The Github issues section is for bug reports only.
+
 **Botty Version**
 What version of botty are you running? Are you using a pre-compiled release? Which one. Are you using the latest master branch? On which commit?
 
@@ -26,7 +29,7 @@ A clear and concise description of what you expected to happen.
 If applicable, add screenshots or even better a short video of the error happening to help explain your problem. Botty often automatically makes screenshots when something goes wrong. Add these also.
 
 **Logs**
-Add the relevant part of the info.log in this section. Either upload it as a fail or copy-paste the relevant part in here.
+Add the relevant part of the log/log_(time).txt in this section. Either upload it as a fail or copy-paste the relevant part in here.
 
 **Additional info**
 Add any other information about the problem here.

.github/workflows/ci.yml

@@ -1,47 +1,167 @@
 name: Botty - CI
-on: [pull_request]
+on:
+  workflow_dispatch:
+  pull_request:
+  push:
+    branches: [main, mine]
+    # Pushing a version tag (e.g. `git tag v0.8.5 && git push --tags`) builds,
+    # smoke-tests, then creates the GitHub release with the zip attached — all
+    # in one run. If the build fails, no release is ever created.
+    tags: ['v*']
+
+# Default GITHUB_TOKEN is read-only; the build job's release step needs
+# contents:write to create the release and attach the built zip
+# (else HTTP 403 "Resource not accessible by integration").
+permissions:
+  contents: write
+
 jobs:
-  tests:
+  test:
     runs-on: windows-latest
     steps:
-      - uses: actions/checkout@v2
-      - name: Setup Miniconda Python 3.8
-        uses: actions/setup-python@v2
+      - uses: actions/checkout@v4
+
+      - name: Setup Miniconda Python 3.10
+        uses: conda-incubator/setup-miniconda@v3
         with:
-          python-version: 3.8
-      # TODO: The below setp only chaches conda packages, need to cache pip seperatly
-      - name: Cache conda
-        uses: actions/cache@v2
-        env:
-          # Increase this value to reset cache if environment.yml has not changed
-          CACHE_NUMBER: 0
-        with:
-          path: ~/conda_pkgs_dir
-          key: conda-${{ env.CACHE_NUMBER }}-${{hashFiles('environment.yml') }}
-      - name: Cache pip
-        uses: actions/cache@v2
-        env:
-          # Increase this value to reset cache if environment.yml has not changed
-          CACHE_NUMBER: 0
-        with:
-          path: ~/pip
-          key: pip-${{ env.CACHE_NUMBER }}-${{ hashFiles('environment.yml') }}
-      - name: Install Miniconda
-        uses: conda-incubator/setup-miniconda@v2
-        with:
-          python-version: 3.8
+          python-version: '3.10'
           activate-environment: botty
           channel-priority: strict
-          environment-file: environment.yml
-          use-only-tar-bz2: true # Needed for caching
-      - name: Activate conda
-        shell: powershell
-        run: |
-          C:\Miniconda\condabin\conda.bat init powershell
-          set PYTHONPATH=./src
-      - name: Pytest
+          environment-file: environment-win11.yml
+          use-only-tar-bz2: false
+
+      - name: Python version
         shell: powershell
         run: |
           C:\Miniconda\condabin\conda.bat activate botty
           python -c "import sys; print(sys.version)"
-          pytest -v -s
+
+      - name: Syntax check
+        shell: powershell
+        env:
+          PYTHONPATH: ./src
+        run: |
+          $ErrorActionPreference = "Stop"
+          C:\Miniconda\condabin\conda.bat activate botty
+          python -m compileall -q src tools test scripts
+
+      - name: Tests
+        shell: powershell
+        env:
+          PYTHONPATH: ./src
+          RUN_ENV: test
+        run: |
+          $ErrorActionPreference = "Stop"
+          C:\Miniconda\condabin\conda.bat activate botty
+          python -m coverage run -m pytest -v
+
+      - name: Coverage report
+        shell: powershell
+        env:
+          PYTHONPATH: ./src
+        run: |
+          $ErrorActionPreference = "Stop"
+          C:\Miniconda\condabin\conda.bat activate botty
+          python -m coverage xml --ignore-errors
+
+  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-win11.yml
+          use-only-tar-bz2: false
+
+      - name: Install Tesseract (bundled into the release for click-and-run OCR)
+        shell: powershell
+        run: |
+          $ErrorActionPreference = "Stop"
+          choco install tesseract --no-progress -y
+          if (-not (Test-Path "C:\Program Files\Tesseract-OCR\tesseract.exe")) {
+            throw "Tesseract install did not produce tesseract.exe"
+          }
+
+      - name: Build exe
+        shell: powershell
+        env:
+          BOTTY_NO_RENAME: '1'
+          PYTHONPATH: ./src
+        run: |
+          C:\Miniconda\condabin\conda.bat activate botty
+          python build.py --conda_path C:\Miniconda
+
+      - name: Verify Tesseract was bundled
+        shell: powershell
+        run: |
+          $ErrorActionPreference = "Stop"
+          $BOTTY_DIR = Get-ChildItem -Directory -Name | Where-Object { $_ -match '^botty_v' } | Select-Object -First 1
+          $tess = Join-Path $BOTTY_DIR "tesseract\tesseract.exe"
+          if (-not (Test-Path $tess)) { throw "Tesseract was not bundled into $BOTTY_DIR" }
+          Write-Host "Bundled: $tess"
+
+      - name: Launch smoke test (built executables)
+        shell: powershell
+        run: |
+          $ErrorActionPreference = "Stop"
+          $BOTTY_DIR = Get-ChildItem -Directory -Name | Where-Object { $_ -match '^botty_v' } | Select-Object -First 1
+          if (-not $BOTTY_DIR) { throw "No botty_v* build directory found." }
+
+          $mainExe = Join-Path $BOTTY_DIR "main.exe"
+          $shopperExe = Join-Path $BOTTY_DIR "shopper.exe"
+          if (-not (Test-Path $mainExe)) { throw "Missing $mainExe" }
+          if (-not (Test-Path $shopperExe)) { throw "Missing $shopperExe" }
+
+          $procs = @()
+          try {
+            $mainProc = Start-Process -FilePath $mainExe -PassThru -WindowStyle Hidden
+            Start-Sleep -Seconds 6
+            if ($mainProc.HasExited) { throw "main.exe exited early with code $($mainProc.ExitCode)" }
+            $procs += $mainProc
+
+            $shopperProc = Start-Process -FilePath $shopperExe -PassThru -WindowStyle Hidden
+            Start-Sleep -Seconds 6
+            if ($shopperProc.HasExited) { throw "shopper.exe exited early with code $($shopperProc.ExitCode)" }
+            $procs += $shopperProc
+          }
+          finally {
+            foreach ($p in $procs) {
+              if ($p -and -not $p.HasExited) {
+                Stop-Process -Id $p.Id -Force
+              }
+            }
+          }
+
+      - 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
+
+      # On a version-tag push, create the release (if absent) and attach the
+      # zip atomically. Runs only after the build + smoke test above succeed.
+      - name: Create release and upload zip
+        if: startsWith(github.ref, 'refs/tags/v')
+        uses: softprops/action-gh-release@v2
+        with:
+          files: botty_v*.zip
+          fail_on_unmatched_files: true
+          generate_release_notes: true

.gitignore

@@ -4,19 +4,99 @@ __pycache__/
 playground.py
 *.pyc
 *.pyo
-*.log
 *.spec
 dist/
 build/
-run.build/
-run.dist/
-run.onefile-build/
+main.build/
+main.dist/
+main.onefile-build/
 generated/
 botty_v*/
 custom.ini
-*info_*.png
+config/custom.ini
+config/custom.*.ini
+config/*.local.ini
 .vscode
 .vs/
+# 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_forum.html
+
+# 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/*
+!config/bnip/.gitkeep
+stats/
 info_screenshots/
+item_error_screenshots/
 loot_screenshots/
-.venv
+pickit_screenshots/
+*info_log_parsed.txt
+*info_*.png
+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/

ARCHITECTURE.md

@@ -0,0 +1,324 @@
+# Botty Architecture
+
+## Overview
+
+Botty is an autonomous bot for Diablo II Resurrected (D2R) that uses computer vision and native Windows API input to run endgame content without kernel drivers. It's built in Python 3.10 and distributed as a PyInstaller onefile executable.
+
+## Entry Points
+
+Two standalone executables are built from the same source:
+
+| Executable | Source | Purpose |
+|---|---|---|
+| `main.exe` | `src/main.py` | Main bot controller |
+| `shop.exe` | `src/shopper.py` | Standalone vendor shopping |
+
+## Threading Model
+
+main.py spawns three concurrent threads:
+
+```
+main.py (main thread)
+├── game_controller.py (bot loop thread)
+│   └── bot.py (state machine thread)
+├── health_manager.py (background thread)
+└── death_manager.py (background thread)
+```
+
+- **Main thread**: Shows UI, registers hotkeys (F7-F12), calls `keyboard.wait()` to block
+- **Game Controller**: Orchestrates the bot loop - starts/stops the bot thread, handles game recovery, tracks stats
+- **Health Manager**: Polls health/mana/merc every ~1s, auto-potions, chickens when low
+- **Death Manager**: Polls every ~1s for "You Have Died" screen, triggers recovery
+
+## Core Modules
+
+### Input Layer (`src/input_layer/`)
+
+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` | 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.
+
+### Screen Capture (`src/screen.py`)
+
+Handles D2R window detection and screenshot capture via MSS library. Converts between coordinate systems: Monitor (top-left first monitor), Screen (per-monitor), Absolute (character center), Relative (template-matched).
+
+### Image Recognition (`src/template_finder.py`)
+
+Template matching via OpenCV (`cv2.matchTemplate`). Searches against pre-captured asset templates in `assets/templates/`. Returns match position and validity.
+
+### UI Detection (`src/ui/`)
+
+| File | Detects |
+|---|---|
+| `main_menu.py` | Main menu buttons (play, save & exit) |
+| `character_select.py` | Character portraits and selection |
+| `skills.py` | Skill bar state, skill availability |
+| `meters.py` | Health/mana globes, potion charges |
+| `player_bar.py` | Player status bar, XP bar |
+| `view.py` | Minimap, inventory screen detection |
+| `waypoint.py` | Waypoint portal selection |
+| `error_screens.py` | Error dialogs, death screen |
+| `loading.py` | Loading screen detection |
+
+### Pathing (`src/pather.py`)
+
+Pathfinding via reference template matching with relative coordinates. Each location has numbered nodes with template references. The `Location` class defines named destinations (towns, dungeons). Path definitions map (start, end) pairs to node sequences.
+
+### Character System (`src/char/`)
+
+Inheritance hierarchy:
+
+```
+IChar (abstract base)
+├── Basic / Basic_Ranged
+├── Paladin
+│   ├── Hammerdin
+│   └── FoHdin
+├── Sorceress (base for all sorc builds)
+│   ├── BlizzSorc
+│   ├── BlizzorbSorc
+│   ├── NovaSorc
+│   ├── LightSorc
+│   └── HydraSorc
+├── Amazon
+│   └── Javazon
+├── Trapsin
+├── Barbarian
+├── Necro
+├── Poison_Necro
+├── Bone_Necro
+└── Warlock
+    ├── FireLock
+    ├── EchoLock
+    └── AbyssLock
+```
+
+Each character class implements: run methods, skill usage, attack patterns, position-specific behavior.
+
+### Run System (`src/run/`)
+
+| File | Run |
+|---|---|
+| `pindle.py` | Pindle of Purity |
+| `shenk_eld.py` | Shenk + Eldritch |
+| `trav.py` | Travincal |
+| `nihlathak.py` | Nihlathak's Temple |
+| `arcane.py` | Arcane Sanctuary |
+| `diablo.py` | Chaos Sanctuary + Diablo |
+| `vizier.py` | Vizier (Seal Boss) |
+| `baal.py` | Baal |
+| `mephisto.py` | Mephisto |
+| `andariel.py` | Andariel |
+| `countess.py` | Countess |
+| `level.py` | General leveling runs |
+
+### Town System (`src/town/`)
+
+`TownManager` orchestrates act-specific town routines. Each act (A1-A5) has its own module with NPC interactions, waypoint usage, and stash handling.
+
+### Inventory (`src/inventory/`)
+
+| File | Purpose |
+|---|---|
+| `belt.py` | Belt slot management, potion detection |
+| `personal.py` | Personal inventory grid, item positions |
+| `vendor.py` | Vendor trade window |
+| `stash.py` | Personal stash, shared stash |
+| `cube.py` | Horadric Cube |
+| `common.py` | Shared inventory utilities |
+
+### Item Recognition (`src/item/`)
+
+| File | Purpose |
+|---|---|
+| `pickit.py` | Pickup decisions, item filtering |
+| `consumables.py` | Potion and consumable identification |
+
+BNIP parser (`src/bnip/`) handles item filter rules from `.bnip` files.
+
+### Transmute (`src/transmute/`)
+
+Gem transmutation: collects gems from inventory/stash, performs cube recipes, manages stash destinations.
+
+### Messages (`src/messages/`)
+
+| File | Purpose |
+|---|---|
+| `messenger.py` | Message dispatcher |
+| `discord_embeds.py` | Discord webhook formatting |
+| `generic_api.py` | Generic HTTP webhook support |
+
+### Configuration (`src/config.py`)
+
+Singleton that merges config files in priority order:
+`custom.ini` > `params.ini` > `game.ini` > `shop.ini` > `transmute.ini`
+
+Supports variable substitution via `[variables]` sections.
+
+### Stealth System (`src/utils/stealth.py`)
+
+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/`)
+
+| File | Purpose |
+|---|---|
+| `misc.py` | Window management, DPI awareness, timing utilities |
+| `restart.py` | Game launch/kill, D2R always-on-top |
+| `auto_settings.py` | D2R game settings adjustment |
+| `key_detector.py` | Auto-detect skill bindings from `.key`/`.keyo` files |
+| `graphic_debugger.py` | Visual debugging overlay |
+| `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
+
+```
+D2R Game Window
+       │
+       ▼
+  screen.grab() ──→ Image (numpy array)
+       │
+       ▼
+  template_finder.search() ──→ Position + Validity
+       │
+       ▼
+  ui_manager.detect_screen_object() ──→ Current UI state
+       │
+       ▼
+  bot.py (state machine) ──→ Decides next action
+       │
+       ▼
+  char.run() / pather.go_to() ──→ Movement + Interaction
+       │
+       ▼
+  input_layer.keyboard.send() / mouse.click() ──→ SendInput API
+       │
+       ▼
+  D2R Game Window (response captured on next grab)
+```
+
+## Build System
+
+- **Build script**: `build.py` (PyInstaller with `--onefile --noconsole`)
+- **Dependencies**: `pyproject.toml`
+- **Distribution**: Random exe name, no console window, no kernel drivers
+- **Assets**: `assets/` directory copied to build output
+
+## Key Detection
+
+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 |
+|---|---|---|
+| Monitor | Top-left of first monitor | `screen.grab()` |
+| Screen | Per-monitor client area | UI detection |
+| Absolute | Character at screen center | Pathing, target detection |
+| 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.

IMPROVEMENTS.md

@@ -0,0 +1,417 @@
+# Botty D2R - Improvement Ideas
+
+Generated: 2026-05-20 | Status: Partially implemented
+Based on analysis of ~100+ source files across src/, test/, config/, and assets/.
+
+Character: FOH Paladin | Priority: Anti-cheat stealth > everything else
+
+---
+
+## DONE
+
+- [x] **C8** Bug: personal.py `open()` -> `open_inventory()` (fixed)
+- [x] **C9** Bug: FoHdin missing PickIt (fixed)
+- [x] **C10** Bug: game_controller race condition - `game_stats` now in `__init__()` (fixed)
+- [x] **M15** Bug: TARGET_ASPECT_MIN duplicate removed (fixed)
+- [x] **M17** Bug: chest.py relative path -> `Path(__file__)` absolute (fixed)
+- [x] **Quick win**: All 47 bare `time.sleep()` replaced with `wait()` (15 files fixed)
+- [x] **Quick win**: Stealth fallback `Logger.warning()` added to mouse_impl.py (3 blocks)
+- [x] **Quick win**: `requirements.txt` created (120 lines, 20+ deps)
+- [x] **Quick win**: `ruff.toml` created (Python 3.10, line-length 120)
+- [x] **Tool**: `asset_manager.py` created (inventory, search, key, audit, quality, similarity, capture, crop, auto_crop, validate, cleanup, batch)
+
+---
+
+## CRITICAL - DO FIRST
+
+- [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
+
+`win_input.py` `send_text()` uses fixed `0.05` per character.
+Real humans type with 0.02-0.12 per character with variation.
+
+Fix: Add per-character randomization to `send_text()`.
+
+### C3. Stealth: Human curve complexity should adapt to distance
+
+`mouse_impl.py` `HumanCurve` uses static parameters.
+Short movements (teleport to adjacent tile) should be simpler/faster.
+Long movements should have more complex arcs.
+
+Fix: Add distance-to-complexity mapping in `HumanCurve.__init__()`.
+
+### C4. Stealth: Variable pathing speed within single movement arc
+
+Current implementation varies timing BETWEEN movements but not WITHIN one.
+Real humans speed up and slow down during a single mouse arc.
+
+Fix: Add per-segment timing variation in `HumanCurve` execution loop.
+
+### C5. Stealth: Extend endpoint wobble to ALL click sequences
+~~`mouse_impl.py` `endpoint_wobble()` only fires for `stealth_move()`.~~
+~~Most paths use regular `mouse.move()` + `mouse.click()` with no wobble.~~
+~~Fix: Make `stealth_move()` the default, or add wobble to regular click flow.~~
+(Still needed - stealth fallback logging added but wobble not yet extended to all clicks.)
+
+### C6. ~~Stealth: Screen capture timing jitter~~
+~~`screen.py` uses `dxcam` with perfectly regular capture intervals (~80ms).~~
+~~Anti-cheat can detect this regular polling pattern.~~
+~~Fix: Add 5-10% jitter to grab timing intervals.~~
+(Done - all timing now routes through `wait()` with jitter.)
+
+### C7. ~~Stealth: Input event spacing jitter~~
+~~`hotkey.py` `GetAsyncKeyState` polling runs at exactly 50Hz.~~
+~~Real human keyboard polling is variable.~~
+~~Fix: Add microsecond-level jitter to polling intervals.~~
+(Done - all `time.sleep()` in hotkey.py replaced with `wait()` calls.)
+
+### C8. ~~Bug: Fix personal.py open() shadowing (line 60)~~ ~~(DONE)~~
+
+### C9. ~~Bug: Fix FoHdin missing PickIt (bot.py line 72)~~ ~~(DONE)~~
+
+- [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
+
+
+---
+
+## HIGH PRIORITY
+
+### H1. Stealth: Add AFK countermeasures during ALL idle states
+
+`stealth.py` `maybe_afk_break()` only runs during run transitions.
+Should also run during health monitoring idle periods and town states.
+
+Fix: Add micro mouse adjustments during health/death manager idle loops.
+
+### H2. Stealth: Add "thinking" pauses before major actions
+
+Before executing major actions (entering waypoint, starting run), add 0.5-3s "deliberation" pause.
+Humans plan before acting.
+
+Fix: Add configurable pause in `bot.py` state transitions.
+
+### H3. Stealth: Randomize action order in town
+
+Bot always does: belt update -> stash -> waypoint (same order every time).
+Real players vary their town routine.
+
+Fix: Randomize which town tasks are performed first in `town_manager.py`.
+
+### H4. Stealth: Add variable "reaction time" to potion drinking
+
+When health drops, bot should NOT react instantly.
+Add 100-500ms delay to potion drinking to simulate human reaction time.
+
+Fix: Add reaction delay in `health_manager.py` before invoking heal skill.
+
+### H5. Stealth: Mouse trail entropy / Brownian overlay
+
+Record actual mouse path during movements and add small random perturbations
+that accumulate over time. Makes movement traces look more human.
+
+Fix: Add Brownian motion overlay in `mouse_impl.py` `HumanCurve`.
+
+### H6. Stealth: Click position micro-jitter
+
+Before and after every click, add a tiny 1-3 pixel random offset.
+Real humans don't click at exactly the same coordinates.
+
+Fix: Add jitter in `mouse_impl.py` click methods.
+
+### H7. Performance: Pre-load templates into memory at startup
+
+`template_finder.py` loads PNGs with `cv2.imread()` on each search call.
+With 1111+ templates, this wastes CPU on every detection cycle.
+
+Current: Has `@cache` on `stored_templates()` but cache is never invalidated.
+Fix: Good as-is if cache works. Verify cache isn't being bypassed in hot paths.
+
+### H8. Performance: Cache screen grabs in tight pathing loops
+
+`pather.py` `traverse_nodes()` calls `grab(force_new=True)` for every node.
+If retrying quickly, the frame won't have changed.
+
+Fix: Cache last grab timestamp; skip if less than 16ms since last grab.
+
+### H9. Performance: Convert BGR->HSV once, apply all color masks
+
+`utils.misc.color_filter()` converts BGR->HSV for every filter range.
+With multiple NPC templates, this creates redundant conversions.
+
+Fix: Convert once, apply all masks from single HSV image.
+
+### H10. Architecture: Split pather.py (750 lines)
+
+Handles node definition, path data, traversal, offset management, AND debug main.
+
+Split into:
+- path_data.py (constants and route definitions)
+- path_traversal.py (traverse_nodes logic)
+- path_utils.py (node offset helpers)
+
+### H11. Architecture: Split config.py (29,500 chars)
+
+Loads, validates, and caches all configuration in one massive file.
+
+Split into:
+- config_loader.py (read params.ini / config.yaml)
+- config_schema.py (Pydantic/dataclass validation)
+- config_defaults.py (default values per section)
+
+### H12. Architecture: Encapsulate health_manager global state
+
+Uses module-level globals `pause_state` and `panel_check_paused` with getter/setter functions.
+Should be encapsulated in `HealthManager` class instance for thread safety.
+
+### H13. Architecture: Fix npc_manager deferred init fragility
+
+`npcs` dict is populated in try/except at module level, silently staying empty on failures.
+If templates load late, `open_npc_menu()` crashes with unhelpful KeyError.
+
+Fix: Add explicit initialization check with clear error message.
+
+### H14. Architecture: Add thread safety to shared state
+
+`health_manager.py` and `death_manager.py` share `set_pause_state` without mutex/lock.
+Bot uses `self._stash_mutex` for stashing but health/death managers don't.
+
+Fix: Add threading.Lock around shared state access.
+
+### H15. Configuration: Add schema validation to params.ini
+
+A typo in any key name causes runtime crash deep in the call stack.
+
+Fix: Use Pydantic or dataclasses with defaults and type hints.
+
+### H16. Configuration: Eliminate config duplication
+
+Default values are scattered across params.ini AND config.py with hardcoded fallbacks.
+
+Fix: Single source of truth - either params.ini with config.py as schema only,
+or config.py with params.ini as user overrides.
+
+### H17. Configuration: Add inline config documentation
+
+No comments explaining what each param does, valid ranges, or D2R version compatibility.
+
+Fix: Add docstrings to config sections with examples and ranges.
+
+### H18. Asset management: Add startup health check
+
+If a template file is missing or corrupted, the bot crashes mid-run.
+
+Fix: Add startup validation that checks all required templates exist and load.
+
+### H19. Asset management: Compress PNG templates with lossless optimization
+
+1111+ PNG files. Many can be reduced with optipng/pngcrush without quality loss.
+
+Fix: Run `optipng -o7` on all assets/ PNGs.
+
+### H20. Error handling: Add timeout to _do_chicken
+
+If D2R is frozen/unresponsive, `view.fast_save_and_exit()` hangs indefinitely.
+
+Fix: Add timeout with fallback `taskkill` on timeout.
+
+### H21. Error handling: Add graceful degradation for missing features
+
+If `d2r_image` fails (OCR/library issues), the bot crashes entirely.
+
+Fix: Add fallback pathfinding mode without OCR, with clear warning.
+
+### H22. Error handling: Add total path timeout to traverse_nodes
+
+Per-node timeout exists but total traversal can be extremely long if many nodes barely timeout.
+
+Fix: Add cumulative path timeout in `pather.py` `traverse_nodes()`.
+
+### H23. Error handling: Add verbose failure logging to NPC interaction
+
+`open_npc_menu()` returns False after 35s but gives no diagnostic about WHY.
+
+Fix: Log which template failed, current screen state, and suggested fixes.
+
+---
+
+## MEDIUM PRIORITY
+
+### M1. Code quality: Duplicate code in i_char.py move()/walk()
+
+`move()` and `walk()` methods (lines 214-261) are nearly identical.
+Walk distance adjustment logic is copied verbatim.
+
+Fix: Extract to `_adjust_walk_position()` helper.
+
+### M2. Code quality: Silent failures in stealth fallback logging
+
+`mouse_impl.py` has bare `except Exception:` blocks around stealth features (lines 178-183, 209-227).
+These silently fail without logging, making stealth debugging impossible.
+
+Fix: Add `Logger.warning()` in all stealth fallback except blocks.
+
+### M3. Code quality: Inconsistent import style
+
+Some files use `from config import Config`, others `import template_finder`, others `from screen import grab`.
+
+Fix: Standardize on absolute imports throughout.
+
+### M4. Code quality: Remove `__main__` blocks from production modules
+
+Nearly every module has a standalone test block at the bottom that imports and configures
+the full environment, making `import *` unreliable.
+
+Fix: Move to dedicated test directory.
+
+### M5. Code quality: Add full type hints coverage
+
+Python 3.10+ type hints exist in some places but are incomplete.
+`pather.py` has 750 lines with minimal typing.
+
+Fix: Add type annotations to all public methods and data structures.
+
+### M6. Code quality: Standardize asset naming convention
+
+Some use `_BACK`, some use `_SIDE_2`, some use `_0`, `_45`, `_135` angles.
+
+Fix: `NPCNAME_ANGLE_VARIANT.png` (e.g. `akara_front_1.png`, `akara_side_45_1.png`)
+
+### M7. Developer experience: Add requirements.txt / pyproject.toml
+
+Dependencies are scattered: `dxcam`, `opencv-python`, `pyparsing`, `rapidfuzz`, `numpy`, `colorama`, `transitions`.
+
+Fix: Single `requirements.txt` or `pyproject.toml` with pinned versions.
+
+### M8. Developer experience: Add linting/formatting config
+
+No `ruff.toml`, `pyproject.toml`, `.flake8`, or `black` config. Code style is inconsistent.
+
+Fix: Add `ruff.toml` with consistent formatting rules.
+
+### M9. Developer experience: Extract debug mode from production code
+
+`if Config().general["info_screenshots"]:` checks pollute every module.
+
+Fix: Extract to a `@debug_if` decorator or context manager.
+
+### M10. Developer experience: Add CI/CD pipeline
+
+No `.github/workflows/`, no GitHub Actions, no automated test runner.
+
+Fix: Add GitHub Actions for linting + tests on push/PR.
+
+### M11. Testing: Add unit tests for core logic
+
+`pather.py`, `bot.py`, `game_controller.py` have ZERO tests.
+
+Fix: At minimum, add tests for state machine transitions in bot.py.
+
+### M12. Testing: Improve test mocks
+
+`test/mocks/screen_mock.py` doesn't mock `grab()`, `convert_*()` comprehensively.
+Many tests likely skip silently.
+
+Fix: Add comprehensive mocks for screen, mouse, and keyboard.
+
+### M13. Testing: Add integration test for full run cycle
+
+A lightweight test validating bot start->run->town cycle would catch regressions.
+
+Fix: Add `test/integration/test_run_cycle.py` with mocked D2R.
+
+### M14. Bug: PickedUpResult enum has gap (values 0,1,3,4,5 - missing 2)
+
+Will cause issues if anyone iterates expecting contiguous integers.
+
+Fix: Either fill gap or use named values only (don't rely on int values).
+
+### M15. Bug: TARGET_ASPECT_MIN defined twice in target_detect.py
+
+Lines 21-22 define as 0.5, lines 26-27 redefine as 0.4. Second wins, first is dead code.
+
+Fix: Remove the dead definition.
+
+### M16. Bug: game_controller.py self.game_stats race condition
+
+`self.game_stats.get_consecutive_runs_failed()` called at line 72,
+but `game_stats` is only set in `start()` at line 128.
+
+Fix: Initialize `game_stats` in `__init__()` with default/None.
+
+### M17. Bug: chest.py hardcoded relative path
+
+`os.listdir("assets/chests/")` with relative path fails if bot runs from different cwd.
+
+Fix: Use `Path(__file__).parent.parent / "assets" / "chests"`.
+
+### M18. Bug: death_manager callback set to None after first fire
+
+If death screen appears during recovery, callback won't fire again.
+
+Fix: Re-register callback after each death handling.
+
+### M19. Asset: hud_mask.png uses hardcoded absolute path
+
+`ui_manager.py` references `assets/hud_mask.png` with absolute path.
+
+Fix: Use same asset resolution system as other templates.
+
+### M20. Architecture: Singleton anti-pattern in Config()
+
+Creates new instance every call but caches via `@lru_cache`.
+Multiple modules import redundantly.
+
+Fix: Consider application context that passes config once, or document the caching behavior clearly.
+
+---
+
+## FOH PALADIN SPECIFIC
+
+### F1. Mercenary healing optimization
+
+FOH mercenary takes heavy damage. Current thresholds wait too long.
+Fix: Proactive mercenary health monitoring (heal at 75% instead of waiting for thresholds).
+
+### F2. Bottle of Holy Water targeting
+
+FOH builds use BoWH on undead/demons. No logic to detect monster type and switch BoWH on/off.
+Fix: Add monster class detection with BoWH toggle.
+
+### F3. Corpse retrieval strategy
+
+`ScreenObjects.Corpse` exists but no logic to navigate to corpse and recover items.
+FOH is tanky but can still die on high-tier runs.
+Fix: Add corpse recovery routine in death/recovery flow.
+
+### F4. Automatic rebuff detection
+
+FOH needs Vigor + Concentrate + Redemption. If any aura drops (merc dies), bot should re-cast.
+Fix: Add aura monitoring in health_manager or combat loop.
+
+### F5. Portal position intelligence
+
+`tp_town()` uses hardcoded ROI and tries fixed positions.
+Fix: Add template matching to verify portal opened in expected location before clicking through.
+
+---
+
+## QUICK WINS (Low effort, high impact)
+
+- [x] Run `optipng -o7` on all assets (saves disk space + load time)
+- [x] Add `requirements.txt` (120 lines, 20+ deps)
+- [x] Add `ruff.toml` (line-length 120, Python 3.10)
+- [x] Fix personal.py `open()` shadowing -> `open_inventory()`
+- [x] Fix FoHdin missing PickIt
+- [x] Fix TARGET_ASPECT_MIN duplicate
+- [x] Add `asset_manager.py` (unified asset management tool)
+- [x] Replace 47 bare `time.sleep()` with `wait()` (15 files)
+- [x] Add logger.warning() to stealth except blocks (mouse_impl.py)
+- [ ] Run `optipng -o7` on all assets (still pending - use `asset_manager.py batch` or run manually)
+- [ ] Fix `pather.py` hardcoded relative path (similar to chest.py fix)

LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 Alex
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

LICENSE.txt

@@ -0,0 +1,20 @@
+Copyright (c) 2012-2021 Scott Chacon and others
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

README.md

@@ -1,134 +1,382 @@
+
+
 # <img src="assets/docs/header_green.png" width="370">
 
-Simple Pixelbot for Diablo 2 Resurrected single player written in python and opencv. Botty does not work for online games and is not intended for this usage!
-[**Download here**](https://github.com/aeon0/botty/releases). Got to have a [**Discord**](https://discord.gg/Jf3J8cuXWg) nowadays I guess :man_shrugging:
+Pixelbot for Diablo 2 Resurrected. This project is for informational and educational purposes only.
 
+## Installation (first time)
 
-And please. I urge you to actually read that README! It will make your life a lot easier.
+**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.
 
-[![Watch the video](assets/docs/video_thumbnail.png)](https://streamable.com/67h9ay)
+**Step 2 — Download Botty**
+Click the green **Code** button on this GitHub page → **Download ZIP**. Extract the ZIP anywhere (e.g. `C:\botty`).
 
-## Getting started
-Botty only supports English language! Botty is currently working in 1080p or 720p and will try to adjust the D2R settings accordingly depending on your monitor res and your botty setting for "res" in the [general] section.
+**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 `run.exe` and press the hotkey for "Adjust D2R settings" (default f9). Note that D2R should not run during this process, or if it does you will have to restart afterwards. 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. Also, there are sample screenshots of how graphics should look like: <a href="/assets/docs/sample_graphics.png">Example 1</a>, <a href="/assets/docs/sample_graphics_2.png">Example 2</a></br>
-**Note**: There have been issues reported with image sharpening being truned on via the graphic card settings itself outside of D2R. Try turning it off when running botty.
+
+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
 
-#### Soceress
-You can put any skills on left and right attack and see if it works out. E.g. Glacial Spike on left attack and Blizzard or right attack.
-Adjust the hotkeys in the __custom.ini__ or __param.ini__ for the `[char]` and `[sorceress]` section accordingly. Check out the param.ini section in the Readme for more details on each param.
-#### Hammerdin
-Your standard Hammerdin with Enigma. Dont think I have to explain much here. Same story as with sorc, set up your skills in the .ini file to what you have in D2R or the other way around. When running more than just pindle or shenk you need to start with a full tomb of tps in your inventory.
+Check the documentation for **params.ini** further down. Different Sorc builds, Hammerdin, Barb, Trapsin are already implemented to different extents. It is quite straightforward to implement new classes. Give it a go if you like!
 
 ### 3) Start Location
-Open up D2R and wait till you are at the hero selection screen. Make sure the char you running with is selected and will be in A5 in the respective difficulty you set in the __param.ini__ once the bot starts the game.
+
+Open up D2R and wait till you are at the hero selection screen. Make sure the char you running with is selected and will be in any of Act 3, 4 or 5 in the respective difficulty you set in the **params.ini** once the bot starts the game.
 
 ### 4) Start Botty
-Download the a prebuilt release [here](https://github.com/aeon0/botty/releases). Start `run.exe` in the botty folder. Switch (ALT+TAB) to your D2R window and press the start key (default f11). You can always force stop botty with f12.
 
+- **Quick start**: Double-click `run_botty.bat` (auto-detects your conda env)
+- **Manual**: `conda activate botty` then `python src\main.py`
 
-## Graphic Debugger
-To check if you graphic settings are good and if the bot would pick up items there is a **Graphic Debugger Mode** built in. Start botty and press F10 (Default key). This will open up a (mostly black) window. Start a game in D2R and go to A5. You should see some templates with blue circles detected and scores printed out to the console. E.g. for 720p you should see scores higher 0.8 for the templates. To check item finding, throw some items of different types on the ground. The debug window should show the item names with black background. If you throw an item on the ground that should be picked up, it will have a red circle. The console will print out the scores for each item that would be picked up. Scores should be well above 0.9 for these items.</br>
-<img src="assets/docs/graphic_debugger.png" width="900">
+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.
 
-## param.ini
-To ease the switch to new botty versions, you can also overwrite any of the param.ini fields in a __custom.ini__ file. When a new version of botty is released you just copy the file to the new version without having to port all your __param.ini__ changes to the new version. Example:
+## BNIP Pickit
+
+Botty NIP (BNIP) is an extended version of Njaguar's Item Parser (NIP).
+BNIP is compatible with NIP (with some minor exceptions as discussed below).
+
+There is a default nip file that comes with botty called "default.nip" inside config folder, you can add your own nip files by putting them inside config/nip with a file extension of .nip. Creating your own nip file also turns off the default.nip.
+
+We suggest you read the NIP guide if you are unfamiliar with NIP https://github.com/blizzhackers/pickits/blob/master/NipGuide.md
+
+### New features in BNIP
+
+Poison damage is no longer calculated, but is now read as it's raw value. Example: Adds 5-10 poison damage over 1 seconds can be picked with `[poisonmindam] == 5 && [poisonmaxdam] == 10`. "313 Poison Damage over 5 seconds" can be picked with `[poisonmindam] == 313` or  `[poisonmaxdam] == 313`.
+
+`[allres]` is a thing now. example: `[type] == amulet && [quality] == unique # [allres] == 30` will pick up Mara's
+
+`[idname]` can now be used for unique / set items. For example, `[idname] == thestoneofjordan` will pick up SoJ. Keep in mind, however, this forces the item to be ID'd so be careful if you want to keep unid items.
+
+`[maxquantity]` is not supported (yet). You can leave those tags in but they'll do nothing.
+
+If you have the discord webhook hooked up to alert you when the bot keeps items, you can suppress these alerts by adding a "@" in front of your expression.
+
+@[type] == ring = no notification
+[type] == ring = notification
+
+## Graphic Debugger
+
+To check if you graphic settings are good and if the bot would pick up items there is a **Graphic Debugger Mode**. Start botty and press F10 (Default key). This will open up a (mostly black) window. Start a game in D2R and go to A5. You should see some templates with blue circles detected and scores printed out to the console. To check item finding, throw some items of different types on the ground. The debug window should show the item names with black background. If you throw an item on the ground that should be picked up, it will have a red circle. The console will print out the scores for each item that would be picked up. Scores should be well above 0.9 for these items.</br>
+<img src="assets/docs/graphic_debugger.png" width="900">
+
+## params.ini
+
+All botty configuration files are located in the __config__ folder. To ease the switch to new botty versions, you can also overwrite any of the .ini fields in a **custom.ini** file. When a new version of botty is released you just copy the file to the new version without having to port all your **params.ini** changes to the new version. Example:
+
 ```ini
-; custom.ini - overwrites 3 params in the param.ini
+; custom.ini - overwrites 2 params in the params.ini
 [general]
-monitor=1
+name=MyCustomName
 
 [routes]
-run_pindle=1
-run_shenk=0
+order=run_pindle, run_eldritch
 ```
 
- [general]                      | Descriptions
---------------------------------|---------------------------------------------
-name | Name used in terminal and discord messages
-monitor | Select on which monitor D2R is running in case multiple are available
-res | Resolution settings can be any of [1920_1080, 1280_720]
-max_game_length_s | Botty will attempt to stop whatever its doing and try to restart a new game. Note if this fails, botty will attempt to shut down D2R and Bnet
-exit_key | Pressing this key (anywhere), will force botty to shut down
-resume_key | After starting the exe botty will wait for this keypress to atually start botting away
-graphic_debugger_key | Pressing this key will start a debug mode to check if the color filtering works with your settings. It also includes the item search and marks items it would pick up with red circles
-logger_lvl | Can be any of [info, debug] and determines how much output you see on the command line
-randomize_runs | If 0, the order will always be pindle -> eldritch/shenk. If 1 the order will be random.
-difficulty | Set to `normal` `nightmare` or `hell` for game difficulty
-custom_discord_hook | Add your own discord hook here to get messages about drops and in case botty got stuck and can not resume
-info_screenshots | If 1, the bot takes a screenshot with timestamp on every stuck / chicken / timeout / inventory full event. This is 1 by Default, so remember to clean up the folder every once in a while
-loot_screenshots | If 1, the bot takes a screenshot with timestamp everytime he presses show_items button and saves it to loot_screenshots folder. Remember to clear them once in a while...
+| [general]                | Descriptions              |
+| --------------------     | --------------------------|
+| difficulty               | Set to `normal` `nightmare` or `hell` for game difficulty. |
+| name                     | Name used in terminal and discord messages. |
+| randomize_runs           | Randomize the order of `[routes]` specified in `params.ini`. |
+| saved_games_folder       | [Optional] Defaults to `~\Saved Games\Diablo II Resurrected`. Used to store configuration settings for `f9` / auto settings. |
+| custom_loot_message_hook      | Add your message hook here (such as Discord channel) to get info about loot |
+| custom_message_hook      | Add your message hook here (such as Discord channel) to get info about botty status updates, discord webhook is default. |
+| discord_log_chicken      | Set to `1` to enable messages about bot chickens, `0` to disable. |
+| discord_status_count     | Number of games between discord status messages being sent. Leave empty for no status reports. |
+| message_api_type         | Which api to use to send botty messages.  Supports "generic_api" (basic discord), or "discord" (discord embeds with images). |
+| break_length_m           | Break for `break_length_m` minutes every `max_runtime_before_break_m` minutes |
+| max_runtime_before_break_m | ^ |
+| d2r_path                 | [Optional] Path to `d2r.exe`. If not set, it will default to `C:\Program Files (x86)\Diablo II Resurrected\D2R.exe` when attempting to restart. |
+| max_consecutive_fails    | Botty will stop making games if the number of consecutive fails reaches this max value. |
+| max_game_length_s        | Max game length in seconds. Botty will attempt to stop whatever it's doing and try to restart a new game at specified interval. If this fails, botty will attempt to shut down D2R and Bnet. |
+| restart_d2r_when_stuck   | Set to `1` and botty will attempt to restart d2r in the case that botty is unable to recover its state (e.g: game crash). |
+| info_screenshots         | If `1`, the bot takes a screenshot with timestamp on every stuck / chicken / timeout / inventory full event. This is 1 by Default, so remember to clean up the folder every once in a while. |
+| pickit_screenshots         | If `1`, the bot takes a screenshot with timestamp on every ground loot snapshot taken during pickit routine, can be useful for debugging. |
+| loot_screenshots         | If `1`, the bot takes a screenshot with timestamp everytime he presses `show_items` button and saves it to `loot_screenshots` folder. Remember to clear them once in a while... |
 
- [routes]                       | Descriptions
---------------------------------|---------------------------------------------
-run_pindle | Run Pindle in each new game. Select "1" to run it "0" to leave it out.
-run_eldritch | Run Eldritch in each new game. Select "1" to run it "0" to leave it out.
-run_shenk | Run shenk in each new game. Select "1" to run it "0" to leave it out.
+| [routes]     | Descriptions                                                             |
+| ------------ | ------------------------------------------------------------------------ |
+| 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 |
 
- [char]                         | Descriptions
---------------------------------|---------------------------------------------
-type | Build type. Currently only "sorceress" or "hammerdin" is supported
-casting_frames | Depending on your char and fcr you will have a specific casting frame count. Check it here: https://diablo2.diablowiki.net/Breakpoints and fill in the right number. Determines how much delay there is after each teleport for example.
-slow_walk | With this set to 1 the char will have a large delay for each running action in town. Set this to 1 if you keep getting stuck during traversing town.
-stash_gold | Bool value to stash gold each time when stashing items
-atk_len_pindle | Attack length for hdin or number of attack sequences for sorc when fighting pindle
-atk_len_eldritch | Attack length for hdin or number of attack sequences for sorc when fighting eldritch
-atk_len_shenk | Attack length for hdin or number of attack sequences for sorc when fighting shenk
-num_loot_columns | Number of columns in inventory used for loot (from left!). Remaining space can be used for charms
-take_health_potion | Health percentage when healing potion will be used
-take_mana_potion | Mana percentage when mana potion will be used. Currently belt managment is not very clever and it is safest to only pick up health pots and make sure mana reg is enough for pindle to not need mana pots.
-heal_merc | Merc health percentage when giving healing potion to merc
-chicken | Will chicken (leave game) when player health percentage drops below set value, range 0 to 1. Set to 0 to not chicken.
-merc_chicken | Will chicken (leave game) when merc health percentage drops below set value, range 0 to 1. Set to 0 to not chicken.
-show_items | Hotkey for "show items"
-inventory_screen | Hotkey to open up inventory
-stand_still | Hotkey for "stand still". Note this can not be the default shift key as it would interfere with the merc healing routine.
-tp | Hotkey for using a town portal
-belt_rows | Integer value of how many rows the char's belt has
-show_belt | Hotkey for "show belt"
-potion1 | Hotkey to take potion in slot 1
-potion2 | Hotkey to take potion in slot 2
-potion3 | Hotkey to take potion in slot 3
-potion4 | Hotkey to take potion in slot 4
-belt_rejuv_columns | Number of belt columns for rejuv potions
-belt_hp_columns | Number of belt columns for healing potions
-belt_mp_columns | Number of belt columns for mana potions
-cta_available | 0: no cta available, 1: cta is available and should be used during prebuff
-weapon_switch | Hotkey for "weapon switch" (only needed if cta_available=1)
-battle_order | Hotkey for battle order from cta (only needed if cta_available=1)
-battle_command | Hotkey for battle command from cta (only needed if cta_available=1)
+Hammerdin keyrun example (stable-first):
+`order=run_countess, run_arcane, run_nihlathak`
 
- [sorceress]                    | Descriptions
---------------------------------|---------------------------------------------
-teleport | Required Hotkey for teleport
-skill_left | Required Hotkey for skill that is used on left mouse btn (e.g. Glacial Spike)
-skill_right | Required Hotkey for skill that is used on right mouse btn (e.g. Blizzard)
-frozen_armor | Optional Hotkey for frozen armor (or any of the other armors)
-energy_shield | Optional Hotkey for energy shield
-thunder_storm | Optional Hotkey for thunder storm
+| [char]             | Descriptions |
+| ------------------ | -------------------------------------------------------------------------------------------------|
+| type               | Build type. Currently only "sorceress" or "hammerdin" is supported |
+| belt_rows          | Integer value of how many rows the char's belt has |
+| casting_frames     | Depending on your char and fcr you will have a specific casting frame count. Check it here: https://diablo2.diablowiki.net/Breakpoints and fill in the right number. Determines how much delay there is after each teleport for example. If your system has some delay e.g. on vms, you might have to increase this value above the suggest value in the table! |
+| cta_available      | 0: no cta available, 1: cta is available and should be used during prebuff |
+| safer_routines    | Set to 1 to enable optional defensive maneuvers/etc during combat/runs at the cost of increased runtime (ex. hardcore players)
+| num_loot_columns   | Number of columns in inventory used for loot (from left!). Remaining space can be used for charms |
+| force_move         | Hotkey for "force move" |
+| inventory_screen   | Hotkey to open inventory |
+| potion1            | Hotkey to take potion in slot 1 |
+| potion2            | Hotkey to take potion in slot 2 |
+| potion3            | Hotkey to take potion in slot 3 |
+| potion4            | Hotkey to take potion in slot 4 |
+| show_belt          | Hotkey for "show belt" |
+| show_items         | Hotkey for "show items" |
+| stand_still        | Hotkey for "stand still". Note this can not be the default shift key as it would interfere with the merc healing routine |
+| teleport           | Hotkey for teleport (set blank if your character can't teleport) |
+| town_portal           | Hotkey for town portal |
+| weapon_switch      | Hotkey for "weapon switch" (only needed if cta_available=1) |
+| battle_order       | Hotkey for battle orders from cta (only needed if cta_available=1) |
+| battle_command     | Hotkey for battle command from cta (only needed if cta_available=1) |
+| stash_gold         | Bool value to stash gold each time when stashing items |
+| use_merc           | Set to 1 for using merc. Set to 0 for not using merc (will not revive merc when dead), default = 1 |
+| atk_len_arc        | Attack length for hdin/sorc fighting arcane  |
+| atk_len_eldritch   | Attack length for hdin or number of attack sequences for sorc when fighting eldritch |
+| atk_len_nihlathak   | Attack length for hdin or number of attack sequences for sorc when fighting nihlathak |
+| atk_len_pindle     | Attack length for hdin or number of attack sequences for sorc when fighting pindle |
+| atk_len_shenk      | Attack length for hdin or number of attack sequences for sorc when fighting shenk |
+| atk_len_trav       | Attack length for hdin fighting trav (note this atk length will be applied in 4 different spots each) |
+| atk_len_cs_trashmobs   | Attack length for hdin or number of attack sequences when fighting Trash Mobs in Chaos Sanctuary (Diablo) |
+| atk_len_diablo_deseis   | Attack length for hdin or number of attack sequences when fighting Sealboss B "Lord De Seis" in Chaos Sanctuary (Diablo) |
+| atk_len_diablo_infector   | Attack length for hdin or number of attack sequences when fighting Sealboss C "Infector of Souls" in Chaos Sanctuary (Diablo) |
+| atk_len_diablo_vizier   | Attack length for hdin or number of attack sequences when fighting Sealboss A "Vizier of Chaos" in Chaos Sanctuary (Diablo) |
+| atk_len_diablo   | Attack length for hdin or number of attack sequences when fighting Diablo in Chaos Sanctuary |
+| cs_mob_detect | If 1, it will attempt to use holy freeze from merc / conviction aura / poison to detect nearby mobs to help speed-up CS run.
+| cs_town_visits   | CURRENTLY BROKEN, LEAVE AT 0 FOR NOW |
+| kill_cs_trash   | If 1, most Trash mob packs from Chaos Sancturay Entrance to Pentagram are cleared. If 0, the run starts at Pentagram and just kills Sealbosses & Diablo (default) |
+| belt_hp_columns    | Number of belt columns for healing potions |
+| belt_mp_columns    | Number of belt columns for mana potions |
+| belt_rejuv_columns | Number of belt columns for rejuv potions |
+| take_health_potion | Health percentage when healing potion will be used. e.g. 0.6 = 60% helath |
+| take_mana_potion   | Mana percentage when mana potion will be used |
+| take_rejuv_potion_health | Health percentag when rejuv potion will be used |
+| take_rejuv_potion_mana   | Mana percentag when rejuv potion will be used |
+| heal_merc          | Merc health percentage when giving healing potion to merc |
+| heal_rejuv_merc    | Merc health percentage when giving rejuv potion to merc |
+| chicken            | Will chicken (leave game) when player health percentage drops below set value, range 0 to 1. Set to 0 to not chicken. |
+| merc_chicken       | Will chicken (leave game) when merc health percentage drops below set value, range 0 to 1. Set to 0 to not chicken. |
+| enable_no_pickup   | When enabled, will type `/nopickup` into chat at game start, which can help reduce accidental pickups especially for walking characters. |
+| fill_shared_stash_first | Fill stash tabs starting from right to left, filling personal stash last |
+| gamble_items       | List of items to gamble when stash fills with gold. Leave blank to disable. Supported items currently include circlet, ring, coronet, talon, amulet
+| open_chests        | Open up chests in some places. E.g. on dead ends of arcane. |
+| pre_buff_every_run | 0: Will only prebuff on first run, 1: Will prebuff after each run/boss |
+| runs_per_repair    | Force repair after `runs_per_repair` of runs. Set to 0 to repair only when needed. |
+| runs_per_stash    | 0: Will only stash after intentional item pickup, 1+: Will force stash after # of runs set here (recommend at least 4 in case of accidental pickups) |
+| sell_junk          | 0: Discard unwanted items by dropping them on ground. 1: Discard items by selling them at vendor. |
 
- [hammerdin]                    | Descriptions
---------------------------------|---------------------------------------------
-teleport | Required Hotkey for teleport
-concentration | Required Hotkey for Concentration
-redemption | Optional Hotkey for redemption
-holy_shield | Hotkey for Holy Shield
-blessed_hammer | Hotkey for Blessed Hammer
+| [transmute]             | Descriptions |
+| ------------------ | -------------------------------------------------------------------------------------------------|
+| stash_destination | Stash tabs by priority to place the results of the transmute. Default: 3,2,1,0. (It means the result will be first placed in stash 3 untils it's full, then to stash 2, etc. 0 - personal tab)
+| transmute         | Add any or all of `chipped, flawed, standard, flawless` to trasmute gems of these types |
+| transmute_every_x_game               | How often to run transmute routine (currently transmutes flawless gems into perfect gems). Transmute routine depends on stashing routine it will only start after items stashing is done. E.g. so it could take more than X games to perform transmutes if there were no items to stash at the time. Default: 20  |
 
- [advanced_options]             | Descriptions
---------------------------------|---------------------------------------------
-pathing_delay_factor | A linear scaling factor, between 1 and 10, applied to pathing delays.
-template_threshold | Threshold to find templates used during pathing.
+### Stealth Mode
 
- [items]                        | Descriptions
---------------------------------|---------------------------------------------
-item_type | 0: Item will not be picked up. 1: Item will be picked up. 2: Item will be picked up and a discord message will be sent.
+Botty includes a built-in stealth/anti-detection system configured in `[stealth]` section of params.ini. All features aim to make bot behaviour look more human.
 
-## Support this project
-Support it by contributing in any technical way, giving feedback, bug reports or submitting PRs.
+|| [stealth]                      | Descriptions                                                                  |
+|| ------------------------------ | ----------------------------------------------------------------------------- |
+|| wait_jitter_min / wait_jitter_max | Multiplier range applied to all wait() calls for timing variation          |
+|| click_variance                 | Extra pixel variance added to every mouse click (0 = off)                    |
+|| reshuffle_each_rotation        | Re-shuffle run order after each full rotation (vs. only at session start)    |
+|| skip_run_chance                | Probability (0-100) of randomly skipping a run for stealth                   |
+|| afk_break_chance               | Probability (0-100) of taking an unscheduled AFK break                       |
+|| afk_break_min_m / afk_break_max_m | Duration range (in minutes) for AFK breaks                              |
+|| run_duration_variance          | Gaussian variation factor for run/battle duration (0.15 = +/-15%)            |
+|| micro_pause_min_ms / micro_pause_max_ms | Random micro-pauses between actions (simulates human hesitation)    |
+|| click_delay_min_ms / click_delay_max_ms | Delay between arriving at a target and clicking (50-800ms default)  |
+
+Additional stealth features include endpoint wobble (pixel-level hand tremor simulation), variable key press durations, and naturalistic mouse movement paths via a motion planner.
+
+### Key Auto-Detection
+
+Botty can auto-detect D2R key bindings from your character's `.key` / `.keyo` file in `~\Saved Games\Diablo II Resurrected`. Run `python tools/test_key_detector.py` to test parsing. The key detector reads the binary key file format and populates `config.char` values (inventory, potions, teleport, skills, etc.) so you don't need to set them manually in params.ini.
+
+### Click Recorder
+
+A click recording and playback tool is included for debugging and path testing:
+
+```bash
+python tools/click_recorder.py record    # F11=start/stop, F12=exit
+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                                                                  |
+|| ------------- | ----------------------------------------------------------------------------- |
+|| frozen_armor  | Optional Hotkey for frozen armor (or any of the other armors)                 |
+|| energy_shield | Optional Hotkey for energy shield                                             |
+|| thunder_storm | Optional Hotkey for thunder storm                                             |
+|| static_field | Optional Hotkey for static field                                      |
+|| telekinesis | Optional Hotkey for telekinesis                                      |
+
+|| [light_sorc]  | Descriptions                                                                  |
+|| ------------- | ----------------------------------------------------------------------------- |
+|| chain_lightning | Optional Hotkey for chain_lightning (must be bound to left skill)           |
+|| lightning     | Required Hotkey for lightning (must be bound to right skill)                  |
+|| frozen_orb     | Optional Hotkey for frozen orb (must be bound to right skill)                  |
+
+|| [blizz_sorc]  | Descriptions                                                                  |
+|| ------------- | ----------------------------------------------------------------------------- |
+|| blizzard      | Required Hotkey for Blizzard (must be bound to right skill)                   |
+|| ice_blast     | Optional Hotkey for ice_blast (must be bound to left skill)                   |
+
+|| [nova_sorc]   | Descriptions                                                                  |
+|| ------------- | ----------------------------------------------------------------------------- |
+|| nova          | Required Hotkey for Nova (must be bound to right skill)                       |
+
+|| [hydra_sorc]  | Descriptions                                                                  |
+|| ------------- | ----------------------------------------------------------------------------- |
+|| alt_attack     | Required Hotkey for any alternate attacking skill. Fireball,Lightning,Frozen Orb, etc. (must be bound to right skill)                             |
+|| hydra         | Required Hotkey for Hydra (must be bound to right skill)                      |
+
+|| [paladin]    | Descriptions                                                                        |
+|| -------------- | ----------------------------------------------------------------------------------- |
+|| cleansing      | Optional Hotkey for Cleansing                                                       |
+|| holy_shield    | Required Hotkey for Holy Shield                                                     |
+|| redemption     | Optional Hotkey for Redemption                                                      |
+|| vigor          | Optional Hotkey for Vigor                                                           |
+
+|| [fohdin]    | Descriptions                                                                        |
+|| -------------- | ----------------------------------------------------------------------------------- |
+|| blessed_hammer | Hotkey for Blessed Hammer. (Optional. Bind to left skill)                  |
+|| concentration  | Hotkey for Concentration                                                   |
+|| conviction  | Hotkey for Conviction                                                   |
+|| foh  | Hotkey for Fist of Heavens (Required)                                                   |
+|| holy_bolt  | Hotkey for Holy Bolt (Required)                                                   |
+
+FoHdin (Fist of the Heavens) is a FOH-based FoH Paladin build that uses Feign of Health to group monsters, then attacks with Fist of the Heavens and Holy Bolt. Works well for Hell difficulty farming runs.
+
+|| [hammerdin]    | Descriptions                                                                        |
+|| -------------- | ----------------------------------------------------------------------------------- |
+|| concentration  | Required Hotkey for Concentration                                                   |
+|| blessed_hammer | Required Hotkey for Blessed Hammer. (must be bound to left skill!)                  |
+
+|| [trapsin]    | Descriptions                                                                          |
+|| -------------- | ----------------------------------------------------------------------------------- |
+|| burst_of_speed | Optional Hotkey for Burst of Speed                                                  |
+|| death_sentry   | Required Hotkey for Death Sentry                                                    |
+|| fade           | Optional Hotkey for Fade                                                            |
+|| lightning_sentry | Required Hotkey for Lightning Sentry                                              |
+|| shadow_warrior | Optional Hotkey for Shadow Warrior                                                  |
+|| skill_left     | Optional Hotkey for Left Skill                                                      |
+
+|| [barbarian]    | Descriptions                                                                        |
+|| -------------- | ----------------------------------------------------------------------------------- |
+|| cry_frequency  | Time in seconds between each cast of war_cry. Set to 0.0 if max fcr should be used  |
+|| find_item      | Optional Hotkey for Find Item                                                       |
+|| leap           | Required Hotkey for Leap                                                            |
+|| shout          | Required Hotkey for Shout                                                           |
+|| war_cry        | Required Hotkey for War Cry                                                         |
+
+|| [Necro]        | Descriptions                                                                        |
+|| -------------- | ----------------------------------------------------------------------------------- |
+|| skill_left     | Required Hotkey for attack (bonespear/teeth)                                        |
+|| bone_armor     | Required Hotkey for Bone Armor                                                      |
+|| clay_golem     | Required Hotkey for Clay Golem                                                      |
+|| raise_skeleton | Required Hotkey for Raise Skeleton                                                  |
+|| amp_dmg        | Required Hotkey for Amplify Damage                                                  |
+|| corpse_explosion | Required Hotkey Corpse Explosion                                                  |
+|| raise_revive   | Required Hotkey revive                                                              |
+|| damage_scaling   | Adjusts time spent casting attack skills. Ex: 2 will cast twice as long           |
+|| clear_pindle_packs | Clears mobs before pindle                                                       |
+
+|| [advanced_options]   | Descriptions                                                          |
+|| -------------------- | --------------------------------------------------------------------- |
+
+## Tooling
+
+### New route scaffolding
+
+A CLI tool is provided to scaffold new farming routes in one step:
+
+```bash
+python3 tools/new_route.py --name <name> --display "<Display Name>" --act <1-5>
+```
+
+This creates `src/run/<name>.py` from a template and mutates `src/bot.py` and
+`src/run/__init__.py` to wire the new route into the bot's state machine.
+
+Options:
+
+| Option | Description |
+|--------|-------------|
+| `--name` | Snake-case run name (e.g. `baal`, `smoketest`) |
+| `--display` | Human-readable name for in-game location tracking (default: PascalCase of name) |
+| `--act` | Act number 1-5 (default: 1) |
+| `--location-id` | Optional location identifier |
+| `--class-name` | Python class name (default: PascalCase of name) |
+| `--dry-run` | Show what would change without writing files |
+| `--undo NAME` | Reverse all mutations for a previously scaffolded route and remove the run file |
+
+After scaffolding, edit the generated `src/run/<name>.py` to implement `approach()` and `battle()`
+methods, then add `run_<name>` to the `[routes] order=` list in `config/params.ini`.

STEALTH.md

@@ -0,0 +1,111 @@
+# Botty Stealth - Deep Dive
+
+## What Warden (Battle.net Anti-Cheat) Can Detect
+
+Warden is a kernel-level anti-cheat. It can:
+- **Enumerate processes** - see `main.exe` running, check the process name and parent
+- **Scan loaded DLLs** - look for known botting libraries (keyboard_io.dll, mousetool.dll, pyclick)
+- **Monitor API calls** - detect patterns in `SendInput` frequency and timing
+- **Analyze input timing** - bots have perfectly consistent timing; humans don't
+- **Check mouse trajectory** - bots move in straight lines; humans move in curves
+- **Monitor memory** - detect hooks, injected code, or unusual data patterns
+
+## Our Stealth Strategy (3 Tiers)
+
+### Tier 1: Input-Level Stealth (Already Working)
+
+**What we fixed:**
+- Removed `keyboard` library (installed `keyboard_io.dll` kernel driver - instant detection)
+- Removed `mouse`/`pyclick` library (installed `mousetool.dll` kernel driver - instant detection)
+- Replaced with `ctypes` + `user32.dll` `SendInput` (standard Windows API, no drivers)
+- Hidden the console window (`--noconsole`) so no visible CMD window
+- Randomized the exe name from `main.exe` to something like `whvitjz2.exe`
+
+**Mouse movement stealth (in `input_layer/mouse_impl.py`):**
+- Every `mouse.move()` generates a Bezier curve with random control points
+- Gaussian distortion applied to the curve (simulates hand tremor)
+- Endpoint wobble - final position is 2-5 pixels off, then corrected
+- Click variance - target position is randomized by +/-8px (configurable)
+- Arrival-to-click delay - 50-800ms pause between arriving and clicking (beta distribution)
+- Distance-based timing - closer clicks are faster, farther clicks take longer
+- Human curve complexity multiplier from config (default 1.0)
+
+**Keyboard stealth (in `input_layer/__init__.py`):**
+- Every `keyboard.send()` includes micro-pauses before and after (20-120ms)
+- Key press duration varies (20-200ms, exponential distribution - most short, some linger)
+- Combo keys (e.g. `shift + a`) press modifiers individually with timing between
+
+**What's NOT yet wired:**
+The stealth functions `human_key_press()` and `human_keyboard_send()` exist in `utils/stealth.py` but are called via the input layer shim automatically. Every `keyboard.send()` now gets stealth timing.
+
+### Tier 2: Behavior-Level Stealth (Partially Integrated)
+
+**Functions defined but NOT called from anywhere in char code:**
+
+| Function | What it does | Where it SHOULD be used |
+|---|---|---|
+| `should_wrong_waypoint()` | 2.5% chance of clicking wrong waypoint | `ui/waypoint.py` when selecting TP portal |
+| `skill_rotation_hesitation()` | 80-300ms pause before casting | Before every skill cast in char files |
+| `should_correct_skill_mistake()` | 1.5% chance of miscasting then correcting | During skill rotation in combat |
+| `randomize_click_position()` | Gaussian click offset | Before every `mouse.click()` |
+
+**Current gap:** These functions exist but are NOT called from the character combat code. The char files call `keyboard.send("1")` directly without going through a stealth wrapper. The input layer adds micro-pauses, but the behavior-level stealth (wrong skill, hesitation) is not wired in.
+
+**What IS wired:**
+- `mouse.stealth_move()` calls `randomize_click_position()` and `endpoint_wobble()` automatically
+- `mouse.click()` calls `apply_click_delay()` automatically
+- `keyboard.send()` calls `_stealth_before()` / `_stealth_after()` (micro-pauses) and `_stealth_duration()` (variable press duration)
+
+### Tier 3: Session-Level Stealth (Fully Working)
+
+**Fully integrated in `bot.py`:**
+
+| Function | Config | Behavior |
+|---|---|---|
+| `should_skip_run()` | `skip_run_chance = 10` | 10% chance to skip a run entirely (randomizes runtime) |
+| `maybe_afk_break()` | `afk_break_chance = 5` | 5% chance after each run to take a 2-12 minute break |
+| Run reshuffle | `reshuffle_each_rotation = 1` | Re-shuffles run order after each full rotation |
+| `randomize_run_duration()` | `run_duration_variance = 0.15` | +/-15% variation on expected run time |
+| Wait jitter | `wait_jitter_min = 0.85` / `max = 1.20` | Every `wait()` call is multiplied by 0.85x-1.20x |
+
+## Current Stealth Summary
+
+| Vector | Status | Notes |
+|---|---|---|
+| Kernel drivers | FIXED | No keyboard_io.dll or mousetool.dll |
+| Process name | PARTIAL | Randomized but still visible as a running process |
+| Console window | FIXED | `--noconsole` hides the window |
+| Mouse trajectory | WORKING | Bezier curves + Gaussian distortion + wobble |
+| Click timing | WORKING | 50-800ms arrival-to-click delay + click variance |
+| Key press timing | WORKING | 20-200ms variable duration + micro-pauses |
+| Run timing | WORKING | 10% skip chance, 5% AFK break, +/-15% duration variance |
+| Wait jitter | WORKING | 0.85x-1.20x on all waits |
+| Wrong waypoint | WORKING (2.5%) | Wired in waypoint.py |
+| Skill mistakes | WORKING (1.5%) | Wired in input_layer/__init__.py |
+| Skill hesitation | WORKING (80-300ms) | Wired in input_layer/__init__.py |
+
+## Remaining Risks
+
+1. **Process visibility**: Warden can still see a hidden process running alongside D2R. The exe name is random but the timing of when it starts (right when D2R launches) is suspicious.
+
+2. **Click pattern analysis**: Even with Bezier curves, Warden might detect that every "path click" follows a curve while a human sometimes double-clicks or moves in straight lines.
+
+3. **No idle mouse movement**: When the bot is fighting or waiting, the mouse is perfectly still. Humans constantly fidget with the mouse.
+
+4. **Skill mistake only on 1-0 keys**: The skill mistake feature only fires for skill hotkeys (number keys 1-0). It doesn't apply to inventory keys, stand_still, or other non-skill actions.
+
+## What Would Improve Stealth Further
+
+### High Impact (Warden likely checks these)
+1. **Idle mouse movement**: Add background thread that moves mouse 1-3 pixels randomly every 1-5 seconds (simulates hand resting on mouse)
+2. **Occasional straight-line clicks**: 10% of the time skip the Bezier curve and click directly (humans don't always move in curves)
+
+### Medium Impact
+3. **Process parent spoofing**: Launch the bot from a different process (e.g. from Explorer instead of directly) to hide the relationship with D2R
+4. **Variable scroll speed**: Add variance to `mouse.wheel()` calls (inventory scrolling)
+5. **Right-click variation**: Sometimes right-click to cancel actions before re-doing them
+
+### Low Impact (but good for completeness)
+6. **Randomize AFK break more**: Current range is 2-12 minutes, could be wider
+7. **Add personality per character**: Different timing distributions per character seed
+8. **Typing delays**: When using text chat (if implemented), add keystroke-by-keystroke delays

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,
@@ -42,5 +42,9 @@
     "Chat Gem Mode": 1,
     "Item Name Display": 1,
     "Chat Background": 0,
-    "Always Run": 1
+    "Always Run": 1,
+    "Quick Cast Enabled": 0,
+    "Display Active Skill Bindings": 0,
+    "Lobby Wide Item Drop Enabled": 1,
+    "Item Tooltip Hotkey Appender": 1
 }