Skip to content

Add cartography table plugin (lint-fixed version of #3778)#3826

Open
plainprince wants to merge 25 commits intoPrismarineJS:masterfrom
plainprince:fix/cartography-table-lint
Open

Add cartography table plugin (lint-fixed version of #3778)#3826
plainprince wants to merge 25 commits intoPrismarineJS:masterfrom
plainprince:fix/cartography-table-lint

Conversation

@plainprince
Copy link
Copy Markdown
Contributor

@plainprince plainprince commented Feb 8, 2026

Summary

  • Rebased version of Added ability to use cartography table #3778 by @AboveAphid with lint fixes applied
  • Adds cartography table plugin (lib/plugins/cartography_table.js) enabling bot interaction with cartography tables (put/take maps, modifiers, and outputs)
  • Registers the plugin in lib/loader.js and documents the API in docs/api.md

Changes from #3778

Fixed all standard lint errors in cartography_table.js:

  • Added missing spaces before function parentheses (space-before-function-paren)
  • Removed blank line padding inside block (padded-blocks)

Note on 1.13.2 CI failure

The useChests test failure on 1.13.2 (27 !== 305) in #3778 is a pre-existing flaky test unrelated to these changes -- master CI passes consistently.

Test plan

  • npm run lint passes locally
  • CI lint check passes
  • All Minecraft version tests pass (cartography table changes are additive and don't affect existing tests)

Made with Cursor

Based on PrismarineJS#3778 by @AboveAphid. Adds ability to use cartography tables
(put/take maps, modifiers, and outputs). Fixes lint errors
(space-before-function-paren, padded-blocks) that caused CI to fail.

Co-authored-by: Cursor <cursoragent@cursor.com>
@rom1504
Copy link
Copy Markdown
Member

rom1504 commented Mar 28, 2026

Please add tests

@plainprince plainprince force-pushed the fix/cartography-table-lint branch from cc06c55 to 6d3818e Compare April 14, 2026 10:20
@plainprince
Copy link
Copy Markdown
Contributor Author

Well, it seems the CI has "some" problems.
Working on them...

Simeon Kummer added 6 commits April 15, 2026 17:08
… operations

- Fix activateItem: use actual bot entity rotation in use_item packet
- Fix heldItemChanged test: use bot._setSlot instead of updateSlot
- Add wait after cartography take operations for transaction handling
- Add 'minecraft:generic' to allowed window types for 1.19+
- Use moveSlotItem instead of transfer for put operations
- Fix chunk packet generation for newer versions (bitMap computing)
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