fix(editor): canonicalize pasted profile urls

Author: ion-lion

frontend/packages/client/src/hm-resolver.test.ts

@@ -0,0 +1,41 @@
+import {describe, expect, it, vi} from 'vitest'
+import {resolveHypermediaUrl} from './hm-resolver'
+
+const PROFILE_URL = 'https://hyper.media/hm/z6MkwRWdU6HY11Qzi9SQZrwSx3rxFw94ipYyiN7Dtbrcg2yU/:profile'
+
+describe('resolveHypermediaUrl', () => {
+  it('resolves public gateway profile URLs locally', async () => {
+    const fetchMock = vi.fn()
+    vi.stubGlobal('fetch', fetchMock)
+
+    const resolved = await resolveHypermediaUrl(PROFILE_URL)
+
+    expect(fetchMock).not.toHaveBeenCalled()
+    expect(resolved?.id).toBe('hm://z6MkwRWdU6HY11Qzi9SQZrwSx3rxFw94ipYyiN7Dtbrcg2yU/:profile')
+    expect(resolved?.hmId).toMatchObject({
+      uid: 'z6MkwRWdU6HY11Qzi9SQZrwSx3rxFw94ipYyiN7Dtbrcg2yU',
+      path: [':profile'],
+      hostname: 'hyper.media',
+      latest: true,
+    })
+  })
+
+  it('resolves public gateway document URLs locally', async () => {
+    const fetchMock = vi.fn()
+    vi.stubGlobal('fetch', fetchMock)
+
+    const resolved = await resolveHypermediaUrl(
+      'https://hyper.media/hm/z6MkwRWdU6HY11Qzi9SQZrwSx3rxFw94ipYyiN7Dtbrcg2yU/docs/example?v=bafyDoc#block1',
+    )
+
+    expect(fetchMock).not.toHaveBeenCalled()
+    expect(resolved?.id).toBe('hm://z6MkwRWdU6HY11Qzi9SQZrwSx3rxFw94ipYyiN7Dtbrcg2yU/docs/example')
+    expect(resolved?.hmId).toMatchObject({
+      uid: 'z6MkwRWdU6HY11Qzi9SQZrwSx3rxFw94ipYyiN7Dtbrcg2yU',
+      path: ['docs', 'example'],
+      version: 'bafyDoc',
+      blockRef: 'block1',
+      latest: false,
+    })
+  })
+})

frontend/packages/client/src/hm-resolver.ts

@@ -40,12 +40,14 @@ export async function resolveHypermediaUrl(url: string, opts?: ResolveOptions):
   let blockRange: {start: number; end: number} | {expanded: boolean} | null = null
   let panel: string | null = null
   let parsedHostname: string | null = null
+  let directHmId: UnpackedHypermediaId | null = null
   try {
     const parsedUrl = new URL(url)
     parsedHostname = parsedUrl.hostname
     const hasVersion = parsedUrl.searchParams.has('v')
     const hasLatest = parsedUrl.searchParams.has('l')
     panel = parsedUrl.searchParams.get('panel')
+    directHmId = parsedUrl.pathname.startsWith('/hm/') ? unpackHmId(url) : null
 
     // Extract blockRef and blockRange from fragment first
     if (parsedUrl.hash) {
@@ -67,6 +69,23 @@ export async function resolveHypermediaUrl(url: string, opts?: ResolveOptions):
     // If URL parsing fails, continue with defaults
   }
 
+  // Public gateway URLs already contain the canonical hm id in the path.
+  // Resolve them locally so paste/autolink keeps working even when the
+  // gateway does not expose x-hypermedia-* headers for special views such as
+  // /:profile.
+  if (directHmId) {
+    return {
+      id: directHmId.id,
+      hmId: directHmId,
+      version: directHmId.version,
+      title: null,
+      target: null,
+      authors: null,
+      type: null,
+      panel,
+    }
+  }
+
   // Try domain resolver first (fast, cached, works offline).
   if (opts?.domainResolver && parsedHostname) {
     try {

frontend/packages/editor/src/tiptap-extension-link/helpers/pasteHandler.test.ts

@@ -1,5 +1,5 @@
 import {describe, expect, it} from 'vitest'
-import {restoreBlockRangeSuffix} from './pasteHandler'
+import {getPastedHypermediaId, restoreBlockRangeSuffix} from './pasteHandler'
 
 describe('restoreBlockRangeSuffix', () => {
   it('reattaches a [start:end] block range linkifyjs truncates', () => {
@@ -43,3 +43,23 @@ describe('restoreBlockRangeSuffix', () => {
     expect(restoreBlockRangeSuffix(href, full)).toBe('https://site.example/doc#blockId[20:52]')
   })
 })
+
+describe('getPastedHypermediaId', () => {
+  const gwUrl = {get: () => 'http://localhost:56001'} as any
+
+  it('parses public gateway profile URLs independent of the configured gateway', () => {
+    const id = getPastedHypermediaId('https://hyper.media/hm/z6Mktest/:profile', gwUrl)
+    expect(id?.uid).toBe('z6Mktest')
+    expect(id?.path).toEqual([':profile'])
+  })
+
+  it('parses configured gateway profile URLs', () => {
+    const id = getPastedHypermediaId('http://localhost:56001/hm/z6Mktest/:profile', gwUrl)
+    expect(id?.uid).toBe('z6Mktest')
+    expect(id?.path).toEqual([':profile'])
+  })
+
+  it('returns null for regular web URLs', () => {
+    expect(getPastedHypermediaId('https://example.com/hm/z6Mktest/:profile', gwUrl)).toBeNull()
+  })
+})

frontend/packages/editor/src/tiptap-extension-link/helpers/pasteHandler.ts

@@ -186,10 +186,7 @@ export function pasteHandler(options: PasteHandlerOptions): Plugin {
             }
           : null
 
-        const unpackedHmId =
-          isHypermediaScheme(textContent) || isPublicGatewayLink(textContent, options.gwUrl)
-            ? unpackHmId(textContent)
-            : null
+        const unpackedHmId = getPastedHypermediaId(textContent, options.gwUrl)
 
         if (!selection.empty && options.linkOnPaste) {
           const pastedLink = unpackedHmId
@@ -231,7 +228,13 @@ export function pasteHandler(options: PasteHandlerOptions): Plugin {
 
         // If transformPasted already added a link mark, skip further processing
         // unless we have a link that might need special handling (twitter, instagram, video, etc)
-        if (firstChildIsText && firstChildContainsLinkMark && !(link && selection.empty && !unpackedHmId)) {
+        // or a hypermedia URL that should be canonicalized into an hm:// link.
+        if (
+          firstChildIsText &&
+          firstChildContainsLinkMark &&
+          !(link && selection.empty && !unpackedHmId) &&
+          !(selection.empty && unpackedHmId)
+        ) {
           return false
         }
 
@@ -329,6 +332,31 @@ export function pasteHandler(options: PasteHandlerOptions): Plugin {
 
             return true
           }
+
+          const normalizedHmUrl = packHmId(hmId(unpackedHmId.uid, unpackedHmId))
+          view.dispatch(
+            tr.insertText(normalizedHmUrl, pos).addMark(
+              pos,
+              pos + normalizedHmUrl.length,
+              options.editor.schema.mark('link', {
+                href: normalizedHmUrl,
+              }),
+            ),
+          )
+
+          view.dispatch(
+            view.state.tr.scrollIntoView().setMeta(linkMenuPluginKey, {
+              activate: true,
+              ref: normalizedHmUrl,
+              items: getLinkMenuItems({
+                isLoading: false,
+                sourceUrl: normalizedHmUrl,
+                hmId: unpackHmId(normalizedHmUrl),
+                gwUrl: options.gwUrl,
+              }),
+            }),
+          )
+          return true
         }
 
         // Check if the link is hm link or web URL
@@ -673,6 +701,26 @@ export function restoreBlockRangeSuffix(href: string, fullText: string): string
   return match ? href + match[0] : href
 }
 
+export function getPastedHypermediaId(text: string, gwUrl: StateStream<string>): UnpackedHypermediaId | null {
+  const trimmed = text.trim()
+  if (!trimmed) return null
+
+  if (isHypermediaScheme(trimmed) || isPublicGatewayLink(trimmed, gwUrl) || isPublicHypermediaLink(trimmed)) {
+    return unpackHmId(trimmed)
+  }
+
+  return null
+}
+
+function isPublicHypermediaLink(text: string) {
+  try {
+    const url = new URL(text)
+    return (url.protocol === 'https:' || url.protocol === 'http:') && url.hostname === 'hyper.media'
+  } catch {
+    return false
+  }
+}
+
 async function fetchEntityTitle(hmId: UnpackedHypermediaId, grpcClient: GRPCClient, blockRef?: string | null) {
   const document = await grpcClient.documents.getDocument({
     account: hmId.uid,