fix: parse Codex JSONL responses in Ralph loop (#89)

Author: LarsCowe

ralph/drivers/claude-code.sh

@@ -112,7 +112,49 @@ driver_supports_sessions() {
     return 0  # true
 }
 
-# Stream filter for live output (jq filter for JSON streaming)
+# Claude Code supports stream-json live output.
+driver_supports_live_output() {
+    return 0  # true
+}
+
+# Prepare command arguments for live stream-json output.
+driver_prepare_live_command() {
+    LIVE_CMD_ARGS=()
+    local skip_next=false
+
+    for arg in "${CLAUDE_CMD_ARGS[@]}"; do
+        if [[ "$skip_next" == "true" ]]; then
+            LIVE_CMD_ARGS+=("stream-json")
+            skip_next=false
+        elif [[ "$arg" == "--output-format" ]]; then
+            LIVE_CMD_ARGS+=("$arg")
+            skip_next=true
+        else
+            LIVE_CMD_ARGS+=("$arg")
+        fi
+    done
+
+    if [[ "$skip_next" == "true" ]]; then
+        return 1
+    fi
+
+    LIVE_CMD_ARGS+=("--verbose" "--include-partial-messages")
+}
+
+# Stream filter for raw Claude stream-json events.
 driver_stream_filter() {
-    echo '.content // empty | select(type == "string")'
+    echo '
+        if .type == "stream_event" then
+            if .event.type == "content_block_delta" and .event.delta.type == "text_delta" then
+                .event.delta.text
+            elif .event.type == "content_block_start" and .event.content_block.type == "tool_use" then
+                "\n\n⚡ [" + .event.content_block.name + "]\n"
+            elif .event.type == "content_block_stop" then
+                "\n"
+            else
+                empty
+            end
+        else
+            empty
+        end'
 }

ralph/drivers/codex.sh

@@ -75,7 +75,16 @@ driver_supports_sessions() {
     return 0  # true - Codex supports session resume
 }
 
+# Codex JSONL output is already suitable for live display.
+driver_supports_live_output() {
+    return 0  # true
+}
+
+driver_prepare_live_command() {
+    LIVE_CMD_ARGS=("${CLAUDE_CMD_ARGS[@]}")
+}
+
 # Codex outputs JSONL events
 driver_stream_filter() {
-    echo 'select(.type == "message") | .content // empty'
+    echo 'select(.type == "item.completed" and .item.type == "agent_message") | (.item.text // ([.item.content[]? | select(.type == "output_text") | .text] | join("\n")) // empty)'
 }

ralph/drivers/copilot.sh

@@ -82,6 +82,11 @@ driver_supports_sessions() {
     return 1  # false — session IDs not capturable from -p output
 }
 
+# Copilot CLI does not expose structured live output for jq streaming.
+driver_supports_live_output() {
+    return 1  # false
+}
+
 # Copilot CLI outputs plain text only (no JSON streaming).
 # Passthrough filter — no transformation needed.
 driver_stream_filter() {

ralph/drivers/cursor.sh

@@ -5,7 +5,7 @@
 # Known limitations:
 # - CLI is in beta — binary name and flags may change
 # - NDJSON stream format assumes {type: "text", content: "..."} events
-# - Session ID capture from output not yet validated
+# - Session continuity is disabled until Cursor exposes a stable capturable session ID
 
 driver_name() {
     echo "cursor"
@@ -85,11 +85,6 @@ driver_build_command() {
     # NDJSON streaming output
     CLAUDE_CMD_ARGS+=("--output-format" "stream-json")
 
-    # Session resume — gated on CLAUDE_USE_CONTINUE to respect --no-continue flag
-    if [[ "$CLAUDE_USE_CONTINUE" == "true" && -n "$session_id" ]]; then
-        CLAUDE_CMD_ARGS+=("--resume" "$session_id")
-    fi
-
     # Build prompt with context prepended
     local prompt_content
     if driver_running_on_windows; then
@@ -107,7 +102,15 @@ $prompt_content"
 }
 
 driver_supports_sessions() {
-    return 0  # true — Cursor supports --resume
+    return 1  # false — session IDs are not capturable from current NDJSON output
+}
+
+driver_supports_live_output() {
+    return 0  # true
+}
+
+driver_prepare_live_command() {
+    LIVE_CMD_ARGS=("${CLAUDE_CMD_ARGS[@]}")
 }
 
 # Cursor CLI outputs NDJSON events