Migration restaurant-comparison-tool to Tinyfish SDK #171
Migration restaurant-comparison-tool to Tinyfish SDK #171KrishnaAgarwal7531 wants to merge 3 commits intotinyfish-io:mainfrom
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 — SDK idiomatic, no Mino, build passes, Vite leftovers removed, README references .env.example. Old #139 issues all addressed.
Two fixes needed:
1. Remove GROQ_API_KEY from .env.example
Lines 5-7 list GROQ_API_KEY but there are zero Groq references anywhere in src/. The app only needs TINYFISH_API_KEY. Having a dead key misleads users into signing up for Groq they don't need.
2. README sign-up URL
Line 96 shows https://tinyfish.ai — should be https://agent.tinyfish.ai/api-keys to match .env.example and the canonical URL.
Nits (non-blocking):
eslint.config.jsstill importseslint-plugin-react-refresh(Vite leftover) —npm run lintwould failnext-env.d.tsis committed — normally.gitignored
Removes the last Vite artefact from the App Router migration. src/App.tsx was a near-identical duplicate of src/app/page.tsx left over from the Vite entry point — nothing imported it, now it's gone.
Also fixes the README setup flow: setup step now points to .env.example with a cp command instead of instructing users to hand-author a .env.local, and the sign-up link corrected from agent.tinyfish.ai to tinyfish.ai to match .env.example. README fully rewritten to match the current codebase.