fix(report): render output_structured as formatted HTML in drill-down#277
Merged
Conversation
The HTML report was dumping raw JSON from output_structured in a <pre> block under "View transcript". Now renders structured sections inline per-phase with formatted HTML: - Triage: approach, complexity, code area, target files, risks - Plan: numbered task list, verification commands - Implement: files changed with M/A/D prefixes, commit messages - Verify: verdict badge, command results with pass/fail, code issues - Review: verdict badge, finding count strip_session_data now preserves a curated allowlist of output_structured keys (STRUCTURED_DISPLAY_FIELDS) instead of removing the entire dict. Structured sections render even without --include-sessions. 💖 Generated with Crush Assisted-by: Crush:claude-opus-4-6
The review structured section was empty because it only checked for os.comments (Alcove format) but not os.findings (SODA format). Now renders each finding with severity badge, file location, and issue text inside the expandable review section. 💖 Generated with Crush Assisted-by: Crush:claude-opus-4-6
pipeline_phases from SessionMeta was alphabetically sorted by the adapter (set iteration order), overriding the correct execution sequence. sort_phases now always uses PHASE_ORDER as the canonical reference. pipeline_phases only contributes unknown phase names (e.g. Alcove bridge steps) which are appended after known phases. Also adds patch, follow-up, create-pr, and await-ci to PHASE_ORDER. 💖 Generated with Crush Assisted-by: Crush:claude-opus-4-6
Phases now sort by (generation, PHASE_ORDER) instead of (PHASE_ORDER, generation). This produces a timeline that reads as it actually happened: triage(1) → plan(1) → implement(1) → verify(1) → implement(2) → verify(2) → review(2) instead of grouping all implements together then all verifies together. 💖 Generated with Crush Assisted-by: Crush:claude-opus-4-6
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.
Summary
Fixes the raw JSON dump visible in drill-down phase sections. The HTML report now renders
output_structuredas formatted, readable HTML inline per-phase instead of raw JSON in a<pre>block.What changed
strip_session_datanow preserves a curated allowlist (STRUCTURED_DISPLAY_FIELDS) instead of removingoutput_structuredentirely, so structured sections render even without--include-sessions.Test plan
💖 Generated with Crush