Revert "Ignore pyright reportMissingImport error"

[mulkieran]

Summary by CodeRabbit

• Chores
  • Removed workaround configuration, transitioning to default environment handling for type checking.

[coderabbitai]

Walkthrough

The executionEnvironments workaround block has been removed from the Pyright configuration in pyproject.toml. This eliminates a previous bug mitigation, allowing Pyright to rely on default environment handling instead.

Changes

Cohort / File(s)	Change Summary
Pyright configuration cleanup pyproject.toml	Removed executionEnvironments block from [tool.pyright] section that was previously used as a workaround for a Pyright bug

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• pyright: Turn off reportMissingImports only in tests directory #1215: Modifies the same Pyright configuration in pyproject.toml by adding an executionEnvironments block to scope reportMissingImports to tests, directly contrasting with this PR's removal of the workaround.

Poem

🐰 A buggy workaround fades away,
Pyright's defaults now have their say,
Config grows cleaner, lean and bright,
One less quirk to set things right! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The PR title "Revert 'Ignore pyright reportMissingImport error'" directly corresponds to the changeset, which removes the executionEnvironments block under [tool.pyright] that was a workaround for a Pyright bug. The title is clear and specific, indicating that the primary change is undoing a previous configuration that suppressed a specific Pyright error, which aligns with the actual modifications shown in the summary. The title is concise, readable, and contains no vague or misleading language.

✨ Finishing touches

🧪 Generate unit tests (beta)

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

---

📜 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 d7b8869 and 74c5f4a.

📒 Files selected for processing (1)

• pyproject.toml (0 hunks)

💤 Files with no reviewable 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)

---

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.}

[mulkieran]

pyright checks are passing now, no need to wait for the rest.