soia-client 1.0.7__tar.gz → 1.0.9__tar.gz

{soia_client-1.0.7 → soia_client-1.0.9}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.4
 Name: soia-client
-Version: 1.0.7
+Version: 1.0.9
 Author-email: Tyler Fibonacci <gepheum@gmail.com>
 License: MIT License
         
@@ -31,3 +31,4 @@ Classifier: Programming Language :: Python :: 3
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 License-File: LICENSE
+Dynamic: license-file

{soia_client-1.0.7 → soia_client-1.0.9}/README

@@ -1 +1,5 @@
 python3 -m unittest tests/**.py
+
+pyright
+flake8
+black-formatter

{soia_client-1.0.7 → soia_client-1.0.9}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "soia-client"
-version = "1.0.7"
+version = "1.0.9"
 description = ""
 readme = "README.md"
 authors = [{ name = "Tyler Fibonacci", email = "gepheum@gmail.com" }]

{soia_client-1.0.7 → soia_client-1.0.9}/soia_client.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.4
 Name: soia-client
-Version: 1.0.7
+Version: 1.0.9
 Author-email: Tyler Fibonacci <gepheum@gmail.com>
 License: MIT License
         
@@ -31,3 +31,4 @@ Classifier: Programming Language :: Python :: 3
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 License-File: LICENSE
+Dynamic: license-file

{soia_client-1.0.7 → soia_client-1.0.9}/soia_client.egg-info/SOURCES.txt

@@ -16,7 +16,6 @@ soialib/spec.py
 soialib/timestamp.py
 soialib/impl/__init__.py
 soialib/impl/arrays.py
-soialib/impl/encoding.py
 soialib/impl/enums.py
 soialib/impl/function_maker.py
 soialib/impl/optionals.py

{soia_client-1.0.7 → soia_client-1.0.9}/soialib/impl/arrays.py

@@ -73,16 +73,13 @@ class _ArrayAdapter(TypeAdapter):
         return attr_expr
 
     def to_json_expr(self, in_expr: ExprLike, readable: bool) -> ExprLike:
-        e = Expr.join("_e")
-        item_to_json = self.item_adapter.to_json_expr(e, readable)
-        if item_to_json == e:
+        item_to_json = self.item_adapter.to_json_expr("_e", readable)
+        if Expr.join(item_to_json) == Expr.join("_e"):
             return in_expr
         return Expr.join(
             "[",
             item_to_json,
-            " for ",
-            e,
-            " in ",
+            " for _e in ",
             in_expr,
             "]",
         )
@@ -93,8 +90,8 @@ class _ArrayAdapter(TypeAdapter):
         return Expr.join(
             listuple_class_local,
             "([",
-            self.item_adapter.from_json_expr(Expr.join("_e")),
-            " for e in ",
+            self.item_adapter.from_json_expr("_e"),
+            " for _e in ",
             json_expr,
             "] or ",
             empty_listuple_local,

{soia_client-1.0.7 → soia_client-1.0.9}/soialib/impl/enums.py

@@ -1,20 +1,10 @@
-# TODO: test
-# TODO: unrecognized fields (and handles removed fields)
-
+import copy
 from collections.abc import Callable, Sequence
 from dataclasses import FrozenInstanceError, dataclass
 from typing import Any, Final, Union
 
 from soialib import spec as _spec
-from soialib.impl import primitives
-from soialib.impl.function_maker import (
-    BodyBuilder,
-    BodySpan,
-    Expr,
-    ExprLike,
-    Line,
-    make_function,
-)
+from soialib.impl.function_maker import BodyBuilder, Expr, ExprLike, Line, make_function
 from soialib.impl.repr import repr_impl
 from soialib.impl.type_adapter import TypeAdapter
 
@@ -40,7 +30,6 @@ class EnumAdapter(TypeAdapter):
 
         private_is_enum_attr = _name_private_is_enum_attr(spec.id)
         self.private_is_enum_attr = private_is_enum_attr
-        # TODO: comment
         setattr(base_class, private_is_enum_attr, True)
 
         # Add the constants.
@@ -81,7 +70,10 @@ class EnumAdapter(TypeAdapter):
             setattr(base_class, f"wrap_{value_field.spec.name}", wrap_fn)
 
         base_class._fj = _make_from_json_fn(
-            self.all_constant_fields, value_fields, base_class
+            self.all_constant_fields,
+            value_fields,
+            set(self.spec.removed_numbers),
+            base_class,
         )
 
         # Mark finalization as done.
@@ -116,7 +108,6 @@ class EnumAdapter(TypeAdapter):
 
     def from_json_expr(self, json_expr: ExprLike) -> Expr:
         fn_name = "_fj"
-        # TODO: comment
         from_json_fn = getattr(self.gen_class, fn_name, None)
         if from_json_fn:
             return Expr.join(Expr.local("_fj?", from_json_fn), "(", json_expr, ")")
@@ -149,7 +140,6 @@ def _make_base_class(spec: _spec.Enum) -> type:
             raise FrozenInstanceError(self.__class__.__qualname__)
 
         def __eq__(self, other: Any) -> bool:
-            # TODO: make it work with unrecognized versus UNKNOWN
             if isinstance(other, BaseClass):
                 return other.kind == self.kind and other.value == self.value
             return NotImplemented
@@ -164,7 +154,7 @@ def _make_base_class(spec: _spec.Enum) -> type:
 
 
 def _make_constant_class(base_class: type, spec: _spec.ConstantField) -> type:
-    class ConstantClass(base_class):
+    class Constant(base_class):
         __slots__ = ()
 
         kind: Final[str] = spec.name
@@ -183,7 +173,37 @@ def _make_constant_class(base_class: type, spec: _spec.ConstantField) -> type:
         def __repr__(self) -> str:
             return f"{base_class.__qualname__}.{spec.attribute}"
 
-    return ConstantClass
+    return Constant
+
+
+def _make_unrecognized_class(base_class: type) -> type:
+    """Wraps around an unrecognized dense JSON.
+
+    Looks and acts just like the UNKNOWN constant, except that its JSON representation
+    is the original unrecognized dense JSON.
+    """
+
+    class Unrecognized(base_class):
+        __slots__ = ("_dj",)
+
+        kind: Final[str] = "?"
+        _number: Final[int] = 0
+        # dense JSON
+        _dj: Any
+        # readable JSON
+        _rj: Final[str] = "?"
+        # has value
+        _hv: Final[bool] = False
+
+        def __init__(self, dj: Any):
+            # Do not call super().__init__().
+            object.__setattr__(self, "_dj", copy.deepcopy(dj))
+            object.__setattr__(self, "value", None)
+
+        def __repr__(self) -> str:
+            return f"{base_class.__qualname__}.UNKNOWN"
+
+    return Unrecognized
 
 
 def _make_value_class(
@@ -193,7 +213,7 @@ def _make_value_class(
 ) -> type:
     number = field_spec.number
 
-    class ValueClass(base_class):
+    class Value(base_class):
         __slots__ = ()
 
         kind: Final[str] = field_spec.name
@@ -213,7 +233,7 @@ def _make_value_class(
                 body = value_repr.repr
             return f"{base_class.__qualname__}.wrap_{field_spec.name}({body})"
 
-    ret = ValueClass
+    ret = Value
 
     ret._dj = property(
         make_function(
@@ -272,7 +292,7 @@ def _make_wrap_fn(field: _ValueField) -> Callable[[Any], Any]:
     builder.append_ln(
         Expr.local("setattr", object.__setattr__),
         "(ret, 'value', ",
-        field.field_type.to_frozen_expr(Expr.join("value")),
+        field.field_type.to_frozen_expr("value"),
         ")",
     )
     builder.append_ln("return ret")
@@ -286,9 +306,13 @@ def _make_wrap_fn(field: _ValueField) -> Callable[[Any], Any]:
 def _make_from_json_fn(
     constant_fields: Sequence[_spec.ConstantField],
     value_fields: Sequence[_ValueField],
+    removed_numbers: set[int],
     base_class: type,
 ) -> Callable[[Any], Any]:
+    unrecognized_class = _make_unrecognized_class(base_class)
+    unrecognized_class_local = Expr.local("Unrecognized", unrecognized_class)
     obj_setattr_local = Expr.local("obj_settatr", object.__setattr__)
+    removed_numbers_local = Expr.local("removed_numbers", removed_numbers)
 
     key_to_constant: dict[Union[int, str], Any] = {}
     for field in constant_fields:
@@ -318,21 +342,26 @@ def _make_from_json_fn(
         builder.append_ln("  if json == 0:")
         builder.append_ln("    return ", unknown_constant_local)
     else:
+        # `json.__class__ is int` is significantly faster than `isinstance(json, int)`
         builder.append_ln("  if json.__class__ is int:")
         builder.append_ln("    try:")
         builder.append_ln("      return ", key_to_constant_local, "[json]")
         builder.append_ln("    except:")
-        # TODO: handle unrecognized fields!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-        builder.append_ln("      return ...")
+        if removed_numbers:
+            builder.append_ln("      if json in ", removed_numbers_local, ":")
+            builder.append_ln("        return ", unknown_constant_local)
+        builder.append_ln("      return ", unrecognized_class_local, "(json)")
 
     def append_number_branches(numbers: list[int], indent: str) -> None:
         if len(numbers) == 1:
             number = numbers[0]
             field = key_to_field[number]
             value_class_local = Expr.local("cls?", field.value_class)
-            value_expr = field.field_type.from_json_expr(Expr.join("json[1]"))
+            value_expr = field.field_type.from_json_expr("json[1]")
             builder.append_ln(f"{indent}ret = ", value_class_local, "()")
-            builder.append_ln(indent, obj_setattr_local, "(ret, ", value_expr, ")")
+            builder.append_ln(
+                indent, obj_setattr_local, '(ret, "value", ', value_expr, ")"
+            )
             builder.append_ln(f"{indent}return ret")
         else:
             indented = f"  {indent}"
@@ -344,18 +373,24 @@ def _make_from_json_fn(
             builder.append_ln(f"{indent}else:")
             append_number_branches(numbers[mid_index:], indented)
 
+    # `json.__class__ is list` is significantly faster than `isinstance(json, list)`
     builder.append_ln("  elif json.__class__ is list:")
+    builder.append_ln("    number = json[0]")
     if not value_fields:
-        # TODO: handle unrecognized fields!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-        builder.append_ln("    return ...")
+        # The field was either removed or is an unrecognized field.
+        if removed_numbers:
+            builder.append_ln("    if number in ", removed_numbers_local, ":")
+            builder.append_ln("      return ", unknown_constant_local)
+        builder.append_ln("    return ", unrecognized_class_local, "(json)")
     else:
         if len(value_fields) == 1:
             builder.append_ln(f"    if number != {value_fields[0].spec.number}:")
         else:
             builder.append_ln("    if number not in ", value_keys_local, ":")
-        # TODO: handle unrecognized fields!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-        builder.append_ln("      return ...")
-        builder.append_ln("    number = json[0]")
+        if removed_numbers:
+            builder.append_ln("      if number in ", removed_numbers_local, ":")
+            builder.append_ln("        return ", unknown_constant_local)
+        builder.append_ln("      return ", unrecognized_class_local, "(json)")
         append_number_branches(sorted(numbers), "    ")
 
     # READABLE FORMAT
@@ -367,7 +402,7 @@ def _make_from_json_fn(
         builder.append_ln("    try:")
         builder.append_ln("      return ", key_to_constant_local, "[json]")
         builder.append_ln("    except:")
-        # TODO: comment
+        # In readable mode, drop unrecognized values and use UNKNOWN instead.
         builder.append_ln("      return ", unknown_constant_local)
 
     def append_name_branches(names: list[str], indent: str) -> None:
@@ -377,30 +412,38 @@ def _make_from_json_fn(
             value_class_local = Expr.local("cls?", field.value_class)
             value_expr = field.field_type.from_json_expr("json['value']")
             builder.append_ln(f"{indent}ret = ", value_class_local, "()")
-            builder.append_ln(indent, obj_setattr_local, "(ret, ", value_expr, ")")
+            builder.append_ln(
+                indent, obj_setattr_local, '(ret, "value", ', value_expr, ")"
+            )
             builder.append_ln(f"{indent}return ret")
         else:
             indented = f"  {indent}"
             mid_index = int(len(names) / 2)
             mid_name = names[mid_index - 1]
             operator = "==" if mid_index == 1 else "<="
-            builder.append_ln(f"{indent}if name {operator} '{mid_name}':")
+            builder.append_ln(f"{indent}if kind {operator} '{mid_name}':")
             append_name_branches(names[0:mid_index], indented)
             builder.append_ln(f"{indent}else:")
             append_name_branches(names[mid_index:], indented)
 
     builder.append_ln("  elif isinstance(json, dict):")
     if not value_fields:
-        # TODO: comment
         builder.append_ln("    return ", unknown_constant_local)
     else:
-        builder.append_ln("    name = json['name']")
-        builder.append_ln("    if name not in ", value_keys_local, ":")
-        # TODO: comment
+        builder.append_ln("    kind = json['kind']")
+        builder.append_ln("    if kind not in ", value_keys_local, ":")
         builder.append_ln("      return ", unknown_constant_local)
         builder.append_ln("    else:")
         append_name_branches(sorted(names), "      ")
 
+    # In the unlikely event that json.loads() returns an instance of a subclass of int.
+    builder.append_ln("  elif isinstance(json, int):")
+    builder.append_ln("    json = int(json)")
+    builder.append_ln("  elif isinstance(json, list):")
+    builder.append_ln("    json = list(json)")
+    builder.append_ln("  else:")
+    builder.append_ln("    return TypeError()")
+
     return make_function(
         name="from_json",
         params=["json"],

{soia_client-1.0.7 → soia_client-1.0.9}/soialib/impl/function_maker.py

@@ -1,6 +1,6 @@
 import itertools
 from dataclasses import dataclass
-from typing import Any, Callable, Dict, Iterable, List, Optional, Sequence, Tuple, Union
+from typing import Any, Callable, Iterable, Optional, Sequence, Union
 
 
 def make_function(
@@ -49,12 +49,12 @@ def make_function(
 
     locals = make_locals()
 
-    def line_to_code(l: Union[str, Line]) -> str:
-        if isinstance(l, str):
-            return l
-        return l._to_code(locals)
+    def line_to_code(ln: Union[str, Line]) -> str:
+        if isinstance(ln, str):
+            return ln
+        return ln._to_code(locals)
 
-    body_str = "\n    ".join(line_to_code(l) for l in body) if body else "pass"
+    body_str = "\n    ".join(line_to_code(ln) for ln in body) if body else "pass"
     text = f"""
 def __create_function__({', '.join(locals.locals.keys())}):
   def {name}({', '.join(p._to_code(locals) for p in params)}):
@@ -120,15 +120,11 @@ class LineSpan:
 _EMPTY_LINE_SPAN = LineSpan(())
 
 
-# TODO: comment
 LineSpanLike = Union[str, LineSpan]
 
 
 # A type alias to use when the line span is a Python expression.
 Expr = LineSpan
-
-
-# TODO: comment
 ExprLike = LineSpanLike
 
 
@@ -156,7 +152,7 @@ class BodyBuilder:
         self._lines.append(Line.join(*spans))
 
     def extend(self, other: Iterable[Union[str, Line]], indent: str = ""):
-        self._lines.extend(Line.join(indent, l) for l in other)
+        self._lines.extend(Line.join(indent, ln) for ln in other)
         return self
 
     def build(self) -> Body:

{soia_client-1.0.7 → soia_client-1.0.9}/soialib/impl/optionals.py

@@ -4,7 +4,6 @@ from typing import TypeVar
 from weakref import WeakValueDictionary
 
 from soialib import spec
-from soialib.impl.encoding import NULL_WIRE
 from soialib.impl.function_maker import Expr, ExprLike
 from soialib.impl.type_adapter import TypeAdapter
 

{soia_client-1.0.7 → soia_client-1.0.9}/soialib/impl/primitives.py

@@ -33,9 +33,9 @@ class _BoolAdapter(AbstractPrimitiveAdapter):
         readable: bool,
     ) -> ExprLike:
         if readable:
-            return Expr.join("(1 if ", in_expr, " else 0)")
+            return Expr.join("(True if ", in_expr, " else False)")
         else:
-            return in_expr
+            return Expr.join("(1 if ", in_expr, " else 0)")
 
     def from_json_expr(self, json_expr: ExprLike) -> Expr:
         return Expr.join("(True if ", json_expr, " else False)")
@@ -52,31 +52,72 @@ class _AbstractIntAdapter(AbstractPrimitiveAdapter):
         return "0"
 
     def to_frozen_expr(self, arg_expr: ExprLike) -> Expr:
-        return Expr.join("(", arg_expr, " | 0)")
+        # Must accept float inputs and turn them into ints.
+        return Expr.join("(0).__class__(", arg_expr, ")")
 
     def is_not_default_expr(self, arg_expr: ExprLike, attr_expr: ExprLike) -> ExprLike:
         return arg_expr
 
-    def to_json_expr(self, in_expr: ExprLike, readable: bool) -> ExprLike:
-        return in_expr
-
     def from_json_expr(self, json_expr: ExprLike) -> Expr:
-        return Expr.join("(", json_expr, " | 0)")
+        # Must accept float inputs and string inputs and turn them into ints.
+        return Expr.join(
+            "(0).__class__(",
+            json_expr,
+            ")",
+        )
 
 
 @dataclass(frozen=True)
 class _Int32Adapter(_AbstractIntAdapter):
-    pass
+    def to_json_expr(self, in_expr: ExprLike, readable: bool) -> Expr:
+        return Expr.join(
+            "(-2147483648 if ",
+            in_expr,
+            " <= -2147483648 else ",
+            in_expr,
+            " if ",
+            in_expr,
+            " < 2147483647 else 2147483647)",
+        )
+
+
+def _int64_to_json(i: int) -> int | str:
+    if i < -9007199254740991:  # min safe integer in JavaScript
+        if i <= -9223372036854775808:
+            return "-9223372036854775808"
+        else:
+            return str(i)
+    elif i <= 9007199254740991:  # max safe integer in JavaScript
+        return i
+    elif i < 9223372036854775807:
+        return str(i)
+    else:
+        return "9223372036854775807"
 
 
 @dataclass(frozen=True)
 class _Int64Adapter(_AbstractIntAdapter):
-    pass
+    def to_json_expr(self, in_expr: ExprLike, readable: bool) -> Expr:
+        return Expr.join(Expr.local("int64_to_json", _int64_to_json), "(", in_expr, ")")
+
+
+def _uint64_to_json(i: int) -> int | str:
+    if i <= 0:
+        return 0
+    elif i <= 9007199254740991:  # max safe integer in JavaScript
+        return i
+    elif i < 18446744073709551615:
+        return f"{i}"
+    else:
+        return "18446744073709551615"
 
 
 @dataclass(frozen=True)
 class _Uint64Adapter(_AbstractIntAdapter):
-    pass
+    def to_json_expr(self, in_expr: ExprLike, readable: bool) -> Expr:
+        return Expr.join(
+            Expr.local("uint64_to_json", _uint64_to_json), "(", in_expr, ")"
+        )
 
 
 INT32_ADAPTER: Final[TypeAdapter] = _Int32Adapter()
@@ -203,7 +244,7 @@ class _BytesAdapter(AbstractPrimitiveAdapter):
 
     def from_json_expr(self, json_expr: ExprLike) -> Expr:
         return Expr.join(
-            Expr.local("fromhex", _BytesAdapter._fromhex_fn), "(", json_expr, ")"
+            Expr.local("fromhex", _BytesAdapter._fromhex_fn), "(", json_expr, ' or "")'
         )
 
 

{soia_client-1.0.7 → soia_client-1.0.9}/soialib/impl/repr.py

@@ -31,7 +31,6 @@ def repr_impl(input: Any) -> ReprResult:
             return ReprResult(f"[\n{body}]", complex=True)
     elif isinstance(input, str):
         if "\n" in input:
-            # TODO: comment
             lines = input.split("\n")
             body = "".join(f"  {repr(line)},\n" for line in lines)
             return ReprResult(f"'\\n'.join([\n{body}])", complex=True)