feat: add REFLEX_HOT_RELOAD_OVERRIDE_PATHS to fully override reload paths#6639
Conversation
…aths When set, get_reload_paths() returns these paths directly, taking precedence over include/exclude paths and the config-derived defaults.
Greptile SummaryThis PR introduces a
Confidence Score: 5/5Safe to merge — the change is an additive opt-in feature with no effect when the env var is unset. The override guard is an isolated early-return that is inert by default (empty list is falsy). The implementation mirrors the existing include/exclude path handling exactly: same Path.absolute conversion, same EnvVar[list[Path]] typing, same debug log pattern. No existing behavior is modified. No files require special attention. Important Files Changed
Reviews (3): Last reviewed commit: "add changelog fragments" | Re-trigger Greptile |
Merging this PR will not alter performance
Comparing Footnotes
|
Summary
Adds a
REFLEX_HOT_RELOAD_OVERRIDE_PATHSenvironment variable (colon-separated list of paths). When set,get_reload_paths()returns these paths directly, taking precedence overREFLEX_HOT_RELOAD_INCLUDE_PATHS,REFLEX_HOT_RELOAD_EXCLUDE_PATHS, and the config-derived defaults.This gives embedders full control over what the dev-mode file watcher observes — e.g. pointing it at a dedicated control directory instead of relying on the app's project layout.
Linear: ENG-9752