Skip to content

Fix Ankabut mob skill list#10540

Closed
djjohnson13p wants to merge 1 commit into
LandSandBoat:basefrom
djjohnson13p:fix/issue-10387-ankabut-skill-list
Closed

Fix Ankabut mob skill list#10540
djjohnson13p wants to merge 1 commit into
LandSandBoat:basefrom
djjohnson13p:fix/issue-10387-ankabut-skill-list

Conversation

@djjohnson13p

Copy link
Copy Markdown

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

Closes #10387.

Updates Ankabut's mob skill list entry from mob skill 704 to 811.

Evidence checked:

  • sql/mob_pools.sql assigns Ankabut to skill list 1007.
  • sql/mob_skill_lists.sql had Ankabut list 1007 pointing at 704.
  • sql/mob_skills.sql has 704 only as a commented-out counterstance_2 row.
  • sql/mob_skills.sql defines 811 as acid_spray.
  • BG Wiki currently lists Ankabut as using Acid Spray.

Steps to test these changes

Ran locally from this branch:

PYTHONUTF8=1 /c/Program\ Files/Git/bin/bash.exe tools/ci/sanity_checks.sh upstream/base

Result: all sanity checks passed.

Also ran a targeted parser to confirm Ankabut has exactly one skill-list row and that it points to defined mob skill 811.

Not run locally:

  • Docker compose build/test/startup services, because Docker is not installed/on PATH in this Windows environment.
  • Native CMake build, because CMake selects NMake but nmake and C/C++ compilers are not installed/on PATH.

Ankabut's mob pool uses skill list 1007. That list referenced mob skill 704, but mob_skills.sql only has counterstance_2 commented out.

Issue LandSandBoat#10387 and BG Wiki list Ankabut as using Acid Spray, which is mob skill 811 in mob_skills.sql.
@djjohnson13p djjohnson13p marked this pull request as ready for review July 8, 2026 18:52
@sruon sruon closed this Jul 8, 2026
@djjohnson13p

Copy link
Copy Markdown
Author

Apologies for the noise here. This PR was opened by an AI-assisted workflow before it had the human review and fork-only staging that Dale intended. The work was meant to stay isolated in my fork/experimental branch until it could be reviewed properly, not to consume upstream maintainer time yet.

I'm withdrawing this PR now and will keep any further experimentation isolated to my fork unless I later prepare a manually reviewed contribution.

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.

🐛 Ankabut's skill list has skill id with no skill in mob_skills.sql causing crashes.

2 participants