Skip to content

fix(deps): update @todoist/comms-sdk to 5.0.1 - #121

Merged
scottlovegrove merged 1 commit into
mainfrom
scottl/bump-comms-sdk-5.0.1
Nov 11, 2026
Merged

scottlovegrove merged 1 commit into
mainfrom
scottl/bump-comms-sdk-5.0.1

Conversation

@scottlovegrove

Copy link
Copy Markdown
Collaborator

Updates @todoist/comms-sdk from 5.0.0 to 5.0.1, which picks up todoist/comms-sdk-typescript#145 (todoist/comms-sdk-typescript#144).

Why

The search API can send snippet: null for a result. SDK 5.0.0 required a string, so one such result made api.search.search() throw for the whole page. 5.0.1 accepts null and returns it as an empty string, so the rest of the page and its cursor come through. The SDK type is still string, so no CLI code changes.

Evidence

  • The lockfile diff touches only @todoist/comms-sdk.
  • npm run verify passed (lint, types, build, skill sync, tests).
  • The regression test lives in the SDK.
  • Not run: tdc search against the real Comms API.

5.0.1 accepts a null `snippet` on search results instead of failing
the whole page, so search keeps the other results and the cursor.
@scottlovegrove scottlovegrove self-assigned this Nov 11, 2026

@doistbot doistbot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This PR updates fix(deps): update @todoist/comms-sdk to 5.0.1. No issues were flagged in the reviewed diff.

Share Feedback • Review Logs

@scottlovegrove scottlovegrove added the 🚢 Ship PR requires passing checks before merge, but no independent human review. label Nov 11, 2026
@scottlovegrove
scottlovegrove merged commit 964a89c into main Nov 11, 2026
6 checks passed
@scottlovegrove
scottlovegrove deleted the scottl/bump-comms-sdk-5.0.1 branch November 11, 2026 18:47
doist-release-bot Bot added a commit that referenced this pull request Nov 11, 2026
## [5.0.2](v5.0.1...v5.0.2) (2026-11-11)

### Bug Fixes

* **deps:** update @todoist/comms-sdk to 5.0.1 ([#121](#121)) ([964a89c](964a89c))
@doist-release-bot

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released 🚢 Ship PR requires passing checks before merge, but no independent human review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants