Skip to content

fix: escape agent-controlled Rich output - #46

Merged
royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
Rayan-and-beyond:fix/rich-escape-29
Sep 16, 2026
Merged

royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
Rayan-and-beyond:fix/rich-escape-29

Conversation

@Rayan-and-beyond

Copy link
Copy Markdown
Contributor

Summary

  • escape agent-controlled descriptions and finding text before Rich renders them
  • escape prompt/result content in show while preserving the CLI's own formatting
  • add the requested fail-first regression for bracketed descriptions

Closes #29

Validation

  • fail-first: focused bracketed-description regression failed before the fix
  • PYTHONPATH=. uv run pytest -q tests/test_cli.py -k bracketed_description — 1 passed
  • PYTHONPATH=. uv run pytest -q — 52 passed
  • uv run ruff check agentrace/cli.py tests/test_cli.py — passed
  • git diff --check — passed

The repository-wide Ruff scan still reports pre-existing findings outside these touched lines.

@royalpinto007
royalpinto007 merged commit 6bf4101 into AgentPostmortem:main Sep 16, 2026
8 checks passed
@github-actions

Copy link
Copy Markdown

This is in, @Rayan-and-beyond. Really appreciate it.

Good, focused change. Exactly the kind of thing Agentrace needs.

If Agentrace is useful to you, a star goes a long way.

Happy to see more from you whenever you have the time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Agent-controlled text printed as Rich markup unescaped

2 participants