Extract FileExtension routine from Accept content negotiation

Add a dedicated FileExtension routine that handles URL extension
mapping (.json, .html, etc.) independently from Accept header
negotiation. Only declared extensions are stripped during route
matching, so dots in paths like /users/john.doe no longer get
mangled.

Multiple FileExtension routines can cascade for compound extensions
like .json.en — each peels its extension from right to left.

Decouple AbstractAccept from IgnorableFileExtension entirely,
making it pure header-based content negotiation. Promote
IgnorableFileExtension from marker interface to a real interface
with getExtensions() so route matching knows exactly which
extensions to strip.

Author: alganet

docs/README.md

@@ -403,8 +403,40 @@ $r3->any('/listeners/*', function ($user) { /***/ });
 Since there are three routes with the `$user` parameter, `when` will
 verify them all automatically by name.
 
+## File Extensions
+
+Use the `fileExtension` routine to map URL extensions to response transformations:
+```php
+$r3->get('/users/*', function($name) {
+    return ['name' => $name];
+})->fileExtension([
+    '.json' => 'json_encode',
+    '.html' => function($data) { return "<h1>{$data['name']}</h1>"; },
+]);
+```
+
+Requesting `/users/alganet.json` strips the `.json` extension, passes `alganet` as the
+parameter, and applies `json_encode` to the response.
+
+Only declared extensions are stripped. A URL like `/users/john.doe` with no `.doe` declared
+will match normally with `john.doe` as the full parameter.
+
+### Multiple Extensions
+
+Multiple `fileExtension` routines can cascade for compound extensions like `.json.en`.
+Declare the outermost extension (rightmost in the URL) first:
+```php
+$r3->get('/page/*', $handler)
+    ->fileExtension(['.en' => $translateEn, '.pt' => $translatePt])
+    ->fileExtension(['.json' => 'json_encode', '.html' => $render]);
+```
+
+Requesting `/page/about.json.en` strips `.en` (first routine), then `.json` (second routine),
+and applies both callbacks in order.
+
 ## Content Negotiation
 
+Content negotiation uses HTTP Accept headers to select the appropriate response format.
 Respect\Rest supports four distinct types of Accept header content-negotiation:
 Mimetype, Encoding, Language and Charset:
 ```php

example/full.php

@@ -19,6 +19,8 @@
  *   GET  /boom                      → Exception route
  *   GET  /status                    → Static value
  *   GET  /time                      → PSR-7 injection
+ *   GET  /data/users.json          → File extension (JSON)
+ *   GET  /data/users.html          → File extension (HTML)
  */
 
 require __DIR__ . '/../vendor/autoload.php';
@@ -138,6 +140,17 @@ public function get(string $id): string
     };
 });
 
+// --- File Extensions ---
+
+$r3->get('/data/*', function (string $resource) {
+    return ['resource' => $resource, 'items' => ['a', 'b', 'c']];
+})->fileExtension([
+    '.json' => 'json_encode',
+    '.html' => function (array $data) {
+        return "<h1>{$data['resource']}</h1><ul><li>" . implode('</li><li>', $data['items']) . '</li></ul>';
+    },
+]);
+
 // --- Content Negotiation ---
 
 $r3->get('/json', function () {

src/Routes/AbstractRoute.php

@@ -14,10 +14,13 @@
 use Respect\Rest\Routines\Routinable;
 use Respect\Rest\Routines\Unique;
 
+use function array_map;
+use function array_merge;
 use function array_pop;
 use function array_shift;
 use function end;
 use function explode;
+use function implode;
 use function is_a;
 use function is_string;
 use function ltrim;
@@ -45,6 +48,7 @@
  * @method self authBasic(mixed ...$args)
  * @method self by(mixed ...$args)
  * @method self contentType(mixed ...$args)
+ * @method self fileExtension(mixed ...$args)
  * @method self lastModified(mixed ...$args)
  * @method self through(mixed ...$args)
  * @method self userAgent(mixed ...$args)
@@ -166,16 +170,27 @@ public function match(DispatchContext $context, array &$params = []): bool
         $params = [];
         $matchUri = $context->path();
 
+        $allExtensions = [];
         foreach ($this->routines as $routine) {
-            if (!($routine instanceof IgnorableFileExtension)) {
+            if (!$routine instanceof IgnorableFileExtension) {
                 continue;
             }
 
-            $matchUri = preg_replace(
-                '#(\.[\w\d\-_.~\+]+)*$#',
-                '',
-                $context->path(),
-            ) ?? $context->path();
+            $allExtensions = array_merge($allExtensions, $routine->getExtensions());
+        }
+
+        if ($allExtensions !== []) {
+            $escaped = array_map(preg_quote(...), $allExtensions);
+            $pattern = '#(' . implode('|', $escaped) . ')+$#';
+            $matchUri = preg_replace($pattern, '', $context->path()) ?? $context->path();
+
+            $suffix = substr($context->path(), strlen($matchUri));
+            if ($suffix !== '') {
+                $context->request = $context->request->withAttribute(
+                    'respect.ext.remaining',
+                    $suffix,
+                );
+            }
         }
 
         if (!preg_match($this->regexForMatch, $matchUri, $params)) {

src/Routines/AbstractAccept.php

@@ -7,14 +7,12 @@
 use Respect\Rest\DispatchContext;
 
 use function array_keys;
-use function array_pop;
 use function array_slice;
 use function arsort;
 use function explode;
 use function preg_replace;
 use function str_replace;
 use function str_starts_with;
-use function stripos;
 use function strpos;
 use function strtolower;
 use function substr;
@@ -26,30 +24,13 @@
 abstract class AbstractAccept extends AbstractCallbackMediator implements
     ProxyableBy,
     ProxyableThrough,
-    Unique,
-    IgnorableFileExtension
+    Unique
 {
     public const string ACCEPT_HEADER = '';
 
-    protected string $requestUri = '';
-
     /** @param array<int, mixed> $params */
     public function by(DispatchContext $context, array $params): mixed
     {
-        $unsyncedParams = $context->params;
-        $extensions = $this->filterKeysContain('.');
-
-        if (empty($extensions) || empty($unsyncedParams)) {
-            return null;
-        }
-
-        $unsyncedParams[] = str_replace(
-            $extensions,
-            '',
-            array_pop($unsyncedParams),
-        );
-        $context->params = $unsyncedParams;
-
         return null;
     }
 
@@ -66,8 +47,6 @@ public function through(DispatchContext $context, array $params): mixed
      */
     protected function identifyRequested(DispatchContext $context, array $params): array
     {
-        $this->requestUri = $context->path();
-
         $headerName = $this->getAcceptHeaderName();
         $acceptHeader = $context->request->getHeaderLine($headerName);
 
@@ -93,7 +72,7 @@ protected function identifyRequested(DispatchContext $context, array $params): a
     /** @return array<int, string> */
     protected function considerProvisions(string $requested): array
     {
-        return $this->getKeys(); // no need to split see authorize
+        return $this->getKeys();
     }
 
     /** @param array<int, mixed> $params */
@@ -105,10 +84,6 @@ protected function notifyApproved(
     ): void {
         $this->rememberNegotiatedCallback($context, $this->getCallback($provided));
 
-        if (strpos($provided, '.') !== false) {
-            return;
-        }
-
         $headerType = $this->getNegotiatedHeaderType();
 
         $contentHeader = 'Content-Type';
@@ -138,16 +113,10 @@ protected function notifyDeclined(
 
     protected function authorize(string $requested, string $provided): mixed
     {
-        // negotiate on file extension
-        if (strpos($provided, '.') !== false) {
-            return stripos($this->requestUri, $provided) !== false;
-        }
-
         if ($requested === '*') {
             return true;
         }
 
-        // normal matching requirements
         return $requested == $provided;
     }
 

src/Routines/FileExtension.php

@@ -0,0 +1,77 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Respect\Rest\Routines;
+
+use Respect\Rest\DispatchContext;
+use SplObjectStorage;
+
+use function str_ends_with;
+use function strlen;
+use function substr;
+
+final class FileExtension extends CallbackList implements
+    ProxyableBy,
+    ProxyableThrough,
+    IgnorableFileExtension
+{
+    private const string REMAINING_ATTRIBUTE = 'respect.ext.remaining';
+
+    /** @var SplObjectStorage<DispatchContext, callable>|null */
+    private SplObjectStorage|null $negotiated = null;
+
+    /** @return array<int, string> */
+    public function getExtensions(): array
+    {
+        return $this->getKeys();
+    }
+
+    /** @param array<int, mixed> $params */
+    public function by(DispatchContext $context, array $params): mixed
+    {
+        $remaining = (string) $context->request->getAttribute(self::REMAINING_ATTRIBUTE, '');
+
+        if ($remaining === '') {
+            return null;
+        }
+
+        foreach ($this->getKeys() as $ext) {
+            if (!str_ends_with($remaining, $ext)) {
+                continue;
+            }
+
+            $remaining = substr($remaining, 0, -strlen($ext));
+            $context->request = $context->request->withAttribute(
+                self::REMAINING_ATTRIBUTE,
+                $remaining,
+            );
+            $this->remember($context, $this->getCallback($ext));
+
+            return null;
+        }
+
+        return null;
+    }
+
+    /** @param array<int, mixed> $params */
+    public function through(DispatchContext $context, array $params): mixed
+    {
+        if (!$this->negotiated instanceof SplObjectStorage || !$this->negotiated->offsetExists($context)) {
+            return null;
+        }
+
+        return $this->negotiated[$context];
+    }
+
+    private function remember(DispatchContext $context, callable $callback): void
+    {
+        if (!$this->negotiated instanceof SplObjectStorage) {
+            /** @var SplObjectStorage<DispatchContext, callable> $storage */
+            $storage = new SplObjectStorage();
+            $this->negotiated = $storage;
+        }
+
+        $this->negotiated[$context] = $callback;
+    }
+}

src/Routines/IgnorableFileExtension.php

@@ -6,4 +6,6 @@
 
 interface IgnorableFileExtension
 {
+    /** @return array<int, string> Extensions this routine handles, e.g. ['.json', '.html'] */
+    public function getExtensions(): array;
 }