LotusHacks 2026 2nd Place (Overall & TinyFish Enterprise Track): TinyDetective#94
LotusHacks 2026 2nd Place (Overall & TinyFish Enterprise Track): TinyDetective#94injaneity wants to merge 4 commits 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.
Hey @injaneity — awesome project, congrats on the LotusHacks placement! Two things need to be addressed before we can merge:
1. Must use the official TinyFish Python SDK (blocking)
The cookbook requires all projects to use the official TinyFish SDK rather than hand-rolling HTTP clients. Currently services/tinyfish_client.py is a 225-line custom urllib client hitting /v1/automation/run-async and /v1/runs/batch directly.
Please replace this with the tinyfish Python SDK — add it to pyproject.toml and refactor the adapters to use it. This is our most common review item and a hard requirement for all cookbook submissions.
2. Remove vendored jsPDF — use a CDN instead (blocking)
frontend/vendor/jspdf.umd.min.js is a full copy of jsPDF 2.5.1 committed to the repo (~400 lines of minified JS). We don't want vendored third-party libraries in the cookbook.
In frontend/index.html, replace:
<script src="/static/vendor/jspdf.umd.min.js"></script>with:
<script src="https://cdnjs.cloudflare.com/ajax/libs/jspdf/2.5.1/jspdf.umd.min.js"></script>Then delete the frontend/vendor/ directory.
Everything else looks solid — the agent architecture, adapters, and orchestration are well done. Just these two items and we're good to go!
Summary
TinyDetective/cookbook project based on the existing TinyDetective repoValidation
uv run pytestinTinyDetective/Contributing Guide Check
CONTRIBUTING.md