fix: use VT100 box-drawing characters and semantic colors for output - #124
Open
rsnodgrass wants to merge 1 commit into
Open
fix: use VT100 box-drawing characters and semantic colors for output#124rsnodgrass wants to merge 1 commit into
rsnodgrass wants to merge 1 commit into
Conversation
rsnodgrass
force-pushed
the
fix/box-drawing-chars
branch
from
May 9, 2026 03:43
bfce31f to
1a2aa07
Compare
Replace ASCII pipes and dashes with the original VT100 box-drawing characters (├, └, │, ─, ┼) for crisp tree and table rendering. Tree output now correctly uses └── for last siblings and omits the vertical continuation line below terminal nodes. Added a space after the connector and before action suffixes for readability. Table output uses semantic colors per action type (green for add, red for delete, yellow for update, magenta for recreate, cyan for import/moved) via tablewriter Rich() API. Colors automatically respect NO_COLOR via the fatih/color library. Co-Authored-By: SageOx <ox@sageox.ai>
rsnodgrass
force-pushed
the
fix/box-drawing-chars
branch
from
May 9, 2026 03:50
1a2aa07 to
60b52fc
Compare
rsnodgrass
marked this pull request as ready for review
May 9, 2026 04:09
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
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
|---and|with VT100 box-drawing characters (├──,└──,│) in tree output for crisp rendering└──and omit the vertical bar below terminal nodesname (+)instead ofname(+)) for readability│,─,┼) instead of+,-,|for non-markdown modeNO_COLORenv via thefatih/colorlibrary (already a dependency)Before / After
Tree
Before:
After:
Table
Before:
After (with semantic colors):
graph LR A[writer/tree.go] -->|"├── └── │"| B[Crisp tree lines] C[writer/table.go] -->|"│ ─ ┼ + Rich()"| D[Box-drawing + colors] E[tree/tree.go] -->|"AddChild spacing"| F[Drawable tree spacing] B --> G[Better terminal output] D --> G F --> GTest plan
go test ./...)NO_COLOR=1disables table colorsCo-Authored-By: SageOx ox@sageox.ai