Skip to content

fix(create-update): reject unsupported mentions#244

Open
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/create-update-mentions-compat
Open

fix(create-update): reject unsupported mentions#244
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/create-update-mentions-compat

Conversation

@Gujiassh
Copy link

Summary

  • remove the unsupported UpdateMention / mentions_list fields from the shared create_update mutation so plain update creation no longer fails against monday's current schema
  • fail fast with a clear tool-level error when mentionsList is provided instead of sending an API request that the schema cannot satisfy
  • keep normal update creation and parentId replies working, and cover the compatibility change with focused create-update tests

Validation

  • cd packages/agent-toolkit && npx jest src/core/tools/platform-api-tools/create-update-tool/create-update-tool.test.ts --runInBand
  • cd packages/agent-toolkit && npx eslint src/core/tools/platform-api-tools/create-update-tool/create-update.graphql.ts src/core/tools/platform-api-tools/create-update-tool/create-update-tool.ts src/core/tools/platform-api-tools/create-update-tool/create-update-tool.test.ts src/core/tools/platform-api-tools/create-update-tool-ui/create-update-ui-tool.ts --max-warnings=0
  • cd packages/agent-toolkit && yarn build

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
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