Skip to content

chore(demo): refresh caro-demo video — drift in src/safety/patterns.rs, website/src/ui/tokens.css - #1429

Open
wildcard wants to merge 2 commits into
mainfrom
chore/caro-demo-refresh-2026-09
Open

wildcard wants to merge 2 commits into
mainfrom
chore/caro-demo-refresh-2026-09

Conversation

@wildcard

@wildcard wildcard commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Description

Monthly maintenance re-render of website/public/caro-demo.mp4 and its poster, triggered by the automated drift check on 2026-09-01. The video bakes in facts about the product (pattern count, brand colors, CLI output strings, install commands) that have drifted since the last render (commit 4c45fc56, 2026-04-26).

This PR requires human review — drift is semantic (pattern count claim changed) and cosmetic (brand palette changed). Do not auto-merge.

Motivation

The demo video on the landing page was claiming "52+ patterns" while src/safety/patterns.rs now defines 67 patterns. The brand accent color has also changed from orange (#ff8c42) to signal-red (#ef3333), making the video look off-brand.

Changes Made

  • demos/remotion-video/src/tokens.ts: updated accent/accentDark/accentSoft to signal-red palette matching current website/src/ui/tokens.css
  • demos/remotion-video/src/scenes/SceneSafety.tsx: updated Scene 3 caption from "52+ patterns" to "67+ patterns"
  • website/public/caro-demo.mp4: re-rendered from updated Remotion source (1.8 MB)
  • website/public/caro-demo-poster.png: re-rendered poster (frame 60, 90 KB)
  • demos/remotion-video/.baseline-manifest.json: refreshed tripwire SHAs and semantic claims anchored to commit be07b22

Drift Report

caro-demo drift check
  last rendered: 2026-04-26T02:07:00Z (commit 4c45fc56)
  total watched: 7  ok: 3  drifted: 4  missing: 0

  ⚠ DRIFTED: website/src/ui/tokens.css
  ✓ website/src/components/landing/LPDemo.astro
  ⚠ DRIFTED: src/main.rs
  ✓ .claude/beta-testing/test-cases.yaml
  ⚠ DRIFTED: src/safety/patterns.rs
  ⚠ DRIFTED: homebrew-tap/README.md
  ✓ install.sh

Side-by-side: what changed in each drifted file

File Drift type Old (baked into video) New (current source)
src/safety/patterns.rs Semantic 52 patterns → caption: "52+ patterns. Blocked before damage." 67 patterns → caption updated to "67+ patterns. Blocked before damage."
website/src/ui/tokens.css Cosmetic --accent: #ff8c42 (orange) --accent: #ef3333 (signal red, --caro-red-500)
src/main.rs Output-equivalent "✗ command blocked by safety validator (Critical)" Same runtime output — file changed (other code) but the CLI string in Scene 3 is unchanged
homebrew-tap/README.md Output-equivalent Version string updated to 1.5.0 Scene 4 shows cargo install caro (version-agnostic) — no visible change in the video

Type of Change

  • CI/CD or tooling (changes to build, release, or development tools)

Checklist

Code Quality

  • No Rust code changed — this is a Remotion/video asset update

Testing

  • Video renders cleanly (900 frames, 1.8 MB output)
  • Poster renders at frame 60
  • Baseline manifest refreshed with updated SHA tripwires

Documentation

  • Baseline manifest updated with new claims_baked_into_video values

Additional Context

Technical Decisions

  • Used pre-installed chrome-headless-shell from PLAYWRIGHT_BROWSERS_PATH to avoid downloading a second Chromium in the CI container.
  • Scene 2 queries and Scene 3 block message are unchanged — test-cases.yaml and the main.rs CLI string are stable.

Routine link

This PR was opened by the monthly caro-demo-video maintenance routine (scheduled task, 2026-09-01).


Reviewer Checklist

  • Scene 3 caption reads "67+ patterns" (correct, matches grep -c 'description:' src/safety/patterns.rs)
  • Accent color in the video reads as signal-red (not orange)
  • Scene 2 queries and outputs are still accurate
  • Scene 4 install command still reads cargo install caro
  • File size ≤ 6 MB (current: 1.8 MB ✓)

Generated by Claude Code


Summary by cubic

Re-renders the caro-demo landing video so it stops showing stale product facts: the Scene 3 caption now reads "67+ patterns" instead of "52+ patterns", and the accent color shifts from orange to signal-red to match current brand tokens. This is a demo asset refresh only; no runtime code changes.

  • Updates the Remotion source (demos/remotion-video/src/tokens.ts and SceneSafety.tsx) before re-rendering.
  • Replaces the video and poster under website/public/ with fresh 900-frame renders.
  • Refreshes .baseline-manifest.json with new SHA tripwires and claim values anchored to commit be07b22, and corrects the claim-text and watched-file purpose fields that the render script left at "52+ patterns".
  • Updates the token table in .claude/skills/caro-demo-video/SKILL.md, which still listed the old orange palette, to the current signal-red values.
  • Scene 2 queries, CLI block message, and install command are unchanged.

Written for commit 32c459a. Summary will update on new commits.

Review in cubic

…s.css

Semantic drift (pattern count: 52 → 67) and cosmetic drift (brand accent
color: #ff8c42 orange → #ef3333 signal-red). Updated Remotion source to
match reality, re-rendered the 30s video and poster.

Source changes:
- demos/remotion-video/src/tokens.ts: accent/accentDark/accentSoft updated
  to signal-red palette per website/src/ui/tokens.css
- demos/remotion-video/src/scenes/SceneSafety.tsx: caption updated from
  "52+ patterns" to "67+ patterns" per current src/safety/patterns.rs count

Generated by monthly maintenance routine.
@vercel

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
caro-foss-website Ready Ready Preview Sep 1, 2026 12:17pm UTC
5 Skipped Deployments
Project Deployment Actions Updated
caro-docs Ignored Ignored Sep 1, 2026 12:17pm UTC
caro-slides Ignored Ignored Sep 1, 2026 12:17pm UTC
caro-storybook Ignored Ignored Sep 1, 2026 12:17pm UTC
cmdai Ignored Ignored Sep 1, 2026 12:17pm UTC
cmdai-saas Ignored Ignored Sep 1, 2026 12:17pm UTC

Request Review

@greptile-apps greptile-apps Bot 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.

Your trial has ended. Reactivate Greptile to resume code reviews.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Nice work! 🚀

@github-actions github-actions Bot added website Related to website size/S Small PR (10-50 lines) labels Sep 1, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-01T12:15:34.539521Z 227cc31 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 227cc31b2a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread demos/remotion-video/.baseline-manifest.json Outdated
Comment thread demos/remotion-video/src/tokens.ts
- .baseline-manifest.json: pattern_count_claim_text and watched-file
  purpose were still "52+ patterns" after render-and-ship.sh updated
  only the numeric field; corrected to "67+ patterns" so future drift
  checks compare against the actual baked-in caption text.
- .claude/skills/caro-demo-video/SKILL.md: visual token table still
  showed the old orange palette (#ff8c42 / #ff6b35); updated to the
  current signal-red values (#ef3333 / #e63636) so the next maintenance
  agent mirrors the correct colors into tokens.ts.

Addresses Codex P2 findings on PR #1429.

wildcard commented Sep 1, 2026

Copy link
Copy Markdown
Owner Author

Security Audit — pre-existing failure, not caused by this PR

The Security Audit check failed with "2 vulnerabilities found" in cxx (RUSTSEC-2026-0190, RUSTSEC-2026-0202) plus 11 unsoundness/yanked warnings in event-listener, lru, memmap2, scc, and others. All advisory dates range from May–July 2026.

This PR touches only demos/remotion-video/src/ TypeScript files, demos/remotion-video/.baseline-manifest.json, .claude/skills/caro-demo-video/SKILL.md, and the rendered website/public/caro-demo.{mp4,png} — no Rust source files and no Cargo.toml/Cargo.lock changes. The vulnerability in Cargo.lock predates this branch entirely.

Fixing these advisories requires a dependency-update PR targeting the Rust workspace — out of scope here. This PR is safe to merge once the video content is approved.


Generated by Claude Code

@cubic-dev-ai cubic-dev-ai Bot 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.

Review completed against the latest diff

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread demos/remotion-video/.baseline-manifest.json
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Website Claims Verification Report

Run ID: 33506780471
SHA: a119621
Date: 2026-09-01

Platform Results

claims-report-macos-latest

{
  "suite": "website-claims",
  "platform": "macos-latest",
  "timestamp": "2026-09-01T12:31:22Z",
  "passed": 84,
  "skipped": 0,
  "warnings": 1,
  "run_id": "33506780471",
  "sha": "a11962146c4886b073e2bac4b9a5065abbe09d9b"
}

claims-report-ubuntu-latest

{
  "suite": "website-claims",
  "platform": "ubuntu-latest",
  "timestamp": "2026-09-01T12:20:18Z",
  "passed": 84,
  "skipped": 0,
  "warnings": 1,
  "run_id": "33506780471",
  "sha": "a11962146c4886b073e2bac4b9a5065abbe09d9b"
}

Documentation

Next Steps

  1. Review warnings and address any gaps
  2. Update website if claims are inaccurate
  3. Implement missing features if claims are aspirational

@wildcard wildcard added the path:auto caro-merge-review-integrate dispatcher classification label Sep 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation path:auto caro-merge-review-integrate dispatcher classification size/S Small PR (10-50 lines) website Related to website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants