wexample-wex-addon-dev-python 13.3.1__tar.gz → 15.0.0__tar.gz

{wexample_wex_addon_dev_python-13.3.1 → wexample_wex_addon_dev_python-15.0.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: wexample-wex-addon-dev-python
-Version: 13.3.1
+Version: 15.0.0
 Summary: Extends wex with commands to format, lint, and rename Python symbols, plus workdir types for Python packages and suites
 Author-Email: weeger <contact@wexample.com>
 License: MIT
@@ -18,9 +18,9 @@ Requires-Dist: networkx
 Requires-Dist: pylint
 Requires-Dist: pyright
 Requires-Dist: wexample-api>=6.8.0
-Requires-Dist: wexample-filestate-python>=9.0.0
-Requires-Dist: wexample-wex-addon-ai>=15.0.0
-Requires-Dist: wexample-wex-addon-app>=31.1.0
+Requires-Dist: wexample-filestate-python>=9.1.0
+Requires-Dist: wexample-wex-addon-ai>=16.0.0
+Requires-Dist: wexample-wex-addon-app>=32.0.0
 Provides-Extra: dev
 Requires-Dist: pytest; extra == "dev"
 Requires-Dist: pytest-cov; extra == "dev"
@@ -28,7 +28,7 @@ Description-Content-Type: text/markdown
 
 # wex_addon_dev_python
 
-Version: 13.3.1
+Version: 15.0.0
 
 `wex_addon_dev_python` extends the wex CLI with a `python::` command group that formats source files with isort and black (`python::code/format`), checks them with mypy, pylint, and pyright (`python::code/check`), and renames Python symbols — packages, modules, classes, functions, constants — while propagating every affected import statement across the workdir (`python::code/rename`). It also ships `PythonPackageWorkdir` and `PythonPackagesSuiteWorkdir`, two workdir types that let wex manage a single PDM-based package or a multi-package suite as a first-class project structure. The intended users are wex operators who develop and maintain Python packages within the wexample ecosystem.
 
@@ -215,12 +215,14 @@ src/wexample_wex_addon_dev_python/selection/abstract_python_code_selection.py es
 
 src/wexample_wex_addon_dev_python/selection/python_code_performance_selection.py inherits that baseline and adds two constraints: the file must be under `/src/`, and its AST must contain at least one non-trivial function. Trivial is defined as: `pass`, `...`, docstring-only, `raise NotImplementedError`, a same-name `super()` pass-through, or any `@abstractmethod`.
 
+src/wexample_wex_addon_dev_python/selection/package_description_missing_selection.py selects the package's own `.wex/config.yml`, and only when `global.description` is empty. Selecting the config file rather than the package makes the item what is to be edited, so the agent writes that one field and leaves the rest of the file alone. The description is copied verbatim into `pyproject.toml` and shown on the package registry.
+
+This replaced a filestate operation that called the agent from inside `apply_operation`. Writing a description is a job given to an agent, not a file state to converge on: rectification retried it every pass, and an agent declining to invent a description for a package with no code kept the hole open, which kept the retry going.
+
 ### Supporting pieces
 
 src/wexample_wex_addon_dev_python/common/pypi_registry_gateway.py is a thin HTTP client over the PEP 503 Simple index. It accepts optional Basic auth (`username` + `token`) and exposes `has_version(package, version) -> bool`.
 
-src/wexample_wex_addon_dev_python/operation/generated_description_operation.py is a filestate operation that invokes a `ClaudeAgent` to produce a one-sentence `global.description` and writes it to the app config. The model is called only during `apply_operation`, not during dry-run, so a preview is always offline. Failures (network, model, length over 200 chars) leave the key unset and log a warning rather than aborting the rectification.
-
 src/wexample_wex_addon_dev_python/services/python/app_service.py contributes the initial `pyproject.toml` and `src/app/` scaffolding when a new Python app service is set up.
 
 src/wexample_wex_addon_dev_python/const/tags.py declares the `DomainTag` constants (`domain:format`, `domain:language-python`, `domain:lint`, `domain:service`) used in command `@command(tags=[…])` decorators.
@@ -255,9 +257,9 @@ Visit the [Wexample Suite documentation](https://docs.wexample.com) for the comp
 - pylint: 
 - pyright: 
 - wexample-api: >=6.8.0
-- wexample-filestate-python: >=9.0.0
-- wexample-wex-addon-ai: >=15.0.0
-- wexample-wex-addon-app: >=31.1.0
+- wexample-filestate-python: >=9.1.0
+- wexample-wex-addon-ai: >=16.0.0
+- wexample-wex-addon-app: >=32.0.0
 
 ## Versioning & Compatibility Policy
 

{wexample_wex_addon_dev_python-13.3.1 → wexample_wex_addon_dev_python-15.0.0}/README.md

@@ -1,6 +1,6 @@
 # wex_addon_dev_python
 
-Version: 13.3.1
+Version: 15.0.0
 
 `wex_addon_dev_python` extends the wex CLI with a `python::` command group that formats source files with isort and black (`python::code/format`), checks them with mypy, pylint, and pyright (`python::code/check`), and renames Python symbols — packages, modules, classes, functions, constants — while propagating every affected import statement across the workdir (`python::code/rename`). It also ships `PythonPackageWorkdir` and `PythonPackagesSuiteWorkdir`, two workdir types that let wex manage a single PDM-based package or a multi-package suite as a first-class project structure. The intended users are wex operators who develop and maintain Python packages within the wexample ecosystem.
 
@@ -187,12 +187,14 @@ src/wexample_wex_addon_dev_python/selection/abstract_python_code_selection.py es
 
 src/wexample_wex_addon_dev_python/selection/python_code_performance_selection.py inherits that baseline and adds two constraints: the file must be under `/src/`, and its AST must contain at least one non-trivial function. Trivial is defined as: `pass`, `...`, docstring-only, `raise NotImplementedError`, a same-name `super()` pass-through, or any `@abstractmethod`.
 
+src/wexample_wex_addon_dev_python/selection/package_description_missing_selection.py selects the package's own `.wex/config.yml`, and only when `global.description` is empty. Selecting the config file rather than the package makes the item what is to be edited, so the agent writes that one field and leaves the rest of the file alone. The description is copied verbatim into `pyproject.toml` and shown on the package registry.
+
+This replaced a filestate operation that called the agent from inside `apply_operation`. Writing a description is a job given to an agent, not a file state to converge on: rectification retried it every pass, and an agent declining to invent a description for a package with no code kept the hole open, which kept the retry going.
+
 ### Supporting pieces
 
 src/wexample_wex_addon_dev_python/common/pypi_registry_gateway.py is a thin HTTP client over the PEP 503 Simple index. It accepts optional Basic auth (`username` + `token`) and exposes `has_version(package, version) -> bool`.
 
-src/wexample_wex_addon_dev_python/operation/generated_description_operation.py is a filestate operation that invokes a `ClaudeAgent` to produce a one-sentence `global.description` and writes it to the app config. The model is called only during `apply_operation`, not during dry-run, so a preview is always offline. Failures (network, model, length over 200 chars) leave the key unset and log a warning rather than aborting the rectification.
-
 src/wexample_wex_addon_dev_python/services/python/app_service.py contributes the initial `pyproject.toml` and `src/app/` scaffolding when a new Python app service is set up.
 
 src/wexample_wex_addon_dev_python/const/tags.py declares the `DomainTag` constants (`domain:format`, `domain:language-python`, `domain:lint`, `domain:service`) used in command `@command(tags=[…])` decorators.
@@ -227,9 +229,9 @@ Visit the [Wexample Suite documentation](https://docs.wexample.com) for the comp
 - pylint: 
 - pyright: 
 - wexample-api: >=6.8.0
-- wexample-filestate-python: >=9.0.0
-- wexample-wex-addon-ai: >=15.0.0
-- wexample-wex-addon-app: >=31.1.0
+- wexample-filestate-python: >=9.1.0
+- wexample-wex-addon-ai: >=16.0.0
+- wexample-wex-addon-app: >=32.0.0
 
 ## Versioning & Compatibility Policy
 

{wexample_wex_addon_dev_python-13.3.1 → wexample_wex_addon_dev_python-15.0.0}/pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "pdm.backend"
 
 [project]
 name = "wexample-wex-addon-dev-python"
-version = "13.3.1"
+version = "15.0.0"
 description = "Extends wex with commands to format, lint, and rename Python symbols, plus workdir types for Python packages and suites"
 authors = [
     { name = "weeger", email = "contact@wexample.com" },
@@ -27,9 +27,9 @@ dependencies = [
     "pylint",
     "pyright",
     "wexample-api>=6.8.0",
-    "wexample-filestate-python>=9.0.0",
-    "wexample-wex-addon-ai>=15.0.0",
-    "wexample-wex-addon-app>=31.1.0",
+    "wexample-filestate-python>=9.1.0",
+    "wexample-wex-addon-ai>=16.0.0",
+    "wexample-wex-addon-app>=32.0.0",
 ]
 
 [project.readme]

{wexample_wex_addon_dev_python-13.3.1 → wexample_wex_addon_dev_python-15.0.0}/src/wexample_wex_addon_dev_python/python_addon_manager.py

@@ -2,23 +2,22 @@ from __future__ import annotations
 
 from typing import TYPE_CHECKING
 
+from wexample_wex_addon_ai.agent.agent_contributing_addon_mixin import (
+    AgentContributingAddonMixin,
+)
 from wexample_wex_addon_ai.formatter.formatter_contributing_addon_mixin import (
     FormatterContributingAddonMixin,
 )
-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.formatter.abstract_formatter import AbstractFormatter
-    from wexample_wex_addon_ai.selection.abstract_selection import AbstractSelection
 
 
 class PythonAddonManager(
+    AgentContributingAddonMixin,
     FormatterContributingAddonMixin,
-    SelectionContributingAddonMixin,
     AbstractAddonManager,
 ):
     def get_formatter_classes(self) -> list[type[AbstractFormatter]]:
@@ -52,12 +51,15 @@ class PythonAddonManager(
             EachPythonFileMiddleware,
         ]
 
-    def get_selection_classes(self) -> list[type[AbstractSelection]]:
+    def get_process_selection_classes(self) -> list[type]:
+        from wexample_wex_addon_dev_python.selection.package_description_missing_selection import (
+            PackageDescriptionMissingSelection,
+        )
         from wexample_wex_addon_dev_python.selection.python_code_performance_selection import (
             PythonCodePerformanceSelection,
         )
 
-        return [PythonCodePerformanceSelection]
+        return [PackageDescriptionMissingSelection, PythonCodePerformanceSelection]
 
     def get_workdir_types(self) -> dict[str, type]:
         from wexample_wex_addon_dev_python.workdir.python_package_workdir import (

{wexample_wex_addon_dev_python-13.3.1 → wexample_wex_addon_dev_python-15.0.0}/src/wexample_wex_addon_dev_python/resources/ai/agents/describer/agent.yml

@@ -6,3 +6,4 @@ allowed_tools:
   - Read
   - Glob
   - Grep
+  - Edit

{wexample_wex_addon_dev_python-13.3.1 → wexample_wex_addon_dev_python-15.0.0}/src/wexample_wex_addon_dev_python/resources/ai/agents/describer/context.j2

@@ -1,7 +1,7 @@
 ## Domain: describing one Python package in one sentence
 
-You are given a package directory. You read it and answer with a single sentence saying
-what the package is for. That sentence lands in `global.description`, from which it is
+You are given a package directory. You read it and write a single sentence saying what
+the package is for. That sentence lands in `global.description`, from which it is
 copied verbatim into `pyproject.toml` and shown on the package registry.
 
 ### Read before you write
@@ -28,9 +28,12 @@ file tree and rectifies the disk to match it" beats "provides file management ut
 
 ### Answering
 
-The sentence goes into `report_value`, and only what you pass there is written to the
-config file. Text you type outside that call is read by nobody.
+The target is the package's `.wex/config.yml`. When you are asked to do the work, set
+`global.description` there to the sentence — that one field, nothing else in the file
+touched — then report it as a change (`described the package`). When you are asked to
+change nothing, report the missing description as a finding (`description_missing`) and
+the sentence you would write as a change.
 
 If the package is too empty to describe honestly — no source, no README, nothing but
-scaffolding — end your turn without calling `report_value`. The host writes nothing and
-leaves the field for a human. Guessing is the one failure that matters here.
+scaffolding — write nothing and report it as a finding, code `nothing_to_describe`. The
+field stays for a human. Guessing is the one failure that matters here.

{wexample_wex_addon_dev_python-13.3.1 → wexample_wex_addon_dev_python-15.0.0}/src/wexample_wex_addon_dev_python/selection/abstract_python_code_selection.py

@@ -2,13 +2,11 @@ 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 (
+from wexample_wex_addon_process.selection.abstract_code_from_directory_selection import (
     AbstractCodeFromDirectorySelection,
 )
 
 
-@base_class
 class AbstractPythonCodeSelection(AbstractCodeFromDirectorySelection):
     """Shared base for every Python-source Selection (perf, tests, …).
 
@@ -23,7 +21,7 @@ class AbstractPythonCodeSelection(AbstractCodeFromDirectorySelection):
     - 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
+    Concrete subclasses just pick a `name` and, if needed, extend
     `get_excluded_path_fragments()` / `get_excluded_file_names()` with their
     use-case specifics via `super() + (…)`.
     """

wexample_wex_addon_dev_python-15.0.0/src/wexample_wex_addon_dev_python/selection/package_description_missing_selection.py

@@ -0,0 +1,40 @@
+from __future__ import annotations
+
+from typing import TYPE_CHECKING, ClassVar
+
+from wexample_wex_addon_process.selection.abstract_process_selection import (
+    AbstractProcessSelection,
+)
+
+if TYPE_CHECKING:
+    from pathlib import Path
+
+
+class PackageDescriptionMissingSelection(AbstractProcessSelection):
+    """The package's `.wex/config.yml`, when its `global.description` is empty.
+
+    The description is copied verbatim into `pyproject.toml` and shown on the
+    package registry. Selecting the config file itself makes the item what is to be
+    edited: the agent writes the one field, and nothing else in the file.
+    """
+
+    label: ClassVar[str] = "Python package without a description"
+    name: ClassVar[str] = "package-description-missing"
+
+    def instructions(self, workdir: Path, item: Path) -> str:
+        return (
+            f"Package directory: {workdir}\n"
+            f"Write the description as `global.description` in this file, and change "
+            f"nothing else in it."
+        )
+
+    def items(self, workdir: Path) -> list[Path]:
+        import yaml
+        from wexample_app.const.globals import WORKDIR_SETUP_DIR
+
+        config = workdir / WORKDIR_SETUP_DIR / "config.yml"
+        if not config.is_file():
+            return []
+        values = yaml.safe_load(config.read_text(encoding="utf-8")) or {}
+        description = (values.get("global") or {}).get("description")
+        return [] if description else [config]

{wexample_wex_addon_dev_python-13.3.1 → wexample_wex_addon_dev_python-15.0.0}/src/wexample_wex_addon_dev_python/selection/python_code_performance_selection.py

@@ -3,8 +3,6 @@ from __future__ import annotations
 import ast
 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,
 )
@@ -17,7 +15,6 @@ if TYPE_CHECKING:
 _FUNC_TYPES: tuple[type, ...] = (ast.FunctionDef, ast.AsyncFunctionDef)
 
 
-@base_class
 class PythonCodePerformanceSelection(AbstractPythonCodeSelection):
     """Python source files candidate for the performance sweep.
 
@@ -27,7 +24,8 @@ class PythonCodePerformanceSelection(AbstractPythonCodeSelection):
     `vendor/`), extend the inherited methods via `super() + (…)`.
     """
 
-    SELECTION_NAME: ClassVar[str] = "python-performance"
+    label: ClassVar[str] = "Python code worth optimizing"
+    name: ClassVar[str] = "python-performance"
 
     def get_required_path_fragments(self) -> tuple[str, ...]:
         """Only library code under `/src/` — the modern Python layout for any

{wexample_wex_addon_dev_python-13.3.1 → wexample_wex_addon_dev_python-15.0.0}/src/wexample_wex_addon_dev_python/workdir/python_package_workdir.py

@@ -27,6 +27,16 @@ if TYPE_CHECKING:
 class PythonPackageWorkdir(PythonWorkdir):
     _project_info_cache = None
 
+    @staticmethod
+    def path_belongs_to_tests(package_path: Path, file_path: Path) -> bool:
+        """Whether a file lives under the package's own `tests/` directory.
+
+        Only that one directory, at the package root. A `testing/` package under `src/`
+        is shipped code — other packages import it to build their own suites — and has
+        every reason to be judged like the rest.
+        """
+        return (package_path / "tests") in file_path.parents
+
     def check_publish_prerequisites(self) -> None:
         import shutil
 
@@ -162,6 +172,13 @@ class PythonPackageWorkdir(PythonWorkdir):
         - import <pkg>(.<sub>)* [as alias]
 
         Returns a list of SearchResult with file, line and column for each match.
+
+        `tests/` is left out. What this feeds is the runtime dependency graph — what a
+        consumer installing the package receives — and the test suite is not part of it:
+        setuptools does not ship it and coverage omits it. A package a test imports
+        belongs in the `dev` extra, which this graph is right not to know about; judging
+        a test import against `project.dependencies` asks a correctly declared package
+        to declare it twice, in the wrong place.
         """
         import re
 
@@ -173,7 +190,14 @@ class PythonPackageWorkdir(PythonWorkdir):
             rf"|import\s+{escaped}(?:\.[\w\.]+)?(?:\s+as\s+\w+)?\b"
             rf")"
         )
-        return self.search_in_codebase(pattern, regex=True, flags=re.MULTILINE)
+        results = self.search_in_codebase(pattern, regex=True, flags=re.MULTILINE)
+        package_path = self.get_path()
+
+        return [
+            result
+            for result in results
+            if not self.path_belongs_to_tests(package_path, result.item.get_path())
+        ]
 
     def search_in_codebase(
         self, string: str, *, regex: bool = False, flags: int = 0

{wexample_wex_addon_dev_python-13.3.1 → wexample_wex_addon_dev_python-15.0.0}/src/wexample_wex_addon_dev_python/workdir/python_workdir.py

@@ -118,14 +118,9 @@ class PythonWorkdir(
             PythonOptionsProvider,
         )
 
-        from wexample_wex_addon_dev_python.options_provider.generative_options_provider import (
-            GenerativeOptionsProvider,
-        )
-
         options = super().get_options_providers()
 
         options.append(PythonOptionsProvider)
-        options.append(GenerativeOptionsProvider)
 
         return options
 
@@ -212,16 +207,11 @@ class PythonWorkdir(
         from wexample_wex_addon_dev_python.file.python_pyproject_toml_file import (
             PythonPyprojectTomlFile,
         )
-        from wexample_wex_addon_dev_python.option.generated_description_option import (
-            GeneratedDescriptionOption,
-        )
 
         raw_value = super().prepare_value(raw_value=raw_value)
 
         self.append_agents(config=raw_value)
 
-        raw_value[GeneratedDescriptionOption.get_name()] = True
-
         children = raw_value["children"]
 
         self.add_gitignore_rules(

wexample_wex_addon_dev_python-15.0.0/tests/unit/workdir/test_python_package_workdir.py

@@ -0,0 +1,53 @@
+from __future__ import annotations
+
+from pathlib import Path
+
+import pytest
+
+PACKAGE = Path("/suite/packages/storage")
+
+
+def _belongs_to_tests(file_path: str) -> bool:
+    from wexample_wex_addon_dev_python.workdir.python_package_workdir import (
+        PythonPackageWorkdir,
+    )
+
+    return PythonPackageWorkdir.path_belongs_to_tests(PACKAGE, Path(file_path))
+
+
+def test_the_suite_that_set_this_off_is_out_of_scope() -> None:
+    """storage's test imports wexample-wallet, declared where it belongs: the dev extra.
+
+    Judging it against `project.dependencies` asked a correct package to declare the
+    same thing twice, once in the wrong place.
+    """
+    assert _belongs_to_tests("/suite/packages/storage/tests/test_storage.py")
+
+
+@pytest.mark.parametrize(
+    "path",
+    [
+        "/suite/packages/storage/tests/unit/deep/test_thing.py",
+        "/suite/packages/storage/tests/conftest.py",
+    ],
+)
+def test_anything_under_tests_is_out_of_scope(path: str) -> None:
+    assert _belongs_to_tests(path)
+
+
+@pytest.mark.parametrize(
+    "path",
+    [
+        "/suite/packages/storage/src/wexample_storage/common/storage.py",
+        # Shipped, and imported by other packages to build their own suites.
+        "/suite/packages/storage/src/wexample_storage/testing/abstract_test.py",
+        "/suite/packages/storage/src/wexample_storage/tests_helper.py",
+    ],
+)
+def test_shipped_code_stays_in_scope(path: str) -> None:
+    assert not _belongs_to_tests(path)
+
+
+def test_another_package_tests_directory_is_not_ours() -> None:
+    """The check is anchored on this package's root, not on the name `tests`."""
+    assert not _belongs_to_tests("/suite/packages/wallet/tests/test_wallet.py")

wexample_wex_addon_dev_python-13.3.1/src/wexample_wex_addon_dev_python/operation/generated_description_operation.py

@@ -1,127 +0,0 @@
-from __future__ import annotations
-
-from typing import TYPE_CHECKING
-
-from wexample_filestate.enum.scopes import Scope
-from wexample_filestate.operation.abstract_operation import AbstractOperation
-from wexample_helpers.classes.mixin.has_package_resources import HasPackageResources
-from wexample_helpers.classes.private_field import private_field
-from wexample_helpers.decorator.base_class import base_class
-
-if TYPE_CHECKING:
-    from pathlib import Path
-
-CONFIG_KEY_SECTION = "global"
-CONFIG_KEY_NAME = "description"
-DESCRIPTION_MAX_LENGTH = 200
-DESCRIPTION_PLACEHOLDER = "<one sentence, written by the describer agent on apply>"
-DESCRIPTION_UNKNOWN = "UNKNOWN"
-
-
-@base_class
-class GeneratedDescriptionOperation(HasPackageResources, AbstractOperation):
-    """Write `global.description` into the app config, asking an agent for the sentence.
-
-    The model call lives here rather than in the option that builds this operation:
-    `FileStateDryRunResult` never calls `apply_operation`, so a preview stays offline and
-    free, and shows the placeholder instead of a sentence it would then have to produce a
-    second time on apply.
-    """
-
-    _previous: str | None = private_field(
-        default=None,
-        description="Value this operation replaced, for `undo`. None when the key was absent.",
-    )
-
-    @classmethod
-    def get_scopes(cls) -> list[Scope]:
-        return [Scope.CONTENT]
-
-    def apply_operation(self) -> None:
-        description = self._generate()
-        if description is None:
-            return
-
-        config_file = self.target.get_config_file()
-        content = config_file.read_parsed() or {}
-        section = content.setdefault(CONFIG_KEY_SECTION, {})
-
-        self._previous = section.get(CONFIG_KEY_NAME)
-        section[CONFIG_KEY_NAME] = description
-        config_file.write_parsed(content)
-
-        self.target.success(f"Description: {description}")
-
-    def undo(self) -> None:
-        config_file = self.target.get_config_file()
-        content = config_file.read_parsed() or {}
-        section = content.get(CONFIG_KEY_SECTION)
-        if not isinstance(section, dict):
-            return
-
-        if self._previous is None:
-            section.pop(CONFIG_KEY_NAME, None)
-        else:
-            section[CONFIG_KEY_NAME] = self._previous
-        config_file.write_parsed(content)
-
-    def _agent_config_path(self) -> Path:
-        return self.get_resource_path("ai", "agents", "describer", "agent.yml")
-
-    def _generate(self) -> str | None:
-        """The sentence, or None when nothing should be written.
-
-        Every failure mode lands on None: the hole stays, the pass reports it and moves on.
-        A rectification must not fail because a model was unreachable.
-        """
-        from wexample_wex_addon_ai.agent.claude_agent import ClaudeAgent
-        from wexample_wex_addon_ai.file.agent_yaml_file import AgentYamlFile
-
-        try:
-            agent = ClaudeAgent(
-                io=self.target.io,
-                app_workdir=self.target,
-                config_file=AgentYamlFile.create_from_path(
-                    path=self._agent_config_path()
-                ),
-            )
-            sentence = agent.run_value_draft(self._prompt())
-        except Exception as exception:
-            self.target.warning(
-                f"Could not generate a description: {exception}. "
-                f"Leaving {CONFIG_KEY_SECTION}.{CONFIG_KEY_NAME} unset."
-            )
-            return None
-
-        return self._validate(sentence)
-
-    def _prompt(self) -> str:
-        return (
-            f"Package directory: {self.target.get_path()}\n"
-            f"Package name: {self.target.get_package_name()}\n"
-            f"\n"
-            f"Read the package and report its one-sentence description."
-        )
-
-    def _validate(self, sentence: str | None) -> str | None:
-        """Guard the one shape that would still poison a config file: a runaway sentence.
-
-        `report_value` already delimits the answer, so nothing has to be extracted from the
-        agent's prose — a missing report is simply `None`.
-        """
-        if sentence is None:
-            self.target.warning(
-                f"The describer reported nothing — "
-                f"leaving {CONFIG_KEY_SECTION}.{CONFIG_KEY_NAME} unset."
-            )
-            return None
-
-        if len(sentence) > DESCRIPTION_MAX_LENGTH:
-            self.target.warning(
-                f"The describer answered {len(sentence)} characters, over the "
-                f"{DESCRIPTION_MAX_LENGTH} allowed — leaving "
-                f"{CONFIG_KEY_SECTION}.{CONFIG_KEY_NAME} unset."
-            )
-            return None
-
-        return sentence

wexample_wex_addon_dev_python-13.3.1/src/wexample_wex_addon_dev_python/option/generated_description_option.py

@@ -1,77 +0,0 @@
-from __future__ import annotations
-
-from collections.abc import Callable
-from typing import TYPE_CHECKING, Any, Union
-
-from wexample_config.config_option.abstract_config_option import AbstractConfigOption
-from wexample_filestate.enum.scopes import Scope
-from wexample_filestate.option.mixin.option_mixin import OptionMixin
-from wexample_helpers.decorator.base_class import base_class
-
-if TYPE_CHECKING:
-    from wexample_filestate.const.types_state_items import TargetFileOrDirectoryType
-    from wexample_filestate.operation.abstract_operation import AbstractOperation
-
-
-@base_class
-class GeneratedDescriptionOption(OptionMixin, AbstractConfigOption):
-    """Ask an agent for `global.description` when the app config has none.
-
-    Only ever fills a hole: a description already written — by a human or by an earlier
-    run — is left alone, which is what makes the option convergent. It fires once per
-    package, then never again. To have one rewritten, clear the key.
-
-    Declared on the workdir, not on `pyproject.toml`: the toml's `description` is already
-    derived from this key by `PythonPyprojectTomlFile._enforce_project_metadata`, so
-    filling the config is enough for the value to reach the manifest through the ordinary
-    deterministic path.
-    """
-
-    @classmethod
-    def get_scopes(cls) -> list[Scope]:
-        return [Scope.CONTENT]
-
-    @staticmethod
-    def get_raw_value_allowed_type() -> Any:
-        return Union[bool, Callable]
-
-    def applicable_on_file(self) -> bool:
-        return False
-
-    def applicable_on_missing(self) -> bool:
-        return False
-
-    def create_required_operation(
-        self, target: TargetFileOrDirectoryType, scopes: set[Scope]
-    ) -> AbstractOperation | None:
-        from wexample_wex_addon_dev_python.operation.generated_description_operation import (
-            CONFIG_KEY_NAME,
-            CONFIG_KEY_SECTION,
-            DESCRIPTION_PLACEHOLDER,
-            GeneratedDescriptionOperation,
-        )
-
-        value = self.get_value()
-        raw = value.raw if value is not None else None
-        if callable(raw):
-            raw = raw(target)
-        if not bool(raw):
-            return None
-
-        existing = target.get_config().search(
-            f"{CONFIG_KEY_SECTION}.{CONFIG_KEY_NAME}"
-        )
-        if not existing.is_none() and existing.get_str().strip():
-            return None
-
-        return GeneratedDescriptionOperation(
-            option=self,
-            target=target,
-            description=(
-                f"Write {CONFIG_KEY_SECTION}.{CONFIG_KEY_NAME}: "
-                f"{DESCRIPTION_PLACEHOLDER}"
-            ),
-        )
-
-    def get_description(self) -> str:
-        return "Have an agent write the app description when none is set"

wexample_wex_addon_dev_python-13.3.1/src/wexample_wex_addon_dev_python/options_provider/generative_options_provider.py

@@ -1,26 +0,0 @@
-from __future__ import annotations
-
-from typing import TYPE_CHECKING
-
-from wexample_config.options_provider.abstract_options_provider import (
-    AbstractOptionsProvider,
-)
-
-if TYPE_CHECKING:
-    from wexample_config.config_option.abstract_config_option import (
-        AbstractConfigOption,
-    )
-
-
-class GenerativeOptionsProvider(AbstractOptionsProvider):
-    @classmethod
-    def get_options(cls) -> list[type[AbstractConfigOption]]:
-        if "_options_cache" not in cls.__dict__:
-            from wexample_wex_addon_dev_python.option.generated_description_option import (
-                GeneratedDescriptionOption,
-            )
-
-            cls._options_cache: list[type[AbstractConfigOption]] = [
-                GeneratedDescriptionOption
-            ]
-        return cls._options_cache