@puredesktop/puredesktop-ui-bridge 1.0.0-beta.1 → 2.1.0

package/src/bridge/collectionDocumentHtml.test.ts

@@ -1,71 +1,71 @@
-import { describe, expect, it, vi } from 'vitest'
-import {
-  absoluteCollectionAssetPath,
-  COLLECTION_ASSET_SRC_ATTR,
-  normalizeCollectionDocumentHtml,
-  prepareCollectionDocumentHtml,
-  relativeAssetSrcForHtml,
-  relativizeCollectionAssetPath,
-} from './collectionDocumentHtml.js'
-
-describe('collectionDocumentHtml', () => {
-  const collectionPath = '/home/user/MyBook.book'
-
-  it('resolves relative and in-collection absolute paths', () => {
-    expect(relativeAssetSrcForHtml('assets/fig.png', collectionPath)).toBe(
-      'assets/fig.png',
-    )
-    expect(
-      relativeAssetSrcForHtml(
-        `${collectionPath}/assets/fig.png`,
-        collectionPath,
-      ),
-    ).toBe('assets/fig.png')
-    expect(
-      relativeAssetSrcForHtml('https://example.com/x.png', collectionPath),
-    ).toBe(null)
-  })
-
-  it('relativizes absolute paths under the collection root', () => {
-    expect(
-      relativizeCollectionAssetPath(
-        `${collectionPath}/assets/fig.png`,
-        collectionPath,
-      ),
-    ).toBe('assets/fig.png')
-    expect(
-      relativizeCollectionAssetPath('/tmp/other.png', collectionPath),
-    ).toBeNull()
-  })
-
-  it('builds absolute asset paths', () => {
-    expect(absoluteCollectionAssetPath(collectionPath, 'assets/fig.png')).toBe(
-      `${collectionPath}/assets/fig.png`,
-    )
-  })
-
-  it('prepareCollectionDocumentHtml injects data URLs and storage attribute', async () => {
-    const html = '<p><img src="assets/fig.png" alt="fig"></p>'
-    const readBinary = vi.fn().mockResolvedValue({
-      mimeType: 'image/png',
-      base64: 'abc',
-    })
-
-    const display = await prepareCollectionDocumentHtml(
-      html,
-      collectionPath,
-      readBinary,
-    )
-    expect(readBinary).toHaveBeenCalledWith(`${collectionPath}/assets/fig.png`)
-    expect(display).toContain('data:image/png;base64,abc')
-    expect(display).toContain(`${COLLECTION_ASSET_SRC_ATTR}="assets/fig.png"`)
-  })
-
-  it('normalizeCollectionDocumentHtml restores relative src from storage attribute', () => {
-    const html = `<p><img src="data:image/png;base64,abc" ${COLLECTION_ASSET_SRC_ATTR}="assets/fig.png"></p>`
-    const stored = normalizeCollectionDocumentHtml(html, collectionPath)
-    expect(stored).toContain('src="assets/fig.png"')
-    expect(stored).not.toContain(COLLECTION_ASSET_SRC_ATTR)
-    expect(stored).not.toContain('data:image/png')
-  })
-})
+import { describe, expect, it, vi } from 'vitest'
+import {
+  absoluteCollectionAssetPath,
+  COLLECTION_ASSET_SRC_ATTR,
+  normalizeCollectionDocumentHtml,
+  prepareCollectionDocumentHtml,
+  relativeAssetSrcForHtml,
+  relativizeCollectionAssetPath,
+} from './collectionDocumentHtml.js'
+
+describe('collectionDocumentHtml', () => {
+  const collectionPath = '/home/user/MyBook.book'
+
+  it('resolves relative and in-collection absolute paths', () => {
+    expect(relativeAssetSrcForHtml('assets/fig.png', collectionPath)).toBe(
+      'assets/fig.png',
+    )
+    expect(
+      relativeAssetSrcForHtml(
+        `${collectionPath}/assets/fig.png`,
+        collectionPath,
+      ),
+    ).toBe('assets/fig.png')
+    expect(
+      relativeAssetSrcForHtml('https://example.com/x.png', collectionPath),
+    ).toBe(null)
+  })
+
+  it('relativizes absolute paths under the collection root', () => {
+    expect(
+      relativizeCollectionAssetPath(
+        `${collectionPath}/assets/fig.png`,
+        collectionPath,
+      ),
+    ).toBe('assets/fig.png')
+    expect(
+      relativizeCollectionAssetPath('/tmp/other.png', collectionPath),
+    ).toBeNull()
+  })
+
+  it('builds absolute asset paths', () => {
+    expect(absoluteCollectionAssetPath(collectionPath, 'assets/fig.png')).toBe(
+      `${collectionPath}/assets/fig.png`,
+    )
+  })
+
+  it('prepareCollectionDocumentHtml injects data URLs and storage attribute', async () => {
+    const html = '<p><img src="assets/fig.png" alt="fig"></p>'
+    const readBinary = vi.fn().mockResolvedValue({
+      mimeType: 'image/png',
+      base64: 'abc',
+    })
+
+    const display = await prepareCollectionDocumentHtml(
+      html,
+      collectionPath,
+      readBinary,
+    )
+    expect(readBinary).toHaveBeenCalledWith(`${collectionPath}/assets/fig.png`)
+    expect(display).toContain('data:image/png;base64,abc')
+    expect(display).toContain(`${COLLECTION_ASSET_SRC_ATTR}="assets/fig.png"`)
+  })
+
+  it('normalizeCollectionDocumentHtml restores relative src from storage attribute', () => {
+    const html = `<p><img src="data:image/png;base64,abc" ${COLLECTION_ASSET_SRC_ATTR}="assets/fig.png"></p>`
+    const stored = normalizeCollectionDocumentHtml(html, collectionPath)
+    expect(stored).toContain('src="assets/fig.png"')
+    expect(stored).not.toContain(COLLECTION_ASSET_SRC_ATTR)
+    expect(stored).not.toContain('data:image/png')
+  })
+})

package/src/bridge/collectionDocumentHtml.ts

@@ -1,118 +1,118 @@
-import { COLLECTION_ASSET_SRC_ATTR } from '@purescience/platform-editor'
-
-export { COLLECTION_ASSET_SRC_ATTR }
-
-export function isDisplayableAbsoluteSource(src: string): boolean {
-  return (
-    src.startsWith('http://') ||
-    src.startsWith('https://') ||
-    src.startsWith('data:') ||
-    src.startsWith('blob:')
-  )
-}
-
-export function absoluteCollectionAssetPath(
-  collectionPath: string,
-  relativePath: string,
-): string {
-  const clean = relativePath.replace(/\\/g, '/').replace(/^\/+/, '')
-  return `${collectionPath.replace(/\/+$/, '')}/${clean}`
-}
-
-export function relativizeCollectionAssetPath(
-  absolutePath: string,
-  collectionPath: string,
-): string | null {
-  const normalizedAbsolute = absolutePath.replace(/\\/g, '/')
-  const normalizedRoot = collectionPath.replace(/\/+$/, '')
-  if (!normalizedAbsolute.startsWith(`${normalizedRoot}/`)) return null
-  return normalizedAbsolute.slice(normalizedRoot.length + 1)
-}
-
-/** Relative path stored in HTML, or null if src should be left unchanged. */
-export function relativeAssetSrcForHtml(
-  src: string,
-  collectionPath: string,
-): string | null {
-  const trimmed = src.trim()
-  if (!trimmed || isDisplayableAbsoluteSource(trimmed)) return null
-  if (trimmed.startsWith('/')) {
-    return relativizeCollectionAssetPath(trimmed, collectionPath)
-  }
-  return trimmed.replace(/\\/g, '/').replace(/^\/+/, '')
-}
-
-export type ReadCollectionBinary = (
-  absolutePath: string,
-) => Promise<{ mimeType: string; base64: string }>
-
-/** Load relative image paths as data URLs for the iframe editor. */
-export async function prepareCollectionDocumentHtml(
-  html: string,
-  collectionPath: string,
-  readBinary: ReadCollectionBinary,
-): Promise<string> {
-  if (!html.trim() || !collectionPath.trim()) return html
-  const doc = new DOMParser().parseFromString(html, 'text/html')
-  let changed = false
-
-  for (const img of Array.from(doc.querySelectorAll('img'))) {
-    const src = img.getAttribute('src')?.trim()
-    if (!src || isDisplayableAbsoluteSource(src)) continue
-
-    const relative = relativeAssetSrcForHtml(src, collectionPath)
-    if (!relative) continue
-
-    try {
-      const absolute = absoluteCollectionAssetPath(collectionPath, relative)
-      const result = await readBinary(absolute)
-      img.setAttribute('src', `data:${result.mimeType};base64,${result.base64}`)
-      img.setAttribute(COLLECTION_ASSET_SRC_ATTR, relative)
-      changed = true
-    } catch {
-      // Keep relative src when the file is missing.
-    }
-  }
-
-  return changed ? doc.body.innerHTML : html
-}
-
-/** Persist relative `img src` values (strip injected data URLs). */
-export function normalizeCollectionDocumentHtml(
-  html: string,
-  collectionPath: string,
-): string {
-  if (!html.trim() || !collectionPath.trim()) return html
-  const doc = new DOMParser().parseFromString(html, 'text/html')
-  let changed = false
-
-  for (const img of Array.from(doc.querySelectorAll('img'))) {
-    const stored = img.getAttribute(COLLECTION_ASSET_SRC_ATTR)?.trim()
-    if (stored) {
-      img.setAttribute('src', stored)
-      img.removeAttribute(COLLECTION_ASSET_SRC_ATTR)
-      changed = true
-      continue
-    }
-
-    const src = img.getAttribute('src')?.trim()
-    if (!src) continue
-
-    if (src.startsWith('file://')) {
-      try {
-        const relative = relativizeCollectionAssetPath(
-          decodeURIComponent(new URL(src).pathname),
-          collectionPath,
-        )
-        if (relative) {
-          img.setAttribute('src', relative)
-          changed = true
-        }
-      } catch {
-        // ignore malformed file URLs
-      }
-    }
-  }
-
-  return changed ? doc.body.innerHTML : html
-}
+import { COLLECTION_ASSET_SRC_ATTR } from '@purescience/platform-editor'
+
+export { COLLECTION_ASSET_SRC_ATTR }
+
+export function isDisplayableAbsoluteSource(src: string): boolean {
+  return (
+    src.startsWith('http://') ||
+    src.startsWith('https://') ||
+    src.startsWith('data:') ||
+    src.startsWith('blob:')
+  )
+}
+
+export function absoluteCollectionAssetPath(
+  collectionPath: string,
+  relativePath: string,
+): string {
+  const clean = relativePath.replace(/\\/g, '/').replace(/^\/+/, '')
+  return `${collectionPath.replace(/\/+$/, '')}/${clean}`
+}
+
+export function relativizeCollectionAssetPath(
+  absolutePath: string,
+  collectionPath: string,
+): string | null {
+  const normalizedAbsolute = absolutePath.replace(/\\/g, '/')
+  const normalizedRoot = collectionPath.replace(/\/+$/, '')
+  if (!normalizedAbsolute.startsWith(`${normalizedRoot}/`)) return null
+  return normalizedAbsolute.slice(normalizedRoot.length + 1)
+}
+
+/** Relative path stored in HTML, or null if src should be left unchanged. */
+export function relativeAssetSrcForHtml(
+  src: string,
+  collectionPath: string,
+): string | null {
+  const trimmed = src.trim()
+  if (!trimmed || isDisplayableAbsoluteSource(trimmed)) return null
+  if (trimmed.startsWith('/')) {
+    return relativizeCollectionAssetPath(trimmed, collectionPath)
+  }
+  return trimmed.replace(/\\/g, '/').replace(/^\/+/, '')
+}
+
+export type ReadCollectionBinary = (
+  absolutePath: string,
+) => Promise<{ mimeType: string; base64: string }>
+
+/** Load relative image paths as data URLs for the iframe editor. */
+export async function prepareCollectionDocumentHtml(
+  html: string,
+  collectionPath: string,
+  readBinary: ReadCollectionBinary,
+): Promise<string> {
+  if (!html.trim() || !collectionPath.trim()) return html
+  const doc = new DOMParser().parseFromString(html, 'text/html')
+  let changed = false
+
+  for (const img of Array.from(doc.querySelectorAll('img'))) {
+    const src = img.getAttribute('src')?.trim()
+    if (!src || isDisplayableAbsoluteSource(src)) continue
+
+    const relative = relativeAssetSrcForHtml(src, collectionPath)
+    if (!relative) continue
+
+    try {
+      const absolute = absoluteCollectionAssetPath(collectionPath, relative)
+      const result = await readBinary(absolute)
+      img.setAttribute('src', `data:${result.mimeType};base64,${result.base64}`)
+      img.setAttribute(COLLECTION_ASSET_SRC_ATTR, relative)
+      changed = true
+    } catch {
+      // Keep relative src when the file is missing.
+    }
+  }
+
+  return changed ? doc.body.innerHTML : html
+}
+
+/** Persist relative `img src` values (strip injected data URLs). */
+export function normalizeCollectionDocumentHtml(
+  html: string,
+  collectionPath: string,
+): string {
+  if (!html.trim() || !collectionPath.trim()) return html
+  const doc = new DOMParser().parseFromString(html, 'text/html')
+  let changed = false
+
+  for (const img of Array.from(doc.querySelectorAll('img'))) {
+    const stored = img.getAttribute(COLLECTION_ASSET_SRC_ATTR)?.trim()
+    if (stored) {
+      img.setAttribute('src', stored)
+      img.removeAttribute(COLLECTION_ASSET_SRC_ATTR)
+      changed = true
+      continue
+    }
+
+    const src = img.getAttribute('src')?.trim()
+    if (!src) continue
+
+    if (src.startsWith('file://')) {
+      try {
+        const relative = relativizeCollectionAssetPath(
+          decodeURIComponent(new URL(src).pathname),
+          collectionPath,
+        )
+        if (relative) {
+          img.setAttribute('src', relative)
+          changed = true
+        }
+      } catch {
+        // ignore malformed file URLs
+      }
+    }
+  }
+
+  return changed ? doc.body.innerHTML : html
+}

package/src/bridge/collectionImagePaste.test.ts

@@ -1,32 +1,32 @@
-import { describe, expect, it } from 'vitest'
-import {
-  buildPastedFigureRelativePath,
-  normalizeCollectionImageMimeType,
-  stemFromImageFileName,
-} from './collectionImagePaste.js'
-
-describe('collectionImagePaste helpers', () => {
-  it('normalizes mime types from file names', () => {
-    expect(normalizeCollectionImageMimeType(undefined, 'photo.PNG')).toBe(
-      'image/png',
-    )
-    expect(normalizeCollectionImageMimeType('image/jpeg', 'x.bin')).toBe(
-      'image/jpeg',
-    )
-  })
-
-  it('builds stable relative paths under the figures subdir', () => {
-    expect(
-      buildPastedFigureRelativePath(
-        'assets/figures',
-        'My Diagram.png',
-        'image/png',
-        1_700_000_000_000,
-      ),
-    ).toBe('assets/figures/My-Diagram-1700000000000.png')
-  })
-
-  it('derives stems from file names', () => {
-    expect(stemFromImageFileName('figures/My Photo.jpg')).toBe('My-Photo')
-  })
-})
+import { describe, expect, it } from 'vitest'
+import {
+  buildPastedFigureRelativePath,
+  normalizeCollectionImageMimeType,
+  stemFromImageFileName,
+} from './collectionImagePaste.js'
+
+describe('collectionImagePaste helpers', () => {
+  it('normalizes mime types from file names', () => {
+    expect(normalizeCollectionImageMimeType(undefined, 'photo.PNG')).toBe(
+      'image/png',
+    )
+    expect(normalizeCollectionImageMimeType('image/jpeg', 'x.bin')).toBe(
+      'image/jpeg',
+    )
+  })
+
+  it('builds stable relative paths under the figures subdir', () => {
+    expect(
+      buildPastedFigureRelativePath(
+        'assets/figures',
+        'My Diagram.png',
+        'image/png',
+        1_700_000_000_000,
+      ),
+    ).toBe('assets/figures/My-Diagram-1700000000000.png')
+  })
+
+  it('derives stems from file names', () => {
+    expect(stemFromImageFileName('figures/My Photo.jpg')).toBe('My-Photo')
+  })
+})