Merge pull request #617 from dmnyc/fix/note-content-newlines

fix(notes): preserve authored newlines in quoted posts

Author: spe1020

package.json

@@ -1,7 +1,7 @@
 {
   "name": "zap.cooking",
   "license": "MIT",
-  "version": "4.2.695",
+  "version": "4.2.696",
   "private": true,
   "scripts": {
     "dev": "vite dev",

src/components/FoodstrFeedOptimized.svelte

@@ -62,6 +62,7 @@
     type ReferencedNote
   } from '$lib/shareNoteImage';
   import { optimizeImageUrl, getOptimalFormat } from '$lib/imageOptimizer';
+  import { stripQuotedNoteReferences } from '$lib/feed/noteContent';
   import { compressedCacheManager, COMPRESSED_FEED_CACHE_CONFIG } from '$lib/compressedCache';
   import FeedErrorBoundary from './FeedErrorBoundary.svelte';
   import FeedPostSkeleton from './FeedPostSkeleton.svelte';
@@ -1007,14 +1008,7 @@
   // Get content without the quoted note reference (so it doesn't render as inline embed)
   function getContentWithoutQuote(content: string): string {
     try {
-      if (!content) return '';
-      return content
-        .replace(
-          /nostr:(nevent1[023456789acdefghjklmnpqrstuvwxyz]+|note1[023456789acdefghjklmnpqrstuvwxyz]+)/g,
-          ''
-        )
-        .replace(/\s+/g, ' ')
-        .trim();
+      return stripQuotedNoteReferences(content);
     } catch {
       return content || '';
     }

src/lib/feed/noteContent.test.ts

@@ -0,0 +1,29 @@
+import { describe, expect, it } from 'vitest';
+import { stripQuotedNoteReferences } from './noteContent';
+
+const NEVENT =
+  'nostr:nevent1qvzqqqqqqypzqvv660neqc6dh6r0zndec2v4kfhw833z30j4lzwycll2ntxqr4g2qy88wumn8ghj7mn0wvhxcmmv9uq3wamnwvaz7tmjv4kxz7fwwpexjmtpdshxuet59uqzpwelkuhmeara2plp6xpapjk3m9hvzzsapqtfq2wmfjk6k7euzjdscjpx7f';
+
+describe('stripQuotedNoteReferences', () => {
+  it('preserves paragraphs and profile mentions when removing a quoted event', () => {
+    const content =
+      'What do you call a sauce recipe on nostr:npub1xxdd8eusvdxmaph3fkuu9x2mymhrcc3ghe2l38zv0l4f4nqp659qskkt7a? \n\nOpen sauce. 🍝\n' +
+      NEVENT;
+
+    expect(stripQuotedNoteReferences(content)).toBe(
+      'What do you call a sauce recipe on nostr:npub1xxdd8eusvdxmaph3fkuu9x2mymhrcc3ghe2l38zv0l4f4nqp659qskkt7a? \n\nOpen sauce. 🍝'
+    );
+  });
+
+  it('preserves line breaks after a leading quote reference', () => {
+    expect(stripQuotedNoteReferences(`${NEVENT}\nFirst paragraph\n\nSecond paragraph`)).toBe(
+      'First paragraph\n\nSecond paragraph'
+    );
+  });
+
+  it('removes note references without changing ordinary spacing', () => {
+    const note = `nostr:note1${'q'.repeat(58)}`;
+
+    expect(stripQuotedNoteReferences(`Keep  these words\n${note}`)).toBe('Keep  these words');
+  });
+});

src/lib/feed/noteContent.ts

@@ -0,0 +1,9 @@
+const QUOTED_NOTE_REFERENCE =
+  /nostr:(?:nevent1[023456789acdefghjklmnpqrstuvwxyz]+|note1[023456789acdefghjklmnpqrstuvwxyz]+)/g;
+
+/** Remove embedded quote references without collapsing the author's line breaks. */
+export function stripQuotedNoteReferences(content: string): string {
+  if (!content) return '';
+
+  return content.replace(QUOTED_NOTE_REFERENCE, '').trim();
+}