Repository navigation
cc-dazzle: one frame for both reply paths - #77
Merged
Merged
Conversation
The stretchtext path and the plain path in the AssistantMessage handler drew the same frame from two copies, and each worked out the surface style on its own. Both now use one helper and one style. Claude-Session: https://claude.ai/code/session_01Nad9wduaNHB473kmw7nKF3
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.
The reply handler in cc-dazzle drew the same frame twice, once for stretchtext replies and once for plain ones, and worked out the surface style twice. A review of the mod found it. Both paths now share one helper, and replies look the same as before.
I did not cache the project root lookup on tool rows. It can change mid-session after
/cdor a worktree move, and no event says when.Tested
mise run verifypasses, andmise run test:dazzlepasses.