We've just hit a serious bug in Growthbook's handling of invalid JSON in conditions. If there's invalid JSON in the targeting for an experiment, the UI accepts it, but the ultimate output of the CDN completely omits the condition ‼️

Yeah, we should be validating/preventing this up front in the targeting modal. We didn't want a typo in one feature to prevent the entire payload from rendering in the CDN, which is why we're trying to recover gracefully there. We're defaulting to omitting the condition, but I can see why that might cause issues. We have a pretty good UI workflow for JSON feature flags that I think we can re-use for advanced targeting conditions. If you make a small, easily correctable typo (e.g. extra comma), we fix it automatically and ask you to confirm the fix and save again.

We would significantly prefer things to break more seriously if something isn't working - at least the entire feature. Hidden/silent failure is incredibly dangerous

yes, agreed. That's what we do for most things, not entirely sure why conditions were treated differently. We should have, at the very least, defaulted to a condition that always evaluated to false, in essence forcing the rule to skip

James, I went ahead and made a GitHub issue for this here: https://github.com/growthbook/growthbook/issues/1934 Thanks again for letting us know about this.