Skip to content

Pull strategy migration#2310

Merged
Celandriel merged 13 commits intomod-playerbots:test-stagingfrom
kadeshar:pull-strategy-migration
Apr 17, 2026
Merged

Pull strategy migration#2310
Celandriel merged 13 commits intomod-playerbots:test-stagingfrom
kadeshar:pull-strategy-migration

Conversation

@kadeshar
Copy link
Copy Markdown
Collaborator

@kadeshar kadeshar commented Apr 15, 2026

Pull Request Description

Pull strategy migration from cmangos for tank specializations

How to Test the Changes

  1. Invite bot tank
  2. Use reset boAI or nc +pull,+pull back + co +pull,+pull back
  3. Order bot to pull using command pull my target or pull rti target
  4. Bot should run to mob, use ranged skill and back to point where he started pull
    Without pull back strategy bot run to mob, use ranged skill and wait on mob until he come to bot

Impact Assessment

  • Does this change increase per-bot/per-tick processing or risk scaling poorly with thousands of bots?

      • No, not at all
      • Minimal impact (explain below)
      • Moderate impact (explain below)
  • Does this change modify default bot behavior?

      • No
      • Yes (explain why)
  • Does this change add new decision branches or increase maintenance complexity?

      • No
      • Yes (explain below)

AI Assistance

Was AI assistance used while working on this change?

    • No
    • Yes (explain below)

Help with migration and solving some problems

Final Checklist

    • Stability is not compromised.
    • Performance impact is understood, tested, and acceptable.
    • Added logic complexity is justified and explained.
    • Any new bot dialogue lines are translated.
    • Documentation updated if needed (Conf comments, WiKi commands).

Notes for Reviewers

Stability test after randomize new bots
obraz

@kadeshar kadeshar added Requires WIKI update The wiki documentation needs to updated the changes in the PR Multibot requires update labels Apr 15, 2026
Celandriel
Celandriel previously approved these changes Apr 17, 2026
@Celandriel Celandriel merged commit 19249e9 into mod-playerbots:test-staging Apr 17, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Added translation Multibot requires update Requires WIKI update The wiki documentation needs to updated the changes in the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants