Bump @sourceacademy/conductor to ^0.8.2 - #858
Merged
Merged
Conversation
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.
Contributor
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
🤖 Generated with Claude Code