@cyber-dash-tech/revela 0.8.2 → 0.8.4

package/README.md

@@ -205,7 +205,7 @@ Review the current deck state with:
 
 Minimum readiness conditions:
 
-- topic, audience, slide count, language, and visual style/design are decided
+- topic, audience, language, visual style/design, and slide plan are decided
 - source materials are identified or explicitly deemed unnecessary
 - research need is assessed
 - needed research findings have been read and reflected in the slide specs
@@ -222,7 +222,6 @@ The gate checks:
 - `writeReadiness.blockers` is empty
 - the deck `outputPath` exactly matches the target `decks/*.html` path
 - all `requiredInputs` booleans are true
-- `slides.length` matches `slideCount` when a slide count is set
 - every slide has title, layout, components, and structured content
 - every needed research axis is `done`, `read`, or `skipped`
 

package/README.zh-CN.md

@@ -204,7 +204,7 @@ Revela 使用工作区根目录的 `DECKS.json` 做跨会话记忆和 deck 生
 
 最小 readiness 条件：
 
-- topic、audience、slide count、language、visual style/design 已确定
+- topic、audience、language、visual style/design 和 slide plan 已确定
 - source materials 已识别，或明确不需要源材料
 - research need 已评估
 - 需要调研时，相关 findings 已读取并反映到逐页规格中
@@ -221,7 +221,6 @@ Revela 使用工作区根目录的 `DECKS.json` 做跨会话记忆和 deck 生
 - `writeReadiness.blockers` 为空
 - deck `outputPath` 精确匹配目标 `decks/*.html` 路径
 - 所有 `requiredInputs` 布尔值都是 true
-- 如果设置了 `slideCount`，`slides.length` 必须匹配
 - 每页都有 title、layout、components 和结构化 content
 - 每个 needed research axis 都是 `done`、`read` 或 `skipped`
 

package/lib/commands/init.ts

@@ -16,9 +16,10 @@ export function buildInitPrompt({
 Goal:
 - Build or update ${DECKS_STATE_FILE}, the workspace-level machine-readable state file for slide deck work.
 - Use the \`revela-decks\` tool for state updates. Do not write or patch ${DECKS_STATE_FILE} directly.
-- Capture stable project context, available source materials, the current deck spec, slide plan, and open questions for future sessions.
+- Capture stable project context, professional/domain context, topic, audience, available source materials, the current deck artifact, slide specs, and open questions for future sessions.
 - Do not treat initialization as permission to write a slide deck; the current deck must pass a later readiness review.
 - ${DECKS_STATE_FILE} is the source of truth for the single current workspace deck.
+- \`slides\` is the only slide-plan source of truth. Do not track or infer a separate slide count field.
 
 Current state:
 - ${mode}
@@ -45,8 +46,9 @@ Workflow:
 4. For selected PDF/PPTX/DOCX/XLSX files, call \`revela-extract-document-materials\` before deciding what to summarize.
 5. Read only the materials needed to form a conservative workspace memory. Do not exhaustively read every file if the workspace is large.
 6. Call \`revela-decks\` with action \`init\` to create ${DECKS_STATE_FILE} if needed.
-7. If this conversation already contains a concrete deck task, call \`revela-decks\` with action \`upsertDeck\` and later \`upsertSlides\` only for explicit deck spec information. Do not pass or ask for a deck key; the tool uses the workspace folder name internally. Do not mark readiness ready during init.
-8. Report what was initialized or updated and list any open questions.
+7. If this conversation or the workspace contains a concrete deck task or an existing deck artifact, call \`revela-decks\` with action \`upsertDeck\` and later \`upsertSlides\` for explicit deck information. Do not pass or ask for a deck key; the tool uses the workspace folder name internally. Do not mark readiness ready during init.
+8. When adopting an existing HTML deck, analyze the artifact and create one conservative \`SlideSpec\` per identifiable slide/page. The \`SlideSpec[]\` itself is the worklist; do not create a separate target slide count.
+9. Report what was initialized or updated and list any open questions.
 
 Memory rules:
 - Only write facts supported by workspace files into ${DECKS_STATE_FILE} workspace state, source materials, deck memory, and open questions.

package/lib/decks-state.ts

@@ -45,7 +45,6 @@ export interface DeckSpec {
   goal: string
   audience?: string
   language?: string
-  slideCount?: number
   outputPath: string
   theme: {
     design?: string
@@ -65,7 +64,6 @@ export interface DeckSpec {
 export interface RequiredInputs {
   topicClarified: boolean
   audienceClarified: boolean
-  slideCountDecided: boolean
   languageDecided: boolean
   visualStyleSelected: boolean
   sourceMaterialsIdentified: boolean
@@ -179,17 +177,15 @@ export function normalizeWorkspaceDeckState(state: DecksState, workspaceRoot: st
 
 export function defaultRequiredInputs(overrides?: Partial<RequiredInputs>): RequiredInputs {
   return {
-    topicClarified: false,
-    audienceClarified: false,
-    slideCountDecided: false,
-    languageDecided: false,
-    visualStyleSelected: false,
-    sourceMaterialsIdentified: false,
-    researchNeedAssessed: false,
-    researchFindingsRead: false,
-    slidePlanConfirmed: false,
-    designLayoutsFetched: false,
-    ...overrides,
+    topicClarified: overrides?.topicClarified ?? false,
+    audienceClarified: overrides?.audienceClarified ?? false,
+    languageDecided: overrides?.languageDecided ?? false,
+    visualStyleSelected: overrides?.visualStyleSelected ?? false,
+    sourceMaterialsIdentified: overrides?.sourceMaterialsIdentified ?? false,
+    researchNeedAssessed: overrides?.researchNeedAssessed ?? false,
+    researchFindingsRead: overrides?.researchFindingsRead ?? false,
+    slidePlanConfirmed: overrides?.slidePlanConfirmed ?? false,
+    designLayoutsFetched: overrides?.designLayoutsFetched ?? false,
   }
 }
 
@@ -201,7 +197,6 @@ export function createDeckSpec(input: Partial<DeckSpec> & { slug: string }): Dec
     goal: input.goal ?? "",
     audience: input.audience,
     language: input.language,
-    slideCount: input.slideCount,
     outputPath: normalizeDeckPath(input.outputPath || `decks/${slug}.html`),
     theme: input.theme ?? {},
     requiredInputs: defaultRequiredInputs(input.requiredInputs),
@@ -419,9 +414,6 @@ function computeDeckBlockers(deck: DeckSpec): string[] {
     if (value !== true) blockers.push(`requiredInputs.${key} is not true`)
   }
 
-  if (typeof deck.slideCount === "number" && deck.slideCount > 0 && deck.slides.length !== deck.slideCount) {
-    blockers.push(`slides length ${deck.slides.length} does not match slideCount ${deck.slideCount}`)
-  }
   if (deck.slides.length === 0) blockers.push("slides are missing")
   for (const slide of deck.slides) {
     if (!slide.title.trim()) blockers.push(`Slide ${slide.index} title is missing`)

package/lib/edit/deck-state.ts

@@ -32,7 +32,6 @@ export function ensureEditableDeckState(workspaceRoot: string, deck: EditableDec
     goal: existing?.goal || `Edit existing Revela deck ${deck.slug}.`,
     audience: existing?.audience || "Existing deck viewers",
     language: existing?.language || "en",
-    slideCount: existing?.slideCount || inferSlideCount(deck.absoluteFile),
     outputPath: deck.file,
     theme: {
       design: existing?.theme?.design || safeActiveDesign(),
@@ -42,7 +41,6 @@ export function ensureEditableDeckState(workspaceRoot: string, deck: EditableDec
       ...existing?.requiredInputs,
       topicClarified: true,
       audienceClarified: true,
-      slideCountDecided: true,
       languageDecided: true,
       visualStyleSelected: true,
       sourceMaterialsIdentified: true,
@@ -67,10 +65,6 @@ export function ensureEditableDeckState(workspaceRoot: string, deck: EditableDec
   }
 }
 
-function inferSlideCount(filePath: string): number {
-  return inferSlides(filePath).length
-}
-
 function inferSlides(filePath: string): SlideSpec[] {
   const html = readFileSync(filePath, "utf-8")
   const chunks = html.match(/<section\b[\s\S]*?<\/section>/gi) || [html]

package/lib/edit/open.ts

@@ -78,6 +78,7 @@ function openEditableDeckInternal(
   const session = editServer.getOrCreateSession({
     client: options.client,
     sessionID: options.sessionID,
+    workspaceRoot: options.workspaceRoot,
     deck,
   })
   const url = `${editServer.baseUrl}/edit?token=${encodeURIComponent(session.token)}`

package/lib/edit/server.ts

@@ -1,6 +1,7 @@
 import { randomBytes } from "crypto"
-import { readFileSync, statSync } from "fs"
-import { resolve, sep } from "path"
+import { existsSync, readFileSync, statSync } from "fs"
+import { fileURLToPath } from "url"
+import { dirname, extname, isAbsolute, resolve, sep } from "path"
 import type { EditableDeck } from "./resolve-deck"
 import { buildEditPrompt, type EditCommentPayload } from "./prompt"
 
@@ -9,6 +10,11 @@ const SESSION_TTL_MS = 2 * 60 * 60 * 1000
 const IDLE_STOP_MS = 30 * 60 * 1000
 export const LIVE_EDITOR_IDLE_MS = 10 * 1000
 
+interface EditAsset {
+  id: string
+  absoluteFile: string
+}
+
 interface EditSession {
   token: string
   client: any
@@ -16,13 +22,17 @@ interface EditSession {
   deck: string
   file: string
   absoluteFile: string
+  workspaceRoot: string
+  assets: Map<string, EditAsset>
+  assetKeys: Map<string, string>
+  nextAssetId: number
   createdAt: number
   lastActiveAt: number
 }
 
 export interface EditServerHandle {
   baseUrl: string
-  getOrCreateSession(input: { client: any; sessionID: string; deck: EditableDeck }): EditServerSessionResult
+  getOrCreateSession(input: { client: any; sessionID: string; workspaceRoot: string; deck: EditableDeck }): EditServerSessionResult
 }
 
 export interface EditServerSessionResult {
@@ -57,6 +67,7 @@ export function startEditServer(): EditServerHandle {
         existing.session.sessionID = input.sessionID
         existing.session.deck = input.deck.slug
         existing.session.file = input.deck.file
+        existing.session.workspaceRoot = resolve(input.workspaceRoot)
         return {
           token: existing.token,
           reused: true,
@@ -72,6 +83,10 @@ export function startEditServer(): EditServerHandle {
         deck: input.deck.slug,
         file: input.deck.file,
         absoluteFile: input.deck.absoluteFile,
+        workspaceRoot: resolve(input.workspaceRoot),
+        assets: new Map(),
+        assetKeys: new Map(),
+        nextAssetId: 1,
         createdAt: Date.now(),
         lastActiveAt: Date.now(),
       })
@@ -131,7 +146,13 @@ async function handleRequest(req: Request): Promise<Response> {
   if (url.pathname === "/deck" && req.method === "GET") {
     const session = validateSession(url.searchParams.get("token"))
     if (!session.ok) return session.response
-    return htmlResponse(readFileSync(session.value.absoluteFile, "utf-8"))
+    return handleDeck(session.value)
+  }
+
+  if (url.pathname === "/__revela_asset" && (req.method === "GET" || req.method === "HEAD")) {
+    const session = validateSession(url.searchParams.get("token"))
+    if (!session.ok) return session.response
+    return handleAsset(session.value, url.searchParams.get("id"), req.method)
   }
 
   if (url.pathname === "/api/comment" && req.method === "POST") {
@@ -149,6 +170,203 @@ async function handleRequest(req: Request): Promise<Response> {
   return textResponse("Not found", 404)
 }
 
+function handleDeck(session: EditSession): Response {
+  session.assets.clear()
+  session.assetKeys.clear()
+  session.nextAssetId = 1
+  const html = readFileSync(session.absoluteFile, "utf-8")
+  return htmlResponse(rewriteLocalAssetRefs(html, {
+    session,
+    sourceFile: session.absoluteFile,
+    contentType: "html",
+  }))
+}
+
+function handleAsset(session: EditSession, id: string | null, method: string): Response {
+  if (!id) return textResponse("Missing asset id", 400)
+  const asset = session.assets.get(id)
+  if (!asset) return textResponse("Asset not found", 404)
+  if (!existsSync(asset.absoluteFile)) return textResponse("Asset file not found", 404)
+  if (!statSync(asset.absoluteFile).isFile()) return textResponse("Asset is not a file", 404)
+
+  const mime = mimeTypeForPath(asset.absoluteFile)
+  const headers = {
+    "content-type": mime,
+    "cache-control": "no-store, max-age=0",
+  }
+  if (method === "HEAD") return new Response(null, { status: 200, headers })
+
+  if (mime === "text/css") {
+    const css = readFileSync(asset.absoluteFile, "utf-8")
+    return new Response(rewriteLocalAssetRefs(css, {
+      session,
+      sourceFile: asset.absoluteFile,
+      contentType: "css",
+    }), { status: 200, headers })
+  }
+
+  return new Response(new Uint8Array(readFileSync(asset.absoluteFile)), { status: 200, headers })
+}
+
+function rewriteLocalAssetRefs(content: string, input: { session: EditSession; sourceFile: string; contentType: "html" | "css" }): string {
+  const baseDir = dirname(input.sourceFile)
+  let rewritten = rewriteCssUrls(content, input.session, baseDir)
+  if (input.contentType === "css") return rewritten
+
+  rewritten = rewriteHtmlAssetAttributes(rewritten, input.session, baseDir)
+  rewritten = rewriteSrcsetAttributes(rewritten, input.session, baseDir)
+  return rewritten
+}
+
+function rewriteHtmlAssetAttributes(html: string, session: EditSession, baseDir: string): string {
+  const attrPattern = /\b(src|href|poster)\s*=\s*("([^"]*)"|'([^']*)'|([^\s>]+))/gi
+  return html.replace(attrPattern, (match, name: string, raw: string, doubleQuoted?: string, singleQuoted?: string, unquoted?: string) => {
+    const value = doubleQuoted ?? singleQuoted ?? unquoted ?? ""
+    const assetUrl = assetUrlForRef(value, session, baseDir)
+    if (!assetUrl) return match
+    const quote = doubleQuoted !== undefined ? '"' : singleQuoted !== undefined ? "'" : ""
+    const escaped = quote ? assetUrl.replace(/&/g, "&amp;") : assetUrl
+    return `${name}=${quote}${escaped}${quote}`
+  })
+}
+
+function rewriteSrcsetAttributes(html: string, session: EditSession, baseDir: string): string {
+  const srcsetPattern = /\bsrcset\s*=\s*("([^"]*)"|'([^']*)'|([^\s>]+))/gi
+  return html.replace(srcsetPattern, (match, raw: string, doubleQuoted?: string, singleQuoted?: string, unquoted?: string) => {
+    const value = doubleQuoted ?? singleQuoted ?? unquoted ?? ""
+    const rewritten = rewriteSrcset(value, session, baseDir)
+    if (rewritten === value) return match
+    const quote = doubleQuoted !== undefined ? '"' : singleQuoted !== undefined ? "'" : ""
+    const escaped = quote ? rewritten.replace(/&/g, "&amp;") : rewritten
+    return `srcset=${quote}${escaped}${quote}`
+  })
+}
+
+function rewriteSrcset(value: string, session: EditSession, baseDir: string): string {
+  return value.split(",").map((part) => {
+    const trimmed = part.trim()
+    if (!trimmed) return part
+    const pieces = trimmed.split(/\s+/)
+    const assetUrl = assetUrlForRef(pieces[0], session, baseDir)
+    if (!assetUrl) return part
+    return [assetUrl, ...pieces.slice(1)].join(" ")
+  }).join(", ")
+}
+
+function rewriteCssUrls(content: string, session: EditSession, baseDir: string): string {
+  const cssUrlPattern = /url\(\s*("([^"]*)"|'([^']*)'|([^\s)]+))\s*\)/gi
+  return content.replace(cssUrlPattern, (match, raw: string, doubleQuoted?: string, singleQuoted?: string, unquoted?: string) => {
+    const value = doubleQuoted ?? singleQuoted ?? unquoted ?? ""
+    const assetUrl = assetUrlForRef(value, session, baseDir)
+    if (!assetUrl) return match
+    return `url("${assetUrl.replace(/"/g, "%22")}")`
+  })
+}
+
+function assetUrlForRef(ref: string, session: EditSession, baseDir: string): string | null {
+  const absoluteFile = resolveLocalAssetRef(ref, session.workspaceRoot, baseDir)
+  if (!absoluteFile || !existsSync(absoluteFile) || !statSync(absoluteFile).isFile()) return null
+  const key = resolve(absoluteFile)
+  let id = session.assetKeys.get(key)
+  if (!id) {
+    id = String(session.nextAssetId++)
+    session.assetKeys.set(key, id)
+    session.assets.set(id, { id, absoluteFile: key })
+  }
+  return `/__revela_asset?token=${encodeURIComponent(session.token)}&id=${encodeURIComponent(id)}`
+}
+
+function resolveLocalAssetRef(ref: string, workspaceRoot: string, baseDir: string): string | null {
+  const trimmed = ref.trim()
+  if (!trimmed || isSkippedAssetRef(trimmed)) return null
+
+  const pathPart = stripQueryAndHash(trimmed)
+  if (!pathPart) return null
+
+  if (pathPart.startsWith("file://")) {
+    try {
+      return resolve(fileURLToPath(pathPart))
+    } catch {
+      return null
+    }
+  }
+
+  const decodedPath = safeDecodeUri(pathPart)
+  if (isWindowsAbsolutePath(decodedPath)) return resolve(decodedPath)
+
+  if (isAbsolute(decodedPath)) {
+    const absolute = resolve(decodedPath)
+    if (existsSync(absolute)) return absolute
+    return resolve(workspaceRoot, `.${decodedPath}`)
+  }
+
+  return resolve(baseDir, decodedPath)
+}
+
+function stripQueryAndHash(ref: string): string {
+  const hashIndex = ref.indexOf("#")
+  const withoutHash = hashIndex >= 0 ? ref.slice(0, hashIndex) : ref
+  const queryIndex = withoutHash.indexOf("?")
+  return queryIndex >= 0 ? withoutHash.slice(0, queryIndex) : withoutHash
+}
+
+function safeDecodeUri(value: string): string {
+  try {
+    return decodeURI(value)
+  } catch {
+    return value
+  }
+}
+
+function isSkippedAssetRef(ref: string): boolean {
+  return /^(?:https?:|data:|blob:|mailto:|tel:|javascript:|#)/i.test(ref) || ref.startsWith("//")
+}
+
+function isWindowsAbsolutePath(value: string): boolean {
+  return /^[a-zA-Z]:[\\/]/.test(value)
+}
+
+function mimeTypeForPath(filePath: string): string {
+  switch (extname(filePath).toLowerCase()) {
+    case ".html":
+    case ".htm":
+      return "text/html; charset=utf-8"
+    case ".css":
+      return "text/css"
+    case ".js":
+      return "application/javascript"
+    case ".jpg":
+    case ".jpeg":
+      return "image/jpeg"
+    case ".png":
+      return "image/png"
+    case ".gif":
+      return "image/gif"
+    case ".webp":
+      return "image/webp"
+    case ".svg":
+      return "image/svg+xml"
+    case ".woff":
+      return "font/woff"
+    case ".woff2":
+      return "font/woff2"
+    case ".ttf":
+      return "font/ttf"
+    case ".otf":
+      return "font/otf"
+    case ".mp4":
+      return "video/mp4"
+    case ".webm":
+      return "video/webm"
+    case ".mp3":
+      return "audio/mpeg"
+    case ".wav":
+      return "audio/wav"
+    default:
+      return "application/octet-stream"
+  }
+}
+
 function handleDeckVersion(session: EditSession): Response {
   try {
     const version = readDeckVersion(session)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cyber-dash-tech/revela",
-  "version": "0.8.2",
+  "version": "0.8.4",
   "description": "OpenCode plugin that turns AI into an HTML slide deck generator",
   "type": "module",
   "main": "./index.ts",

package/skill/SKILL.md

@@ -23,7 +23,7 @@ Before writing any HTML, ask the user these questions **in a single message**
 
 1. **Topic** — What is the presentation about?
 2. **Audience** — Who will see it? (e.g. investors, team, conference, class)
-3. **Slide count** — How many slides? (suggest 6–10 if unsure)
+3. **Scope** — How broad should the deck be? If the user has a preferred length, treat it as guidance only.
 4. **Language** — What language should the slides be in?
 5. **Reference materials** — Do you have any reference files to draw content from?
    (PDF research reports, Excel data, Word documents, PowerPoint decks, images)
@@ -43,7 +43,7 @@ research or writing HTML. The brief should capture:
 - working thesis or angle, if one has emerged
 - key questions the deck must answer
 - known workspace sources from `DECKS.json`, user attachments, or visible files
-- desired output shape, slide count, and visual direction
+- desired output shape, approximate scope, and visual direction
 
 If the brief is unclear, ask 1–3 targeted clarification questions. Do not force
 the user to provide a research topic command; the working topic emerges from the
@@ -82,7 +82,7 @@ All subsequent file paths in this session use the current workspace deck:
 Create or update the active deck in `DECKS.json` through `revela-decks` actions
 `upsertDeck` and `upsertSlides`. Keep the deck spec current as work progresses:
 - `goal` — purpose and decision/context
-- `audience`, `language`, `slideCount`, `outputPath`, and `theme`
+- `audience`, `language`, `outputPath`, and `theme`
 - `requiredInputs` — checklist state for prewrite readiness
 - `researchPlan` — axes, status, and findings files
 - `slides` — confirmed per-slide title, purpose, layout, components, content, evidence, visuals, and status
@@ -227,7 +227,7 @@ The exact visual style for each section comes from the active design.
 When the user asks for N slides, distribute them across these sections.
 A 6-slide deck might be: Cover → Background → Content × 3 → Closing.
 An 8-slide deck might be: Cover → TOC → Background → Content × 3 → Summary → Closing.
-Never skip Cover, Background, or Closing regardless of slide count.
+Never skip Cover, Background, or Closing regardless of deck length.
 
 **Every `<section class="slide">` must include a `slide-qa` attribute.** Set
 `slide-qa="true"` for content-heavy layouts (those marked ✓ in the Layout Index
@@ -247,7 +247,7 @@ core rules and the visual design below.
 
 **When a domain definition is present:**
 - Follow its report structure instead of the default "Required Slide Structure" above.
-  The domain defines its own sections, ordering, and slide count distribution.
+  The domain defines its own sections, ordering, and deck length guidance.
 - Follow its AI logic rules (e.g. terminology, evidence standards, risk frameworks).
 - The domain's visual preferences are **suggestions only** — the active Design's
   visual rules always take precedence for colors, fonts, animations, and layout.

package/tools/decks.ts

@@ -28,7 +28,6 @@ export default tool({
     goal: tool.schema.string().optional().describe("For upsertDeck: deck goal."),
     audience: tool.schema.string().optional().describe("For upsertDeck: deck audience."),
     language: tool.schema.string().optional().describe("For upsertDeck: deck language."),
-    slideCount: tool.schema.number().optional().describe("For upsertDeck: expected slide count."),
     outputPath: tool.schema.string().optional().describe("For upsertDeck: target output path, normally decks/{workspace-name}.html."),
     design: tool.schema.string().optional().describe("For upsertDeck: active design name."),
     domain: tool.schema.string().optional().describe("For upsertDeck: active domain name."),
@@ -37,7 +36,6 @@ export default tool({
     requiredInputs: tool.schema.object({
       topicClarified: tool.schema.boolean().optional(),
       audienceClarified: tool.schema.boolean().optional(),
-      slideCountDecided: tool.schema.boolean().optional(),
       languageDecided: tool.schema.boolean().optional(),
       visualStyleSelected: tool.schema.boolean().optional(),
       sourceMaterialsIdentified: tool.schema.boolean().optional(),
@@ -111,7 +109,6 @@ export default tool({
           goal: args.goal ?? existing?.goal ?? "",
           audience: args.audience ?? existing?.audience,
           language: args.language ?? existing?.language,
-          slideCount: args.slideCount ?? existing?.slideCount,
           outputPath: args.outputPath ?? existing?.outputPath,
           theme: {
             design: args.design ?? existing?.theme?.design,