Skip to content

fix(agent-toolkit): clarify full board data helper description#417

Open
ebbsanchez wants to merge 1 commit into
mondaycom:masterfrom
ebbsanchez:fix/full-board-data-description
Open

fix(agent-toolkit): clarify full board data helper description#417
ebbsanchez wants to merge 1 commit into
mondaycom:masterfrom
ebbsanchez:fix/full-board-data-description

Conversation

@ebbsanchez

Copy link
Copy Markdown

Summary

  • replace the hard "DO NOT CALL THIS TOOL DIRECTLY" warning on get_full_board_data with guidance for the intended UI-display workflow
  • add a focused metadata test so the helper description keeps explaining when agents should use narrower tools instead

Closes #366

Verification

  • ../../node_modules/.bin/jest -c jest.config.ts src/core/tools/platform-api-tools/full-board-data-tool/full-board-data-tool.test.ts --runInBand
  • ../../node_modules/.bin/jest -c jest.config.ts src/core/tools/tool-description-safety.test.ts --runInBand

Note: verified locally under Node 22.17.1 because this machine does not have the repo CI version Node 20.18.1 installed.

@ebbsanchez

Copy link
Copy Markdown
Author

Hi, gentle follow-up after a couple of weeks. This is still ready on my side; happy to adjust or close if it is not aligned with the project direction.

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.

Confusing warning "INTERNAL USE ONLY - DO NOT CALL THIS TOOL DIRECTLY"

2 participants