Tint transcript turns in speaker colors for visual distinction

[alexkroman]

Summary

Redesigned the visual rendering of conversation transcripts so that entire speaker turns (both label and body text) are tinted in the speaker's color, rather than just the label prefix. This makes "you:" and "agent:" lines visually distinct at a glance, even on terminals with limited color support.

Key Changes

• Theme colors: Introduced a new AGENT color (#14B8A6, teal) to replace the previous secondary accent (Cobolt 300) for agent labels. The teal was chosen specifically to downgrade to ANSI cyan on 16-color terminals, staying clearly distinct from the brand purple used for "you" lines. Previously, both speakers used near-identical cobolt purples that collapsed to the same hue after downsampling.

• Rendering: Modified _labeled() in aai_cli/agent/render.py to apply the style to the entire line (label + body) rather than just the label prefix. This ensures the whole turn reads as a cohesive colored block.

• Tests: Added test_human_whole_turn_is_tinted_in_speaker_color() to verify that both the label and body text are wrapped in the speaker's color SGR codes, and that the two speakers render in contrasting hues. Added test_you_and_agent_stay_distinct_after_downsampling() to ensure the colors remain visually distinct even after downsampling to 16-color ANSI palettes.

Implementation Details

• The AGENT teal (#14B8A6) was chosen outside the cobolt purple family on purpose: it downsamples to ANSI cyan, which is clearly distinct from the purple/blue that "you" downsamples to, ensuring readability across all terminal capabilities.
• The aai.you style retains the brand purple and remains reserved (never reused for diarized speakers).
• Updated theme documentation to clarify that the entire transcript turn is now tinted, not just the prefix.

https://claude.ai/code/session_01AFrwHX8EdECmkNsU8LqJNf

[aikido-pr-checks]

In apt_retry, the warning says 'retrying' even on the 3rd failed attempt, but no retry occurs afterward. This creates misleading failure diagnostics.

Show fix

Suggested change

	echo "::warning::apt-get $* failed or stalled (attempt ${attempt}/3); retrying" >&2
	sleep "$((attempt * 5))"
	if [ "$attempt" -lt 3 ]; then
	echo "::warning::apt-get $* failed or stalled (attempt ${attempt}/3); retrying" >&2
	sleep "$((attempt * 5))"
	fi

Details

✨ AI Reasoning
​The retry loop is bounded to three attempts. When the third attempt fails, the warning text still states that it is retrying, but no further attempt exists and the function immediately proceeds to terminal failure. That makes the status message inconsistent with the actual execution path and can mislead CI diagnosis.

_{Reply @AikidoSec feedback: [FEEDBACK] to get better review comments in the future.
Reply @AikidoSec ignore: [REASON] to ignore this issue.
More info}