@cyber-dash-tech/revela 0.17.16 → 0.17.17

package/lib/html-export/deck-detect.ts

@@ -22,55 +22,81 @@ export async function detectDeckHtmlWithBrowser(browser: Browser, htmlFilePath:
   try {
     await page.goto(pathToFileURL(htmlFilePath).href, { waitUntil: "domcontentloaded", timeout: 15000 })
     return await page.evaluate(() => {
+      const CANVAS_WIDTH = 1920
+      const CANVAS_HEIGHT = 1080
+      const DIMENSION_TOLERANCE = 2
+      const isDeckCanvasSize = (el: HTMLElement): boolean => {
+        const rect = el.getBoundingClientRect()
+        return (
+          Math.abs(rect.width - CANVAS_WIDTH) <= DIMENSION_TOLERANCE &&
+          Math.abs(rect.height - CANVAS_HEIGHT) <= DIMENSION_TOLERANCE
+        )
+      }
+
       const slides = Array.from(document.querySelectorAll(".slide")) as HTMLElement[]
       if (slides.length === 0) {
         return { isDeck: false, slideCount: 0, reason: "no .slide elements found" }
       }
 
-      const missingCanvas = slides.findIndex((slide) => !slide.querySelector(".slide-canvas"))
-      if (missingCanvas >= 0) {
-        return {
-          isDeck: false,
-          slideCount: slides.length,
-          reason: `.slide ${missingCanvas + 1} has no .slide-canvas`,
+      const indexValues = slides.map((slide) => slide.getAttribute("data-slide-index"))
+      const seen = new Set<number>()
+      for (let i = 0; i < indexValues.length; i++) {
+        const raw = indexValues[i]
+        if (raw === null || raw.trim() === "") {
+          return {
+            isDeck: false,
+            slideCount: slides.length,
+            reason: `slide ${i + 1} is missing data-slide-index`,
+          }
         }
-      }
-
-      const indexValues = slides
-        .map((slide) => slide.getAttribute("data-slide-index"))
-        .filter((value): value is string => value !== null && value.trim() !== "")
 
-      if (indexValues.length > 0) {
-        if (indexValues.length !== slides.length) {
+        const parsed = Number(raw)
+        if (!Number.isInteger(parsed) || parsed < 1) {
           return {
             isDeck: false,
             slideCount: slides.length,
-            reason: "some slides have data-slide-index and some do not",
+            reason: `slide ${i + 1} has invalid data-slide-index "${raw}"`,
           }
         }
-
-        const seen = new Set<number>()
-        for (let i = 0; i < indexValues.length; i++) {
-          const parsed = Number(indexValues[i])
-          if (!Number.isInteger(parsed) || parsed < 1) {
-            return {
-              isDeck: false,
-              slideCount: slides.length,
-              reason: `slide ${i + 1} has invalid data-slide-index "${indexValues[i]}"`,
-            }
+        if (seen.has(parsed)) {
+          return {
+            isDeck: false,
+            slideCount: slides.length,
+            reason: `duplicate data-slide-index "${parsed}"`,
           }
-          if (seen.has(parsed)) {
-            return {
-              isDeck: false,
-              slideCount: slides.length,
-              reason: `duplicate data-slide-index "${parsed}"`,
-            }
+        }
+        if (parsed !== i + 1) {
+          return {
+            isDeck: false,
+            slideCount: slides.length,
+            reason: `slide ${i + 1} has data-slide-index "${parsed}", expected "${i + 1}"`,
           }
-          seen.add(parsed)
         }
+        seen.add(parsed)
       }
 
-      return { isDeck: true, slideCount: slides.length, reason: "valid deck contract" }
+      let usedSlideAsCanvas = false
+      for (let i = 0; i < slides.length; i++) {
+        const slide = slides[i]
+        if (slide.querySelector(".slide-canvas")) continue
+        if (isDeckCanvasSize(slide)) {
+          usedSlideAsCanvas = true
+          continue
+        }
+        return {
+          isDeck: false,
+          slideCount: slides.length,
+          reason: `.slide ${i + 1} has no .slide-canvas and is not 1920x1080`,
+        }
+      }
+
+      return {
+        isDeck: true,
+        slideCount: slides.length,
+        reason: usedSlideAsCanvas
+          ? "valid deck contract: slide-as-canvas"
+          : "valid deck contract: slide-canvas",
+      }
     })
   } finally {
     await page.close().catch(() => undefined)

package/lib/pdf/export.ts

@@ -9,7 +9,7 @@
  *    rewrite the HTML to use file:// local paths — avoids CDN/CORS/headless issues
  * 3. Navigate to the patched HTML file
  * 4. For each .slide: force .reveal.visible, wait 800ms, screenshot .slide-canvas
- *    using offsetParent-chain absolute coordinates
+ *    or slide-as-canvas fallback using offsetParent-chain absolute coordinates
  * 5. Assemble screenshots into a multi-page PDF (16:9, 1920×1080pt per page) via pdf-lib
  * 6. Write PDF alongside the HTML file (same directory, .html → .pdf)
  * 7. Clean up temp dir
@@ -335,24 +335,23 @@ export async function exportDeckToPdf(htmlFilePath: string): Promise<Omit<Export
       // Wait for CSS transitions and JS rendering (ECharts animations, etc.)
       await new Promise((r) => setTimeout(r, 800))
 
-      // Compute .slide-canvas absolute position by walking the offsetParent chain.
+      // Compute screenshot target absolute position by walking the offsetParent chain.
       // getBoundingClientRect() returns viewport-relative coords (always near 0,0) —
       // unusable as screenshot clip coordinates without adding scrollY.
       // offsetParent walk gives document-absolute coords that Puppeteer clip expects.
       const clipRect = await page.evaluate((i: number) => {
         const slide = document.querySelectorAll(".slide")[i] as HTMLElement | null
         if (!slide) return null
-        const canvas = slide.querySelector(".slide-canvas") as HTMLElement | null
-        if (!canvas) return null
+        const target = (slide.querySelector(".slide-canvas") as HTMLElement | null) ?? slide
         let top = 0
         let left = 0
-        let el: HTMLElement | null = canvas
+        let el: HTMLElement | null = target
         while (el) {
           top += el.offsetTop
           left += el.offsetLeft
           el = el.offsetParent as HTMLElement | null
         }
-        return { x: left, y: top, width: canvas.offsetWidth, height: canvas.offsetHeight }
+        return { x: left, y: top, width: target.offsetWidth, height: target.offsetHeight }
       }, idx)
 
       if (clipRect && clipRect.width > 0 && clipRect.height > 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cyber-dash-tech/revela",
-  "version": "0.17.16",
+  "version": "0.17.17",
   "description": "OpenCode plugin for trusted narrative artifacts from local sources, research, and evidence",
   "type": "module",
   "main": "./index.ts",

package/plugins/revela/.mcp.json

@@ -2,7 +2,7 @@
   "mcpServers": {
     "revela": {
       "command": "npx",
-      "args": ["-y", "@cyber-dash-tech/revela@0.17.16", "mcp"]
+      "args": ["-y", "@cyber-dash-tech/revela@0.17.17", "mcp"]
     }
   }
 }