@cyber-dash-tech/revela 0.15.0 → 0.15.1

package/plugin.ts

@@ -49,9 +49,8 @@ import { buildPptxNotesPrompt, handlePptx, parsePptxArgs, resolvePptxDeck } from
 import { handleEdit } from "./lib/commands/edit"
 import { handleInspect } from "./lib/commands/inspect"
 import { handleRefine } from "./lib/commands/refine"
-import { formatDeckHtmlContractReport, validateDeckHtmlContract } from "./lib/deck-html/contract"
-import { ensureEditableDeckOpenForChange } from "./lib/edit/open"
-import { hasLiveEditorSessionForFile } from "./lib/edit/server"
+import { formatArtifactQAReport, runArtifactQA } from "./lib/qa/artifact"
+import { ensureRefineDeckOpenForChange } from "./lib/refine/open"
 import { handleDesignsPreview } from "./lib/commands/designs-preview"
 import {
   parseDesignsNewArgs,
@@ -62,7 +61,7 @@ import {
 import { buildInitPrompt } from "./lib/commands/init"
 import { buildResearchPrompt } from "./lib/commands/research"
 import { handleBrief, parseBriefArgs } from "./lib/commands/brief"
-import { buildNarrativeViewPrompt, handleNarrative, parseNarrativeArgs } from "./lib/commands/narrative"
+import { buildNarrativeViewPrompt, handleNarrative, parseNarrativeArgs, parseStoryArgs } from "./lib/commands/narrative"
 import { parseRememberArgs, buildRememberPrompt } from "./lib/commands/remember"
 import { buildDeckPrompt, buildDeckReviewPrompt, buildReviewPrompt } from "./lib/commands/review"
 import {
@@ -73,7 +72,6 @@ import {
 } from "./lib/decks-memory"
 import {
   buildDecksStatePromptLayer,
-  checkDeckStateWriteReadiness,
   DECKS_STATE_FILE,
   extractDecksStateTargetsFromPatch,
   hasDecksState,
@@ -98,7 +96,6 @@ import pptxTool from "./tools/pptx"
 import createEditTool from "./tools/edit"
 import { RESEARCH_PROMPT, RESEARCH_AGENT_SIGNATURE } from "./lib/agents/research-prompt"
 import { NARRATIVE_REVIEWER_PROMPT, NARRATIVE_REVIEWER_SIGNATURE } from "./lib/agents/narrative-reviewer-prompt"
-import { formatReport, runComplianceQA } from "./lib/qa"
 import { extractDesignClasses } from "./lib/design/designs"
 import { log, childLog } from "./lib/log"
 
@@ -154,53 +151,29 @@ const server: Plugin = (async (pluginCtx) => {
   const client = pluginCtx.client
   const workspaceRoot = pluginCtx.directory
   const blockedDeckWrites = new Map<string, string>()
-  const blockedDeckPatches = new Map<string, string>()
+  const blockedPatches = new Map<string, string>()
 
-  async function appendComplianceReport(filePath: string, output: any): Promise<void> {
-    if (!isDeckHtmlPath(filePath)) return
+  async function runPostWriteArtifactQA(filePath: string, output: any): Promise<boolean> {
+    if (!isDeckHtmlPath(filePath)) return true
 
     try {
       let vocabulary
       try {
         vocabulary = extractDesignClasses()
       } catch {
-        // Design may not be installed or may have no markers — skip compliance.
+        // Design may not be installed or may have no markers — skip compliance vocabulary.
       }
 
-      const report = runComplianceQA(filePath, vocabulary)
-      if (report.totalIssues === 0) return
-
-      appendToolResult(
-        output,
-        "---\n\n**[revela design compliance]** Static check completed:\n\n" +
-        formatReport(report)
-      )
+      const report = await runArtifactQA({ workspaceRoot, filePath, vocabulary })
+      appendToolResult(output, "---\n\n" + formatArtifactQAReport(report))
+      return report.passed
     } catch (e) {
-      childLog("compliance").warn("static compliance failed", {
-        filePath,
-        error: e instanceof Error ? e.message : String(e),
-      })
-    }
-  }
-
-  async function appendDeckHtmlContractReport(filePath: string, output: any): Promise<void> {
-    if (!isDeckHtmlPath(filePath)) return
-
-    try {
-      const report = validateDeckHtmlContract(workspaceRoot, filePath)
-      if (report.status === "valid" || report.status === "skipped") return
-
-      appendToolResult(
-        output,
-        "---\n\n**[revela deck HTML contract]** Slide identity check failed:\n\n" +
-        formatDeckHtmlContractReport(report) +
-        "\n\nFix every `<section class=\"slide\">` to use the matching 1-based `data-slide-index` from DECKS.json before inspection or export."
-      )
-    } catch (e) {
-      childLog("deck-contract").warn("deck HTML contract report failed", {
+      childLog("artifact-qa").warn("post-write artifact QA failed", {
         filePath,
         error: e instanceof Error ? e.message : String(e),
       })
+      appendToolResult(output, "---\n\n## Artifact QA: FAILED\n\nError running artifact QA: " + (e instanceof Error ? e.message : String(e)))
+      return false
     }
   }
 
@@ -229,17 +202,17 @@ const server: Plugin = (async (pluginCtx) => {
     input.output.parts.push({ type: "text", text: input.visibleText } as any)
   }
 
-  function ensureEditorOpenAfterDeckChange(filePath: string, sessionID: string): void {
+  function ensureRefineOpenAfterDeckChange(filePath: string, sessionID: string): void {
     if (!isDeckHtmlPath(filePath) || !sessionID) return
 
     try {
-      ensureEditableDeckOpenForChange("", {
+      ensureRefineDeckOpenForChange("", {
         client,
         sessionID,
         workspaceRoot,
       })
     } catch (e) {
-      childLog("edit").warn("failed to ensure visual editor after deck change", {
+      childLog("refine").warn("failed to ensure Refine after deck change", {
         filePath,
         error: e instanceof Error ? e.message : String(e),
       })
@@ -409,8 +382,9 @@ const server: Plugin = (async (pluginCtx) => {
         return
       }
       if (sub === "story") {
-        if (param) {
-          await send("`/revela story` does not accept arguments yet. It opens the current workspace story UI. Use `/revela review` for a readiness report.")
+        const parsed = parseStoryArgs(param)
+        if (!parsed.ok) {
+          await send(parsed.error)
           throw new Error("__REVELA_STORY_USAGE_HANDLED__")
         }
         queueWorkflowCommand({
@@ -418,7 +392,7 @@ const server: Plugin = (async (pluginCtx) => {
           name: "story",
           mode: "narrative",
           visibleText: "Open Revela story workspace.",
-          hiddenPrompt: buildNarrativeViewPrompt({ workspaceRoot, language: "en" }),
+          hiddenPrompt: buildNarrativeViewPrompt({ workspaceRoot, language: parsed.args.language }),
           output,
         })
         return
@@ -531,7 +505,7 @@ const server: Plugin = (async (pluginCtx) => {
       }
       if (sub === "edit") {
         if (param) {
-          await send("`/revela edit` is deprecated and does not accept a target. Use `/revela refine` for the unified refinement workspace.")
+          await send("`/revela edit` has been removed. Use `/revela refine` for the unified reading, inspection, and editing workspace.")
           throw new Error("__REVELA_EDIT_USAGE_HANDLED__")
         }
         await handleEdit({ client, sessionID, workspaceRoot }, send)
@@ -850,7 +824,7 @@ const server: Plugin = (async (pluginCtx) => {
 
     // ── Pre-tool processing ────────────────────────────────────────────────
     // - read: intercept DOCX/PPTX/XLSX before read executes.
-    // - write/apply_patch: gate decks/*.html on DECKS.json readiness.
+    // - write/apply_patch: protect DECKS.json, but do not block deck HTML edits.
     "tool.execute.before": async (input, output) => {
       log.info("[hook] tool.execute.before fired", { tool: input.tool, enabled: ctx.enabled, isResearch: ctx.isResearchAgent })
       if (!ctx.enabled) return
@@ -875,32 +849,6 @@ Next step: use \`revela-decks\` with action \`init\`, \`upsertDeck\`, \`upsertSl
           childLog("decks-state").warn("blocked direct DECKS.json write", { filePath, blockedPath })
           return
         }
-        if (!isDeckHtmlPath(filePath)) return
-        if (hasLiveEditorSessionForFile(workspaceRoot, filePath)) return
-
-        const readiness = checkDeckStateWriteReadiness(workspaceRoot, filePath) ?? {
-          ready: false,
-          slug: basename(filePath, ".html") || "deck",
-          blocker: `No ${DECKS_STATE_FILE} exists. Use revela-decks init/upsertDeck/upsertSlides/review before writing deck HTML.`,
-          blockers: [`No ${DECKS_STATE_FILE} exists.`],
-        }
-        if (readiness.ready) return
-
-        const blockedDir = join(workspaceRoot, ".opencode", "revela", "blocked-writes")
-        mkdirSync(blockedDir, { recursive: true })
-        const blockedPath = join(blockedDir, `${readiness.slug}.blocked.md`)
-        ;(output.args as any).filePath = blockedPath
-        ;(output.args as any).content = `# Revela Blocked Deck Write
-
-The attempted write to \`${filePath}\` was blocked.
-
-Reason: ${readiness.blocker}
-
-Next step: use \`revela-decks\` or \`/revela review\` to update ${DECKS_STATE_FILE}, then write only after the matching deck has \`writeReadiness.status\` set to \`ready\` and no blockers.
-`
-        blockedDeckWrites.set(filePath, readiness.blocker)
-        childLog("decks-memory").warn("blocked deck write", { filePath, blockedPath, blocker: readiness.blocker })
-        return
       }
 
       if (input.tool === "apply_patch") {
@@ -925,49 +873,10 @@ Next step: use \`revela-decks\` or \`/revela review\` to update ${DECKS_STATE_FI
 +Next step: use \`revela-decks\` with action \`init\`, \`upsertDeck\`, \`upsertSlides\`, or \`review\`.
 *** End Patch`
           setPatchTextArg(args, blockedPatch)
-          blockedDeckPatches.set(blockedRelativePath, blocker)
+          blockedPatches.set(blockedRelativePath, blocker)
           childLog("decks-state").warn("blocked direct DECKS.json patch", { targets: stateTargets, blockedPath: blockedRelativePath })
           return
         }
-
-        const targets = extractDeckHtmlTargetsFromPatch(patchText)
-        if (targets.length === 0) return
-        if (targets.every((target) => hasLiveEditorSessionForFile(workspaceRoot, target))) return
-
-        const blocked = targets
-          .map((target) => ({
-            target,
-            readiness: checkDeckStateWriteReadiness(workspaceRoot, target) ?? {
-              ready: false,
-              slug: basename(target, ".html") || "deck",
-              blocker: `No ${DECKS_STATE_FILE} exists. Use revela-decks init/upsertDeck/upsertSlides/review before patching deck HTML.`,
-              blockers: [`No ${DECKS_STATE_FILE} exists.`],
-            },
-          }))
-          .find((item) => !item.readiness.ready)
-        if (!blocked) return
-
-        const blockedDir = join(workspaceRoot, ".opencode", "revela", "blocked-writes")
-        mkdirSync(blockedDir, { recursive: true })
-        const blockedRelativePath = `.opencode/revela/blocked-writes/${blocked.readiness.slug}-${Date.now()}.blocked.md`
-        const blockedPatch = `*** Begin Patch
-*** Add File: ${blockedRelativePath}
-+# Revela Blocked Deck Patch
-+
-+The attempted patch touching \`${blocked.target}\` was blocked.
-+
-+Reason: ${blocked.readiness.blocker}
-+
-+Next step: use \`revela-decks\` or \`/revela review\` to update ${DECKS_STATE_FILE}, then patch only after the matching deck has \`writeReadiness.status\` set to \`ready\` and no blockers.
-*** End Patch`
-        setPatchTextArg(args, blockedPatch)
-        blockedDeckPatches.set(blockedRelativePath, blocked.readiness.blocker)
-        childLog("decks-memory").warn("blocked deck patch", {
-          target: blocked.target,
-          blockedPath: blockedRelativePath,
-          blocker: blocked.readiness.blocker,
-        })
-        return
       }
 
       if (input.tool === "read") {
@@ -987,7 +896,7 @@ Next step: use \`revela-decks\` or \`/revela review\` to update ${DECKS_STATE_FI
     // PDF: extract text, remove base64. Images: jimp compress.
     //
     // Also reports writes/patches blocked by the DECKS.json prewrite gate and
-    // runs lightweight static design compliance after successful deck changes.
+    // runs artifact QA before opening Refine after successful deck changes.
     "tool.execute.after": async (input, output) => {
       if (!ctx.enabled) return
 
@@ -1004,7 +913,7 @@ Next step: use \`revela-decks\` or \`/revela review\` to update ${DECKS_STATE_FI
         return
       }
 
-      // ── Report blocked deck writes and run static compliance ──────────
+      // ── Report blocked state writes and run artifact QA ───────────────
       if (input.tool === "write") {
         const filePath: string = input.args?.filePath ?? ""
         const blockedReason = blockedDeckWrites.get(filePath)
@@ -1018,20 +927,19 @@ Next step: use \`revela-decks\` or \`/revela review\` to update ${DECKS_STATE_FI
           )
           return
         }
-        await appendComplianceReport(filePath, output)
-        await appendDeckHtmlContractReport(filePath, output)
-        ensureEditorOpenAfterDeckChange(filePath, extractSessionID(input))
+        const qaPassed = await runPostWriteArtifactQA(filePath, output)
+        if (qaPassed) ensureRefineOpenAfterDeckChange(filePath, extractSessionID(input))
         return
       }
 
-      if (input.tool === "apply_patch" && blockedDeckPatches.size > 0) {
-        const [blockedPath, blockedReason] = blockedDeckPatches.entries().next().value ?? []
-        if (blockedPath) blockedDeckPatches.delete(blockedPath)
+      if (input.tool === "apply_patch" && blockedPatches.size > 0) {
+        const [blockedPath, blockedReason] = blockedPatches.entries().next().value ?? []
+        if (blockedPath) blockedPatches.delete(blockedPath)
         appendToolResult(
           output,
-          "---\n\n**[revela prewrite gate]** Deck HTML patch was blocked.\n\n" +
+          "---\n\n**[revela prewrite gate]** Patch was blocked.\n\n" +
           `${blockedReason}\n\n` +
-          "Run `/revela review` or complete the same DECKS.json review workflow before patching the deck."
+          "Use the `revela-decks` tool for controlled workspace state changes."
         )
         return
       }
@@ -1040,18 +948,16 @@ Next step: use \`revela-decks\` or \`/revela review\` to update ${DECKS_STATE_FI
         const patchText = extractPatchTextArg(input.args as Record<string, unknown>)
         const targets = patchText ? extractDeckHtmlTargetsFromPatch(patchText) : []
         for (const target of targets) {
-          await appendComplianceReport(target, output)
-          await appendDeckHtmlContractReport(target, output)
-          ensureEditorOpenAfterDeckChange(target, extractSessionID(input))
+          const qaPassed = await runPostWriteArtifactQA(target, output)
+          if (qaPassed) ensureRefineOpenAfterDeckChange(target, extractSessionID(input))
         }
         return
       }
 
       if (input.tool === "edit") {
         const filePath = extractEditFilePath(input.args)
-        await appendComplianceReport(filePath, output)
-        await appendDeckHtmlContractReport(filePath, output)
-        ensureEditorOpenAfterDeckChange(filePath, extractSessionID(input))
+        const qaPassed = await runPostWriteArtifactQA(filePath, output)
+        if (qaPassed) ensureRefineOpenAfterDeckChange(filePath, extractSessionID(input))
         return
       }
     },

package/skill/NARRATIVE_SKILL.md

@@ -124,7 +124,7 @@ Use `/revela refine` for post-artifact reading, inspection, and editing.
 - Reading should explain source, support strength, caveat, unsupported scope, narrative purpose, related risks/objections, research gaps, and artifact coverage.
 - Pure artifact polish may stay artifact-level: layout, typography, spacing, crop, visual hierarchy, export mechanics, and deck contract fixes.
 - Meaning-changing edits must update canonical narrative first, then run story readiness/approval or explicit override, then remake affected artifacts.
-- Deprecated `/revela edit` and `/revela inspect` route to Refine.
+- `/revela edit` has been removed; use `/revela refine`. Deprecated `/revela inspect` routes to Refine.
 
 ## Design Surface
 

package/skill/SKILL.md

@@ -460,16 +460,11 @@ deck HTML writes or patches. If the tool result reports compliance issues, fix
 them immediately by removing the offending classes and replacing them with the
 closest component from the Component Index.
 
-Do not run `revela-qa` after writing or editing HTML unless the user explicitly
-asks for diagnostics. PDF/PPTX export commands run hard-error pre-export QA
-automatically and will report overflow issues that must be fixed before exporting.
-
-### Inline Editing
-
-**Always include inline editing** in every generated presentation. The complete
-reference implementation is provided in the active design's `@design:foundation`
-section. Follow it exactly — pay attention to the hover-delay pattern, editable
-element selector list, and `window.getEditedHTML()` definition.
+Deck HTML writes and patches automatically run Artifact QA. If hard errors are
+reported, fix them immediately with the smallest patch; Refine opens only after
+hard errors pass. Do not add deck-local inline editing JavaScript, `contenteditable`
+handlers, `editable` classes, or `window.getEditedHTML()` implementations. Post-
+artifact editing belongs in `/revela refine`, not inside generated deck HTML.
 
 ### Image Rules
 

package/tools/decks.ts

@@ -1,6 +1,7 @@
 import { tool } from "@opencode-ai/plugin"
 import {
   createDeckSpec,
+  confirmDeckPlan,
   DECKS_STATE_FILE,
   normalizeWorkspaceDeckState,
   readOrCreateDecksState,
@@ -67,7 +68,7 @@ export default tool({
     "It stores workspace narrative state, active deck specs, per-slide content/layout/components, and computes narrative or deck readiness.",
   args: {
     action: tool.schema
-      .enum(["read", "init", "upsertDeck", "upsertSlides", "upsertNarrative", "compileDeckPlan", "backfillClaimRefs", "review", "reviewNarrative", "approveNarrative", "deriveResearchGaps", "upsertResearchGaps", "updateResearchGap", "closeResearchGap", "applyEvidenceCandidates", "attachResearchFindings", "remember"])
+      .enum(["read", "init", "upsertDeck", "upsertSlides", "upsertNarrative", "compileDeckPlan", "confirmDeckPlan", "backfillClaimRefs", "review", "reviewNarrative", "approveNarrative", "deriveResearchGaps", "upsertResearchGaps", "updateResearchGap", "closeResearchGap", "applyEvidenceCandidates", "attachResearchFindings", "remember"])
       .describe("Action to perform on DECKS.json."),
     summary: tool.schema.boolean().optional().describe("For read: return a compact summary instead of full state."),
     goal: tool.schema.string().optional().describe("For upsertDeck: deck goal."),
@@ -251,8 +252,8 @@ export default tool({
     findingsFile: tool.schema.string().optional().describe("For attachResearchFindings: workspace-relative researches/{topic}/{axis}.md file to attach to researchPlan."),
     researchAxis: tool.schema.string().optional().describe("For attachResearchFindings: researchPlan axis to attach the findings file to. Required when filename matching would be ambiguous."),
     researchStatus: tool.schema.enum(["done", "read"]).optional().describe("For attachResearchFindings: optional explicit status to set on the matched research axis."),
-    approvalNote: tool.schema.string().optional().describe("For approveNarrative: optional note explaining the approval or override."),
-    approvalBy: tool.schema.enum(["user", "override"]).optional().describe("For approveNarrative: use override only for explicit render overrides, not normal strategic approval."),
+    approvalNote: tool.schema.string().optional().describe("For approveNarrative or confirmDeckPlan: optional note explaining the approval, override, or deck plan confirmation."),
+    approvalBy: tool.schema.enum(["user", "override"]).optional().describe("For approveNarrative or confirmDeckPlan: use override only for explicit render overrides, not normal strategic approval or deck plan confirmation."),
     approvalScope: tool.schema.enum(["narrative", "render_override"]).optional().describe("For approveNarrative: narrative approval or explicit render override scope."),
     gapId: tool.schema.string().optional().describe("For updateResearchGap/closeResearchGap: canonical research gap id."),
     researchGaps: tool.schema.array(tool.schema.object({
@@ -428,6 +429,31 @@ export default tool({
         return JSON.stringify({ ok: true, path: DECKS_STATE_FILE, result: compiled.result, deck: compiled.state.activeDeck ? compiled.state.decks[compiled.state.activeDeck] : undefined, narrative: compiled.state.narrative }, null, 2)
       }
 
+      if (args.action === "confirmDeckPlan") {
+        if (args.approvalBy && args.approvalBy !== "user") return JSON.stringify({ ok: false, error: "confirmDeckPlan requires approvalBy=user" })
+        const confirmed = confirmDeckPlan(state, {
+          approvedBy: "user",
+          note: args.approvalNote,
+        })
+        if (confirmed.result.confirmed) {
+          recordWorkspaceAction(confirmed.state, {
+            type: "deck.plan_confirmed",
+            actor: "revela-decks",
+            inputs: { activeDeck: state.activeDeck, approvalBy: "user" },
+            outputs: {
+              slug: confirmed.result.slug,
+              narrativeHash: confirmed.result.narrativeHash,
+              planHash: confirmed.result.planHash,
+            },
+            status: "success",
+            summary: args.approvalNote?.trim() || "User confirmed the compiled deck plan.",
+            nodeIds: [confirmed.state.narrative?.id, confirmed.result.slug ? `deck:${confirmed.result.slug}` : undefined].filter((item): item is string => Boolean(item)),
+          })
+        }
+        writeDecksState(workspaceRoot, confirmed.state)
+        return JSON.stringify({ ok: confirmed.result.confirmed, path: DECKS_STATE_FILE, result: confirmed.result, deck: confirmed.state.activeDeck ? confirmed.state.decks[confirmed.state.activeDeck] : undefined }, null, 2)
+      }
+
       if (args.action === "backfillClaimRefs") {
         const backfilled = backfillSlideClaimRefsFromCoverage(state)
         writeDecksState(workspaceRoot, backfilled.state)

package/tools/narrative-view.ts

@@ -10,7 +10,7 @@ export default tool({
     "Render Revela's read-only narrative claim-flow UI from the current deterministic narrative map plus an optional localized display model. " +
     "This tool validates display IDs against DECKS.json, opens a local HTML view, and never mutates workspace state.",
   args: {
-    language: tool.schema.string().describe("UI language request from /revela narrative. May be any language tag or language name, such as en, zh-CN, fr, de, Korean, Arabic, or Portuguese-BR."),
+    language: tool.schema.string().describe("UI language request from /revela story or /revela narrative. May be any language tag or language name, such as en, zh-CN, fr, de, Korean, Arabic, or Portuguese-BR."),
     narrativeHash: tool.schema.string().optional().describe("Narrative hash from the prompt projection. Used to detect stale display prompts."),
     displayModel: tool.schema.object({
       version: tool.schema.number().describe("Must be 1."),

package/tools/qa.ts

@@ -1,7 +1,7 @@
 /**
  * tools/qa.ts
  *
- * revela-qa — Hard-error quality assurance for generated slide HTML files.
+ * revela-qa — Artifact quality assurance for generated slide HTML files.
  *
  * Exposed as a manual diagnostic tool. Export commands run pre-export QA automatically.
  */
@@ -9,15 +9,16 @@
 import { tool } from "@opencode-ai/plugin"
 import { resolve } from "path"
 import { existsSync } from "fs"
-import { runQA, formatReport } from "../lib/qa"
+import { extractDesignClasses } from "../lib/design/designs"
+import { formatArtifactQAReport, runArtifactQA } from "../lib/qa/artifact"
 
 export default tool({
   description:
-    "Run hard-error checks on a generated slide HTML file. " +
+    "Run artifact QA on a generated slide HTML file. " +
     "Opens the file in a headless browser and measures actual rendered geometry. " +
-    "Checks for element overflow. " +
+    "Checks deck contract, component compliance, exact 1920x1080 canvas, scrollbars, element overflow, text overflow, and content/evidence density warnings. " +
     "Returns a structured report with specific issues and fix instructions. " +
-    "Normally PDF/PPTX export commands run this automatically; call it directly only for explicit diagnostics.",
+    "Deck writes and PDF/PPTX export commands run QA automatically; call it directly for explicit diagnostics.",
   args: {
     file: tool.schema
       .string()
@@ -39,20 +40,25 @@ export default tool({
     }
 
     try {
-      const report = await runQA(filePath)
-      const formatted = formatReport(report)
+      let vocabulary
+      try {
+        vocabulary = extractDesignClasses()
+      } catch {
+        // Design may not be installed or may have no markers.
+      }
+      const report = await runArtifactQA({ workspaceRoot: directory || process.cwd(), filePath, vocabulary })
+      const formatted = formatArtifactQAReport(report)
 
       // Prepend a compact JSON summary for programmatic use if needed
       const jsonSummary = JSON.stringify({
-        totalIssues: report.totalIssues,
-        errors: report.errorCount,
+        passed: report.passed,
+        errors: report.hardErrorCount,
         warnings: report.warningCount,
-        slidesWithIssues: report.slides.filter((s) => s.issues.length > 0).map((s) => s.index + 1),
       })
 
       return `<!-- QA Summary: ${jsonSummary} -->\n\n${formatted}`
     } catch (err: any) {
-      return `Error running hard-error QA: ${err?.message ?? String(err)}\n\nMake sure Chrome is installed at /Applications/Google Chrome.app`
+      return `Error running artifact QA: ${err?.message ?? String(err)}\n\nMake sure Chrome is installed at /Applications/Google Chrome.app`
     }
   },
 })