Skip to content

ref(tsconfig): Apply the same strictness everywhere except mcp-core - #2239

Open
JoshuaKGoldberg wants to merge 2 commits into
josh/tsconfig-shared-basefrom
josh/tsconfig-shared-strictness
Open

JoshuaKGoldberg wants to merge 2 commits into
josh/tsconfig-shared-basefrom
josh/tsconfig-shared-strictness

Conversation

@JoshuaKGoldberg

@JoshuaKGoldberg JoshuaKGoldberg commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

Moves the strictness flags into the shared base so every package except mcp-core checks with them, and folds apps/local's oxlint config into the root config. And of course, all introduced type checking reports are fixed.

Note: apps/local's source folder gets a lint-only tsconfig so type-aware lint checks its colocated tests with the app's options instead of TypeScript's defaults. mcp-core keeps noUncheckedIndexedAccess off until the next PR in this stack.

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

vercel Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
sentry-local Skipped Skipped Nov 13, 2026 4:10pm UTC

Request Review

@JoshuaKGoldberg

Copy link
Copy Markdown
Member Author

@cursor 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 296120a. Configure here.

…rictness

# Conflicts:
#	packages/codemod-cli3to4/tsconfig.json

This branch was successfully deployed

1 active and 1 inactive deployments
Actions — b2f22493 Deployed Nov 13, 2026 by JoshuaKGoldberg via eval #1864
Preview — b2f22493 Deployed Nov 13, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant