I haven't really got the bandwidth to look at this right now, sorry :)

cc @serhiy-storchaka @pitrou

I agree, we should drop the defaults part.

>>> import sys, inspect
>>> def f(x=1):
...    print(inspect.getargvalues(sys._getframe()))
...
>>> f()
ArgInfo(args=['x'], varargs=None, keywords=None, locals={'x': 1})

Current API does not show defaults, only locals. But, locals are stored as .f_locals, it is a stable API. Users can use .f_locals to populate defaults, it is quite easy. Later we can add defaults support if users ask for it.

Showing defaults is also not safe in this case by default, for example we can leak things like passwords or secret keys.

I agree, we should drop the defaults part.

I looked at tests where the frame is captured whereas frame locals are not modified, so "it just works" magically. But if I look at the test which modify a variable, I now understand how wrong it is. A frame local is not a function default parameter value: they are two different things. signature() is the signature, before the function is called. Frame locals is the "live state" of a frame, it's unrelated.

Moreover, frame locals can contain sensitive information (like a password) which should not be exposed in a signature.

Updated, now defaults are not shown. Right now this is very close to getargvalues:

>>> import sys, inspect
>>> def f(x=1):
...    print(inspect.getargvalues(sys._getframe()))
...
>>> f()
ArgInfo(args=['x'], varargs=None, keywords=None, locals={'x': 1})

And we are trying to replace this old function, so I guess it is fine :)

(This has a merge conflict)

@sobolevn: Can you please try to fix the merge conflict?

This PR is stale because it has been open for 30 days with no activity.