Fixes typing issues and null check in github workflow (#9685)

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: arii <342438+arii@users.noreply.github.com>

Author: arii

.github/workflows/reusable-gemini-review.yml

@@ -398,7 +398,6 @@ jobs:
           PR_DIFF_FILE: pr.diff
           FAILED_CHECKS_JSON: ${{ env.FAILED_CHECKS_JSON }}
           # Pass other context vars
-        env:
           GEMINI_THOUGHT_SIGNATURE: ${{ needs.analyze-changes.outputs.thought-signature }}
         run: |
           pnpm tsx scripts/gemini-client.ts \
@@ -425,7 +424,7 @@ jobs:
             REVIEW_COMMENT=$(jq -r '.reviewComment // empty' review_result.json)
 
             # Check if there's a comment to post
-            if [ -n "$REVIEW_COMMENT" ]; then
+            if [[ -n "$REVIEW_COMMENT" && "$REVIEW_COMMENT" != "null" ]]; then
               # Create the commit hash line
               COMMIT_HASH_MSG="> Reviewed commit: \`${HEAD_SHA}\`"
 

scripts/gemini-client.ts

@@ -340,7 +340,13 @@ export async function generateContentWithFallback({
       const text = result.response.text()
 
       // Capture thought signature from the response if present
-      const capturedSignature = (result.response.candidates?.[0] as any)?.thought_signature
+      const capturedSignature = (
+        result.response.candidates?.[0] as NonNullable<
+          typeof result.response.candidates
+        >[number] & {
+          thought_signature?: string
+        }
+      )?.thought_signature
 
       return { text, thoughtSignature: capturedSignature }
     } catch (error: unknown) {