wexample-wex-addon-dev-php 12.1.0__tar.gz → 12.2.0__tar.gz

{wexample_wex_addon_dev_php-12.1.0 → wexample_wex_addon_dev_php-12.2.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: wexample-wex-addon-dev-php
-Version: 12.1.0
+Version: 12.2.0
 Summary: Extends wex with PHP-specific development commands for Composer, Laravel, Symfony, and WordPress services running in Docker containers
 Author-Email: weeger <contact@wexample.com>
 License: MIT
@@ -14,7 +14,7 @@ Requires-Dist: cattrs>=23.1.0
 Requires-Dist: wexample-filestate-php>=6.4.0
 Requires-Dist: wexample-filestate-symfony>=2.0.0
 Requires-Dist: wexample-wex-addon-ai>=15.0.0
-Requires-Dist: wexample-wex-addon-app>=31.0.0
+Requires-Dist: wexample-wex-addon-app>=31.1.0
 Requires-Dist: wexample-wex-addon-dev-javascript>=8.1.0
 Provides-Extra: dev
 Requires-Dist: pytest; extra == "dev"
@@ -23,7 +23,7 @@ Description-Content-Type: text/markdown
 
 # wex_addon_dev_php
 
-Version: 12.1.0
+Version: 12.2.0
 
 `wex_addon_dev_php` extends the wex CLI with PHP-specific development commands for Composer, Laravel, Symfony, and WordPress services running in Docker containers. It is aimed at PHP developers who manage their apps through wex and need framework-aware automation — dependency wiring, migration runs, lock-file refreshes — executed directly inside the running service containers. The addon registers four workdir types (`php`, `php-package`, `php-packages-suite`, `php-symfony`) and contributes its commands through the standard `PhpAddonManager` plugin point.
 
@@ -276,7 +276,7 @@ Visit the [Wexample Suite documentation](https://docs.wexample.com) for the comp
 - wexample-filestate-php: >=6.4.0
 - wexample-filestate-symfony: >=2.0.0
 - wexample-wex-addon-ai: >=15.0.0
-- wexample-wex-addon-app: >=31.0.0
+- wexample-wex-addon-app: >=31.1.0
 - wexample-wex-addon-dev-javascript: >=8.1.0
 
 ## Versioning & Compatibility Policy

{wexample_wex_addon_dev_php-12.1.0 → wexample_wex_addon_dev_php-12.2.0}/README.md

@@ -1,6 +1,6 @@
 # wex_addon_dev_php
 
-Version: 12.1.0
+Version: 12.2.0
 
 `wex_addon_dev_php` extends the wex CLI with PHP-specific development commands for Composer, Laravel, Symfony, and WordPress services running in Docker containers. It is aimed at PHP developers who manage their apps through wex and need framework-aware automation — dependency wiring, migration runs, lock-file refreshes — executed directly inside the running service containers. The addon registers four workdir types (`php`, `php-package`, `php-packages-suite`, `php-symfony`) and contributes its commands through the standard `PhpAddonManager` plugin point.
 
@@ -253,7 +253,7 @@ Visit the [Wexample Suite documentation](https://docs.wexample.com) for the comp
 - wexample-filestate-php: >=6.4.0
 - wexample-filestate-symfony: >=2.0.0
 - wexample-wex-addon-ai: >=15.0.0
-- wexample-wex-addon-app: >=31.0.0
+- wexample-wex-addon-app: >=31.1.0
 - wexample-wex-addon-dev-javascript: >=8.1.0
 
 ## Versioning & Compatibility Policy

{wexample_wex_addon_dev_php-12.1.0 → wexample_wex_addon_dev_php-12.2.0}/pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "pdm.backend"
 
 [project]
 name = "wexample-wex-addon-dev-php"
-version = "12.1.0"
+version = "12.2.0"
 description = "Extends wex with PHP-specific development commands for Composer, Laravel, Symfony, and WordPress services running in Docker containers"
 authors = [
     { name = "weeger", email = "contact@wexample.com" },
@@ -23,7 +23,7 @@ dependencies = [
     "wexample-filestate-php>=6.4.0",
     "wexample-filestate-symfony>=2.0.0",
     "wexample-wex-addon-ai>=15.0.0",
-    "wexample-wex-addon-app>=31.0.0",
+    "wexample-wex-addon-app>=31.1.0",
     "wexample-wex-addon-dev-javascript>=8.1.0",
 ]
 

{wexample_wex_addon_dev_php-12.1.0 → wexample_wex_addon_dev_php-12.2.0}/src/wexample_wex_addon_dev_php/workdir/php_package_workdir.py

@@ -27,6 +27,16 @@ if TYPE_CHECKING:
 
 @base_class
 class PhpPackageWorkdir(WithRunnerWorkdirMixin, PhpWorkdir):
+    @staticmethod
+    def line_opens_a_comment(line: str) -> bool:
+        """Whether a line is a comment, judged on the characters it opens with.
+
+        A line-level test and not a parse: it catches the docblock body and the `//`
+        line, which is where a class gets *mentioned* rather than used. A name trailing
+        real code after a `//` on the same line stays reported — rarer, and for a
+        dependency check erring toward reporting is the right way round.
+        """
+        return line.lstrip().startswith(("*", "//", "#", "/*"))
     def get_package_import_name(self) -> str:
         """Get the full package import name with vendor prefix."""
         return f"{string_to_pascal_case(self.get_vendor_name())}\\{string_to_pascal_case(self.get_project_name())}"
@@ -60,12 +70,37 @@ class PhpPackageWorkdir(WithRunnerWorkdirMixin, PhpWorkdir):
     def search_imports_in_codebase(
         self, searched_package: PhpPackageWorkdir
     ) -> list[SearchResult]:
-        """Find PHP `use`/qualified references to the given package."""
+        """Find PHP `use`/qualified references to the given package.
+
+        Two shapes count as an import: the `use` statement, and the fully qualified
+        name written inline (`new \\Wexample\\Foo\\Bar()`). The second is why matches
+        have to be sifted afterwards — a fully qualified name reads the same inside a
+        docblock, and an `@deprecated` note pointing at the class that replaces this
+        one is documentation, not a dependency.
+
+        The `use` branch needs no sifting: it is anchored on `^\\s*use`, which a
+        docblock's leading `*` already prevents from matching.
+        """
         import re
 
         pkg = re.escape(searched_package.get_package_import_name())
         pattern = rf"(?m)^\s*use\s+{pkg}(?:\\\\[\w]+)*\s*;|{pkg}(?:\\\\[\w]+)*(?!\w)"
-        return self.search_in_codebase(pattern, regex=True, flags=re.MULTILINE)
+        results = self.search_in_codebase(pattern, regex=True, flags=re.MULTILINE)
+
+        lines_by_path: dict[str, list[str]] = {}
+        kept: list[SearchResult] = []
+
+        for result in results:
+            path = str(result.item.get_path())
+            if path not in lines_by_path:
+                lines_by_path[path] = result.item.read_text().splitlines()
+
+            lines = lines_by_path[path]
+            line = lines[result.line - 1] if result.line <= len(lines) else ""
+            if not self.line_opens_a_comment(line):
+                kept.append(result)
+
+        return kept
 
     def search_in_codebase(
         self, string: str, *, regex: bool = False, flags: int = 0

wexample_wex_addon_dev_php-12.2.0/tests/workdir/test_php_package_workdir.py

@@ -0,0 +1,54 @@
+from __future__ import annotations
+
+import pytest
+
+# The line that set this off: an `@deprecated` note naming the class that replaces
+# this one. Verbatim from symfony-helpers' SystemParameter.php.
+DEPRECATION_NOTE = (
+    " * @deprecated use `Wexample\\SymfonyPlatform\\Entity\\Configuration`, which holds a"
+)
+
+
+def test_a_docblock_body_is_a_comment() -> None:
+    assert _line_opens_a_comment(DEPRECATION_NOTE)
+
+
+def test_a_name_trailing_real_code_stays_reported() -> None:
+    """Documented limit: only the line's opening decides, so this is still an import."""
+    assert not _line_opens_a_comment(
+        "$c = new \\Wexample\\SymfonyPlatform\\Entity\\Configuration(); // built here"
+    )
+
+
+@pytest.mark.parametrize(
+    "line",
+    [
+        "// use Wexample\\SymfonyPlatform\\Entity\\Configuration;",
+        "# Wexample\\SymfonyPlatform\\Entity\\Configuration",
+        "/* Wexample\\SymfonyPlatform\\Entity\\Configuration */",
+        "    * @see Wexample\\SymfonyPlatform\\Entity\\Configuration",
+    ],
+)
+def test_every_comment_opener_is_recognised(line: str) -> None:
+    assert _line_opens_a_comment(line)
+
+
+@pytest.mark.parametrize(
+    "line",
+    [
+        "use Wexample\\SymfonyPlatform\\Entity\\Configuration;",
+        "    use Wexample\\SymfonyPlatform\\Entity\\Configuration;",
+        "        $c = new \\Wexample\\SymfonyPlatform\\Entity\\Configuration();",
+        "class SystemParameter extends AbstractEntity",
+    ],
+)
+def test_real_code_is_not_a_comment(line: str) -> None:
+    assert not _line_opens_a_comment(line)
+
+
+def _line_opens_a_comment(line: str) -> bool:
+    from wexample_wex_addon_dev_php.workdir.php_package_workdir import (
+        PhpPackageWorkdir,
+    )
+
+    return PhpPackageWorkdir.line_opens_a_comment(line)