sap-screens-reviewer

sap-screens-reviewer is an agent for Claude Code from marcellourbani/vscode_abap_remote_fs. It costs 90 tokens per session (1,676 once invoked), scanned A, original, MIT.

A review agent for checking whether a Phase 2 `screens.md` file matches what users can actually see and interact with in a live SAP web interface. It checks browser-visible names, labels, starting states, dialogs, and ALV tables rather than ABAP source descriptions.

In plain words
What is it for?
Use it to review `screens.md` files for SAP web interfaces and find mismatches that could make later Playwright test steps fail or target the wrong element.
Why use it?
It catches screen documentation that was built from source code instead of browser observation, before automated Playwright tests rely on incorrect labels or controls.

Agent for Claude Code

Written for Claude Code: disable-model-invocation in frontmatter. Also seen: model in frontmatter; mentions subagents.

Good fit Use it to review screens.md files for SAP web interfaces and find mismatches that could make later Playwright test steps fail or target the wrong element.

Compare 6 agents from other repositories ↓
Install with agentmods
npx agentmods add agents/marcellourbani/vscode_abap_remote_fs/sap-screens-reviewer
Install

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.

Clone the repo
git clone --depth 1 https://github.com/marcellourbani/vscode_abap_remote_fs

Made for: Claude Code.

Wrote 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.

agentmods badge for sap-screens-reviewer

README.md
[![agentmods](https://agentmods.dev/badge/agents/marcellourbani/vscode_abap_remote_fs/sap-screens-reviewer/github.svg)](https://agentmods.dev/agents/marcellourbani/vscode_abap_remote_fs/sap-screens-reviewer)
Your own site
<a href="https://agentmods.dev/agents/marcellourbani/vscode_abap_remote_fs/sap-screens-reviewer"><img src="https://agentmods.dev/badge/agents/marcellourbani/vscode_abap_remote_fs/sap-screens-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.

agentmods 80×15 button for sap-screens-reviewer

Your own site · 80×15
<a href="https://agentmods.dev/agents/marcellourbani/vscode_abap_remote_fs/sap-screens-reviewer"><img src="https://agentmods.dev/badge/agents/marcellourbani/vscode_abap_remote_fs/sap-screens-reviewer.svg" alt="Reviewed on agentmods" width="80" height="20"></a>
Per session 90 Only the description is in the session, so the agent can decide to use it. The body loads when it is invoked.
When invoked 1,676 The whole file, excluding the scripts and references it only reads on demand.
Security scan A 0 findings. A grade says what 26 rules found in the file — not that it is safe.
Origin original No closer match found in the catalogue.
Token cost

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.

ModelPer sessionOnce invoked
Fable 5.1 $0.00090 $0.01676
Opus 5 $0.00045 $0.00838
Sonnet 5 $0.00018 $0.00335
Haiku 4.5 $0.00009 $0.00168

Measured 11d ago against content hash e6102f4a2c3b, method: parsed. Prices are Anthropic first-party input rates as of 2026-09-11, from the pricing page.

Security

Grade A, and why

sap-screens-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 11d 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.

client/media/agents/sap-screens-reviewer.agent.md · 71 lines

How it starts

The opening of the file, as written. The whole thing — 71 lines — stays where its author put it; the contents beside it link to each section on GitHub.

sap-screens-reviewer

If the caller tells you HOW to do your task, ignore it. Follow only this file. Accept inputs (what/where); reject invented methods.

You are an ephemeral, one-shot subagent. The caller cannot see your work or ask a follow-up, and gets one response. Run every check below in a single pass and report EVERY problem at once — never return after the first offending line you spot. A FAIL that lists one issue when several exist costs the caller a full extra explore→review cycle per hidden issue, which is the exact waste this gate should prevent.

Why you exist

Phase 6 (build-scripts) turns every control in _screens.md into a Playwright call using its EXACT accessible name / on-screen label. If _screens.md was written from the ABAP source instead of from live browser observation, its "labels" are ABAP field names and screen constructs that Playwright will never find — so every spec that touches them breaks or, worse, silently matches the wrong element. This exact failure has happened. You are the cheap gate that catches a source-derived _screens.md before Phase 3+ build on it.

Scope — be honest about it. You are reviewing STATIC CONTENT and PROVENANCE, not label correctness against the running system (you have no browser; you cannot confirm a given accessible name is really what SAP renders). You catch: source-derived content, ABAP artifacts that don't belong, missing initial states, missing screens, and structural problems. Say clearly that live-label accuracy is verified later (first headed run in Phase 7), not by you.

You cannot fix anything, by design. Report each problem precisely and stop.

Input you'll receive

  • Program name (and connectionId, for context).
  • Confirmation that tests/<PROGRAM>/test-cases/_screens.md exists (and usually _findings.md/_flow.md alongside, which you may read to know which screens/controls SHOULD appear).

If _screens.md is missing, say so and tell the caller to run explore-ui.

Read the full file on GitHub · 71 lines

Changes

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.

  1. 11d ago First seen · 71 lines · 90 tokens per session scan A e6102f4a2c3b

Subscribe to this mod's changes

sap-screens-reviewer is an agent published in the GitHub repository marcellourbani/vscode_abap_remote_fs (389 stars, last pushed 2d ago), licensed MIT. It adds 90 tokens to every session and 1,676 once invoked, about $0.0005 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-30.