Add newline character handling for markdown table cells

Co-authored-by: MaestroError <46760939+MaestroError@users.noreply.github.com>

Author: Copilot

src/Services/PropertiesTable.php

@@ -29,9 +29,9 @@ public function convertPropertiesToMarkdownTable(array $properties): string
                 continue;
             }
 
-            // Escape pipe characters in property name and value
-            $escapedName = $this->escapePipeCharacters($name);
-            $escapedValue = $this->escapePipeCharacters($value);
+            // Escape special characters in property name and value
+            $escapedName = $this->escapeTableCellCharacters($name);
+            $escapedValue = $this->escapeTableCellCharacters($value);
 
             $markdown .= "| {$escapedName} | {$escapedValue} |\n";
         }
@@ -295,13 +295,19 @@ private function formatUser(?array $user): string
     }
 
     /**
-     * Escape pipe characters in markdown table cells
+     * Escape special characters in markdown table cells
      *
      * @param  string  $text  Text to escape
      * @return string Escaped text
      */
-    private function escapePipeCharacters(string $text): string
+    private function escapeTableCellCharacters(string $text): string
     {
-        return str_replace('|', '\|', $text);
+        // Escape pipe characters to prevent table structure corruption
+        $text = str_replace('|', '\|', $text);
+
+        // Replace newline characters with spaces to maintain table structure
+        $text = str_replace(["\r\n", "\r", "\n"], ' ', $text);
+
+        return $text;
     }
 }

tests/Services/PropertiesTableTest.php

@@ -653,3 +653,159 @@
 
     expect($result)->toContain('| Calculated | Result \| Text |');
 });
+
+test('replaces newline characters with spaces in property values', function () {
+    $properties = [
+        'Description' => [
+            'id' => 'rich',
+            'type' => 'rich_text',
+            'rich_text' => [
+                ['plain_text' => "Line 1\nLine 2"],
+            ],
+        ],
+    ];
+
+    $result = $this->propertiesTable->convertPropertiesToMarkdownTable($properties);
+
+    expect($result)->toContain('| Description | Line 1 Line 2 |');
+});
+
+test('replaces carriage return with spaces in property values', function () {
+    $properties = [
+        'Description' => [
+            'id' => 'rich',
+            'type' => 'rich_text',
+            'rich_text' => [
+                ['plain_text' => "Line 1\rLine 2"],
+            ],
+        ],
+    ];
+
+    $result = $this->propertiesTable->convertPropertiesToMarkdownTable($properties);
+
+    expect($result)->toContain('| Description | Line 1 Line 2 |');
+});
+
+test('replaces CRLF with spaces in property values', function () {
+    $properties = [
+        'Description' => [
+            'id' => 'rich',
+            'type' => 'rich_text',
+            'rich_text' => [
+                ['plain_text' => "Line 1\r\nLine 2"],
+            ],
+        ],
+    ];
+
+    $result = $this->propertiesTable->convertPropertiesToMarkdownTable($properties);
+
+    expect($result)->toContain('| Description | Line 1 Line 2 |');
+});
+
+test('replaces multiple newlines with spaces in property values', function () {
+    $properties = [
+        'Description' => [
+            'id' => 'rich',
+            'type' => 'rich_text',
+            'rich_text' => [
+                ['plain_text' => "Line 1\n\nLine 2\n\nLine 3"],
+            ],
+        ],
+    ];
+
+    $result = $this->propertiesTable->convertPropertiesToMarkdownTable($properties);
+
+    expect($result)->toContain('| Description | Line 1  Line 2  Line 3 |');
+});
+
+test('replaces newlines in property names', function () {
+    $properties = [
+        "Multi\nLine\nName" => [
+            'id' => 'title',
+            'type' => 'title',
+            'title' => [
+                ['plain_text' => 'Test Value'],
+            ],
+        ],
+    ];
+
+    $result = $this->propertiesTable->convertPropertiesToMarkdownTable($properties);
+
+    expect($result)->toContain('| Multi Line Name | Test Value |');
+});
+
+test('handles both newlines and pipe characters together', function () {
+    $properties = [
+        "Name | Title\nWith Newline" => [
+            'id' => 'rich',
+            'type' => 'rich_text',
+            'rich_text' => [
+                ['plain_text' => "Value | with\npipe and\nnewlines"],
+            ],
+        ],
+    ];
+
+    $result = $this->propertiesTable->convertPropertiesToMarkdownTable($properties);
+
+    expect($result)->toContain('| Name \| Title With Newline | Value \| with pipe and newlines |');
+});
+
+test('replaces newlines in select property values', function () {
+    $properties = [
+        'Type' => [
+            'id' => 'select',
+            'type' => 'select',
+            'select' => [
+                'id' => 'LOQu',
+                'name' => "Option\nWith\nNewlines",
+                'color' => 'pink',
+            ],
+        ],
+    ];
+
+    $result = $this->propertiesTable->convertPropertiesToMarkdownTable($properties);
+
+    expect($result)->toContain('| Type | Option With Newlines |');
+});
+
+test('replaces newlines in multi_select property values', function () {
+    $properties = [
+        'Tags' => [
+            'id' => 'multi_select',
+            'type' => 'multi_select',
+            'multi_select' => [
+                [
+                    'id' => 'tag1',
+                    'name' => "Tag\n1",
+                    'color' => 'blue',
+                ],
+                [
+                    'id' => 'tag2',
+                    'name' => "Tag\n2",
+                    'color' => 'green',
+                ],
+            ],
+        ],
+    ];
+
+    $result = $this->propertiesTable->convertPropertiesToMarkdownTable($properties);
+
+    expect($result)->toContain('| Tags | Tag 1, Tag 2 |');
+});
+
+test('replaces newlines in people names', function () {
+    $properties = [
+        'Person' => [
+            'id' => 'people',
+            'type' => 'people',
+            'people' => [
+                ['name' => "John\nDoe"],
+                ['name' => "Jane\nSmith"],
+            ],
+        ],
+    ];
+
+    $result = $this->propertiesTable->convertPropertiesToMarkdownTable($properties);
+
+    expect($result)->toContain('| Person | John Doe, Jane Smith |');
+});