From 7c79b331f61411166281dc028752807a339c6bec Mon Sep 17 00:00:00 2001
From: Paulo Castellano <paulo@castellanos.llc>
Date: Fri, 17 Jul 2026 16:17:06 -0300
Subject: [PATCH] refactor(posts): drop the frontend URL trim, rely on the
 backend's clean uri

The link-preview endpoint re-runs UrlDetector on whatever it receives and returns the exact, trimmed URL as card.uri, so the composer only needs a rough match to detect and dedup a link. Removes the firstUrl helper's punctuation/paren trimming that mirrored the backend tokenizer, leaving one trivial regex.
---
 resources/js/composables/useLinkCard.ts | 30 ++++++-------------------
 1 file changed, 7 insertions(+), 23 deletions(-)

diff --git a/resources/js/composables/useLinkCard.ts b/resources/js/composables/useLinkCard.ts
index 17f9b541..25dda490 100644
--- a/resources/js/composables/useLinkCard.ts
+++ b/resources/js/composables/useLinkCard.ts
@@ -13,35 +13,19 @@ export interface LinkCard {
 }
 
 /**
- * The first http(s) URL in `text`, or null. Trailing sentence punctuation and an
- * unmatched closing paren are trimmed so the detected URL matches the link's
- * rendered span — this mirrors the backend `UrlDetector`, keeping the preview
- * card's URL identical to the one the post actually links to.
- */
-const firstUrl = (text: string): string | null => {
-    const match = text.match(/https?:\/\/[^\s]+/);
-
-    if (!match) {
-        return null;
-    }
-
-    const url = match[0].replace(/[.,;:!?]$/, '');
-
-    return url.endsWith(')') && !url.includes('(') ? url.slice(0, -1) : url;
-};
-
-/**
- * Live link-preview card for the composer. Resolves the first link in `content`
- * to its OpenGraph card, but only when there is no attached media (media
- * suppresses the link card on every platform). Returns the card and a loading
- * flag for the editor previews to render.
+ * Live link-preview card for the composer. Detects the first link in `content`
+ * — a rough match is enough, because the backend re-detects it and returns the
+ * exact, trimmed URL as `card.uri` — and resolves its OpenGraph card, but only
+ * when no media is attached (media suppresses the link card on every platform).
  */
 export const useLinkCard = (content: Ref<string>, media: Ref<MediaItem[]>) => {
     const card = ref<LinkCard | null>(null);
     const loading = ref(false);
     const http = useHttp<{ url: string }, LinkCard | null>({ url: '' });
 
-    const url = computed(() => (media.value.length > 0 ? null : firstUrl(content.value)));
+    const url = computed(() =>
+        media.value.length > 0 ? null : (content.value.match(/https?:\/\/\S+/)?.[0] ?? null),
+    );
 
     const fetchCard = async (target: string): Promise<void> => {
         loading.value = true;