Skip to content

runtime: correct and complete a few binding docs - #76

Open
ResurrectedTrader wants to merge 1 commit into
mainfrom
binding-doc-fixes
Open

ResurrectedTrader wants to merge 1 commit into
mainfrom
binding-doc-fixes

Conversation

@ResurrectedTrader

Copy link
Copy Markdown
Owner

Summary

Doc-comment-only fixes in the JS bindings (they feed the API docs and the d.ts):

  • unit.revive() sends the player resurrect request (0x41), not a hireling revive.
  • clickMap names its click types: 0 / 1 / 2 = left down / held / up, 3 / 4 / 5 = right down / held / up.
  • getMouseCoords() reports the screen space drawables use; screenToAutomap() takes game (world) coordinates despite its name and returns the automap's screen point.

Verification

  • build.ps1 Release (Win32, the 1.14d DLL): builds
  • build.ps1 test: 169/169 pass
  • build.ps1 check-format: clean

🤖 Generated with Claude Code

- unit.revive() sends the player resurrect request (0x41), not a
  hireling revive.
- clickMap names its click types (0 / 1 / 2 left down / held / up,
  3 / 4 / 5 right down / held / up).
- getMouseCoords() reports the screen space drawables use;
  screenToAutomap() takes game (world) coordinates despite its name and
  returns the automap's screen point.

Doc comments only.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

This branch has not been deployed

No deployments
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.

1 participant