johba/disinto
1
0
Fork 0
Code Issues 4 Pull requests Projects Releases Packages Wiki Activity Actions

fix: ci_required_for_pr should paginate /pulls/{pr}/files (#273)

Browse source 
Replace single-page curl fetch with codeberg_api_all() which paginates
through all pages of PR files. This ensures large PRs with 50+ files
correctly detect code files on all pages, preventing CI from being
incorrectly bypassed.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
openhands 2026-03-20 06:55:00 +00:00
parent e0a1c67224
commit 2a3e5eef90
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
lib/ci-helpers.sh
View file

@ -24,9 +24,9 @@ diff_has_code_files() {
# Returns 0 if PR has code files (CI required), 1 if non-code only (CI not required). # Returns 0 if PR has code files (CI required), 1 if non-code only (CI not required).
ci_required_for_pr() { ci_required_for_pr() {
local pr_num="$1" local pr_num="$1"
local files local files all_json
files=$(curl -sf -H "Authorization: token ${CODEBERG_TOKEN}" \ all_json=$(codeberg_api_all "/pulls/${pr_num}/files") || return 0
"${CODEBERG_API}/pulls/${pr_num}/files" | jq -r '.[].filename' 2>/dev/null) || return 0 files=$(printf '%s' "$all_json" | jq -r '.[].filename' 2>/dev/null) || return 0
if [ -z "$files" ]; then if [ -z "$files" ]; then
return 0 # empty file list — require CI as safety default return 0 # empty file list — require CI as safety default
fi fi