optimizely / javascript-sdk / 28954433134

78%

Build Type

push

github

Committed by web-flow

Commit Message

[FSSDK-12880] Refactor project config creation (#1164)

* [FSSDK-12880] Refactor project config creation

- Simplify createProjectConfig to accept a datafile string directly
  instead of string | object, inlining datafile validation
- Remove the separate config_validator module and its tests
- Break monolithic config builder into focused parse functions:
  parseAudienceConfig, parseAttributeConfig, parseGroupConfig,
  parseRolloutConfig, parseIntegrationConfig, parseExperimentConfig,
  parseFeatureFlagConfig
- Eliminate mutation-safe datafile copy in favor of direct construction
- Refactor redundant iterations over experiments and feature flags
- Remove lodash dev dependency

Benchmarks show ~10% faster config parsing for 3-4 MB datafiles.
Removing createMutationSafeDatafileCopy also fixes a memory issue
where it allocated ~4x the datafile size in shallow copies per call,
causing OOM under repeated parsing of large datafiles.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Coverage Stats

1547 of 3017 branches covered (51.28%)

Branch coverage included in aggregate %.

138 of 140 new or added lines in 7 files covered. (98.57%)

2 existing lines in 2 files now uncovered.

8990 of 10540 relevant lines covered (85.29%)

261.79 hits per line