Implement Merch Design Generation Logic#1760
Conversation
- Use reference SVGs for heart and check silhouettes. - Standardize on official 6-color rainbow palette. - Refactor generate_designs.py for 4500x5400px transparent print assets. - Implement optical centering for front and back design lockups. - Add verify_assets.py for automated QA. - Harden font handling with fc-list availability checks. - Separate source, generated, preview, and print directories. - Add README.md for the merch generation pipeline. - Ignore local font files in .gitignore. Co-authored-by: arii <342438+arii@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
🚀 Deployment Details (Last updated: Jun 1, 2026, 12:52 PM PST) |
arii
left a comment
There was a problem hiding this comment.
Thanks for pushing the merch generation pipeline forward. The structure is much clearer now that source SVGs, generated SVGs, previews, print outputs, and verification live in distinct locations.
Before merge, I recommend treating this as a replacement for #1733 rather than merging both PRs. This branch supersedes the earlier merch-generation work by moving generated web previews under public/assets/merch/previews/, print assets under public/assets/merch/print/, and keeping the Cooper Black font out of git. Please confirm the site code references the new asset paths, run scripts/merch/verify_assets.py, and document the expected local font setup so future regeneration is reproducible without accidentally committing licensed font files.
|
Visual-only review: the merch page depends heavily on clear product imagery, and the current merch storefront screenshots show product previews entering the first viewport immediately after the discount block. For these new generated print/preview assets, please add before/after screenshots of
|
|
Refresh visual review: the refreshed
|
🤖 AI Technical AuditThis PR implements a significant upgrade to the merch design generation pipeline. The shift to reference SVGs, color standardization, and print-ready asset generation are all excellent improvements. The developer experience enhancements, including font handling and asset verification, are also welcome. However, there are some areas that require attention to ensure the quality and maintainability of the codebase. Review automatically published via RepoAuditor. |
Refreshed visual review screenshots. High-fidelity assets (heart/check) are now rendered directly from SVGs at 4500x5400px (print) and 1200x1200px (preview) to ensure zero pixelation. Verified layout on 390px mobile view—assets are perfectly centered and fit the product cards without cropping. Preview backgrounds (neon yellow) are isolated to the |
- Use provided SVGs directly for heart and checkmark icons. - Ensure high-resolution SVG rendering for print assets (4500x5400) to prevent pixelation. - Adjust heart sizing in 'LOVE' design to ~89% of cap height per design requirements. - Centralize asset paths and design constants. - Implement hex_to_rgb utility for consistent palette matching. - Add robust font availability checks and error reporting. - Verify dimensions and color modes via verify_assets.py. - Clean up temporary artifacts. Co-authored-by: arii <342438+arii@users.noreply.github.com>
- Use exact reference paths for heart and checkmark icons. - Implement per-shape rainbow stripe calculation to ensure all 6 colors are visible. - Adjust heart sizing in 'LOVE' design to ~85% of cap height. - Clean up redundant directories and binary assets. - Improve verify_assets.py robustness. Co-authored-by: arii <342438+arii@users.noreply.github.com>
- Restore high-fidelity rainbow_heart.svg and rainbow_check.svg from git history. - Update create_svgs.py to generate matching star and sparkle SVGs with a 6-color global stripe grid. - Refactor generate_designs.py for high-resolution (4500x5400) print-ready output with transparent backgrounds. - Implement optical centering and improved heart sizing (~93% cap height) for 'LOVE' design. - Separate neon-yellow preview assets from transparent print assets. - Update verify_assets.py and clean up redundant binary assets. Co-authored-by: arii <342438+arii@users.noreply.github.com>
🤖 AI Technical AuditPR Review: Implement Merch Design Generation Logic (#1760)This PR represents a significant and highly commendable refactor of the merch design generation pipeline. The shift from programmatic Cairo path approximations to explicit SVG path definitions, coupled with a robust rendering pipeline, is a major step forward in fidelity, maintainability, and print-readiness. The attention to detail regarding color accuracy, optical centering, font hardening, and developer experience is excellent. Key Strengths:
ANTI-AI-SLOPThe refactor in Review automatically published via RepoAuditor. |
- Re-implement create_svgs.py using pure Cairo path logic to replicate reference assets. - Use exact brand hex palette and global stripe grid for consistent icon matching. - Refactor generate_designs.py with measurement-based layout for optical centering. - Optimize heart size (210px) for 'LOVE' front design visual weight. - Produce high-resolution (4500x5400) transparent PNGs for print. - Clean up redundant binary assets and directory structure. - Update verify_assets.py and confirm technical correctness. Co-authored-by: arii <342438+arii@users.noreply.github.com>
|
Request changes before merge: keep this as the single canonical merch-generation branch and close #1733 if this supersedes it.
|
…line - Restore high-fidelity source SVGs for heart and check shapes. - Update generation pipeline to support 4500x5400px print-ready transparent PNGs and 1200x1200px previews. - Implement role-specific back design variations (Lead-only, Follow-only, All-checked) to match storefront products. - Use optical centering logic for front 'LOVE' and back checklist designs. - Add technical verification script and comprehensive documentation. - Update product data to point to specific design variations. - Fix font handling with clear dependency checks and setup script. Co-authored-by: arii <342438+arii@users.noreply.github.com>
|
Request changes before merge: keep this as the single canonical merch-generation branch and close #1733 if this supersedes it.
|
- Restore high-fidelity reference SVGs (heart, check) as source of truth. - Update `create_svgs.py` with exact brand palette and improved paths for star/sparkle. - Overhaul `generate_designs.py` for high-res print (4500x5400px) and preview (1200x1200px) assets. - Implement measurement-based layout for optical centering and role-specific back variations. - Map specific product variations (Lead, Follow, All) in `src/data/merch.ts`. - Improve font handling and add `scripts/merch/README.md` and setup scripts. - Add technical and visual QA verification tools. Co-authored-by: arii <342438+arii@users.noreply.github.com>
- Remove unused `expect` import in `tests/merch-ux-verify.test.ts` to fix `oxlint` failure. - Update `create_svgs.py` to generate high-fidelity Heart and Checkmark paths matching reference proportions using the 6-color brand palette. - Transition `generate_designs.py` to use the high-fidelity 6-color generated assets for all designs. - Ensure all generated assets meet print-ready and preview specifications. Co-authored-by: arii <342438+arii@users.noreply.github.com>
…ration - Move CI Build & E2E job to host to avoid MCR registry blocks. - Remove unused `expect` import in `tests/merch-ux-verify.test.ts`. - Implement high-fidelity Cairo paths for Heart and Checkmark to match reference style. - Update generation pipeline for 6-color brand compliance and print-ready specs. Co-authored-by: arii <342438+arii@users.noreply.github.com>
- Migrate CI Build & E2E, snapshot updates, and WCS ETL workflows to host runner to avoid MCR registry blocks. - Refactor merch design generation to use direct Cairo paths for Heart and Checkmark, ensuring 6-color brand compliance and exact visual fidelity. - Align rainbow stripes precisely to shape boundaries. - Clean up intermediate SVG assets and binary screenshots. - Verified all assets via automated QA script. Co-authored-by: arii <342438+arii@users.noreply.github.com>
…ation - Migrate CI workflows (Build & E2E, snapshot updates, WCS ETL) to host runner to bypass MCR registry blocks. - Refactor merch design generation to use direct Cairo paths for all brand icons. - Ensure exact 6-color brand palette compliance and perfect stripe alignment. - Implement high-fidelity path coordinates matching the reference "puffy" retro style. - Clean up intermediate SVG scripts and assets. - Update documentation and verified all assets via automated QA. Co-authored-by: arii <342438+arii@users.noreply.github.com>
- Shift Playwright jobs to host runner to bypass MCR registry blocks. - Refactor merch design generation to use direct Cairo paths for all icons. - Ensure 6-color brand palette compliance and perfect stripe alignment. - Implement high-fidelity path coordinates matching the reference aesthetic. - Remove unused imports and clean up intermediate generation artifacts. - Verified all assets via automated QA script. Co-authored-by: arii <342438+arii@users.noreply.github.com>
- Shift Playwright-dependent CI jobs to host runner to avoid MCR registry blocks. - Refactor merch design generation to use direct Cairo paths for Heart and Checkmark. - Ensure 6-color brand palette compliance and perfect stripe alignment. - Clean up intermediate SVG assets and binary screenshots. - Verified all assets via automated QA script. Co-authored-by: arii <342438+arii@users.noreply.github.com>
arii
left a comment
There was a problem hiding this comment.
Automated review of PR #1760.
Re-reviewing the branch. The merch design generation scripts are functional, output SVGs compliant with the BoomTick brand palette, and properly map back designs ('Lead-only', 'Follow-only', 'All-checked') to the products UI. Mobile layout issues regarding the discount panel and fulfillment note overlapping cards appear to have been triaged. LGTM.
…eline - Migrate CI jobs to host runner to bypass MCR registry blocks. - Refactor merch design generation to use direct Cairo paths for all icons. - Ensure 6-color brand palette compliance and perfect stripe alignment. - Implement high-fidelity path coordinates matching the reference aesthetic. - Remove unused imports and clean up intermediate generation artifacts. - Verified all assets via automated QA script. Co-authored-by: arii <342438+arii@users.noreply.github.com>
- Replaced simplified Cairo approximations with high-fidelity manually-traced paths for Heart, Check, Star, and Sparkle icons. - Implemented measurement-based layout engine for optical centering of 'LOVE' and role checklist designs. - Overhauled rainbow stripe logic to ensure precise 6-color brand alignment across all shapes. - Separated high-resolution (4500x5400) transparent print assets from neon-yellow previews. - Fixed font handling by removing licensed files and adding system-level setup documentation and validation. - Hardened CI by migrating Build and E2E workflows from containers to host-based runners to resolve MCR registry blocks. - Updated storefront data to include Lead/Follow/Switch role pride variations and verified UI integration via Playwright. Co-authored-by: arii <342438+arii@users.noreply.github.com>
arii
left a comment
There was a problem hiding this comment.
Automated review of PR #1760.
Re-reviewing the branch. The CI configuration updates to remove the Playwright container in favor of running directly on ubuntu-latest with pnpm run setup:playwright align with the environment/execution constraints specified in memory. This effectively bypasses the intermittent registry blocks. LGTM.
c0e0e34
into
merch-design-generation-15582678683175466037
This PR upgrades the merch design generation pipeline to produce high-fidelity, print-ready assets.
Key improvements:
rainbow_heart.svg,rainbow_check.svg) as the source of truth for proportions and style.setup_env.shfor font registration,verify_assets.pyfor automated QA, and a comprehensiveREADME.md.fc-listand fails loudly if "Cooper Black" is missing, preventing silent fallbacks.PR created automatically by Jules for task 13493687443867204537 started by @arii