Fixed pyright errors across modules

[larsewi]

Summary

• Added __getattr__ with -> Any return type to AttributeObject so pyright accepts dynamic attribute access
• Fixed Result.NOTKEPT typo in gpg module (NOT_KEPT)
• Fixed evaluate_promise parameter name mismatches in ansible and systemd modules to match base class signature
• Fixed create_attribute_object signature in json module to match base class
• Fixed evaluate_command_policy return type annotation in iptables module (Result -> str)

🤖 Generated with Claude Code

[olehermanse]

Looks good overall, please improve the first commit a bit.

[larsewi]

"getattr is only invoked if the attribute wasn't found the usual ways"

• https://stackoverflow.com/questions/3278077/difference-between-getattr-and-getattribute/3278092

from cfengine_module_library import AttributeObject
a = AttributeObject({"x": 1, "y": 2})
print("Accessing x:", a.x)
print("Accessing y:", a.y)
print("Accessing z:")
a.z

$ PYTHONPATH=libraries/python python3 test_attribute_object.py
Accessing x: 1
Accessing y: 2
Accessing z:
Traceback (most recent call last):
  File "/home/larsewi/ntech/modules/test_attribute_object.py", line 8, in <module>
    a.z
  File "/home/larsewi/ntech/modules/libraries/python/cfengine_module_library.py", line 100, in __getattr__
    raise AttributeError(
AttributeError: 'AttributeObject' object has no attribute 'z'

[olehermanse]

@larsewi please put it in a comment - the code looks weird as is.