Fix typing of action runner. Add 'build artifact action'. Move 'FormatFilesSave' handler in builtin handlers. Add 'get_actions_by_source' and 'get_actions_for_language' methods in IActionRunner. Add 'current_result' in RunActionContext. Improve terms: declaration and definiton of action and handlers.

Author: Aksem

.vscode/tasks.json

@@ -188,7 +188,7 @@ async def run_on_single_file(
     async def run(
         self,
         payload: lint_files_action.LintFilesRunPayload,
-        run_context: code_action.RunActionWithPartialResultsContext,
+        run_context: lint_files_action.LintFilesRunContext,
     ) -> None:
         if self.config.select is not None and len(self.config.select) == 0:
             # empty set of rules is selected, no need to run flake8

extensions/fine_python_flake8/fine_python_flake8/action.py

@@ -195,7 +195,7 @@ async def _run_dmypy_on_project(
     async def run(
         self,
         payload: lint_action.LintRunPayload,
-        run_context: code_action.RunActionWithPartialResultsContext,
+        run_context: lint_action.LintRunContext,
     ) -> None:
         file_paths = [file_path async for file_path in payload]
 

extensions/fine_python_mypy/fine_python_mypy/action.py

@@ -0,0 +1,95 @@
+import dataclasses
+
+from finecode_extension_api import code_action
+from finecode_extension_api.actions import build_artifact_action
+from finecode_extension_api.interfaces import (
+    icommandrunner,
+    iextensionrunnerinfoprovider,
+    ilogger,
+    iprojectinfoprovider,
+)
+
+
+@dataclasses.dataclass
+class BuildArtifactPyHandlerConfig(code_action.ActionHandlerConfig): ...
+
+
+class BuildArtifactPyHandler(
+    code_action.ActionHandler[
+        build_artifact_action.BuildArtifactAction,
+        BuildArtifactPyHandlerConfig,
+    ]
+):
+    def __init__(
+        self,
+        config: BuildArtifactPyHandlerConfig,
+        command_runner: icommandrunner.ICommandRunner,
+        project_info_provider: iprojectinfoprovider.IProjectInfoProvider,
+        extension_runner_info_provider: iextensionrunnerinfoprovider.IExtensionRunnerInfoProvider,
+        logger: ilogger.ILogger,
+    ) -> None:
+        self.config = config
+        self.command_runner = command_runner
+        self.project_info_provider = project_info_provider
+        self.extension_runner_info_provider = extension_runner_info_provider
+        self.logger = logger
+
+    async def run(
+        self,
+        payload: build_artifact_action.BuildArtifactRunPayload,
+        run_context: build_artifact_action.BuildArtifactRunContext,
+    ) -> build_artifact_action.BuildArtifactRunResult:
+        # Use current project if src_artifact_def_path is not provided
+        src_artifact_def_path = payload.src_artifact_def_path
+        if src_artifact_def_path is None:
+            src_artifact_def_path = (
+                self.project_info_provider.get_current_project_def_path()
+            )
+
+        # Get the project directory (parent of pyproject.toml)
+        project_dir = src_artifact_def_path.parent
+
+        self.logger.info(f"Building artifact in {project_dir}")
+
+        # Get the python interpreter from the current venv
+        venv_dir = self.extension_runner_info_provider.get_current_venv_dir_path()
+        python_path = self.extension_runner_info_provider.get_venv_python_interpreter(
+            venv_dir
+        )
+
+        # Run python -m build
+        process = await self.command_runner.run(
+            cmd=f"{python_path} -m build",
+            cwd=project_dir,
+        )
+        await process.wait_for_end()
+
+        exit_code = process.get_exit_code()
+        if exit_code != 0:
+            error_output = process.get_error_output()
+            raise code_action.ActionFailedException(
+                f"Build failed with exit code {exit_code}: {error_output}"
+            )
+
+        # Parse the build output to get the produced file names
+        # Example line: "Successfully built pkg-1.0.tar.gz and pkg-1.0-py3-none-any.whl"
+        dist_dir = project_dir / "dist"
+        build_output_paths = []
+
+        output = process.get_output()
+        for line in output.splitlines():
+            if line.startswith("Successfully built "):
+                files_part = line[len("Successfully built ") :]
+                file_names = [f.strip() for f in files_part.split(" and ")]
+                build_output_paths = [dist_dir / name for name in file_names]
+                break
+
+        if not build_output_paths:
+            # Fallback: return the dist directory if parsing failed
+            build_output_paths = [dist_dir]
+
+        self.logger.info(f"Build completed. Output: {build_output_paths}")
+
+        return build_artifact_action.BuildArtifactRunResult(
+            build_output_paths=build_output_paths
+        )

extensions/fine_python_package_info/fine_python_package_info/build_artifact_py_handler.py

@@ -65,7 +65,7 @@ async def run(
 
         # Get registries using the action
         get_registries_action = self.action_runner.get_action_by_name(
-            "get_src_artifact_registries"
+            "get_src_artifact_registries", get_src_artifact_registries_action.GetSrcArtifactRegistriesAction
         )
         registries_payload = (
             get_src_artifact_registries_action.GetSrcArtifactRegistriesRunPayload(
@@ -127,7 +127,7 @@ async def run(
             except KeyError as exception:
                 raise code_action.ActionFailedException("File object has no 'filename' key") from exception
 
-            is_published_by_dist_path = {dist_path: dist_path in published_file_names for dist_path in dist_artifact_paths}
+            is_published_by_dist_path = {dist_path: dist_path.name in published_file_names for dist_path in dist_artifact_paths}
         else:
             is_published_by_dist_path = {dist_path: False for dist_path in dist_artifact_paths}
 

extensions/fine_python_package_info/fine_python_package_info/is_artifact_published_to_registry_py_handler.py

@@ -9,6 +9,7 @@ dependencies = [
     "finecode_extension_api~=0.4.0a0",
     "tomlkit==0.11.*",
     "twine==6.2.*",
+    "build==1.4.*",
 ]
 
 [dependency-groups]

extensions/fine_python_package_info/pyproject.toml

@@ -87,7 +87,7 @@ async def run_on_single_file(
     async def run(
         self,
         payload: lint_files_action.LintFilesRunPayload,
-        run_context: code_action.RunActionWithPartialResultsContext,
+        run_context: lint_files_action.LintFilesRunContext,
     ) -> None:
         file_paths = [file_path async for file_path in payload]
 

extensions/fine_python_pyrefly/fine_python_pyrefly/lint_files_handler.py

@@ -80,7 +80,7 @@ async def run_on_single_file(
     async def run(
         self,
         payload: lint_files_action.LintFilesRunPayload,
-        run_context: code_action.RunActionWithPartialResultsContext,
+        run_context: lint_files_action.LintFilesRunContext,
     ) -> None:
         file_paths = [file_path async for file_path in payload]
 

extensions/fine_python_ruff/fine_python_ruff/lint_files_handler.py

@@ -4,6 +4,7 @@
 from .dump_config import DumpConfigHandler
 from .dump_config_save import DumpConfigSaveHandler
 from .format import FormatHandler
+from .format_files_save_handler import SaveFormatFilesHandler
 from .lint import LintHandler
 from .prepare_envs_install_deps import PrepareEnvsInstallDepsHandler
 from .prepare_envs_read_configs import PrepareEnvsReadConfigsHandler
@@ -24,4 +25,5 @@
     "PrepareRunnersReadConfigsHandler",
     "DumpConfigSaveHandler",
     "PublishArtifactHandler",
+    "SaveFormatFilesHandler"
 ]

finecode_builtin_handlers/src/finecode_builtin_handlers/__init__.py

@@ -73,7 +73,7 @@ async def run(
                 # or e.g. files changed according to git + dependencies.
                 files_to_format: list[pathlib.Path] = self.file_editor.get_opened_files()
                 group_project_files_action = self.action_runner.get_action_by_name(
-                    "group_src_artifact_files_by_lang"
+                    "group_src_artifact_files_by_lang", group_src_artifact_files_by_lang_action.GroupSrcArtifactFilesByLangAction
                 )
                 group_src_artifact_files_by_lang_payload = group_src_artifact_files_by_lang_action.GroupSrcArtifactFilesByLangRunPayload(
                     file_paths=files_to_format, langs=langs_supported_by_format
@@ -90,7 +90,7 @@ async def run(
                 # language, use `list_src_artifact_files_by_lang_action` action which returns
                 # only files with supported languages
                 list_src_artifact_file_by_lang_action_instance = (
-                    self.action_runner.get_action_by_name("list_src_artifact_files_by_lang")
+                    self.action_runner.get_action_by_name("list_src_artifact_files_by_lang", list_src_artifact_files_by_lang_action.ListSrcArtifactFilesByLangAction)
                 )
                 list_src_artifact_files_by_lang_payload = (
                     list_src_artifact_files_by_lang_action.ListSrcArtifactFilesByLangRunPayload(
@@ -108,7 +108,7 @@ async def run(
             # format target are files, format them
             files_to_format = payload.file_paths
             group_src_artifact_files_by_lang_action_instance = (
-                self.action_runner.get_action_by_name("group_src_artifact_files_by_lang")
+                self.action_runner.get_action_by_name("group_src_artifact_files_by_lang", group_src_artifact_files_by_lang_action.GroupSrcArtifactFilesByLangAction)
             )
             group_src_artifact_files_by_lang_payload = (
                 group_src_artifact_files_by_lang_action.GroupSrcArtifactFilesByLangRunPayload(
@@ -130,7 +130,7 @@ async def run(
                     # TODO: handle errors
                     # TODO: handle KeyError?
                     action = self.action_runner.get_action_by_name(
-                        format_files_prefix + lang
+                        format_files_prefix + lang, format_files_action.FormatFilesAction
                     )
                     format_files_payload = format_files_action.FormatFilesRunPayload(
                         file_paths=lang_files, save=payload.save