Preserve code blocks during validation, apply line-breaking after

- Defer line-breaking until after validation passes in generate-presentation.js
- Add preserveCode parameter to parseMarkdownContent() in markdown-parser.js
- Code blocks remain intact for validation, preventing false positives
- Line-breaking applied only to validated presentations before deployment

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: ofriw

scripts/generate-presentation.js

@@ -722,21 +722,11 @@ async function generatePresentationWithClaude(prompt, outputPath) {
 
         console.log(`  ✅ Valid presentation JSON (${presentation.slides.length} slides)`);
 
-        // Apply deterministic line breaking
-        console.log('  🔧 Applying line breaking...');
-        const { presentation: processedPresentation, stats } = processPresentation(presentation);
-
-        // Log statistics
-        if (stats.linesShortened > 0) {
-          console.log(`  ✂️  Fixed ${stats.linesShortened} long lines (max reduction: ${stats.maxReduction} chars)`);
-        } else {
-          console.log('  ✅ No lines exceeded 60 characters');
-        }
-
-        // Write back the processed presentation
-        writeFileSync(outputPath, JSON.stringify(processedPresentation, null, 2), 'utf-8');
+        // Write the unmodified presentation to file for validation
+        // Line breaking will happen after validation passes
+        writeFileSync(outputPath, JSON.stringify(presentation, null, 2), 'utf-8');
 
-        resolve(processedPresentation);
+        resolve(presentation);
       } catch (parseError) {
         reject(new Error(`Failed to parse JSON: ${parseError.message}\nContent preview: ${fileContent?.slice(0, 200)}`));
         return;
@@ -1181,8 +1171,8 @@ async function generatePresentation(filePath, manifest, config) {
   console.log(`\n📄 Generating presentation: ${relativePath}`);
 
   try {
-    // Parse content
-    const content = parseMarkdownContent(filePath);
+    // Parse content (preserve code blocks for visual presentation)
+    const content = parseMarkdownContent(filePath, true);
 
     if (content.length < 100) {
       console.log(`  ⚠️  Skipping - content too short`);
@@ -1297,24 +1287,38 @@ async function generatePresentation(filePath, manifest, config) {
       console.log(`  ✅ All ${codeSourceValidation.codeSlidesChecked} code slide(s) verified against source`);
     }
 
+    // Apply deterministic line breaking (AFTER validation passes)
+    console.log('  🔧 Applying line breaking...');
+    const { presentation: processedPresentation, stats } = processPresentation(presentation);
+
+    // Log statistics
+    if (stats.linesShortened > 0) {
+      console.log(`  ✂️  Fixed ${stats.linesShortened} long lines (max reduction: ${stats.maxReduction} chars)`);
+    } else {
+      console.log('  ✅ No lines exceeded 60 characters');
+    }
+
+    // Write the line-broken version back to output file
+    writeFileSync(outputPath, JSON.stringify(processedPresentation, null, 2), 'utf-8');
+
     // Copy to static directory for deployment
     const staticPath = join(STATIC_OUTPUT_DIR, dirname(relativePath), outputFileName);
     mkdirSync(dirname(staticPath), { recursive: true });
-    writeFileSync(staticPath, JSON.stringify(presentation, null, 2), 'utf-8');
+    writeFileSync(staticPath, JSON.stringify(processedPresentation, null, 2), 'utf-8');
 
     // Update manifest
     const presentationUrl = `/presentations/${join(dirname(relativePath), outputFileName)}`;
     manifest[relativePath] = {
       presentationUrl,
-      slideCount: presentation.slides.length,
-      estimatedDuration: presentation.metadata.estimatedDuration,
-      title: presentation.metadata.title,
+      slideCount: processedPresentation.slides.length,
+      estimatedDuration: processedPresentation.metadata.estimatedDuration,
+      title: processedPresentation.metadata.title,
       generatedAt: new Date().toISOString()
     };
 
     console.log(`  ✅ Generated: ${presentationUrl}`);
-    console.log(`  📊 Slides: ${presentation.slides.length}`);
-    console.log(`  ⏱️  Duration: ${presentation.metadata.estimatedDuration}`);
+    console.log(`  📊 Slides: ${processedPresentation.slides.length}`);
+    console.log(`  ⏱️  Duration: ${processedPresentation.metadata.estimatedDuration}`);
 
     return outputPath;
 

scripts/lib/markdown-parser.js

@@ -120,9 +120,10 @@ export function extractCodeSummary(code, language) {
 /**
  * Parse MDX/MD file and extract clean text content
  * @param {string} filePath - Path to MDX/MD file
- * @returns {string} Cleaned text content with code blocks described
+ * @param {boolean} preserveCode - If true, keeps code blocks intact instead of converting to descriptions
+ * @returns {string} Cleaned text content with code blocks described (or preserved if preserveCode=true)
  */
-export function parseMarkdownContent(filePath) {
+export function parseMarkdownContent(filePath, preserveCode = false) {
   const content = readFileSync(filePath, 'utf-8');
 
   // Remove frontmatter
@@ -135,42 +136,45 @@ export function parseMarkdownContent(filePath) {
   // Remove remaining HTML tags (lowercase = HTML elements)
   cleaned = cleaned.replace(/<[^>]+>/g, '');
 
-  // First pass: Find all code blocks and their contexts
-  const codeBlocks = [];
-  const regex = /```[\s\S]*?```/g;
-  let match;
-
-  while ((match = regex.exec(cleaned)) !== null) {
-    const precedingStart = Math.max(0, match.index - 200);
-    const precedingContext = cleaned.substring(precedingStart, match.index);
-
-    const followingEnd = Math.min(cleaned.length, match.index + match[0].length + 200);
-    const followingContext = cleaned.substring(match.index + match[0].length, followingEnd);
+  // Code block processing: preserve or describe based on mode
+  if (!preserveCode) {
+    // First pass: Find all code blocks and their contexts
+    const codeBlocks = [];
+    const regex = /```[\s\S]*?```/g;
+    let match;
+
+    while ((match = regex.exec(cleaned)) !== null) {
+      const precedingStart = Math.max(0, match.index - 200);
+      const precedingContext = cleaned.substring(precedingStart, match.index);
+
+      const followingEnd = Math.min(cleaned.length, match.index + match[0].length + 200);
+      const followingContext = cleaned.substring(match.index + match[0].length, followingEnd);
+
+      codeBlocks.push({
+        original: match[0],
+        index: match.index,
+        precedingContext,
+        followingContext
+      });
+    }
 
-    codeBlocks.push({
-      original: match[0],
-      index: match.index,
-      precedingContext,
-      followingContext
-    });
-  }
+    // Second pass: Replace code blocks with descriptions
+    let offset = 0;
+    for (const block of codeBlocks) {
+      const description = describeCodeBlock(block.original, block.precedingContext, block.followingContext);
+      const adjustedIndex = block.index + offset;
 
-  // Second pass: Replace code blocks with descriptions
-  let offset = 0;
-  for (const block of codeBlocks) {
-    const description = describeCodeBlock(block.original, block.precedingContext, block.followingContext);
-    const adjustedIndex = block.index + offset;
+      cleaned = cleaned.substring(0, adjustedIndex) +
+                description +
+                cleaned.substring(adjustedIndex + block.original.length);
 
-    cleaned = cleaned.substring(0, adjustedIndex) +
-              description +
-              cleaned.substring(adjustedIndex + block.original.length);
+      offset += description.length - block.original.length;
+    }
 
-    offset += description.length - block.original.length;
+    // Remove inline code (only when converting code blocks to descriptions)
+    cleaned = cleaned.replace(/`[^`]+`/g, (match) => match.replace(/`/g, ''));
   }
 
-  // Remove inline code
-  cleaned = cleaned.replace(/`[^`]+`/g, (match) => match.replace(/`/g, ''));
-
   // Remove images
   cleaned = cleaned.replace(/!\[.*?\]\(.*?\)/g, '[Image]');