Merge branch 'main' into fix/load-project-skills-git-root

Author: enyst

.github/scripts/check_sdk_api_breakage.py

@@ -551,8 +551,20 @@ def _compute_breakages(old_root, new_root, cfg: PackageConfig) -> tuple[int, int
     except Exception as e:
         raise RuntimeError(f"Failed to resolve root module '{pkg}'") from e
 
-    old_exports = _extract_exported_names(old_mod)
     new_exports = _extract_exported_names(new_mod)
+    try:
+        old_exports = _extract_exported_names(old_mod)
+    except ValueError as e:
+        # The API breakage check relies on a curated public surface defined via
+        # __all__. If the previous release didn't define (or couldn't statically
+        # evaluate) __all__, we can't compute meaningful breakages.
+        #
+        # In this situation, skip rather than failing the entire workflow.
+        print(
+            f"::notice title={title}::Skipping breakage check; previous release "
+            f"has no statically-evaluable {pkg}.__all__: {e}"
+        )
+        return 0, 0
 
     removed = sorted(old_exports - new_exports)
 

tests/ci_scripts/test_check_sdk_api_breakage.py

@@ -210,6 +210,23 @@ def test_removed_public_method_with_deprecation_is_not_undeprecated(tmp_path):
     assert undeprecated == 0
 
 
+def test_missing_all_in_previous_release_skips_breakage_check(tmp_path):
+    """If previous release lacks __all__, skip instead of failing workflow."""
+    old_pkg = tmp_path / "old" / "openhands" / "sdk"
+    old_pkg.mkdir(parents=True)
+    (tmp_path / "old" / "openhands" / "__init__.py").write_text("")
+    (old_pkg / "__init__.py").write_text("# no __all__ in previous release\n")
+
+    _write_pkg_init(tmp_path, "new", ["Foo"])
+
+    old_root = griffe.load("openhands.sdk", search_paths=[str(tmp_path / "old")])
+    new_root = griffe.load("openhands.sdk", search_paths=[str(tmp_path / "new")])
+
+    total_breaks, undeprecated = _prod._compute_breakages(old_root, new_root, _SDK_CFG)
+    assert total_breaks == 0
+    assert undeprecated == 0
+
+
 def test_parse_version_simple():
     v = _parse_version("1.2.3")
     assert v.major == 1