typing-exclusion-worker
Python typing exclusion worker: remove assigned mypy exclusion modules in small scoped batches, fix typing issues, run validation, and produce a structured completion summary. Use when running parallel typing-debt workers or when asked to remove modules from pyproject mypy exclusion overrides.
Install
npx skills add https://github.com/getsentry/skills --skill typing-exclusion-workerSKILL.md
Typing Exclusion Worker
Purpose
Execute one assigned typing batch safely and predictably:
- remove only assigned modules from mypy exclusions,
- fix surfaced typing issues in scope,
- run required checks,
- return a consistent summary for the manager/orchestrator.
Inputs Required
Before starting, confirm these inputs exist in the task prompt:
- worktree/branch name,
- exact module list to remove from exclusion,
- ownership/domain boundary,
- expected validation commands (if customized).
If any are missing, ask for them before editing.
Scope Rules (Hard Constraints)
- Only remove assigned module entries from the mypy exclusion list in
pyproject.toml. - Keep code changes in assigned scope unless a direct dependency is required to pass typing/tests.
- Do not expand to cross-team modules unless explicitly approved by the manager.
- Avoid blanket
# type: ignore; if unavoidable, use narrowignore[code]with a short reason.
Execution Workflow
Apply exclusion change
- Remove assigned modules from the exclusion override in
pyproject.toml.
- Remove assigned modules from the exclusion override in
Run mypy on assigned scope
- Prefer targeted paths first for fast feedback.
- Fix errors using explicit typing patterns (
isinstancenarrowing, accurate return types, typed class attrs, relation-safe model access).
Run tests for touched area
- Execute targeted pytest for modified modules/tests.
- Fix regressions before continuing.
Run pre-commit on changed files
- Run
pre-commit run --files <changed files>. - If hooks auto-fix files, rerun until clean.
- Run
Final verification
- Re-run targeted mypy and tests after final edits.
- Ensure no unrelated files were changed.
Python Typing Best Practices
- Prefer precise types over
Any. - Use type narrowing on unions before attribute access.
- Keep method overrides signature-compatible with base classes.
- Annotate class attributes in tests/helpers when inference is weak.
- Use relation objects (
obj.related) when stubs do not expose raw*_idattributes.
Required Output Template
Return this exact structure at the end of each batch:
## Batch Summary
- Branch/worktree: `<name>`
- Ownership/domain: `<team-or-domain>`
### Modules Removed From Exclusion
- `<module.path.one>`
- `<module.path.two>`
### Files Changed
- `<path>`
- `<path>`
### Key Typing Fixes
- `<short rationale + fix>`
- `<short rationale + fix>`
### Validation
- `mypy`: `<pass/fail + scope>`
- `pre-commit --files`: `<pass/fail>`
- `pytest`: `<pass/fail + scope>`
### Notes
- Remaining blockers: `<none or details>`
- Any new ignore entries: `<none or file + ignore code + reason>`
Stop Conditions (Escalate to Manager)
Stop and report instead of widening scope when:
- fixes require touching another team/domain,
- exclusion conflicts in
pyproject.tomlcannot be resolved safely, - error volume indicates batch is too large and should be split.
Related skills
improve-codebase-architecturemattpocock1MScan a codebase for deepening opportunities, present them as a visual HTML report, then grill through whichever one you pick.codebase-designmattpocock691KShared vocabulary for designing deep modules. Use when the user wants to design or improve a module's interface, find deepening opportunities, decide where a seam goes, make code more testable or AI-navigable, or when another skill needs the deep-module vocabulary.web-design-guidelinesvercel-labs676KReview UI code for Web Interface Guidelines compliance. Use when asked to "review my UI", "check accessibility", "audit design", "review UX", or "check my site against best practices".code-reviewmattpocock631KReview the changes since a fixed point (commit, branch, tag, or merge-base) along two axes: Standards (does the code follow this repo's documented coding standards?) and Spec (does the code match what the originating issue/spec asked for?). Runs both reviews in parallel sub-agents and reports them side by side. Use when the user wants to review a branch, a PR, work-in-progress changes, or asks to \"review since X\".