From 3119ef0f004b576d67737bebf517947f8819b4fa Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Tue, 22 Jul 2025 18:54:44 -0400
Subject: Render message markdown to HTML inside of `<Message />`.

This simplifies data flow, at the potential expense of re-rendering HTML more often than strictly necessary. Requiring every path that produces a message-shaped object to pre-render markdown made things more interdependent than intended and slowed me down.
---
 ui/routes/(swatch)/.swatch/Message/+page.svelte    | 25 ++++------------------
 ui/routes/(swatch)/.swatch/MessageRun/+page.svelte |  4 +---
 2 files changed, 5 insertions(+), 24 deletions(-)

(limited to 'ui/routes/(swatch)')

diff --git a/ui/routes/(swatch)/.swatch/Message/+page.svelte b/ui/routes/(swatch)/.swatch/Message/+page.svelte
index aab17d5..bc27522 100644
--- a/ui/routes/(swatch)/.swatch/Message/+page.svelte
+++ b/ui/routes/(swatch)/.swatch/Message/+page.svelte
@@ -2,7 +2,6 @@
   import { DateTime } from 'luxon';
 
   import EventCapture from '$lib/swatch/event-capture.svelte.js';
-  import { render } from '$lib/markdown.js';
 
   import Message from '$lib/components/Message.svelte';
   import EventLog from '$lib/components/swatch/EventLog.svelte';
@@ -12,7 +11,7 @@
   // Astonishingly, `DateTime.fromISO` does not throw on invalid inputs. It generates an "Invalid
   // DateTime" sentinel value, instead.
   let at = $derived(DateTime.fromISO(atInput));
-  let renderedBodyInput = $state(
+  let body = $state(
     `Lorem ipsum \`dolor\` sit amet, consectetur adipiscing elit. Nunc quis ante ac leo tristique
 iaculis vel in tortor. Praesent sed interdum ipsum. Pellentesque blandit, sapien at mattis
 facilisis, leo mi gravida erat, in euismod mi lectus non dui. Praesent at justo vel mauris pulvinar
@@ -27,22 +26,6 @@ maximus, nisl eget interdum ornare, enim turpis semper ligula, sed ultricies sem
 dapibus augue. Pellentesque nec tincidunt sem.
 `,
   );
-  /*
-   * Even though `Message` is notionally a generic container for markup, we restrict the swatch to
-   * message-flavoured Markdown. Swatches are available to all users, including
-   * technically-unsophisticated ones, and anything rendered in a swatch runs in the same origin
-   * context and the same cookie context as the rest of the client.
-   *
-   * This makes it possible that a user would be persuaded to enter something into a swatch that
-   * then runs _as them_, interacting with Pilcrow via its API or accessing client-stored data.
-   *
-   * As a proof of concept, `<img src="x" onerror="console.log('oh no')">` should not run the log
-   * statement. With generic HTML entry, it would do so. With our markdown processing, it does not
-   * (the `onerror` attribute is removed). Similarly, `script` elements are prohibited.
-   *
-   * Users who want to experiment with free HTML are encouraged to edit the swatch for themselves.
-   */
-  let renderedBody = $derived(render(renderedBodyInput));
   let editable = $state(true);
   let cssClass = $state('');
 
@@ -75,15 +58,15 @@ dapibus augue. Pellentesque nec tincidunt sem.
   <label>editable <input type="checkbox" bind:checked={editable} /></label>
 
   <label
-    ><p>rendered body (markdown)</p>
-    <textarea bind:value={renderedBodyInput}></textarea>
+    ><p>body (markdown)</p>
+    <textarea bind:value={body}></textarea>
   </label>
 </div>
 
 <h2>rendered</h2>
 
 <div class="component-preview">
-  <Message {id} {at} {renderedBody} {editable} class={cssClass} {deleteMessage} />
+  <Message {id} {at} {body} {editable} class={cssClass} {deleteMessage} />
 </div>
 
 <h2>events</h2>
diff --git a/ui/routes/(swatch)/.swatch/MessageRun/+page.svelte b/ui/routes/(swatch)/.swatch/MessageRun/+page.svelte
index 34118ec..30aeee4 100644
--- a/ui/routes/(swatch)/.swatch/MessageRun/+page.svelte
+++ b/ui/routes/(swatch)/.swatch/MessageRun/+page.svelte
@@ -1,8 +1,6 @@
 <script>
   import { DateTime } from 'luxon';
 
-  import { render } from '$lib/markdown.js';
-
   import MessageRun from '$lib/components/MessageRun.svelte';
   import Message from '$lib/components/Message.svelte';
 
@@ -59,7 +57,7 @@ quis arcu. Ut a dapibus augue. Pellentesque nec tincidunt sem.`,
 <div class="component-preview">
   <MessageRun {sender} class={cssClass}>
     {#each messages.entries() as [index, message]}
-      <Message id="Mplaceholder-{index}" at={DateTime.now()} renderedBody={render(message)} />
+      <Message id="Mplaceholder-{index}" at={DateTime.now()} body={message} />
     {/each}
   </MessageRun>
 </div>
-- 
cgit v1.2.3