[bbbbbbbbbbb114]

What do these changes do?

Fix a runtime introspection mismatch when a caching decorator wraps an instance method. The method call works, but the exposed metadata still describes an unbound function.

For an ordinary bound method, inspect.signature(service.plain) does not include self. Previously, inspect.signature(service.cached) still included it. Also, bypassing the cache required passing the receiver explicitly as service.cached.__wrapped__(service, 7).

Use the standard-library types.MethodType to bind the proxy's original __wrapped__ function to the current receiver.

Are there changes in behavior for the user?

• inspect.signature(service.cached) now matches the ordinary bound method's signature, without the extra self.
• service.cached.__wrapped__(7) calls the original method without requiring the receiver again.
• The change is limited to runtime introspection of bound instance methods. Cache hits, key computation and class-level descriptor logic are unchanged, and it does not introduce eager signature computation.

Related issue number

Checklist

• I think the code is well written
• Unit tests for the changes exist
• Documentation reflects the changes

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

[greptile-apps]

Confidence Score: 5/5

[Low impact] The PR appears safe to merge; no outstanding findings remain.

_{Reviews (3) · Last reviewed commit: "Keep invalid signature metadata from blo..." · Reviewed by Greptile}