Add Claude skill: android-sdk-event-inspection#3220
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f6f04529b4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
How is this PR related to mobile-mcp? |
This comment has been minimized.
This comment has been minimized.
@aleksandr-gringauz we need the mobile-mcp to run the skill. |
@mormubis Can you please be more specific? I don't see anything in the skill that requires the MCP. It uses |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #3220 +/- ##
===========================================
+ Coverage 71.46% 71.57% +0.11%
===========================================
Files 934 934
Lines 34628 34641 +13
Branches 5862 5863 +1
===========================================
+ Hits 24745 24791 +46
+ Misses 8238 8229 -9
+ Partials 1645 1621 -24 🚀 New features to boost your workflow:
|
|
I would say the goal of the skill is to prepare the sample app so that we can extract event information from logcat. The mobile-mcp will be used to interact with the app and generate the events that need to be tested. That’s why MCP is not mentioned directly in the skill, because the skill doesn’t strictly require it. @mormubis, feel free to correct me, but if I’m not mistaken, the idea is for an agent to use the skill to prepare the sample app, then perform some tests by interacting with the device via MCP and extract the necessary information from logcat to verify the results. |
hamorillo
left a comment
hamorillo
left a comment
There was a problem hiding this comment.
🚀 Tested and works well! I’ve left a few minor comments.
There was a problem hiding this comment.
@mormubis thanks for the contribution. I left some comments.
In short - I think most of the skill's content shouldn't be there because it is some knowledge that can either be understood by the agent by itself or should be put in some place else (AGENTS.md).
We don't have AGENTS.md and other tools set up yet in this repo.
I suggest we do this preliminary setup first (I created a ticket RUM-14903, I'll take care of it). And then come back to this skill and see if we can simplify it.
The skill itself looks useful to me.
|
|
||
| This change is gated by `BuildConfig.DEBUG` — it only runs in debug builds. | ||
|
|
||
| ### 2. Configure Build-Time Credentials |
There was a problem hiding this comment.
I think this section isn't related to the skill. If we really need to instruct the agent about it, it should be put into AGENTS.md.
Otherwise we would need to duplicate it in every skill.
|
|
||
| ## Related Tools | ||
|
|
||
| [mobile-mcp](https://github.com/mobile-next/mobile-mcp) — use alongside this skill to interact with the device and generate events (taps, navigation, actions) that you then observe via logcat. |
There was a problem hiding this comment.
Ideally we need to add the mcp to .mcp.json in this repo (it doesn't exist yet, yes).
There was a problem hiding this comment.
I don't want to force people onto mcps (just yet), but if you try to execute the skill, it must tell you what you need to do to install it.
Co-authored-by: Yi LU <[email protected]>
Co-authored-by: Yi LU <[email protected]>
5 participants
Summary
Adds a Claude skill documenting the workflow for inspecting Datadog Android SDK event payloads locally.
The skill covers:
CurlInterceptor(printBody = true)in debug builds to expose full JSON payloads in logcatconfig/{flavor}.json) and credential requirementssample/kotlinappDatadog,DD_LOG,Curl)rumApplicationId, wrong activity name, no body output)File
.claude/skills/android-sdk-event-inspection/SKILL.mdRelated