All skills
jakubkrehel avatar

/interface-review

@267330e
by Jakub Kreheljakubkrehel/skills7.4k stars
275

Reviews your work across multiple categories like UI, typography, layout, color, writing and accessibility and gives you a detailed analysis of the findings.

Use this Skill: https://skilld.dev/gh/jakubkrehel/skills/interface-review

This session only. Nothing lands on disk.

removed-signals.md

≈727 tokens on demand. Your agent reads this file only when SKILL.md points to it.

Removed signals

What to look for on the - side of a hunk and which skill owns the judgement. A row here is a lead, never a finding. Route the removal to its owner and report it only once that skill confirms the interface got worse.

Removed from the - side Owner What to check
aria-label, aria-labelledby, aria-describedby, aria-live, role= better-accessibility The control or region lost its accessible name, description, or announcement
alt=, <label, for=, scope= better-accessibility Image, field, or table cell lost its programmatic association
<button>, <a>, <nav>, <main>, <ul> replaced by div or span better-accessibility Keyboard and assistive-technology behavior was traded for styling
:focus-visible, :focus, outline, tabindex better-accessibility Keyboard users lost the focus indicator or the element left the tab order
prefers-reduced-motion, prefers-contrast better-accessibility Motion or contrast now ignores the user's system preference
Logical properties swapped for left / right better-layout Direction-aware layout was dropped
lang=, dir= better-typography Language metadata or text direction was dropped
text-wrap, line-clamp, overflow-wrap, tabular-nums, font-feature-settings better-typography Text rendering, wrapping, or numeral alignment silently changed
A color token swapped for a literal, or a token swapped for a lighter one better-colors The rendered contrast pair may now fail; measure it
A user-facing string deleted or shortened better-writing A label, error, or empty state lost the information it carried

Equivalent replacements

These clear the signal. Check for them before routing anything, or the report fills with refactors reported as regressions:

  • aria-label giving way to aria-labelledby pointing at visible text.
  • An explicit role dropped because the element became the native equivalent, role="button" going as a div becomes a <button>.
  • outline replaced by a box-shadow focus ring that still meets the focus-indicator rule.
  • tabindex="0" dropped from an element that is now natively focusable.
  • A color literal replaced by a token that measures the same rendered pair.
  • A physical property replaced by its logical counterpart, which is the fix rather than the regression.
  • A string moved into the translation catalogue rather than deleted.

Searching the removed side

Restrict the search to deleted lines so additions do not mask a removal:

git diff -U0 "$BASE"...HEAD -- '*.tsx' '*.css' | grep -E '^-[^-]' | grep -E 'aria-|role=|alt=|focus|tabindex|prefers-'

Read the surrounding hunk before deciding. A removed attribute is meaningless without the element it came from, and -U0 deliberately hides that context.

Source: SKILL.md on GitHub

1 warning1mo3 checks · Risk SAFE
  • Gen Agent Trust Hub1mo

    The skill facilitates interface reviews by executing git and GitHub CLI commands to analyze code changes and pull request metadata. It is generally safe but carries a minor risk of indirect prompt injection due to its handling of untrusted text from pull request descriptions and commit messages.

  • Socket1mo

    No alerts

  • Snyk1mo

    Risk: MEDIUM · 1 issue

Signed by skilld at 267330e. This ties the file your Agent reads to that commit on GitHub. It does not review the instructions.

Last checked against GitHub last month.

Steadyupdated last month
disable-model-invocation
true

README badge

README badge for jakubkrehel/skills/interface-review