Black fails to format an un-formatted file with walrus operator

Describe the bug

If a file is not black-formatted and it contains a walrus operator, Black fails. If a file is already black-formatted and it contains a walrus operator, Black succeeds.

It happens on my machine on version 22.6.0. It doesn’t happen in playground.

To Reproduce

With this code, which is already black-formatted, all is good:

if 3 > (a := 3):
    pass

x = [1, 2]

$ black black_walrus.py 
All done! ✨ 🍰 ✨
1 file left unchanged.

But with this code, which is not black-formatted (notice the magic comma), Black fails:

if 3 > (a := 3):
    pass

x = [1, 2,]

$ black white_walrus.py 
error: cannot format white_walrus.py: cannot use --safe with this file; failed to parse source file AST: invalid syntax (<unknown>, line 1)
This could be caused by running Black with an older Python version that does not support new syntax used in your source file.

Oh no! 💥 💔 💥
1 file failed to reformat.

Expected behavior

In the second example, Black should re-format the file with no errors.

Environment

$ black --version
black, 22.6.0 (compiled: yes)
Python (CPython) 3.6.10

$ python --version
Python 3.6.10 :: Anaconda, Inc.

OS: Linux Mint 20.3, kernel 5.4.0-120-generic

Additional context

Since it works correctly in the playground, I suppose something is wrong with my local installation, but I have no idea what it can be.

[ichard26]

The reason is because our main parser (blib2to3) is extremely lax (mostly to support Python 3.10 syntax without rewriting the whole parser and therefore codebase) so it doesn’t care if the Python version is too low, it’ll happily parse newer syntax. For the AST equivalence safety check though, it uses the built-in ast module which obviously does not support syntax beyond the running Python. We only run the safety check if changes were made so that’s why if no changes are made, Black “works.”

I think there’s something to this. For example, I have 3.9 but I’m able to run Black on match statements with --target-version py310. However, it only works when there’s nothing to change, but fails with “cannot parse” when the target version is left out and “invalid syntax” when there are changes.

Yeah we do lock the very coarse Python version configuration of blib2to3 to the target-version, but by default without a target-version specified, it should select all grammars. Are you sure you’re running Black in a clean directory (without pyproject.toml?)

It would be good if Black informed that targeting a higher version of Python that the Black itself is running on is not supported. Actually, it could also work for auto-detected target Python version.

I like the idea, but doing it for auto-detected Python is a bit harder since we would want to warn the user only once. That’s difficult to do when the files are being processed in different processes at the same time thanks to multiprocessing.

[michcio1234]

Yeah, that’s what was happening to me. It was not about the magic comma; it was about any reformatting at all. I think I even tried with --target-version py39 and got the same behaviour (I’m not 100% sure though).

It would be good if Black informed that targeting a higher version of Python that the Black itself is running on is not supported. Actually, it could also work for auto-detected target Python version.