fix: route progress markdown detail through _item_row (missed fourth site) - #62
Merged
Merged
Conversation
…s missed fourth site) A retro probe (2026-08-29) found the multi-line-detail defect #61 fixed for triage / assumption_review / confirm still live in _progress_lines: a detail carrying newlines was interpolated into the bullet, so every line starting `-` parsed as a NEW bullet. Both progress row styles (task-status and report) now route through _item_row — single-line details render byte-identical, multi-line ones become the indented block. Shipped via the first live consumer run of the hunk-review-as-triage encoding (board on thread q-forms-hunk-review-001's ruling): both hunks chair-ruled `apply`, rulings validated through collect_form_response. Also records the theme-cap bar chair-ruled at the 2026-08-28 retro in the theme docstring: raises buy FAMILIES, not rules. 825 tests passing. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
1 participant
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.
A retro probe found the multi-line-detail defect #61 fixed for
triage/assumption_review/confirmstill live in_progress_lines: a detail carrying newlines was interpolated into the bullet, so every removed line of a diff parsed as a new bullet. Both progress row styles now route through_item_row— single-line details render byte-identical, multi-line ones become the indented block. One regression test added.Also records the theme-cap bar chair-ruled at the 2026-08-28 retro in the
theme.pydocstring: raises buy families, not rules.Provenance: this change shipped through the first live consumer run of the hunk-review-as-triage encoding ruled in
docs/reports/roundtable/q-forms-hunk-review-001.md— hunks presented as a triage board (id = path@sha:range, diff indetail), both chair-ruledapply, rulings validated throughcollect_form_response. No wall hit; the encoding's one pinch was tool freshness (the plugin's MCP renderer serves released 0.8.0, which lacks the #61 widget fix — the board was rendered from worktree source).825 tests passing;
pre-commit runclean (black-pinned).🤖 Generated with Claude Code