Skip to content

Port upstream fixes to the hr-better-* skills and hr-explain-interface - #636

Merged
bompus merged 2 commits into
mainfrom
port-better-skills-2026-11-13
Nov 13, 2026
Merged

bompus merged 2 commits into
mainfrom
port-better-skills-2026-11-13

Conversation

@bompus

@bompus bompus commented Nov 13, 2026 •

Copy link
Copy Markdown
Owner

Moves the five hr-better-* skills and hr-explain-interface to jakubkrehel/skills a72290d. Each upstream hunk was judged against this fork, not copied.

Ported: accessibility's 2.2.2 timer wording (auto-updating content needs a control however long it runs, limited to content shown alongside other content, with WCAG's exemption for essential updates) in SKILL.md and motion-and-zoom.md, and a note that a custom dialog renders outside the inert container; colors' @supports example uses the same oklch value as the first; typography's text-box examples name both edges (trim-start cap text, trim-end text alphabetic) and say why.

Left out: upstream's layout 100dvh keyboard note (its "stable chrome" advice conflicts with the sticky-footer recipe here), its hit-areas.md label rule, which this fork already states, and its explain-interface token recipe, which this fork already covers by reading themed and computed values.

upstream_ref, each NOTICE.md and THIRD_PARTY_NOTICES.md name a72290d; hr-better-writing has no upstream change and moves with the group. bun run check passes; no skill description changed.

@bompus bompus added author:anthropic Pull request written by a anthropic model author-model:sonnet-5-5 A model that wrote part of this pull request labels Nov 13, 2026

@bompus-ai-review bompus-ai-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bompus AI Review (GPT-6.1 Terra, max reasoning; SWE-2 High, high reasoning): 2 finding(s).

Comment thread skills/hr-better-layout/SKILL.md Resolved
Comment thread skills/hr-better-accessibility/SKILL.md Resolved
…ide-other-content limit and essential exemption

@bompus-ai-review bompus-ai-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bompus AI Review (GPT-6.1 Terra, max reasoning): 0 finding(s).

Review 2: only high findings and above are posted from here on.

@bompus
bompus merged commit 83028cf into main Nov 13, 2026
2 checks passed
@bompus
bompus deleted the port-better-skills-2026-11-13 branch November 13, 2026 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author:anthropic Pull request written by a anthropic model author-model:sonnet-5-5 A model that wrote part of this pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant