Skip to content

feat(read-docs): add include_comments to fetch doc comments and replies#281

Merged
RoniLandau merged 2 commits intomasterfrom
feat/read-docs-include-comments
Mar 26, 2026
Merged

feat(read-docs): add include_comments to fetch doc comments and replies#281
RoniLandau merged 2 commits intomasterfrom
feat/read-docs-include-comments

Conversation

@RoniLandau
Copy link
Copy Markdown
Collaborator

@RoniLandau RoniLandau commented Mar 26, 2026

Summary

  • Adds include_comments (boolean, default false) and comments_limit (number, default 50) parameters to the read_docs tool
  • When enabled, fetches all comments and replies on a document via the backing board's items_page → items → updates → replies GraphQL path (using the doc's object_id as the board ID)
  • Each comment includes id, text_body, body, created_at, creator, item_id, item_name, and nested replies
  • Fixes pre-existing timeouttimeoutMs typos in search-tool and get-board-items-page-tool that blocked builds

Test plan

  • 7 new tests covering: comments with replies, default behavior, comments_limit passthrough, skipping empty items, error handling
  • All 30 tests pass (23 existing + 7 new)
  • Manual test with a real doc that has comments

🤖 Generated with Claude Code

const smartSearchRes = await this.mondayApi.request<SearchItemsDevQuery>(searchItemsDev, smartSearchVariables, {
versionOverride: 'dev',
timeout: SEARCH_TIMEOUT
timeoutMs: SEARCH_TIMEOUT
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.

why did you change it?

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.

automatic change because of the wrong type

…d replies

Doc comments are stored at the item level within the doc's backing board.
This adds an `include_comments` parameter (default: false) that fetches
all comments and replies via the board's items_page → updates → replies
GraphQL path using the doc's object_id. Also adds `comments_limit` to
control pagination.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@RoniLandau RoniLandau force-pushed the feat/read-docs-include-comments branch from bd8992a to de96c6e Compare March 26, 2026 11:12
@RoniLandau RoniLandau merged commit da513b3 into master Mar 26, 2026
1 check 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