Compile jq expression once, reuse in writer

PersistentPreRunE now caches the compiled *gojq.Code and passes it
to newJQWriterWithCode, avoiding a redundant parse + compile cycle.

Author: robzolkos

internal/commands/jq.go

@@ -31,6 +31,25 @@ func newJQWriter(dest io.Writer, filter string) (*jqWriter, error) {
 	return &jqWriter{dest: dest, code: code}, nil
 }
 
+// newJQWriterWithCode creates a jqWriter using a pre-compiled *gojq.Code.
+// Used when the expression has already been validated and compiled (e.g. in PersistentPreRunE).
+func newJQWriterWithCode(dest io.Writer, code *gojq.Code) *jqWriter {
+	return &jqWriter{dest: dest, code: code}
+}
+
+// compileJQ parses and compiles a jq expression, returning the compiled code.
+func compileJQ(filter string) (*gojq.Code, error) {
+	query, err := gojq.Parse(filter)
+	if err != nil {
+		return nil, errors.ErrJQValidation(err)
+	}
+	code, err := gojq.Compile(query, gojq.WithEnvironLoader(os.Environ))
+	if err != nil {
+		return nil, errors.ErrJQValidation(err)
+	}
+	return code, nil
+}
+
 // Write intercepts JSON output, applies the jq filter, and writes filtered results.
 // String results print as plain text; everything else prints as compact single-line JSON.
 // Error envelopes (ok: false) pass through unfiltered so error messages are never hidden.

internal/commands/root.go

@@ -77,20 +77,20 @@ Use fizzy to manage boards, cards, comments, and more from your terminal.`,
 	PersistentPreRunE: func(cmd *cobra.Command, args []string) error {
 		// Early jq validation: check flag conflicts first (actionable message),
 		// then parse + compile before RunE so invalid expressions are rejected
-		// with no side effects.
+		// with no side effects. The compiled code is reused below to avoid
+		// parsing the expression twice.
+		var jqCode *gojq.Code
 		if cfgJQ != "" {
 			if cfgIDsOnly {
 				return errors.ErrJQConflict("--ids-only")
 			}
 			if cfgCount {
 				return errors.ErrJQConflict("--count")
 			}
-			q, err := gojq.Parse(cfgJQ)
+			var err error
+			jqCode, err = compileJQ(cfgJQ)
 			if err != nil {
-				return errors.ErrJQValidation(err)
-			}
-			if _, err := gojq.Compile(q, gojq.WithEnvironLoader(os.Environ)); err != nil {
-				return errors.ErrJQValidation(err)
+				return err
 			}
 		}
 
@@ -103,23 +103,15 @@ Use fizzy to manage boards, cards, comments, and more from your terminal.`,
 			// Test mode — preserve test buffer as writer.
 			outWriter = &testBuf
 			var w io.Writer = &testBuf
-			if cfgJQ != "" {
-				jw, err := newJQWriter(&testBuf, cfgJQ)
-				if err != nil {
-					return err
-				}
-				w = jw
+			if jqCode != nil {
+				w = newJQWriterWithCode(&testBuf, jqCode)
 			}
 			out = output.New(output.Options{Format: format, Writer: w})
 		} else {
 			outWriter = os.Stdout
 			var w io.Writer = os.Stdout
-			if cfgJQ != "" {
-				jw, err := newJQWriter(os.Stdout, cfgJQ)
-				if err != nil {
-					return err
-				}
-				w = jw
+			if jqCode != nil {
+				w = newJQWriterWithCode(os.Stdout, jqCode)
 			}
 			out = output.New(output.Options{Format: format, Writer: w})
 		}