fix(harness-replays): use github.event.commits for push event detect-changes #499
Merged
core-lead
merged 2 commits from 2026-05-11 15:49:49 +00:00
ci/fix-detect-changes-commits-array into main
2 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
| 9a078e1163 |
Merge branch 'main' into ci/fix-detect-changes-commits-array
All checks were successful
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 21s
Harness Replays / detect-changes (pull_request) Successful in 21s
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 16s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 19s
CI / Detect changes (pull_request) Successful in 1m6s
sop-tier-check / tier-check (pull_request) Successful in 20s
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m5s
Harness Replays / Harness Replays (pull_request) Successful in 8s
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 1m10s
CI / Platform (Go) (pull_request) Successful in 9s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 1m11s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 13s
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 11s
CI / Python Lint & Test (pull_request) Successful in 12s
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 9s
CI / Canvas (Next.js) (pull_request) Successful in 16s
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 54s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 5s
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 5s
audit-force-merge / audit (pull_request) Successful in 15s
|
|||
| 5a70d1a1be |
fix(harness-replays): use github.event.commits for push event detect-changes
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 20s
cascade-list-drift-gate / check (pull_request) Successful in 25s
CI / Detect changes (pull_request) Successful in 1m16s
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m14s
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 1m11s
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 18s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 1m8s
Harness Replays / detect-changes (pull_request) Successful in 20s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 21s
sop-tier-check / tier-check (pull_request) Successful in 20s
CI / Canvas (Next.js) (pull_request) Successful in 9s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 8s
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 1m11s
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 13s
Harness Replays / Harness Replays (pull_request) Successful in 8s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
E2E Staging External Runtime / E2E Staging External Runtime (pull_request) Successful in 5m17s
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 2m50s
E2E API Smoke Test / E2E API Smoke Test (pull_request) Failing after 4m54s
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 5m39s
CI / Python Lint & Test (pull_request) Failing after 7m39s
CI / Platform (Go) (pull_request) Failing after 9m22s
Gitea Compare API rejects SHA-to-branch comparisons (returns "BaseNotExist"). The previous push-event fix (PR #497) used github.event.before (SHA) as BASE and GITHUB_REF (branch name) as HEAD — which fails. Fix: for push events, extract changed files directly from github.event.commits array (each commit has added/removed/ modified file lists). This is already in-memory from the push event payload — no extra API call needed. Pull request path continues to use Compare API (branch-to-branch works fine). New script: .gitea/scripts/push-commits-diff-files.py |
