pyright: Turn off reportMissingImports only in tests directory

[mulkieran]

Summary by CodeRabbit

• Chores
  • Adjusted static analysis configuration so missing-import suppression applies only within the test environment, improving type-check accuracy in other areas.
• Tests
  • Tightened type-checking behavior for tests to surface import issues during development.

No user-facing changes.

[coderabbitai]

Walkthrough

Replaces the global Pyright reportMissingImports setting with an executionEnvironments list in pyproject.toml, adding a single environment rooted at tests that sets reportMissingImports = "none", and moves the related bug comment into that environment block.

Changes

Cohort / File(s)	Summary of Changes
Pyright config scoping pyproject.toml	Removed global reportMissingImports setting and added an executionEnvironments entry rooted at tests that sets reportMissingImports = "none"; relocated the existing bug-reference comment into that environment block.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• Ignore pyright reportMissingImport error #1210 — Also modifies pyproject.toml to control Pyright reportMissingImports, but applies it globally rather than scoping it to an execution environment.

Suggested labels

trivial

"I hopped through configs with tidy feet,
I fenced missing imports to the tests' neat suite.
No global weeds, just a planted row,
Carrots of clarity in tidy tow. 🥕🐇"

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e5b7bbc and 2be9158.

📒 Files selected for processing (1)

• pyproject.toml (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• pyproject.toml

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: test-runs (3.13)
• GitHub Check: test-runs (3.12)

✨ Finishing Touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

pyproject.toml (1)

10-10: Prefer severity string over boolean for future-proofing.

Use "none" instead of false to align with documented rule severities and avoid potential schema drift.

-  { root  = "tests", reportMissingImports = false }
+  { root  = "tests", reportMissingImports = "none" }

Docs/examples show per-environment rule severities as strings. (docs.basedpyright.com)

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c68dcda and e5b7bbc.

📒 Files selected for processing (1)

• pyproject.toml (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: test-runs (3.13)
• GitHub Check: test-runs (3.12)

🔇 Additional comments (1)

pyproject.toml (1)

8-11: Scoped suppression looks good.

Limiting reportMissingImports to tests via executionEnvironments matches the PR goal and the TOML inline-table syntax here is valid for Pyright configs. (fpfortkamp.com)