@uniweb/content-writer 0.3.0 → 0.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniweb/content-writer",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "ProseMirror document structure to Markdown converter",
   "type": "module",
   "main": "src/index.js",

package/src/normalize.js

@@ -193,14 +193,8 @@ function mapEditorNode(node) {
     case 'Icon':
       return mapUniwebIcon(node)
     case 'WarningBlock':
-    case 'details': {
-      const attrs = EDITOR_CONTAINERS[node.type](node)
-      return {
-        type: 'inset_block',
-        attrs,
-        content: node.type === 'details' ? detailsBody(node) : asBlocks(node.content),
-      }
-    }
+    case 'details':
+      return EDITOR_CONTAINERS[node.type](node)
     case 'emoji':
       // The node carries the character alongside a name; the character is the
       // whole markdown form. Guarded rather than assumed to be populated.
@@ -256,8 +250,25 @@ const NAMED_INLINE_STYLES = new Set(['accent', 'highlight', 'callout', 'muted'])
  * missing entry here is a visible gap rather than a silent assumption.
  */
 const EDITOR_CONTAINERS = {
-  WarningBlock: node => ({ component: 'Alert', type: node.attrs?.type || 'info' }),
-  details: () => ({ component: 'Details' }),
+  // A callout's SEVERITY is the concept, not a parameter on one — a warning and
+  // a note are different kinds of thing, so they are different tags. This is
+  // where the editor's `type` attribute goes.
+  WarningBlock: node => ({
+    type: 'concept_block',
+    attrs: { tag: node.attrs?.type || 'info' },
+    content: asBlocks(node.content),
+  }),
+  // The summary becomes the leading HEADING rather than a paragraph, which is
+  // what makes the mapping lossless in both directions: a concept block's items
+  // come from its headings, so the summary recovers as that item's title and
+  // the body as its paragraphs. Flattening it to a paragraph — what this did
+  // while the target was an inset — made the two indistinguishable on the way
+  // back.
+  details: node => ({
+    type: 'concept_block',
+    attrs: { tag: 'details' },
+    content: detailsBody(node),
+  }),
 }
 
 /** Inline node types, for deciding whether a container's body needs wrapping. */
@@ -289,11 +300,18 @@ function asBlocks(content) {
 }
 
 /**
- * Flatten the editor's `details` › `detailsSummary` + `detailsContent` into the
- * container's body, summary first.
+ * Flatten the editor's `details` › `detailsSummary` + `detailsContent` into a
+ * concept block's body, summary first — as a HEADING.
  *
- * The summary leads rather than riding on an attribute: `detailsSummary` can
- * carry marks, and `summary="…"` would flatten emphasis and links.
+ * The heading is what makes the mapping lossless. A concept block's items come
+ * from its headings, so a summary written as `# …` recovers as that item's
+ * title and the body as its paragraphs; written as a paragraph (which is what
+ * this did while the target was an inset) the two become indistinguishable on
+ * the way back, and a disclosure loses which half was its label.
+ *
+ * The summary stays inline content rather than moving to an attribute for the
+ * original reason: `detailsSummary` can carry marks, and `summary="…"` would
+ * flatten emphasis and links.
  *
  * @param {Object} node
  * @returns {Array}
@@ -301,8 +319,11 @@ function asBlocks(content) {
 function detailsBody(node) {
   const body = []
   for (const child of node.content || []) {
-    if (child.type === 'detailsSummary') body.push(...asBlocks(child.content))
-    else if (child.type === 'detailsContent') body.push(...(child.content || []))
+    if (child.type === 'detailsSummary') {
+      if (child.content?.length) {
+        body.push({ type: 'heading', attrs: { level: 1 }, content: child.content })
+      }
+    } else if (child.type === 'detailsContent') body.push(...(child.content || []))
     else body.push(child)
   }
   return body

package/tests/roundtrip-source.test.js

@@ -263,22 +263,34 @@ describe('containers — the block form of an inset', () => {
     expect(rt(source)).toBe(source)
   })
 
-  test('the editor container shapes normalize onto it', () => {
+  test('the editor containers normalize onto CONCEPT blocks, not insets', () => {
+    // Changed 2026-07-31. These used to become `@Alert` and `@Details` — an
+    // inset naming the component that renders it, which is a rendering
+    // decision living in content. A concept block names what the content IS
+    // and leaves rendering to the foundation.
     const text = t => ({ type: 'text', text: t })
     const doc = node => proseMirrorToMarkdown({ type: 'doc', content: [node] }).trim()
 
-    // WarningBlock ↔ @Alert, defaulting to info, NOT warning — a name-derived
-    // mapping drops exactly this.
+    // The editor's `type` becomes the TAG: a warning and a note are different
+    // kinds of thing, not one kind with a parameter. `info` is still the
+    // default, and still not `warning` — a name-derived mapping drops that.
     expect(doc({ type: 'WarningBlock', attrs: {}, content: [text('Careful.')] }))
-      .toBe('```@Alert{type=info}\nCareful.\n```')
-
-    // The summary is the first block and keeps its marks.
+      .toBe('```md:info\nCareful.\n```')
+    expect(doc({ type: 'WarningBlock', attrs: { type: 'warning' }, content: [text('Careful.')] }))
+      .toBe('```md:warning\nCareful.\n```')
+
+    // The summary becomes the leading HEADING, which is what makes this
+    // lossless: a concept block's items come from its headings, so the summary
+    // recovers as the item's title and the body as its paragraphs. As a
+    // paragraph — what this produced while the target was an inset — the two
+    // were indistinguishable coming back.
     expect(doc({
       type: 'details',
       content: [
         { type: 'detailsSummary', content: [text('Breaking '), { type: 'text', text: 'changes', marks: [{ type: 'bold' }] }] },
         { type: 'detailsContent', content: [{ type: 'paragraph', content: [text('Body.')] }] },
       ],
-    })).toBe('```@Details\nBreaking **changes**\n\nBody.\n```')
+    })).toBe('```md:details\n# Breaking **changes**\n\nBody.\n```')
   })
+
 })