fix(server): handle prefix problem (#326)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **Bug Fixes**
- Enhanced URL prefix handling for more precise matching of requests
based on the expected structure.

- **Tests**
- Introduced new test cases to validate URL prefix matching behavior,
ensuring only correctly formatted paths are recognized.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Author: unnoq

packages/server/src/adapters/standard/handler.test.ts

@@ -340,20 +340,103 @@ describe('standardHandler', () => {
     expect(init).toHaveBeenCalledWith(options)
   })
 
-  it('should check prefix first', async () => {
-    const result = await handler.handle({
-      method: 'GET',
-      headers: {
-        Accept: 'application/json',
-      },
-      url: new URL('http://localhost/users/1'),
-      body: vi.fn(),
-      signal,
-    }, {
-      context: { db: 'postgres' },
-      prefix: '/invalid',
+  describe('prefix', () => {
+    it('require match prefix', async () => {
+      matcher.match.mockResolvedValue({
+        path: ['ping'],
+        procedure: ping,
+        params: { id: '__id__' },
+      })
+
+      const result = await handler.handle({
+        method: 'GET',
+        headers: {
+          Accept: 'application/json',
+        },
+        url: new URL('http://localhost/users/1'),
+        body: vi.fn(),
+        signal,
+      }, {
+        context: { db: 'postgres' },
+        prefix: '/prefix',
+      })
+
+      expect(result).toEqual({ matched: false, response: undefined })
+      expect(matcher.match).not.toHaveBeenCalled()
+    })
+
+    it('must be separate with slash', async () => {
+      matcher.match.mockResolvedValue({
+        path: ['ping'],
+        procedure: ping,
+        params: { id: '__id__' },
+      })
+
+      const result = await handler.handle({
+        method: 'GET',
+        headers: {
+          Accept: 'application/json',
+        },
+        url: new URL('http://localhost/prefixusers/1'),
+        body: vi.fn(),
+        signal,
+      }, {
+        context: { db: 'postgres' },
+        prefix: '/prefix',
+      })
+
+      expect(result).toEqual({ matched: false, response: undefined })
+      expect(matcher.match).not.toHaveBeenCalled()
+    })
+
+    it('support prefix exact match', async () => {
+      matcher.match.mockResolvedValue({
+        path: ['ping'],
+        procedure: ping,
+        params: { id: '__id__' },
+      })
+
+      const result = await handler.handle({
+        method: 'GET',
+        headers: {
+          Accept: 'application/json',
+        },
+        url: new URL('http://localhost/prefix'),
+        body: vi.fn(),
+        signal,
+      }, {
+        context: { db: 'postgres' },
+        prefix: '/prefix',
+      })
+
+      expect(result.matched).toEqual(true)
+      expect(matcher.match).toHaveBeenCalledOnce()
+      expect(matcher.match).toHaveBeenCalledWith('GET', '/')
     })
 
-    expect(result).toEqual({ matched: false, response: undefined })
+    it('support prefix ending with slash', async () => {
+      matcher.match.mockResolvedValue({
+        path: ['ping'],
+        procedure: ping,
+        params: { id: '__id__' },
+      })
+
+      const result = await handler.handle({
+        method: 'GET',
+        headers: {
+          Accept: 'application/json',
+        },
+        url: new URL('http://localhost/prefix/something'),
+        body: vi.fn(),
+        signal,
+      }, {
+        context: { db: 'postgres' },
+        prefix: '/prefix/',
+      })
+
+      expect(result.matched).toEqual(true)
+      expect(matcher.match).toHaveBeenCalledOnce()
+      expect(matcher.match).toHaveBeenCalledWith('GET', '/something')
+    })
   })
 })

packages/server/src/adapters/standard/handler.ts

@@ -72,13 +72,15 @@ export class StandardHandler<T extends Context> {
   }
 
   async handle(request: StandardLazyRequest, options: StandardHandleOptions<T>): Promise<StandardHandleResult> {
-    if (options.prefix && !request.url.pathname.startsWith(options.prefix)) {
+    const prefix = (options.prefix?.replace(/\/$/, '') || undefined) as HTTPPath | undefined
+
+    if (prefix && !request.url.pathname.startsWith(`${prefix}/`) && request.url.pathname !== prefix) {
       return { matched: false, response: undefined }
     }
 
     return intercept(
       this.rootInterceptors,
-      { ...options, request },
+      { ...options, request, prefix },
       async (interceptorOptions) => {
         let isDecoding = false