Storybook fix-linting-types-on-pr

Checks out a PR (including fork PRs), fixes all linting and TypeScript errors, then pushes the changes back. Use when asked to fix lint, types, or TS errors on a PR.

install
source · Clone the upstream repo
git clone https://github.com/storybookjs/storybook
Claude Code · Install into ~/.claude/skills/
T=$(mktemp -d) && git clone --depth=1 https://github.com/storybookjs/storybook "$T" && mkdir -p ~/.claude/skills && cp -r "$T/.agents/skills/fix-linting-types-on-pr" ~/.claude/skills/storybookjs-storybook-fix-linting-types-on-pr && rm -rf "$T"
manifest: .agents/skills/fix-linting-types-on-pr/SKILL.md
source content

Fix Linting and TypeScript Issues on a PR

Checks out a PR, auto-fixes linting and TypeScript issues, and pushes the fixes.

Step 1 — Get the PR number

If the user provided a PR number, use it directly. Otherwise ask for it.

Step 2 — Check out the PR

Use

gh pr checkout
so it works for both fork and non-fork PRs:

gh pr checkout <PR_NUMBER>

This automatically sets up the correct remote tracking and switches to the PR branch, even when the PR comes from a fork.

Step 3 — Install dependencies

yarn

Step 4 — Compile the repo

Compiling first ensures TS declarations referenced by the linter are present:

yarn nx run-many -t compile

Step 5 — Fix linting errors

yarn lint

Step 6 — Fix TypeScript errors

Run the TypeScript checker to surface remaining type errors:

yarn nx run-many -t check

Read the output carefully. Fix each type error manually by editing the relevant file(s). Common fixes:

  • Add or correct type annotations
  • Fix incorrect generics
  • Resolve
    any
    assignments that violate strict mode
  • Add missing imports or re-exports

Re-run

yarn nx run-many -t check
after each batch of edits to confirm errors are resolved.

Step 7 — Commit the fixes

Stage only the files you changed:

git add <files-you-modified>
git commit -m "Maintenance: Fix linting and TypeScript errors"

Do not use

git add -A
— avoid accidentally staging unrelated files.

Step 8 — Push the fixes

For fork PRs

gh pr checkout
sets up the correct upstream tracking. Push directly:

git push

Notes

  • Only fix errors that are clearly linting or TypeScript issues. Do not refactor logic.
  • If a TypeScript error requires a non-trivial code change, surface it to the user and ask before proceeding.
  • If
    gh pr checkout
    fails due to permissions on a fork, inform the user — they may need to grant write access to the fork or the maintainer must push directly.
  • After pushing, confirm with the user that the CI is green.