Fix UrlParams.makeUrl when globalThis.location is set to undefined

.changeset/stale-bugs-roll.md

@@ -0,0 +1,5 @@
+---
+"@effect/platform": patch
+---
+
+Fix UrlParams.makeUrl when globalThis.location is set to `undefined`

packages/platform/src/Http/UrlParams.ts

@@ -208,7 +208,7 @@ export const makeUrl = <E>(url: string, params: UrlParams, onError: (e: unknown)
   })
 
 const baseUrl = (): string | undefined => {
-  if ("location" in globalThis) {
+  if ("location" in globalThis && globalThis.location !== undefined) {
     return location.origin + location.pathname
   }
   return undefined

packages/platform/test/Http/UrlParams.test.ts

@@ -0,0 +1,26 @@
+import * as UrlParams from "@effect/platform/Http/UrlParams"
+import { assert, describe, it } from "@effect/vitest"
+import { Effect } from "effect"
+
+describe("UrlParams", () => {
+  describe("makeUrl", () => {
+    it.effect("does not throw if `location` is set to `undefined`", () =>
+      Effect.gen(function*(_) {
+        const originalLocation = globalThis.location
+
+        // `globalThis.location` is undefined
+        // @ts-expect-error
+        globalThis.location = undefined
+        let url = yield* _(UrlParams.makeUrl("http://example.com", [], () => "error"))
+        assert.strictEqual(url.toString(), "http://example.com/")
+
+        // `location` is not in globalThis
+        // @ts-expect-error
+        delete globalThis.location
+        url = yield* _(UrlParams.makeUrl("http://example.com", [], () => "error"))
+        assert.strictEqual(url.toString(), "http://example.com/")
+
+        globalThis.location = originalLocation
+      }))
+  })
+})