@cyber-dash-tech/revela 0.17.0 → 0.17.1

package/lib/refine/server.ts

@@ -253,35 +253,105 @@ async function handleAssetSave(req: Request, session: EditSession): Promise<Resp
   const candidate = normalizeImageCandidate(body?.candidate ?? body)
   if (!candidate) return jsonResponse({ ok: false, error: "Valid image candidate is required" }, 400)
   const purpose = normalizeMediaPurpose(body?.purpose) || candidate.purpose || "illustration"
-  const result = await saveMediaAsset({
-    topic: session.deck,
+  const brief = body?.brief || `Saved from ${candidate.provider} for Review asset placement.`
+  const saved = await saveAssetCandidateUrls({
+    session,
+    candidate,
     id: body?.id || candidate.candidateId,
-    type: "image",
     purpose,
-    brief: body?.brief || `Saved from ${candidate.provider} for Review asset placement.`,
-    status: "success",
-    sourceUrl: candidate.imageUrl,
+    brief,
     alt: body?.alt || candidate.alt || candidate.title,
     notes: body?.notes,
-    provider: candidate.provider,
-    sourcePageUrl: candidate.sourcePageUrl,
-    license: candidate.license,
-    attribution: candidate.attribution,
-    width: candidate.width,
-    height: candidate.height,
-  }, session.workspaceRoot)
+  })
 
   session.lastActiveAt = Date.now()
   scheduleIdleStop()
+  const result = saved.result
   if (!result.ok) return jsonResponse({ ok: false, error: result.error }, 400)
   if (result.status !== "success" || !result.path) {
-    return jsonResponse({ ok: false, error: `Failed to save asset: ${result.status}` }, 400)
+    return jsonResponse({ ok: false, error: failedAssetSaveMessage(result.status, saved.failures) }, 400)
   }
   const asset = savedAssetForResult(session, result.assetId)
+    ?? savedAssetFallback(session, {
+      id: result.assetId,
+      path: result.path,
+      sourceUrl: saved.sourceUrl,
+      purpose,
+      brief,
+      candidate,
+    })
   if (!asset) return jsonResponse({ ok: false, error: "Saved asset was not found in workspace assets." }, 500)
   return jsonResponse({ ok: true, asset, result })
 }
 
+async function saveAssetCandidateUrls(input: {
+  session: EditSession
+  candidate: ImageCandidate
+  id: string
+  purpose: MediaPurpose
+  brief: string
+  alt?: string
+  notes?: string
+}): Promise<{
+  result: Awaited<ReturnType<typeof saveMediaAsset>>
+  sourceUrl?: string
+  failures: Array<{ url: string; status: string }>
+}> {
+  const urls = uniqueAssetUrls([input.candidate.imageUrl, input.candidate.thumbnailUrl])
+  const failures: Array<{ url: string; status: string }> = []
+  let lastResult: Awaited<ReturnType<typeof saveMediaAsset>> | undefined
+
+  for (const sourceUrl of urls) {
+    const result = await saveMediaAsset({
+      topic: input.session.deck,
+      id: input.id,
+      type: "image",
+      purpose: input.purpose,
+      brief: input.brief,
+      status: "success",
+      sourceUrl,
+      alt: input.alt,
+      notes: input.notes,
+      provider: input.candidate.provider,
+      sourcePageUrl: input.candidate.sourcePageUrl,
+      license: input.candidate.license,
+      attribution: input.candidate.attribution,
+      width: input.candidate.width,
+      height: input.candidate.height,
+    }, input.session.workspaceRoot)
+    lastResult = result
+    if (result.ok && result.status === "success" && result.path) return { result, sourceUrl, failures }
+    failures.push({ url: sourceUrl, status: result.ok ? result.failureReason ?? result.status : result.error })
+  }
+
+  return {
+    result: lastResult ?? { ok: false, error: "No downloadable image URL was provided" },
+    failures,
+  }
+}
+
+function uniqueAssetUrls(values: Array<string | undefined>): string[] {
+  const seen = new Set<string>()
+  return values.flatMap((value) => {
+    const url = value?.trim()
+    if (!url || seen.has(url)) return []
+    seen.add(url)
+    return [url]
+  })
+}
+
+function failedAssetSaveMessage(status: string, failures: Array<{ url: string; status: string }>): string {
+  if (!failures.length) return `Failed to save asset: ${status}`
+  const details = failures
+    .map((failure) => `${shortUrl(failure.url)}: ${failure.status}`)
+    .join("; ")
+  return `Failed to save asset: ${status} (${details})`
+}
+
+function shortUrl(value: string): string {
+  return value.length <= 96 ? value : `${value.slice(0, 93)}...`
+}
+
 function handleAssetList(session: EditSession): Response {
   session.lastActiveAt = Date.now()
   scheduleIdleStop()
@@ -292,6 +362,39 @@ function savedAssetForResult(session: EditSession, assetId: string): (MediaAsset
   return listSavedAssets(session).find((asset) => asset.id === assetId) ?? null
 }
 
+function savedAssetFallback(
+  session: EditSession,
+  input: {
+    id: string
+    path: string | null
+    sourceUrl?: string
+    purpose: MediaPurpose
+    brief: string
+    candidate: ImageCandidate
+  },
+): (MediaAssetRecord & { previewUrl?: string; deckPath?: string }) | null {
+  if (!input.path) return null
+  return {
+    id: input.id,
+    type: "image",
+    purpose: input.purpose,
+    brief: input.brief,
+    status: "success",
+    path: input.path,
+    sourceUrl: input.sourceUrl ?? input.candidate.imageUrl,
+    alt: input.candidate.alt || input.candidate.title,
+    provider: input.candidate.provider,
+    sourcePageUrl: input.candidate.sourcePageUrl,
+    license: input.candidate.license,
+    attribution: input.candidate.attribution,
+    width: input.candidate.width,
+    height: input.candidate.height,
+    savedAt: new Date().toISOString(),
+    previewUrl: assetUrlForRef(input.path, session, session.workspaceRoot) ?? undefined,
+    deckPath: relative(dirname(session.absoluteFile), resolve(session.workspaceRoot, input.path)).replace(/\\/g, "/"),
+  }
+}
+
 function listSavedAssets(session: EditSession): Array<MediaAssetRecord & { previewUrl?: string; deckPath?: string }> {
   const manifestPath = resolve(session.workspaceRoot, "assets", slugify(session.deck), "media-manifest.json")
   if (!existsSync(manifestPath)) return []
@@ -891,6 +994,7 @@ export function renderRefineShell(token: string, defaultMode: RefineMode = "edit
     .skeleton-line.long { width: 92%; }
     .asset-card.is-saving::after { content: ""; position: absolute; inset: 0; background: rgba(15,23,42,.32); }
     .asset-card.is-saving .asset-save { z-index: 1; }
+    .asset-card.is-saved-candidate .asset-thumb { opacity: .72; }
     @keyframes spin { to { transform: rotate(360deg); } }
     @keyframes shimmer { 0% { background-position: 200% 0; } 100% { background-position: -200% 0; } }
     .asset-search { display: grid; grid-template-columns: minmax(0, 1fr) 118px; gap: 8px; }
@@ -913,6 +1017,7 @@ export function renderRefineShell(token: string, defaultMode: RefineMode = "edit
     .asset-search-title span { color: #756f66; font-size: 12px; line-height: 1.35; }
     .asset-back { width: auto; padding: 9px 11px; border-radius: 999px; background: #ebe4d8; color: #111827; box-shadow: none; }
     .asset-save { position: absolute; left: 7px; right: 7px; bottom: 7px; width: auto; padding: 7px 8px; border-radius: 10px; font-size: 11px; background: rgba(17,24,39,.9); color: #fbfaf7; box-shadow: 0 8px 16px rgba(31,41,51,.2); opacity: .96; }
+    .asset-save.saved { background: rgba(77,97,56,.94); color: #fbfaf7; cursor: default; }
     .asset-empty { grid-column: 1 / -1; margin: 0; color: #756f66; font-size: 12px; line-height: 1.45; }
     .edit-assets { padding: 10px; border: 1px solid #d8d2c6; border-radius: 16px; background: #f7f3ea; }
     .edit-assets .panel { gap: 8px; }
@@ -2000,9 +2105,13 @@ export function renderRefineShell(token: string, defaultMode: RefineMode = "edit
         }
         state.assetCandidates.forEach((candidate, index) => {
           const card = assetCard(candidate, false, index);
+          const savedAsset = savedAssetForCandidate(candidate);
           if (state.assetSavingIndex === index) {
             card.classList.add('is-saving');
             appendAssetSaveButton(card, 'Saving...', 'Saving to workspace', () => {}, true);
+          } else if (savedAsset) {
+            card.classList.add('is-saved-candidate');
+            appendAssetSaveButton(card, '✅ Saved', 'Asset already saved to Local Assets', () => {}, false, 'saved');
           } else {
             appendAssetSaveButton(card, 'Save', 'Save to workspace', () => saveCandidate(index));
           }
@@ -2025,7 +2134,11 @@ export function renderRefineShell(token: string, defaultMode: RefineMode = "edit
           });
           const body = await res.json().catch(() => ({}));
           if (!res.ok || !body.ok) throw new Error(body.error || 'Failed to save asset');
-          await loadSavedAssets();
+          const listed = await loadSavedAssets();
+          if (body.asset && (!listed || !findSavedAsset(body.asset.id))) {
+            mergeSavedAsset(body.asset);
+            renderSavedAssets();
+          }
           const path = body.asset && (body.asset.path || body.asset.deckPath);
           setStatus(path ? 'Saved to ' + path + '. Use it from Local Assets.' : 'Asset saved. Use it from Local Assets.');
         } catch (error) {
@@ -2043,12 +2156,36 @@ export function renderRefineShell(token: string, defaultMode: RefineMode = "edit
           if (!res.ok || !body.ok) throw new Error(body.error || 'Failed to list assets');
           state.savedAssets = Array.isArray(body.assets) ? body.assets : [];
           renderSavedAssets();
+          return true;
         } catch (error) {
-          const message = '<p class="asset-empty">' + escapeHtml(error && error.message ? error.message : String(error)) + '</p>';
-          els.editSavedAssets.innerHTML = message;
+          if (!state.savedAssets.length) {
+            const message = '<p class="asset-empty">' + escapeHtml(error && error.message ? error.message : String(error)) + '</p>';
+            els.editSavedAssets.innerHTML = message;
+          }
+          return false;
         }
       }
 
+      function mergeSavedAsset(asset) {
+        if (!asset || !asset.id) return;
+        const next = state.savedAssets.filter((existing) => existing.id !== asset.id);
+        next.unshift(asset);
+        state.savedAssets = next;
+      }
+
+      function savedAssetForCandidate(candidate) {
+        const id = slugifyAssetId(candidate && candidate.candidateId);
+        if (!id) return null;
+        return state.savedAssets.find((asset) => asset.id === id) || null;
+      }
+
+      function slugifyAssetId(value) {
+        return String(value || '')
+          .toLowerCase()
+          .replace(/[^a-z0-9]+/g, '-')
+          .replace(/^-+|-+$/g, '');
+      }
+
       function renderSavedAssets() {
         renderSavedAssetGrid(els.editSavedAssets, 'No local assets yet. Click + to search assets.');
       }
@@ -2108,12 +2245,12 @@ export function renderRefineShell(token: string, defaultMode: RefineMode = "edit
         }
       }
 
-      function appendAssetSaveButton(card, text, label, onClick, loading) {
+      function appendAssetSaveButton(card, text, label, onClick, loading, variant) {
         const button = document.createElement('button');
         button.type = 'button';
-        button.className = 'asset-save';
+        button.className = variant ? 'asset-save ' + variant : 'asset-save';
         button.innerHTML = loading ? '<span class="spinner" aria-hidden="true"></span><span>' + escapeHtml(text) + '</span>' : escapeHtml(text);
-        button.disabled = !!loading;
+        button.disabled = !!loading || variant === 'saved';
         button.setAttribute('aria-label', label);
         button.title = label;
         button.addEventListener('click', onClick);

package/package.json

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

package/skill/SKILL.md

@@ -95,7 +95,8 @@ Before writing HTML, the confirmed plan must include:
   non-structural slides assigned to each chapter.
 - One row per slide with title, purpose, narrative role, content summary, layout,
   components, primary/supporting claim ids, evidence binding ids or source
-  summary, visual intent, and caveats/unsupported scope.
+  summary, `content.data.visualIntent`, `visuals[]`, and caveats/unsupported
+  scope.
 - A low-fidelity layout sketch for every slide when requested by the handoff
   prompt.
 
@@ -110,6 +111,10 @@ Rules for the slide plan:
   "overview of topic".
 - Every content slide must carry a distinct claim, evidence item, comparison,
   risk, or action.
+- Treat `content.data.visualIntent` and `visuals[]` as required render
+  instructions, not optional decoration. Do not downgrade a planned metric card,
+  evidence table, comparison grid, risk matrix, steps view, chart, or media brief
+  into generic bullets unless the user revises and reconfirms the plan.
 - Normal content slides should usually contain 2-4 semantic boxes/cards unless
   intentionally using a focus layout.
 - If a chapter lacks enough substance for its allocated slides, reduce the slide

package/tools/narrative-view.ts

@@ -22,6 +22,18 @@ export default tool({
         claimFlow: tool.schema.string().optional(),
         flowNote: tool.schema.string().optional(),
         selectedClaim: tool.schema.string().optional(),
+        selectedEvidence: tool.schema.string().optional(),
+        evidenceList: tool.schema.string().optional(),
+        gap: tool.schema.string().optional(),
+        gaps: tool.schema.string().optional(),
+        noEvidence: tool.schema.string().optional(),
+        selectEvidencePrompt: tool.schema.string().optional(),
+        sourceTrace: tool.schema.string().optional(),
+        evidenceSource: tool.schema.string().optional(),
+        whyThisSupports: tool.schema.string().optional(),
+        linkedGaps: tool.schema.string().optional(),
+        selectedGap: tool.schema.string().optional(),
+        noLinkedGaps: tool.schema.string().optional(),
         claim: tool.schema.string().optional(),
         claimId: tool.schema.string().optional(),
         status: tool.schema.string().optional(),
@@ -51,6 +63,10 @@ export default tool({
         riskOrGapSummary: tool.schema.string().optional(),
         researchGapsSummary: tool.schema.string().optional().describe("Display-only localized summary of research gaps tied to this claim."),
       })).optional(),
+      researchGapCards: tool.schema.array(tool.schema.object({
+        gapId: tool.schema.string().describe("Existing canonical research gap id. Must match the deterministic map."),
+        displayQuestion: tool.schema.string().optional().describe("Display-only localized gap question. Do not change source facts, ids, targets, or gap meaning."),
+      })).optional(),
       relations: tool.schema.array(tool.schema.object({
         fromClaimId: tool.schema.string(),
         toClaimId: tool.schema.string(),