Skip to content

fix(ci): use git diff for path filtering#2029

Merged
wesbillman merged 1 commit into
mainfrom
chotchkies/ci-paths-filter-git-mode
Jul 17, 2026
Merged

fix(ci): use git diff for path filtering#2029
wesbillman merged 1 commit into
mainfrom
chotchkies/ci-paths-filter-git-mode

Conversation

@cameronhotchkies

Copy link
Copy Markdown
Contributor

Why

Fork pull requests fail before CI fan-out because dorny/paths-filter calls GitHub's pull-request files API with the workflow token and receives a 500 response.

What

  • Force dorny/paths-filter into git-diff mode with token: ''
  • Fetch both parents required to diff the pull request merge commit
  • Move the fake Postgres credential out of inline URIs to satisfy secret scanning on workflow edits

Risk Assessment

Low — this only changes path detection and test-only workflow environment wiring. The main risk is incorrect job selection if the shallow checkout lacks a required parent; fetch-depth: 2 supplies both merge parents.

References

  • git diff --check origin/main...HEAD passed
  • Pre-push desktop, mobile, Tauri, and Rust unit tests passed; integration setup was blocked because Docker was not running locally

Generated with Codex

Force dorny/paths-filter to avoid the pull request files REST API so fork PRs do not fail before the real CI jobs are scheduled.

Move the fake CI Postgres password out of inline connection URIs so the workflow edit passes the repository secret scanner.

Co-authored-by: Cameron Hotchkies <chotchkies@block.xyz>
Signed-off-by: Cameron Hotchkies <chotchkies@block.xyz>
@cameronhotchkies
cameronhotchkies marked this pull request as ready for review July 17, 2026 16:32
@cameronhotchkies
cameronhotchkies requested a review from a team as a code owner July 17, 2026 16:32
@wesbillman
wesbillman merged commit 92040e5 into main Jul 17, 2026
26 checks passed
@wesbillman
wesbillman deleted the chotchkies/ci-paths-filter-git-mode branch July 17, 2026 17:07
kalvinnchau added a commit that referenced this pull request Jul 17, 2026
* origin/main: (31 commits)
  chore(release): release Buzz Desktop version 0.4.14 (#2036)
  Strip media metadata on clients and reject it at the relay (#2006)
  fix(desktop): restore Doctor ACP login discovery (#2017)
  feat(desktop): animate starter team onboarding (#2032)
  fix(ci): use git diff for path filtering (#2029)
  fix(desktop): include exe dir and ~/.local/bin in provider discovery (#2007)
  [codex] Hold Git concurrency permits through streaming (BUZZ-SEC-018) (#1916)
  [codex] Enforce shared relay admission limits (BUZZ-SEC-019) (#1917)
  [codex] Block banned actors from moderation commands (BUZZ-SEC-007) (#1915)
  [codex] Fix relay WebSocket admission limits (#1682)
  fix(desktop): close mention popup after an exact name + trailing space (#2030)
  test(desktop): harden observer archive policy regression coverage (#1994)
  [codex] Bind release publishers to immutable tags (BUZZ-SEC-039) (#1914)
  fix: resolve invite downloads by platform (#2001)
  Bug-bash: mention/code-span, feed titles, autocomplete, edit-mentions, draft routing, and right-click media Download (#2027)
  fix(desktop): keep mesh allowlist on transient roster query failure (#2024)
  Polish Buzz theme and sidebar (#1971)
  Welcome new users with a live agent team kickoff (#1998)
  fix(desktop): case-insensitive relay media origin match, release 0.4.13 (#2023)
  feat(desktop): show owners in agent hover cards (#2015)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants