Add cherry blossoms experiment report (CC vs Codex)#1039
Add cherry blossoms experiment report (CC vs Codex)#1039sonichi wants to merge 1 commit intomassgen:mainfrom
Conversation
Side-by-side comparison of Claude Code (Sonnet 4.6) vs Codex (GPT-5.4) generating a 1200x600 cherry blossom SVG illustration. CC won 2-0 — both agents voted for CC after Codex's SVG had unresolved template placeholders in blossom elements. Includes: coordination timeline, vote reasoning, stats ($4.95 cost, 3.8M input tokens, 6 rounds). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
📝 WalkthroughWalkthroughA new static HTML documentation page was added to report on the "MassGen Experiment: Cherry Blossoms SVG," featuring experiment details, agent comparison cards, SVG visualizations, coordination timeline, vote results, and statistics with inline CSS styling. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/presentation/experiments/cherry-blossoms/index.html`:
- Around line 107-112: Replace the unsafe <object> embeds for generated SVG
previews with inert <img> tags: locate the <object data="cc-initial.svg"
type="image/svg+xml"> and <object data="codex-initial.svg" type="image/svg+xml">
(and the other occurrence at the later SVG embed) and change them to <img
src="..."> elements preserving the surrounding container classes/attributes and
adding meaningful alt text (e.g., "CC SVG preview", "Codex SVG preview") so the
SVG is rendered as a static image with no active script surface.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 367ca19a-af62-4fc8-a7e7-00299e784481
⛔ Files ignored due to path filters (3)
docs/presentation/experiments/cherry-blossoms/cc-initial.svgis excluded by!**/*.svg,!**/*.svgdocs/presentation/experiments/cherry-blossoms/codex-initial.svgis excluded by!**/*.svg,!**/*.svgdocs/presentation/experiments/cherry-blossoms/final.svgis excluded by!**/*.svg,!**/*.svg
📒 Files selected for processing (1)
docs/presentation/experiments/cherry-blossoms/index.html
| <object data="cc-initial.svg" type="image/svg+xml">CC SVG</object> | ||
| </div> | ||
| <div class="svg-panel"> | ||
| <h3 class="agent-name codex" style="padding-left:1rem">Codex</h3> | ||
| <object data="codex-initial.svg" type="image/svg+xml">Codex SVG</object> | ||
| </div> |
There was a problem hiding this comment.
Use <img> instead of <object> for generated SVG previews.
Line 107, Line 111, and Line 184 embed model-generated SVG using <object>, which can expose active SVG content risks. For static report rendering, use <img> to reduce scriptable surface.
Suggested hardening diff
- <object data="cc-initial.svg" type="image/svg+xml">CC SVG</object>
+ <img src="cc-initial.svg" alt="Claude Code initial cherry blossom SVG output" loading="lazy">
- <object data="codex-initial.svg" type="image/svg+xml">Codex SVG</object>
+ <img src="codex-initial.svg" alt="Codex initial cherry blossom SVG output" loading="lazy">
- <object data="final.svg" type="image/svg+xml" style="width:100%; height:400px; background:white;">Final SVG</object>
+ <img src="final.svg" alt="Final winning cherry blossom SVG from Claude Code" style="width:100%; height:400px; background:white;" loading="lazy">Also applies to: 184-184
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/presentation/experiments/cherry-blossoms/index.html` around lines 107 -
112, Replace the unsafe <object> embeds for generated SVG previews with inert
<img> tags: locate the <object data="cc-initial.svg" type="image/svg+xml"> and
<object data="codex-initial.svg" type="image/svg+xml"> (and the other occurrence
at the later SVG embed) and change them to <img src="..."> elements preserving
the surrounding container classes/attributes and adding meaningful alt text
(e.g., "CC SVG preview", "Codex SVG preview") so the SVG is rendered as a static
image with no active script surface.
Summary
docs/presentation/experiments/cherry-blossoms/Files
index.html— experiment report page (dark theme, inline CSS)cc-initial.svg— Claude Code's initial outputcodex-initial.svg— Codex's initial outputfinal.svg— winning output (CC)Stats
cwdper agent, no--cwd-context rwLive preview
Also hosted at: https://sutando.ai/massgen-demo/
🤖 Generated with Claude Code
Summary by CodeRabbit
Release Notes