content: add Harmonia case study - #130
Conversation
Full write-up of the Harmonia music-intelligence project, built on top of the draft in #57 but verified and corrected against the actual repo: Next.js 16 (not 15), Groq gpt-oss-20b for classification (not llama-3.3-70b), the real four-app monorepo shape, and the actual waitlist-gated product flow. No product screenshots exist yet (blocked on harmonia#142's design refactor), so the case study ships with two hand-authored diagrams instead: the six-stage pipeline architecture, and a chart of the real before/after numbers from the adaptive-clustering fix (harmonia#111 -> #329). The "Engineering Challenges" section links each of the five real bugs to both its issue and its fix PR, with the actual figures pulled from those issues (16 retry requests, 56 fan-out workers, 33/33 classify failures, 2-3 vs 25-35 playlists). next.config.ts now allows SVG through next/image for this one local, site-owned asset.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (3)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe change adds a Harmonia project case-study page and introduces a curated order for featured projects. Other projects continue to use the existing status-then-id ordering. ChangesHarmonia project listing
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This PR adds a Harmonia case study and project artwork without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review. 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
Full details: Description checkExplanation The description provides a detailed summary, issue reference, implementation details, test results, and remaining visual-check status. It does not reproduce every template heading or checklist item, but it contains the required information. Full details: Linked Issues checkExplanation The PR satisfies the documented Harmonia case-study requirements, including Resolution Review the excluded assets Full details: Docstring CoverageExplanation Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 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 |
|
harmonia is actually also put it as top 1 and then dukkani and then zerolocker and then findmalek and then rest like u want |
status: "draft" -> "published" per Malek's correction. Also adds FEATURED_PROJECT_ORDER (Harmonia, Dukkani, Zero Locker, FindMalek) to pin those at the top of the /projects grid in that exact order, with everything else still falling back to the existing status-then-id sort.
|
Both done — status is now |
|
for the image of harmonia, there is a logo in repo of harmonia and for the opengraph pls use this https://github.com/FindMalek/harmonia/blob/main/packages/email/public/email/images/feature-update-hero.png |
Replaces the hand-authored SVG diagram used as the project card image with the actual marketing hero image from the repo (packages/email/ public/email/images/feature-update-hero.png) -- real logo, wordmark, and tagline. Recompressed to JPEG (1.37MB -> 178KB) since it's a flat raster with no transparency need. Also drops the dangerouslyAllowSVG next.config.ts change from the earlier commit -- no longer needed now that the card image isn't an SVG going through next/image. The hand-drawn pipeline diagram stays as in-body content (rendered as a plain <img> via markdown, unaffected).
|
Swapped the card thumbnail to the real hero image from the repo ( |
1 participant
Closes #57
Summary
data/projects/12-harmonia.mdx: full case study, built on content: Harmonia case study — full-context MDX draft #57's draft but re-verified against the liveFindMalek/harmoniarepo and corrected — Next.js 16 (draft said 15), Groqgpt-oss-20bfor classification (draft saidllama-3.3-70b-versatile), the real 4-app shape (api/dashboard/web/admin— draft was missingadmin), and the actual waitlist-gated onboarding flow.web.harmonia.malek.engineering, not the API host the draft guessed), real solo-project stats (165 commits, 337 PRs, 193 issues).next.config.ts:dangerouslyAllowSVG+ a strict CSP, scoped to allow the one local, site-owned SVG asset throughnext/image(used for the project card).status: "draft"kept — the product itself is genuinely pre-launch/invite-only, not a case-study-completeness flag.Test plan
pnpm lint— no new errorspnpm build— passes, Harmonia added to static project pathsnext/image's optimizer), the case study page renders all real data (models, issue numbers, stats), and the homepage project card shows the new thumbnailSummary by CodeRabbit
New Features
Improvements