Skip to content

fix(add_content_to_doc): ignore null block ids#274

Open
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/add-content-null-block-ids
Open

fix(add_content_to_doc): ignore null block ids#274
Gujiassh wants to merge 1 commit intomondaycom:masterfrom
Gujiassh:fix/add-content-null-block-ids

Conversation

@Gujiassh
Copy link

Summary

  • ignore null block_ids when formatting successful add-content-to-doc responses
  • base the created block count and returned block ID list on non-null block ids only
  • add regression coverage for successful mutation responses containing null block ids

Testing

  • npx jest src/core/tools/platform-api-tools/add-content-to-doc-tool/add-content-to-doc-tool.test.ts
  • npx eslint src/core/tools/platform-api-tools/add-content-to-doc-tool/add-content-to-doc-tool.ts src/core/tools/platform-api-tools/add-content-to-doc-tool/add-content-to-doc-tool.test.ts
  • npm run build

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

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