fix: resolve hang by prioritizing authentication and unsetting conflicting credentials (minimal)

Author: cocosheng-g

action.yml

@@ -316,7 +316,7 @@ runs:
         if [[ -n "${GOOGLE_CLOUD_ACCESS_TOKEN:-}" ]]; then
           unset GEMINI_API_KEY
           unset GOOGLE_API_KEY
-          # Unset credential file pointers that might cause conflicts with the access token.
+          # Unset credential file pointers that might cause conflicts with the access token and hang.
           unset CLOUDSDK_AUTH_CREDENTIAL_FILE_OVERRIDE
           unset GOOGLE_GHA_CREDS_PATH
         elif [[ "${GOOGLE_GENAI_USE_VERTEXAI:-false}" == "true" && -n "${GOOGLE_API_KEY:-}" ]]; then
@@ -332,16 +332,32 @@ runs:
         # We capture stdout (JSON) to TEMP_STDOUT and stderr to TEMP_STDERR
         if [[ "${GEMINI_DEBUG}" = true ]]; then
           echo "::warning::Gemini CLI debug logging is enabled. This will stream responses, which could reveal sensitive information if processed with untrusted inputs."
+          echo "::: Start Gemini CLI STDOUT :::"
           if ! gemini --debug --yolo --prompt "${PROMPT}" --output-format json 2> >(tee "${TEMP_STDERR}" >&2) | tee "${TEMP_STDOUT}"; then
             FAILED=true
           fi
+          # Wait for async stderr logging to complete. This is because process substitution in Bash is async so let tee finish writing to ${TEMP_STDERR}
+          sleep 1
+          echo "::: End Gemini CLI STDOUT :::"
         else
           if ! gemini --yolo --prompt "${PROMPT}" --output-format json 2> "${TEMP_STDERR}" 1> "${TEMP_STDOUT}"; then
             FAILED=true
           fi
         fi
 
+        # Create the artifacts directory and copy full logs
+        mkdir -p gemini-artifacts
+        cp "${TEMP_STDOUT}" gemini-artifacts/stdout.log
+        cp "${TEMP_STDERR}" gemini-artifacts/stderr.log
+        if [[ -f .gemini/telemetry.log ]]; then
+          cp .gemini/telemetry.log gemini-artifacts/telemetry.log
+        else
+          # Create an empty file so the artifact upload doesn't fail if telemetry is missing
+          touch gemini-artifacts/telemetry.log
+        fi
+
         # Parse JSON output to extract response and errors
+        # If output is not valid JSON, RESPONSE will be empty and we'll rely on stderr for errors
         RESPONSE=""
         ERROR_JSON=""
         if jq -e . "${TEMP_STDOUT}" >/dev/null 2>&1; then
@@ -357,21 +373,32 @@ runs:
            fi
         fi
 
-        # Use a unique delimiter to avoid collisions for multiline outputs
+        if { [[ -s "${TEMP_STDERR}" ]] && [[ -z "${ERROR_JSON}" ]]; }; then
+          echo "::warning::Gemini CLI stderr contains data but no valid JSON error object was extracted"
+        fi
+
+        if { [[ -s "${TEMP_STDOUT}" ]] && ! jq -e . "${TEMP_STDOUT}" >/dev/null 2>&1; }; then
+          echo "::warning::Gemini CLI stdout was not valid JSON"
+        fi
+
+
+        # Set the captured response as a step output, supporting multiline
+        # Use a unique delimiter to avoid collisions
         EOF_DELIMITER="gh_gemini_out_$(cat /dev/urandom | tr -dc 'a-zA-Z0-9' | fold -w 8 | head -n 1)"
 
         echo "gemini_response<<${EOF_DELIMITER}" >> "${GITHUB_OUTPUT}"
         if [[ -n "${RESPONSE}" ]]; then
           echo "${RESPONSE}" >> "${GITHUB_OUTPUT}"
-        elif [[ -s "${TEMP_STDOUT}" ]]; then
+        else
           cat "${TEMP_STDOUT}" >> "${GITHUB_OUTPUT}"
         fi
         echo "${EOF_DELIMITER}" >> "${GITHUB_OUTPUT}"
 
+        # Set the captured errors as a step output, supporting multiline
         echo "gemini_errors<<${EOF_DELIMITER}" >> "${GITHUB_OUTPUT}"
         if [[ -n "${ERROR_JSON}" ]]; then
           echo "${ERROR_JSON}" >> "${GITHUB_OUTPUT}"
-        elif [[ -s "${TEMP_STDERR}" ]]; then
+        else
           cat "${TEMP_STDERR}" >> "${GITHUB_OUTPUT}"
         fi
         echo "${EOF_DELIMITER}" >> "${GITHUB_OUTPUT}"
@@ -417,6 +444,9 @@ runs:
              ERROR_MSG=$(jq -r '.message // .' <<< "${ERROR_JSON}")
              echo "::error title=Gemini CLI execution failed::${ERROR_MSG}"
           fi
+          echo "::: Start Gemini CLI STDERR :::"
+          cat "${TEMP_STDERR}"
+          echo "::: End Gemini CLI STDERR :::"
           exit 1
         fi
       env: