CtrlK
BlogDocsLog inGet started
Tessl Logo

review-wall-of-apps-prs

Audit maintainer-side Wall of Apps pull requests in App-Store-Connect-CLI. Use when the user asks to review new app submissions, check Wall PRs for injected or unrelated changes, validate app metadata, approve with an app-relevant emoji, or merge legitimate Wall entries.

65

Quality

77%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

Low

Low-risk findings worth noting

Fix and improve this skill with Tessl

tessl review fix ./.agents/skills/review-wall-of-apps-prs/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

67%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A well-structured, lean review runbook with explicit validation gates and a sound multi-step workflow. Its main weakness is actionability: several validation steps tell Claude what to check but not the concrete commands, queries, or expected outputs to perform the check, leaving it between abstract direction and fully executable guidance.

Suggestions

Make validation steps executable: for 'verify the app name and destination URL against the public App Store/TestFlight', give the specific check (e.g. the exact bot command or API call to run) and what a passing result looks like.

Deduplicate the approval gates: the 'Approve and merge' and 'Automation contract' sections re-list the same checks (make check-wall-of-apps, GitHub checks, review threads) — state the gate once and have the automation section reference it to tighten conciseness.

Add an explicit verify→fix→retry loop for a failing 'make check-wall-of-apps' run (run, read failure, apply the smallest correction, re-run) to push workflow clarity from a clear sequence to a full feedback loop.

DimensionReasoningScore

Conciseness

The body is efficient and assumes Claude's competence — it does not explain what a PR, worktree, or squash merge is, and each line carries guidance. It earns a 4 rather than 5 because a few sentences restate the approval gates across the 'Approve and merge' and 'Automation contract' sections (e.g. re-listing 'make check-wall-of-apps', GitHub checks, and review threads), which could be tightened by referencing the shared gate once. Not a 3 since there is no real over-explanation of known concepts.

4 / 5

Actionability

Guidance is concrete in places — explicit commands like 'make check-wall-of-apps', 'worktree only when a fix is required', 'squash strategy' — but the core review work is described as imperatives rather than executable steps (e.g. 'Verify the app name and destination URL against the public App Store' gives no command, bot query, or expected output to check against). It sits between anchors: more concrete than the score-2 'high-level hints' but missing the copy-paste-ready specifics that define a 4.

3 / 5

Workflow Clarity

The process is clearly sequenced (Discover and classify → Validate the entry → Approve and merge) with explicit validation checkpoints ('Run make check-wall-of-apps on the exact PR head before approval', 'confirm... latest head contains only the legitimate Wall change', 'No actionable unresolved review thread remains') and a merge-one-at-a-time feedback loop. It is not a 5 because error-recovery detail for a failed check is light ('Verify bot findings... fix only proven omissions') and the checklist, while present, is not framed as a verify→fix→retry loop.

4 / 5

Progressive Disclosure

Content is well-organized into clearly headed sections (Discover and classify, Validate the entry, Approve and merge, Automation contract, Hand off) at appropriate length with no nested or buried references, and there are no bundle files to misplace. It does not reach 5 because nothing is split out to reference files — all guidance is inline, which is reasonable for this length but lacks the 'well-signaled one-level-deep references' the top anchor rewards; a 3 would require content that clearly belongs in a separate file to be inlined, which is not the case here.

4 / 5

Total

15

/

20

Passed

Description

87%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A strong, specific description that clearly answers both what the skill does and when to use it, with concrete maintainer-facing trigger phrases and a distinct, well-bounded niche. Minor synonym/extension coverage in trigger terms is the only thing keeping it from being perfect across every dimension.

DimensionReasoningScore

Specificity

The description names the domain and lists several concrete actions — 'Audit', 'check Wall PRs for injected or unrelated changes', 'validate app metadata', 'approve with an app-relevant emoji', 'merge legitimate Wall entries' — giving broad coverage with only minor gaps (e.g. escalation/reporting is implied rather than enumerated). Not a 5 because coverage is not fully comprehensive, and not a 3 because it goes well beyond 1-2 actions.

4 / 5

Completeness

It explicitly states both what the skill does ('Audit maintainer-side Wall of Apps pull requests... validate app metadata, approve... merge') and when to use it via a clear 'Use when the user asks to review new app submissions, check Wall PRs..., approve..., or merge...' clause with concrete triggers. This matches the score-5 anchor for answering both what and when with concrete trigger phrases; a 4 would require the 'when' to be less explicit, which it is not.

5 / 5

Trigger Term Quality

It surfaces natural user phrases like 'review new app submissions', 'check Wall PRs', 'approve', and 'merge', which a maintainer would actually say; it is just missing a few common synonyms or the literal file extension phrasing users might use. Not a 5 because synonym/extension coverage is incomplete, and not a 3 because the keyword set is solid rather than merely 'some relevant'.

4 / 5

Distinctiveness Conflict Risk

The niche is highly specific — maintainer-side review of Wall of Apps PRs in a named repo (App-Store-Connect-CLI) — with distinct triggers ('Wall PRs', 'Wall entries', 'app-relevant emoji') that make false-positive matching against unrelated skills unlikely. A 4 would imply minor overlap risk with closely related skills, but the named scope and terminology keep conflict risk minimal.

5 / 5

Total

18

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository
rorkai/App-Store-Connect-CLI
Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.