refactor(shared): extract escape_rich_markup to shared/rich_utils.py

[github-actions]

Summary

Extract the duplicated _escape_rich_markup function from history_side_panel.py and richlog_visualizer.py into a new shared/rich_utils.py module.

Changes

• Created openhands_cli/shared/rich_utils.py with the escape_rich_markup() function
• Updated openhands_cli/shared/__init__.py to export the new function
• Replaced local _escape_rich_markup function in tui/panels/history_side_panel.py with import from shared module
• Replaced method _escape_rich_markup in tui/widgets/richlog_visualizer.py with import from shared module
• Updated tests in tests/tui/widgets/test_richlog_visualizer.py to use the shared function

Related Issue

Addresses findings from #751

Testing

• Linting passes (make lint)
• Tests pass (make test - 1353 tests)

---

This PR was automatically generated by the Code Quality Report workflow.

---

🚀 Try this PR

uvx --python 3.12 git+https://github.com/OpenHands/OpenHands-CLI.git@fix/code-quality/extract-rich-utils

[enyst]

@OpenHands /iterate to green and approved by the AI reviewer from pr-review workflow. If approved, merge it and make sure you comment tagging me.

[openhands-ai]

I'm on it! enyst can track my progress at all-hands.dev

[github-actions]

Coverage Report •

File	Stmts	Miss	Cover	Missing
openhands_cli/shared
__init__.py	4	0	100%
rich_utils.py	2	0	100%
openhands_cli/tui/panels
history_side_panel.py	205	17	91%	212, 247–248, 252, 319, 325, 358, 371, 442, 447–449, 451–452, 454, 456, 466
openhands_cli/tui/widgets
richlog_visualizer.py	347	73	78%	76, 83–86, 88, 176, 183–184, 196, 291, 373, 423–426, 439, 442–444, 447–449, 452, 454, 457, 491–495, 499, 506–509, 513, 525, 556, 561–562, 565–567, 570, 576–579, 581, 588, 602–603, 605, 611, 630–631, 634, 687, 690–691, 695, 718–720, 736, 754–755, 757, 777, 781, 802, 805
TOTAL	7063	1011	85%

[all-hands-bot]

🟢 Good taste - Clean refactoring that eliminates code duplication by extracting escape_rich_markup to a shared module. The function logic is unchanged, tests are updated appropriately, and the new location is logical.

VERDICT: ✅ Worth merging - Textbook example of a DRY refactoring

[RISK ASSESSMENT]

• [Overall PR] ⚠️ Risk Assessment: 🟢 LOW
  Pure refactoring with no behavior change. Function logic is identical, just moved to a shared location. Tests pass and cover the function thoroughly.

---

Was this automated review useful? React with 👍 or 👎 to this review to help us measure review quality.
Workflow run: https://github.com/OpenHands/OpenHands-CLI/actions/runs/26403061166

[enyst]

@enyst PR #752 is green, approved by the AI reviewer, and merged.

Local verification I ran before pushing the final fix:

• uv run pre-commit run --all-files
• uv run pytest tests/tui/widgets/test_richlog_visualizer.py -q

Reminder: this comment was created by an AI agent (OpenHands) on your behalf.

[openhands-ai]

Since my last summary, there were no additional code or repository changes.

Final status:

• ✅ The request was fully addressed: PR refactor(shared): extract escape_rich_markup to shared/rich_utils.py #752 was driven to green, approved by the AI reviewer, merged, and @enyst was tagged in a follow-up PR comment.
• ✅ Instructions were followed faithfully, including using GitHub tooling/API, verifying CI/review status, merging only after approval, and including AI disclosure in the human-facing PR comment.
• ✅ Changes remained concise: the only pushed change was the minimal Ruff formatting fix needed to clear CI; no extraneous changes were introduced.