Commit Graph

2 Commits

Author SHA1 Message Date
Ignacy Łątka f2b0c0be1a feat(android-binary): probe OS-default SDK install locations when env vars are absent (#293)
## Summary

- Extends `androidRoots()` to fall back to OS-default Android SDK
install paths after `ANDROID_HOME` / `ANDROID_SDK_ROOT` come up empty
- Covers Android Studio defaults (`~/Library/Android/sdk` on macOS,
`~/Android/Sdk` on Linux) and common system-wide paths
(`/opt/android-sdk`, `/usr/lib/android-sdk`, etc.)
- Fixes the case where an MCP server spawned by a GUI process (Claude
Code desktop, VS Code) doesn't inherit the shell-exported
`ANDROID_HOME`, causing argent to silently fail to find `adb`/`emulator`
even on a correctly configured machine

## Stacking

Stacked on `feat/linux-host-support`. Base will be retargeted to `main`
after that PR merges.

## Test plan

- [ ] New tests in `android-binary.test.ts` cover the Studio-default
paths and the `ANDROID_HOME` priority invariant
- [ ] `npm run test -w packages/tool-server` passes
2026-06-03 17:14:35 +02:00
Ignacy Łątka 837d088744 test: cover #148 (android) (#186)
This PR adds tests for #148.

## Summary

Moves the Android emulator branch's test coverage out of #148 so that PR
can be reviewed without the test churn. This branch adds the test files
back on top of the (now test-stripped) base branch.

Tests inseparable from source changes (`describe-tool`,
`workspace-reader`, `workspace-reader-integration`) and the
`boot-simulator.test.ts` deletion stayed on #148 - base PR's CI passes
on its own.

## Test plan

- [x] `npx tsc --build` — clean
- [x] `npx prettier --check .` — clean
- [x] `npm test --workspaces --if-present` — 585/585 tool-server tests
pass; all other workspaces green

---------

Co-authored-by: Paweł Fornagiel <pawel.fornagiel2@gmail.com>
2026-05-12 14:21:09 +02:00