sentry-devenv 1.24.0__py3-none-any.whl → 1.25.0__py3-none-any.whl

devenv/lib/colima.py

@@ -104,9 +104,9 @@ def check() -> ColimaStatus:
     if not os.getenv("CI"):
         macos_version = platform.mac_ver()[0]
         macos_major_version = int(macos_version.split(".")[0])
-        if macos_major_version < 14:
+        if macos_major_version < 15:
             raise SystemExit(
-                f"macos >= 14 is required to use colima, found {macos_version}"
+                f"macos >= 15 is required to use colima, found {macos_version}"
             )
 
     docker_executable = shutil.which("docker")
@@ -196,7 +196,8 @@ def start(restart: bool = False) -> ColimaStatus:
             "1.1.1.1",
             # ideally we keep ~ ro, but currently the "default" vm
             # is shared across repositories, so for ease of use we'll let home rw
-            f"--mount=/var/folders:w,/private/tmp/colima:w,{home}:w,/tmp/:w",
+            f"--mount=/var/folders:w,/private/tmp/colima:w,{home}:w,/tmp/sentry-profiles:w",
+            # note: it is not allowed by lima to add top-level root directories like /tmp!
             *args,
         ),
         pathprepend=f"{root}/bin",

devenv/lib/node.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import json
 import os
 import shutil
 import tempfile
@@ -9,7 +10,7 @@ from devenv.lib import fs
 from devenv.lib import proc
 
 
-_shims = ("node", "npm", "npx", "yarn", "pnpm")
+_shims = ("node", "npm", "npx")
 
 
 def _install(url: str, sha256: str, into: str) -> None:
@@ -32,7 +33,7 @@ def _install(url: str, sha256: str, into: str) -> None:
 def uninstall(binroot: str) -> None:
     shutil.rmtree(f"{binroot}/node-env", ignore_errors=True)
 
-    for shim in (*_shims, "yarn"):
+    for shim in (*_shims, "pnpm"):
         fp = f"{binroot}/{shim}"
         try:
             os.remove(fp)
@@ -71,6 +72,51 @@ def installed(version: str, binroot: str) -> bool:
         return False
 
 
+def installed_pnpm(version: str, binroot: str) -> bool:
+    if shutil.which(
+        "pnpm", path=binroot
+    ) != f"{binroot}/pnpm" or not os.path.exists(
+        f"{binroot}/node-env/bin/pnpm"
+    ):
+        return False
+
+    stdout = proc.run((f"{binroot}/pnpm", "--version"), stdout=True)
+    installed_version = stdout.strip()
+    return version == installed_version
+
+
+def install_pnpm(reporoot: str) -> None:
+    binroot = fs.ensure_binroot(reporoot)
+
+    with open(f"{reporoot}/package.json") as f:
+        package_json = json.load(f)
+        pnpm = package_json["packageManager"]
+        pnpm_version = pnpm.split("@")[-1]
+
+    if installed_pnpm(pnpm_version, binroot):
+        return
+
+    print(f"installing pnpm {pnpm_version}...")
+
+    # {binroot}/npm is a devenv-managed shim, so
+    # this install -g ends up putting pnpm into
+    # .devenv/bin/node-env/bin/pnpm which is pointed
+    # to by the {binroot}/pnpm shim
+    proc.run(
+        (f"{binroot}/npm", "install", "-g", f"pnpm@{pnpm_version}"), stdout=True
+    )
+
+    fs.write_script(
+        f"{binroot}/pnpm",
+        """#!/bin/sh
+export PATH={binroot}/node-env/bin:"${{PATH}}"
+export NPM_CONFIG_PREFIX={binroot}/node-env
+exec {binroot}/node-env/bin/pnpm "$@"
+""",
+        shell_escape={"binroot": binroot},
+    )
+
+
 def install(version: str, url: str, sha256: str, reporoot: str) -> None:
     binroot = fs.ensure_binroot(reporoot)
 
@@ -81,9 +127,8 @@ def install(version: str, url: str, sha256: str, reporoot: str) -> None:
     uninstall(binroot)
     _install(url, sha256, binroot)
 
-    # NPM_CONFIG_PREFIX is needed to ensure npm install -g yarn
-    # puts yarn into our node-env.
-
+    # NPM_CONFIG_PREFIX is needed to ensure npm install -g pnpm
+    # puts pnpm into our node-env.
     for shim in _shims:
         fs.write_script(
             f"{binroot}/{shim}",
@@ -97,51 +142,3 @@ exec {binroot}/node-env/bin/{shim} "$@"
 
     if not installed(version, binroot):
         raise SystemExit(f"failed to install node {version}!")
-
-
-def installed_yarn(version: str, binroot: str) -> bool:
-    if shutil.which(
-        "yarn", path=binroot
-    ) != f"{binroot}/yarn" or not os.path.exists(
-        f"{binroot}/node-env/bin/yarn"
-    ):
-        return False
-
-    with open(f"{binroot}/yarn", "r") as f:
-        sample = f.read(64)
-        if "VOLTA_HOME" in sample:
-            print("volta-based yarn detected")
-            return False
-
-    stdout = proc.run((f"{binroot}/yarn", "--version"), stdout=True)
-    installed_version = stdout.strip()
-    if version == installed_version:
-        return True
-
-    print(f"installed yarn {installed_version} is unexpected!")
-    return False
-
-
-def install_yarn(version: str, reporoot: str) -> None:
-    binroot = fs.ensure_binroot(reporoot)
-
-    if installed_yarn(version, binroot):
-        return
-
-    # not explicitly uninstalling here, following steps
-    # will pave over it
-    print(f"installing yarn {version}...")
-
-    proc.run((f"{binroot}/npm", "install", "-g", f"yarn@{version}"))
-
-    fs.write_script(
-        f"{binroot}/yarn",
-        """#!/bin/sh
-export PATH={binroot}/node-env/bin:"${{PATH}}"
-exec {binroot}/node-env/bin/yarn "$@"
-""",
-        shell_escape={"binroot": binroot},
-    )
-
-    if not installed_yarn(version, binroot):
-        raise SystemExit(f"failed to install yarn {version}!")

{sentry_devenv-1.24.0.dist-info → sentry_devenv-1.25.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sentry_devenv
-Version: 1.24.0
+Version: 1.25.0
 Summary: Utilities for setting up a Sentry development environment
 Author-email: Joshua Li <joshua.li@sentry.io>, Ian Woodard <ian.woodard@sentry.io>, Buck Evan <buck.evan@sentry.io>
 Classifier: Programming Language :: Python :: 3
@@ -335,16 +335,16 @@ def main(context: dict[str, str]) -> int:
         cfg["node"][f"{constants.SYSTEM_MACHINE}_sha256"],
         reporoot,
     )
-    node.install_yarn(cfg["node"]["yarn_version"], reporoot)
+
+    node.install_pnpm(reporoot)
 
     print("installing node dependencies...")
     proc.run(
         (
-            ".devenv/bin/yarn",
+            f"{reporoot}/.devenv/bin/pnpm",
             "install",
             "--frozen-lockfile",
-            "--no-progress",
-            "--non-interactive",
+            "--reporter=append-only",
         ),
     )
 
@@ -366,7 +366,6 @@ linux_x86_64 = https://storage.googleapis.com/sentry-dev-infra-assets/node/node-
 linux_x86_64_sha256 = efc0f295dd878e510ab12ea36bbadc3db03c687ab30c07e86c7cdba7eed879a9
 # used for autoupdate
 version = v20.13.1
-yarn_version = 1.22.22
 ```
 
 ### brew

{sentry_devenv-1.24.0.dist-info → sentry_devenv-1.25.0.dist-info}/RECORD

@@ -21,7 +21,7 @@ devenv/checks/test.py,sha256=AvxoA2W5qTO2ZpoXmjScLiJxXD3Y_limYp5yTjGSZ9U,701
 devenv/lib/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 devenv/lib/archive.py,sha256=75MjlDuwHVaazbNUr8vJmqTyl4Jz6KRd_fUcpwvPwec,4710
 devenv/lib/brew.py,sha256=_7recvYh-LnUJC0eKnYK7F8AzGRL5bi_Khv0f_mgrlM,1795
-devenv/lib/colima.py,sha256=xd9EpysU2JlD0mD8rDv78QeetaUs7q4HOJXxXnAb1Mc,7534
+devenv/lib/colima.py,sha256=vK1QItRGg8C46m0krtApvBsu3Op5FeuroHCZSIirZPw,7640
 devenv/lib/config.py,sha256=5eXYwM44AJ3w2VfVbCDqchYOL1zAjazNcirtM6kxxGw,3344
 devenv/lib/context.py,sha256=4EDImK9pvTp3FqeNigevC9lbJsgdDO9D1t2_Y4NyPlc,207
 devenv/lib/direnv.py,sha256=HDRUaIR0PMAQYiiTaw3gEQuh6Tdb7BUJmV7dBZdPztM,1325
@@ -31,7 +31,7 @@ devenv/lib/gcloud.py,sha256=pJ28_L0gUS1lZna-gIHeJPo9bEfSHVXh63CoRQGyAw8,3552
 devenv/lib/github.py,sha256=AMtcz6gTVKy7qvCP1zTUdHQlnSM32TVaji0aOXJ_F9A,3463
 devenv/lib/limactl.py,sha256=wMA_Q4DRPjmBPe0JSj1xPwYesFa9MT_vVSdaBAAKn5w,4256
 devenv/lib/modules.py,sha256=shoqgja0b-xzXr601Ryz0JOy3Vxbw53I3cRYezVt77g,859
-devenv/lib/node.py,sha256=sp5QbExF0H03UnfLUmFown42_NhyPWKw0H776W7987U,4160
+devenv/lib/node.py,sha256=Rz5A5tep3YEfTfyVnrcAZEZCD6-rmdG4MdgpwRC033E,4119
 devenv/lib/proc.py,sha256=eU7DHIk_66wgkHRmdtKbBKZTa6q4tl9QJbrlnQOQCls,2743
 devenv/lib/repository.py,sha256=cDCgcZ6lw0bcWC6DrcDp3tz_MWObmgAyFq39TeU7YHY,1966
 devenv/lib/rosetta.py,sha256=Lm-tDFut86o6l-Y55d5kTacRHTwrv5MFEymQpy7Qihc,1170
@@ -70,11 +70,12 @@ tests/lib/test_brew.py,sha256=g2Sz-yg4-212qlsKg0vfHf604bQyRjUElsf3khvXrTQ,5711
 tests/lib/test_direnv.py,sha256=GKpEbsHgaRZWevYrsa0oOGQFCsyllCLX1vGdg3igvAc,1551
 tests/lib/test_fs.py,sha256=qhX9CQTTSht1f6ZJfmpsSFICTwPs--h_mLlS_1jnZQg,1670
 tests/lib/test_github.py,sha256=IMEG2cmRaK_PMJprFE_ZMqPnZ0StwWqgznhhT_zVk3U,4526
+tests/lib/test_node.py,sha256=0cmWS-YqTXtQ5p6dk2Hc5qTl8XQGt3CGGfVy1rrsE9k,1082
 tests/lib/test_proc.py,sha256=XH6OnxKPSSm3HvDjYHApputMKwgOE8lYqDuK2vK0Z0U,2626
 tests/lib/test_repository.py,sha256=gUi1lkY7bha5WwZ5xcnENOlFYboVV8TFW1lCESvS0VA,787
 tests/lib/test_venv.py,sha256=3XGpINFBkn6Yq4OycsF-Ki4xMWAINt2g69uOHtkDU6k,3266
-sentry_devenv-1.24.0.dist-info/METADATA,sha256=w5wVhVx5bE8mhGjpkyNegoQxclAtzbdL9zsac6fytew,15500
-sentry_devenv-1.24.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-sentry_devenv-1.24.0.dist-info/entry_points.txt,sha256=StsIuNugcoEU8jsPy6H9ECjzoDzOVKXl1vUaYTIGbzM,44
-sentry_devenv-1.24.0.dist-info/top_level.txt,sha256=dOQExvIA0fj_EQjCrMTS7CCHNH7WZFHxEU0M7LlNcKQ,16
-sentry_devenv-1.24.0.dist-info/RECORD,,
+sentry_devenv-1.25.0.dist-info/METADATA,sha256=mr9-PPOWcU4UtufPHtYt9_8JYT82XI4CIjLJFNh0w0A,15437
+sentry_devenv-1.25.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+sentry_devenv-1.25.0.dist-info/entry_points.txt,sha256=StsIuNugcoEU8jsPy6H9ECjzoDzOVKXl1vUaYTIGbzM,44
+sentry_devenv-1.25.0.dist-info/top_level.txt,sha256=dOQExvIA0fj_EQjCrMTS7CCHNH7WZFHxEU0M7LlNcKQ,16
+sentry_devenv-1.25.0.dist-info/RECORD,,

tests/lib/test_node.py

@@ -0,0 +1,37 @@
+from __future__ import annotations
+
+import os
+import pathlib
+from unittest.mock import patch
+
+from devenv.lib import node
+from devenv.lib.repository import Repository
+
+
+def test_install_pnpm(tmp_path: pathlib.Path) -> None:
+    repo = Repository(f"{tmp_path}/test")
+
+    binroot = f"{repo.path}/.devenv/bin"
+    os.makedirs(binroot)
+    open(f"{binroot}/node", "w").close()
+
+    with patch("devenv.lib.archive.download"), patch(
+        "devenv.lib.archive.unpack",
+        side_effect=lambda archive_file, tmpd, perform_strip1, strip1_new_prefix: os.makedirs(
+            f"{tmpd}/{strip1_new_prefix}"
+        ),
+    ), patch("devenv.lib.node.os.path.exists"), patch(
+        "devenv.lib.direnv.proc.run", side_effect=["0.0.0"]  # node --version
+    ):
+        node.install("0.0.0", "bar", "baz", repo.path)
+
+    with open(f"{binroot}/npm", "r") as f:
+        text = f.read()
+        assert (
+            text
+            == f"""#!/bin/sh
+export PATH={binroot}/node-env/bin:"${{PATH}}"
+export NPM_CONFIG_PREFIX={binroot}/node-env
+exec {binroot}/node-env/bin/npm "$@"
+"""
+        )