umputun / revdiff
88%
master: 90%

LAST BUILD BRANCH: refs/tags/v1.6.1
DEFAULT BRANCH: master
Repo Added
Token zpy25XdTlXkHx5U3syYGatR9U5X0sk8oe regen
Build 747 Last
Files 71
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH style-extraction
branch: style-extraction
CHANGE BRANCH
x
Sync Branches

coverage: 88.4% (+0.2%) from 88.217%
24277629943

Pull #92

github

umputun 
fix(style): address Copilot review findings on PR #92

Apply four fixes from Copilot's PR review:

1. FileStatusMark color leak (renderer.go): when Colors.Normal is
   unset, the status color had no terminating ANSI sequence and
   bled into subsequent text on the same line. Fall back to ResetFg
   ("\033[39m") when normalFg is empty. Added regression test
   TestRenderer_FileStatusMark_emptyNormalFallsBackToResetFg.

2. FileReviewedMark color leak (renderer.go): same bug pattern as
   FileStatusMark — AddFg checkmark color would bleed when Normal
   was empty. Same fix. Added regression subtest
   emptyNormalFallsBackToResetFg.

3. Pin go-pkgz/enum generator version (enums.go): the go:generate
   directives used @latest, making regeneration non-reproducible.
   Pinned to @v0.7.0 matching the version in go.mod. Verified
   regeneration produces identical output.

4. Drop tautological NotNil assertion (resolver_test.go): the
   TestResolver_Style_coversAllKeys test asserted require.NotNil on
   lipgloss.Style return values, but lipgloss.Style is a struct
   value type that can never be nil. The assertion always passed.
   Replaced with `_ = r.Style(k)` to match the sibling
   TestResolver_Color_coversAllKeys pattern — the exhaustiveness
   check's real purpose is "no panic for any defined key", not a
   nil check.
Pull Request #92: refactor(ui): extract style sub-package with Resolver/Renderer/SGR

880 of 996 new or added lines in 19 files covered. (88.35%)

1 existing line in 1 file now uncovered.

6066 of 6862 relevant lines covered (88.4%)

382.9 hits per line

Relevant lines Covered
6862 RELEVANT LINES 6066 COVERED LINES
382.9 HITS PER LINE
Source Files on style-extraction

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
24277629943 style-extraction fix(style): address Copilot review findings on PR #92 Apply four fixes from Copilot's PR review: 1. FileStatusMark color leak (renderer.go): when Colors.Normal is unset, the status color had no terminating ANSI sequence and bled into subse... Pull #92 umputun github
88.4
24277631009 style-extraction fix(style): address Copilot review findings on PR #92 Apply four fixes from Copilot's PR review: 1. FileStatusMark color leak (renderer.go): when Colors.Normal is unset, the status color had no terminating ANSI sequence and bled into subse... Pull #92 umputun github
88.4
24277447099 style-extraction docs(plan): reflect ansi.go merged into color.go An early draft of the plan scoped a separate ansi.go + ansi_test.go for the Write helper, private ansiColor, and hexVal. During implementation those were consolidated into color.go / color_test.go ... Pull #92 umputun github
88.39
See All Builds (653)

Badge your Repo: revdiff

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh