feat(marketing): workflow marketplace with ReactFlow previews and SEO pages#3542
feat(marketing): workflow marketplace with ReactFlow previews and SEO pages#3542georgi wants to merge 2 commits into
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
nodetool-marketing | 8bdc0f5 | Commit Preview URL Branch Preview URL |
Jun 22 2026, 09:26 AM |
georgi
left a comment
georgi
left a comment
There was a problem hiding this comment.
CI checks passed and the scope is clean (marketing-only, 65 files). However, the base is ~185 commits behind main.
Please rebase onto current main to pick up recent changes. Once rebased and CI stays green, this is ready to merge.
Generated by Claude Code
|
CI Quality Gate is failing and this PR is 5 days stale. The workflow marketplace feature is large (65 files, +1938/-553). Needs a rebase onto current main, CI fix, and the manual test checklist items verified before merging. Generated by Claude Code |
georgi
left a comment
georgi
left a comment
There was a problem hiding this comment.
Clean and self-contained in marketing/ — zero risk to the core platform. SEO sitemap integration is a nice touch. Has merge conflicts from recent merges to main — please rebase and this is ready to merge.
Generated by Claude Code
georgi
left a comment
georgi
left a comment
There was a problem hiding this comment.
The marketplace feature and color migration are solid, but two issues before merge:
- Placeholder video IDs — All demo videos point to
dQw4w9WgXcQ(Rick Astley rickroll). These need real demo videos or removal before shipping to production. - Bundle size —
import * as Icons from "lucide-react"inWorkflowPreviewFlow.tsxbundles the entire icon library (~200KB). Use selective imports or a tree-shaking-safe dynamic lookup instead.
Minor: all ogImage values are /preview.png — consider workflow-specific OG images for better social sharing.
Generated by Claude Code
georgi
left a comment
georgi
left a comment
There was a problem hiding this comment.
CI checks passed but the merge state is stale. This is a large marketing feature (65 files). Please rebase on latest main to refresh the mergeability check, then this needs a feature review of the marketplace pages.
Generated by Claude Code
georgi
left a comment
georgi
left a comment
There was a problem hiding this comment.
Good feature work overall — clean component decomposition, solid SEO (JSON-LD, OG tags, sitemap), correct ReactFlow usage with SSR guard. A few things need fixing before merge:
Must fix:
- Placeholder videos — All 7 workflows use
id: "dQw4w9WgXcQ"(Rick Astley). Replace with real demo videos or remove thevideofield entirely and let the "Demo video coming soon" fallback render. This should not ship to production. import * as Icons from "lucide-react"— This barrel-imports ~1000 icons into the client bundle. Since only ~15 icons are actually used (driven by the data catalog), create a curated map of just those icons.
Needs answer:
3. proOptions: { hideAttribution: true } — This requires a ReactFlow Pro license. Does the project have one? If not, remove it.
Suggestions (non-blocking):
4. The bulk slate → neutral Tailwind color rename across ~40 files inflates the diff and buries the actual feature changes. Would be cleaner as a separate PR.
5. No tests added. At minimum, the filter logic and getWorkflowBySlug should have unit tests.
6. The dangerouslySetInnerHTML for JSON-LD is safe today (static data), but consider a helper that escapes </script> sequences if this ever pulls from a CMS.
Generated by Claude Code
|
Multiple CI failures and merge conflicts need addressing:
Scoped to Generated by Claude Code |
georgi
left a comment
georgi
left a comment
There was a problem hiding this comment.
Large PR (65 files, 1,938 additions) for the marketing workflow marketplace. The Quality Gate never ran, so CI status is unknown. Needs a rebase onto current main, a full CI run, and manual testing of the marketplace pages before merge.
Generated by Claude Code
georgi
left a comment
georgi
left a comment
There was a problem hiding this comment.
This PR is 15 days old with 65 files changed and no recent code updates. CI has no Quality Gate run and likely has merge conflicts against current main. Needs a rebase and CI run before it can be considered for merge.
Generated by Claude Code
Review: Approved, but needs rebaseThe core marketplace feature (workflow pages, components, data) looks good. However the branch has extensive conflicts with current main — particularly the "styling" commit which touches 30+ marketing files that have diverged significantly. The marketplace feature commit itself (workflow gallery, detail pages, ReactFlow previews, sitemap integration) is clean and well-scoped. Action needed: Rebase onto current main. The first commit cherry-picks cleanly with only 2 minor conflicts (page.tsx navigation array + sitemap.ts). The "styling" commit has 30+ conflicts across the marketing site and will need manual resolution by the author. Generated by Claude Code |
… pages Each /workflows/[slug] page ships a fancy ReactFlow preview (custom marketplace node with icon/hue/badge, animated edges), a YouTube demo video with click-to-play poster, HowTo + BreadcrumbList JSON-LD, OpenGraph + Twitter cards, related workflows, and a share bar — built to win on SEO, GEO (Perplexity/Google AI Overviews), and social previews. Six initial workflows: brand asset pack, video-to-shorts, AI product photographer, multilingual voice-over, RAG support bot, music video generator. Index page has category filters and full-text search. All pages prerendered as static HTML and added to sitemap.xml.
georgi
force-pushed
the
claude/nice-fermi-IqSxI
branch
from
fa5fafd to
8bdc0f5
Compare
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
| @@ -1,26 +1,108 @@ | |||
| "use client"; | |||
| import React, { useEffect, useState } from "react"; | |||
| import React, { useEffect, useState, useRef } from "react"; | |||
georgi
left a comment
georgi
left a comment
There was a problem hiding this comment.
Marketing-specific CI (typecheck, lint, build, Playwright, Lighthouse) all pass — looks good overall. One minor issue: unused useRef import in marketing/src/app/creatives/page.tsx. Please fix that and this is ready to merge.
Generated by Claude Code
georgi
left a comment
georgi
left a comment
There was a problem hiding this comment.
This has been open for 19 days and the base SHA is significantly behind main. The marketing-only scope limits blast radius, but it likely has merge conflicts at this point.
Before merging:
- Rebase onto current main and resolve any conflicts
- Verify
cd marketing && npm run buildstill succeeds after rebase - Spot-check that the ReactFlow preview components render (the workflow data/types are hardcoded, so they should survive the rebase)
The code itself looks reasonable — scoped entirely to marketing/, no backend changes. Happy to merge once rebased and build-verified.
Generated by Claude Code
georgi
left a comment
georgi
left a comment
There was a problem hiding this comment.
This PR has been open since June 6 and CI is red (quality gate failure). The branch is significantly behind main. Needs a rebase onto current main and CI must pass before this can be merged.
Generated by Claude Code
2 participants
Summary
Adds a public workflow marketplace to the marketing site — a browsable gallery of NodeTool workflows with live ReactFlow graph previews and per-workflow SEO landing pages.
What's included
/workflowsindex (app/workflows/page.tsx,layout.tsx) — gallery with filtering/workflows/[slug]detail pages (app/workflows/[slug]/page.tsx) — per-workflow SEO pages, wired intositemap.tscomponents/workflows/):WorkflowPreviewFlow— ReactFlow-based graph previewWorkflowCard,WorkflowFilters,WorkflowShareBar,WorkflowVideo,WorkflowFlowClientlib/workflows/data.ts,types.ts) — workflow catalog and shared typesStats
65 files changed, +1938 / −553. New surface is scoped entirely to
marketing/.Test plan
cd marketing && npm run buildsucceeds/workflowsrenders the gallery and filters work/workflows/[slug]renders ReactFlow preview + share bar🤖 Generated with Claude Code