-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[NoQA] Add agent-device glue-code skill for mobile testing #87662
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
rlinoz
merged 15 commits into
Expensify:main
from
callstack-internal:agent-device-local-skill
Apr 22, 2026
+26
−1
Merged
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
6f273b5
Install agent-device skill and add App-specific mobile testing wrapper
kacper-mikolajczak 4bfce77
Address PR review findings for agent-device skills
kacper-mikolajczak c0d5618
Trim wrapper skill to user-directed context layer
kacper-mikolajczak a63542b
Update build instructions to Rock workflow, HybridApp only
kacper-mikolajczak 7af0dc5
Drop dogfood skill, macos-desktop and remote-tenancy references
kacper-mikolajczak e756806
Sync agent-device skill with v0.12.x and improve app-testing startup …
kacper-mikolajczak 9b21ec1
Trim app-testing skill: defer device bootstrap to base agent-device
kacper-mikolajczak 18a1a81
Lean glue-code skill: drop inlined base skill, keep only pre-flight gate
kacper-mikolajczak 3537d95
Broaden agent-device skill scope and add project config
kacper-mikolajczak d4abc1b
Remove agent-device.json from PR
kacper-mikolajczak 4477236
Drop footnote and how-this-works section from agent-device SKILL.md
kacper-mikolajczak 1307ae5
Inject agent-device pre-flight context automatically
kacper-mikolajczak a9934fb
Surface agent-device skill in CLAUDE.md and README
kacper-mikolajczak 3fc5701
Allow agent-device skill path echo in permissions allowlist
kacper-mikolajczak 2939ad1
Merge remote-tracking branch 'upstream/main' into agent-device-local-…
kacper-mikolajczak File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| --- | ||
| name: agent-device | ||
| description: Drive iOS and Android devices for the Expensify App - testing, debugging, performance profiling, bug reproduction, and feature verification. Use when the developer needs to interact with the mobile app on a device. | ||
| allowed-tools: Bash(agent-device *) Bash(npm root *) | ||
| --- | ||
|
|
||
| # agent-device | ||
|
|
||
| ## Pre-flight | ||
|
|
||
| `agent-device` CLI version: !`agent-device --version 2>&1 || echo "NOT_INSTALLED"` | ||
|
|
||
| Canonical skill reference path (read these files directly for device automation guidance - bootstrap, exploration, verification, debugging): !`echo "$(npm root -g)/agent-device/skills/agent-device"` | ||
|
|
||
| > If the version line above shows `NOT_INSTALLED` or a command-not-found error, **STOP** and instruct the developer to install it: `npm install -g agent-device`. All device interaction depends on it. | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -48,6 +48,8 @@ For detailed setup instructions for each platform, see the following guides: | |
| * **📱 iOS Development**: [iOS Setup Instructions](contributingGuides/SETUP_IOS.md) | ||
| * **🤖 Android Development**: [Android Setup Instructions](contributingGuides/SETUP_ANDROID.md) | ||
|
|
||
| **Optional AI-assisted mobile testing:** If you use Claude Code, the [`/agent-device` skill](.claude/skills/agent-device/SKILL.md) drives iOS and Android simulators or devices for interactive testing, debugging, and performance profiling. Requires `npm install -g agent-device`. | ||
|
|
||
|
Comment on lines
+51
to
+52
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. NAB: We should start documenting our skills better -- outside scope here of course |
||
| ## General Troubleshooting | ||
| 1. If you are having issues with **_Getting Started_**, please reference [React Native's Documentation](https://reactnative.dev/docs/environment-setup) | ||
| 2. If you are running into CORS errors like (in the browser dev console) | ||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like to remove
testing, debugging, performance profilingfor now. I worry Melvin will start triggering this on workflows before we have good instructions for it to follow.Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Skill for now is only relevant for local development. Idea to broaden the use-cases, was to devs explore the agent-device more. We can remove it, but it should not be an issue until Melvin actually picks up on it, wdyt?