Skip to content

Bump @sourceacademy/conductor to ^0.8.2 - #858

Merged
martin-henz merged 1 commit into
conductor-migrationfrom
bump/conductor-0.8.2
Jul 28, 2026
Merged

Bump @sourceacademy/conductor to ^0.8.2#858
martin-henz merged 1 commit into
conductor-migrationfrom
bump/conductor-0.8.2

Conversation

@martin-henz

Copy link
Copy Markdown
Member

Summary

Bumps the `@sourceacademy/conductor` catalog pin (`.yarnrc.yml`) from `^0.8.0` to `^0.8.2`, and refreshes `yarn.lock` to actually resolve it (it was still pinned to `0.8.0` despite the range technically allowing newer patches).

0.8.2 includes source-academy/conductor#55: terminal STATUS messages (`STOPPED`/`ERROR`) now carry `sentCount` — the number of output/result/error messages the runner has actually sent — letting a host wait for genuine delivery before tearing the runner down, instead of guessing with a fixed delay.

Without it, back-to-back output (even as simple as `print(1); print(2)`) could silently lose the last message or two whenever a terminal status won whatever race landed it on the host first, ahead of a message already sent on a separate channel. This was the root cause of a duplicate-print bug observed on the deployed `modules-conductor` build.

Verification

  • `yarn.lock` now resolves `@sourceacademy/conductor` to `0.8.2`
  • `tsc` + full test suite pass for every package that depends on conductor: `lib/buildtools`, `lib/modules-lib`, `lib/testplugin`, and the `binary_tree`, `csg`, `curve`, `matrix`, `midi`, `plotly`, `repeat`, `rune`, `scrabble`, `sound` bundles + their `Csg`/`Curve`/`Matrix`/`Plotly`/`Rune`/`Sound` tabs
    • Note: `binary_tree`'s test suite has one pre-existing failure (`is_tree > returns true for a tree round-tripped back in as DataType.ARRAY...`) — confirmed by diffing against an unmodified checkout at the same commit that it fails identically with or without this bump, so it's unrelated (from already-merged binary_tree: accept DataType.ARRAY as equally valid to PAIR for tree nodes #813) and out of scope here.
  • `yarn constraints` clean
  • `yarn lint:all` — no new errors/warnings

🤖 Generated with Claude Code

0.8.2 includes the sentCount fix (source-academy/conductor#55): terminal
STATUS messages now carry the count of output/result/error messages the
runner has actually sent, letting a host wait for genuine delivery
before tearing the runner down instead of guessing with a fixed delay.
Without it, back-to-back output (even print(1); print(2)) could lose
the last message or two once STOPPED/ERROR won whatever race landed it
on the host first.
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@martin-henz
martin-henz merged commit e797221 into conductor-migration Jul 28, 2026
@martin-henz
martin-henz deleted the bump/conductor-0.8.2 branch July 28, 2026 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant