docs: update docs for AH protection, Safe Mode, and destruction system

Update CHANGELOG, FEATURES, and CurseForge description with new v3.5
features: AH value protection, Safe Mode template, destruction system v1,
centralized AH lookups, and friendly self-test messages.

Author: Mikolaj Graf

CHANGELOG.md

@@ -5,6 +5,13 @@
 ### Added
 - **Tooltip item status** (`showTooltipStatus`, default: on) — Shows ASP classification in item tooltips: "Will sell (quality filter)", "Protected (uncollected transmog)", "On never-sell list", etc. Works for items in bags, equipped gear, and merchant windows. Togglable in Settings > Display.
 - **Compact mode** (`compactMode`, default: off) — Condensed popup showing item count, total value, per-quality breakdown, and a one-click Sell button. Toggle between compact and detailed views via a button on either popup or `/asp compact`. All filters and protections still apply.
+- **AH value protection** (`ahProtectionEnabled`, default: off) — Protects items worth more than a configurable threshold on the AH from being auto-sold. Requires TSM or Auctionator. Popup rows where AH value exceeds a configurable multiplier of vendor price are color-coded, with a tooltip showing "Worth listing: AH Xg, vendor Yg (Nx)".
+- **Safe Mode template** — New profile template for new users: grays only, all protections on. Wizard defaults to Safe Mode when no template or profile is selected.
+- **Destruction system v1** — Complete rewrite of the auto-destroy feature with separate destroy filters (quality, ilvl, max vendor value), a never-destroy list, countdown confirmation popup, and a bag pressure valve that auto-triggers when free slots drop below a configurable threshold. Items are destroyed one per tick with cursor verification for safety.
+
+### Improved
+- **Self-test messages** — API failure messages are now user-friendly and reassuring instead of technical ("Transmog detection paused — Blizzard changed an API. Sell rules are more conservative until updated.").
+- **Centralized AH lookup** — TSM/Auctionator price queries consolidated into `ns:GetAHValue()` and `ns:HasAHAddon()`, replacing duplicated code in Popup, PopupFilters, and Overlays.
 
 ### Fixed
 - **Bindings.xml parsing error** — Removed invalid `header` attribute from Binding element. The section header is provided by the `BINDING_HEADER_AUTOSELLPLUS` global.

CURSEFORGE_DESCRIPTION.md

@@ -9,10 +9,14 @@ Lightweight, modular, and self-testing on login so it keeps working through patc
 ## Selling
 
 - **Popup preview** -- sortable columns (name, ilvl, vendor price, AH value), checkboxes, one-click Sell All
+- **Compact mode** -- condensed popup showing item count, total value, per-quality breakdown, and a one-click Sell button. Toggle with `/asp compact` or the button on either popup
 - **Three sell modes** -- interactive popup (default), one-click, or fully automatic with configurable delay
 - **Quality filters** -- gray through epic, each with independent ilvl thresholds
+- **Relative ilvl threshold** -- set a single sell threshold as a percentage of your equipped ilvl instead of per-quality sliders
 - **Category filters** -- consumables, trade goods, quest items, miscellaneous
 - **Expansion and slot filters** -- narrow results by expansion or equipment slot
+- **Sell collected transmog** -- optionally sell items whose appearances you've already collected
+- **Sell known collectibles** -- optionally sell already-known mounts, pets, and toys
 - **Confirmations** -- epic, high-value, and Sell All Junk show a confirmation with item count and gold total
 - **Buyback safety** -- highest-value items are sold first so buyback slots hold the most valuable items
 - **Dry run** -- preview what would be sold without actually selling
@@ -27,9 +31,12 @@ Lightweight, modular, and self-testing on login so it keeps working through patc
 - **Transmog** -- source-level appearance checking; uncollected items are never sold
 - **Equipment sets** -- items in any saved gear set are protected
 - **BoE** -- unbound Bind on Equip items are protected
+- **Warband items** -- optionally protect all warband and account-bound items
+- **Mount equipment** -- mount equipment items are protected by default
 - **Soulbound-only mode** -- optionally skip all unbound BoE items, useful for dungeon farmers keeping BoE for AH
 - **Quest items** -- items in the Quest Items category are protected by default
 - **Current expansion materials** -- optionally protect Trade Goods from the current expansion
+- **AH value protection** -- protects items worth listing on the AH (requires TSM or Auctionator). Color-coded popup rows show when AH value exceeds vendor price
 - **Refundable** -- items in the vendor refund window are skipped
 - **Addon hooks** -- AllTheThings and CanIMogIt integration for enhanced transmog detection
 
@@ -47,6 +54,9 @@ Lightweight, modular, and self-testing on login so it keeps working through patc
 
 ## Bag Management
 
+- **Destruction system** -- destroy worthless items when not at a vendor. Separate protection chain, countdown confirmation, cursor-safe one-at-a-time deletion
+- **Bag pressure valve** -- auto-triggers destruction confirmation when free slots drop below a threshold
+- **Never-destroy list** -- per-item blacklist for destruction, independent from never-sell
 - **Bag space guard** -- automatically suggests selling the cheapest junk when free slots are low
 - **Stack limits** -- set max quantities per item, excess goes into the sell queue
 - **Free slot alerts** -- chat message or on-screen warning when bag space is low
@@ -56,6 +66,7 @@ Lightweight, modular, and self-testing on login so it keeps working through patc
 
 ## Tracking
 
+- **Tooltip item status** -- colored status line on item tooltips showing how ASP classifies each item (will sell, protected, on never-sell list, etc.)
 - **Session tracker** -- gold/hour calculation in the minimap button tooltip
 - **Sale history** -- scrollable panel showing the last 200 sales; shift-click to link items in chat
 - **Per-character stats** -- lifetime sales totals visible across alts
@@ -65,7 +76,7 @@ Lightweight, modular, and self-testing on login so it keeps working through patc
 
 ## Profiles
 
-- **Built-in templates** -- Raid Farmer, Transmog Hunter, Leveling Alt, Gold Farmer
+- **Built-in templates** -- Safe Mode, Raid Farmer, Transmog Hunter, Leveling Alt, Gold Farmer
 - **Named profiles** -- save and load settings that persist across sessions
 - **Auto-load** -- last loaded profile restores automatically on login
 - **Instance auto-profiles** -- auto-switch profiles when entering raids, dungeons, battlegrounds, or open world
@@ -87,13 +98,16 @@ Type `/asp` or `/autosell` for the full command list. Key commands:
 - `/asp session` -- View session stats
 - `/asp log ui` -- Open sale history
 - `/asp keep [id] [count]` -- Set stack limits
+- `/asp compact` -- Toggle compact popup mode
+- `/asp destroy` -- Destroy eligible junk items
+- `/asp neverdestroy add/remove/list` -- Manage never-destroy list
 - `/asp wizard` -- Re-run setup wizard
 
 ---
 
 ## Integrations
 
-- **TSM / Auctionator** -- AH price column in popup
+- **TSM / Auctionator** -- AH price column in popup, AH value protection
 - **Bagnon / AdiBags / ArkInventory / Baganator** -- junk mark overlays in bag frames
 - **AllTheThings / CanIMogIt** -- enhanced transmog protection
 - **Leatrix Plus** -- conflict detection

FEATURES.md

@@ -21,7 +21,7 @@ AutoSellPlus is a World of Warcraft addon that shows a popup when visiting a mer
 13. [Stack Limits](#stack-limits)
 14. [Value-Based Eviction](#value-based-eviction)
 15. [Auto-Repair](#auto-repair)
-16. [Auto-Destroy](#auto-destroy)
+16. [Destruction System](#destruction-system)
 17. [Bag Space Monitoring](#bag-space-monitoring)
 18. [Minimap Button](#minimap-button)
 19. [Bag Gold Display](#bag-gold-display)
@@ -44,6 +44,9 @@ AutoSellPlus is a World of Warcraft addon that shows a popup when visiting a mer
 36. [Quest Item Protection](#quest-item-protection)
 37. [Tooltip Item Status](#tooltip-item-status)
 38. [Compact Mode](#compact-mode)
+39. [AH Value Protection](#ah-value-protection)
+40. [Safe Mode Template](#safe-mode-template)
+41. [Destruction System](#destruction-system)
 
 ---
 
@@ -390,24 +393,64 @@ Automatically repairs all gear when visiting a repair-capable merchant.
 
 ---
 
-## Auto-Destroy
-
-Safely destroy junk items that have no vendor value (or when not at a vendor).
-
-### Behavior
-1. Scans bags for destroyable items matching quality and value criteria
-2. Respects never-sell list and stack limits
-3. Destroys max 5 items per invocation (safety limit)
-4. Optional confirmation dialog
+## Destruction System
+
+Complete item destruction system for clearing worthless items when not at a vendor.
+
+### Protection Chain
+Items pass through a separate destruction protection chain before being eligible:
+1. Never-destroy list (manual blacklist)
+2. Never-sell list (also blocks destruction)
+3. Equipment set items (if `destroyProtectEquipmentSets` enabled)
+4. Uncollected transmog (if `destroyProtectTransmog` enabled)
+5. BoE items (if `destroyProtectBoE` enabled)
+6. Refundable items (always skipped)
+7. Quality filter (`destroyMaxQuality`)
+8. Item level filter (`destroyMaxIlvl`)
+9. Vendor value filter (`destroyMaxVendorValue`)
+
+### Countdown Confirmation
+Before destroying, a confirmation popup appears with:
+- Red-themed dialog listing items to destroy (up to 10 shown, remainder counted)
+- Vendor value being lost
+- Countdown timer on the Destroy button (configurable, default 3 seconds)
+- Cancel button to abort
+
+### Cursor Safety
+Each item is destroyed one at a time (0.3s per tick) with full cursor verification:
+1. `ClearCursor()` to ensure clean state
+2. `PickupContainerItem()` to pick up the item
+3. `GetCursorInfo()` to verify the correct item is on the cursor
+4. `DeleteCursorItem()` only if itemID matches
+5. Skips with a warning if cursor has wrong item (e.g., player dragging something)
+
+### Bag Pressure Valve
+Automatically triggers the destruction confirmation when free bag slots drop below a configured threshold:
+- Fires on `BAG_UPDATE_DELAYED` events
+- 60-second cooldown between triggers to avoid spam
+- Only triggers when destruction is enabled and not already in progress
+- Shows the same confirmation popup (never auto-destroys silently)
+
+### Never-Destroy List
+Separate per-item blacklist for destruction (independent from never-sell list):
+- `/asp neverdestroy add <ItemLink>` — add item to never-destroy list
+- `/asp neverdestroy remove <ItemLink>` — remove item
+- `/asp neverdestroy list` — show all items on the list
+- Also manageable via Settings > Lists > Never-Destroy tab
 
 ### Settings
-- `autoDestroyEnabled` (default: off)
-- `autoDestroyMaxQuality` (default: 0 / Poor only) - max quality to destroy (0-4)
-- `autoDestroyMaxValue` (default: 0 / unlimited) - max copper value to destroy
-- `autoDestroyConfirm` (default: on) - require confirmation
+- `destroyEnabled` (default: off) — enable destruction feature
+- `destroyMaxQuality` (default: 0 / Poor only) — max quality to destroy (0-4)
+- `destroyMaxIlvl` (default: 0 / disabled) — max ilvl for equippable items
+- `destroyMaxVendorValue` (default: 0 / unlimited) — max total vendor value (copper)
+- `destroyConfirmCountdown` (default: 3) — seconds before Destroy button activates
+- `destroyFreeSlotTrigger` (default: 0 / disabled) — bag pressure valve threshold
+- `destroyProtectTransmog` (default: on) — protect uncollected transmog
+- `destroyProtectBoE` (default: on) — protect bind-on-equip items
+- `destroyProtectEquipmentSets` (default: on) — protect equipment set items
 
 ### Command
-- `/asp destroy` - trigger item destruction
+- `/asp destroy` — trigger item destruction
 
 ---
 
@@ -715,6 +758,10 @@ All commands use `/asp` or `/autosell` as prefix.
 | `onlySoulbound` | bool | false | Only sell soulbound items, skip all unbound BoE |
 | `protectQuestItems` | bool | true | Never sell Quest Items category |
 | `protectCurrentExpMaterials` | bool | false | Never sell current expansion Trade Goods |
+| `protectMountEquipment` | bool | true | Never sell mount equipment items |
+| `protectWarband` | bool | false | Protect warband and account-bound items |
+| `sellCollectedTransmog` | bool | false | Sell items with already-collected transmog |
+| `sellKnownCollectibles` | bool | false | Sell already-known mounts, pets, and toys |
 
 ### Safety & Confirmation
 | Setting | Type | Default | Description |
@@ -758,13 +805,25 @@ All commands use `/asp` or `/autosell` as prefix.
 | `freeSlotAlertMode` | string | "chat" | Alert mode: "chat" or "screen" |
 | `evictionEnabled` | bool | false | Enable value-based eviction |
 
-### Auto-Destroy
+### Destruction
+| Setting | Type | Default | Description |
+|---------|------|---------|-------------|
+| `destroyEnabled` | bool | false | Enable destruction feature |
+| `destroyMaxQuality` | int | 0 | Max quality to destroy (0-4) |
+| `destroyMaxIlvl` | int | 0 | Max ilvl for equippable items (0 = disabled) |
+| `destroyMaxVendorValue` | int | 0 | Max total vendor value in copper (0 = unlimited) |
+| `destroyConfirmCountdown` | int | 3 | Seconds before Destroy button activates |
+| `destroyFreeSlotTrigger` | int | 0 | Free slot threshold for pressure valve (0 = disabled) |
+| `destroyProtectTransmog` | bool | true | Protect uncollected transmog from destruction |
+| `destroyProtectBoE` | bool | true | Protect bind-on-equip items from destruction |
+| `destroyProtectEquipmentSets` | bool | true | Protect equipment set items from destruction |
+
+### AH Value Protection
 | Setting | Type | Default | Description |
 |---------|------|---------|-------------|
-| `autoDestroyEnabled` | bool | false | Enable destruction feature |
-| `autoDestroyMaxQuality` | int | 0 | Max quality to destroy (0-4) |
-| `autoDestroyMaxValue` | int | 0 | Max copper value to destroy |
-| `autoDestroyConfirm` | bool | true | Require confirmation |
+| `ahProtectionEnabled` | bool | false | Protect items worth listing on AH |
+| `ahProtectionThreshold` | int | 10000 | AH value threshold in copper |
+| `ahHighlightMultiplier` | int | 2 | AH/vendor ratio to color-code popup rows |
 
 ---
 
@@ -839,8 +898,9 @@ charStats["Charactername - Realmname"] = {
 12. **PopupFilters.lua** - Popup filter logic and smart defaults
 13. **ConfirmList.lua** - Confirmation dialog item list panel
 14. **Popup.lua** - Merchant popup frame, item row rendering, sell action
-15. **Selling.lua** - Auto-sell logic, confirmations, eviction, destruction
-16. **Core.lua** - Event handling, sell queue processing, slash commands
+15. **Selling.lua** - Auto-sell logic, confirmations, eviction
+16. **Destroy.lua** - Item destruction system, countdown confirmation, bag pressure valve
+17. **Core.lua** - Event handling, sell queue processing, slash commands
 
 ### Event Flow
 ```
@@ -985,6 +1045,44 @@ The compact popup has independent position and scale saved separately from the f
 
 ---
 
+## AH Value Protection
+
+When enabled (`ahProtectionEnabled`, default: off), items worth more than a configurable threshold on the auction house are protected from being auto-sold. Requires TradeSkillMaster (TSM) or Auctionator.
+
+### How It Works
+1. During sell evaluation, each item's AH market value is looked up via TSM's `DBMarket` or Auctionator's price API
+2. If the AH value exceeds `ahProtectionThreshold` (default: 1g), the item is protected from selling
+3. In the popup, rows where AH value exceeds `ahHighlightMultiplier` times the vendor price are color-coded in light blue
+4. Hovering a color-coded row shows a tooltip: "Worth listing: AH Xg, vendor Yg (Nx)"
+
+### Settings
+- `ahProtectionEnabled` (default: off) — enable AH value protection
+- `ahProtectionThreshold` (default: 10000 copper / 1g) — minimum AH value to trigger protection
+- `ahHighlightMultiplier` (default: 2) — AH/vendor price ratio for popup row coloring
+
+### Requirements
+- TSM or Auctionator must be installed and have price data available
+- Without either addon, AH protection is silently skipped (items sell normally)
+
+---
+
+## Safe Mode Template
+
+A new profile template designed for new users or anyone who wants maximum safety.
+
+### Settings Applied
+- **Quality**: Grays only (whites, greens, blues, epics all disabled)
+- **Protections**: All protections enabled (transmog, equipment sets, BoE, quest items, mount equipment, warband, current expansion materials)
+- **Confirmations**: Epic confirm and high-value confirm enabled (threshold: 1g)
+- **Mode**: Popup mode with buyback warning
+- **Categories**: Only equippable items (no consumables, trade goods, quest items, misc)
+
+### Wizard Integration
+- Safe Mode is rendered first in the wizard template list with a green accent
+- If the user completes the wizard without selecting any template or profile, Safe Mode is automatically applied as the default
+
+---
+
 ## Competitive Analysis & Roadmap
 
 AutoSellPlus is already exceptionally comprehensive, covering 90%+ of player wishlists from recent [r/wowaddons](https://www.reddit.com/r/wowaddons/comments/1qtqsze/legacy_vendor_a_simple_addon_to_clean_up_old/) threads (filters, marking, safeties, profiles). It outclasses Aardvark, Scrap, and Legacy Vendor in proactive tools like eviction and loot-marking previews.