chore: log the required flag errors to stderr

Author: rhamzeh

cmd/model/get.go

@@ -79,7 +79,7 @@ func init() {
 	getCmd.Flags().Var(&getOutputFormat, "format", `Authorization model output format. Can be "fga" or "json"`)
 
 	if err := flags.SetFlagRequired(getCmd, "store-id", "cmd/models/get", false); err != nil {
-		fmt.Printf("%v\n", err)
+		_, _ = fmt.Fprintln(os.Stderr, err)
 		os.Exit(1)
 	}
 }

cmd/model/list.go

@@ -114,7 +114,7 @@ func init() {
 	listCmd.Flags().StringArray("field", []string{"id", "created_at"}, "Fields to display, choices are: id, created_at and model") //nolint:lll
 
 	if err := flags.SetFlagRequired(listCmd, "store-id", "cmd/models/list", false); err != nil {
-		fmt.Printf("%v\n", err)
+		_, _ = fmt.Fprintln(os.Stderr, err)
 		os.Exit(1)
 	}
 }

cmd/model/test.go

@@ -101,7 +101,7 @@ func init() {
 	testCmd.Flags().Bool("suppress-summary", false, "Suppress the plain text summary output")
 
 	if err := flags.SetFlagRequired(testCmd, "tests", "cmd/models/test", false); err != nil {
-		fmt.Printf("%v\n", err)
+		_, _ = fmt.Fprintln(os.Stderr, err)
 		os.Exit(1)
 	}
 }

cmd/model/write.go

@@ -108,7 +108,7 @@ func init() {
 	writeCmd.Flags().Var(&writeInputFormat, "format", `Authorization model input format. Can be "fga", "json", or "modular"`) //nolint:lll
 
 	if err := flags.SetFlagRequired(writeCmd, "store-id", "cmd/model/write", false); err != nil {
-		fmt.Printf("%v\n", err)
+		_, _ = fmt.Fprintln(os.Stderr, err)
 		os.Exit(1)
 	}
 }

cmd/query/list-users.go

@@ -143,7 +143,7 @@ func init() {
 		listUsersCmd,
 		[]string{"object", "relation", "user-filter"},
 		"cmd/query/list-users", false); err != nil {
-		fmt.Printf("%v\n", err)
+		_, _ = fmt.Fprintln(os.Stderr, err)
 		os.Exit(1)
 	}
 }

cmd/query/query.go

@@ -51,7 +51,7 @@ func init() {
 	)
 
 	if err := flags.SetFlagRequired(QueryCmd, "store-id", "cmd/query/query", true); err != nil {
-		fmt.Printf("%v\n", err)
+		_, _ = fmt.Fprintln(os.Stderr, err)
 		os.Exit(1)
 	}
 }

cmd/store/delete.go

@@ -74,7 +74,7 @@ func init() {
 	deleteCmd.Flags().Bool("force", false, "Force delete without confirmation")
 
 	if err := flags.SetFlagRequired(deleteCmd, "store-id", "cmd/store/delete", false); err != nil {
-		fmt.Printf("%v\n", err)
+		_, _ = fmt.Fprintln(os.Stderr, err)
 		os.Exit(1)
 	}
 }

cmd/store/export.go

@@ -195,7 +195,7 @@ func init() {
 	exportCmd.Flags().Uint("max-tuples", defaultMaxTupleCount, "max number of tuples to return in the output")
 
 	if err := flags.SetFlagRequired(exportCmd, "store-id", "cmd/store/export", false); err != nil {
-		fmt.Printf("%v\n", err)
+		_, _ = fmt.Fprintln(os.Stderr, err)
 		os.Exit(1)
 	}
 }

cmd/store/get.go

@@ -67,7 +67,7 @@ func init() {
 	getCmd.Flags().String("store-id", "", "Store ID")
 
 	if err := flags.SetFlagRequired(getCmd, "store-id", "cmd/store/get", false); err != nil {
-		fmt.Printf("%v\n", err)
+		_, _ = fmt.Fprintln(os.Stderr, err)
 		os.Exit(1)
 	}
 }

cmd/store/import.go

@@ -341,7 +341,7 @@ func init() {
 	importCmd.Flags().Int("max-parallel-requests", tuple.MaxParallelRequests, "Max number of requests to issue to the server in parallel.") //nolint:lll
 
 	if err := flags.SetFlagRequired(importCmd, "file", "cmd/store/import", false); err != nil {
-		fmt.Printf("%v\n", err)
+		_, _ = fmt.Fprintln(os.Stderr, err)
 		os.Exit(1)
 	}
 }