feat: add funnelIds to remoteConfig and handle development defaults

[ilasw]

No description provided.

[cursor]

Bug: Development Mode Funnel ID Retrieval Bug

In development mode (!isProd && !isTest), the resolveDynamicFunnelId function's logic for returning static funnel IDs is flawed. If GROWTHBOOK_API_CONFIG_CLIENT_KEY is set, remoteConfig.vars attempts to fetch from GrowthBook instead of providing the intended hardcoded development defaults. This leads to two issues:

1. If GrowthBook doesn't return funnelIds, the function falls back to 'off' instead of the expected static values, contradicting the code comment.
2. Accessing remoteConfig.vars before remoteConfig.init() completes can throw a "remote config not initialized" error, creating a race condition during application startup when GROWTHBOOK_API_CONFIG_CLIENT_KEY is present.

src/routes/boot.ts#L789-L794

daily-api/src/routes/boot.ts

Lines 789 to 794 in c684915

	});
	if (!isProd && !isTest) {
	// In development, we use a static value for the feature key
	return remoteConfig?.vars?.funnelIds?.[featureKey] || 'off';
	}
	return gbClient.getFeatureValue(featureKey, 'off');

Fix in Cursor • Fix in Web

---

Comment bugbot run to trigger another review on this PR
Was this report helpful? Give feedback by reacting with 👍 or 👎