[Autofix error] Invalid code produce autofix error

[qarmin]

Ruff a45753f

# flake8: noqa: F405
fom opts import *  # noqa

with

ruff file.py --fix

error: Autofix introduced a syntax error. Reverting all changes.

This indicates a bug in `ruff`. If you could open an issue at:

    https://github.com/charliermarsh/ruff/issues/new?title=%5BAutofix%20error%5D

...quoting the contents of `Desktop/RunEveryCommand/Ruff/Broken/94966__init__ (9 413th copy)0.py`, along with the `pyproject.toml` settings and executed command, we'd be very appreciative!

[charliermarsh]

It might be a bug that we're considering that noqa to be unused though, it should be suppressing an E999 error...? Not sure why that's not getting raised here.

[JonathanPlasse]

I think # noqa should never suppress E999, that seems to be the problem.

[charliermarsh]

I think it should, that was used as a workaround for a while for match statement. But we're raising a RUF100, then fixing that, then Ruff thinks we introduced a syntax error.

[charliermarsh]

Ahh, the issue is # flake8: noqa: F405. That suppress all errors, just like in Flake8.

[charliermarsh]

The issue is here:

Err(parse_error) => {
    if settings.rules.enabled(Rule::SyntaxError) {
        pycodestyle::rules::syntax_error(&mut diagnostics, &parse_error);
    }

    // If the syntax error is ignored, suppress it (regardless of whether
    // `Rule::SyntaxError` is enabled).
    if !rule_is_ignored(
        Rule::SyntaxError,
        parse_error.location.row(),
        &directives.noqa_line_for,
        locator,
    ) {
        error = Some(parse_error);
    }
}

That needs to respect the whole file ignores, not just the inline ignores.