fix: escape file paths in GraphQL expressions to prevent query syntax errors

[njlazzar-su]

Summary

Escape file paths before interpolating them into GraphQL query strings to prevent syntax errors.

Problem

In both fetch_file_contents_batch() and fetch_file_contents_with_base(), file paths from PRs are interpolated directly into GraphQL expression strings using f-strings:

expression = f"{head_sha}:{path}"
file_fields.append(f"file{i}: object(expression: \"{expression}\") {{ ... }}")

If a filename contains double quotes (e.g., src/"quoted"/file.py) or backslashes (common in Windows-originated PRs), the resulting GraphQL query will have broken syntax. This causes the entire batch file fetch to fail, and all files in the PR receive a score of 0 (since content_pair will be None).

Fix

Escape backslashes and double quotes in file paths before interpolation:

safe_path = path.replace("\\", "\\\\").replace("\"" , "\\\"")

Applied to both fetch_file_contents_batch() and fetch_file_contents_with_base().

Impact

Without this fix, any PR containing files with special characters in their paths would silently score 0 for all files in the batch fetch, potentially zeroing out an entire PR score.

[anderdc]

needs to target test branch

[njlazzar-su]

fixed

[njlazzar-su]

Hi @anderdc what are next steps here?