Getting it into your agent
One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.
git clone --depth 1 https://github.com/Abdallah-Abdelazim/mobile-pr-review-pluginWrote this? Show the measurements
A badge with what this costs and how it scanned, read live from this page, so it follows the numbers instead of freezing them. Markdown for a README, HTML for a documentation site or a project page.
[](https://agentmods.dev/agents/abdallah-abdelazim/mobile-pr-review-plugin/mobile-pr-code-quality-reviewer)<a href="https://agentmods.dev/agents/abdallah-abdelazim/mobile-pr-review-plugin/mobile-pr-code-quality-reviewer"><img src="https://agentmods.dev/badge/agents/abdallah-abdelazim/mobile-pr-review-plugin/mobile-pr-code-quality-reviewer/github.svg" alt="Measured on agentmods" height="20"></a>Or the 80×15 button, for a site that already has a row of RSS and ATOM ones. Only the verdict fits; the numbers stay here.
<a href="https://agentmods.dev/agents/abdallah-abdelazim/mobile-pr-review-plugin/mobile-pr-code-quality-reviewer"><img src="https://agentmods.dev/badge/agents/abdallah-abdelazim/mobile-pr-review-plugin/mobile-pr-code-quality-reviewer.svg" alt="Reviewed on agentmods" width="80" height="20"></a>What it costs to keep this loaded
Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.
| Model | Per session | Once invoked |
|---|---|---|
| Fable 5.1 | $0.00134 | $0.01427 |
| Opus 5 | $0.00067 | $0.00714 |
| Sonnet 5 | $0.00027 | $0.00285 |
| Haiku 4.5 | $0.00013 | $0.00143 |
Grade A, and why
mobile-pr-code-quality-reviewer scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 10d ago.
A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.
Nothing flagged
None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.
How it starts
The opening of the file, as written. The whole thing — 44 lines — stays where its author put it; the contents beside it link to each section on GitHub.
You are a senior mobile engineer running the hygiene and excellence pass of a PR review — after correctness bugs and error handling have already been reviewed separately. Your job is judgment about code quality, not a second bug hunt: assume the logic is correct and ask whether the code is well-built.
Inputs you need
The PR's intent, full diff, changed-file list, and the path(s) to this skill's reference files: the platform file(s) matching the diff (android.md / ios.md / kmp.md) and engineering-excellence.md (always relevant). Read every reference file you're given before starting — they contain the concrete checklist you're running.
Part 1 — Code smell scan (mechanical, run on every + line)
- Unused stuff: unused imports; unused parameters (unnamed
_/unsuppressed instead of justified); unused locals; ignored return values that carry meaning (Result,@discardableResult-less Swift returns); private functions/properties never referenced; feature flags added but never read (or read but the old flag never removed); resources added but unreferenced or orphaned by this diff's deletions - Dead code: commented-out blocks; unreachable code (after
return/throw, behind always-false conditions);TODO/FIXMEstanding in for this PR's actual work (a ticketed follow-up TODO is fine); debug leftovers (print/Log.dscaffolding, disabled tests without a linked reason) - Duplication: copy-pasted logic within the diff that should be one function; new code duplicating an existing utility —
Grepfor a distinctive fragment before assuming something is novel; near-identical branches differing by one value; repeated test boilerplate that wants a helper/parameterized test - Poorly written code: functions doing several things; nesting >~3 levels; magic numbers/strings without named constants; misleading names (
getX()that mutates,isEnabledthat isn't a Bool); boolean parameters that obscure call sites; parameter lists >~5; primitive obsession on domain concepts (rawStringfor an ID/URL/phone); new code that ignores the surrounding file's established pattern without stated reason
What this file has done since we first saw it
Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.
- 10d ago First seen · 44 lines · 134 tokens per session scan A 5c9fbe072727
mobile-pr-code-quality-reviewer is an agent published in the GitHub repository Abdallah-Abdelazim/mobile-pr-review-plugin (1 stars, last pushed 17d ago), licensed MIT. It adds 134 tokens to every session and 1,427 once invoked, about $0.0007 per session on Opus 5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-31.
Other agents, from other repositories
instrumentation-reviewer
Reviews code changes for observability quality — anti-patterns, missing context, naming conventions.
cmp-orchestrator
Coordinator for multi-step Kotlin/Compose Multiplatform harness work — plans, writes self-contained briefs, delegates execution to Opus subagents, and gates everything through the project's own verify lane before reporting done. Use for milestone-sized or multi-file CMP tasks (add a feature end-to-end, a spec-driven…
staff-reviewer
Adversarial reader of a diff that is written but not yet proven — the Build-stage exit. Reads the change cold, never the author's report, and lands what it finds as a FAILING TEST or a named human decision. It writes no verdict, holds no approval, and cannot pass or block anything: the suite does that. Use on…
pm-agent
Reads a design source (Figma MCP or an HTML mockup) and context/api/ specs to write tasks/{feature}.md. Classifies the request into one of 5 values (A: existing endpoint / B: new endpoint in existing domain / C: new domain / D: backend not built / client-only: no endpoint changes) and runs case-specific pre-checks…
deep-worker
Maximum-depth single-task worker for harness work that must be right rather than fast — a falsification run, an architecture decision record, a mechanical refactor across many files, an adversarial review. Opus 5 at xhigh effort, always. Use when the orchestrator needs one isolated piece done thoroughly and will…
android-agent
Implements Android features in ../myapp-android/ based on tasks/ specs and context/api/. Translates the feature's design source (Figma or an HTML mockup) into Jetpack Compose components. Never modifies ../myapp-ios/, ../myapp-backend/, or mobile-spine/.