fix(cli): preserve concrete parse errors

Purpose of the change:
- Address CodeRabbit feedback on the CLI unknown-option formatter.
- Ensure concrete parser errors are not replaced by later unknown-option text.

How behavior was before:
- Any parse failure could trigger an argument scan for unknown options.
- A command like -R not-an-int --blah could report --blah instead of the invalid integer value.

Why that was a problem:
- It hid the real parse failure and made CLI diagnostics less accurate.
- The Apprise-style unknown-option formatter was broader than the condition it was meant to handle.

What the new change accomplishes:
- Limits Apprise-style unknown-option formatting to actual unknown-flag parse errors.
- Leaves other parse failures on the standard parser error path.
- Adds regression coverage for invalid integer parsing followed by an unknown option.

How it works:
- Adds a focused parse-error classifier for the Go flag unknown-option error prefix.
- Keeps the existing unknown-option scan only inside that classifier branch.

Author: elibosley

internal/cli/cli.go

@@ -206,9 +206,11 @@ func Run(args []string, stdout, stderr io.Writer) int {
 			printHelp(stdout)
 			return 0
 		}
-		if option := unknownOption(args, fs); option != "" {
-			printUnknownOption(stderr, option)
-			return 2
+		if isUnknownFlagParseError(err) {
+			if option := unknownOption(args, fs); option != "" {
+				printUnknownOption(stderr, option)
+				return 2
+			}
 		}
 		fmt.Fprintln(stderr, err)
 		return 2
@@ -341,6 +343,10 @@ func printUnknownOption(w io.Writer, option string) {
 	fmt.Fprintf(w, "Error: No such option '%s'.\n", option)
 }
 
+func isUnknownFlagParseError(err error) bool {
+	return strings.HasPrefix(err.Error(), "flag provided but not defined: ")
+}
+
 func defaultCliOptions() cliOptions {
 	return cliOptions{
 		notificationType: string(notify.NotifyInfo),

internal/cli/cli_test.go

@@ -160,6 +160,23 @@ func TestCLILegacyFlagFormsStillParse(t *testing.T) {
 	}
 }
 
+func TestCLIParseErrorsAreNotReportedAsLaterUnknownOptions(t *testing.T) {
+	result := runGoCLI("-R", "not-an-int", "--blah")
+
+	if result.code != 2 {
+		t.Fatalf("expected parse failure exit code, got code=%d stdout=%q stderr=%q", result.code, result.stdout, result.stderr)
+	}
+	if result.stdout != "" {
+		t.Fatalf("expected empty stdout, got %q", result.stdout)
+	}
+	if !strings.Contains(result.stderr, `invalid value "not-an-int" for flag -R`) {
+		t.Fatalf("expected invalid integer parse error, got stderr=%q", result.stderr)
+	}
+	if strings.Contains(result.stderr, "No such option") {
+		t.Fatalf("expected parse error not unknown-option error, got stderr=%q", result.stderr)
+	}
+}
+
 func TestRunConvertsMarkdownInputForHTMLTargetFormat(t *testing.T) {
 	testutil.RequirePythonApprise(t)