chore: swap conditions for efficiency, lol

35C4n0r · 35C4n0r · commit 147f55bd268d · 2025-08-01T20:51:44.000+05:30
diff --git a/cmd/server/server.go b/cmd/server/server.go
@@ -78,7 +78,7 @@ func runServer(ctx context.Context, logger *slog.Logger, argsToPass []string) er
 	}
 	if termHeight < 10 {
 		return xerrors.Errorf("term height must be at least 10")
-	} else if termHeight > 930 && agentType == AgentTypeCodex {
+	} else if agentType == AgentTypeCodex && termHeight > 930 {
 		logger.Warn(fmt.Sprintf("Term height is set to %d which may cause issues with Codex. Setting it to 930 instead.", termHeight))
 		termHeight = 930 // codex has a bug where the TUI distorts the screen if the height is too large, see: https://github.com/openai/codex/issues/1608
 	}

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ func runServer(ctx context.Context, logger *slog.Logger, argsToPass []string) er`
`78`	`78`	`}`
`79`	`79`	`if termHeight < 10 {`
`80`	`80`	`return xerrors.Errorf("term height must be at least 10")`
`81`		`- } else if termHeight > 930 && agentType == AgentTypeCodex {`
	`81`	`+ } else if agentType == AgentTypeCodex && termHeight > 930 {`
`82`	`82`	`logger.Warn(fmt.Sprintf("Term height is set to %d which may cause issues with Codex. Setting it to 930 instead.", termHeight))`
`83`	`83`	`termHeight = 930 // codex has a bug where the TUI distorts the screen if the height is too large, see: https://github.com/openai/codex/issues/1608`
`84`	`84`	`}`