source-academy / frontend / 28238933822

41%

push

github

fix: route REPL evaluation through conductor for conductor languages (#4012)

* fix: route evalRepl through evalCodeConductorSaga for conductor languages (closes #4011)

The evalRepl handler was returning early when conductor was enabled, with a
comment saying evalCodeConductorSaga would pick it up — but nothing was
actually calling it. The original conductor integration (PR #3084) handled
this via a take(evalRepl) race inside evalCodeConductorSaga; that was removed
in a later refactor without fixing the routing.

Fix: when conductor is enabled, run the preamble (clearReplInput,
sendReplInputToOutput) then call evalCodeConductorSaga directly with the
REPL code, mirroring how evalEditorSaga routes to evalCodeConductorSaga.

Co-Authored-By: Akshay Narayanan <akshaydocs2007@gmail.com>

* refactor: hoist context selector before conductor branch to avoid duplication

---------

Co-authored-by: Akshay Narayanan <akshaydocs2007@gmail.com>

2592 of 7338 branches covered (35.32%)

Branch coverage included in aggregate %.

1 of 4 new or added lines in 1 file covered. (25.0%)

6122 of 13717 relevant lines covered (44.63%)

27.13 hits per line