gh-130425: Add "Did you mean" suggestion for del obj.attr

[sobolevn]

I am not quite sure about the proper way to test this:

• There are no tests for delattr and setattr suggestions, it feels like this should be a separate issue to add them. I can do it in the next PR, there would be lots of corner cases
• Local testing works:

» ./python.exe     
Python 3.14.0a5+ (heads/main-dirty:38642bff139, Feb 22 2025, 01:18:34) [Clang 15.0.0 (clang-1500.3.9.4)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> class A:
...     pass
... 
... a = A()
... a.abcde = 1
... del a.abcdf # typo
... 
Traceback (most recent call last):
  File "<python-input-0>", line 6, in <module>
    del a.abcdf # typo
        ^^^^^^^
AttributeError: 'A' object has no attribute 'abcdf'. Did you mean: 'abcde'?
>>> 

• Issue: Provide name suggestions for failed attribute deletions #130425

[StanFromIreland]

I agree the tests should be added, a separate issue would be good :-)

[StanFromIreland]

Looks good now!

[picnixz]

I would suggest either using this one and backport the tests from #130455 or use #130799 (which backports this PR).

[sobolevn]

I think that the PR is quite obvious and combining it with tests is not really worth it :(

[picnixz]

Well @encukou actually asked for a merged PR. I would also prefer having both tests and implementation in the same commit so to avoid double commit reversion in case of an issue we didn't forsee.

[cfbolz]

Please add tests. I agree that the implementation is really simple, but eg PyPy relies on the standard library tests a lot to find out what details in behaviour we are missing (and we already implemented delattr suggestions, so even more important in this specific case).

[sobolevn]

Ok, then - let's wait for #130799 to get finished

[sobolevn]

This was a really simple PR that collapsed under the complexity of the review process. Closing. There are now 2 alternative PRs.