[WIP] Updated, implemented, and existing tests pass

python · zooba · Dec 9, 2025 · Nov 20, 2025 · Nov 24, 2025 · Nov 25, 2025
commit 8b83213d37cc2fe47f3cf3f4546edef10e2482e0
diff --git a/src/manage/aliasutils.py b/src/manage/aliasutils.py
@@ -0,0 +1,234 @@
+import os
+
+from .fsutils import ensure_tree, unlink
+from .logging import LOGGER
+from .pathutils import Path
+from .tagutils import install_matches_any
+
+
+def _if_exists(launcher, plat):
+    suffix = "." + launcher.suffix.lstrip(".")
+    plat_launcher = launcher.parent / f"{launcher.stem}{plat}{suffix}"
+    if plat_launcher.is_file():
+        return plat_launcher
+    return launcher
+
+
+def create_alias(cmd, install, alias, target, *, script_code=None, _link=os.link):
+    p = (cmd.global_dir / alias["name"])
+    target = Path(target)
+    ensure_tree(p)
+    launcher = cmd.launcher_exe
+    if alias.get("windowed"):
+        launcher = cmd.launcherw_exe or launcher
+
+    plat = install["tag"].rpartition("-")[-1]
+    if plat:
+        LOGGER.debug("Checking for launcher for platform -%s", plat)
+        launcher = _if_exists(launcher, f"-{plat}")
+    if not launcher.is_file():
+        LOGGER.debug("Checking for launcher for default platform %s", cmd.default_platform)
+        launcher = _if_exists(launcher, cmd.default_platform)
+    if not launcher.is_file():
+        LOGGER.debug("Checking for launcher for -64")
+        launcher = _if_exists(launcher, "-64")
+    LOGGER.debug("Create %s linking to %s using %s", alias["name"], target, launcher)
+    if not launcher or not launcher.is_file():
+        if install_matches_any(install, getattr(cmd, "tags", None)):
+            LOGGER.warn("Skipping %s alias because the launcher template was not found.", alias["name"])
+        else:
+            LOGGER.debug("Skipping %s alias because the launcher template was not found.", alias["name"])
+        return
+
+    try:
+        launcher_bytes = launcher.read_bytes()
+    except OSError:
+        warnings_shown = cmd.scratch.setdefault("aliasutils.create_alias.warnings_shown", set())
+        if str(launcher) not in warnings_shown:
+            LOGGER.warn("Failed to read launcher template at %s.", launcher)
+            warnings_shown.add(str(launcher))
+        LOGGER.debug("Failed to read %s", launcher, exc_info=True)
+        return
+
+    existing_bytes = b''
+    try:
+        with open(p, 'rb') as f:
+            existing_bytes = f.read(len(launcher_bytes) + 1)
+    except FileNotFoundError:
+        pass
+    except OSError:
+        LOGGER.debug("Failed to read existing alias launcher.")
+
+    launcher_remap = cmd.scratch.setdefault("aliasutils.create_alias.launcher_remap", {})
+
+    if existing_bytes == launcher_bytes:
+        # Valid existing launcher, so save its path in case we need it later
+        # for a hard link.
+        launcher_remap.setdefault(launcher.name, p)
+    else:
+        # First try and create a hard link
+        unlink(p)
+        try:
+            _link(launcher, p)
+            LOGGER.debug("Created %s as hard link to %s", p.name, launcher.name)
+        except OSError as ex:
+            if ex.winerror != 17:
+                # Report errors other than cross-drive links
+                LOGGER.debug("Failed to create hard link for command.", exc_info=True)
+            launcher2 = launcher_remap.get(launcher.name)
+            if launcher2:
+                try:
+                    _link(launcher2, p)
+                    LOGGER.debug("Created %s as hard link to %s", p.name, launcher2.name)
+                except FileNotFoundError:
+                    raise
+                except OSError:
+                    LOGGER.debug("Failed to create hard link to fallback launcher")
+                    launcher2 = None
+            if not launcher2:
+                try:
+                    p.write_bytes(launcher_bytes)
+                    LOGGER.debug("Created %s as copy of %s", p.name, launcher.name)
+                    launcher_remap[launcher.name] = p
+                except OSError:
+                    LOGGER.error("Failed to create global command %s.", alias["name"])
+                    LOGGER.debug(exc_info=True)
+
+    p_target = p.with_name(p.name + ".__target__")
+    do_update = True
+    try:
+        do_update = not target.match(p_target.read_text(encoding="utf-8"))
+    except FileNotFoundError:
+        pass
+    except (OSError, UnicodeDecodeError):
+        LOGGER.debug("Failed to read existing target path.", exc_info=True)
+
+    if do_update:
+        p_target.write_text(str(target), encoding="utf-8")
+
+    p_script = p.with_name(p.name + "-script.py")
+    if script_code:
+        do_update = True
+        try:
+            do_update = p_script.read_text(encoding="utf-8") == script_code
+        except FileNotFoundError:
+            pass
+        except (OSError, UnicodeDecodeError):
+            LOGGER.debug("Failed to read existing script file.", exc_info=True)
+        if do_update:
+            p_script.write_text(script_code, encoding="utf-8")
+    else:
+        try:
+            unlink(p_script)
+        except OSError:
+            LOGGER.error("Failed to clean up existing alias. Re-run with -v "
+                         "or check the install log for details.")
+            LOGGER.info("Failed to remove %s.", p_script, exc_info=True)
+
+
+def _parse_entrypoint_line(line):
+    name, sep, rest = line.partition("=")
+    name = name.strip()
+    if name and sep and rest:
+        mod, sep, rest = rest.partition(":")
+        mod = mod.strip()
+        if mod and sep and rest:
+            func, sep, extra = rest.partition("[")
+            func = func.strip()
+            if func:
+                return name, mod, func
+    return None, None, None
+
+
+def _scan(prefix, dirs):
+    for dirname in dirs or ():
+        root = prefix / dirname
+
+        # Scan d for dist-info directories with entry_points.txt
+        dist_info = [d for d in root.listdir() if d.match("*.dist-info") and d.is_dir()]
+        LOGGER.debug("Found %i dist-info directories in %s", len(dist_info), root)
+        entrypoints = [f for f in [d / "entry_points.txt" for d in dist_info] if f.is_file()]
+        LOGGER.debug("Found %i entry_points.txt files in %s", len(entrypoints), root)
+
+        # Filter down to [console_scripts] and [gui_scripts]
+        for ep in entrypoints:
+            try:
+                f = open(ep, "r", encoding="utf-8", errors="strict")
+            except OSError:
+                LOGGER.debug("Failed to read %s", ep, exc_info=True)
+                continue
+
+            with f:
+                alias = None
+                for line in f:
+                    if line.strip() == "[console_scripts]":
+                        alias = dict(windowed=0)
+                    elif line.strip() == "[gui_scripts]":
+                        alias = dict(windowed=1)
+                    elif line.lstrip().startswith("["):
+                        alias = None
+                    elif alias is not None:
+                        name, mod, func = _parse_entrypoint_line(line)
+                        if name and mod and func:
+                            yield (
+                                {**alias, "name": name},
+                                f"import sys; from {mod} import {func}; sys.exit({func}())",
+                            )
+
+
+def scan_and_create_entrypoints(cmd, install, shortcut, _create_alias=create_alias):
+    prefix = install["prefix"]
+    known = cmd.scratch.setdefault("entrypointutils.known", set())
+
+    aliases = list(install.get("alias", ()))
+    alias_1 = [a for a in aliases if not a.get("windowed")]
+    alias_2 = [a for a in aliases if a.get("windowed")]
+
+    # If no windowed targets, we'll use the non-windowed one
+    targets = [prefix / a["target"] for a in [*alias_1[:1], *alias_2[:1], *alias_1[:1]]]
+    if len(targets) < 2:
+        LOGGER.debug("No suitable alias found for %s. Skipping entrypoints",
+                     install["id"])
+        return
+
+    for alias, code in _scan(prefix, shortcut.get("dirs")):
+        # Only create names once per install command
+        n = alias["name"].casefold()
+        if n in known:
+            continue
+        known.add(n)
+
+        # Copy the launcher template and create a standard __target__ file
+        _create_alias(cmd, install, alias, targets[alias.get("windowed", 0)],
+                      script_code=code)
+
+
+def cleanup_entrypoints(cmd, install_shortcut_pairs):
+    seen_names = set()
+    for install, shortcut in install_shortcut_pairs:
+        for alias, code in _scan(install["prefix"], shortcut.get("dirs")):
+            seen_names.add(alias["name"].casefold())
+
+    # Scan existing aliases
+    scripts = cmd.global_dir.glob("*-script.py")
+
+    # Excluding any in seen_names, delete unused aliases
+    for script in scripts:
+        name = script.name.rpartition("-")[0]
+        if name.casefold() in seen_names:
+            continue
+
+        alias = cmd.global_dir / (name + ".exe")
+        if not alias.is_file():
+            continue
+
+        try:
+            unlink(alias)
+            LOGGER.debug("Deleted %s", alias)
+        except OSError:
+            LOGGER.warn("Failed to delete %s", alias)
+        try:
+            unlink(script)
+            LOGGER.debug("Deleted %s", script)
+        except OSError:
+            LOGGER.warn("Failed to delete %s", script)
diff --git a/src/manage/entrypointutils.py b/src/manage/entrypointutils.py