Skip to content

feat(android,ios): Add automatic doctor checks on shell init#24

Merged
abueide merged 1 commit intomainfrom
feat/android-doctor-checks
Apr 21, 2026
Merged

feat(android,ios): Add automatic doctor checks on shell init#24
abueide merged 1 commit intomainfrom
feat/android-doctor-checks

Conversation

@abueide
Copy link
Copy Markdown
Contributor

@abueide abueide commented Apr 21, 2026

Summary

Part of split from #17. Adds lightweight health checks that run automatically on shell init.

Size: 353 lines (8 files)

Changes

  • doctor.sh scripts for comprehensive diagnostics
  • init/doctor.sh for lightweight shell init checks
  • Drift detection for Android config mismatches
  • Silent output if healthy, verbose if issues found
  • Integrated into setup.sh for automatic checks

Impact

Healthy environment:

✅ [OK] Android setup complete
✓ Android

Issues detected:

⚠️  Android issues detected:
  - Config drift: env vars don't match android.lock
Fix: devbox run android:sync
Run 'devbox run doctor' for more details

Related PRs

Part of 6-PR split of #17:

All PRs are independent.

Adds lightweight health checks that run automatically on shell init for
both Android and iOS environments. Shows simple ✓ if everything is good,
or detailed warnings if issues are detected.

Changes:
- doctor.sh scripts for comprehensive diagnostics
- init/doctor.sh for lightweight shell init checks
- Drift detection for Android config mismatches
- Silent output if healthy, verbose if issues found
- Integrated into setup.sh for automatic checks

Example output (healthy):
  ✅ [OK] Android setup complete
  ✓ Android

Example output (with issues):
  ⚠️  Android issues detected:
    - Config drift: env vars don't match android.lock
  Fix: devbox run android:sync
  Run 'devbox run doctor' for more details

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@abueide abueide force-pushed the feat/android-doctor-checks branch from 588e343 to 20be464 Compare April 21, 2026 17:42
@abueide abueide marked this pull request as ready for review April 21, 2026 18:04
@abueide abueide merged commit e8689ee into main Apr 21, 2026
13 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