denise/mod-random-enchants
Archived
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Scrub/reset consumable + Instability Cache refund chance #4

Closed
opened 2026-07-07 05:56:11 +00:00 by denise · 1 comment
denise commented 2026-07-07 05:56:11 +00:00
Owner
Copy link

Context

Deferred from PR #3 (5-line cap + cache append). Instability Caches now append lines; players need a counterweight to cursed/gambling append loops.

Goals

  • Scrub/reset consumable — wipes all custom affix lines (and prefix?) from a targeted item; restores item to unaffixed baseline.
  • 3% cache refund — on successful Instability Cache use, small chance to return the consumed cache item to the player (economy knob via config).
  • Re-enable Affix Artificer (900001) gossip/services once scrub flow exists (currently placeholder; RerollNpc.Enable=0).

Notes

  • Keep existing error-path RefundInstabilityCache behavior (rollback on apply failure).
  • Decide UX: item target spell, NPC service, or both.
  • Economy: scrub should be expensive/rare relative to cache drops.

Depends on

  • PR #3 merged and playtested on develop.

Acceptance

  • Consumable removes all affix rows from item_affix_instance for target item
  • Configurable refund chance on cache success (InstabilityCache.RefundChance or similar)
  • World SQL for scrub item + optional NPC gossip wiring
  • Docs updated (SCALING_AND_LOOT_ECONOMY.md, ADDON_PROTOCOL.md)
## Context Deferred from PR #3 (5-line cap + cache append). Instability Caches now **append** lines; players need a counterweight to cursed/gambling append loops. ## Goals - **Scrub/reset consumable** — wipes all custom affix lines (and prefix?) from a targeted item; restores item to unaffixed baseline. - **3% cache refund** — on successful Instability Cache use, small chance to return the consumed cache item to the player (economy knob via config). - Re-enable **Affix Artificer (900001)** gossip/services once scrub flow exists (currently placeholder; `RerollNpc.Enable=0`). ## Notes - Keep existing error-path `RefundInstabilityCache` behavior (rollback on apply failure). - Decide UX: item target spell, NPC service, or both. - Economy: scrub should be expensive/rare relative to cache drops. ## Depends on - PR #3 merged and playtested on `develop`. ## Acceptance - [ ] Consumable removes all affix rows from `item_affix_instance` for target item - [ ] Configurable refund chance on cache success (`InstabilityCache.RefundChance` or similar) - [ ] World SQL for scrub item + optional NPC gossip wiring - [ ] Docs updated (`SCALING_AND_LOOT_ECONOMY.md`, `ADDON_PROTOCOL.md`)
denise commented 2026-07-10 10:38:09 +00:00
Author
Owner
Copy link

Moved to illidari#8. Archived — use illidari monorepo.

Moved to [illidari#8](https://vcs.ablationsystems.com/denise/illidari/issues/8). Archived — use illidari monorepo.
Commenting is not possible because the repository is archived.
No Branch/Tag specified
Branches Tags
master
develop
feature/pity-instability
feature/affix-cap-5-tier-table
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
denise/mod-random-enchants#4
No description provided.