move http search params apis to ServerRequest module (#2669)

Effect-TS · Apr 30, 2024 · 9deab0a · 9deab0a
1 parent ebd8656
commit 9deab0a
Show file tree

Hide file tree

Showing 13 changed files with 275 additions and 155 deletions.
diff --git a/.changeset/few-lamps-joke.md b/.changeset/few-lamps-joke.md
@@ -0,0 +1,19 @@
+---
+"@effect/platform": minor
+---
+
+move http search params apis to ServerRequest module
+
+If you want to access the search params for a request, you can now use the `Http.request.ParsedSearchParams` tag.
+
+```ts
+import * as Http from "@effect/platform/HttpServer"
+import { Effect } from "effect"
+
+Effect.gen(function* () {
+  const searchParams = yield* Http.request.ParsedSearchParams
+  console.log(searchParams)
+})
+```
+
+The schema method has also been moved to the `ServerRequest` module. It is now available as `Http.request.schemaSearchParams`.
diff --git a/packages/platform-browser/package.json b/packages/platform-browser/package.json
@@ -60,6 +60,6 @@
     }
   },
   "dependencies": {
-    "multipasta": "^0.2.0"
+    "multipasta": "^0.2.1"
   }
 }
diff --git a/packages/platform-node-shared/package.json b/packages/platform-node-shared/package.json
@@ -42,7 +42,7 @@
   },
   "dependencies": {
     "@parcel/watcher": "^2.4.1",
-    "multipasta": "^0.2.0"
+    "multipasta": "^0.2.1"
   },
   "peerDependencies": {
     "@effect/platform": "workspace:^",

diff --git a/packages/platform/package.json b/packages/platform/package.json
@@ -39,9 +39,9 @@
     "coverage": "vitest --coverage"
   },
   "dependencies": {
-    "find-my-way-ts": "^0.1.1",
+    "find-my-way-ts": "^0.1.2",
     "isows": "^1.0.4",
-    "multipasta": "^0.2.0",
+    "multipasta": "^0.2.1",
     "path-browserify": "^1.0.1"
   },
   "peerDependencies": {

diff --git a/packages/platform/src/Http/App.ts b/packages/platform/src/Http/App.ts
@@ -152,9 +152,7 @@ export const toWebHandlerRuntime = <R>(runtime: Runtime.Runtime<R>) => {
         const req = ServerRequest.fromWeb(request)
         ;(req as any)[resolveSymbol] = resolve
         ;(req as any)[rejectSymbol] = reject
-        const fiber = run(
-          Effect.provideService(handled, ServerRequest.ServerRequest, req)
-        )
+        const fiber = run(Effect.provideService(handled, ServerRequest.ServerRequest, req))
         request.signal.addEventListener("abort", () => {
           fiber.unsafeInterruptAsFork(ServerError.clientAbortFiberId)
         })

diff --git a/packages/platform/src/Http/Middleware.ts b/packages/platform/src/Http/Middleware.ts
@@ -103,3 +103,14 @@ export const withTracerDisabledForUrls: {
  * @category constructors
  */
 export const xForwardedHeaders: <R, E>(httpApp: App.Default<E, R>) => App.Default<E, R> = internal.xForwardedHeaders
+
+/**
+ * @since 1.0.0
+ * @category constructors
+ */
+export const searchParamsParser: <E, R>(
+  httpApp: App.Default<E, R>
+) => App.Default<
+  E,
+  Exclude<R, ServerRequest.ParsedSearchParams>
+> = internal.searchParamsParser