Skip to content

feat: [FEEDS-1965] add top_level_comment_count to feeds getActivity and getComments - #359

Merged
JimmyPettersson85 merged 4 commits into
mainfrom
feature/feeds-1965-top-level-comment-count
Sep 29, 2026
Merged

JimmyPettersson85 merged 4 commits into
mainfrom
feature/feeds-1965-top-level-comment-count

Conversation

@JimmyPettersson85

Copy link
Copy Markdown
Contributor

Ticket

https://linear.app/stream/issue/FEEDS-1965

Summary

Regenerates the server SDK for the new read-time top-level comment count on the feeds backend:

Where Field Notes
feeds.getActivity request include_top_level_comment_count?: boolean opt-in
ActivityResponse top_level_comment_count?: number only set by GetActivity when requested; never on feed reads or events
feeds.getComments request include_top_level_comment_count?: boolean opt-in; set it on the first page only (the id_around request for deep links)
GetCommentsResponse comment_count: number always present; whole comment tree, replies included
GetCommentsResponse top_level_comment_count?: number only when requested

top_level_comment_count counts comments made directly on the activity (replies excluded), with the same active/not-deleted rules as comment_count. It is not adjusted per viewer.

Implementation notes

Two commits, so the feature diff is reviewable on its own:

  1. chore: regenerate from the current chat OpenAPI spec: API changes already on chat master since the last regeneration, generated from the chat master spec. No feature changes.
  2. feat: add top_level_comment_count …: the actual change, 16 lines in src/gen/feeds/FeedsApi.ts and src/gen/models/index.ts.

Both generated with chat's tools/openapi (generate-client --language ts, as in generate-openapi.sh) followed by yarn lint:gen; nothing under src/gen is hand-edited. yarn build and yarn lint pass. I ran the chat QA suite locally against this build (yarn-linked) using the typed feeds.getActivity / feeds.getComments methods against a backend built from the chat branch; it passed.

Chat backend PR: branch feature/feeds-1965-top-level-comment-count in GetStream/chat (the chat QA Makefile installs this branch by name).

🤖 Generated with Claude Code

JimmyPettersson85 and others added 2 commits September 29, 2026 11:11
Picks up API changes already on chat master before adding FEEDS-1965
fields in the next commit. No hand edits.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Regenerated for FEEDS-1965: include_top_level_comment_count on
getActivity/getComments, top_level_comment_count on both responses,
and the always-present comment_count on GetCommentsResponse.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JimmyPettersson85
JimmyPettersson85 merged commit bf251c5 into main Sep 29, 2026
17 of 25 checks passed
@JimmyPettersson85
JimmyPettersson85 deleted the feature/feeds-1965-top-level-comment-count branch September 29, 2026 18:38
JimmyPettersson85 pushed a commit that referenced this pull request Sep 29, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.8.9](v0.8.8...v0.8.9)
(2026-09-29)


### Features

* [FEEDS-1965] add top_level_comment_count to feeds getActivity and
getComments
([#359](#359))
([bf251c5](bf251c5))
* regenerate with the flat sendRequest response and no default JSON
content type
([#357](#357))
([9d50ab4](9d50ab4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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