@cyber-dash-tech/revela 0.8.1 → 0.8.3

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

@@ -6,18 +6,15 @@ import {
   DECKS_STATE_FILE,
   normalizeWorkspaceDeckState,
   readOrCreateDecksState,
-  reviewDeckState,
   upsertDeck,
   upsertSlides,
   writeDecksState,
-  type DeckStateReadinessResult,
   type SlideSpec,
 } from "../decks-state"
 import type { EditableDeck } from "./resolve-deck"
 
 export interface EditDeckStatePreflightResult {
   changed: boolean
-  readiness: DeckStateReadinessResult
 }
 
 export function ensureEditableDeckState(workspaceRoot: string, deck: EditableDeck): EditDeckStatePreflightResult {
@@ -27,7 +24,6 @@ export function ensureEditableDeckState(workspaceRoot: string, deck: EditableDec
     throw new Error(`${DECKS_STATE_FILE} already points to ${active.outputPath}. Revela 0.8 expects one deck per workspace; move extra decks to a separate workspace.`)
   }
   const existing = state.decks[deck.slug]
-  const existingReady = existing?.writeReadiness?.status === "ready" && existing.writeReadiness.blockers.length === 0
   let changed = !existing || existing.outputPath !== deck.file
 
   state = upsertDeck(state, {
@@ -36,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(),
@@ -46,7 +41,6 @@ export function ensureEditableDeckState(workspaceRoot: string, deck: EditableDec
       ...existing?.requiredInputs,
       topicClarified: true,
       audienceClarified: true,
-      slideCountDecided: true,
       languageDecided: true,
       visualStyleSelected: true,
       sourceMaterialsIdentified: true,
@@ -64,19 +58,13 @@ export function ensureEditableDeckState(workspaceRoot: string, deck: EditableDec
     changed = true
   }
 
-  const reviewed = reviewDeckState(state, deck.slug)
-  writeDecksState(workspaceRoot, reviewed.state)
+  writeDecksState(workspaceRoot, state)
 
   return {
-    changed: changed || !existingReady,
-    readiness: reviewed.result,
+    changed,
   }
 }
 
-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

@@ -66,9 +66,6 @@ function openEditableDeckInternal(
 ): EnsureEditableDeckOpenResult {
   const deck = resolveEditableDeck(options.workspaceRoot, target)
   const preflight = ensureEditableDeckState(options.workspaceRoot, deck)
-  if (!preflight.readiness.ready) {
-    throw new Error(preflight.readiness.blocker || "Deck is not ready for HTML edits.")
-  }
 
   ctx.enabled = true
   if (!existsSync(ACTIVE_PROMPT_FILE)) {
@@ -88,7 +85,7 @@ function openEditableDeckInternal(
   if (shouldOpen) (options.openUrl ?? openUrl)(url)
 
   const source = deck.source === "decks-state" ? "DECKS.json" : deck.source === "file-path" ? "file path" : "fallback path"
-  const stateNote = preflight.changed ? "Deck state was prepared in DECKS.json before opening the editor." : "Deck state is ready in DECKS.json."
+  const stateNote = preflight.changed ? "Deck state was prepared in DECKS.json for visual editing." : "Deck state already points to this visual edit target."
 
   return {
     deck,

package/lib/edit/server.ts

@@ -1,5 +1,6 @@
 import { randomBytes } from "crypto"
 import { readFileSync, statSync } from "fs"
+import { resolve, sep } from "path"
 import type { EditableDeck } from "./resolve-deck"
 import { buildEditPrompt, type EditCommentPayload } from "./prompt"
 
@@ -85,6 +86,16 @@ export function hasLiveEditorSession(deck: EditableDeck, maxIdleMs = LIVE_EDITOR
   return existing ? isSessionLive(existing.session, maxIdleMs) : false
 }
 
+export function hasLiveEditorSessionForFile(workspaceRoot: string, filePath: string, maxIdleMs = LIVE_EDITOR_IDLE_MS): boolean {
+  if (!filePath) return false
+  const root = resolve(workspaceRoot)
+  const absoluteFile = resolve(root, filePath)
+  if (absoluteFile !== root && !absoluteFile.startsWith(root.endsWith(sep) ? root : root + sep)) return false
+  cleanupExpiredSessions()
+  const existing = findSessionForDeck(absoluteFile)
+  return existing ? isSessionLive(existing.session, maxIdleMs) : false
+}
+
 export function stopEditServer(): void {
   if (idleTimer) clearTimeout(idleTimer)
   idleTimer = undefined

package/package.json

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

package/plugin.ts

@@ -47,6 +47,7 @@ import { handlePdf } from "./lib/commands/pdf"
 import { handlePptx } from "./lib/commands/pptx"
 import { handleEdit } from "./lib/commands/edit"
 import { ensureEditableDeckOpenForChange } from "./lib/edit/open"
+import { hasLiveEditorSessionForFile } from "./lib/edit/server"
 import { handleDesignsPreview } from "./lib/commands/designs-preview"
 import {
   parseDesignsNewArgs,
@@ -540,6 +541,7 @@ Next step: use \`revela-decks\` with action \`init\`, \`upsertDeck\`, \`upsertSl
           return
         }
         if (!isDeckHtmlPath(filePath)) return
+        if (hasLiveEditorSessionForFile(workspaceRoot, filePath)) return
 
         const readiness = checkDeckStateWriteReadiness(workspaceRoot, filePath) ?? {
           ready: false,
@@ -595,6 +597,7 @@ Next step: use \`revela-decks\` or \`/revela review\` to update ${DECKS_STATE_FI
 
         const targets = extractDeckHtmlTargetsFromPatch(patchText)
         if (targets.length === 0) return
+        if (targets.every((target) => hasLiveEditorSessionForFile(workspaceRoot, target))) return
 
         const blocked = targets
           .map((target) => ({

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,