Repository navigation
fix: preserve standalone reply conversation text - #142
Merged
Merged
Conversation
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.
Standalone
jury replysent a prompt and wrote it to a Markdown artifact, but omitted its text from thereply.sentevent. Saved conversations therefore rendered an empty judge message before the reviewer answer. Persist the sent text using the same event schema as autonomous replies.The CLI regression exercises two local fake reviewers and verifies each saved prompt, reviewer isolation, judge attribution and answer. It failed on unchanged master, then passed after the fix. README, changelog and the command coverage map describe the behavior.
Validation:
cdff20bf61404d1eff3ebd69be275d89cf54b00f(71.3 seconds). The reviewer sandbox blocked temporary test files with EPERM, so its review was static; independent full/package checks passed as recorded above.Actual Jury command (from the isolated worktree):
This fix applies to newly sent replies; it does not migrate older event logs.