Skip to content

ref(lint): Enable no-unnecessary-type-assertion - #2125

Draft
JoshuaKGoldberg wants to merge 1 commit into
mainfrom
josh/lint-no-unnecessary-type-assertion
Draft

JoshuaKGoldberg wants to merge 1 commit into
mainfrom
josh/lint-no-unnecessary-type-assertion

Conversation

@JoshuaKGoldberg

Copy link
Copy Markdown
Member

Turns on typescript/no-unnecessary-type-assertion and removes the casts it flags, almost all through oxlint's autofix. Logger and toolkit-core now enable noUncheckedIndexedAccess, matching how the CLI already typechecks their source, so the lint and the typecheck agree on which casts are needed.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JoshuaKGoldberg

Copy link
Copy Markdown
Member Author

@cursor review

@vercel

vercel Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
sentry-local Ready Ready Preview Nov 9, 2026 10:00pm UTC

Request Review

@cursor cursor 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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 50981e3. Configure here.

This branch was successfully deployed

2 active deployments
Preview — 50981e3d Deployed Nov 9, 2026 by vercel[bot]
Actions — 50981e3d Deployed Nov 9, 2026 by JoshuaKGoldberg via eval #1756
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant