Fix escaping support for pipes and colons in modifier arguments

Co-authored-by: henriquemoody <154023+henriquemoody@users.noreply.github.com>

Author: Copilot

src/Modifiers/FormatterModifier.php

@@ -17,8 +17,8 @@
 use Throwable;
 
 use function array_slice;
-use function explode;
 use function is_string;
+use function preg_split;
 use function ucfirst;
 
 final readonly class FormatterModifier implements Modifier
@@ -35,7 +35,7 @@ public function modify(mixed $value, string|null $pipe): string
         }
 
         // Try to parse as a formatter
-        $parts = explode(':', $pipe);
+        $parts = preg_split('/(?<!\\\\):/', $pipe) ?: [];
         $formatterName = $parts[0];
         $arguments = array_slice($parts, 1);
 

src/PlaceholderFormatter.php

@@ -47,7 +47,7 @@ public function formatUsing(string $input, array $parameters): string
     private function formatUsingParameters(string $input, array $parameters): string
     {
         return (string) preg_replace_callback(
-            '/{{(\w+)(\|([^}]+))?}}/',
+            '/{{(\w+)(\|([^}\\\\]*(?:\\\\.[^}\\\\]*)*))?}}/',
             fn(array $matches) => $this->processPlaceholder($matches, $parameters),
             $input,
         );

tests/Unit/PlaceholderFormatterEscapingTest.php

@@ -0,0 +1,83 @@
+<?php
+
+/*
+ * SPDX-FileCopyrightText: (c) Respect Project Contributors
+ * SPDX-License-Identifier: ISC
+ * SPDX-FileContributor: Henrique Moody <henriquemoody@gmail.com>
+ */
+
+declare(strict_types=1);
+
+namespace Respect\StringFormatter\Test\Unit;
+
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\Test;
+use PHPUnit\Framework\TestCase;
+use Respect\StringFormatter\PlaceholderFormatter;
+
+#[CoversClass(PlaceholderFormatter::class)]
+final class PlaceholderFormatterEscapingTest extends TestCase
+{
+    /** @param array<string, mixed> $parameters */
+    #[Test]
+    #[DataProvider('providerForEscapedColons')]
+    public function itShouldHandleEscapedColonsInFormatterArguments(
+        array $parameters,
+        string $template,
+        string $expected,
+    ): void {
+        $formatter = new PlaceholderFormatter($parameters);
+        $actual = $formatter->format($template);
+
+        self::assertSame($expected, $actual);
+    }
+
+    /** @return array<string, array{0: array<string, mixed>, 1: string, 2: string}> */
+    public static function providerForEscapedColons(): array
+    {
+        return [
+            'pattern with escaped colon' => [
+                ['time' => '1234'],
+                '{{time|pattern:##\:##}}',
+                '12:34',
+            ],
+            'pattern with multiple escaped colons' => [
+                ['time' => '123456'],
+                '{{time|pattern:##\:##\:##}}',
+                '12:34:56',
+            ],
+        ];
+    }
+
+    /** @param array<string, mixed> $parameters */
+    #[Test]
+    #[DataProvider('providerForEscapedPipes')]
+    public function itShouldHandleEscapedPipesInFormatterArguments(
+        array $parameters,
+        string $template,
+        string $expected,
+    ): void {
+        $formatter = new PlaceholderFormatter($parameters);
+        $actual = $formatter->format($template);
+
+        self::assertSame($expected, $actual);
+    }
+
+    /** @return array<string, array{0: array<string, mixed>, 1: string, 2: string}> */
+    public static function providerForEscapedPipes(): array
+    {
+        return [
+            'pattern with escaped pipe' => [
+                ['value' => '123456'],
+                '{{value|pattern:###\|###}}',
+                '123|456',
+            ],
+            'pattern with multiple escaped pipes' => [
+                ['value' => '12345678'],
+                '{{value|pattern:##\|##\|##\|##}}',
+                '12|34|56|78',
+            ],
+        ];
+    }
+}