refact: rename and cleanup

Author: sfermigier

packages/hop3-server/src/hop3/core/plugins.py

@@ -11,12 +11,7 @@
 from devtools import debug
 from pluggy import PluginManager
 
-# Temp
-from hop3.plugins.build.dummy_build.builder import DummyBuilder
-from hop3.plugins.deploy.dummy_deploy.deploy import DummyDeployer
-
 from . import hookspecs
-from .hooks import hop3_hook_impl
 
 if TYPE_CHECKING:
     from collections.abc import Iterator
@@ -99,10 +94,6 @@ def get_plugin_manager() -> PluginManager:
 
     pm.add_hookspecs(hookspecs)
 
-    # Register the core plugin first (provides dummy strategies)
-    core_plugin = CorePlugin()
-    pm.register(core_plugin)
-
     # Import all plugin modules and auto-discover plugin instances
     #
     # Plugin Architecture Notes:
@@ -136,21 +127,6 @@ def get_plugin_manager() -> PluginManager:
     return pm
 
 
-class CorePlugin:
-    """The plugin container for Hop3's default strategies."""
-
-    name = "core"
-
-    @hop3_hook_impl
-    def get_builders(self) -> list:
-        # This hook returns classes, not instances.
-        return [DummyBuilder]
-
-    @hop3_hook_impl
-    def get_deployers(self) -> list:
-        return [DummyDeployer]
-
-
 #
 # Convenience Helper Functions
 #

packages/hop3-server/src/hop3/plugins/build/dummy_build/__init__.py

@@ -1,7 +1,10 @@
 # Copyright (c) 2025, Abilian SAS
 #
 # SPDX-License-Identifier: Apache-2.0
-"""Native build plugin package.
+"""Language toolchains plugin package.
+
+Provides Level 2 toolchains (PythonToolchain, NodeToolchain, etc.)
+that are used by LocalBuilder to build applications.
 
 The plugin instance is exported from plugin.py, not from this __init__.py.
 This avoids duplicate registration during package discovery, since

packages/hop3-server/src/hop3/plugins/build/dummy_build/builder.py

@@ -9,14 +9,14 @@
 from hop3.toolchains import TOOLCHAIN_CLASSES
 
 
-class NativeBuildPlugin:
+class LanguageToolchainsPlugin:
     """Plugin that provides language toolchains for various languages.
 
     This plugin provides Level 2 toolchains (PythonToolchain, NodeToolchain, etc.)
     that are used by LocalBuilder to build applications.
     """
 
-    name = "native-build"
+    name = "language-toolchains"
 
     @hop3_hook_impl
     def get_language_toolchains(self) -> list:
@@ -29,4 +29,4 @@ def get_language_toolchains(self) -> list:
 
 
 # Auto-register plugin instance when module is imported
-plugin = NativeBuildPlugin()
+plugin = LanguageToolchainsPlugin()

…3/plugins/build/native_build/__init__.py …ns/build/language_toolchains/__init__.pypackages/hop3-server/src/hop3/plugins/build/native_build/__init__.py renamed to packages/hop3-server/src/hop3/plugins/build/language_toolchains/__init__.py packages/hop3-server/src/hop3/plugins/build/native_build/__init__.py renamed to packages/hop3-server/src/hop3/plugins/build/language_toolchains/__init__.py

@@ -52,36 +52,16 @@ def test_get_builder_with_node_project(tmp_path: Path):
     assert builder.context == context
 
 
-def test_get_builder_no_suitable_builder(tmp_path: Path):
-    """Test that get_builder uses DummyBuilder when .dummy-build marker exists."""
-    # Create source directory without any recognized project files
-    src_dir = tmp_path / "src"
-    src_dir.mkdir()
-    (src_dir / "README.md").write_text("# Test Project\n")
-
-    # Create .dummy-build marker file to explicitly request DummyBuilder
-    # (DummyBuilder only accepts if this marker exists)
-    (src_dir / ".dummy-build").touch()
-
-    # Create DeploymentContext
-    context = DeploymentContext(app_name="test-app", source_path=src_dir, app_config={})
-
-    # With .dummy-build marker, DummyBuilder should accept
-    builder = get_builder(context)
-    assert builder.name == "dummy"
-
-
 def test_get_builder_no_builder_raises_error(tmp_path: Path):
     """Test that get_builder raises RuntimeError when no builder accepts."""
     # Create source directory without any recognized project files
-    # and WITHOUT .dummy-build marker
     src_dir = tmp_path / "src"
     src_dir.mkdir()
     (src_dir / "README.md").write_text("# Test Project\n")
 
     # Create DeploymentContext
     context = DeploymentContext(app_name="test-app", source_path=src_dir, app_config={})
 
-    # Without any recognized files or .dummy-build marker, should raise RuntimeError
+    # Without any recognized files, should raise RuntimeError
     with pytest.raises(RuntimeError, match="Could not find a suitable builder"):
         get_builder(context)

…op3/plugins/build/native_build/plugin.py …gins/build/language_toolchains/plugin.pypackages/hop3-server/src/hop3/plugins/build/native_build/plugin.py renamed to packages/hop3-server/src/hop3/plugins/build/language_toolchains/plugin.py packages/hop3-server/src/hop3/plugins/build/native_build/plugin.py renamed to packages/hop3-server/src/hop3/plugins/build/language_toolchains/plugin.py

@@ -80,7 +80,7 @@ def test_uwsgi_deployer_has_app_access(tmp_path: Path):
 
 
 def test_deployment_strategy_priority(tmp_path: Path):
-    """Test that UWSGIDeployer is selected before DummyDeployer."""
+    """Test that UWSGIDeployer is selected for various artifact types."""
     app = Mock(spec=App)
     app.name = "test-app"
 

packages/hop3-server/src/hop3/plugins/deploy/dummy_deploy/__init__.py

@@ -109,7 +109,7 @@ def test_uwsgi_deployer_has_app_access(self, tmp_path: Path):
         assert deployer.app.name == "test-app"
 
     def test_deployment_strategy_priority_for_multiple_artifacts(self, tmp_path: Path):
-        """Test that UWSGIDeployer is selected before DummyDeployer.
+        """Test that UWSGIDeployer is selected for various artifact types.
 
         ARRANGE:
             - Create real App instance