Work toward alignment with Java implementation

Author: ctrueden

src/appose/builder/__init__.py

@@ -82,7 +82,7 @@ def __init__(
             verb = "interrupted" if isinstance(cause, KeyboardInterrupt) else "failed"
             message = f"{noun} {verb}"
         super().__init__(message)
-        self._builder: Builder | None = builder
+        self.builder: Builder | None = builder
         self.__cause__: Exception | None = cause
 
 
@@ -253,20 +253,6 @@ def file(self, path: str | Path) -> Builder:
         except Exception as e:
             raise BuildException(self, cause=e)
 
-    @abstractmethod
-    def content(self, content: str) -> Builder:
-        """
-        Specifies configuration file content to build from.
-        The scheme will be auto-detected from content syntax.
-
-        Args:
-            content: Configuration file content
-
-        Returns:
-            This builder instance, for fluent-style programming
-        """
-        ...
-
     def url(self, url: str) -> Builder:
         """
         Specifies a URL to fetch configuration content from.
@@ -288,6 +274,20 @@ def url(self, url: str) -> Builder:
         except Exception as e:
             raise BuildException(self, cause=e)
 
+    @abstractmethod
+    def content(self, content: str) -> Builder:
+        """
+        Specifies configuration file content to build from.
+        The scheme will be auto-detected from content syntax.
+
+        Args:
+            content: Configuration file content
+
+        Returns:
+            This builder instance, for fluent-style programming
+        """
+        ...
+
     @abstractmethod
     def scheme(self, scheme: str) -> Builder:
         """
@@ -605,9 +605,6 @@ def __init__(self):
         super().__init__()
         self._custom_bin_paths: list[str] = []
 
-    def env_type(self) -> str:
-        return "custom"
-
     def bin_paths(self, *paths: str) -> SimpleBuilder:
         """
         Appends additional binary paths to search for executables.
@@ -651,6 +648,9 @@ def inherit_running_java(self) -> SimpleBuilder:
             self._env_vars["JAVA_HOME"] = java_home
         return self
 
+    def env_type(self) -> str:
+        return "custom"
+
     def build(self) -> Environment:
         """Build the simple environment."""
         base = self._resolve_env_dir()

src/appose/builder/mamba.py

@@ -51,19 +51,6 @@ class MambaBuilder(BaseBuilder):
     def env_type(self) -> str:
         return "mamba"
 
-    def channels(self, *channels: str) -> MambaBuilder:
-        """
-        Adds conda channels to search for packages.
-
-        Args:
-            channels: Channel names (e.g., "conda-forge", "bioconda")
-
-        Returns:
-            This builder instance
-        """
-        super().channels(*channels)
-        return self
-
     def build(self) -> Environment:
         """
         Builds the Mamba environment.
@@ -95,7 +82,7 @@ def build(self) -> Environment:
         is_conda_dir = (env_dir / "conda-meta").is_dir()
         if is_conda_dir:
             # Environment already exists, just wrap it
-            return self._create_environment(env_dir, mamba)
+            return self._create_environment(mamba, env_dir)
 
         # Building a new environment - config content is required
         if self._content is None:
@@ -153,7 +140,7 @@ def build(self) -> Environment:
             # Step 3: Update environment from yml
             mamba.update(env_dir, env_yaml)
 
-            return self._create_environment(env_dir, mamba)
+            return self._create_environment(mamba, env_dir)
 
         except (IOError, KeyboardInterrupt) as e:
             raise BuildException(self, cause=e)
@@ -186,7 +173,7 @@ def wrap(self, env_dir: str | Path) -> Environment:
         self.base(env_path)
         return self.build()
 
-    def _create_environment(self, env_dir: Path, mamba: Mamba) -> Environment:
+    def _create_environment(self, mamba: Mamba, env_dir: Path) -> Environment:
         """
         Creates an Environment for the given Mamba/conda directory.
 

src/appose/builder/pixi.py

@@ -81,18 +81,6 @@ def pypi(self, *packages: str) -> PixiBuilder:
         self._pypi_packages.extend(packages)
         return self
 
-    def channels(self, *channels: str) -> PixiBuilder:
-        """
-        Adds conda channels to search for packages.
-
-        Args:
-            channels: Channel names (e.g., "conda-forge", "bioconda")
-
-        Returns:
-            This builder instance
-        """
-        return super().channels(*channels)
-
     def build(self) -> Environment:
         """
         Builds the Pixi environment.
@@ -153,7 +141,7 @@ def build(self) -> Environment:
                 and not self._pypi_packages
             ):
                 # Environment already exists, just use it
-                return self._create_environment(env_dir, pixi)
+                return self._create_environment(pixi, env_dir)
 
             # Handle source-based build (file or content)
             if self._content is not None:
@@ -197,7 +185,7 @@ def build(self) -> Environment:
                 # Programmatic package building
                 if is_pixi_dir:
                     # Already initialized, just use it
-                    return self._create_environment(env_dir, pixi)
+                    return self._create_environment(pixi, env_dir)
 
                 if not env_dir.exists():
                     env_dir.mkdir(parents=True, exist_ok=True)
@@ -239,7 +227,7 @@ def build(self) -> Environment:
                             'Add .conda("appose") or .pypi("appose") to your builder.',
                         )
 
-            return self._create_environment(env_dir, pixi)
+            return self._create_environment(pixi, env_dir)
 
         except (IOError, KeyboardInterrupt) as e:
             raise BuildException(self, cause=e)
@@ -281,7 +269,7 @@ def wrap(self, env_dir: str | Path) -> Environment:
         self.base(env_path)
         return self.build()
 
-    def _create_environment(self, env_dir: Path, pixi: Pixi) -> Environment:
+    def _create_environment(self, pixi: Pixi, env_dir: Path) -> Environment:
         """
         Creates an Environment for the given Pixi directory.
 

src/appose/builder/uv.py

@@ -54,9 +54,6 @@ def __init__(self):
         self._python_version: str | None = None
         self._packages: list[str] = []
 
-    def env_type(self) -> str:
-        return "uv"
-
     def python(self, version: str) -> UvBuilder:
         """
         Specifies the Python version to use for the virtual environment.
@@ -83,19 +80,8 @@ def include(self, *packages: str) -> UvBuilder:
         self._packages.extend(packages)
         return self
 
-    def channels(self, *indexes: str) -> UvBuilder:
-        """
-        Adds PyPI index URLs for package discovery.
-
-        These are alternative or additional package indexes beyond the default pypi.org.
-
-        Args:
-            indexes: Index URLs (e.g., custom PyPI mirrors or private package repositories)
-
-        Returns:
-            This builder instance
-        """
-        return super().channels(*indexes)
+    def env_type(self) -> str:
+        return "uv"
 
     def build(self) -> Environment:
         """

src/appose/service.py

@@ -57,16 +57,6 @@ def __init__(self, message: str, task: "Task") -> None:
         super().__init__(message)
         self.task: Task = task
 
-    @property
-    def status(self) -> "TaskStatus":
-        """Returns the status of the failed task."""
-        return self.task.status
-
-    @property
-    def task_error(self) -> str | None:
-        """Returns the error message from the task, if available."""
-        return self.task.error
-
 
 class Service:
     """

tests/test_task_exception.py

@@ -61,9 +61,9 @@ def test_task_exception_on_failure():
 
         # Verify we can access the task and its details through the exception
         assert e.task is task
-        assert e.status == TaskStatus.FAILED
-        assert e.task_error is not None
-        assert "NameError" in e.task_error or "nameerror" in e.task_error.lower()
+        assert e.task.status == TaskStatus.FAILED
+        assert e.task.error is not None
+        assert "NameError" in e.task.error or "nameerror" in e.task.error.lower()
 
 
 def test_no_exception_on_success():