Migration research-sentry to Tinyfish SDK#204
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
simantak-dabhade
left a comment
There was a problem hiding this comment.
Clean migration. Fixed optional chaining on comparison.points, removed next-env.d.ts, fixed README typos. Build passes.
Migrates Research Sentry to the official TinyFish SDK and fixes the error handling that was swallowing all failure modes as silent null returns. The library now throws a typed TinyFishError with a code field — missing key, run failed, timeout, stream error, and no result are all distinguishable by the caller.
Also cleans up leftover traces of the old OpenAI dependency — the app uses Groq for all LLM calls and transcription (including Whisper via Groq's endpoint), not OpenAI. README, tech stack table, and env file all updated to reflect this. .env.local.example renamed to .env.example per the cookbook convention, and bare .env added to .gitignore so a cp .env.example .env workflow doesn't leak keys.