@openhands/extensions 0.20.0 → 0.21.0

package/tests/test_github_skill_installation.py

@@ -0,0 +1,63 @@
+"""Verify shared support is included in independently installed skills."""
+
+import json
+import subprocess
+import sys
+from pathlib import Path
+
+import pytest
+from openhands.sdk.skills import install_skill
+
+ROOT = Path(__file__).resolve().parents[1]
+
+
+def test_installed_triage_skill_contains_its_bundle(tmp_path):
+    name = "github-issue-triage"
+    install_skill(source=str(ROOT / "skills" / name), installed_dir=tmp_path)
+    scripts = tmp_path / name / "scripts"
+    manifest = json.loads(
+        (ROOT / f"automations/catalog/{name}/manifest.json").read_text()
+    )
+
+    for filename, source in manifest["setup"]["bundle"]["files"].items():
+        installed = scripts / filename
+        assert installed.is_file() and not installed.is_symlink()
+        assert installed.read_bytes() == (ROOT / source).read_bytes()
+
+    result = subprocess.run(
+        [sys.executable, "-c", "import worker"],
+        cwd=scripts,
+        text=True,
+        capture_output=True,
+        check=False,
+    )
+    assert result.returncode == 0, result.stderr
+
+
+@pytest.mark.parametrize(
+    "name",
+    [
+        "github-repo-monitor",
+        "github-agents-md-maintainer",
+        "github-issue-to-pr",
+        "github-pr-reviewer",
+    ],
+)
+def test_installed_github_skill_contains_shared_client(tmp_path, name):
+    install_skill(source=str(ROOT / "skills" / name), installed_dir=tmp_path)
+    scripts = tmp_path / name / "scripts"
+    helper = scripts / "github_client.py"
+
+    assert helper.is_file() and not helper.is_symlink()
+    assert (
+        helper.read_bytes()
+        == (ROOT / "skills/github/scripts/github_client.py").read_bytes()
+    )
+    result = subprocess.run(
+        [sys.executable, "-c", "import main"],
+        cwd=scripts,
+        text=True,
+        capture_output=True,
+        check=False,
+    )
+    assert result.returncode == 0, result.stderr

package/tests/test_github_triage_delivery.py

@@ -0,0 +1,53 @@
+"""Contract tests for independent triage delivery."""
+
+import json
+from unittest.mock import Mock
+
+import pytest
+from github_automation_helpers import worker
+
+
+@pytest.mark.parametrize(
+    "ready,criteria",
+    [(True, ["One observable result"]), (False, ["Clarify account ownership"])],
+)
+def test_triage_preserves_labels_and_only_readies_clear_work(
+    tmp_path, monkeypatch, ready, criteria
+):
+    cls = worker("github-issue-triage", tmp_path, monkeypatch).IssueTriage
+    run = object.__new__(cls)
+    issue = {
+        "number": 1,
+        "title": "A small feature",
+        "body": "",
+        "labels": [{"name": "enhancement"}],
+    }
+    run.open_issues = lambda: [issue]
+    run.gh_pages = lambda path: []
+    run.dependencies_complete = lambda issue: True
+    result = {
+        "ready": ready,
+        "priority": "normal",
+        "acceptance_criteria": criteria,
+        "rationale": "Small independent change",
+    }
+    run.evidence = tmp_path
+    run.conversation = Mock()
+    run.conversation.run.side_effect = lambda **kwargs: (
+        tmp_path / "triage.json"
+    ).write_text(json.dumps(result))
+    calls = []
+    run.gh = lambda *args: calls.append(args)
+    run.comment = lambda *args: calls.append(("comment", *args))
+
+    run.run()
+
+    updates = [call for call in calls if call[0] == "PATCH"]
+    assert bool(updates) is ready
+    if ready:
+        assert set(updates[0][2]["labels"]) == {
+            "enhancement",
+            "priority:normal",
+            "ready-for-dev",
+        }
+    assert criteria[0] in next(call[2] for call in calls if call[0] == "comment")