perf(utils/url): use slice + indexOf for getPath() (#2376)

* perf(utils/url): use `slice` + `indexOf` for `getPath()`

* denoify

benchmarks/utils/.gitignore

@@ -0,0 +1,2 @@
+yarn.lock
+bun.lockb

benchmarks/utils/package.json

@@ -0,0 +1,5 @@
+{
+  "devDependencies": {
+    "mitata": "^0.1.11"
+  }
+}

benchmarks/utils/src/get-path.ts

@@ -0,0 +1,20 @@
+import { run, group, bench } from 'mitata'
+
+bench('noop', () => {})
+
+const request = new Request('http://localhost/about/me')
+
+group('getPath', () => {
+  bench('slice + indexOf', () => {
+    const url = request.url
+    const queryIndex = url.indexOf('?', 8)
+    url.slice(url.indexOf('/', 8), queryIndex === -1 ? undefined : queryIndex)
+  })
+
+  bench('regexp', () => {
+    const match = request.url.match(/^https?:\/\/[^/]+(\/[^?]*)/)
+    match ? match[1] : ''
+  })
+})
+
+run()

deno_dist/utils/url.ts

@@ -70,9 +70,10 @@ export const getPattern = (label: string): Pattern | null => {
 }
 
 export const getPath = (request: Request): string => {
-  // Optimized: RegExp is faster than indexOf() + slice()
-  const match = request.url.match(/^https?:\/\/[^/]+(\/[^?]*)/)
-  return match ? match[1] : ''
+  // Optimized: indexOf() + slice() is faster than RegExp
+  const url = request.url
+  const queryIndex = url.indexOf('?', 8)
+  return url.slice(url.indexOf('/', 8), queryIndex === -1 ? undefined : queryIndex)
 }
 
 export const getQueryStrings = (url: string): string => {

src/utils/url.ts

@@ -70,9 +70,10 @@ export const getPattern = (label: string): Pattern | null => {
 }
 
 export const getPath = (request: Request): string => {
-  // Optimized: RegExp is faster than indexOf() + slice()
-  const match = request.url.match(/^https?:\/\/[^/]+(\/[^?]*)/)
-  return match ? match[1] : ''
+  // Optimized: indexOf() + slice() is faster than RegExp
+  const url = request.url
+  const queryIndex = url.indexOf('?', 8)
+  return url.slice(url.indexOf('/', 8), queryIndex === -1 ? undefined : queryIndex)
 }
 
 export const getQueryStrings = (url: string): string => {