Skip to content

feat: add activity processor tag extraction thresholds - #339

Merged
JimmyPettersson85 merged 1 commit into
mainfrom
feature/feeds-1792-configurable-tag-extraction-thresholds
Aug 24, 2026
Merged

feat: add activity processor tag extraction thresholds#339
JimmyPettersson85 merged 1 commit into
mainfrom
feature/feeds-1792-configurable-tag-extraction-thresholds

Conversation

@JimmyPettersson85

Copy link
Copy Markdown
Contributor

Ticket

Summary

Regenerates src/gen from the chat OpenAPI spec to pick up two new optional fields on ActivityProcessorConfig:

min_text_length?: number;
min_word_count?: number;

They let a feed group gate text_interest_tags extraction on how much text an activity actually has. Today there is no gate at all — a one-character activity triggers a full LLM call and whatever topics the model invents land in interest_tags, which is what a customer reported as junk tags. Both fields default to 0 (disabled), so existing behaviour is unchanged and the knob is opt-in.

Notes for review

Branch name matches the chat branch on purpose. tests/qa/Makefile resolves VIDEO_JS_SERVER_SDK_BRANCH by looking for a stream-node branch with the same name as the chat branch, and swaps @stream-io/node-sdk to it for the QA run. That is how GetStream/chat#16074's QA tests exercise min_text_length / min_word_count before this SDK is released. (This PR supersedes #338, which sat on a non-matching branch name.)

The regeneration diff is only those 10 lines — src/gen was already in sync with master, so nothing unrelated came along.

Both fields are optional, so this is additive and backwards compatible: existing callers constructing an ActivityProcessorConfig keep compiling untouched.

Verification

  • yarn generate:open-api — regenerated, diff scoped to ActivityProcessorConfig
  • yarn build — clean
  • yarn lint — clean, no new warnings
  • yarn test — 18 failed / 3 passed / 3 skipped, identical to main. Every failure is secretOrPrivateKey must have a value from missing STREAM_API_SECRET locally; verified by stashing the change and re-running. A type-only addition cannot affect these.

🤖 Generated with Claude Code

Regenerate the models from the chat OpenAPI spec, picking up the optional
min_text_length and min_word_count fields on ActivityProcessorConfig.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@JimmyPettersson85
JimmyPettersson85 merged commit 740fda1 into main Aug 24, 2026
25 of 26 checks passed
@JimmyPettersson85
JimmyPettersson85 deleted the feature/feeds-1792-configurable-tag-extraction-thresholds branch August 24, 2026 14:35
oliverlaz pushed a commit that referenced this pull request Aug 24, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.8.3](v0.8.2...v0.8.3)
(2026-08-24)


### Features

* add activity processor tag extraction thresholds
([#339](#339))
([740fda1](740fda1))

---
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.

2 participants