feat: add warning for excessive term height in Codex

35C4n0r · 35C4n0r · commit cd38beac037b · 2025-08-01T19:33:16.000+05:30
diff --git a/cmd/server/server.go b/cmd/server/server.go
@@ -78,6 +78,9 @@ 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 {
+		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
 	}
 
 	var process *termexec.Process

Original file line number	Diff line number	Diff line change
`@@ -78,6 +78,9 @@ 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 {`
	`82`	`+ logger.Warn(fmt.Sprintf("Term height is set to %d which may cause issues with Codex. Setting it to 930 instead.", termHeight))`
	`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`
`81`	`84`	`}`
`82`	`85`
`83`	`86`	`var process *termexec.Process`