claude-dev-env 1.37.0 → 1.38.0

package/skills/pr-converge/scripts/test_check_pr_mergeability.py

@@ -1,126 +0,0 @@
-"""Tests for check_pr_mergeability.
-
-Covers:
-- gh pr view is invoked with the documented mergeability --json field list
-- the parsed JSON object is returned with mergeable/mergeStateStatus/headRefOid keys
-- subprocess errors propagate
-"""
-
-from __future__ import annotations
-
-import importlib.util
-import json
-import subprocess
-from pathlib import Path
-from types import ModuleType
-from unittest.mock import MagicMock, patch
-
-import pytest
-
-
-def _load_module() -> ModuleType:
-    module_path = Path(__file__).parent / "check_pr_mergeability.py"
-    spec = importlib.util.spec_from_file_location("check_pr_mergeability", module_path)
-    assert spec is not None
-    assert spec.loader is not None
-    module = importlib.util.module_from_spec(spec)
-    spec.loader.exec_module(module)
-    return module
-
-
-check_pr_mergeability_module = _load_module()
-
-
-def _completed(stdout: str) -> subprocess.CompletedProcess:
-    process = MagicMock(spec=subprocess.CompletedProcess)
-    process.stdout = stdout
-    process.returncode = 0
-    return process
-
-
-def test_should_invoke_gh_pr_view_with_mergeability_field_list() -> None:
-    payload = json.dumps(
-        {
-            "mergeable": "MERGEABLE",
-            "mergeStateStatus": "CLEAN",
-            "headRefOid": "abc123",
-        }
-    )
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(payload)
-        check_pr_mergeability_module.check_pr_mergeability(
-            owner="acme", repo="widget", number=42
-        )
-    invoked_argv = mock_run.call_args[0][0]
-    assert invoked_argv[0:3] == ["gh", "pr", "view"]
-    assert "--json" in invoked_argv
-    fields_arg = invoked_argv[invoked_argv.index("--json") + 1]
-    for required_field in ("mergeable", "mergeStateStatus", "headRefOid"):
-        assert required_field in fields_arg
-
-
-def test_should_pass_pr_number_and_repo_arg_for_explicit_targeting() -> None:
-    payload = json.dumps(
-        {
-            "mergeable": "MERGEABLE",
-            "mergeStateStatus": "CLEAN",
-            "headRefOid": "abc123",
-        }
-    )
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(payload)
-        check_pr_mergeability_module.check_pr_mergeability(
-            owner="acme", repo="widget", number=42
-        )
-    invoked_argv = mock_run.call_args[0][0]
-    assert invoked_argv[3] == "42"
-    assert "--repo" in invoked_argv
-    repo_arg_value = invoked_argv[invoked_argv.index("--repo") + 1]
-    assert repo_arg_value == "acme/widget"
-
-
-def test_should_return_parsed_json_object_with_mergeability_keys() -> None:
-    payload = {
-        "mergeable": "CONFLICTING",
-        "mergeStateStatus": "DIRTY",
-        "headRefOid": "deadbeef",
-    }
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(json.dumps(payload))
-        mergeability_state = check_pr_mergeability_module.check_pr_mergeability(
-            owner="acme", repo="widget", number=42
-        )
-    assert mergeability_state == payload
-    assert mergeability_state["mergeable"] == "CONFLICTING"
-    assert mergeability_state["mergeStateStatus"] == "DIRTY"
-    assert mergeability_state["headRefOid"] == "deadbeef"
-
-
-def test_should_raise_when_gh_subprocess_fails() -> None:
-    failure = subprocess.CalledProcessError(
-        returncode=1, cmd=["gh"], stderr="auth failure"
-    )
-    with patch("subprocess.run", side_effect=failure):
-        with pytest.raises(subprocess.CalledProcessError):
-            check_pr_mergeability_module.check_pr_mergeability(
-                owner="acme", repo="widget", number=42
-            )
-
-
-def test_should_pass_imported_constant_directly_without_local_alias() -> None:
-    payload = json.dumps(
-        {
-            "mergeable": "MERGEABLE",
-            "mergeStateStatus": "CLEAN",
-            "headRefOid": "abc",
-        }
-    )
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(payload)
-        check_pr_mergeability_module.check_pr_mergeability(
-            owner="acme", repo="widget", number=42
-        )
-    invoked_argv = mock_run.call_args[0][0]
-    fields_arg = invoked_argv[invoked_argv.index("--json") + 1]
-    expected_fields = check_pr_mergeability_module.MERGEABILITY_FIELDS
-    assert fields_arg is expected_fields

package/skills/pr-converge/scripts/test_fetch_bugbot_inline_comments.py

@@ -1,443 +0,0 @@
-"""Tests for fetch_bugbot_inline_comments.
-
-Covers:
-- gh command uses --paginate --slurp on the comments endpoint
-- only cursor[bot] inline comments are returned
-- comments not anchored to the requested commit are filtered out
-- comments on the same commit but from an older Bugbot review are filtered out
-- multi-page responses are flattened correctly
-- subprocess errors propagate
-"""
-
-from __future__ import annotations
-
-import importlib.util
-import json
-import subprocess
-from pathlib import Path
-from types import ModuleType
-from unittest.mock import MagicMock, patch
-
-import pytest
-
-
-def _load_module() -> ModuleType:
-    module_path = Path(__file__).parent / "fetch_bugbot_inline_comments.py"
-    spec = importlib.util.spec_from_file_location(
-        "fetch_bugbot_inline_comments", module_path
-    )
-    assert spec is not None
-    assert spec.loader is not None
-    module = importlib.util.module_from_spec(spec)
-    spec.loader.exec_module(module)
-    return module
-
-
-fetch_bugbot_inline_comments_module = _load_module()
-
-
-def _completed(stdout: str) -> subprocess.CompletedProcess:
-    process = MagicMock(spec=subprocess.CompletedProcess)
-    process.stdout = stdout
-    process.returncode = 0
-    return process
-
-
-def _default_review_for_head(*, commit: str, review_id: int) -> list[dict]:
-    return [
-        {
-            "review_id": review_id,
-            "commit_id": commit,
-            "submitted_at": "2026-01-01T00:00:00Z",
-            "body": "Cursor Bugbot has reviewed your changes and found 0 potential issue",
-            "classification": "clean",
-        }
-    ]
-
-
-def test_should_invoke_gh_with_paginate_slurp_against_comments_endpoint() -> None:
-    pages_payload = json.dumps([[]])
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=_default_review_for_head(commit="abc123", review_id=1),
-    ), patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-            owner="acme", repo="widget", number=42, current_head="abc123"
-        )
-    invoked_argv = mock_run.call_args[0][0]
-    assert invoked_argv[0] == "gh"
-    assert invoked_argv[1] == "api"
-    assert "repos/acme/widget/pulls/42/comments?per_page=100" in invoked_argv[2]
-    assert "--paginate" in invoked_argv
-    assert "--slurp" in invoked_argv
-
-
-def test_should_filter_to_cursor_bot_only() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 100,
-                    "user": {"login": "copilot-pull-request-reviewer[bot]"},
-                    "commit_id": "abc123",
-                    "pull_request_review_id": 1,
-                    "body": "copilot finding",
-                    "path": "x.py",
-                    "line": 5,
-                },
-                {
-                    "id": 101,
-                    "user": {"login": "cursor[bot]"},
-                    "commit_id": "abc123",
-                    "pull_request_review_id": 1,
-                    "body": "bugbot finding",
-                    "path": "x.py",
-                    "line": 6,
-                },
-            ]
-        ]
-    )
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=_default_review_for_head(commit="abc123", review_id=1),
-    ), patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_inline_comments = (
-            fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-                owner="acme", repo="widget", number=42, current_head="abc123"
-            )
-        )
-    assert len(all_inline_comments) == 1
-    assert all_inline_comments[0]["comment_id"] == 101
-
-
-def test_should_filter_out_comments_not_on_current_head() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 200,
-                    "user": {"login": "cursor[bot]"},
-                    "commit_id": "old_sha",
-                    "pull_request_review_id": 1,
-                    "body": "stale finding",
-                    "path": "x.py",
-                    "line": 5,
-                },
-                {
-                    "id": 201,
-                    "user": {"login": "cursor[bot]"},
-                    "commit_id": "current_sha",
-                    "pull_request_review_id": 2,
-                    "body": "fresh finding",
-                    "path": "x.py",
-                    "line": 6,
-                },
-            ]
-        ]
-    )
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=_default_review_for_head(commit="current_sha", review_id=2),
-    ), patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_inline_comments = (
-            fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-                owner="acme", repo="widget", number=42, current_head="current_sha"
-            )
-        )
-    assert len(all_inline_comments) == 1
-    assert all_inline_comments[0]["comment_id"] == 201
-
-
-def test_should_ignore_inline_comments_from_older_bugbot_review_on_same_commit() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 300,
-                    "user": {"login": "cursor[bot]"},
-                    "commit_id": "same_sha",
-                    "pull_request_review_id": 10,
-                    "body": "stale dirty thread",
-                    "path": "x.py",
-                    "line": 1,
-                },
-                {
-                    "id": 301,
-                    "user": {"login": "cursor[bot]"},
-                    "commit_id": "same_sha",
-                    "pull_request_review_id": 11,
-                    "body": "current clean thread",
-                    "path": "x.py",
-                    "line": 2,
-                },
-            ]
-        ]
-    )
-    reviews_newest_first = [
-        {
-            "review_id": 11,
-            "commit_id": "same_sha",
-            "submitted_at": "2026-01-02T00:00:00Z",
-            "body": "clean",
-            "classification": "clean",
-        },
-        {
-            "review_id": 10,
-            "commit_id": "same_sha",
-            "submitted_at": "2026-01-01T00:00:00Z",
-            "body": "Cursor Bugbot has reviewed your changes and found 1 potential issue",
-            "classification": "dirty",
-        },
-    ]
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=reviews_newest_first,
-    ), patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_inline_comments = (
-            fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-                owner="acme", repo="widget", number=42, current_head="same_sha"
-            )
-        )
-    assert [each_comment["comment_id"] for each_comment in all_inline_comments] == [301]
-
-
-def test_should_return_empty_when_no_bugbot_review_exists_for_commit() -> None:
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=[
-            {
-                "review_id": 1,
-                "commit_id": "other_sha",
-                "submitted_at": "2026-01-01T00:00:00Z",
-                "body": "",
-                "classification": "clean",
-            }
-        ],
-    ), patch("subprocess.run") as mock_run:
-        all_inline_comments = (
-            fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-                owner="acme", repo="widget", number=42, current_head="missing_sha"
-            )
-        )
-    assert all_inline_comments == []
-    mock_run.assert_not_called()
-
-
-def test_should_flatten_across_pages() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 1,
-                    "user": {"login": "cursor[bot]"},
-                    "commit_id": "abc",
-                    "pull_request_review_id": 9,
-                    "body": "a",
-                    "path": "f.py",
-                    "line": 1,
-                }
-            ],
-            [
-                {
-                    "id": 2,
-                    "user": {"login": "cursor[bot]"},
-                    "commit_id": "abc",
-                    "pull_request_review_id": 9,
-                    "body": "b",
-                    "path": "f.py",
-                    "line": 2,
-                },
-                {
-                    "id": 3,
-                    "user": {"login": "cursor[bot]"},
-                    "commit_id": "abc",
-                    "pull_request_review_id": 9,
-                    "body": "c",
-                    "path": "f.py",
-                    "line": 3,
-                },
-            ],
-        ]
-    )
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=_default_review_for_head(commit="abc", review_id=9),
-    ), patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_inline_comments = (
-            fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-                owner="acme", repo="widget", number=42, current_head="abc"
-            )
-        )
-    assert [each_comment["comment_id"] for each_comment in all_inline_comments] == [
-        1,
-        2,
-        3,
-    ]
-
-
-def test_should_match_login_case_insensitively() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 300,
-                    "user": {"login": "Cursor"},
-                    "commit_id": "abc123",
-                    "pull_request_review_id": 1,
-                    "body": "uppercase login",
-                    "path": "x.py",
-                    "line": 5,
-                },
-                {
-                    "id": 301,
-                    "user": {"login": "CURSOR[bot]"},
-                    "commit_id": "abc123",
-                    "pull_request_review_id": 1,
-                    "body": "screaming login",
-                    "path": "x.py",
-                    "line": 6,
-                },
-            ]
-        ]
-    )
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=_default_review_for_head(commit="abc123", review_id=1),
-    ), patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_inline_comments = (
-            fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-                owner="acme", repo="widget", number=42, current_head="abc123"
-            )
-        )
-    assert {each_comment["comment_id"] for each_comment in all_inline_comments} == {300, 301}
-
-
-def test_should_match_login_containing_cursor_substring() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 400,
-                    "user": {"login": "internal-cursor-fork[bot]"},
-                    "commit_id": "abc123",
-                    "pull_request_review_id": 1,
-                    "body": "non-canonical login still matches",
-                    "path": "x.py",
-                    "line": 5,
-                }
-            ]
-        ]
-    )
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=_default_review_for_head(commit="abc123", review_id=1),
-    ), patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_inline_comments = (
-            fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-                owner="acme", repo="widget", number=42, current_head="abc123"
-            )
-        )
-    assert len(all_inline_comments) == 1
-    assert all_inline_comments[0]["comment_id"] == 400
-
-
-def test_should_exclude_login_without_cursor_substring() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 500,
-                    "user": {"login": "copilot-pull-request-reviewer[bot]"},
-                    "commit_id": "abc123",
-                    "pull_request_review_id": 1,
-                    "body": "copilot finding",
-                    "path": "x.py",
-                    "line": 5,
-                },
-                {
-                    "id": 501,
-                    "user": {"login": "dependabot[bot]"},
-                    "commit_id": "abc123",
-                    "pull_request_review_id": 1,
-                    "body": "dependency bump",
-                    "path": "x.py",
-                    "line": 6,
-                },
-            ]
-        ]
-    )
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=_default_review_for_head(commit="abc123", review_id=1),
-    ), patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_inline_comments = (
-            fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-                owner="acme", repo="widget", number=42, current_head="abc123"
-            )
-        )
-    assert all_inline_comments == []
-
-
-def test_should_raise_when_gh_subprocess_fails() -> None:
-    failure = subprocess.CalledProcessError(
-        returncode=1, cmd=["gh"], stderr="auth failure"
-    )
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=_default_review_for_head(commit="abc", review_id=1),
-    ), patch("subprocess.run", side_effect=failure):
-        with pytest.raises(subprocess.CalledProcessError):
-            fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-                owner="acme", repo="widget", number=42, current_head="abc"
-            )
-
-
-def test_should_return_entries_whose_keys_are_strings() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 101,
-                    "user": {"login": "cursor[bot]"},
-                    "commit_id": "abc123",
-                    "pull_request_review_id": 1,
-                    "body": "bugbot finding",
-                    "path": "x.py",
-                    "line": 6,
-                }
-            ]
-        ]
-    )
-    with patch.object(
-        fetch_bugbot_inline_comments_module,
-        "fetch_bugbot_reviews",
-        return_value=_default_review_for_head(commit="abc123", review_id=1),
-    ), patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_inline_comments = (
-            fetch_bugbot_inline_comments_module.fetch_bugbot_inline_comments(
-                owner="acme", repo="widget", number=42, current_head="abc123"
-            )
-        )
-    assert len(all_inline_comments) == 1
-    first_comment_entry = all_inline_comments[0]
-    assert isinstance(first_comment_entry, dict)
-    assert all(isinstance(each_key, str) for each_key in first_comment_entry.keys())