refactor: add ctx:write_php_exec_wrapper utility (#409)

7 files changed, 25 insertions, 74 deletions

diff --git a/lua/mason-core/installer/context.lua b/lua/mason-core/installer/context.lua
index 9e1df72f..7d0a505b 100644
--- a/lua/mason-core/installer/context.lua
+++ b/lua/mason-core/installer/context.lua
@@ -216,6 +216,21 @@ function InstallContext:write_node_exec_wrapper(new_executable_rel_path, script_
 end
 
 ---@param new_executable_rel_path string Relative path to the executable file to create.
+---@param script_rel_path string Relative path to the PHP script.
+function InstallContext:write_php_exec_wrapper(new_executable_rel_path, script_rel_path)
+    if not self.fs:file_exists(script_rel_path) then
+        error(("Cannot write PHP exec wrapper for path %q as it doesn't exist."):format(script_rel_path), 0)
+    end
+    return self:write_shell_exec_wrapper(
+        new_executable_rel_path,
+        ("php %q"):format(path.concat {
+            self.package:get_install_path(),
+            script_rel_path,
+        })
+    )
+end
+
+---@param new_executable_rel_path string Relative path to the executable file to create.
 ---@param module string The python module to call.
 function InstallContext:write_pyvenv_exec_wrapper(new_executable_rel_path, module)
     local pip3 = require "mason-core.managers.pip3"
diff --git a/lua/mason-registry/php-cs-fixer/init.lua b/lua/mason-registry/php-cs-fixer/init.lua
index 0929a2b3..84dd0078 100644
--- a/lua/mason-registry/php-cs-fixer/init.lua
+++ b/lua/mason-registry/php-cs-fixer/init.lua
@@ -3,7 +3,6 @@ local _ = require "mason-core.functional"
 local github = require "mason-core.managers.github"
 local std = require "mason-core.managers.std"
 local platform = require "mason-core.platform"
-local path = require "mason-core.path"
 
 return Pkg.new {
     name = "php-cs-fixer",
@@ -31,16 +30,7 @@ return Pkg.new {
                 ctx:link_bin("php-cs-fixer", "php-cs-fixer")
             end,
             win = function()
-                ctx:link_bin(
-                    "php-cs-fixer",
-                    ctx:write_shell_exec_wrapper(
-                        "php-cs-fixer",
-                        ("php %q"):format(path.concat {
-                            ctx.package:get_install_path(),
-                            "php-cs-fixer.phar",
-                        })
-                    )
-                )
+                ctx:link_bin("php-cs-fixer", ctx:write_php_exec_wrapper("php-cs-fixer", "php-cs-fixer.phar"))
             end,
         }
     end,
diff --git a/lua/mason-registry/phpcbf/init.lua b/lua/mason-registry/phpcbf/init.lua
index 8cae909a..6dd062f5 100644
--- a/lua/mason-registry/phpcbf/init.lua
+++ b/lua/mason-registry/phpcbf/init.lua
@@ -3,13 +3,10 @@ local _ = require "mason-core.functional"
 local github = require "mason-core.managers.github"
 local std = require "mason-core.managers.std"
 local platform = require "mason-core.platform"
-local path = require "mason-core.path"
 
 return Pkg.new {
     name = "phpcbf",
-    desc = _.dedent [[
-        PHP_CodeSniffer(phpcbf) automatically corrects coding standard violations that would be detected by PHP_CodeSniffer(phpcs).
-    ]],
+    desc = [[phpcbf automatically corrects coding standard violations that would be detected by phpcs.]],
     homepage = "https://github.com/squizlabs/PHP_CodeSniffer",
     languages = { Pkg.Lang.PHP },
     categories = { Pkg.Cat.Formatter },
@@ -29,16 +26,7 @@ return Pkg.new {
                 ctx:link_bin("phpcbf", "phpcbf")
             end,
             win = function()
-                ctx:link_bin(
-                    "phpcbf",
-                    ctx:write_shell_exec_wrapper(
-                        "phpcbf",
-                        ("php %q"):format(path.concat {
-                            ctx.package:get_install_path(),
-                            "phpcbf.phar",
-                        })
-                    )
-                )
+                ctx:link_bin("phpcbf", ctx:write_php_exec_wrapper("phpcbf", "phpcbf.phar"))
             end,
         }
     end,
diff --git a/lua/mason-registry/phpcs/init.lua b/lua/mason-registry/phpcs/init.lua
index 17515ba2..41f82f7d 100644
--- a/lua/mason-registry/phpcs/init.lua
+++ b/lua/mason-registry/phpcs/init.lua
@@ -3,13 +3,10 @@ local _ = require "mason-core.functional"
 local github = require "mason-core.managers.github"
 local std = require "mason-core.managers.std"
 local platform = require "mason-core.platform"
-local path = require "mason-core.path"
 
 return Pkg.new {
     name = "phpcs",
-    desc = _.dedent [[
-        PHP_CodeSniffer(phpcs) tokenizes PHP, JavaScript and CSS files to detect violations of a defined standard.
-    ]],
+    desc = [[phpcs tokenizes PHP, JavaScript and CSS files to detect violations of a defined standard.]],
     homepage = "https://github.com/squizlabs/PHP_CodeSniffer",
     languages = { Pkg.Lang.PHP },
     categories = { Pkg.Cat.Linter },
@@ -29,16 +26,7 @@ return Pkg.new {
                 ctx:link_bin("phpcs", "phpcs")
             end,
             win = function()
-                ctx:link_bin(
-                    "phpcs",
-                    ctx:write_shell_exec_wrapper(
-                        "phpcs",
-                        ("php %q"):format(path.concat {
-                            ctx.package:get_install_path(),
-                            "phpcs.phar",
-                        })
-                    )
-                )
+                ctx:link_bin("phpcs", ctx:write_php_exec_wrapper("phpcs", "phpcs.phar"))
             end,
         }
     end,
diff --git a/lua/mason-registry/phpmd/init.lua b/lua/mason-registry/phpmd/init.lua
index a7810c54..4543711f 100644
--- a/lua/mason-registry/phpmd/init.lua
+++ b/lua/mason-registry/phpmd/init.lua
@@ -3,7 +3,6 @@ local _ = require "mason-core.functional"
 local github = require "mason-core.managers.github"
 local std = require "mason-core.managers.std"
 local platform = require "mason-core.platform"
-local path = require "mason-core.path"
 
 return Pkg.new {
     name = "phpmd",
@@ -30,16 +29,7 @@ return Pkg.new {
                 ctx:link_bin("phpmd", "phpmd")
             end,
             win = function()
-                ctx:link_bin(
-                    "phpmd",
-                    ctx:write_shell_exec_wrapper(
-                        "phpmd",
-                        ("php %q"):format(path.concat {
-                            ctx.package:get_install_path(),
-                            "phpmd.phar",
-                        })
-                    )
-                )
+                ctx:link_bin("phpmd", ctx:write_php_exec_wrapper("phpmd", "phpmd.phar"))
             end,
         }
     end,
diff --git a/lua/mason-registry/phpstan/init.lua b/lua/mason-registry/phpstan/init.lua
index bc0bf3ff..688bd75a 100644
--- a/lua/mason-registry/phpstan/init.lua
+++ b/lua/mason-registry/phpstan/init.lua
@@ -29,16 +29,7 @@ return Pkg.new {
                 ctx:link_bin("phpstan", "phpstan")
             end,
             win = function()
-                ctx:link_bin(
-                    "phpstan",
-                    ctx:write_shell_exec_wrapper(
-                        "phpstan",
-                        ("php %q"):format(path.concat {
-                            ctx.package:get_install_path(),
-                            "phpstan.phar",
-                        })
-                    )
-                )
+                ctx:link_bin("phpstan", ctx:write_php_exec_wrapper("phpstan", "phpstan.phar"))
             end,
         }
     end,
diff --git a/lua/mason-registry/twigcs/init.lua b/lua/mason-registry/twigcs/init.lua
index ea947c15..4a0a638a 100644
--- a/lua/mason-registry/twigcs/init.lua
+++ b/lua/mason-registry/twigcs/init.lua
@@ -3,14 +3,12 @@ local _ = require "mason-core.functional"
 local github = require "mason-core.managers.github"
 local std = require "mason-core.managers.std"
 local platform = require "mason-core.platform"
-local path = require "mason-core.path"
 
 return Pkg.new {
     name = "twigcs",
     desc = _.dedent [[
-        The missing checkstyle for twig!
-        Twigcs aims to be what phpcs is to php.
-        It checks your codebase for violations on coding standards.
+        The missing checkstyle for twig! Twigcs aims to be what phpcs is to php. It checks your codebase for violations
+        on coding standards.
     ]],
     homepage = "https://github.com/friendsoftwig/twigcs",
     languages = { Pkg.Lang.Twig },
@@ -31,16 +29,7 @@ return Pkg.new {
                 ctx:link_bin("twigcs", "twigcs")
             end,
             win = function()
-                ctx:link_bin(
-                    "twigcs",
-                    ctx:write_shell_exec_wrapper(
-                        "twigcs",
-                        ("php %q"):format(path.concat {
-                            ctx.package:get_install_path(),
-                            "twigcs.phar",
-                        })
-                    )
-                )
+                ctx:link_bin("twigcs", ctx:write_php_exec_wrapper("twigcs", "twigcs.phar"))
             end,
         }
     end,