Skip to content

fix(ValeTotem): correct bow selection and withdraw race (#400)#402

Merged
chsami merged 1 commit intomainfrom
development
Apr 18, 2026
Merged

fix(ValeTotem): correct bow selection and withdraw race (#400)#402
chsami merged 1 commit intomainfrom
development

Conversation

@chsami
Copy link
Copy Markdown
Owner

@chsami chsami commented Apr 18, 2026

  • Select the configured bow option by name via Rs2Widget.handleProcessingInterface so the hotkey is resolved from the live fletching interface rather than a hardcoded child ID. The previous scheme mapped SHORTBOW → child 15 with hotkey-index 1, but widget 270,13's sparse dynamic children could produce "3" at that slot for some log types, causing shortbow selection to press the longbow key.
  • Bank withdraws now sleepUntil on the concrete predicate (hasKnife, hasLogBasket, log count reaches requested target) instead of the generic waitForInventoryChanges + re-check. Rs2Bank.withdrawOne/withdrawX return immediately after queueing the menu click, so under server lag the follow-up check would read stale state and spuriously report "no knife available" right after the knife was withdrawn.

Bumps plugin to 1.0.10.

- Select the configured bow option by name via
  Rs2Widget.handleProcessingInterface so the hotkey is resolved from the
  live fletching interface rather than a hardcoded child ID. The previous
  scheme mapped SHORTBOW → child 15 with hotkey-index 1, but widget 270,13's
  sparse dynamic children could produce "3" at that slot for some log types,
  causing shortbow selection to press the longbow key.
- Bank withdraws now sleepUntil on the concrete predicate (hasKnife,
  hasLogBasket, log count reaches requested target) instead of the generic
  waitForInventoryChanges + re-check. Rs2Bank.withdrawOne/withdrawX return
  immediately after queueing the menu click, so under server lag the
  follow-up check would read stale state and spuriously report "no knife
  available" right after the knife was withdrawn.

Bumps plugin to 1.0.10.
@chsami chsami merged commit 219e9e3 into main Apr 18, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants