?
PLOW-16
feature,visualization
Created: 2025-12-21 Updated: 2025-12-21
Relationships Loading...
Attachments
Loading...
Comments (2)
code-verification-qa · 2025-12-21
[QA] ✅ VERIFICATION PASSED All acceptance criteria met: - render_workflow_ascii() returns string with ASCII diagram (597 chars) - Output contains all 12 workflow step names - Renders without errors - print_workflow() convenience function works correctly - All 32 visualization tests pass (11 new tests added) - phart v1.1.4 dependency properly integrated Test Results: 32/32 tests passing Manual Testing: All functions work as expected Code Quality: Clean implementation with comprehensive docstrings Minor Note: One unused loop variable detected by ruff (line 180) - non-blocking style issue Full QA report: /tmp/qa-PLOW-16.md Status: Ready for production
agent · 2025-12-21
Implemented render_workflow_ascii() and print_workflow() functions in visualization.py. ## Changes Made **src/trckr_plow/visualization.py:** - Added import for phart.ASCIIRenderer - Added render_workflow_ascii() function (line 148-184): - Builds workflow graph using existing build_workflow_graph() - Creates display graph with human-readable node labels from 'name' attribute - Uses phart's ASCIIRenderer to render the graph as ASCII art - Returns the ASCII diagram as a string - Added print_workflow() convenience function (line 187-200): - Calls render_workflow_ascii() and prints to stdout **tests/test_visualization.py:** - Added 11 new tests for both functions - TestRenderWorkflowAscii: 8 tests for render_workflow_ascii() - TestPrintWorkflow: 3 tests for print_workflow() ## Verification - All 517 tests pass - visualization.py has 100% code coverage - Output contains expected step names: Check Status, Pick Issue, Check Issue Status, etc.