Skip to content

feat: add activity processor tag extraction thresholds - #338

Closed
JimmyPettersson85 wants to merge 1 commit into
mainfrom
feat/feeds-1792-tag-extraction-thresholds
Closed

feat: add activity processor tag extraction thresholds#338
JimmyPettersson85 wants to merge 1 commit into
mainfrom
feat/feeds-1792-tag-extraction-thresholds

Conversation

@JimmyPettersson85

@JimmyPettersson85JimmyPettersson85 commented Aug 22, 2026

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

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

Copy link
Copy Markdown
ContributorAuthor

Superseded by #339, which carries the identical commit on feature/feeds-1792-configurable-tag-extraction-thresholds.

The branch name has to match the chat branch: tests/qa/Makefile resolves VIDEO_JS_SERVER_SDK_BRANCH by looking for a stream-node branch of the same name and swaps the SDK to it, which is how GetStream/chat#16074's QA tests exercise the new fields before this SDK is released. Closing this one and deleting its branch.

@JimmyPettersson85
JimmyPettersson85 deleted the feat/feeds-1792-tag-extraction-thresholds branch August 22, 2026 21:55
Sign up for freeto 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

@JimmyPettersson85