Infinite loop/recursion error when accessing LockFile properties via __getitem__

[mcrichton]

Affects:

plette version 1.0.0

Expected:

accessing lockfile["default"] should return the same value as lockfile.default

Actual:

accessing lockfile["default"] causes an infinite loop (RecursionError)

Cause:

• https://github.com/sarugaku/plette/blob/master/src/plette/lockfiles.py#L141
  • regression from

    plette/src/plette/lockfiles.py

    Line 109 in bfda17b

    value = self._data[key]

my_lockfile["default"] -> __getitem__(self, "default") -> value = self["default"] -> __getitem__(self, "default") -> ...

Repro:

Using (an admittedly outdated version of) requirementslib with latest plette. Affected line here: https://github.com/sarugaku/requirementslib/blob/d6b5165bfc6f043b1156c209d85f3e09b60aeaa8/src/requirementslib/models/lockfile.py#L79

Can also be easily repro'd in tests by doing lock["default"]

Notes:

The tests in https://github.com/sarugaku/plette/blob/master/tests/test_lockfiles.py do not catch this issue, as they use lock.<...> rather than lock[<...>]

[oz123]

This should be fixed by #55

Please verify by:

python -i -m plette -f Pipfile.lock
>>> lockfile.dev_packages
>>> lockfile.develop

[mcrichton]

Tested and working, thank you.

The only change I had to make was removing the (unused) import tomllib as I am using Python 3.10: mcrichton@b395ca8

[oz123]

Fixed in v2.0.0