fix(plan): normalize the runtime feature flag to a boolean

Author: Sg312

apps/sim/lib/core/config/env-flags.test.ts

@@ -7,16 +7,23 @@ vi.hoisted(() => {
   vi.stubEnv('NEXT_PUBLIC_APP_URL', 'https://self-hosted.example')
   vi.stubEnv('NEXT_PUBLIC_FORCE_HOSTED', 'true')
   vi.stubEnv('NODE_ENV', 'production')
+  vi.stubEnv('MSHIP_PLAN_MODE', 'true')
 })
 
 vi.unmock('@/lib/core/config/env')
 vi.unmock('@/lib/core/config/env-flags')
 
-import { isHosted, isProd } from '@/lib/core/config/env-flags'
+import { isHosted, isPlanModeEnabled, isProd } from '@/lib/core/config/env-flags'
 
 describe('hosted detection', () => {
   it('follows the configured URL and ignores the development override in production', () => {
     expect(isProd).toBe(true)
     expect(isHosted).toBe(false)
   })
 })
+
+describe('Plan mode deployment flag', () => {
+  it('normalizes the runtime string to the boolean required by mode selectors', () => {
+    expect(isPlanModeEnabled).toBe(true)
+  })
+})

apps/sim/lib/core/config/env-flags.ts

@@ -729,4 +729,5 @@ export const isLiveEnterpriseSearchEnabled =
 
 /** Private organizational planning, enabled only by this deployment. */
 export const isPlanModeEnabled =
-  typeof window === 'undefined' && (env.MSHIP_PLAN_MODE ?? process.env.NODE_ENV === 'development')
+  typeof window === 'undefined' &&
+  (envBoolean(env.MSHIP_PLAN_MODE) ?? process.env.NODE_ENV === 'development')