Speed up

Author: jpsca

.gitignore

@@ -17,3 +17,4 @@ dist/
 pip-wheel-metadata
 coverage.xml
 
+profile_results.prof

profile_formidable.py

@@ -0,0 +1,176 @@
+"""
+Profiling script for formidable.
+
+Exercises the main code paths:
+1. Form initialization (field discovery, cloning)
+2. Request data parsing (flat dict -> nested)
+3. Field setting & validation
+4. NestedForms (multiple sub-forms)
+5. SlugField (unicode normalization)
+6. HTML rendering
+
+Run:
+    uv run python profile_formidable.py
+
+Outputs:
+    - Console: top 40 cumulative-time entries
+    - profile_results.prof: full cProfile dump (for snakeviz, etc.)
+"""
+
+import cProfile
+import pstats
+
+import formidable as f
+
+
+# -- Define realistic forms --------------------------------------------------
+
+class AddressForm(f.Form):
+    street = f.TextField()
+    city = f.TextField()
+    zip_code = f.TextField(required=False)
+    country = f.TextField()
+
+
+class ContactForm(f.Form):
+    name = f.TextField()
+    email = f.EmailField()
+    phone = f.TextField(required=False)
+    age = f.IntegerField(required=False)
+    website = f.URLField(required=False)
+    slug = f.SlugField(required=False)
+    subscribe = f.BooleanField(required=False)
+    notes = f.TextField(required=False)
+
+    address = f.FormField(AddressForm)
+    addresses = f.NestedForms(AddressForm, min_items=1, max_items=10)
+
+
+# -- Build test data ----------------------------------------------------------
+
+def make_flat_reqdata(n_addresses=5):
+    """Simulate flat request data as it would arrive from an HTML form."""
+    data = {
+        "name": "José García-López",
+        "email": "jose@example.com",
+        "phone": "+1-555-0123",
+        "age": "42",
+        "website": "https://example.com/~jose",
+        "slug": "Héllo Wörld! Ça va très bien, merci αβγδ",
+        "subscribe": "on",
+        "notes": "Some notes here",
+        # FormField (single nested)
+        "address[street]": "123 Main St",
+        "address[city]": "Springfield",
+        "address[zip_code]": "62704",
+        "address[country]": "US",
+    }
+    # NestedForms (multiple nested)
+    for i in range(n_addresses):
+        prefix = f"addresses[{i}]"
+        data[f"{prefix}[street]"] = f"{100 + i} Oak Avenue"
+        data[f"{prefix}[city]"] = f"City {i}"
+        data[f"{prefix}[zip_code]"] = f"{10000 + i}"
+        data[f"{prefix}[country]"] = "US"
+
+    return data
+
+
+# -- Workloads ----------------------------------------------------------------
+
+def workload_init_only(iterations=1000):
+    """Measure form class instantiation (no data)."""
+    for _ in range(iterations):
+        ContactForm()
+
+
+def workload_parse_set_validate(iterations=500):
+    """Measure parse + set + validate cycle."""
+    data = make_flat_reqdata(n_addresses=5)
+    for _ in range(iterations):
+        form = ContactForm(data)
+        form.is_valid
+
+
+def workload_large_nested(iterations=100):
+    """Measure with many nested forms."""
+    data = make_flat_reqdata(n_addresses=50)
+    for _ in range(iterations):
+        form = ContactForm(data)
+        form.is_valid
+
+
+def workload_render_html(iterations=500):
+    """Measure HTML rendering helpers."""
+    data = make_flat_reqdata(n_addresses=3)
+    form = ContactForm(data)
+    for _ in range(iterations):
+        for field in form:
+            field.label("Label")
+            field.text_input()
+            field.error_tag()
+
+
+def workload_slug(iterations=2000):
+    """Measure slug field processing."""
+    from formidable.fields.slug import slugify
+    texts = [
+        "Héllo Wörld! Ça va très bien",
+        "αβγδεζηθικλμνξοπρστυφχψω",
+        "The Quick Brown Fox Jumps Over The Lazy Dog 123!@#$%",
+        "ñoño año español café résumé naïve",
+        "ა ბ გ დ ე ვ ზ თ ი კ ლ მ ნ ო პ",
+    ]
+    for _ in range(iterations):
+        for text in texts:
+            slugify(text)
+
+
+def workload_parser(iterations=2000):
+    """Measure raw parser performance."""
+    from formidable.parser import parse
+    data = make_flat_reqdata(n_addresses=20)
+    for _ in range(iterations):
+        parse(data)
+
+
+def run_all():
+    """Run all workloads together for a combined profile."""
+    workload_init_only()
+    workload_parse_set_validate()
+    workload_large_nested()
+    workload_render_html()
+    workload_slug()
+    workload_parser()
+
+
+# -- Main --------------------------------------------------------------------
+
+if __name__ == "__main__":
+    print("Profiling formidable...")
+    print("=" * 70)
+
+    profiler = cProfile.Profile()
+    profiler.enable()
+    run_all()
+    profiler.disable()
+
+    # Save for visualization tools (snakeviz, pyprof2calltree, etc.)
+    profiler.dump_stats("profile_results.prof")
+
+    # Print summary
+    stats = pstats.Stats(profiler)
+    stats.strip_dirs()
+    stats.sort_stats("cumulative")
+
+    print("\n TOP 40 BY CUMULATIVE TIME")
+    print("=" * 70)
+    stats.print_stats(40)
+
+    print("\n TOP 30 BY TOTAL (SELF) TIME")
+    print("=" * 70)
+    stats.sort_stats("tottime")
+    stats.print_stats(30)
+
+    print(f"\nFull profile saved to: profile_results.prof")
+    print("Visualize with: uv run snakeviz profile_results.prof")

pyproject.toml

@@ -4,7 +4,7 @@ requires = ["setuptools"]
 
 [project]
 name = "formidable"
-version = "0.15.1"
+version = "0.16.0"
 description = "Small but powerful library for rendering and processing web forms"
 authors = [{ name = "Juan-Pablo Scaletti", email = "juanpablo@jpscaletti.com" }]
 license = "MIT"
@@ -23,7 +23,6 @@ requires-python = ">=3.12,<4"
 dependencies = [
     "idna>=3.10",
     "markupsafe>=3.0.3",
-    "writeadoc",
 ]
 
 [project.optional-dependencies]

src/formidable/fields/base.py

@@ -2,9 +2,9 @@
 Formidable | Copyright (c) 2025 Juan-Pablo Scaletti
 """
 
+import itertools
 import typing as t
 from collections.abc import Iterable
-from uuid import uuid4
 
 from markupsafe import Markup
 
@@ -15,6 +15,9 @@
     from ..form import Form
 
 
+_field_counter = itertools.count()
+
+
 class Field:
     """
     Base class for all form fields.
@@ -55,7 +58,12 @@ def __init__(
         self.default = default
         self.value = self.default_value
         self.messages = messages if messages is not None else {}
-        self.id = f"f{uuid4().hex}"
+        self.id = f"f{next(_field_counter)}"
+
+    def __copy__(self):
+        clone = object.__new__(self.__class__)
+        clone.__dict__ = self.__dict__.copy()
+        return clone
 
     def __repr__(self):
         attrs = [
@@ -742,9 +750,7 @@ def _render_html_attrs(self, attrs: dict[str, t.Any]) -> str:
         html_props = []
         clean_attrs = {}
         for key, value in attrs.items():
-            if key == "class_":
-                key = "class"
-            key = key.replace("_", "-")
+            key = "class" if key == "class_" else key.replace("_", "-")
             if isinstance(value, bool):
                 if value:
                     html_props.append(key)

src/formidable/fields/formfield.py

@@ -40,7 +40,7 @@ def __init__(
 
     def __copy__(self):
         clone = object.__new__(self.__class__)
-        clone.__dict__.update(self.__dict__)
+        clone.__dict__ = self.__dict__.copy()
         clone.form = self.FormClass()
         return clone
 

src/formidable/fields/nested.py

@@ -68,7 +68,7 @@ def __init__(
 
     def __copy__(self):
         clone = object.__new__(self.__class__)
-        clone.__dict__.update(self.__dict__)
+        clone.__dict__ = self.__dict__.copy()
         clone.empty_form = self.FormClass()
         clone.forms = []
         return clone

src/formidable/fields/slug.py

@@ -302,19 +302,29 @@
 }
 
 
+_rx_non_word = re.compile(r"[^\w\s-]")
+_rx_sep = re.compile(r"[-\s]+")
+
+# Pre-built translation table for str.translate() — much faster than
+# iterating char-by-char with CHAR_MAP.get() in Python.
+_TRANSLATE_TABLE = str.maketrans(
+    {ord(k): v for k, v in CHAR_MAP.items()}
+)
+
+
 def slugify(value: str) -> str:
     """
     A very simple function to convert a string to a slug.
     """
     value = unicodedata.normalize("NFKC", value).lower()
     # Replace some non-ASCII characters
-    value = "".join(CHAR_MAP.get(c, c) for c in value)
+    value = value.translate(_TRANSLATE_TABLE)
     # Remove any remaining non-ASCII characters
     value = value.encode("ascii", "ignore").decode("ascii")
     # Replace non-word characters with hyphens
-    value = re.sub(r"[^\w\s-]", "", value)
+    value = _rx_non_word.sub("", value)
     # Replace whitespace and hyphens with a single hyphen
-    return re.sub(r"[-\s]+", "-", value).strip("-_")
+    return _rx_sep.sub("-", value).strip("-_")
 
 
 class SlugField(TextField):