fix: Seeding script improvements and Storybook configuration cleanup

- Allowed dry-run validation in seed.ts without Supabase keys
- Corrected formatting and column counts in lessons-template.csv
- Broadened Storybook story discovery pattern
- Updated seedLessons for null-safety during local validation

Author: Pavillion

.storybook/main.ts

@@ -1,10 +1,7 @@
 import type { StorybookConfig } from '@storybook/nextjs-vite'
 
 const config: StorybookConfig = {
-  stories: [
-    '../src/components/**/*.mdx',
-    '../src/components/**/*.stories.@(js|jsx|mjs|ts|tsx)',
-  ],
+  stories: ['../src/**/*.mdx', '../src/**/*.stories.@(js|jsx|mjs|ts|tsx)'],
   addons: [
     '@chromatic-com/storybook',
     '@storybook/addon-vitest',

content/lessons-template.csv

@@ -1,3 +1,3 @@
 unit_id,lesson_order,lesson_title,intro_video_url,passing_score,exercise_id,exercise_type,prompt,audio_url,options,correct_option_index,explanation,sentence_parts,correct_order,distractors,correct_answer,acceptable_answers,pairs,correct_transcript
-UNIT_ID_HERE,1,Greetings,,0.8,,multiple_choice,Choose the correct article for Hund,,Der Hund|Die Hund|Das Hund,0,Use Der for masculine nouns.,,,,,,,
-UNIT_ID_HERE,1,Greetings,,0.8,,word_bank,Build the sentence,,,,Ich|trinke|Wasser,0|1|2,heute,,,,
+UNIT_ID_HERE,1,Greetings,,0.8,,multiple_choice,Choose the correct article for Hund,,Der Hund|Die Hund|Das Hund,0,Use Der for masculine nouns,,,,,,,
+UNIT_ID_HERE,1,Greetings,,0.8,,word_bank,Build the sentence,,,,,Ich|trinke|Wasser,0|1|2,heute,,,,

scripts/seed.ts

@@ -254,15 +254,18 @@ async function main() {
     process.env.SUPABASE_URL || process.env.NEXT_PUBLIC_SUPABASE_URL
   const serviceRoleKey = process.env.SUPABASE_SERVICE_ROLE_KEY
 
-  if (!supabaseUrl || !serviceRoleKey) {
+  if (!options.dryRun && (!supabaseUrl || !serviceRoleKey)) {
     throw new Error(
-      'Missing SUPABASE_URL (or NEXT_PUBLIC_SUPABASE_URL) or SUPABASE_SERVICE_ROLE_KEY'
+      'Missing SUPABASE_URL (or NEXT_PUBLIC_SUPABASE_URL) or SUPABASE_SERVICE_ROLE_KEY. Required for actual seeding.'
     )
   }
 
-  const client = createClient(supabaseUrl, serviceRoleKey, {
-    auth: { persistSession: false },
-  })
+  const client =
+    supabaseUrl && serviceRoleKey
+      ? createClient(supabaseUrl, serviceRoleKey, {
+          auth: { persistSession: false },
+        })
+      : null
 
   const resolvedPath = path.resolve(process.cwd(), options.filePath)
   const csv = await fs.readFile(resolvedPath, 'utf8')
@@ -273,14 +276,17 @@ async function main() {
     trim: true,
   }) as CsvRow[]
 
-  if (rows.length === 0) {
-    throw new Error('CSV contains no rows')
+  if (!Array.isArray(rows) || rows.length === 0) {
+    throw new Error('CSV yielded no valid rows')
   }
 
   const units = new Map<string, Map<number, LessonSeed>>()
 
-  rows.forEach((row, index) => {
+  let index = 0
+  for (const row of rows) {
     const line = index + 2
+    index += 1
+
     const unitId = normalize(row.unit_id) || options.unitId
     if (!unitId) {
       throw new Error(`Line ${line}: missing unit_id or --unit flag`)
@@ -352,7 +358,7 @@ async function main() {
 
     const exercise = parseExercise(row, line)
     lesson.exercises.push(exercise)
-  })
+  }
 
   for (const [unitId, lessonMap] of units) {
     const items = Array.from(lessonMap.values())
@@ -368,14 +374,10 @@ async function main() {
         try {
           content = LessonContentSchema.parse(input)
         } catch (error) {
-          if (error instanceof ZodError) {
-            const details = error.issues
-              .map((issue) => `${issue.path.join('.')}: ${issue.message}`)
-              .join(' | ')
-            throw new Error(
-              `Lesson ${lesson.orderIndex} (${lesson.title}) failed validation: ${details}`
-            )
-          }
+          console.error(
+            `Validation failed for Lesson ${lesson.orderIndex} (${lesson.title}):`
+          )
+          console.error(error)
           throw error
         }
 
@@ -395,6 +397,6 @@ async function main() {
 }
 
 main().catch((error) => {
-  console.error(error instanceof Error ? error.message : error)
+  console.error(error)
   process.exitCode = 1
 })

src/features/admin/seed.ts

@@ -17,7 +17,7 @@ interface SeedParams {
  * NOTE: This should only be run by admin users or via a protected server script.
  */
 export async function seedLessons(
-  client: SupabaseClient,
+  client: SupabaseClient | null,
   { unitId, items, dryRun = false }: SeedParams
 ) {
   console.log(
@@ -53,6 +53,10 @@ export async function seedLessons(
     content: item.content as unknown as Record<string, unknown>,
   }))
 
+  if (!client) {
+    throw new Error('Database client is not initialized')
+  }
+
   const { error } = await client
     .from('lessons')
     .upsert(records, { onConflict: 'unit_id, order_index' })