Added find blocks function and a check if bot is placing a block where it is standing#250
Conversation
|
@demid-ns, can you check my pr? |
Yes, I will in next few days |
There was a problem hiding this comment.
Pull request overview
This PR updates the Minecraft MCP server’s block interaction tools to (1) prevent the bot from placing blocks in its own occupied space, and (2) expand the block-finding capability to optionally return multiple nearby matches.
Changes:
- Added a self-placement guard to
place-block(blocks placing at the bot’s feet and the block above). - Renamed
find-blocktofind-blocksand added optionalcountsupport for returning multiple block positions. - Updated tests and README command list to reflect the new tool name and behavior.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
src/tools/block-tools.ts |
Adds place-block self-placement check; renames and extends find-block tool to support multiple results. |
tests/block-tools.test.ts |
Updates tool registration expectations; adds coverage for self-placement guard and multi-result find-blocks. |
README.md |
Updates available command list from find-block to find-blocks. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Hey @agrofx1 thanks for the contribution. I apologise that the review took so much time |
Changes:
I commited in a main branch because it is a small change