fix(env): normalize URLs before comparing to avoid spurious mismatch warnings

Use new URL().href to normalize both the override and profile URL before
comparing, so trailing slashes and host-case differences don't produce
false positives. Falls back to raw string comparison when either URL is
malformed.

Also pin the test to a concrete literal ("https://api.clerk.com") instead
of the self-referencing getPlapiBaseUrl() call, and strengthen the positive
warning case by asserting the override host appears in the message.

Author: rafa-thayto

packages/cli-core/src/lib/environment.test.ts

@@ -1,5 +1,5 @@
 import { test, expect, describe, beforeEach, afterEach } from "bun:test";
-import { getPlapiBaseUrl, warnIfPlatformApiUrlOverride } from "./environment.ts";
+import { warnIfPlatformApiUrlOverride } from "./environment.ts";
 import { setMode } from "../mode.ts";
 import { useCaptureLog } from "../test/lib/stubs.ts";
 
@@ -23,6 +23,7 @@ describe("warnIfPlatformApiUrlOverride", () => {
     warnIfPlatformApiUrlOverride();
     expect(captured.err).toContain("CLERK_PLATFORM_API_URL");
     expect(captured.err).toContain("production");
+    expect(captured.err).toContain("api.staging.example.com");
   });
 
   test("does not warn when no override is set", () => {
@@ -31,8 +32,7 @@ describe("warnIfPlatformApiUrlOverride", () => {
   });
 
   test("does not warn when the override equals the active env URL", () => {
-    const profileUrl = getPlapiBaseUrl(); // no override set → active env URL
-    process.env.CLERK_PLATFORM_API_URL = profileUrl;
+    process.env.CLERK_PLATFORM_API_URL = "https://api.clerk.com";
     warnIfPlatformApiUrlOverride();
     expect(captured.err).not.toContain("CLERK_PLATFORM_API_URL");
   });

packages/cli-core/src/lib/environment.ts

@@ -152,7 +152,14 @@ export function warnIfPlatformApiUrlOverride(): void {
   const override = process.env.CLERK_PLATFORM_API_URL;
   if (!override) return;
   const envName = getCurrentEnvName();
-  if (override === getCurrentEnv().platformApiUrl) return;
+  const normalize = (u: string) => {
+    try {
+      return new URL(u).href;
+    } catch {
+      return u;
+    }
+  };
+  if (normalize(override) === normalize(getCurrentEnv().platformApiUrl)) return;
   log.warn(
     `CLERK_PLATFORM_API_URL is routing requests to ${override}, but credentials stay keyed to the "${envName}" environment — the "${envName}" token will be sent to that host.`,
   );