test: stub posthog and clarify nock usage in setup-test-env

Generated-by: aiautocommit

Author: iloveitaly

web/test/setup-test-env.ts

@@ -6,12 +6,17 @@ import * as matchers from "@testing-library/jest-dom/matchers"
 import "@testing-library/jest-dom/vitest"
 import { cleanup } from "@testing-library/react"
 
-// without nock, posthog and other services will attempt to load and cause tests to be flakey
+// happy-dom blocks external <script src="..."> execution by default, which prevents SDK bundles
+// like PostHog/Clerk from loading in tests.
+//
+// nock is still useful as a second layer: it blocks accidental external HTTP calls (fetch/xhr/http)
+// so tests remain deterministic even if runtime behavior changes.
 nock.disableNetConnect()
 nock.enableNetConnect((host: string) => {
   return host.startsWith("127.0.0.1") || host.startsWith("localhost")
 })
 
+// Keep this explicit PostHog stub for stability across environment/runtime differences.
 nock(/posthog\.com/)
   .persist()
   .get(/.*/)