Skip to content

feat(skills): ship an agent skill and usage rules in the package - #153

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-11893-ship-agent-skills-and-usage-guidance-inside-the-react-ui
Oct 31, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-11893-ship-agent-skills-and-usage-guidance-inside-the-react-ui

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Closes CL-11893.

What changed

  • skills/react-ui/SKILL.md (agentskills.io frontmatter): what the library is, plus a task → reference map.
  • skills/react-ui/AGENTS.md: rules for agents using the library: reuse first, props as contract, tokens, text props for wording, accessibility, one action per control, imports via exports.
  • skills/react-ui/references/*.md: generated by scripts/agent-references.mjs from dist/*.d.ts during build/prepack and gitignored, so they can't drift from the shipped code.
    • Each covers components with props, a Types section, and hooks and helpers.
    • Generation fails on a public module no group covers, a group with no component, or a file over 16 KB.
  • files includes skills; README has a section for coding agents; the root AGENTS.md layout lists the skill.
  • typescript is pinned to 5.9.3, because the references depend on its output.

Verified

  • lint, format:check, typecheck, test, build, dep-guard, contrast-test pass; bun install --frozen-lockfile is clean.
  • After deleting dist/ and the references, npm pack --dry-run regenerates all 11 reference files into the tarball.
  • Hand-written guidance is about 6 KB.
  • Independent review; all findings fixed.

skills/react-ui holds a task-to-reference skill, consumer rules for
agents, and component references generated from the built declarations
at build and pack time. Generation fails on public components no group
covers. typescript is pinned exactly because the references depend on
its output.
@linear-code

linear-code Bot commented Oct 31, 2026

Copy link
Copy Markdown

CL-11893

@corbits-triage

corbits-triage Bot commented Oct 31, 2026 •

Copy link
Copy Markdown
Contributor

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Notes:

  • added code matches a code pattern: scripts/agent-references.mjs:373 Invented fallback `?? []`; handle the missing value instead.
  • changes sensitive paths: locks: bun.lock
  • Risk: medium (locks paths, no tests)

Triage will approve.

Changed since the last verdict (Fail at 331a75d): 1 new commit.

@corbits-triage corbits-triage Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Triage requests changes:

  • CI green: required checks are failing

@corbits-triage
corbits-triage Bot dismissed their stale review October 31, 2026 20:41

Nothing triage asked to change still fails, so this request no longer stands.

@TheGreatAxios
TheGreatAxios merged commit f5e3513 into main Oct 31, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant