@pyreon/connector-document 0.24.5 → 0.24.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pyreon/connector-document",
-  "version": "0.24.5",
+  "version": "0.24.6",
   "description": "Bridge between @pyreon/pyreon styled components and @pyreon/document rendering",
   "license": "MIT",
   "repository": {
@@ -13,8 +13,7 @@
     "!lib/**/*.map",
     "!lib/analysis",
     "README.md",
-    "LICENSE",
-    "src"
+    "LICENSE"
   ],
   "type": "module",
   "sideEffects": false,
@@ -22,7 +21,6 @@
   "types": "./lib/index.d.ts",
   "exports": {
     ".": {
-      "bun": "./src/index.ts",
       "import": "./lib/index.js",
       "types": "./lib/index.d.ts"
     }
@@ -42,11 +40,11 @@
     "typecheck": "tsc --noEmit"
   },
   "devDependencies": {
-    "@pyreon/core": "^0.24.5",
-    "@pyreon/document": "^0.24.5",
-    "@pyreon/reactivity": "^0.24.5",
+    "@pyreon/core": "^0.24.6",
+    "@pyreon/document": "^0.24.6",
+    "@pyreon/reactivity": "^0.24.6",
     "@pyreon/test-utils": "^0.13.11",
-    "@pyreon/typescript": "^0.24.5",
+    "@pyreon/typescript": "^0.24.6",
     "@vitest/browser-playwright": "^4.1.4",
     "@vitus-labs/tools-rolldown": "^2.4.0"
   },
@@ -54,7 +52,7 @@
     "node": ">= 22"
   },
   "dependencies": {
-    "@pyreon/core": "^0.24.5",
-    "@pyreon/document": "^0.24.5"
+    "@pyreon/core": "^0.24.6",
+    "@pyreon/document": "^0.24.6"
   }
 }

package/src/__tests__/connector-document.browser.test.tsx

@@ -1,146 +0,0 @@
-/** @jsxImportSource @pyreon/core */
-import { h } from '@pyreon/core'
-import { signal } from '@pyreon/reactivity'
-import { afterEach, describe, expect, it } from 'vitest'
-import { extractDocumentTree } from '../extractDocumentTree'
-import { resolveStyles } from '../resolveStyles'
-
-// Real-Chromium smoke for @pyreon/connector-document.
-//
-// The pre-existing unit tests construct mock vnode objects by hand
-// (`{ type, props, children }`) — fast but they bypass the real
-// `@pyreon/core` `h()` runtime. PR #197 fixed a class of bug that
-// hand-mock tests can't catch: `_documentProps` only appears on a
-// vnode AFTER rocketstyle's `.attrs()` HOC runs, so the JSX
-// vnode (what `h()` returns) only carries USER-provided props.
-//
-// This suite builds vnodes with the real `h()` runtime and verifies:
-//   1. Path A: a documentType component with `_documentProps` already
-//      set on the JSX vnode extracts cleanly.
-//   2. Path B: `extractDocumentTree` invokes the component and reads
-//      `_documentProps` from the post-call vnode (the rocketstyle
-//      attrs-style path) — proving the workaround documented in the
-//      source still functions in a real browser.
-//   3. Function-valued `_documentProps` (reactive accessors) are
-//      resolved to LIVE values at extraction time — covers the
-//      "metadata reads current signal" path used by document-primitives.
-//   4. resolveStyles converts rocketstyle theme objects to plain
-//      style records in the browser bundle (no Node-only dep leaks).
-
-describe('@pyreon/connector-document in real browser', () => {
-  afterEach(() => {
-    // No globals to reset — extractDocumentTree is pure.
-  })
-
-  it('Path A — reads _documentProps off the JSX vnode WITHOUT invoking the component', () => {
-    // Strict Path A: the component function itself must NEVER touch
-    // `_documentProps` — `_documentProps` is supplied directly on the
-    // JSX vnode props. To prove the extractor takes the fast path and
-    // doesn't call the component, we install a throwing body: if
-    // extractNode falls through to Path B (invoking the component),
-    // the test fails with the throw. Passing proves Path A is taken.
-    const Heading = (_props: any) => {
-      throw new Error('Path A must not invoke the component')
-    }
-    ;(Heading as any)._documentType = 'heading'
-
-    const tree = h(
-      Heading,
-      { _documentProps: { level: 1 }, $rocketstyle: { color: '#000' } },
-      'Hi',
-    )
-    const result = extractDocumentTree(tree)
-
-    expect(result.type).toBe('heading')
-    expect(result.props).toEqual({ level: 1 })
-    expect(result.children[0]).toBe('Hi')
-    expect(result.styles?.color).toBe('#000')
-  })
-
-  it('Path B — invokes the component to recover _documentProps from the post-call vnode', () => {
-    // Simulates the rocketstyle attrs-HOC pattern: USER passes
-    // `{ title, author }` as JSX props; the component itself
-    // injects `_documentProps` into its return vnode via h().
-    const DocDocument = (props: any) =>
-      h(
-        'div',
-        {
-          // Post-attrs props — never visible on the JSX vnode itself.
-          _documentProps: {
-            title: props.title,
-            author: props.author,
-          },
-        },
-        props.children,
-      )
-    ;(DocDocument as any)._documentType = 'document'
-
-    const tree = h(DocDocument, { title: 'Manual', author: 'Vít' }, 'body')
-    const result = extractDocumentTree(tree)
-
-    expect(result.type).toBe('document')
-    expect(result.props).toEqual({ title: 'Manual', author: 'Vít' })
-  })
-
-  it('resolves function-valued _documentProps to LIVE values at extraction time', () => {
-    const titleSig = signal('Initial')
-    const DocDocument = (props: any) =>
-      h(
-        'div',
-        {
-          _documentProps: {
-            // Accessor — extractor must call this to read the live signal.
-            title: props.title,
-          },
-        },
-        props.children,
-      )
-    ;(DocDocument as any)._documentType = 'document'
-
-    const tree = h(DocDocument, { title: () => titleSig() }, 'x')
-
-    const before = extractDocumentTree(tree)
-    expect(before.props).toEqual({ title: 'Initial' })
-
-    titleSig.set('Updated')
-    const after = extractDocumentTree(tree)
-    // Same vnode, same accessor — but it now reads the new signal value.
-    expect(after.props).toEqual({ title: 'Updated' })
-  })
-
-  it('flattens transparent (non-documentType) wrappers built with h()', () => {
-    const Section = (props: any) => h('div', props, props.children)
-    ;(Section as any)._documentType = 'section'
-    const Text = (props: any) => h('span', props, props.children)
-    ;(Text as any)._documentType = 'text'
-
-    const tree = h(
-      Section,
-      null,
-      // Plain `h('div')` wrapper has no _documentType — should be
-      // flattened, leaving the inner Text directly under Section.
-      h('div', null, h(Text, null, 'inner')),
-    )
-    const result = extractDocumentTree(tree)
-    expect(result.type).toBe('section')
-    expect(result.children).toHaveLength(1)
-    expect((result.children[0] as any).type).toBe('text')
-    expect((result.children[0] as any).children[0]).toBe('inner')
-  })
-
-  it('resolveStyles produces a plain style record in the browser bundle', () => {
-    const styles = resolveStyles(
-      {
-        color: '#fff',
-        backgroundColor: '#0070f3',
-        fontSize: 24,
-        padding: '8px 16px',
-      },
-      16,
-    )
-    expect(styles.color).toBe('#fff')
-    expect(styles.backgroundColor).toBe('#0070f3')
-    expect(styles.fontSize).toBe(24)
-    expect(styles.padding).toEqual([8, 16])
-  })
-})

package/src/__tests__/cssValueParser.test.ts

@@ -1,136 +0,0 @@
-import { describe, expect, it } from 'vitest'
-import {
-  parseBoxModel,
-  parseCssDimension,
-  parseFontWeight,
-  parseLineHeight,
-} from '../cssValueParser'
-
-describe('parseCssDimension', () => {
-  it('passes through numbers', () => {
-    expect(parseCssDimension(14)).toBe(14)
-    expect(parseCssDimension(0)).toBe(0)
-    expect(parseCssDimension(-5)).toBe(-5)
-    expect(parseCssDimension(1.5)).toBe(1.5)
-  })
-
-  it('parses px values', () => {
-    expect(parseCssDimension('14px')).toBe(14)
-    expect(parseCssDimension('0px')).toBe(0)
-    expect(parseCssDimension('-5px')).toBe(-5)
-    expect(parseCssDimension('1.5px')).toBe(1.5)
-  })
-
-  it('parses rem values', () => {
-    expect(parseCssDimension('1rem')).toBe(16)
-    expect(parseCssDimension('1.5rem')).toBe(24)
-    expect(parseCssDimension('0.5rem')).toBe(8)
-    expect(parseCssDimension('2rem', 20)).toBe(40)
-  })
-
-  it('parses em values', () => {
-    expect(parseCssDimension('1em')).toBe(16)
-    expect(parseCssDimension('2em')).toBe(32)
-  })
-
-  it('parses pt values', () => {
-    expect(parseCssDimension('12pt')).toBeCloseTo(16)
-    expect(parseCssDimension('9pt')).toBeCloseTo(12)
-  })
-
-  it('parses bare number strings', () => {
-    expect(parseCssDimension('14')).toBe(14)
-    expect(parseCssDimension('0')).toBe(0)
-    expect(parseCssDimension('-5')).toBe(-5)
-  })
-
-  it('returns undefined for unresolvable values', () => {
-    expect(parseCssDimension('auto')).toBeUndefined()
-    expect(parseCssDimension('100%')).toBeUndefined()
-    expect(parseCssDimension('calc(100% - 20px)')).toBeUndefined()
-    expect(parseCssDimension('var(--spacing)')).toBeUndefined()
-  })
-
-  it('trims whitespace', () => {
-    expect(parseCssDimension(' 14px ')).toBe(14)
-    expect(parseCssDimension('  1rem  ')).toBe(16)
-  })
-})
-
-describe('parseBoxModel', () => {
-  it('returns undefined for null/undefined', () => {
-    expect(parseBoxModel(undefined)).toBeUndefined()
-    expect(parseBoxModel(null as any)).toBeUndefined()
-  })
-
-  it('passes through numbers', () => {
-    expect(parseBoxModel(8)).toBe(8)
-    expect(parseBoxModel(0)).toBe(0)
-  })
-
-  it('parses single value', () => {
-    expect(parseBoxModel('8px')).toBe(8)
-    expect(parseBoxModel('1rem')).toBe(16)
-  })
-
-  it('parses two values (vertical horizontal)', () => {
-    expect(parseBoxModel('8px 16px')).toEqual([8, 16])
-  })
-
-  it('parses three values (top horizontal bottom)', () => {
-    expect(parseBoxModel('8px 16px 12px')).toEqual([8, 16, 12, 16])
-  })
-
-  it('parses four values', () => {
-    expect(parseBoxModel('8px 16px 12px 4px')).toEqual([8, 16, 12, 4])
-  })
-
-  it('returns undefined when any part is unresolvable', () => {
-    expect(parseBoxModel('8px auto')).toBeUndefined()
-    expect(parseBoxModel('8px 16px auto 4px')).toBeUndefined()
-  })
-})
-
-describe('parseFontWeight', () => {
-  it('passes through numbers', () => {
-    expect(parseFontWeight(400)).toBe(400)
-    expect(parseFontWeight(700)).toBe(700)
-  })
-
-  it('handles string keywords', () => {
-    expect(parseFontWeight('normal')).toBe('normal')
-    expect(parseFontWeight('bold')).toBe('bold')
-  })
-
-  it('parses numeric strings', () => {
-    expect(parseFontWeight('400')).toBe(400)
-    expect(parseFontWeight('700')).toBe(700)
-  })
-
-  it('returns undefined for null/undefined', () => {
-    expect(parseFontWeight(undefined)).toBeUndefined()
-  })
-
-  it('returns undefined for unrecognized values', () => {
-    expect(parseFontWeight('lighter')).toBeUndefined()
-  })
-})
-
-describe('parseLineHeight', () => {
-  it('passes through numbers (unitless)', () => {
-    expect(parseLineHeight(1.5)).toBe(1.5)
-    expect(parseLineHeight(2)).toBe(2)
-  })
-
-  it('parses px values', () => {
-    expect(parseLineHeight('24px')).toBe(24)
-  })
-
-  it("returns undefined for 'normal'", () => {
-    expect(parseLineHeight('normal')).toBeUndefined()
-  })
-
-  it('returns undefined for null/undefined', () => {
-    expect(parseLineHeight(undefined)).toBeUndefined()
-  })
-})