refactor: centralize command execution logic in cobra commands

Consolidate the command execution logic by setting `cmd.Run` directly in the constructor functions instead of within the `Command` method. This improves maintainability and consistency across all command implementations.

Author: vldcreation

cmd/encode.go

@@ -38,12 +38,13 @@ func NewEncodeCommand() *encodeCmd {
 
 	cmd.MarkPersistentFlagRequired("source")
 
+	cmd.Run = apps.Execute
+
 	apps.cmd = cmd
 	return apps
 }
 
 func (c *encodeCmd) Command() *cobra.Command {
-	c.cmd.Run = c.Execute
 	return c.cmd
 }
 

cmd/find.go

@@ -34,12 +34,13 @@ func NewFindCommand() *findCommand {
 
 	cmd.MarkPersistentFlagRequired("lang")
 
+	cmd.Run = apps.Execute
+
 	apps.cmd = cmd
 	return apps
 }
 
 func (c *findCommand) Command() *cobra.Command {
-	c.cmd.Run = c.Execute
 	return c.cmd
 }
 

cmd/password_gen.go

@@ -28,12 +28,12 @@ func NewGeneratePasswordCommand() *generatePasswordCmd {
 	cmd.PersistentFlags().IntVarP(&apps.len, "len", "l", 0, "Password length (words or chars)")
 	cmd.PersistentFlags().IntVarP(&apps.passType, "type", "t", 0, "Password type (0: word, 1: phrase, 2: word with special, 3: phrase with special, 4: secure)")
 
+	cmd.Run = apps.Execute
 	apps.cmd = cmd
 	return apps
 }
 
 func (c *generatePasswordCmd) Command() *cobra.Command {
-	c.cmd.Run = c.Execute
 	return c.cmd
 }
 

cmd/pull.go

@@ -32,12 +32,13 @@ func NewPullCommand() *pullCmd {
 	cmd.MarkPersistentFlagRequired("user")
 	cmd.MarkPersistentFlagRequired("repo")
 
+	cmd.Run = apps.Execute
+
 	apps.cmd = cmd
 	return apps
 }
 
 func (c *pullCmd) Command() *cobra.Command {
-	c.cmd.Run = c.Execute
 	return c.cmd
 }
 

cmd/runtest.go

@@ -34,13 +34,13 @@ func NewRunTestCommand() *runtestCmd {
 	cmd.MarkPersistentFlagRequired("file")
 	cmd.MarkPersistentFlagRequired("input")
 	cmd.MarkPersistentFlagRequired("output")
+	cmd.Run = apps.Execute
 
 	apps.cmd = cmd
 	return apps
 }
 
 func (c *runtestCmd) Command() *cobra.Command {
-	c.cmd.Run = c.Execute
 	return c.cmd
 }
 

cmd/setup.go

@@ -26,12 +26,13 @@ func NewSetupCommand() *setupCommand {
 
 	cmd.MarkPersistentFlagRequired("lang")
 
+	cmd.Run = apps.Execute
+
 	apps.cmd = cmd
 	return apps
 }
 
 func (c *setupCommand) Command() *cobra.Command {
-	c.cmd.Run = c.Execute
 	return c.cmd
 }
 

cmd/sharefile.go

@@ -27,12 +27,13 @@ func NewShareFileCommand() *shareFileCmd {
 	cmd.PersistentFlags().StringVarP(&apps.port, "port", "P", "9000", "Port of server")
 	cmd.MarkPersistentFlagRequired("root")
 
+	cmd.Run = apps.Execute
+
 	apps.cmd = cmd
 	return apps
 }
 
 func (c *shareFileCmd) Command() *cobra.Command {
-	c.cmd.Run = c.Execute
 	return c.cmd
 }
 

cmd/track.go

@@ -26,12 +26,13 @@ func NewTrackCommand() *trackCmd {
 
 	cmd.MarkPersistentFlagRequired("config")
 
+	cmd.Run = apps.Execute
+
 	apps.cmd = cmd
 	return apps
 }
 
 func (c *trackCmd) Command() *cobra.Command {
-	c.cmd.Run = c.Execute
 	return c.cmd
 }