Address PR review feedback

- Check write error from fmt.Fprintln for non-string jq results
- Move flag conflict checks before Parse/Compile so the user sees
  the actionable conflict message first

Author: robzolkos

internal/commands/jq.go

@@ -68,7 +68,9 @@ func (w *jqWriter) Write(p []byte) (int, error) {
 			if err != nil {
 				return 0, errors.ErrJQRuntime(fmt.Errorf("result not serializable: %w", err))
 			}
-			fmt.Fprintln(w.dest, string(raw))
+			if _, err := fmt.Fprintln(w.dest, string(raw)); err != nil {
+				return 0, err
+			}
 		}
 	}
 	return len(p), nil

internal/commands/root.go

@@ -75,22 +75,23 @@ var rootCmd = &cobra.Command{
 Use fizzy to manage boards, cards, comments, and more from your terminal.`,
 	Version: "dev",
 	PersistentPreRunE: func(cmd *cobra.Command, args []string) error {
-		// Early jq validation: parse + compile before RunE so invalid
-		// expressions are rejected with no side effects.
+		// Early jq validation: check flag conflicts first (actionable message),
+		// then parse + compile before RunE so invalid expressions are rejected
+		// with no side effects.
 		if cfgJQ != "" {
+			if cfgIDsOnly {
+				return errors.ErrJQConflict("--ids-only")
+			}
+			if cfgCount {
+				return errors.ErrJQConflict("--count")
+			}
 			q, err := gojq.Parse(cfgJQ)
 			if err != nil {
 				return errors.ErrJQValidation(err)
 			}
 			if _, err := gojq.Compile(q, gojq.WithEnvironLoader(os.Environ)); err != nil {
 				return errors.ErrJQValidation(err)
 			}
-			if cfgIDsOnly {
-				return errors.ErrJQConflict("--ids-only")
-			}
-			if cfgCount {
-				return errors.ErrJQConflict("--count")
-			}
 		}
 
 		// Resolve output format from parsed flags (must happen post-parse).