source-academy / frontend / 30558935769

44%

push

github

Allow deployments to pin the Conductor feature flags (#4205)

* Allow deployments to pin the Conductor feature flags

The Conductor and directory feature flags could only be set per-browser,
via /features, so a deployment had no way to configure them for every
user.

Feature flags can now carry a lockedValue supplied by build-time
configuration, which takes precedence over both the persisted user
override and the flag's default. Locked flags are read-only on /features
and cannot be written back into persisted state.

REACT_APP_CONDUCTOR_ENABLE (TRUE/FALSE/unset) pins conductor.enable;
while it is TRUE, REACT_APP_LANGUAGE_DIRECTORY_URL,
REACT_APP_PLUGIN_DIRECTORY_URL and REACT_APP_MODULES_DIRECTORY_URL pin
the corresponding directory URLs. Leaving them unset preserves today's
behaviour exactly.

The SICP JS override in selectConductorEnable still wins over a pinned
flag, since Source is not in the Language Directory yet and a forced-on
Conductor would break the textbook's runnable snippets.

Also replaces LanguageDirectorySaga's duplicate copy of the default
directory URL with the flag's own defaultValue, so pinning the URL to
that same public value keeps using the bundled npm copy instead of
silently starting a network fetch.

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

* Fix formatting

* Update README to be more accurate

* Guard resetFlag on locked feature flags

A deployment-pinned flag could still have its persisted override deleted
via resetFlag, even though setFlag was already blocked. Guard both
reducer actions so a locked flag never modifies persisted user state.

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

---------

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
Co-authored-by: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>

2951 of 7734 branches covered (38.16%)

Branch coverage included in aggregate %.

16 of 16 new or added lines in 9 files covered. (100.0%)

6813 of 14467 relevant lines covered (47.09%)

29.78 hits per line