source-academy / frontend / 28697479282

push

github

feat(playground): drive the conductor stepper from its side-content tab (#4035)

* feat(stepper): integrate Conductor plugin tabs and dynamic loading of web plugins

* Update src/styles/_workspace.scss

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Update src/commons/sagas/helpers/conductorEvaluatorCache.ts

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Update src/features/conductor/pluginTabRegistry.ts

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

* chore(conductor): stop committing evaluator/plugin bundles to frontend

The evaluators and the stepper web-plugin bundle are built from their source
repos (js-slang, py-slang, plugins) and deployed to GitHub Pages. The frontend
references those URLs through the hosted language- and plugin-directories, whose
flag defaults already point at source-academy.github.io. The local copies under
public/evaluators, public/languages and public/plugins were only a turnkey
local-test harness and must not live in the frontend repo.

Untrack them and gitignore the dev-fixture paths so dev.sh can keep copying them
locally without re-adding them to the PR.

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

* fix(conductor): address PR review feedback

- Fix absolute js-slang portal path to relative (breaks CI for other devs)
- Fix corrupted pluginTabRegistry.ts type definition; add iconName validation
- Wrap conduit.terminate() in try-catch to prevent stuck running state
- Guard registerPluginTabIfPresent against stale conductor race condition
- Add defensive checks in shim files for missing globalThis bootstrap vars
- Extract CONDUCTOR_STEPPER_TAB_ID constant to avoid hardcoded string drift
- Add flagConductorEnable side-effects to set local directory URLs

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat(conductor): dynamic plugin tabs + f... (continued)

2624 of 7498 branches covered (35.0%)

Branch coverage included in aggregate %.

28 of 60 new or added lines in 6 files covered. (46.67%)

6233 of 14126 relevant lines covered (44.12%)

26.54 hits per line