Updated logging (#116)

* a41a3e []: updated logging

Author: arvind-choudhary-h

cmd/artifact/command/list.go

@@ -2,10 +2,12 @@ package command
 
 import (
 	"context"
+	"fmt"
 
 	client "github.com/harness/harness-cli/internal/api/ar"
 	client2 "github.com/harness/harness-cli/util/client"
 	"github.com/harness/harness-cli/util/common/printer"
+	"github.com/rs/zerolog/log"
 
 	"github.com/spf13/cobra"
 )
@@ -41,6 +43,11 @@ func NewListArtifactCmd(c *client.ClientWithResponses) *cobra.Command {
 				return err
 			}
 
+			if response.JSON200 == nil || response.JSON200.Data.Artifacts == nil {
+				log.Debug().Msgf("Unable to list artifacts")
+				return fmt.Errorf("unable to list artifacts")
+			}
+
 			err = printer.Print(response.JSON200.Data.Artifacts, *response.JSON200.Data.PageIndex,
 				*response.JSON200.Data.PageCount, *response.JSON200.Data.ItemCount, true, [][]string{
 					{"name", "Artifact"},

cmd/hc/main.go

@@ -10,14 +10,10 @@ import (
 	"runtime/pprof"
 	"time"
 
-	"github.com/harness/harness-cli/cmd/api"
 	"github.com/harness/harness-cli/cmd/artifact"
 	"github.com/harness/harness-cli/cmd/auth"
-	"github.com/harness/harness-cli/cmd/organisation"
-	"github.com/harness/harness-cli/cmd/project"
 	"github.com/harness/harness-cli/cmd/registry"
 	"github.com/harness/harness-cli/config"
-	"github.com/harness/harness-cli/module/ar/migrate/types"
 	"github.com/harness/harness-cli/util/templates"
 
 	"github.com/rs/zerolog"
@@ -34,8 +30,9 @@ func main() {
 	var verbose bool
 
 	rootCmd := &cobra.Command{
-		Use:   "hc",
-		Short: "CLI tool for Harness",
+		Use:          "hc",
+		Short:        "CLI tool for Harness",
+		SilenceUsage: true,
 		Long: templates.LongDesc(`
       Harness CLI is a tool to interact with Harness Resources.
 
@@ -69,14 +66,17 @@ func main() {
 				}
 			}
 
-			// Set up logging if verbose mode is enabled
+			// Set up logging based on verbose flag
 			if verbose {
 				logWriter := zerolog.ConsoleWriter{
 					Out:        os.Stderr,
 					TimeFormat: time.RFC3339,
 					NoColor:    false,
 				}
-				log.Logger = log.Output(logWriter).Hook(types.ErrorHook{})
+				log.Logger = log.Output(logWriter)
+			} else {
+				// Disable logging when verbose is not enabled
+				log.Logger = zerolog.Nop()
 			}
 
 			return initProfiling()
@@ -131,16 +131,16 @@ func main() {
 	rootCmd.AddCommand(auth.GetRootCmd())
 	rootCmd.AddCommand(registry.GetRootCmd())
 	rootCmd.AddCommand(artifact.GetRootCmd())
-	rootCmd.AddCommand(project.GetRootCmd())
-	rootCmd.AddCommand(organisation.GetRootCmd())
-	rootCmd.AddCommand(api.GetRootCmd())
+	//rootCmd.AddCommand(project.GetRootCmd())
+	//rootCmd.AddCommand(organisation.GetRootCmd())
+	//rootCmd.AddCommand(api.GetRootCmd())
 	rootCmd.AddCommand(versionCmd())
 	rootCmd.AddCommand(upgradeCmd())
 
 	flags := rootCmd.PersistentFlags()
 
 	addProfilingFlags(flags)
-	zerolog.TimeFieldFormat = zerolog.TimeFormatUnix
+	//zerolog.TimeFieldFormat = zerolog.TimeFormatUnix
 
 	if err := rootCmd.Execute(); err != nil {
 		fmt.Println(err)

util/common/printer/json.go

@@ -35,7 +35,7 @@ func DefaultJsonOptions() JsonOptions {
 		Indent:         true,
 		IndentPrefix:   "",
 		IndentSize:     2,
-		ShowPagination: true,
+		ShowPagination: false,
 	}
 }