Skip to content

fix(android): Fix device filtering and update min device to API 24#22

Merged
abueide merged 1 commit intomainfrom
fix/device-filtering-logic
Apr 21, 2026
Merged

fix(android): Fix device filtering and update min device to API 24#22
abueide merged 1 commit intomainfrom
fix/device-filtering-logic

Conversation

@abueide
Copy link
Copy Markdown
Contributor

@abueide abueide commented Apr 21, 2026

Summary

Part of split from #17. Fixes device filtering to properly respect ANDROID_DEVICES env var.

Size: 406 lines (6 files)

Changes

  • Add filename metadata to device definitions for filtering
  • Filter devices by filename (min.json → min) in devices.sh and avd.sh
  • Add early failure detection when all devices are filtered out
  • Update minimum Android API from 21 to 24 (React Native requirement)
  • Update test validation for API 24

Impact

Before: ANDROID_DEVICES=max would fail to match devices
After: ANDROID_DEVICES=max correctly matches max.json device

Related PRs

Part of 6-PR split of #17:

All PRs are independent.

Fixes device filtering logic to properly respect ANDROID_DEVICES env var
and updates minimum Android API from 21 to 24.

Changes:
- Add filename metadata to device definitions for filtering
- Filter devices by filename in devices.sh and avd.sh
- Add early failure detection when all devices are filtered out
- Update min device from API 21 to API 24 (React Native minimum)
- Update test validation to check for pixel_api24

Before: ANDROID_DEVICES=max would fail to match devices
After: ANDROID_DEVICES=max correctly matches max.json device

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@abueide abueide force-pushed the fix/device-filtering-logic branch from f01087d to d633046 Compare April 21, 2026 18:11
@abueide abueide marked this pull request as ready for review April 21, 2026 18:16
@abueide abueide merged commit c9b80de into main Apr 21, 2026
10 of 14 checks passed
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.

1 participant