[vitali87]

Summary

Follow-up to #124: the remaining Rich outputs that print plan or user text unescaped — the Split Plan table cells, the dependency-tree labels, and the editor's Moved … / Hunk … not found / Group '…' not found messages — now go through rich.markup.escape. A title such as Add [core] typing or a path like src/[id].py is printed as written instead of losing the bracketed part as a style tag. The markdown dependency graph in PR bodies is plain text and untouched.

Stacked on #124 (fix/show-assignment-type-markup).

Still unescaped, noted for a later PR: the status table, merge summary lines, Edited plan is invalid: {exc}-style error echoes, and LOC-bound warnings.

Test plan

• TestRichEscapingOfPlanText: table, tree, and editor messages keep bracketed text — all three fail on the base
• 448 tests pass, ruff clean
• Local review: 5/5

[coderabbitai]

Warning

Review limit reached

Next included review available in 1 minute.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: c19159e9-da6a-42fb-94af-76068403a250

📥 Commits

Reviewing files that changed from the base of the PR and between 92103d1 and 069a82d.

⛔ Files ignored due to path filters (1)

• uv.lock is excluded by !**/*.lock

📒 Files selected for processing (64)

• .github/workflows/split-score.yml
• README.md
• action.yml
• pr_split/cli.py
• pr_split/config.py
• pr_split/constants.py
• pr_split/diff_ops/new_file_split.py
• pr_split/diff_ops/parser.py
• pr_split/diff_ops/reconstructor.py
• pr_split/exceptions.py
• pr_split/git_ops/__init__.py
• pr_split/git_ops/branches.py
• pr_split/git_ops/prs.py
• pr_split/graph.py
• pr_split/logs.py
• pr_split/per_group.py
• pr_split/plan_store.py
• pr_split/planner/__init__.py
• pr_split/planner/chunker.py
• pr_split/planner/client.py
• pr_split/planner/new_file_pieces.py
• pr_split/planner/partitioning.py
• pr_split/planner/repair.py
• pr_split/planner/scoring.py
• pr_split/planner/symbols.py
• pr_split/planner/validator.py
• pr_split/recover.py
• pr_split/restack.py
• pr_split/schemas.py
• pr_split/stack_move.py
• pyproject.toml
• scripts/score_pr.py
• tests/conftest.py
• tests/network_guards.py
• tests/test_chunker.py
• tests/test_cli_coverage.py
• tests/test_cli_helpers.py
• tests/test_cli_new_features.py
• tests/test_client.py
• tests/test_config.py
• tests/test_default_strategy.py
• tests/test_description_refresh.py
• tests/test_diff_base.py
• tests/test_diff_parser_extended.py
• tests/test_git_branches.py
• tests/test_git_prs.py
• tests/test_git_prs_coverage.py
• tests/test_graph.py
• tests/test_local_provider.py
• tests/test_new_file_split.py
• tests/test_partitioning.py
• tests/test_per_group.py
• tests/test_plan_store.py
• tests/test_reconstructor.py
• tests/test_recover.py
• tests/test_repair.py
• tests/test_restack.py
• tests/test_schemas.py
• tests/test_score_pr.py
• tests/test_score_pr_events.py
• tests/test_scoring.py
• tests/test_stack_move.py
• tests/test_symbols.py
• tests/test_validator.py

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[vitali87]

This change is already in main: it landed via the merge train #243, which includes this PR's head commit. GitHub can't mark a stacked PR merged when it lands that way, so I'm closing it as done.

---

Generated by Claude Code