Skip to content

fix: scene data passthrough + ISR write explosion (dev → main)#637

Merged
codercatdev merged 2 commits intomainfrom
dev
Mar 13, 2026
Merged

fix: scene data passthrough + ISR write explosion (dev → main)#637
codercatdev merged 2 commits intomainfrom
dev

Conversation

@codercatdev
Copy link
Contributor

Two urgent fixes

1. Video scene data passthrough (mapInputProps)

mapInputProps() was cherry-picking 6 fields per scene, dropping sceneType, code, comparison, mockup, list, infographicUrl, wordTimestamps. Now spreads full scene object. All 6 Remotion scene components will render correctly.

2. ISR write explosion fix

  • Removed generateStaticParams from 11 pages (eliminated 450+ pre-rendered ISR pages)
  • Removed export const revalidate from 17 pages (all use defineLive/sanityFetch)
  • Scoped webhook to skip pipeline doc types (automatedVideo, contentIdea, etc.)
  • ISR writes drop from ~15,000-50,000/day to ~0

Build Verification

  • ✅ TypeScript: zero errors (tsc --noEmit)
  • ✅ revalidateTag fix: restored { expire: 0 } second arg for Next.js 16
  • Node 22.22.1, pnpm 10.32.1

Two urgent fixes:\n\n1. mapInputProps() now spreads full scene object (...s) instead of cherry-picking 6 fields. This passes sceneType, code, comparison, mockup, list, infographicUrl, and wordTimestamps through to Remotion — enabling all 6 scene components (CodeMorph, ComparisonGrid, IsometricMockup, DynamicList, InfographicScene, narration).\n\n2. ISR write explosion fix: removed generateStaticParams from 11 pages, removed revalidate exports from 17 pages (all use defineLive/sanityFetch for real-time updates). Webhook now skips pipeline doc types (automatedVideo, contentIdea, sponsorLead, config singletons) that were triggering mass revalidation on every cron run.
@vercel
Copy link

vercel bot commented Mar 13, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
codingcat-dev Ignored Ignored Mar 13, 2026 2:42am

@codercatdev codercatdev merged commit a23f23e into main Mar 13, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant