feat: EpicVideo component automation (#574)

* add admin set-videos command for product video sync

Co-authored-by: me <me@kentcdodds.com>

* clean up set-videos test global unstub handling

Co-authored-by: me <me@kentcdodds.com>

* refactor admin tests to using disposables

Co-authored-by: me <me@kentcdodds.com>

* rename using bindings to satisfy lint

Co-authored-by: me <me@kentcdodds.com>

* add dry-run mode for admin set-videos

Co-authored-by: me <me@kentcdodds.com>

* print set-videos failures when not silent

Co-authored-by: me <me@kentcdodds.com>

* enable automatic vitest mock and global cleanup

Co-authored-by: me <me@kentcdodds.com>

* expand set-videos mismatch diagnostics

Co-authored-by: me <me@kentcdodds.com>

* map set-videos by lesson slots not file count

Co-authored-by: me <me@kentcdodds.com>

* extract shared admin workshop utilities

Co-authored-by: me <me@kentcdodds.com>

* chore: format admin command files

Co-authored-by: me <me@kentcdodds.com>

* Extract formatProductLessonUrl utility

Co-authored-by: me <me@kentcdodds.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>

Author: kentcdodds

docs/launch.md

@@ -21,6 +21,14 @@ To validate launch readiness locally, run:
 
 `npx epicshop admin launch-readiness`
 
+To auto-set workshop videos from the current product lesson order, run:
+
+`npx epicshop admin set-videos`
+
+This command only inserts/updates the `EpicVideo` directly below the file title,
+and leaves any additional `EpicVideo` components in the file unchanged. Use
+`--dry-run` to preview what would change without writing files.
+
 You must also add the `product.slug` and `product.host` to the `epicshop`
 section in `package.json`:
 

packages/workshop-cli/src/cli.ts

@@ -979,7 +979,7 @@ const cli = yargs(args)
 				.positional('subcommand', {
 					describe: 'Admin subcommand',
 					type: 'string',
-					choices: ['launch-readiness'],
+					choices: ['launch-readiness', 'set-videos'],
 				})
 				.option('workshop-dir', {
 					alias: 'w',
@@ -1005,10 +1005,24 @@ const cli = yargs(args)
 						'Skip checking that EpicVideo urls return 200 to HEAD (network required)',
 					default: false,
 				})
+				.option('dry-run', {
+					type: 'boolean',
+					description:
+						'Preview set-videos changes without writing files (set-videos only)',
+					default: false,
+				})
 				.example(
 					'$0 admin launch-readiness',
 					'Check workshop launch readiness (hidden command)',
 				)
+				.example(
+					'$0 admin set-videos',
+					'Set top EpicVideo embeds from product lesson order (hidden command)',
+				)
+				.example(
+					'$0 admin set-videos --dry-run',
+					'Preview top EpicVideo changes without writing files',
+				)
 		},
 		async (
 			argv: ArgumentsCamelCase<{
@@ -1017,6 +1031,7 @@ const cli = yargs(args)
 				silent?: boolean
 				skipRemote?: boolean
 				skipHead?: boolean
+				dryRun?: boolean
 			}>,
 		) => {
 			const { findWorkshopRoot } = await import('./commands/workshops.js')
@@ -1049,6 +1064,15 @@ const cli = yargs(args)
 						if (!result.success) process.exit(1)
 						break
 					}
+					case 'set-videos': {
+						const { setVideos } = await import('./commands/admin.js')
+						const result = await setVideos({
+							silent: argv.silent,
+							dryRun: argv.dryRun,
+						})
+						if (!result.success) process.exit(1)
+						break
+					}
 					default: {
 						console.error(
 							chalk.red(`❌ Unknown admin subcommand: ${argv.subcommand}`),

packages/workshop-cli/src/commands/admin.ts

@@ -3,3 +3,8 @@ export {
 	type LaunchReadinessOptions,
 	type LaunchReadinessResult,
 } from './admin/launch-readiness.js'
+export {
+	setVideos,
+	type SetVideosOptions,
+	type SetVideosResult,
+} from './admin/set-videos.js'

packages/workshop-cli/src/commands/admin/launch-readiness.test.ts

@@ -2,7 +2,7 @@ import fs from 'node:fs/promises'
 import os from 'node:os'
 import path from 'node:path'
 
-import { afterEach, expect, test, vi } from 'vitest'
+import { expect, test, vi } from 'vitest'
 
 vi.mock('@epic-web/workshop-utils/compile-mdx.server', async () => {
 	const fs = await import('node:fs/promises')
@@ -84,56 +84,49 @@ async function createWorkshopFixture({
 		`# Step Solution\n\n<EpicVideo url="https://${productHost}/workshops/${productSlug}/step-solution" />\n`,
 	)
 
-	return root
+	return {
+		root,
+		async [Symbol.asyncDispose]() {
+			await fs.rm(root, { recursive: true, force: true })
+		},
+	}
 }
 
-afterEach(async () => {
-	vi.unstubAllGlobals()
-})
-
 test('passes with configured product + videos (skip remote)', async () => {
-	const workshopRoot = await createWorkshopFixture()
-
-	try {
-		await expect(
-			launchReadiness({
-				workshopRoot,
-				silent: true,
-				skipRemote: true,
-				skipHead: true,
-			}),
-		).resolves.toEqual(expect.objectContaining({ success: true }))
-	} finally {
-		await fs.rm(workshopRoot, { recursive: true, force: true })
-	}
+	await using workshop = await createWorkshopFixture()
+
+	await expect(
+		launchReadiness({
+			workshopRoot: workshop.root,
+			silent: true,
+			skipRemote: true,
+			skipHead: true,
+		}),
+	).resolves.toEqual(expect.objectContaining({ success: true }))
 })
 
 test('fails when epicshop.product.slug missing', async () => {
-	const workshopRoot = await createWorkshopFixture({
+	await using workshop = await createWorkshopFixture({
 		includeProductSlug: false,
 	})
 
-	try {
-		await expect(
-			launchReadiness({
-				workshopRoot,
-				silent: true,
-				skipRemote: true,
-				skipHead: true,
-			}),
-		).resolves.toEqual(expect.objectContaining({ success: false }))
-	} finally {
-		await fs.rm(workshopRoot, { recursive: true, force: true })
-	}
+	await expect(
+		launchReadiness({
+			workshopRoot: workshop.root,
+			silent: true,
+			skipRemote: true,
+			skipHead: true,
+		}),
+	).resolves.toEqual(expect.objectContaining({ success: false }))
 })
 
 test('fails when a required MDX file has no EpicVideo embed (and prints helpful path)', async () => {
-	const workshopRoot = await createWorkshopFixture()
+	await using workshop = await createWorkshopFixture()
 
 	// Remove the EpicVideo embed from the step problem README.
 	await writeFile(
 		path.join(
-			workshopRoot,
+			workshop.root,
 			'exercises',
 			'01.first-exercise',
 			'01.problem',
@@ -144,30 +137,26 @@ test('fails when a required MDX file has no EpicVideo embed (and prints helpful
 
 	const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
 
-	try {
-		const result = await launchReadiness({
-			workshopRoot,
-			silent: false,
-			skipRemote: true,
-			skipHead: true,
-		})
-
-		expect(result.success).toBe(false)
-		const output = logSpy.mock.calls.map((c) => c.join(' ')).join('\n')
-		expect(output).toContain('No <EpicVideo url="..."> embed found')
-		expect(output).toContain(
-			'exercises/01.first-exercise/01.problem/README.mdx',
-		)
-	} finally {
-		logSpy.mockRestore()
-		await fs.rm(workshopRoot, { recursive: true, force: true })
-	}
+	const result = await launchReadiness({
+		workshopRoot: workshop.root,
+		silent: false,
+		skipRemote: true,
+		skipHead: true,
+	})
+
+	expect(result.success).toBe(false)
+	const output = logSpy.mock.calls.map((c) => c.join(' ')).join('\n')
+	expect(output).toContain('No <EpicVideo url="..."> embed found')
+	expect(output).toContain('exercises/01.first-exercise/01.problem/README.mdx')
 })
 
 test('remote lesson check fails when product lesson slug not represented locally', async () => {
 	const productHost = 'www.epicweb.dev'
 	const productSlug = 'test-workshop'
-	const workshopRoot = await createWorkshopFixture({ productHost, productSlug })
+	await using workshop = await createWorkshopFixture({
+		productHost,
+		productSlug,
+	})
 
 	vi.stubGlobal(
 		'fetch',
@@ -191,35 +180,33 @@ test('remote lesson check fails when product lesson slug not represented locally
 
 	const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
 
-	try {
-		const result = await launchReadiness({
-			workshopRoot,
-			silent: false,
-			skipRemote: false,
-			skipHead: true,
-		})
-
-		expect(result.success).toBe(false)
-		const output = logSpy.mock.calls.map((c) => c.join(' ')).join('\n')
-		expect(output).toContain('Missing videos in workshop for product lessons:')
-		expect(output).toContain('missing-lesson')
-		expect(output).toContain(
-			`https://${productHost}/workshops/${productSlug}/functions-section/missing-lesson`,
-		)
-	} finally {
-		logSpy.mockRestore()
-		await fs.rm(workshopRoot, { recursive: true, force: true })
-	}
+	const result = await launchReadiness({
+		workshopRoot: workshop.root,
+		silent: false,
+		skipRemote: false,
+		skipHead: true,
+	})
+
+	expect(result.success).toBe(false)
+	const output = logSpy.mock.calls.map((c) => c.join(' ')).join('\n')
+	expect(output).toContain('Missing videos in workshop for product lessons:')
+	expect(output).toContain('missing-lesson')
+	expect(output).toContain(
+		`https://${productHost}/workshops/${productSlug}/functions-section/missing-lesson`,
+	)
 })
 
 test('warns about extra embeds only for configured workshop (includes offending url + file)', async () => {
 	const productHost = 'www.epicweb.dev'
 	const productSlug = 'test-workshop'
-	const workshopRoot = await createWorkshopFixture({ productHost, productSlug })
+	await using workshop = await createWorkshopFixture({
+		productHost,
+		productSlug,
+	})
 
 	// Add an extra embed for this workshop (should warn) and one outside /workshops (should not).
 	const exerciseIntroPath = path.join(
-		workshopRoot,
+		workshop.root,
 		'exercises',
 		'01.first-exercise',
 		'README.mdx',
@@ -255,56 +242,46 @@ test('warns about extra embeds only for configured workshop (includes offending
 
 	const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
 
-	try {
-		const result = await launchReadiness({
-			workshopRoot,
-			silent: false,
-			skipRemote: false,
-			skipHead: true,
-		})
-
-		expect(result.success).toBe(true)
-		const output = logSpy.mock.calls.map((c) => c.join(' ')).join('\n')
-		expect(output).toContain(
-			`EpicVideo embed not present in the product lesson list: https://${productHost}/workshops/${productSlug}/extra-lesson`,
-		)
-		expect(output).toContain('exercises/01.first-exercise/README.mdx')
-		expect(output).not.toContain('some-post')
-	} finally {
-		logSpy.mockRestore()
-		await fs.rm(workshopRoot, { recursive: true, force: true })
-	}
+	const result = await launchReadiness({
+		workshopRoot: workshop.root,
+		silent: false,
+		skipRemote: false,
+		skipHead: true,
+	})
+
+	expect(result.success).toBe(true)
+	const output = logSpy.mock.calls.map((c) => c.join(' ')).join('\n')
+	expect(output).toContain(
+		`EpicVideo embed not present in the product lesson list: https://${productHost}/workshops/${productSlug}/extra-lesson`,
+	)
+	expect(output).toContain('exercises/01.first-exercise/README.mdx')
+	expect(output).not.toContain('some-post')
 })
 
 test('fails when a required FINISHED.mdx is too short', async () => {
-	const workshopRoot = await createWorkshopFixture()
+	await using workshop = await createWorkshopFixture()
 
 	await writeFile(
-		path.join(workshopRoot, 'exercises', '01.first-exercise', 'FINISHED.mdx'),
+		path.join(workshop.root, 'exercises', '01.first-exercise', 'FINISHED.mdx'),
 		`Short.\n`,
 	)
 
 	const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
 
-	try {
-		const result = await launchReadiness({
-			workshopRoot,
-			silent: false,
-			skipRemote: true,
-			skipHead: true,
-		})
-		expect(result.success).toBe(false)
-		const output = logSpy.mock.calls.map((c) => c.join(' ')).join('\n')
-		expect(output).toContain('File content too short')
-		expect(output).toContain('exercises/01.first-exercise/FINISHED.mdx')
-	} finally {
-		logSpy.mockRestore()
-		await fs.rm(workshopRoot, { recursive: true, force: true })
-	}
+	const result = await launchReadiness({
+		workshopRoot: workshop.root,
+		silent: false,
+		skipRemote: true,
+		skipHead: true,
+	})
+	expect(result.success).toBe(false)
+	const output = logSpy.mock.calls.map((c) => c.join(' ')).join('\n')
+	expect(output).toContain('File content too short')
+	expect(output).toContain('exercises/01.first-exercise/FINISHED.mdx')
 })
 
 test('fails when an EpicVideo url does not return 200 to HEAD', async () => {
-	const workshopRoot = await createWorkshopFixture()
+	await using workshop = await createWorkshopFixture()
 
 	vi.stubGlobal(
 		'fetch',
@@ -317,16 +294,12 @@ test('fails when an EpicVideo url does not return 200 to HEAD', async () => {
 		}),
 	)
 
-	try {
-		await expect(
-			launchReadiness({
-				workshopRoot,
-				silent: true,
-				skipRemote: true,
-				skipHead: false,
-			}),
-		).resolves.toEqual(expect.objectContaining({ success: false }))
-	} finally {
-		await fs.rm(workshopRoot, { recursive: true, force: true })
-	}
+	await expect(
+		launchReadiness({
+			workshopRoot: workshop.root,
+			silent: true,
+			skipRemote: true,
+			skipHead: false,
+		}),
+	).resolves.toEqual(expect.objectContaining({ success: false }))
 })