Skip to content

Live demo link, 30/30 benchmark, correct social preview URL - #15

Merged
CodexManvik merged 1 commit into
mainfrom
live-demo-and-score
Nov 9, 2026
Merged

CodexManvik merged 1 commit into
mainfrom
live-demo-and-score

Conversation

@CodexManvik

Copy link
Copy Markdown
Owner

What

  • README: a "Try it live" link to https://float-chat-ai-teal.vercel.app at the top. The hero screenshot and the data table link there too.
  • Benchmark: re-run against the live API after Router: requests to see profiles go to search #14 deployed. Score: 30/30 (routing 100%, filters 100%, answer checks 100%, median 1.9 s). The LLM wrote 28 of 30 answers. The other two fell back to the data-only summary when Groq's free tier hit its tokens-per-minute limit during the back-to-back run; re-asked 20 s apart, the LLM writes them. The README explains this and quotes the run log label.
  • Social preview fix: metadataBase was hard-coded to https://floatchat.vercel.app, a domain this project doesn't own, so the live site advertised https://floatchat.vercel.app/opengraph-image as its preview image. It now comes from NEXT_PUBLIC_SITE_URL, defaulting to the real site. DEPLOY.md tells forks to set it.
  • Screenshot: docs/screenshots/web-chat-sql.png retaken from the live site. It now shows the prose explanation and the compact answer header.

Checks

  • Web: lint, typecheck, 18 Vitest tests pass.
  • Live site checked end to end: landing page, a streamed chat answer and a float page, with no console errors. CORS allows the Vercel origin.
  • After merge: the live page's og:image should start with https://float-chat-ai-teal.vercel.app.

🤖 Generated with Claude Code

- README: "Try it live" link to https://float-chat-ai-teal.vercel.app at the top; the
  hero screenshot and the data table link to it too.
- Benchmark re-run against the live API after the router fix (#14): 30/30, routing
  100%, median 1.9 s. Two answers fell back to the data-only summary when Groq's
  free tier hit its per-minute token limit; the README says so.
- metadataBase was hard-coded to floatchat.vercel.app, a domain this project does
  not own, so shared links pointed their preview image there. It now comes from
  NEXT_PUBLIC_SITE_URL, defaulting to the real site; DEPLOY.md tells forks to set it.
- Chat screenshot retaken from the live site (prose explanation, compact header).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
float-chat-ai Ready Ready Preview Nov 9, 2026 5:17am UTC

Copilot AI balanced review requested due to automatic review settings November 9, 2026 05:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@coderabbitai

coderabbitai Bot commented Nov 9, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 0dad89dd-cf0c-4010-9bf0-3be78fcab690
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@CodexManvik
CodexManvik merged commit 1bbc373 into main Nov 9, 2026
9 checks passed
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.

2 participants