@cyber-dash-tech/revela 0.8.2 → 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

@@ -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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cyber-dash-tech/revela",
-  "version": "0.8.2",
+  "version": "0.8.3",
   "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,