wexample-wex-addon-dev-python 10.2.0__py3-none-any.whl → 11.0.0__py3-none-any.whl

wexample_wex_addon_dev_python/commands/code/check/mypy.py

@@ -35,12 +35,13 @@ def _code_check_mypy(kernel: Kernel, file_path: str) -> bool:
     # Build and check the file
     source = BuildSource(path=file_path, module=None, text=None)
     result = build.build(sources=[source], options=options, alt_lib_path=None)
-    if result.errors:
+    errors = result.errors
+    if errors:
         kernel.io.log_indent_up()
-        kernel.io.error(f"Mypy errors:")
+        kernel.io.error("Mypy errors:")
         kernel.io.log_indent_up()
 
-        for error in result.errors:
+        for error in errors:
             kernel.io.error(message=error, symbol=False)
 
         kernel.io.log_indent_down(number=2)

wexample_wex_addon_dev_python/commands/code/check/pylint.py

@@ -5,6 +5,16 @@ from typing import TYPE_CHECKING
 if TYPE_CHECKING:
     from wexample_cli.context.execution_context import ExecutionContext
 
+# Pre-computed once at import time; avoids list allocation + join on every call.
+_PYLINT_DISABLE_FLAG = "--disable=" + ",".join([
+    "missing-module-docstring",
+    "import-outside-toplevel",
+    "no-name-in-module",
+    "broad-exception-caught",
+    "c-extension-no-member",
+    "line-too-long",
+])
+
 
 def _code_check_pylint(context: ExecutionContext, file_path: str) -> bool:
     """Check a Python file using pylint for code quality.
@@ -22,23 +32,13 @@ def _code_check_pylint(context: ExecutionContext, file_path: str) -> bool:
 
     # Use subprocess to capture pylint output
     # This avoids issues with pylint's direct printing to stdout
-    # List of warnings to disable
-    disabled_warnings = [
-        "missing-module-docstring",
-        "import-outside-toplevel",
-        "no-name-in-module",
-        "broad-exception-caught",
-        "c-extension-no-member",
-        "line-too-long",
-    ]
-
     cmd = [
         sys.executable,
         "-m",
         "pylint",
         file_path,
         "--output-format=json",
-        f"--disable={','.join(disabled_warnings)}",
+        _PYLINT_DISABLE_FLAG,
     ]
     process = subprocess.run(cmd, capture_output=True, text=True, check=False)
 
@@ -53,19 +53,25 @@ def _code_check_pylint(context: ExecutionContext, file_path: str) -> bool:
     # Parse the JSON output
     results = json.loads(json_output)
 
-    # Filter messages by type
-    errors = [msg for msg in results if msg.get("type") in ("error", "fatal")]
-    warnings = [msg for msg in results if msg.get("type") == "warning"]
-    conventions = [
-        msg for msg in results if msg.get("type") in ("convention", "refactor", "info")
-    ]
+    # Categorise messages in a single pass instead of three separate comprehensions.
+    errors: list = []
+    warnings: list = []
+    conventions: list = []
+    for msg in results:
+        t = msg.get("type")
+        if t in ("error", "fatal"):
+            errors.append(msg)
+        elif t == "warning":
+            warnings.append(msg)
+        elif t in ("convention", "refactor", "info"):
+            conventions.append(msg)
 
     # Display results if any issues found
     if errors or warnings or conventions:
         # Display errors
         if errors:
             context.io.log_indent_up()
-            context.io.error(f"Pylint errors:")
+            context.io.error("Pylint errors:")
             context.io.log_indent_up()
 
             for error in errors:
@@ -80,7 +86,7 @@ def _code_check_pylint(context: ExecutionContext, file_path: str) -> bool:
         # Display warnings with detailed logging
         if warnings:
             context.io.log_indent_up()
-            context.io.warning(f"Pylint warnings:")
+            context.io.warning("Pylint warnings:")
             context.io.log_indent_up()
 
             for warning in warnings:

wexample_wex_addon_dev_python/commands/code/check/pyright.py

@@ -1,5 +1,8 @@
 from __future__ import annotations
 
+import json
+import subprocess
+import sys
 from typing import TYPE_CHECKING
 
 if TYPE_CHECKING:
@@ -16,10 +19,6 @@ def _code_check_pyright(kernel: Kernel, file_path: str) -> bool:
     Returns:
         bool: True if check passes, False otherwise
     """
-    import json
-    import subprocess
-    import sys
-
     # Use subprocess to run pyright
     cmd = [sys.executable, "-m", "pyright", file_path, "--outputjson"]
     process = subprocess.run(cmd, capture_output=True, text=True, check=False)
@@ -42,20 +41,25 @@ def _code_check_pyright(kernel: Kernel, file_path: str) -> bool:
     # Parse the JSON output
     results = json.loads(json_output)
 
-    # Extract diagnostics
-    diagnostics = results.get("diagnostics", [])
-
-    # Filter by severity
-    errors = [diag for diag in diagnostics if diag.get("severity") == "error"]
-    warnings = [diag for diag in diagnostics if diag.get("severity") == "warning"]
-    info = [diag for diag in diagnostics if diag.get("severity") == "information"]
+    # Classify diagnostics in a single pass (avoids 3× iteration and repeated .get("severity"))
+    errors: list = []
+    warnings: list = []
+    info: list = []
+    for diag in results.get("diagnostics", []):
+        sev = diag.get("severity")
+        if sev == "error":
+            errors.append(diag)
+        elif sev == "warning":
+            warnings.append(diag)
+        elif sev == "information":
+            info.append(diag)
 
     # Display results if any issues found
     if errors or warnings or info:
         # Display errors
         if errors:
             kernel.io.log_indent_up()
-            kernel.io.error(f"Pyright errors:")
+            kernel.io.error("Pyright errors:")
 
             for error in errors:
                 line = error.get("range", {}).get("start", {}).get("line", 0) + 1
@@ -70,7 +74,7 @@ def _code_check_pyright(kernel: Kernel, file_path: str) -> bool:
         # Display warnings
         if warnings:
             kernel.io.log_indent_up()
-            kernel.io.warning(f"Pyright warnings:")
+            kernel.io.warning("Pyright warnings:")
 
             for warning in warnings:
                 line = warning.get("range", {}).get("start", {}).get("line", 0) + 1
@@ -85,7 +89,7 @@ def _code_check_pyright(kernel: Kernel, file_path: str) -> bool:
         # Display information
         if info:
             kernel.io.log_indent_up()
-            kernel.io.info(f"Pyright information:")
+            kernel.io.info("Pyright information:")
             kernel.io.log_indent_up()
 
             for item in info:

wexample_wex_addon_dev_python/commands/code/check.py

@@ -74,16 +74,13 @@ def python__code__check(
     }
 
     # Determine which tools to run
-    if tool and tool.lower() in tool_map:
+    tool_lower = tool.lower() if tool else None
+    if tool_lower and tool_lower in tool_map:
         # Run only the specified tool
-        check_functions = [tool_map[tool.lower()]]
+        check_functions = [tool_map[tool_lower]]
     else:
         # Run all tools if no specific tool is specified or if the specified tool is invalid
-        check_functions = [
-            _code_check_mypy,
-            _code_check_pylint,
-            _code_check_pyright,
-        ]
+        check_functions = list(tool_map.values())
 
     # Track overall success
     all_checks_passed = True

wexample_wex_addon_dev_python/commands/code/format/black.py

@@ -1,5 +1,7 @@
 from __future__ import annotations
 
+import subprocess
+import sys
 from typing import TYPE_CHECKING
 
 if TYPE_CHECKING:
@@ -16,11 +18,8 @@ def _code_format_black(kernel: Kernel, file_path: str) -> bool:
     Returns:
         bool: True if formatting succeeds, False otherwise
     """
-    import subprocess
-    import sys
-
     # Use subprocess to run black
-    cmd = [sys.executable, "-m", "black", file_path]
+    cmd = (sys.executable, "-m", "black", file_path)
     process = subprocess.run(cmd, capture_output=True, text=True, check=False)
 
     # Check if the command was successful
@@ -30,17 +29,17 @@ def _code_format_black(kernel: Kernel, file_path: str) -> bool:
         else:
             kernel.io.success(f"Black: {file_path} already well formatted")
         return True
-    else:
-        kernel.io.error(f"Black failed to format {file_path}")
-        kernel.io.log_indent_up()
 
-        if process.stderr:
-            kernel.io.error(f"Error: {process.stderr}", symbol=False)
-        if process.stdout:
-            kernel.io.error(f"Output: {process.stdout}", symbol=False)
+    kernel.io.error(f"Black failed to format {file_path}")
+    kernel.io.log_indent_up()
+
+    if process.stderr:
+        kernel.io.error(f"Error: {process.stderr}", symbol=False)
+    if process.stdout:
+        kernel.io.error(f"Output: {process.stdout}", symbol=False)
 
-        # Add detailed error properties
-        kernel.io.properties({"returncode": process.returncode, "command": cmd})
+    # Add detailed error properties
+    kernel.io.properties({"returncode": process.returncode, "command": cmd})
 
-        kernel.io.log_indent_down()
-        return False
+    kernel.io.log_indent_down()
+    return False

wexample_wex_addon_dev_python/commands/code/format/isort.py

@@ -1,5 +1,7 @@
 from __future__ import annotations
 
+import subprocess
+import sys
 from typing import TYPE_CHECKING
 
 if TYPE_CHECKING:
@@ -16,13 +18,10 @@ def _code_format_isort(kernel: Kernel, file_path: str) -> bool:
     Returns:
         bool: True if formatting succeeds, False otherwise
     """
-    import subprocess
-    import sys
-
     # Use subprocess to run isort
     # --profile=black ensures compatibility with Black formatter
     cmd = [sys.executable, "-m", "isort", "--profile=black", file_path]
-    process = subprocess.run(cmd, capture_output=True, text=True, check=False)
+    process = subprocess.run(cmd, capture_output=True, text=True)
 
     # Check if the command was successful
     if process.returncode == 0:
@@ -31,17 +30,17 @@ def _code_format_isort(kernel: Kernel, file_path: str) -> bool:
         else:
             kernel.io.success(f"isort successfully reformatted imports in {file_path}")
         return True
-    else:
-        kernel.io.error(f"isort failed to format imports in {file_path}")
-        kernel.io.log_indent_up()
 
-        if process.stderr:
-            kernel.io.error(f"Error: {process.stderr}", symbol=False)
-        if process.stdout:
-            kernel.io.error(f"Output: {process.stdout}", symbol=False)
+    kernel.io.error(f"isort failed to format imports in {file_path}")
+    kernel.io.log_indent_up()
+
+    if process.stderr:
+        kernel.io.error(f"Error: {process.stderr}", symbol=False)
+    if process.stdout:
+        kernel.io.error(f"Output: {process.stdout}", symbol=False)
 
-        # Add detailed error properties
-        kernel.io.properties({"returncode": process.returncode, "command": cmd})
+    # Add detailed error properties
+    kernel.io.properties({"returncode": process.returncode, "command": cmd})
 
-        kernel.io.log_indent_down()
-        return False
+    kernel.io.log_indent_down()
+    return False

wexample_wex_addon_dev_python/commands/code/format.py

@@ -61,13 +61,16 @@ def python__code__format(
         "isort": _code_format_isort,
     }
 
+    # Normalise once to avoid repeated .lower() calls
+    tool_lower = tool.lower() if tool else None
+
     # Determine which tools to run
-    if tool and tool.lower() in tool_map:
+    if tool_lower in tool_map:
         # Run only the specified tool
-        format_functions = [tool_map[tool.lower()]]
+        format_functions = [tool_map[tool_lower]]
     else:
-        # Run all tools if no specific tool is specified or if the specified tool is invalid
-        if tool and tool.lower() not in tool_map:
+        # In the else branch, a truthy tool_lower is already known to be absent from tool_map
+        if tool_lower:
             kernel.io.warning(f"Unknown tool '{tool}', running all available tools")
 
         # Run isort first, then black (recommended order)
@@ -85,7 +88,7 @@ def python__code__format(
         format_result = format_function(kernel, file)
 
         # Update overall success status
-        all_formats_passed = all_formats_passed and format_result
+        all_formats_passed &= format_result
 
         # Stop if a format fails and stop_on_failure is True
         if not format_result and stop_on_failure:

wexample_wex_addon_dev_python/commands/code/rename.py

@@ -0,0 +1,134 @@
+from __future__ import annotations
+
+from typing import TYPE_CHECKING
+
+from wexample_cli.const.tags import AudienceTag, EffectTag, ScopeTag
+from wexample_cli.decorator.command import command
+from wexample_cli.decorator.middleware import middleware
+from wexample_cli.decorator.option import option
+from wexample_wex_addon_app.middleware.app_middleware import AppMiddleware
+from wexample_wex_core.const.globals import COMMAND_TYPE_ADDON
+
+from wexample_wex_addon_dev_python.const.tags import DomainTag
+
+if TYPE_CHECKING:
+    from wexample_app.response.abstract_response import AbstractResponse
+    from wexample_cli.context.execution_context import ExecutionContext
+    from wexample_wex_addon_app.workdir.managed_workdir import ManagedWorkdir
+
+
+@option(
+    name="source",
+    type=str,
+    required=True,
+    description="Dotted source name to rename (e.g. 'mylib.helpers').",
+)
+@option(
+    name="target",
+    type=str,
+    required=True,
+    description="Dotted target name (e.g. 'mylib.helper').",
+)
+@option(
+    name="kind",
+    type=str,
+    required=False,
+    default="package",
+    description="Symbol kind to rename. Step 1 supports only 'package'.",
+)
+@option(
+    name="dry_run",
+    type=bool,
+    is_flag=True,
+    default=False,
+    description="Preview impacted files without writing. Default: changes are applied.",
+)
+@middleware(middleware=AppMiddleware)
+@command(
+    type=COMMAND_TYPE_ADDON,
+    description=(
+        "Rename a Python package directory and propagate the change to every "
+        "import statement in the workdir. Dry-run by default."
+    ),
+    tags=[
+        DomainTag.LANGUAGE_PYTHON,
+        EffectTag.WRITE,
+        AudienceTag.AGENT_SAFE,
+        ScopeTag.LOCAL,
+        ScopeTag.PACKAGE,
+    ],
+)
+def python__code__rename(
+    context: ExecutionContext,
+    app_workdir: ManagedWorkdir,
+    source: str,
+    target: str,
+    kind: str = "package",
+    dry_run: bool = False,
+) -> AbstractResponse:
+    from wexample_app.response.failure_response import FailureResponse
+    from wexample_app.response.table_response import TableResponse
+
+    from wexample_wex_addon_dev_python.refactor.python_package_rename_handler import (
+        PythonPackageRenameHandler,
+    )
+    from wexample_wex_addon_dev_python.refactor.symbol_kind import SymbolKind
+
+    try:
+        kind_enum = SymbolKind(kind)
+    except ValueError:
+        return FailureResponse(
+            kernel=context.kernel,
+            message=(
+                f"Unknown --kind {kind!r}. Valid: "
+                f"{', '.join([k.value for k in SymbolKind])}"
+            ),
+        )
+
+    if kind_enum is not SymbolKind.PACKAGE:
+        return FailureResponse(
+            kernel=context.kernel,
+            message=(
+                f"--kind {kind_enum.value} not implemented yet "
+                "(step 1 covers 'package' only)."
+            ),
+        )
+
+    workdir_path = app_workdir.get_path()
+    handler = PythonPackageRenameHandler(
+        workdir_path=workdir_path,
+        source=source,
+        target=target,
+        dry_run=dry_run,
+    )
+    try:
+        report = handler.run()
+    except FileNotFoundError:
+        return FailureResponse(
+            kernel=context.kernel,
+            message=(
+                f"No Python package matching '{source}' under {workdir_path}. "
+                "Check the dotted name (e.g. 'mylib.helpers') and that you're "
+                "in the right workdir."
+            ),
+        )
+    except ValueError as e:
+        # Raised on ambiguous match (multiple packages) or invalid dotted name.
+        return FailureResponse(kernel=context.kernel, message=str(e))
+
+    mode = "APPLIED" if not dry_run else "DRY-RUN"
+    context.io.log(
+        f"[{mode}] {report.package_dir_old} → {report.package_dir_new}"
+    )
+    context.io.log(
+        f"[{mode}] scanned={report.files_scanned} changed={len(report.files_changed)}"
+    )
+
+    rows = [
+        [str(p.relative_to(workdir_path))] for p in report.files_changed
+    ]
+    return TableResponse(
+        kernel=context.kernel,
+        content=rows,
+        headers=["File"],
+    )

wexample_wex_addon_dev_python/commands/examples/validate.py

@@ -27,5 +27,4 @@ def python__examples__validate(
         ExamplePydanticClassWithPublicVarInternallyDefined,
     )
 
-    example_class = ExamplePydanticClassWithPublicVarInternallyDefined()
-    context.kernel.log(example_class)
+    context.kernel.log(ExamplePydanticClassWithPublicVarInternallyDefined())

wexample_wex_addon_dev_python/common/pypi_registry_gateway.py

@@ -1,9 +1,13 @@
 from __future__ import annotations
 
+import base64
+
 from wexample_api.common.abstract_gateway import AbstractGateway
 from wexample_helpers.classes.field import public_field
 from wexample_helpers.decorator.base_class import base_class
 
+_HAS_VERSION_STATUS_CODES: tuple[int, ...] = (200, 404)
+
 
 @base_class
 class PypiRegistryGateway(AbstractGateway):
@@ -27,8 +31,6 @@ class PypiRegistryGateway(AbstractGateway):
     def __attrs_post_init__(self) -> None:
         super().__attrs_post_init__()
         if self.token:
-            import base64
-
             credentials = base64.b64encode(
                 f"{self.username or '__token__'}:{self.token}".encode()
             ).decode()
@@ -43,7 +45,7 @@ class PypiRegistryGateway(AbstractGateway):
         """
         response = self.make_request(
             endpoint=f"simple/{package}/",
-            expected_status_codes=[200, 404],
+            expected_status_codes=_HAS_VERSION_STATUS_CODES,
             fatal_if_unexpected=False,
             quiet=True,
         )

wexample_wex_addon_dev_python/config_value/python_package_readme_config_value.py

@@ -1,15 +1,10 @@
 from __future__ import annotations
 
-from typing import TYPE_CHECKING
-
 from wexample_helpers.decorator.base_class import base_class
 from wexample_wex_addon_app.config_value.app_readme_config_value import (
     AppReadmeConfigValue,
 )
 
-if TYPE_CHECKING:
-    pass
-
 
 @base_class
 class PythonPackageReadmeContentConfigValue(AppReadmeConfigValue):
@@ -22,9 +17,8 @@ class PythonPackageReadmeContentConfigValue(AppReadmeConfigValue):
     def _get_app_homepage(self) -> str:
         """Extract homepage URL from pyproject.toml."""
         project = self.workdir.get_app_config()
-        urls = (
-            project.get("urls", {}) if isinstance(project.get("urls", {}), dict) else {}
-        )
+        raw_urls = project.get("urls", {})
+        urls = raw_urls if isinstance(raw_urls, dict) else {}
         return urls.get("homepage") or urls.get("Homepage") or ""
 
     def _get_project_license(self) -> str | None:

wexample_wex_addon_dev_python/file/python_app_iml_file.py

@@ -29,13 +29,14 @@ class PythonAppImlFile(ImlFile):
         return ({"@type": "sourceFolder", "@forTests": "false"},)
 
     def _default_source_folders(self) -> Iterable[dict[str, Any]]:
+        base = self.MODULE_DIR_URL
         return (
             {
-                "@url": f"{self.MODULE_DIR_URL}/src",
+                "@url": f"{base}/src",
                 "@isTestSource": "false",
             },
             {
-                "@url": f"{self.MODULE_DIR_URL}/tests",
+                "@url": f"{base}/tests",
                 "@isTestSource": "true",
             },
         )

wexample_wex_addon_dev_python/file/python_pyproject_toml_file.py

@@ -105,7 +105,7 @@ class PythonPyprojectTomlFile(AppDependenciesConfigFileMixin, TomlFile):
         deps = self._get_deps_array(optional=optional, group=group)
 
         map = {}
-        for spec in list(deps):
+        for spec in deps:
             req = Requirement(spec)
             # name: version
             map[canonicalize_name(req.name)] = str(req.specifier)
@@ -140,7 +140,7 @@ class PythonPyprojectTomlFile(AppDependenciesConfigFileMixin, TomlFile):
 
         target = canonicalize_name(package_name)
         filtered: list[str] = []
-        for existing in list(deps):
+        for existing in deps:
             try:
                 existing_name = canonicalize_name(Requirement(str(existing)).name)
             except Exception:
@@ -298,12 +298,12 @@ class PythonPyprojectTomlFile(AppDependenciesConfigFileMixin, TomlFile):
         deps_arr = self._dependencies_array()
 
         runtime_pkgs = {
-            package_normalize_name(toml_get_string_value(it)) for it in list(deps_arr)
+            package_normalize_name(toml_get_string_value(it)) for it in deps_arr
         }
-        dev_values = [toml_get_string_value(it).strip() for it in list(dev_arr)]
+        dev_set = {toml_get_string_value(it).strip() for it in dev_arr}
 
         for pkg in ["pytest", "pytest-cov"]:
-            if pkg not in runtime_pkgs and pkg not in dev_values:
+            if pkg not in runtime_pkgs and pkg not in dev_set:
                 dev_arr.append(pkg)
 
         toml_sort_string_array(dev_arr)
@@ -326,7 +326,6 @@ class PythonPyprojectTomlFile(AppDependenciesConfigFileMixin, TomlFile):
         )
 
         deps_arr = self._dependencies_array()
-        toml_sort_string_array(deps_arr)
 
         # Read the keep list from [tool.filestate].keep
         keep_packages: set[str] = set()
@@ -390,8 +389,9 @@ class PythonPyprojectTomlFile(AppDependenciesConfigFileMixin, TomlFile):
         existing_keys = list(d.keys())
 
         # Build the new order: ordered keys first, then remaining keys
+        key_order_set = frozenset(key_order)
         ordered_keys = [k for k in key_order if k in existing_keys]
-        remaining_keys = [k for k in existing_keys if k not in key_order]
+        remaining_keys = [k for k in existing_keys if k not in key_order_set]
         new_order = ordered_keys + remaining_keys
 
         # Reorder by removing and re-adding in the desired order

wexample_wex_addon_dev_python/helpers/pdm.py

@@ -4,8 +4,9 @@ from __future__ import annotations
 def apply_pdm_bin_dir(value: str) -> None:
     import os
 
-    if value not in os.environ.get("PATH", ""):
-        os.environ["PATH"] = f"{value}:{os.environ.get('PATH', '')}"
+    path = os.environ.get("PATH", "")
+    if value not in path:
+        os.environ["PATH"] = f"{value}:{path}"
 
 
 def detect_pdm_bin_dir() -> str | None:

wexample_wex_addon_dev_python/middleware/each_python_file_middleware.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import os
 from typing import TYPE_CHECKING
 
 from wexample_wex_core.middleware.each_file_middleware import EachFileMiddleware
@@ -32,12 +33,13 @@ class EachPythonFileMiddleware(EachFileMiddleware):
     python_extension_only: bool = True
 
     def __init__(self, **kwargs) -> None:
-        # Allow overriding the default settings
-        if "python_extension_only" in kwargs:
-            self.python_extension_only = kwargs.pop("python_extension_only")
-
-        if "ignored_directories" in kwargs:
-            self.ignored_directories = set(kwargs.pop("ignored_directories"))
+        # Allow overriding the default settings (pop avoids a double-lookup)
+        self.python_extension_only = kwargs.pop(
+            "python_extension_only", self.python_extension_only
+        )
+        ignored = kwargs.pop("ignored_directories", None)
+        if ignored is not None:
+            self.ignored_directories = set(ignored)
 
         super().__init__(**kwargs)
 
@@ -65,13 +67,9 @@ class EachPythonFileMiddleware(EachFileMiddleware):
         Returns:
             True if the item should be processed, False otherwise
         """
-        # First check if it's a file (parent class behavior)
-        if not os.path.isfile(item_path):
+        # Cheap string check before the expensive isfile() syscall: skip
+        # non-.py paths immediately when extension filtering is active.
+        if self.python_extension_only and not item_path.endswith(".py"):
             return False
 
-        # If python_extension_only is enabled, check file extension
-        if self.python_extension_only:
-            return item_path.endswith(".py")
-
-        # Otherwise, accept all files
-        return True
+        return os.path.isfile(item_path)

wexample_wex_addon_dev_python/refactor/__init__.py

@@ -0,0 +1,7 @@
+"""Code-refactor primitives.
+
+Step 1: support `kind=package` only. Designed to grow to module / class /
+function / constant / method through the same `SymbolKind` enum.
+"""
+
+from __future__ import annotations