Skip to content

fix(ci): paths-filter outputs not propagating to downstream jobs#72

Merged
abueide merged 2 commits intomainfrom
fix/paths-filter-outputs
May 8, 2026
Merged

fix(ci): paths-filter outputs not propagating to downstream jobs#72
abueide merged 2 commits intomainfrom
fix/paths-filter-outputs

Conversation

@abueide
Copy link
Copy Markdown
Contributor

@abueide abueide commented May 8, 2026

Summary

Fixes the paths-filter composite action not propagating dynamic outputs to the calling workflow, causing all conditional E2E jobs to skip even when matching files changed (e.g. PR #70).

Changes

  • Extract filter logic into paths-filter.sh script called directly by the workflow step
  • Workflow now uses run: bash .github/actions/paths-filter/paths-filter.sh instead of uses: ./.github/actions/paths-filter
  • Keep action.yml as documentation/interface for other repos

Why

GitHub composite actions only expose outputs explicitly declared in their outputs: section. Since filter keys are dynamic (defined by the caller), undeclared keys like android, ios, rn were silently empty. Calling the script directly writes to the caller's $GITHUB_OUTPUT natively.


🤖 Generated with Claude Code

Composite actions don't propagate dynamic step outputs to the caller —
only explicitly declared outputs are visible. Since filter keys are
dynamic, switch to calling paths-filter.sh directly in the workflow
step so outputs write to the caller's GITHUB_OUTPUT natively.
@abueide abueide added the bug Something isn't working label May 8, 2026
The script was rewriting the segkit flake ref to path:./segkit which
resolves relative to the plugin directory (plugins/android/). Since
segkit/ lives at repo root, the correct relative path is ../../segkit.
@abueide abueide merged commit e19d46c into main May 8, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant