Declaration-order-dependent possibly-undefined false positive with nested ifs/matches, decorated any functions and NoReturn
オープン
まだ誰も着手していません。
bug
- 主要言語
- Python
- スター
- 20.6k
- フォーク
- 3.3k
- PR マージ指標
- PR 指標を取得中
説明
Bug Report
Some obtuse code results in a false positive for the possibly-undefined error code.
To Reproduce
Run mypy --enable-error-code possibly-undefined file.py on this file:
decorator = lambda f: f
def unreachable() -> None:
raise NotImplementedError()
def do_it(x: bool, y: None) -> None:
if x:
decorated()
var = 0
else:
match y:
case None:
var = 1
case _:
unreachable()
print(var)
@decorator
def decorated() -> None:
pass
This is as small as I've gotten the reproducer. All these parts seem to be significant:
- the outer
ifand an innermatch - the
decoratedcall being separate to thematch - the
matchbeing amatch(replacing thematchwith the equivalentif y is None: ... else: ...construct works fine) - the
matchhaving exhaustive arms with theunreachablecall being unreachable (as far as mypy can tell) (e.g. deleting thecase None:entirely works fine) - the
unreachablecall existing (inliningcase _: raise NotImplementedError()works fine) decoratorbeing a lambda (or an class with__call__method) (usingdef decorator(f): return fworks fine)decoratedbeing defined afterdo_itin the file (changing the order works fine) (NB. it's justdecorated,decoratorandunreachablecan be defined before or afterdo_itand still reproduce the behaviour)
Expected Behavior
This to pass type checking.
In particular, have same behaviour as any of the very similar ablations mentioned above, e.g. both of these work:
- changing
decorator = lambda f: ftodef decorator(f): return f - moving
decoratedto be beforedo_it
Actual Behavior
file.py:19: error: Name "var" may be undefined [possibly-undefined]
Found 1 error in 1 file (checked 1 source file)
Your Environment
- Mypy version used: 1.11.2
- Mypy command-line flags:
--enable-error-code possibly-undefined - Mypy configuration options from
mypy.ini(and other config files): None - Python version used: 3.10
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
最小の file.py 再現例から始め、報告された診断を確認するために mypy --enable-error-code possibly-undefined を実行します。相互作用を切り分けるため、デコレータの定義、宣言順序、match ブランチを含む、列挙されたアブレーションを比較します。再現例が誤検出なしで型チェックを通過すれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- devtools
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 42/100