ragTechDev · natashaannn · May 13, 2026 · May 11, 2026 · May 11, 2026 · May 13, 2026
diff --git a/.babelrc b/.babelrc
@@ -1,9 +1,14 @@
 {
-  "presets": ["next/babel"],
   "env": {
+    "development": {
+      "presets": ["next/babel"]
+    },
+    "production": {
+      "presets": ["next/babel"]
+    },
     "test": {
       "presets": [
-        ["@babel/preset-env", { "targets": { "node": "current" } }],
+        ["@babel/preset-env", { "targets": { "node": "current" }, "modules": "commonjs" }],
         ["@babel/preset-react", { "runtime": "automatic" }],
         "@babel/preset-typescript"
       ],

diff --git a/.claude/skills/pre-push-audit/SKILL.md b/.claude/skills/pre-push-audit/SKILL.md
@@ -511,7 +511,6 @@ Print the full PR body so the agent or developer can copy it:
 ## Test plan
 <checklist from 8c>
 
-🤖 Generated with [Claude Code](https://claude.com/claude-code)
 ------- (end of template) -------
 ```
 

diff --git a/.eslintrc b/.eslintrc
@@ -2,7 +2,7 @@
   "plugins": ["@remotion"],
   "overrides": [
     {
-      "files": ["remotion/*.{ts,tsx}"],
+      "files": ["remotion/**/*.{ts,tsx}"],
       "extends": ["plugin:@remotion/recommended"]
     }
   ]

diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
@@ -0,0 +1,34 @@
+## Summary
+
+<!-- 2–4 bullets: what changed and why. Focus on effect, not just what the commit messages say. -->
+
+-
+-
+
+## How to review
+
+<!-- Key files changed and what a reviewer should focus on in each. -->
+
+| File | What to check |
+|------|---------------|
+|  |  |
+
+## Test plan
+
+- [ ] `npm test` passes
+- [ ] `npm run test:react` passes (if React/Remotion components changed)
+- [ ] `npm run test:e2e` passes (if Phase 8 user-facing flows changed)
+
+<!-- If Remotion components changed, add manual checks below: -->
+
+Manual verification:
+- [ ] <!-- [REMOTION-VISUAL] Remotion Studio: describe what to scrub and verify -->
+- [ ] <!-- [UI-BROWSER] npm run dev: describe route and interaction to test -->
+
+## Checklist
+
+- [ ] Behaviour parity — relevant `npm run` scripts or Remotion compositions smoke-tested
+- [ ] `tsc --noEmit` passes
+- [ ] No new hardcoded paths in `scripts/`; no new duplicated timing constants in `remotion/`
+- [ ] Type shapes match spec in `docs/PRODUCTION_REFACTOR_PLAN.md` (if types changed)
+- [ ] Scope discipline — only files listed in the implementation doc were touched
diff --git a/.gitignore b/.gitignore
@@ -57,9 +57,11 @@ public/transcribe/output/ # raw Whisper JSON + VTT
 public/thumbnail/         # candidate frames, cutouts, manifest
 public/renders/           # final rendered .mp4 files
 public/output/            # carousel exports
+public/shorts/
+public/sync/output
 
 # ── Editable pipeline outputs (text/JSON) ─────────────────────────────────────
-# public/edit/   → transcript.doc.txt, transcript.json, SRT exports
+public/edit/   → transcript.doc.txt, transcript.json, SRT exports
 public/camera/ → camera-profiles.json, frame snapshots, detections
 #
 # These are committed by default so collaborators (and coding agents) can read

diff --git a/docs/PRODUCTION_REFACTOR_PLAN.md b/docs/PRODUCTION_REFACTOR_PLAN.md
@@ -320,6 +320,7 @@ export type Brand = {
   audio: {
     introOutroMusic: string;
     backgroundMusic: string;
+    hookMusic?: string;
   };
   background: {
     episodeGridAssets: string[];

diff --git a/docs/review-findings/2026-05-13-refactor-s1-brand-loader.md b/docs/review-findings/2026-05-13-refactor-s1-brand-loader.md
@@ -0,0 +1,64 @@
+# Review: refactor/s1-brand-loader
+Date: 2026-05-13
+Reviewer: AI (review-pr skill) — session bias: CLEAN
+PR: NONE (description provided inline)
+
+## Verdict
+CHANGES REQUESTED
+
+## Summary
+This PR adds `brand.audio.hookMusic` resolution to `calculateMetadata` / `calculateShortMetadata`, replaces the `@remotion/sfx` CDN whoosh with a local file, and fixes the Babel/Jest test environment. The core logic is sound and tests pass cleanly. Two issues block merge: the `hookMusic?` field was added to `Brand.audio` without updating the canonical spec, and the new `brand.json` references a non-existent audio file (`background-music.mp3`).
+
+## Blockers (must fix before merge)
+
+### B1 — `hookMusic?` in Brand type diverges from spec
+- **Type:** QUALITY
+- **File:** `remotion/types/brand.ts` line 77 + `docs/PRODUCTION_REFACTOR_PLAN.md`
+- **Finding:** `docs/PRODUCTION_REFACTOR_PLAN.md` defines `Brand.audio` as `{ introOutroMusic: string; backgroundMusic: string; }` with no `hookMusic` field. This PR adds `hookMusic?: string` without updating the spec. Per the CLAUDE.md convention: "If the spec must change, update it first and get agreement before diverging in code. Downstream phase steps depend on specific field names by reference."
+- **Fix:** Add `hookMusic?: string` to the `audio` block in the extended Brand type in `docs/PRODUCTION_REFACTOR_PLAN.md` Phase 0.5. The spec update should precede or accompany the implementation change.
+
+### B2 — `brand.json` references non-existent audio file
+- **Type:** QUALITY
+- **File:** `public/brands/ragtech/brand.json` line 67
+- **Finding:** `"backgroundMusic": "/sounds/background-music.mp3"` points to a file that does not exist in `public/sounds/`. The directory contains `intro-outro-music.mp3`, `jazz-cafe-music.mp3`, and `whoosh.wav` — no `background-music.mp3`. While no component currently reads `brand.backgroundMusic`, this brand.json is being introduced as the canonical config and the broken path will silently fail when Phase 0.5 wires up the audio fields.
+- **Fix:** Change to `"/sounds/jazz-cafe-music.mp3"` (the existing background music track per CLAUDE.md), or add the actual file and document it.
+
+## Warnings (should address)
+
+### W1 — Vacuous `eslint-disable-next-line @typescript-eslint/no-explicit-any`
+- **Type:** QUALITY
+- **File:** `tests/setup.react.ts` line 25
+- **Finding:** `@typescript-eslint/no-explicit-any` is not present in the project's ESLint config (`eslint-config-next` does not enable it; confirmed by grepping `node_modules/eslint-config-next`). The disable comment suppresses nothing and adds noise. This matches the known "eslint-disable comments added for rules not active in the project ESLint config" pattern.
+- **Suggestion:** Remove the comment. If explicit-any is a concern, type the factory as `React.ComponentPropsWithRef<'img'>`.
+
+### W2 — Saloni missing from `hosts` array in `brand.json`
+- **Type:** QUALITY
+- **File:** `public/brands/ragtech/brand.json` (hosts array)
+- **Finding:** CLAUDE.md lists three cohosts: Natasha, Saloni, Victoria. The new `brand.json` hosts array only contains Natasha and Victoria. No component consumes `brand.hosts` yet (confirmed by grep), but this file is being introduced as the canonical brand source of truth; Saloni will be absent when Phase 0.5 consumes it.
+- **Suggestion:** Add Saloni's entry: `{ "name": "Saloni", "role": "Software Developer", "imgSrc": "/assets/team/saloni.PNG", "nameBgColor": "<colour>" }`.
+
+### W3 — Team image paths use lowercase `.png` vs. actual `.PNG` filenames
+- **Type:** QUALITY
+- **File:** `public/brands/ragtech/brand.json` (hosts[*].imgSrc)
+- **Finding:** Actual files on disk are `natasha.PNG`, `saloni.PNG`, `victoria.PNG` (uppercase extension). The brand.json uses `/assets/team/natasha.png` (lowercase). This works on macOS (case-insensitive) but will break on Linux (Docker, CI).
+- **Suggestion:** Align `imgSrc` values to use `.PNG`, or rename the asset files to `.png` for portability.
+
+## Suggestions (optional improvements)
+
+- `remotion/Composition.tsx` and `remotion/ShortFormClip.tsx` both define `const normalizeStaticPath = ...` identically. This duplication predates this PR but the brand-fetch block added here uses it in both files. Consider extracting to `remotion/lib/utils.ts` in the Phase 5/6 cleanup.
+
+## Test plan verification
+
+| Item | Status | Notes |
+|------|--------|-------|
+| `npm test` passes | PASS | 264 passed, 2 skipped |
+| `npm run test:react` passes | PASS | 20 passed, 5 suites |
+| `tsc --noEmit` clean | PASS | No errors |
+| `npm run test:e2e` | SKIPPED | No Phase 8 browser flows changed |
+| [REMOTION-VISUAL] ShortFormClip hook music from brand | NOT RUN | Developer self-attested ✓ |
+| [REMOTION-VISUAL] Transition whoosh from local file | NOT RUN | Developer self-attested ✓ |
+| [REMOTION-VISUAL] ragTechVodcast hook section renders | NOT RUN | Developer self-attested ✓ |
+
+## Patterns observed
+- B1 matches known pattern: **Implementation diverges from documented spec without updating the spec**
+- W1 matches known pattern: **eslint-disable comments added for rules not active in the project ESLint config**
diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -5,6 +5,12 @@ import nextTs from "eslint-config-next/typescript";
 const eslintConfig = defineConfig([
   ...nextVitals,
   ...nextTs,
+  {
+    files: ["remotion/**/*.{ts,tsx}"],
+    rules: {
+      "@next/next/no-img-element": "off",
+    },
+  },
   // Override default ignores of eslint-config-next.
   globalIgnores([
     // Default ignores of eslint-config-next: