freesolo-flash-dev 0.2.39__tar.gz → 0.2.40__tar.gz

{freesolo_flash_dev-0.2.39 → freesolo_flash_dev-0.2.40}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: freesolo-flash-dev
-Version: 0.2.39
+Version: 0.2.40
 Summary: Flash — managed LoRA post-training (SFT/GRPO) for Freesolo environments, driven by the `flash` CLI
 Project-URL: Homepage, https://github.com/freesolo-co/flash
 Project-URL: Repository, https://github.com/freesolo-co/flash

{freesolo_flash_dev-0.2.39 → freesolo_flash_dev-0.2.40}/flash/schema/__init__.py

@@ -88,8 +88,24 @@ def normalize_env_name_segment(value: str) -> str | None:
 
 
 def load_toml(path: str) -> dict[str, Any]:
-    with open(path, "rb") as f:
-        return tomllib.load(f)
+    # a missing path, a directory, or an unreadable file is a user mistake, not a flash bug. only
+    # FileNotFoundError is in _USER_ERRORS, so IsADirectoryError / PermissionError would otherwise
+    # dump a raw traceback (and FileNotFoundError still surfaced a bare [Errno 2] string). route the
+    # whole OSError family through ConfigError so the cli prints one clean `error:` line instead.
+    try:
+        with open(path, "rb") as f:
+            return tomllib.load(f)
+    except FileNotFoundError as exc:
+        raise ConfigError(
+            f"config file not found: {path} (run `flash env setup` to scaffold configs/rl.toml)"
+        ) from exc
+    except IsADirectoryError as exc:
+        raise ConfigError(
+            f"{path} is a directory, not a TOML config (point train at a .toml file)"
+        ) from exc
+    except OSError as exc:
+        # permission denied, ENOTDIR on a path component, symlink loops, etc.
+        raise ConfigError(f"cannot read config {path}: {exc.strerror or exc}") from exc
 
 
 def spec_from_file(

{freesolo_flash_dev-0.2.39 → freesolo_flash_dev-0.2.40}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "freesolo-flash-dev"
-version = "0.2.39"
+version = "0.2.40"
 description = "Flash — managed LoRA post-training (SFT/GRPO) for Freesolo environments, driven by the `flash` CLI"
 readme = "README.md"
 # RunPod Flash supports Python 3.11-3.12 (not 3.13 yet).
@@ -106,7 +106,7 @@ artifacts = ["flash/py.typed"]
 # auto-published by .github/workflows/publish-dev.yml: a push to `dev` whose version isn't yet on
 # PyPI publishes it; pushes that don't change it find it already published and no-op. So cutting a
 # dev-channel release = bump this (and [project].version) and merge to `dev`.
-version = "0.2.39"
+version = "0.2.40"
 
 [tool.pytest.ini_options]
 pythonpath = ["."]

{freesolo_flash_dev-0.2.39 → freesolo_flash_dev-0.2.40}/tests/test_cli_errors.py

@@ -56,6 +56,29 @@ def test_bad_model_is_friendly():
     assert "Traceback (most recent call last)" not in proc.stderr
 
 
+def test_missing_config_is_friendly():
+    with tempfile.TemporaryDirectory() as tmp:
+        proc = _run(
+            ["train", os.path.join(tmp, "nope.toml"), "--dry-run"], env=_logged_out_env(tmp)
+        )
+    assert proc.returncode == 1
+    assert proc.stderr.startswith("error:")
+    assert "config file not found" in proc.stderr
+    # a bare [Errno 2] string and a traceback are both the wrong UX for a mistyped path.
+    assert "Errno" not in proc.stderr
+    assert "Traceback (most recent call last)" not in proc.stderr
+
+
+def test_config_pointed_at_a_directory_is_friendly():
+    with tempfile.TemporaryDirectory() as tmp:
+        # `flash train configs/` or `flash train .` — a directory, not a .toml file.
+        proc = _run(["train", tmp, "--dry-run"], env=_logged_out_env(tmp))
+    assert proc.returncode == 1
+    assert proc.stderr.startswith("error:")
+    assert "is a directory" in proc.stderr
+    assert "Traceback (most recent call last)" not in proc.stderr
+
+
 def test_debug_flag_shows_traceback():
     with tempfile.TemporaryDirectory() as tmp:
         proc = _run(["--debug", "status", "does-not-exist"], env=_logged_out_env(tmp))
@@ -82,10 +105,7 @@ def test_missing_env_id_rejected_client_side():
     with tempfile.TemporaryDirectory() as tmp:
         cfg = os.path.join(tmp, "run.toml")
         with open(cfg, "w") as f:
-            f.write(
-                'model = "Qwen/Qwen3.5-4B"\nalgorithm = "grpo"\n'
-                "[environment]\n[train]\n"
-            )
+            f.write('model = "Qwen/Qwen3.5-4B"\nalgorithm = "grpo"\n[environment]\n[train]\n')
         # A config without [environment] id is rejected before any network call.
         submit = _run(["train", cfg], env=_logged_out_env(tmp))
         assert submit.returncode == 1
@@ -113,7 +133,7 @@ def test_cost_needs_no_live_pricing():
             f.write(
                 'model = "Qwen/Qwen3.5-4B"\nalgorithm = "grpo"\n'
                 '[environment]\nid = "github:freesolo-co/envs@main:gsm8k/environment.py"\n'
-                "[train]\nsteps = 1\nhf_repo = \"owner/runs\"\n"
+                '[train]\nsteps = 1\nhf_repo = "owner/runs"\n'
             )
         proc = _run(["train", cfg, "--cost"], env=_logged_out_env(tmp))
     assert proc.returncode == 0, proc.stdout + proc.stderr