fix(commands): drop double-print of modeline parse failures in root.go

Address review feedback from gemini-code-assist on root.go:255 and
coderabbitai on root.go:257 (same site, two reviewers).

processModelineAndUpdateGlobals printed 'Warning: modeline parsing
failed: <err>' to stderr AND returned errors.Wrapf(err, ...). The
caller threads that wrapped error up to cobra, which surfaces it
through its own stderr writer at command-end. Operators saw the
same underlying error twice — once with a misleading 'Warning:'
prefix, once with the proper command-error rendering.

Drop the explicit Fprintf; let cobra surface the single wrapped
error at the command boundary.

Signed-off-by: Aleksei Sviridkin <f@lex.la>

Author: lexfrei

pkg/commands/root.go

@@ -16,7 +16,6 @@ package commands
 
 import (
 	"context"
-	"fmt"
 	"os"
 	"path/filepath"
 	"time"
@@ -251,8 +250,10 @@ func DetectRootForTemplate(templatePath string) (string, error) {
 func processModelineAndUpdateGlobals(configFile string, nodesFromArgs, endpointsFromArgs, overwrite bool) ([]string, error) {
 	modelineConfig, err := modeline.ReadAndParseModeline(configFile)
 	if err != nil {
-		fmt.Fprintf(os.Stderr, "Warning: modeline parsing failed: %v\n", err)
-
+		// Don't print the error here — cobra surfaces the wrapped
+		// return through stderr at the command level. Printing here
+		// AND returning the wrap caused the same message to appear
+		// twice with a misleading "Warning:" prefix on the first copy.
 		return nil, errors.Wrapf(err, "parsing modeline in %s", configFile)
 	}