Add comments

Author: Archmonger

src/reactpy/_console/rewrite_props.py

@@ -1,7 +1,6 @@
 from __future__ import annotations
 
 import ast
-import re
 from copy import copy
 from keyword import kwlist
 from pathlib import Path
@@ -15,62 +14,73 @@
     rewrite_changed_nodes,
 )
 
-CAMEL_CASE_SUB_PATTERN = re.compile(r"(?<!^)(?=[A-Z])")
-
 
 @click.command()
 @click.argument("paths", nargs=-1, type=click.Path(exists=True))
 def rewrite_props(paths: list[str]) -> None:
-    """Rewrite snake_case to props to camelCase"""
-
+    """Rewrite snake_case props to camelCase in the specified paths"""
     for p in map(Path, paths):
+        # Process each file or recursively process each Python file in directories
         for f in [p] if p.is_file() else p.rglob("*.py"):
             result = generate_rewrite(file=f, source=f.read_text(encoding="utf-8"))
             if result is not None:
                 f.write_text(result)
 
 
 def generate_rewrite(file: Path, source: str) -> str | None:
-    tree = ast.parse(source)
+    """Generate the rewritten source code if changes are detected"""
+    tree = ast.parse(source)  # Parse the source code into an AST
 
-    changed = find_nodes_to_change(tree)
+    changed = find_nodes_to_change(tree)  # Find nodes that need to be changed
     if not changed:
-        return None
+        return None  # Return None if no changes are needed
 
-    new = rewrite_changed_nodes(file, source, tree, changed)
+    new = rewrite_changed_nodes(
+        file, source, tree, changed
+    )  # Rewrite the changed nodes
     return new
 
 
 def find_nodes_to_change(tree: ast.AST) -> list[ChangedNode]:
+    """Find nodes in the AST that need to be changed"""
     changed: list[ChangedNode] = []
     for el_info in find_element_constructor_usages(tree):
+        # Check if the props need to be rewritten
         if _rewrite_props(el_info.props, _construct_prop_item):
+            # Add the changed node to the list
             changed.append(ChangedNode(el_info.call, el_info.parents))
     return changed
 
 
 def conv_attr_name(name: str) -> str:
+    """Convert snake_case attribute name to camelCase"""
     if name in kwlist:
-        return name
+        return name  # Return the name as is if it's a Python keyword
+    # Convert snake_case to CamelCase
     result = name.replace("_", " ").title().replace(" ", "")
+    # Ensure the first letter is lowercase
     result = name[0].lower() + name[1:]
     return result
 
 
 def _construct_prop_item(key: str, value: ast.expr) -> tuple[str, ast.expr]:
+    """Construct a new prop item with the converted key and possibly modified value"""
     if key == "style" and isinstance(value, (ast.Dict, ast.Call)):
+        # Create a copy of the value to avoid modifying the original
         new_value = copy(value)
         if _rewrite_props(
             new_value,
             lambda k, v: (
                 (k, v)
-                # avoid infinite recursion
+                # Avoid infinite recursion
                 if k == "style"
                 else _construct_prop_item(k, v)
             ),
         ):
+            # Update the value if changes were made
             value = new_value
     else:
+        # Convert the key to camelCase
         key = conv_attr_name(key)
     return key, value
 
@@ -79,12 +89,15 @@ def _rewrite_props(
     props_node: ast.Dict | ast.Call,
     constructor: Callable[[str, ast.expr], tuple[str, ast.expr]],
 ) -> bool:
+    """Rewrite the props in the given AST node using the provided constructor"""
+    did_change = False
     if isinstance(props_node, ast.Dict):
-        did_change = False
         keys: list[ast.expr | None] = []
         values: list[ast.expr] = []
+        # Iterate over the keys and values in the dictionary
         for k, v in zip(props_node.keys, props_node.values):
             if isinstance(k, ast.Constant) and isinstance(k.value, str):
+                # Construct the new key and value
                 k_value, new_v = constructor(k.value, v)
                 if k_value != k.value or new_v is not v:
                     did_change = True
@@ -93,20 +106,22 @@ def _rewrite_props(
             keys.append(k)
             values.append(v)
         if not did_change:
-            return False
+            return False  # Return False if no changes were made
         props_node.keys = keys
         props_node.values = values
     else:
         did_change = False
         keywords: list[ast.keyword] = []
+        # Iterate over the keywords in the call
         for kw in props_node.keywords:
             if kw.arg is not None:
+                # Construct the new keyword argument and value
                 kw_arg, kw_value = constructor(kw.arg, kw.value)
                 if kw_arg != kw.arg or kw_value is not kw.value:
                     did_change = True
                 kw = ast.keyword(arg=kw_arg, value=kw_value)
             keywords.append(kw)
         if not did_change:
-            return False
+            return False  # Return False if no changes were made
         props_node.keywords = keywords
     return True

src/reactpy/widgets.py

@@ -73,7 +73,7 @@ def sync_inputs(event: dict[str, Any]) -> None:
 
     inputs: list[VdomDict] = []
     for attrs in attributes:
-        inputs.append(html.input({**attrs, "on_change": sync_inputs, "value": value}))
+        inputs.append(html.input({**attrs, "onChange": sync_inputs, "value": value}))
 
     return inputs