Migration code-reference-finder to Tinyfish SDK - #209
KrishnaAgarwal7531 wants to merge 1 commit into
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.
Review: changes requested
SDK migration is well done — client.search.query() and client.agent.stream() used correctly, old mino-client.ts and openrouter.ts properly deleted, no secrets, no Mino refs.
Build is broken due to a type mismatch and there's a dead file that needs removing.
Blockers
1. apiData removed from type but still referenced
src/lib/goal-builder.ts:53 accesses searchResult.apiData but the SearchResult interface in types.ts no longer has an apiData property. This breaks the build.
Fix: either remove the apiData usage from goal-builder.ts (use searchResult.snippet as fallback), or add apiData?: { body_excerpt?: string } back to the SearchResult interface.
2. Delete dead src/lib/groq-client.ts
This file imports groq-sdk which isn't in package.json and is never imported by any other file. It's completely dead code. Delete it.
3. Remove next-env.d.ts
Auto-generated by Next.js, should not be committed. Delete it and add next-env.d.ts to .gitignore.
Non-blocking
- README says "Tailwind CSS 4" but
package.jsonhastailwindcss ^3.4.17— fix the README. - README constraint checklist mentions "Mino" in a row label — change to just "Raw SSE fetch?"
Once the build passes (npm install && npm run build), this is good to merge.
Migrates all three external dependencies to TinyFish SDK + Groq. The GitHub API and StackExchange API are replaced entirely by client.search.query() with site:github.com and site:stackoverflow.com filters — no more GITHUB_TOKEN, STACKEXCHANGE_KEY, rate limit retries, or sequential delays. OpenRouter is replaced by groq-sdk directly. Mino raw SSE fetch is replaced by client.agent.stream with typed events and result validation. Promise.allSettled for parallel agents. mino-client.ts, openrouter.ts, and the original search.ts are all deleted.