refactor: print stderr via fmt.Fprint with lipgloss.Sprintln instead of lipgloss.Fprintln

This commit is contained in:
2026-08-17 20:11:47 +04:00
parent e13eb9b8a8
commit c195ca2ab2
3 changed files with 23 additions and 27 deletions
+2 -6
View File
@@ -55,10 +55,6 @@ linters:
- wastedassign
settings:
errcheck:
exclude-functions:
- charm.land/lipgloss/v2.Println
- charm.land/lipgloss/v2.Fprintln(os.Stderr)
staticcheck:
checks:
- "all"
@@ -86,9 +82,9 @@ linters:
text: "SA5011"
- linters:
- errcheck
- gosec
text: "G104"
source: "lipgloss\\.(Print|Printf|Println|Fprint|Fprintf|Fprintln)\\("
source: "lipgloss\\.Println\\("
formatters:
enable:
+7 -7
View File
@@ -33,13 +33,13 @@ func main() {
utilsErr.OnType(func(err types.FieldValidationErrors) error {
for _, fieldErr := range err.Errors {
if fieldErr.Value == "" {
lipgloss.Fprintln(os.Stderr,
fmt.Fprint(os.Stderr, lipgloss.Sprintln(
config.StyleYellow.Render(fmt.Sprintf("[VALIDATION] Field '%s': ", fieldErr.Field))+fieldErr.Err.Error(),
)
))
} else {
lipgloss.Fprintln(os.Stderr,
fmt.Fprint(os.Stderr, lipgloss.Sprintln(
config.StyleYellow.Render(fmt.Sprintf("[VALIDATION] Field '%s' (%s): ", fieldErr.Field, fieldErr.Value))+fieldErr.Err.Error(),
)
))
}
}
os.Exit(1)
@@ -59,17 +59,17 @@ func main() {
)
_ = utilsErr.MustHandle(err,
utilsErr.OnType(func(err types.ProxyDialError) error {
lipgloss.Fprintln(os.Stderr, config.StyleRed.Render("[PROXY] ")+err.Error())
fmt.Fprint(os.Stderr, lipgloss.Sprintln(config.StyleRed.Render("[PROXY] ")+err.Error()))
os.Exit(1)
return nil
}),
utilsErr.OnSentinel(types.ErrScriptEmpty, func(err error) error {
lipgloss.Fprintln(os.Stderr, config.StyleRed.Render("[SCRIPT] ")+err.Error())
fmt.Fprint(os.Stderr, lipgloss.Sprintln(config.StyleRed.Render("[SCRIPT] ")+err.Error()))
os.Exit(1)
return nil
}),
utilsErr.OnType(func(err types.ScriptLoadError) error {
lipgloss.Fprintln(os.Stderr, config.StyleRed.Render("[SCRIPT] ")+err.Error())
fmt.Fprint(os.Stderr, lipgloss.Sprintln(config.StyleRed.Render("[SCRIPT] ")+err.Error()))
os.Exit(1)
return nil
}),
+14 -14
View File
@@ -248,7 +248,7 @@ func (config Config) MarshalYAML() (any, error) {
func (config Config) Print() bool {
configYAML, err := yaml.Marshal(config)
if err != nil {
lipgloss.Fprintln(os.Stderr, StyleRed.Render("Error marshaling config to yaml: "+err.Error()))
fmt.Fprint(os.Stderr, lipgloss.Sprintln(StyleRed.Render("Error marshaling config to yaml: "+err.Error())))
os.Exit(1)
}
@@ -267,13 +267,13 @@ func (config Config) Print() bool {
glamour.WithWordWrap(0),
)
if err != nil {
lipgloss.Fprintln(os.Stderr, StyleRed.Render(err.Error()))
fmt.Fprint(os.Stderr, lipgloss.Sprintln(StyleRed.Render(err.Error())))
os.Exit(1)
}
content, err := renderer.Render("```yaml\n" + string(configYAML) + "```")
if err != nil {
lipgloss.Fprintln(os.Stderr, StyleRed.Render(err.Error()))
fmt.Fprint(os.Stderr, lipgloss.Sprintln(StyleRed.Render(err.Error())))
os.Exit(1)
}
@@ -283,7 +283,7 @@ func (config Config) Print() bool {
m, err := p.Run()
if err != nil {
lipgloss.Fprintln(os.Stderr, StyleRed.Render(err.Error()))
fmt.Fprint(os.Stderr, lipgloss.Sprintln(StyleRed.Render(err.Error())))
os.Exit(1)
}
@@ -613,11 +613,11 @@ func ReadAllConfigs() *Config {
_ = utilsErr.MustHandle(err,
utilsErr.OnType(func(err types.CLIUnexpectedArgsError) error {
cliParser.PrintHelp()
lipgloss.Fprintln(os.Stderr,
fmt.Fprint(os.Stderr, lipgloss.Sprintln(
StyleYellow.Render(
"\nUnexpected CLI arguments provided: ",
)+strings.Join(err.Args, ", "),
)
))
os.Exit(1)
return nil
}),
@@ -635,20 +635,20 @@ func ReadAllConfigs() *Config {
_ = utilsErr.MustHandle(err,
utilsErr.OnType(func(err types.ConfigFileReadError) error {
cliParser.PrintHelp()
lipgloss.Fprintln(os.Stderr,
fmt.Fprint(os.Stderr, lipgloss.Sprintln(
StyleYellow.Render(
fmt.Sprintf("\nFailed to read config file (%s): ", configFile.Path())+err.Error(),
),
)
))
os.Exit(1)
return nil
}),
utilsErr.OnType(func(err types.UnmarshalError) error {
lipgloss.Fprintln(os.Stderr,
fmt.Fprint(os.Stderr, lipgloss.Sprintln(
StyleYellow.Render(
fmt.Sprintf("\nFailed to parse config file (%s): ", configFile.Path())+err.Error(),
),
)
))
os.Exit(1)
return nil
}),
@@ -751,13 +751,13 @@ func validateScriptSource(script string) error {
func printParseErrors(parserName string, errors ...types.FieldParseError) {
for _, fieldErr := range errors {
if fieldErr.Value == "" {
lipgloss.Fprintln(os.Stderr,
fmt.Fprint(os.Stderr, lipgloss.Sprintln(
StyleYellow.Render(fmt.Sprintf("[%s] Field '%s': ", parserName, fieldErr.Field))+fieldErr.Err.Error(),
)
))
} else {
lipgloss.Fprintln(os.Stderr,
fmt.Fprint(os.Stderr, lipgloss.Sprintln(
StyleYellow.Render(fmt.Sprintf("[%s] Field '%s' (%s): ", parserName, fieldErr.Field, fieldErr.Value))+fieldErr.Err.Error(),
)
))
}
}
}