runops 2.1.1__tar.gz → 2.1.2__tar.gz

{runops-2.1.1 → runops-2.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: runops
-Version: 2.1.1
+Version: 2.1.2
 Summary: Run ledger and Slurm job submission for HPC simulation projects
 Project-URL: Homepage, https://github.com/Nkzono99/runops
 Project-URL: Documentation, https://github.com/Nkzono99/runops/blob/main/docs/reference.md

{runops-2.1.1 → runops-2.1.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "runops"
-version = "2.1.1"
+version = "2.1.2"
 description = "Run ledger and Slurm job submission for HPC simulation projects"
 readme = "README.md"
 license = "Apache-2.0"

{runops-2.1.1 → runops-2.1.2}/src/runops/__init__.py

@@ -1,3 +1,3 @@
 """runops: run ledger and Slurm job submission for HPC simulation projects."""
 
-__version__ = "2.1.1"
+__version__ = "2.1.2"

{runops-2.1.1 → runops-2.1.2}/src/runops/cli.py

@@ -91,6 +91,19 @@ def _confirm(message: str, yes: bool) -> None:
         raise typer.Exit(1)
 
 
+def _ask_install(root: Path, yes: bool) -> bool:
+    """Ask before creating .venv; default yes. Off a terminal, require -y."""
+    message = (
+        f"Create .venv in {root} and install simulator packages "
+        "(may build simulator binaries and take several minutes)?"
+    )
+    if yes:
+        return True
+    if not sys.stdin.isatty():
+        raise UsageError(f"{message} -- confirmation required; rerun with -y/--yes or --no-install")
+    return bool(typer.confirm(message, default=True))
+
+
 def _resolve(project: Project, refs: list[str]) -> list[Manifest]:
     if not refs:
         raise UsageError("give at least one RUN")
@@ -133,11 +146,14 @@ def init(
             show_default=False,
         )
         sims = [s.strip() for s in answer.split(",") if s.strip()]
-    if sims and not no_install and any(s.split("=")[0] != "generic" for s in sims):
-        _confirm(f"Create .venv in {root} and install simulator packages?", yes)
-    report = init_project(root, sims, site=site, name=name, install=not no_install)
+    install = not no_install
+    if sims and install and any(s.split("=")[0] != "generic" for s in sims):
+        install = _ask_install(root, yes)
+    report = init_project(root, sims, site=site, name=name, install=install)
     if report.site_detected:
         out.print(f"site: {report.site} (guessed from host {hostname()}; use --site to change)")
+    if not install and not no_install:
+        out.print("packages not installed; ask your agent or run `runo update` later")
     for item in report.created:
         out.print(f"created {item}")
     for spec in report.installed:

{runops-2.1.1 → runops-2.1.2}/tests/test_setup_misc.py

@@ -165,3 +165,49 @@ def test_usage_error_exit_code(cli) -> None:  # type: ignore[no-untyped-def]
 def test_cli_does_not_import_click_directly() -> None:
     source = (Path(__file__).parents[1] / "src/runops/cli.py").read_text()
     assert "import click" not in source  # newer typer vendors click
+
+
+def test_init_install_prompt_defaults_to_yes(
+    tmp_path: Path, monkeypatch: pytest.MonkeyPatch, cli
+) -> None:  # type: ignore[no-untyped-def]
+    calls: list[bool] = []
+
+    def fake_install(project, *, upgrade):  # type: ignore[no-untyped-def]
+        calls.append(upgrade)
+        return ["MPIEMSES3D"], []
+
+    monkeypatch.setattr("runops.ops.setup.install_packages", fake_install)
+    monkeypatch.chdir(tmp_path)
+    monkeypatch.setattr(
+        "sys.stdin", _Tty("\nemses\n\n")
+    )  # site: Enter, sims: emses, install: Enter
+    res = cli("init")
+    assert res.code == 0, res.err
+    assert "may build simulator binaries" in res.out
+    assert "[Y/n]" in res.out
+    assert calls == [False]
+    assert "installed MPIEMSES3D" in res.out
+
+
+def test_init_install_declined_keeps_project(
+    tmp_path: Path, monkeypatch: pytest.MonkeyPatch, cli
+) -> None:  # type: ignore[no-untyped-def]
+    monkeypatch.setattr(
+        "runops.ops.setup.install_packages",
+        lambda project, *, upgrade: (_ for _ in ()).throw(AssertionError("must not install")),
+    )
+    monkeypatch.chdir(tmp_path)
+    monkeypatch.setattr("sys.stdin", _Tty("\nemses\nn\n"))
+    res = cli("init")
+    assert res.code == 0, res.err
+    assert "runo update" in res.out
+    assert "emses" in load_toml(tmp_path / "runops.toml")["simulators"]
+
+
+def test_init_install_needs_yes_off_terminal(
+    tmp_path: Path, monkeypatch: pytest.MonkeyPatch, cli
+) -> None:  # type: ignore[no-untyped-def]
+    monkeypatch.chdir(tmp_path)
+    res = cli("init", "emses")
+    assert res.code == 2 and "--yes" in res.err
+    assert not (tmp_path / "runops.toml").exists()

{runops-2.1.1 → runops-2.1.2}/uv.lock

@@ -525,7 +525,7 @@ wheels = [
 
 [[package]]
 name = "runops"
-version = "2.1.1"
+version = "2.1.2"
 source = { editable = "." }
 dependencies = [
     { name = "jinja2" },