Fix goget installer resolution and relink seeded Playwright

Author: pirate

abx_pkg/binprovider_goget.py

@@ -82,7 +82,7 @@ def load_PATH_from_go_env(self) -> Self:
         bin_dir = self.bin_dir
         assert bin_dir is not None
         if self.install_root != DEFAULT_GOPATH or "bin_dir" in self.model_fields_set:
-            self.PATH = self._merge_PATH(bin_dir)
+            self.PATH = self._merge_PATH(bin_dir, PATH=self.PATH)
         else:
             self.PATH = self._merge_PATH(bin_dir, PATH=self.PATH)
         return self
@@ -234,6 +234,21 @@ def default_version_handler(
         abspath = abspath or self.get_abspath(bin_name, quiet=True)
         if not abspath:
             return None
+        if str(bin_name) == self.INSTALLER_BIN:
+            version_provider = (
+                self.get_provider_with_overrides(dry_run=False)
+                if self.dry_run
+                else self
+            )
+            proc = version_provider.exec(
+                bin_name=abspath,
+                cmd=["version"],
+                timeout=timeout,
+                quiet=True,
+            )
+            if proc.returncode != 0:
+                return None
+            return SemVer.parse(proc.stdout.strip() or proc.stderr.strip())
         try:
             installer_abspath = self.INSTALLER_BINARY(no_cache=no_cache).loaded_abspath
             assert installer_abspath

tests/test_playwrightprovider.py

@@ -32,6 +32,18 @@ def copy_seeded_playwright_root(
             symlinks=True,
             copy_function=os.link,
         )
+        copied_bin_dir = install_root / "bin"
+        if not copied_bin_dir.is_dir():
+            return
+        for link_path in copied_bin_dir.iterdir():
+            if not link_path.is_symlink():
+                continue
+            link_target = link_path.resolve(strict=False)
+            if seeded_playwright_root.resolve() not in link_target.parents:
+                continue
+            relative_target = link_target.relative_to(seeded_playwright_root.resolve())
+            link_path.unlink()
+            link_path.symlink_to(install_root / relative_target)
 
     def test_chromium_install_puts_real_browser_into_managed_bin_dir(
         self,