wexample-wex-addon-dev-python 10.1.1__tar.gz → 10.2.0__tar.gz

{wexample_wex_addon_dev_python-10.1.1 → wexample_wex_addon_dev_python-10.2.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: wexample-wex-addon-dev-python
-Version: 10.1.1
+Version: 10.2.0
 Summary: Python dev addon for wex
 Author-Email: weeger <contact@wexample.com>
 License: MIT
@@ -15,10 +15,10 @@ Requires-Dist: griffe>=2.0.2
 Requires-Dist: networkx
 Requires-Dist: pylint
 Requires-Dist: pyright
-Requires-Dist: wexample-api>=6.4.0
-Requires-Dist: wexample-filestate-python>=7.0.0
-Requires-Dist: wexample-wex-addon-ai>=10.0.0
-Requires-Dist: wexample-wex-addon-app>=25.2.0
+Requires-Dist: wexample-api>=6.5.0
+Requires-Dist: wexample-filestate-python>=8.0.0
+Requires-Dist: wexample-wex-addon-ai>=11.0.0
+Requires-Dist: wexample-wex-addon-app>=25.3.0
 Provides-Extra: dev
 Requires-Dist: pytest; extra == "dev"
 Requires-Dist: pytest-cov; extra == "dev"
@@ -26,7 +26,7 @@ Description-Content-Type: text/markdown
 
 # wex_addon_dev_python
 
-Version: 10.1.1
+Version: 10.2.0
 
 Python dev addon for wex
 
@@ -112,10 +112,10 @@ Visit the [Wexample Suite documentation](https://docs.wexample.com) for the comp
 - networkx: 
 - pylint: 
 - pyright: 
-- wexample-api: >=6.4.0
-- wexample-filestate-python: >=7.0.0
-- wexample-wex-addon-ai: >=10.0.0
-- wexample-wex-addon-app: >=25.2.0
+- wexample-api: >=6.5.0
+- wexample-filestate-python: >=8.0.0
+- wexample-wex-addon-ai: >=11.0.0
+- wexample-wex-addon-app: >=25.3.0
 
 ## Versioning & Compatibility Policy
 

{wexample_wex_addon_dev_python-10.1.1 → wexample_wex_addon_dev_python-10.2.0}/README.md

@@ -1,6 +1,6 @@
 # wex_addon_dev_python
 
-Version: 10.1.1
+Version: 10.2.0
 
 Python dev addon for wex
 
@@ -86,10 +86,10 @@ Visit the [Wexample Suite documentation](https://docs.wexample.com) for the comp
 - networkx: 
 - pylint: 
 - pyright: 
-- wexample-api: >=6.4.0
-- wexample-filestate-python: >=7.0.0
-- wexample-wex-addon-ai: >=10.0.0
-- wexample-wex-addon-app: >=25.2.0
+- wexample-api: >=6.5.0
+- wexample-filestate-python: >=8.0.0
+- wexample-wex-addon-ai: >=11.0.0
+- wexample-wex-addon-app: >=25.3.0
 
 ## Versioning & Compatibility Policy
 

{wexample_wex_addon_dev_python-10.1.1 → wexample_wex_addon_dev_python-10.2.0}/pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "pdm.backend"
 
 [project]
 name = "wexample-wex-addon-dev-python"
-version = "10.1.1"
+version = "10.2.0"
 description = "Python dev addon for wex"
 authors = [
     { name = "weeger", email = "contact@wexample.com" },
@@ -24,10 +24,10 @@ dependencies = [
     "networkx",
     "pylint",
     "pyright",
-    "wexample-api>=6.4.0",
-    "wexample-filestate-python>=7.0.0",
-    "wexample-wex-addon-ai>=10.0.0",
-    "wexample-wex-addon-app>=25.2.0",
+    "wexample-api>=6.5.0",
+    "wexample-filestate-python>=8.0.0",
+    "wexample-wex-addon-ai>=11.0.0",
+    "wexample-wex-addon-app>=25.3.0",
 ]
 
 [project.readme]

{wexample_wex_addon_dev_python-10.1.1 → wexample_wex_addon_dev_python-10.2.0}/src/wexample_wex_addon_dev_python/python_addon_manager.py

@@ -2,13 +2,17 @@ from __future__ import annotations
 
 from typing import TYPE_CHECKING
 
+from wexample_wex_addon_ai.selection.selection_contributing_addon_mixin import (
+    SelectionContributingAddonMixin,
+)
 from wexample_wex_core.common.abstract_addon_manager import AbstractAddonManager
 
 if TYPE_CHECKING:
     from wexample_cli.middleware.abstract_middleware import AbstractMiddleware
+    from wexample_wex_addon_ai.selection.abstract_selection import AbstractSelection
 
 
-class PythonAddonManager(AbstractAddonManager):
+class PythonAddonManager(SelectionContributingAddonMixin, AbstractAddonManager):
     def get_local_configurable_keys(self) -> list[dict]:
         from wexample_wex_addon_dev_python.helpers.pdm import (
             apply_pdm_bin_dir,
@@ -32,3 +36,10 @@ class PythonAddonManager(AbstractAddonManager):
         return [
             EachPythonFileMiddleware,
         ]
+
+    def get_selection_classes(self) -> list[type[AbstractSelection]]:
+        from wexample_wex_addon_dev_python.selection.python_code_performance_selection import (
+            PythonCodePerformanceSelection,
+        )
+
+        return [PythonCodePerformanceSelection]

wexample_wex_addon_dev_python-10.2.0/src/wexample_wex_addon_dev_python/selection/abstract_python_code_selection.py

@@ -0,0 +1,45 @@
+from __future__ import annotations
+
+from typing import ClassVar
+
+from wexample_helpers.decorator.base_class import base_class
+from wexample_wex_addon_ai.selection.abstract_code_from_directory_selection import (
+    AbstractCodeFromDirectorySelection,
+)
+
+
+@base_class
+class AbstractPythonCodeSelection(AbstractCodeFromDirectorySelection):
+    """Shared base for every Python-source Selection (perf, tests, …).
+
+    Encapsulates the criteria that are constant across every Python use case in
+    this codebase:
+    - `.py` files only.
+    - Skip `__init__.py` regardless of size — re-export modules carry no logic
+      for any agent we're building today (perf optimization, test writing,
+      docstring extraction would all start by looking at real modules).
+    - Skip Python ecosystem non-source dirs: `__pycache__`, `.venv/`,
+      `.pdm-build/`, plus the `/tests/`/`/build/`/`/dist/` conventions.
+    - 50-byte floor: drops empty stubs (chosen empirically in the shell sweep,
+      kept verbatim until we have a reason to change it).
+
+    Concrete subclasses just pick a `SELECTION_NAME` and, if needed, extend
+    `get_excluded_path_fragments()` / `get_excluded_file_names()` with their
+    use-case specifics via `super() + (…)`.
+    """
+
+    EXTENSIONS: ClassVar[tuple[str, ...]] = (".py",)
+    MIN_SIZE_BYTES: ClassVar[int] = 50
+
+    def get_excluded_file_names(self) -> tuple[str, ...]:
+        return super().get_excluded_file_names() + ("__init__.py",)
+
+    def get_excluded_path_fragments(self) -> tuple[str, ...]:
+        return super().get_excluded_path_fragments() + (
+            "__pycache__",
+            "/tests/",
+            "/build/",
+            "/dist/",
+            "/.venv/",
+            "/.pdm-build/",
+        )

wexample_wex_addon_dev_python-10.2.0/src/wexample_wex_addon_dev_python/selection/python_code_performance_selection.py

@@ -0,0 +1,172 @@
+from __future__ import annotations
+
+from typing import TYPE_CHECKING, ClassVar
+
+from wexample_helpers.decorator.base_class import base_class
+
+from wexample_wex_addon_dev_python.selection.abstract_python_code_selection import (
+    AbstractPythonCodeSelection,
+)
+
+if TYPE_CHECKING:
+    from pathlib import Path
+
+
+@base_class
+class PythonCodePerformanceSelection(AbstractPythonCodeSelection):
+    """Python source files candidate for the performance sweep.
+
+    Inherits every Python-side filter from `AbstractPythonCodeSelection` —
+    nothing perf-specific to declare today beyond the name. If a future need
+    emerges (e.g. include `.pyx` Cython modules, exclude a perf-noisy
+    `vendor/`), extend the inherited methods via `super() + (…)`.
+    """
+
+    SELECTION_NAME: ClassVar[str] = "python-performance"
+
+    def get_required_path_fragments(self) -> tuple[str, ...]:
+        """Only library code under `/src/` — the modern Python layout for any
+        package that uses PEP 517. Drops package-root scripts, `examples/`,
+        `benchmarks/`, `.wex/python/app_manager/*` (wex configuration code, not
+        target of optimization), and anything else outside the package's published
+        surface.
+
+        Symmetric with the exclusion list — defined here rather than in
+        `AbstractPythonCodeSelection` because the `/src/` requirement is opinionated
+        for the perf use case. A test-candidate Selection might want a different
+        positive scope (e.g. include `conftest.py` siblings), so we keep this knob
+        at the concrete level.
+        """
+        return super().get_required_path_fragments() + ("/src/",)
+
+    def is_candidate(self, item: Path) -> bool:
+        """Drop modules that have no executable code for the perf agent to optimize.
+
+        Proxy: "module contains at least one NON-TRIVIAL `FunctionDef` /
+        `AsyncFunctionDef` somewhere in its AST". A trivial function is one
+        whose body is just a stub (`pass`, `...`, docstring only, `raise
+        NotImplementedError`, or `super().__init__()` for `__init__`), or any
+        function decorated with `@abstractmethod`. See `_is_trivial_function`
+        for the exact rules.
+
+        Catches:
+        - Re-export modules (only `from X import Y`).
+        - Pure constant / type-alias modules.
+        - Empty subclasses (`class Foo(Bar): pass`).
+        - Dataclasses without methods, plain Enums, exception classes.
+        - Abstract base classes (every method `@abstractmethod` or `raise
+          NotImplementedError`).
+        - Classes whose only method is `__init__` that just calls `super()`.
+
+        Cost: ~5ms per file (Python's C `ast.parse`). On the ~1200 files of the
+        current sweep that's a few seconds total — orders of magnitude cheaper
+        than one LLM call against any single trivial file, and saves N of them.
+
+        Files we can't decode or parse (UnicodeDecodeError, SyntaxError) are
+        treated as non-candidates — perf optimization on unreadable / broken
+        files isn't what this sweep is for, the operator fixes those first.
+        Other I/O errors (FileNotFoundError, PermissionError) propagate.
+        """
+        import ast
+
+        if not super().is_candidate(item):
+            return False
+        try:
+            source = item.read_text(encoding="utf-8")
+        except UnicodeDecodeError:
+            return False
+        try:
+            tree = ast.parse(source)
+        except SyntaxError:
+            return False
+        return any(
+            isinstance(node, (ast.FunctionDef, ast.AsyncFunctionDef))
+            and not self._is_trivial_function(node)
+            for node in ast.walk(tree)
+        )
+
+    def _is_trivial_function(self, node) -> bool:
+        """True when `node`'s body has no logic worth optimizing.
+
+        Recognized trivial bodies (leading docstring stripped first):
+        - empty (just a docstring and nothing else).
+        - `pass`.
+        - `...` (Ellipsis statement — modern stub idiom).
+        - `raise NotImplementedError` or `raise NotImplementedError(...)`.
+        - `super().__init__(...)` only, when the function IS `__init__`.
+
+        Plus: any decorator named `abstractmethod` (bare or qualified like
+        `abc.abstractmethod`) → trivial regardless of body. Abstract definitions
+        are stubs by contract; the concrete subclass is where the optimizable
+        code lives.
+
+        Doesn't recurse into nested functions — `ast.walk` in `is_candidate`
+        already visits every descendant function independently. Keeps each
+        check decoupled from its container.
+        """
+        import ast
+
+        for deco in node.decorator_list:
+            deco_name = (
+                deco.id
+                if isinstance(deco, ast.Name)
+                else deco.attr
+                if isinstance(deco, ast.Attribute)
+                else None
+            )
+            if deco_name == "abstractmethod":
+                return True
+
+        body = list(node.body)
+        if (
+            body
+            and isinstance(body[0], ast.Expr)
+            and isinstance(body[0].value, ast.Constant)
+            and isinstance(body[0].value.value, str)
+        ):
+            body = body[1:]
+
+        if not body:
+            return True
+        if len(body) != 1:
+            return False
+
+        stmt = body[0]
+
+        if isinstance(stmt, ast.Pass):
+            return True
+
+        if (
+            isinstance(stmt, ast.Expr)
+            and isinstance(stmt.value, ast.Constant)
+            and stmt.value.value is Ellipsis
+        ):
+            return True
+
+        if isinstance(stmt, ast.Raise) and stmt.exc is not None:
+            exc = stmt.exc
+            if isinstance(exc, ast.Name) and exc.id == "NotImplementedError":
+                return True
+            if (
+                isinstance(exc, ast.Call)
+                and isinstance(exc.func, ast.Name)
+                and exc.func.id == "NotImplementedError"
+            ):
+                return True
+
+        if node.name == "__init__" and isinstance(stmt, ast.Expr):
+            call = stmt.value
+            if (
+                isinstance(call, ast.Call)
+                and isinstance(call.func, ast.Attribute)
+                and call.func.attr == "__init__"
+            ):
+                base = call.func.value
+                if (
+                    isinstance(base, ast.Call)
+                    and isinstance(base.func, ast.Name)
+                    and base.func.id == "super"
+                ):
+                    return True
+
+        return False

{wexample_wex_addon_dev_python-10.1.1 → wexample_wex_addon_dev_python-10.2.0}/src/wexample_wex_addon_dev_python/workdir/python_package_workdir.py

@@ -49,7 +49,6 @@ class PythonPackageWorkdir(PythonWorkdir):
             )
 
     def prepare_value(self, raw_value: DictConfig | None = None) -> DictConfig:
-        from wexample_helpers.helpers.array import array_dict_get_by
         from wexample_helpers.helpers.file import file_read
         from wexample_helpers.helpers.module import module_get_path
 
@@ -102,56 +101,44 @@ class PythonPackageWorkdir(PythonWorkdir):
             },
         )
 
-        # Retrieve the '.gitignore' configuration or create it if it doesn't exist
-        config_gitignore = array_dict_get_by("name", ".gitignore", children)
-        if config_gitignore is not None:
-            generic_gitignore_rules = {
-                "Python artifacts": [
-                    "*.egg-info",
-                    "__pycache__/",
-                    "*.py[cod]",
-                    "*.pyo",
-                ],
-                "Build directories": [
-                    "/build/",
-                    "/dist/",
-                    "/pip-wheel-metadata/",
-                    "/.pdm-build/",
-                ],
-                "Virtual environments": [
-                    ".env",
-                    ".venv",
-                    "venv/",
-                ],
-                "Test and coverage artifacts": [
-                    ".tox/",
-                    ".mypy_cache/",
-                    "pytest_cache/",
-                    ".coverage",
-                    "htmlcov/",
-                ],
-                "Editor and IDE settings": [
-                    ".vscode/",
-                    ".idea/",
-                    "*.swp",
-                    "*~",
-                ],
-            }
-
-            should_contain_lines = config_gitignore.setdefault(
-                "should_contain_lines", []
-            )
-            if not isinstance(should_contain_lines, list):
-                raise ValueError("'should_contain_lines' must be a list")
-
-            for category, rules in generic_gitignore_rules.items():
-                category_header = f"# {category}"
-                if category_header not in should_contain_lines:
-                    should_contain_lines.append(category_header)
-
-                for rule in rules:
-                    if rule not in should_contain_lines:
-                        should_contain_lines.append(rule)
+        # Sectioned Python-package gitignore rules. The rectifier (when it
+        # runs on globless files) preserves section headers and sorts
+        # entries within each section; here we glob-heavy patterns make most
+        # of these no-ops on the rectifier side, but the lines themselves
+        # are still ensured by `should_contain_lines`.
+        for section, rules in {
+            "Python artifacts": [
+                "*.egg-info",
+                "__pycache__/",
+                "*.py[cod]",
+                "*.pyo",
+            ],
+            "Build directories": [
+                "/build/",
+                "/dist/",
+                "/pip-wheel-metadata/",
+                "/.pdm-build/",
+            ],
+            "Virtual environments": [
+                ".env",
+                ".venv",
+                "venv/",
+            ],
+            "Test and coverage artifacts": [
+                ".tox/",
+                ".mypy_cache/",
+                "pytest_cache/",
+                ".coverage",
+                "htmlcov/",
+            ],
+            "Editor and IDE settings": [
+                ".vscode/",
+                ".idea/",
+                "*.swp",
+                "*~",
+            ],
+        }.items():
+            self.add_gitignore_rules(children, *rules, section=section)
 
         return raw_value
 

{wexample_wex_addon_dev_python-10.1.1 → wexample_wex_addon_dev_python-10.2.0}/src/wexample_wex_addon_dev_python/workdir/python_workdir.py

@@ -183,7 +183,6 @@ class PythonWorkdir(
         from wexample_filestate.option.children_filter_option import (
             ChildrenFilterOption,
         )
-        from wexample_helpers.helpers.array import array_dict_get_by
 
         from wexample_wex_addon_dev_python.file.python_pyproject_toml_file import (
             PythonPyprojectTomlFile,
@@ -195,15 +194,12 @@ class PythonWorkdir(
 
         children = raw_value["children"]
 
-        # Add rules to .gitignore
-        array_dict_get_by("name", ".gitignore", raw_value["children"]).setdefault(
-            "should_contain_lines", []
-        ).extend(
-            [
-                ".pdm-python",
-                ".python-version",
-                f"/{PYTHON_FILE_PYTEST_COVERAGE_JSON}",
-            ]
+        self.add_gitignore_rules(
+            raw_value["children"],
+            ".pdm-python",
+            ".python-version",
+            f"/{PYTHON_FILE_PYTEST_COVERAGE_JSON}",
+            section="Python",
         )
 
         children.extend(