Skip to content

feat(agent-toolkit): support owners in create_board tool#335

Merged
ron-monday merged 4 commits intomasterfrom
feat/create-board-owners-param
May 7, 2026
Merged

feat(agent-toolkit): support owners in create_board tool#335
ron-monday merged 4 commits intomasterfrom
feat/create-board-owners-param

Conversation

@ron-monday
Copy link
Copy Markdown
Collaborator

Summary

  • add optional owners input to the create_board platform API tool schema
  • pass owners through CreateBoardMutationVariables and into the create_board GraphQL mutation
  • update generated GraphQL variable typing used by the tool so the new param is accepted

Test plan

  • run npm run codegen in packages/agent-toolkit (full codegen sync)
  • run npm run build in packages/agent-toolkit
  • smoke: create_board with owners and confirm request includes owners

Made with Cursor

Expose owners input on create_board and pass it through GraphQL mutation variables so MCP users can set board owners at creation time.

Co-authored-by: Cursor <cursoragent@cursor.com>

getDescription(): string {
return 'Create a monday.com board';
return 'Create a monday.com board, optionally setting owner user IDs';
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should not be in the gneeral description, it's enough that the arg itself has a description

boardKind: input.boardKind,
boardDescription: input.boardDescription,
workspaceId: input.workspaceId,
owners: input.owners,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if there is no owners passed (as it is optional) it should not be added in the variables.

What would the result behavior here be if null is sent? I think it's better to include this only when argument is actually passed (input.owners is defined) and please add tests here to validate behavior

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shaneer, the GraphQL mutation of create_board has no issues with "null" as the value of board_owners_ids

Keep create_board description generic and only include owners in mutation variables when explicitly provided, with tests for both variable-shaping paths.

Co-authored-by: Cursor <cursoragent@cursor.com>
@ron-monday ron-monday requested a review from shaneer May 7, 2026 08:03
Align create_board with schema field board_owner_ids and rename MCP input to boardOwnerIds, then update tests to use BoardKind enum values for type-safe execution.

Co-authored-by: Cursor <cursoragent@cursor.com>
@ron-monday ron-monday merged commit e5daecc into master May 7, 2026
2 checks passed
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.

3 participants