Conversation
There was a problem hiding this comment.
Pull request overview
This PR releases version 1.2.1 with improvements to the STT (Speech-To-Text) model handling, caching behavior changes, and bug fixes. The main focus is on creating a dedicated type system for STT models separate from LM models, removing model caching, and refining image handling in multi-turn conversations.
Key Changes
- Introduced
CactusSTTModeltype to properly distinguish STT models from LM models - Removed model caching from both
CactusLMandCactusSTTclasses for fresher data - Modified image handling to only process images in the last message of a conversation
- Simplified Android file system directory structure
Reviewed changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/types/CactusSTTModel.ts | New type interface for STT models with relevant fields (slug, sizeMb, downloadUrl, etc.) |
| src/types/CactusModel.ts | Added supportsCompletion field to LM model interface |
| src/native/Cactus.ts | Modified to only process images on the last message in a conversation, stripping images from earlier messages |
| src/index.tsx | Exported new CactusSTTModel type for public API |
| src/hooks/useCactusSTT.ts | Updated return type from CactusModel[] to CactusSTTModel[] for type correctness |
| src/constants/packageVersion.ts | Bumped version to 1.2.1 |
| src/classes/CactusSTT.ts | Removed model cache, now uses Database.getSTTModel() and Database.getSTTModels() for fresh data |
| src/classes/CactusLM.ts | Removed model cache for consistency with CactusSTT |
| src/api/Database.ts | Added getSTTModel() and getSTTModels() methods with dedicated STT model response types |
| package.json | Version bump to 1.2.1 |
| example/ios/Podfile.lock | Updated Cactus pod version and checksum |
| android/.../HybridCactusFileSystem.kt | Simplified to use context.filesDir directly instead of external files directory |
| README.md | Updated documentation to reflect cache removal and correct return types |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/api/Database.ts
Outdated
| download_url: string; | ||
| size_mb: number; | ||
| created_at: Date; | ||
| file_name: string; |
There was a problem hiding this comment.
The file_name field in CactusSTTModelResponse interface is defined but never used in the mapping functions getSTTModel() and getSTTModels(). Consider removing this field from the interface if it's not needed, or include it in the mapped CactusSTTModel type if it should be available to consumers.
| file_name: string; |
… messages Signed-off-by: Jakub Mroz <kubamroz124@gmail.com>
Signed-off-by: Jakub Mroz <kubamroz124@gmail.com>
No description provided.