Fix #242: Add type parameters to PRecord for pylance/pyright compatib…

[VYaswanthKumar]

…ility

[tobgu]

Hi!

There seems to be quite a lot of changes going into this PR that I\m not sure are related to the intent as expressed by the PR title. Furthermore some of them are backwards incompatible (see failing tests for some examples) which is pretty much a no-no unless there is good reasons for it.

Thanks

[VYaswanthKumar]

Hi @tobgu,

I've added a new commit (88e2884) that fixes the backward compatibility issues you mentioned. The previous failing tests were caused by:

1. AttributeError: 'str' object has no attribute 'type' - The is_field_ignore_extra_complaint function in _precord.py was being called with PRecord as the first argument instead of CheckedType, causing type checking errors.

2. TypeError: CheckedType.create() got an unexpected keyword argument 'ignore_extra' - The abstract create() method in CheckedType and the CheckedPMap.create() method didn't accept the ignore_extra parameter, even though it was being passed by code that checks for fields supporting this parameter.

Fixes applied:

• Added ignore_extra=False parameter to CheckedType.create() abstract method in _checked_types.py (line 24)
• Added ignore_extra=False parameter to CheckedPMap.create() method in _checked_types.py (line 383)
• Fixed the call to is_field_ignore_extra_complaint in _precord.py (line 111) to use CheckedType instead of PRecord

These changes maintain backward compatibility while properly supporting the ignore_extra feature. Could you please review and approve the workflow so the tests can run? The fixes should resolve all the test failures.

Thank you!

[tobgu]

Looks good, thanks!