fix(logfire): reject an explicit region outside the known set

An out-of-set region reaching getLogfireBaseUrl (from persisted workflow
state, which the LogfireRegion type does not guarantee at runtime) indexed
REGION_BASE_URLS with no membership check and produced "undefined/v2/query",
which the executor would resolve against the app origin. Throw the same clear
error already used for an unrecognized token prefix.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: icecrasher321

apps/sim/tools/logfire/utils.test.ts

@@ -2,6 +2,7 @@
  * @vitest-environment node
  */
 import { describe, expect, it } from 'vitest'
+import type { LogfireRegion } from '@/tools/logfire/types'
 import {
   buildLogfireQueryBody,
   extractLogfireColumns,
@@ -40,6 +41,12 @@ describe('getLogfireBaseUrl', () => {
     )
   })
 
+  it('rejects an explicit region outside the known set rather than building undefined/', () => {
+    expect(() => getLogfireBaseUrl('pylf_v1_us_abc', 'apac' as unknown as LogfireRegion)).toThrow(
+      "Unrecognized Logfire region 'apac'"
+    )
+  })
+
   it('lets a self-hosted host override both the region and the token prefix', () => {
     expect(getLogfireBaseUrl('pylf_v1_eu_abc', 'us', 'https://logfire.example.com')).toBe(
       'https://logfire.example.com'

apps/sim/tools/logfire/utils.ts

@@ -102,7 +102,15 @@ function normalizeHost(host?: string): string | undefined {
 export function getLogfireBaseUrl(apiKey: string, region?: LogfireRegion, host?: string): string {
   const selfHosted = normalizeHost(host)
   if (selfHosted) return selfHosted
-  if (region && region !== 'auto') return REGION_BASE_URLS[region]
+
+  if (region && region !== 'auto') {
+    if (!isCloudRegion(region)) {
+      throw new Error(
+        `Unrecognized Logfire region '${region}'. Set Region to us or eu, or set Host explicitly.`
+      )
+    }
+    return REGION_BASE_URLS[region]
+  }
 
   const detected = TOKEN_REGION_PATTERN.exec(apiKey?.trim() ?? '')?.[1]
   if (!detected) return REGION_BASE_URLS.us