From 70061c4008bdafc3bc484f3152aec4bc505b0f70 Mon Sep 17 00:00:00 2001
From: gmega <giuliano.mega@gmail.com>
Date: Fri, 27 Jun 2025 19:33:52 -0300
Subject: [PATCH] feat: allow setting Codex log level

---
 src/codex.bash       | 12 +++++++++++-
 test/test_codex.bats |  8 ++++++++
 2 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/src/codex.bash b/src/codex.bash
index 117650d..2140d7a 100644
--- a/src/codex.bash
+++ b/src/codex.bash
@@ -21,6 +21,8 @@ if [ ! -f "${_cdx_binary}" ]; then
   exit 1
 fi
 
+echoerr "[codex] Using binary at ${_cdx_binary}"
+
 # Custom prefix for timing logs
 _cdx_timing_prefix=""
 # Log file where timings are aggregated
@@ -32,6 +34,10 @@ _cdx_base_metrics_port=8290
 _cdx_node_start_timeout=30
 # Default options set for Codex nodes
 _cdx_defaultopts=()
+# Log level for Codex nodes.
+_cdx_log_level="INFO"
+
+echoerr "[codex] Node log level is ${_cdx_log_level}"
 
 # PID array for known Codex node processes
 declare -A _cdx_pids
@@ -83,6 +89,10 @@ cdx_clear_defaultopts() {
   _cdx_defaultopts=()
 }
 
+cdx_set_log_level() {
+  _cdx_log_level="$1"
+}
+
 cdx_cmdline() {
   local node_index spr cdx_cmd="${_cdx_binary} --nat:none" opts=("$@")
 
@@ -118,7 +128,7 @@ cdx_cmdline() {
   # shellcheck disable=SC2140
   echo "${cdx_cmd}"\
  "--data-dir=${_cdx_data}/codex-${node_index} --api-port=$(_cdx_api_port "$node_index")"\
- "--disc-port=$(_cdx_disc_port "$node_index") --log-level=INFO"
+ "--disc-port=$(_cdx_disc_port "$node_index") --log-level=${_cdx_log_level}"
 }
 
 cdx_get_spr() {
diff --git a/test/test_codex.bats b/test/test_codex.bats
index 432d86d..d6e6c04 100755
--- a/test/test_codex.bats
+++ b/test/test_codex.bats
@@ -39,6 +39,14 @@ setup() {
 " --api-port=8080 --disc-port=8190 --log-level=INFO"
 }
 
+@test "should modify the Codex log-level when specified" {
+  cdx_set_log_level "DEBUG"
+
+  assert_equal "$(cdx_cmdline 0)" "${_cdx_binary} --nat:none"\
+" --data-dir=${_cdx_output}/data/codex-0"\
+" --api-port=8080 --disc-port=8190 --log-level=DEBUG"
+}
+
 @test "should allow setting of global default options" {
   [[ ! "$(cdx_cmdline 0)" =~ "--metrics --metrics-port=8290 --metrics-address=0.0.0.0" ]]