scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-01 02:02:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 92

fix(ci): use jq filter in close_duplicate_issues to avoid JSON parse error

Browse source 
The script previously called `gh api --paginate` and split the raw
response on newlines before json.loads()'ing each line. This is unsafe
because each page is a JSON array on a single conceptual line, and any
issue `body` field containing literal newlines makes splitlines() cut
the JSON apart, producing fragments that fail with
`Unterminated string starting at: line 1 column ...`.

Pass `--jq '.[]'` so gh emits each issue as a single compact JSON
object per line (with embedded newlines escaped). The loop then parses
one object per line safely.

Co-authored-by: Krrish Dholakia <krrish-berri-2@users.noreply.github.com>
This commit is contained in:
Cursor Agent 2026-05-21 22:12:07 +00:00
parent b55749248d
commit 1672e94901
No known key found for this signature in database
1 changed files with 10 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
.github/scripts/close_duplicate_issues.py vendored
View file

@ -40,27 +40,28 @@ def gh(*args: str) -> str:
def fetch_open_issues(repo: str | None) -> list[dict]:
"""Fetch all open issues (excluding PRs) via gh api --paginate."""
"""Fetch all open issues (excluding PRs) via gh api --paginate.
Uses ``--jq '.[]'`` so each issue is emitted as a single compact JSON
object per line. Splitting on newlines is unsafe on the raw array output
because issue ``body`` fields can contain literal newlines that confuse
line-based JSON parsing.
"""
if repo:
endpoint = (
f"repos/{repo}/issues?state=open&per_page=100&sort=created&direction=asc"
)
else:
endpoint = "repos/{owner}/{repo}/issues?state=open&per_page=100&sort=created&direction=asc"
cmd = ["api", "--paginate", endpoint]
cmd = ["api", "--paginate", "--jq", ".[]", endpoint]
raw = gh(*cmd)
# gh --paginate concatenates JSON arrays, so we may get multiple arrays
issues = []
for line in raw.strip().splitlines():
for line in raw.splitlines():
line = line.strip()
if not line:
continue
parsed = json.loads(line)
if isinstance(parsed, list):
issues.extend(parsed)
else:
issues.append(parsed)
issues.append(json.loads(line))
# Filter out pull requests (they also appear in the issues endpoint)
return [i for i in issues if "pull_request" not in i]