You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
So with NO_COLOR=1 the recipe's own blue/green/yellow highlighting is
correctly dropped, yet diff still writes raw ANSI escapes. The result is
partially-colored output that no NO_COLOR consumer asked for, and escape
bytes leaking into logs, pipes, and ujust check-local-overrides > file.
Reproduced while writing BATS coverage for this recipe (PR #1048). Under NO_COLOR=1 the output for a file present only in /etc is still:
^[[0m^[[0m: custom.conf^[[0m
RESET="\x1b[0m" is also assigned unconditionally, so it is emitted by the
final sed rule even in the NO_COLOR=1 branch — same class of bug.
Recommendation
Make the diff color mode follow NO_COLOR, e.g. select --color="never" when NO_COLOR=1 and --color="always" otherwise
(or use --color="auto" and let diff decide from the tty).
Clear RESET alongside the other palette variables in the NO_COLOR=1
branch so the trailing sed rules emit no escapes at all.
Finding
check-local-overridesinsystem_files/shared/usr/share/ublue-os/just/default.justbranches onNO_COLORfor the palette it applies itself:…but the
diffinvocation two lines later hardcodes:diff -qr \ --suppress-common-lines \ --color="always" \So with
NO_COLOR=1the recipe's own blue/green/yellow highlighting iscorrectly dropped, yet
diffstill writes raw ANSI escapes. The result ispartially-colored output that no
NO_COLORconsumer asked for, and escapebytes leaking into logs, pipes, and
ujust check-local-overrides > file.Reproduced while writing BATS coverage for this recipe (PR #1048). Under
NO_COLOR=1the output for a file present only in/etcis still:RESET="\x1b[0m"is also assigned unconditionally, so it is emitted by thefinal
sedrule even in theNO_COLOR=1branch — same class of bug.Recommendation
diffcolor mode followNO_COLOR, e.g. select--color="never"whenNO_COLOR=1and--color="always"otherwise(or use
--color="auto"and letdiffdecide from the tty).RESETalongside the other palette variables in theNO_COLOR=1branch so the trailing
sedrules emit no escapes at all.check-local-overrides: diff --color=always is unconditional (NO_COLOR gap).Whoever fixes this should flip that test to assert the escape-free output;
it is written to make the fix visible rather than to bless the bug.
This is a production-code change in
default.just, so the quality agent isfiling it rather than patching it.
Priority
Filed by quality agent (hold-gated mode)
🐝 Hive Agent:
quality| Instance:hosted-projectbluefin-knuckle-gjvq| SHA:56961d4— hive: agent=quality backend=copilot model=claude-opus-5