Skip to content

LotusHacks 2026 2nd Place (Overall & TinyFish Enterprise Track): TinyDetective#94

Open
injaneity wants to merge 4 commits into
tinyfish-io:mainfrom
injaneity:injaneity/tinydetective-cookbook
Open

LotusHacks 2026 2nd Place (Overall & TinyFish Enterprise Track): TinyDetective#94
injaneity wants to merge 4 commits into
tinyfish-io:mainfrom
injaneity:injaneity/tinydetective-cookbook

Conversation

@injaneity

Copy link
Copy Markdown

Summary

  • Add a new root-level TinyDetective/ cookbook project based on the existing TinyDetective repo
  • Adapt the project README to match the cookbook convention with title, live link, app description, demo, TinyFish API snippet, run instructions, and architecture diagram
  • Add creators credits and LotusHacks x HackHarvard Vietnam 2026 placement context at the top of the project README
  • Update the main cookbook README recipes table with a TinyDetective entry

Validation

  • uv run pytest in TinyDetective/
  • Result: 17 passed

Contributing Guide Check

  • Project lives in its own root-level folder
  • Project README includes the required sections described in CONTRIBUTING.md
  • Main cookbook README is updated to link to the new project

@coderabbitai

coderabbitai Bot commented Mar 25, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 7dcf614f-4f44-4a7d-b2a8-5d15e81666ba

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@simantak-dabhade simantak-dabhade left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants