refactor: remove dump-raw-comm

Author: xtuc

cmd/ba/main.go

@@ -93,7 +93,7 @@ func makeapp() *cli.App {
 				cli.BoolFlag{Name: "debug", Usage: "Enable debug logging"},
 				cli.BoolFlag{Name: "quiet", Usage: "Decrease verbosity of the output"},
 				cli.BoolFlag{Name: "profile", Usage: "Enable execution profiling"},
-				cli.BoolFlag{Name: "dump-raw-comm", Usage: "Dump all the communication between the agent and the server"},
+				cli.BoolFlag{Name: "watch", Usage: "Enable watch mode"},
 				cli.IntFlag{Name: "duration", Usage: "If set, game will stop after this durarion (in seconds)"},
 			},
 			Action: func(c *cli.Context) error {
@@ -110,7 +110,7 @@ func makeapp() *cli.App {
 					IsDebug:         c.Bool("debug"),
 					IsQuiet:         c.Bool("quiet"),
 					ShouldProfile:   c.Bool("profile"),
-					DumpRaw:         c.Bool("dump-raw-comm"),
+					WatchMode:       c.Bool("watch"),
 					DurationSeconds: c.Int("duration"),
 				}
 

subcommand/train/main.go

@@ -61,9 +61,9 @@ type TrainActionArguments struct {
 	Agentimages     []string
 	IsDebug         bool
 	IsQuiet         bool
+	WatchMode       bool
 	MapName         string
 	ShouldProfile   bool
-	DumpRaw         bool
 	DurationSeconds int
 }
 
@@ -136,6 +136,7 @@ func TrainAction(args TrainActionArguments) (bool, error) {
 		arenaServerUUID,
 		brokerclient,
 		gameDuration,
+		args.IsDebug,
 	)
 
 	for _, dockerImageName := range args.Agentimages {
@@ -184,8 +185,8 @@ func TrainAction(args TrainActionArguments) (bool, error) {
 				fmt.Printf(HeadsUpColor("[headsup] %s\n"), t.Value)
 
 			case arenaserver.EventRawComm:
-				if args.DumpRaw {
-					fmt.Printf(AgentColor("[agent] %s\n"), t.Value)
+				if args.IsDebug {
+					fmt.Printf(DebugColor("[debug from: %s] %s\n"), t.From, t.Value)
 				}
 
 			case arenaserver.EventClose: