Skip to content

feat: wire custom vocabulary from settings into keyword boosting#4237

Open
devin-ai-integration[bot] wants to merge 2 commits intomainfrom
devin/1772024338-keyword-boosting
Open

feat: wire custom vocabulary from settings into keyword boosting#4237
devin-ai-integration[bot] wants to merge 2 commits intomainfrom
devin/1772024338-keyword-boosting

Conversation

@devin-ai-integration
Copy link
Contributor

feat: wire custom vocabulary from settings into keyword boosting

Summary

Previously, useKeywords only extracted keywords from the note's markdown (hashtags + NLP). Custom vocabulary items added in Settings > Memory were stored but never sent to STT providers for keyword boosting.

This wires the visibleVocabs query (memories table, type="vocab") into the useKeywords hook so custom vocab entries are included in the keywords array passed to both streaming and batch transcription.

Custom vocab items are prepended so they survive deduplication in combineKeywords.

Review & Testing Checklist for Human

  • Verify that adding a custom vocab term in Settings > Memory > Custom Vocabulary causes it to appear in the keywords sent during a live transcription session (check websocket params or logs)
  • Verify that removing a custom vocab term stops it from being boosted in subsequent transcriptions
  • Confirm no excessive re-renders — vocabTable is a full table subscription, but vocab changes are infrequent so this should be fine in practice

Notes

  • Also documented the team's custom vocabulary discussion on #3512
  • STT providers typically cap keyword boosting at ~100 words. There's no client-side limit on custom vocab count — worth considering if users add large vocabularies (pre-existing concern, not introduced here).
  • Link to Devin run
  • Requested by @ComputelessComputer

…eline

Co-Authored-By: John <john@hyprnote.com>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@netlify
Copy link

netlify bot commented Feb 25, 2026

Deploy Preview for hyprnote-storybook canceled.

Name Link
🔨 Latest commit db2a221
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/699ef43f2f8bfc000806f6b5

@netlify
Copy link

netlify bot commented Feb 25, 2026

Deploy Preview for hyprnote canceled.

Name Link
🔨 Latest commit db2a221
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/699ef43fc54b2e00089ae335

Co-Authored-By: John <john@hyprnote.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.

1 participant