keystore.py had a small zoo of these:

  File "/usr/local/lib/python3.11/dataclasses.py", line 815, in _get_field
    raise ValueError(f'mutable default {type(f.default)} for field '
ValueError: mutable default <class 'staking_deposit.key_handling.keystore.KeystoreModule'> for field kdf is not allowed: use default_factory

I've changed all instances of this error to use a default_factory.

This is due to this change in 3.11:

* Change field default mutability check, allowing only defaults which are
  :term:`hashable` instead of any object which is not an instance of
  :class:`dict`, :class:`list` or :class:`set`. (Contributed by Eric V. Smith in
  :issue:`44674`.)

I ran into this error (and similar ones) on Python 3.11 too:

ValueError: mutable default <class 'staking_deposit.key_handling.keystore.KeystoreModule'> for field kdf is not allowed: use default_factory

I fixed it in my own fork before looking if there was a PR addressing it. Anyway, I came up with pretty much the same solution, except I used a lambda instead of the partial object and it seems to preserve more readability to me, see here . So just adding my 2 cents, consider using the lambda?

Also, an unnecessary extra @dataclass decorator seems to have sneaked in at line 63 🙂

Your solution is way more elegant @eth2353 ! Thank you, adopted.

@holiman Can I pull you in for review? This changes requirements.txt and some of the code so it works with Python 3.11.