@pyreon/document-primitives 0.13.1 → 0.15.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pyreon/document-primitives",
-  "version": "0.13.1",
+  "version": "0.15.0",
   "description": "Rocketstyle document components — render in browser, export to 18 formats",
   "license": "MIT",
   "repository": {
@@ -10,6 +10,7 @@
   },
   "files": [
     "lib",
+    "!lib/**/*.map",
     "!lib/analysis",
     "README.md",
     "LICENSE",
@@ -41,28 +42,29 @@
     "typecheck": "tsc --noEmit"
   },
   "dependencies": {
-    "@pyreon/connector-document": "^0.13.1"
+    "@pyreon/connector-document": "^0.15.0"
   },
   "devDependencies": {
-    "@pyreon/core": "^0.13.1",
-    "@pyreon/elements": "^0.13.1",
-    "@pyreon/reactivity": "^0.13.1",
-    "@pyreon/rocketstyle": "^0.13.1",
-    "@pyreon/runtime-dom": "^0.13.1",
-    "@pyreon/styler": "^0.13.1",
-    "@pyreon/test-utils": "^0.13.1",
-    "@pyreon/typescript": "^0.13.1",
-    "@pyreon/ui-core": "^0.13.1",
+    "@pyreon/core": "^0.15.0",
+    "@pyreon/elements": "^0.15.0",
+    "@pyreon/manifest": "0.13.1",
+    "@pyreon/reactivity": "^0.15.0",
+    "@pyreon/rocketstyle": "^0.15.0",
+    "@pyreon/runtime-dom": "^0.15.0",
+    "@pyreon/styler": "^0.15.0",
+    "@pyreon/test-utils": "^0.13.2",
+    "@pyreon/typescript": "^0.15.0",
+    "@pyreon/ui-core": "^0.15.0",
     "@vitest/browser-playwright": "^4.1.4",
-    "@vitus-labs/tools-rolldown": "^1.15.4"
+    "@vitus-labs/tools-rolldown": "^2.3.0"
   },
   "peerDependencies": {
-    "@pyreon/core": "^0.13.1",
-    "@pyreon/document": "^0.13.1",
-    "@pyreon/elements": "^0.13.1",
-    "@pyreon/rocketstyle": "^0.13.1",
-    "@pyreon/styler": "^0.13.1",
-    "@pyreon/ui-core": "^0.13.1"
+    "@pyreon/core": "^0.15.0",
+    "@pyreon/document": "^0.15.0",
+    "@pyreon/elements": "^0.15.0",
+    "@pyreon/rocketstyle": "^0.15.0",
+    "@pyreon/styler": "^0.15.0",
+    "@pyreon/ui-core": "^0.15.0"
   },
   "engines": {
     "node": ">= 22"

package/src/__tests__/manifest-snapshot.test.ts

@@ -0,0 +1,45 @@
+import {
+  renderApiReferenceEntries,
+  renderLlmsFullSection,
+  renderLlmsTxtLine,
+} from '@pyreon/manifest'
+import manifest from '../manifest'
+
+describe('gen-docs — document-primitives snapshot', () => {
+  it('renders a llms.txt bullet starting with the package prefix', () => {
+    const line = renderLlmsTxtLine(manifest)
+    expect(line.startsWith('- @pyreon/document-primitives —')).toBe(true)
+  })
+
+  it('renders a llms-full.txt section with the right header', () => {
+    const section = renderLlmsFullSection(manifest)
+    expect(section.startsWith('## @pyreon/document-primitives —')).toBe(true)
+    expect(section).toContain('```typescript')
+  })
+
+  it('renders MCP api-reference entries for every api[] item', () => {
+    const record = renderApiReferenceEntries(manifest)
+    expect(Object.keys(record).sort()).toEqual([
+      'document-primitives/DocButton',
+      'document-primitives/DocCode',
+      'document-primitives/DocColumn',
+      'document-primitives/DocDivider',
+      'document-primitives/DocDocument',
+      'document-primitives/DocHeading',
+      'document-primitives/DocImage',
+      'document-primitives/DocLink',
+      'document-primitives/DocList',
+      'document-primitives/DocListItem',
+      'document-primitives/DocPage',
+      'document-primitives/DocPageBreak',
+      'document-primitives/DocQuote',
+      'document-primitives/DocRow',
+      'document-primitives/DocSection',
+      'document-primitives/DocSpacer',
+      'document-primitives/DocTable',
+      'document-primitives/DocText',
+      'document-primitives/createDocumentExport',
+      'document-primitives/extractDocNode',
+    ])
+  })
+})

package/src/__tests__/primitives-attrs.test.ts

@@ -14,7 +14,7 @@ describe('DocDocument attrs', () => {
   it('sets tag to div', async () => {
     const DocDocument = (await import('../primitives/DocDocument')).default
     const result = renderProps(DocDocument, { children: 'test' })
-    expect(result.tag).toBe('div')
+    expect(result.as).toBe('div')
   })
 
   it('passes title to _documentProps', async () => {
@@ -158,7 +158,7 @@ describe('DocTable attrs', () => {
   it('sets tag to table', async () => {
     const DocTable = (await import('../primitives/DocTable')).default
     const result = renderProps(DocTable, { children: null })
-    expect(result.tag).toBe('table')
+    expect(result.as).toBe('table')
   })
 
   it('defaults columns and rows to empty arrays', async () => {
@@ -195,13 +195,13 @@ describe('DocList attrs', () => {
   it('sets tag to ul by default', async () => {
     const DocList = (await import('../primitives/DocList')).default
     const result = renderProps(DocList, { children: null })
-    expect(result.tag).toBe('ul')
+    expect(result.as).toBe('ul')
   })
 
   it('sets tag to ol when ordered is true', async () => {
     const DocList = (await import('../primitives/DocList')).default
     const result = renderProps(DocList, { ordered: true, children: null })
-    expect(result.tag).toBe('ol')
+    expect(result.as).toBe('ol')
     expect(result._documentProps.ordered).toBe(true)
   })
 
@@ -278,7 +278,7 @@ describe('DocPage attrs', () => {
   it('sets tag to div', async () => {
     const DocPage = (await import('../primitives/DocPage')).default
     const result = renderProps(DocPage, { children: 'page' })
-    expect(result.tag).toBe('div')
+    expect(result.as).toBe('div')
   })
 
   it('passes size and orientation when provided', async () => {
@@ -306,7 +306,7 @@ describe('DocPageBreak attrs', () => {
   it('sets tag to div with empty _documentProps', async () => {
     const DocPageBreak = (await import('../primitives/DocPageBreak')).default
     const result = renderProps(DocPageBreak, { children: null })
-    expect(result.tag).toBe('div')
+    expect(result.as).toBe('div')
     expect(result._documentProps).toEqual({})
   })
 })
@@ -318,7 +318,7 @@ describe('DocQuote attrs', () => {
   it('sets tag to blockquote', async () => {
     const DocQuote = (await import('../primitives/DocQuote')).default
     const result = renderProps(DocQuote, { children: 'quote' })
-    expect(result.tag).toBe('blockquote')
+    expect(result.as).toBe('blockquote')
   })
 
   it('passes borderColor when provided', async () => {
@@ -341,7 +341,7 @@ describe('DocRow attrs', () => {
   it('sets tag to div with empty _documentProps', async () => {
     const DocRow = (await import('../primitives/DocRow')).default
     const result = renderProps(DocRow, { children: null })
-    expect(result.tag).toBe('div')
+    expect(result.as).toBe('div')
     expect(result._documentProps).toEqual({})
   })
 })
@@ -353,7 +353,7 @@ describe('DocColumn attrs', () => {
   it('sets tag to div', async () => {
     const DocColumn = (await import('../primitives/DocColumn')).default
     const result = renderProps(DocColumn, { children: null })
-    expect(result.tag).toBe('div')
+    expect(result.as).toBe('div')
   })
 
   it('passes width to _documentProps when provided', async () => {
@@ -376,7 +376,7 @@ describe('DocSpacer attrs', () => {
   it('sets tag to div', async () => {
     const DocSpacer = (await import('../primitives/DocSpacer')).default
     const result = renderProps(DocSpacer, { children: null })
-    expect(result.tag).toBe('div')
+    expect(result.as).toBe('div')
   })
 
   it('defaults height to 16', async () => {
@@ -399,7 +399,7 @@ describe('DocSection attrs', () => {
   it('sets tag to div', async () => {
     const DocSection = (await import('../primitives/DocSection')).default
     const result = renderProps(DocSection, { children: null })
-    expect(result.tag).toBe('div')
+    expect(result.as).toBe('div')
   })
 
   it('defaults direction to column', async () => {
@@ -445,7 +445,7 @@ describe('DocumentPreview attrs', () => {
   it('sets tag to div', async () => {
     const DocumentPreview = (await import('../DocumentPreview')).default
     const result = renderProps(DocumentPreview, { children: null })
-    expect(result.tag).toBe('div')
+    expect(result.as).toBe('div')
   })
 
   it('defaults size to A4 when not provided', async () => {

package/src/__tests__/useDocumentExport.test.ts

@@ -1,17 +1,22 @@
 import type { DocumentMarker } from '@pyreon/connector-document'
+import { h } from '@pyreon/core'
 import { describe, expect, it } from 'vitest'
 import { createDocumentExport, extractDocNode } from '../useDocumentExport'
 
-// Mock VNode
-const vnode = (
+// Helper: build a real VNode via @pyreon/core's h(). The third arg
+// is an array (kept for parity with the prior mock helper) and
+// spreads into h()'s varargs. Tests run real-h() VNodes through
+// the export pipeline — the mock-only path masked PR #197's silent
+// metadata drop bug.
+const node = (
   type: string | ((...args: any[]) => any),
   props: Record<string, any> = {},
   children: unknown[] = [],
-) => ({ type, props, children })
+) => h(type as any, props, ...(children as any[])) as any
 
-// Mock document-marked component
+// Document-marked component
 const docComponent = (docType: string) => {
-  const fn = (props: any) => vnode('div', props, props.children ? [props.children] : [])
+  const fn = (props: any) => node('div', props, props.children ? [props.children] : [])
   ;(fn as any)._documentType = docType
   return fn as ((...args: any[]) => any) & DocumentMarker
 }
@@ -23,8 +28,8 @@ const DocText = docComponent('text')
 describe('createDocumentExport', () => {
   it('extracts a document tree from template function', () => {
     const doc = createDocumentExport(() =>
-      vnode(DocDocument, { _documentProps: { title: 'Test' } }, [
-        vnode(
+      node(DocDocument, { _documentProps: { title: 'Test' } }, [
+        node(
           DocHeading,
           {
             $rocketstyle: { fontSize: 24, fontWeight: 'bold' },
@@ -32,7 +37,7 @@ describe('createDocumentExport', () => {
           },
           ['Hello'],
         ),
-        vnode(
+        node(
           DocText,
           {
             $rocketstyle: { fontSize: 14, color: '#333' },
@@ -62,7 +67,7 @@ describe('createDocumentExport', () => {
 
   it('can be called multiple times', () => {
     const doc = createDocumentExport(() =>
-      vnode(DocText, { $rocketstyle: { fontSize: 14 } }, ['Static']),
+      node(DocText, { $rocketstyle: { fontSize: 14 } }, ['Static']),
     )
 
     const tree1 = doc.getDocNode()
@@ -74,7 +79,7 @@ describe('createDocumentExport', () => {
 
   it('respects includeStyles option', () => {
     const doc = createDocumentExport(
-      () => vnode(DocHeading, { $rocketstyle: { fontSize: 24 } }, ['Hello']),
+      () => node(DocHeading, { $rocketstyle: { fontSize: 24 } }, ['Hello']),
       { includeStyles: false },
     )
 
@@ -99,8 +104,8 @@ describe('extractDocNode (one-step alias)', () => {
 
   it('extracts a tree from a template function in one call', () => {
     const tree = extractDocNode(() =>
-      vnode(DocDocument, { _documentProps: { title: 'Test' } }, [
-        vnode(DocHeading, { _documentProps: { level: 1 } }, ['Hello']),
+      node(DocDocument, { _documentProps: { title: 'Test' } }, [
+        node(DocHeading, { _documentProps: { level: 1 } }, ['Hello']),
       ]),
     )
 
@@ -114,7 +119,7 @@ describe('extractDocNode (one-step alias)', () => {
 
   it('respects extraction options (includeStyles: false)', () => {
     const tree = extractDocNode(
-      () => vnode(DocHeading, { $rocketstyle: { fontSize: 24 } }, ['Hello']),
+      () => node(DocHeading, { $rocketstyle: { fontSize: 24 } }, ['Hello']),
       { includeStyles: false },
     )
     expect(tree.styles).toBeUndefined()
@@ -125,7 +130,7 @@ describe('extractDocNode (one-step alias)', () => {
     // one-step form internally, so they MUST produce identical
     // output for identical input.
     const template = () =>
-      vnode(DocText, { $rocketstyle: { fontSize: 14 } }, ['Hello'])
+      node(DocText, { $rocketstyle: { fontSize: 14 } }, ['Hello'])
 
     const oneStep = extractDocNode(template)
     const twoStep = createDocumentExport(template).getDocNode()
@@ -150,12 +155,12 @@ describe('extractDocNode (one-step alias)', () => {
     // catch the regression. The three extractions should be deeply
     // equal under any change to the primitive's setup path.
     const template = () =>
-      vnode(
+      node(
         DocDocument,
         { _documentProps: { title: 'Idempotent', author: 'Test' } },
         [
-          vnode(DocHeading, { _documentProps: { level: 1 } }, ['Hello']),
-          vnode(DocText, { $rocketstyle: { fontSize: 14 } }, ['World']),
+          node(DocHeading, { _documentProps: { level: 1 } }, ['Hello']),
+          node(DocText, { $rocketstyle: { fontSize: 14 } }, ['World']),
         ],
       )
 
@@ -174,7 +179,7 @@ describe('extractDocNode (one-step alias)', () => {
     // value of any accessor in _documentProps.
     let counter = 0
     const template = () =>
-      vnode(
+      node(
         DocDocument,
         {
           _documentProps: {
@@ -312,4 +317,50 @@ describe('DocDocument reactive metadata (D1 integration)', () => {
       cleanup()
     }
   })
+
+  it('extractDocNode does NOT invoke a real DocDocument component (T3.1 Path C)', { timeout: 30_000 }, async () => {
+    // The architectural invariant locked in by T3.1 (PR #321):
+    // `extractDocumentTree` consumes a real rocketstyle primitive's
+    // `__rs_attrs` chain directly, never invoking the wrapped component
+    // function. The previous Path B workaround had to call the full
+    // styled wrapper per export to read post-attrs `_documentProps`.
+    //
+    // Spy mechanism: wrap the imported component in a Proxy whose
+    // `apply` trap counts function-call invocations. Property reads
+    // (IS_ROCKETSTYLE, __rs_attrs, _documentType, .meta, .displayName,
+    // etc.) flow through to the original via the default `get` handler,
+    // so extractDocumentTree's contract still works — but any code
+    // path that CALLS the spied component bumps the counter.
+    //
+    // The connector-document tests already pin this with a
+    // `FakeRocketDoc` fixture; this test pins it for a real
+    // rocketstyle primitive end-to-end, closing the abstract /
+    // concrete coverage gap.
+    const { initTestConfig } = await import('@pyreon/test-utils')
+    const { h } = await import('@pyreon/core')
+    const cleanup = initTestConfig()
+    try {
+      const RealDocDocument = (await import('../primitives/DocDocument')).default
+
+      let callCount = 0
+      const SpiedDoc = new Proxy(RealDocDocument, {
+        apply(target, thisArg, args) {
+          callCount++
+          return Reflect.apply(target as (...a: unknown[]) => unknown, thisArg, args as unknown[])
+        },
+      })
+
+      const tree = extractDocNode(() =>
+        h(SpiedDoc as never, { title: 'Hoisted', author: 'Aisha' } as never),
+      )
+
+      expect(tree.type).toBe('document')
+      expect(tree.props.title).toBe('Hoisted')
+      expect(tree.props.author).toBe('Aisha')
+      // The architectural assertion — the styled wrapper must NOT run.
+      expect(callCount).toBe(0)
+    } finally {
+      cleanup()
+    }
+  })
 })