Repository navigation
fix: show both agent icons in exchange headers - #119
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.
What changes
Exchange headers now show the matching icon beside both the judge and jury agent names in the console and standalone demo. Previously, the jury agent's icon appeared before the judge's name and the jury agent had no icon beside its own name.
Validation
npm test: all 193 tests passed.npm run test:package: fresh installed-package checks passed.origin/masterand verified two matching, visible icons in both packaged pages at 1100px and 390px.@agentsdance/codejury@0.8.0from this branch. Live verification withjury review --dir . --trunk master --jury droid --judge codex --rounds 2 --push=false --web=falsecompleted after one round on commitc57be6a: Droid reportedNO NEW FINDINGS; the saved run state isconverged.Limitations
Live review was completed with Droid as reviewer. Claude was unavailable for the initial multi-reviewer attempt. No compatibility changes.