claude-dev-env 1.37.0 → 1.38.0

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

@@ -1,448 +0,0 @@
-"""Tests for reviewer_fetch_core.
-
-Covers:
-- fetch_reviewer_reviews invokes gh against the reviews endpoint with --paginate --slurp
-- login filter applies case-insensitively as a substring on user.login
-- entries missing submitted_at or id are filtered out
-- reviews are sorted newest-first by submitted_at
-- the spec.classify_review callable is invoked for each surviving review
-- subprocess errors propagate
-- fetch_reviewer_inline_comments returns empty when no review for current_head
-- fetch_reviewer_inline_comments only returns comments anchored to the latest review
-- fetch_reviewer_inline_comments invokes gh against the comments endpoint with --paginate --slurp
-"""
-
-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
-
-
-def _load_module() -> ModuleType:
-    module_path = Path(__file__).parent / "reviewer_fetch_core.py"
-    spec = importlib.util.spec_from_file_location("reviewer_fetch_core", 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
-
-
-reviewer_fetch_core_module = _load_module()
-
-
-def _completed(stdout: str) -> subprocess.CompletedProcess:
-    process = MagicMock(spec=subprocess.CompletedProcess)
-    process.stdout = stdout
-    process.returncode = 0
-    return process
-
-
-def _fake_spec(*, login_filter_substring: str = "test") -> object:
-    fake_spec_object = MagicMock()
-    fake_spec_object.login_filter_substring = login_filter_substring
-    fake_spec_object.classify_review = MagicMock(return_value="clean")
-    return fake_spec_object
-
-
-def test_fetch_reviewer_reviews_invokes_gh_with_paginate_slurp_against_reviews_endpoint() -> (
-    None
-):
-    pages_payload = json.dumps([[]])
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        reviewer_fetch_core_module.fetch_reviewer_reviews(
-            _fake_spec(), owner="acme", repo="widget", number=42
-        )
-    invoked_argv = mock_run.call_args[0][0]
-    assert invoked_argv[0] == "gh"
-    assert invoked_argv[1] == "api"
-    assert "repos/acme/widget/pulls/42/reviews?per_page=100" in invoked_argv[2]
-    assert "--paginate" in invoked_argv
-    assert "--slurp" in invoked_argv
-
-
-def test_fetch_reviewer_reviews_filters_by_login_filter_substring_case_insensitively() -> (
-    None
-):
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 1,
-                    "user": {"login": "TestBot[bot]"},
-                    "state": "APPROVED",
-                    "commit_id": "abc",
-                    "submitted_at": "2026-01-01T00:00:00Z",
-                    "body": "uppercase login",
-                },
-                {
-                    "id": 2,
-                    "user": {"login": "other-reviewer"},
-                    "state": "APPROVED",
-                    "commit_id": "abc",
-                    "submitted_at": "2026-01-02T00:00:00Z",
-                    "body": "no match",
-                },
-            ]
-        ]
-    )
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_reviews = reviewer_fetch_core_module.fetch_reviewer_reviews(
-            _fake_spec(login_filter_substring="test"),
-            owner="acme",
-            repo="widget",
-            number=42,
-        )
-    assert len(all_reviews) == 1
-    assert all_reviews[0]["review_id"] == 1
-
-
-def test_fetch_reviewer_reviews_drops_entries_missing_submitted_at() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 1,
-                    "user": {"login": "test[bot]"},
-                    "state": "APPROVED",
-                    "commit_id": "abc",
-                    "body": "no submitted_at",
-                },
-                {
-                    "id": 2,
-                    "user": {"login": "test[bot]"},
-                    "state": "APPROVED",
-                    "commit_id": "abc",
-                    "submitted_at": "2026-01-02T00:00:00Z",
-                    "body": "valid",
-                },
-            ]
-        ]
-    )
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_reviews = reviewer_fetch_core_module.fetch_reviewer_reviews(
-            _fake_spec(), owner="acme", repo="widget", number=42
-        )
-    assert [each_review["review_id"] for each_review in all_reviews] == [2]
-
-
-def test_fetch_reviewer_reviews_drops_entries_missing_id() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "user": {"login": "test[bot]"},
-                    "state": "APPROVED",
-                    "commit_id": "abc",
-                    "submitted_at": "2026-01-01T00:00:00Z",
-                    "body": "no id",
-                },
-                {
-                    "id": 99,
-                    "user": {"login": "test[bot]"},
-                    "state": "APPROVED",
-                    "commit_id": "abc",
-                    "submitted_at": "2026-01-02T00:00:00Z",
-                    "body": "valid",
-                },
-            ]
-        ]
-    )
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_reviews = reviewer_fetch_core_module.fetch_reviewer_reviews(
-            _fake_spec(), owner="acme", repo="widget", number=42
-        )
-    assert [each_review["review_id"] for each_review in all_reviews] == [99]
-
-
-def test_fetch_reviewer_reviews_sorts_newest_first_across_pages() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 10,
-                    "user": {"login": "test[bot]"},
-                    "state": "APPROVED",
-                    "commit_id": "old",
-                    "submitted_at": "2026-01-01T00:00:00Z",
-                    "body": "oldest",
-                }
-            ],
-            [
-                {
-                    "id": 11,
-                    "user": {"login": "test[bot]"},
-                    "state": "CHANGES_REQUESTED",
-                    "commit_id": "new",
-                    "submitted_at": "2026-01-03T00:00:00Z",
-                    "body": "newest",
-                },
-                {
-                    "id": 12,
-                    "user": {"login": "test[bot]"},
-                    "state": "APPROVED",
-                    "commit_id": "mid",
-                    "submitted_at": "2026-01-02T00:00:00Z",
-                    "body": "middle",
-                },
-            ],
-        ]
-    )
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_reviews = reviewer_fetch_core_module.fetch_reviewer_reviews(
-            _fake_spec(), owner="acme", repo="widget", number=42
-        )
-    assert [each_review["submitted_at"] for each_review in all_reviews] == [
-        "2026-01-03T00:00:00Z",
-        "2026-01-02T00:00:00Z",
-        "2026-01-01T00:00:00Z",
-    ]
-
-
-def test_fetch_reviewer_reviews_invokes_classify_callable_per_review() -> None:
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 1,
-                    "user": {"login": "test[bot]"},
-                    "state": "APPROVED",
-                    "commit_id": "abc",
-                    "submitted_at": "2026-01-01T00:00:00Z",
-                    "body": "first",
-                },
-                {
-                    "id": 2,
-                    "user": {"login": "test[bot]"},
-                    "state": "CHANGES_REQUESTED",
-                    "commit_id": "abc",
-                    "submitted_at": "2026-01-02T00:00:00Z",
-                    "body": "second",
-                },
-            ]
-        ]
-    )
-    classify_callable = MagicMock(side_effect=["dirty", "clean"])
-    fake_spec_object = MagicMock()
-    fake_spec_object.login_filter_substring = "test"
-    fake_spec_object.classify_review = classify_callable
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_reviews = reviewer_fetch_core_module.fetch_reviewer_reviews(
-            fake_spec_object, owner="acme", repo="widget", number=42
-        )
-    assert classify_callable.call_count == 2
-    assert {each_review["classification"] for each_review in all_reviews} == {
-        "dirty",
-        "clean",
-    }
-
-
-def test_fetch_reviewer_reviews_propagates_subprocess_errors() -> None:
-    failure = subprocess.CalledProcessError(
-        returncode=1, cmd=["gh"], stderr="auth failure"
-    )
-    with patch("subprocess.run", side_effect=failure):
-        try:
-            reviewer_fetch_core_module.fetch_reviewer_reviews(
-                _fake_spec(), owner="acme", repo="widget", number=42
-            )
-            assert False, "expected CalledProcessError"
-        except subprocess.CalledProcessError:
-            pass
-
-
-def test_fetch_reviewer_inline_comments_returns_empty_when_no_review_for_head() -> None:
-    no_matching_review = [
-        {
-            "review_id": 1,
-            "commit_id": "other_sha",
-            "submitted_at": "2026-01-01T00:00:00Z",
-            "state": "APPROVED",
-            "body": "",
-            "classification": "clean",
-        }
-    ]
-    with patch("subprocess.run") as mock_run:
-        all_inline_comments = reviewer_fetch_core_module.fetch_reviewer_inline_comments(
-            _fake_spec(),
-            owner="acme",
-            repo="widget",
-            number=42,
-            current_head="missing_sha",
-            all_reviews=no_matching_review,
-        )
-    assert all_inline_comments == []
-    mock_run.assert_not_called()
-
-
-def test_fetch_reviewer_inline_comments_invokes_gh_against_comments_endpoint() -> None:
-    pages_payload = json.dumps([[]])
-    matching_review = [
-        {
-            "review_id": 1,
-            "commit_id": "abc123",
-            "submitted_at": "2026-01-01T00:00:00Z",
-            "state": "APPROVED",
-            "body": "",
-            "classification": "clean",
-        }
-    ]
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        reviewer_fetch_core_module.fetch_reviewer_inline_comments(
-            _fake_spec(),
-            owner="acme",
-            repo="widget",
-            number=42,
-            current_head="abc123",
-            all_reviews=matching_review,
-        )
-    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_fetch_reviewer_inline_comments_anchors_to_latest_review_id_for_head() -> None:
-    reviews_newest_first = [
-        {
-            "review_id": 11,
-            "commit_id": "same_sha",
-            "submitted_at": "2026-01-02T00:00:00Z",
-            "state": "APPROVED",
-            "body": "lgtm",
-            "classification": "clean",
-        },
-        {
-            "review_id": 10,
-            "commit_id": "same_sha",
-            "submitted_at": "2026-01-01T00:00:00Z",
-            "state": "CHANGES_REQUESTED",
-            "body": "fix",
-            "classification": "dirty",
-        },
-    ]
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 100,
-                    "user": {"login": "test[bot]"},
-                    "commit_id": "same_sha",
-                    "pull_request_review_id": 10,
-                    "body": "stale",
-                    "path": "x.py",
-                    "line": 1,
-                },
-                {
-                    "id": 101,
-                    "user": {"login": "test[bot]"},
-                    "commit_id": "same_sha",
-                    "pull_request_review_id": 11,
-                    "body": "current",
-                    "path": "x.py",
-                    "line": 2,
-                },
-            ]
-        ]
-    )
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_inline_comments = reviewer_fetch_core_module.fetch_reviewer_inline_comments(
-            _fake_spec(),
-            owner="acme",
-            repo="widget",
-            number=42,
-            current_head="same_sha",
-            all_reviews=reviews_newest_first,
-        )
-    assert [each_comment["comment_id"] for each_comment in all_inline_comments] == [101]
-
-
-def test_fetch_reviewer_inline_comments_filters_login_substring() -> None:
-    matching_review = [
-        {
-            "review_id": 9,
-            "commit_id": "abc",
-            "submitted_at": "2026-01-01T00:00:00Z",
-            "state": "APPROVED",
-            "body": "",
-            "classification": "clean",
-        }
-    ]
-    pages_payload = json.dumps(
-        [
-            [
-                {
-                    "id": 1,
-                    "user": {"login": "test[bot]"},
-                    "commit_id": "abc",
-                    "pull_request_review_id": 9,
-                    "body": "match",
-                    "path": "f.py",
-                    "line": 1,
-                },
-                {
-                    "id": 2,
-                    "user": {"login": "other-reviewer"},
-                    "commit_id": "abc",
-                    "pull_request_review_id": 9,
-                    "body": "no match",
-                    "path": "f.py",
-                    "line": 2,
-                },
-            ]
-        ]
-    )
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(pages_payload)
-        all_inline_comments = reviewer_fetch_core_module.fetch_reviewer_inline_comments(
-            _fake_spec(login_filter_substring="test"),
-            owner="acme",
-            repo="widget",
-            number=42,
-            current_head="abc",
-            all_reviews=matching_review,
-        )
-    assert [each_comment["comment_id"] for each_comment in all_inline_comments] == [1]
-
-
-def test_fetch_reviewer_inline_comments_propagates_subprocess_errors() -> None:
-    matching_review = [
-        {
-            "review_id": 1,
-            "commit_id": "abc",
-            "submitted_at": "2026-01-01T00:00:00Z",
-            "state": "APPROVED",
-            "body": "",
-            "classification": "clean",
-        }
-    ]
-    failure = subprocess.CalledProcessError(
-        returncode=1, cmd=["gh"], stderr="auth failure"
-    )
-    with patch("subprocess.run", side_effect=failure):
-        try:
-            reviewer_fetch_core_module.fetch_reviewer_inline_comments(
-                _fake_spec(),
-                owner="acme",
-                repo="widget",
-                number=42,
-                current_head="abc",
-                all_reviews=matching_review,
-            )
-            assert False, "expected CalledProcessError"
-        except subprocess.CalledProcessError:
-            pass

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

@@ -1,107 +0,0 @@
-"""Tests for reviewer_specs.
-
-Covers:
-- each ReviewerSpec instance carries the documented login_filter_substring
-- bugbot_spec.classify_review uses the dirty-body regex
-- copilot_spec.classify_review dispatches off review state plus body
-- claude_spec.classify_review dispatches off review state plus body
-"""
-
-from __future__ import annotations
-
-import importlib.util
-from pathlib import Path
-from types import ModuleType
-
-
-def _load_module() -> ModuleType:
-    module_path = Path(__file__).parent / "reviewer_specs.py"
-    spec = importlib.util.spec_from_file_location("reviewer_specs", 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
-
-
-reviewer_specs_module = _load_module()
-
-
-def test_bugbot_spec_uses_cursor_login_filter_substring() -> None:
-    assert reviewer_specs_module.bugbot_spec.login_filter_substring == "cursor"
-
-
-def test_copilot_spec_uses_copilot_login_filter_substring() -> None:
-    assert reviewer_specs_module.copilot_spec.login_filter_substring == "copilot"
-
-
-def test_claude_spec_uses_claude_login_filter_substring() -> None:
-    assert reviewer_specs_module.claude_spec.login_filter_substring == "claude"
-
-
-def test_bugbot_classify_returns_dirty_when_body_matches_findings_pattern() -> None:
-    review_payload = {
-        "body": "Cursor Bugbot has reviewed your changes and found 2 potential issues.",
-    }
-    assert reviewer_specs_module.bugbot_spec.classify_review(review_payload) == "dirty"
-
-
-def test_bugbot_classify_returns_clean_when_body_lacks_findings_pattern() -> None:
-    review_payload = {
-        "body": "Bugbot reviewed your changes and found no new issues!",
-    }
-    assert reviewer_specs_module.bugbot_spec.classify_review(review_payload) == "clean"
-
-
-def test_copilot_classify_returns_clean_when_state_is_approved() -> None:
-    review_payload = {"state": "APPROVED", "body": "lgtm"}
-    assert reviewer_specs_module.copilot_spec.classify_review(review_payload) == "clean"
-
-
-def test_copilot_classify_returns_dirty_when_state_is_changes_requested() -> None:
-    review_payload = {"state": "CHANGES_REQUESTED", "body": "fix this"}
-    assert reviewer_specs_module.copilot_spec.classify_review(review_payload) == "dirty"
-
-
-def test_copilot_classify_returns_dirty_when_state_is_commented_with_body() -> None:
-    review_payload = {"state": "COMMENTED", "body": "minor nit"}
-    assert reviewer_specs_module.copilot_spec.classify_review(review_payload) == "dirty"
-
-
-def test_copilot_classify_returns_clean_when_state_is_commented_with_empty_body() -> (
-    None
-):
-    review_payload = {"state": "COMMENTED", "body": ""}
-    assert reviewer_specs_module.copilot_spec.classify_review(review_payload) == "clean"
-
-
-def test_copilot_classify_returns_clean_when_state_is_unknown() -> None:
-    review_payload = {"state": "DISMISSED", "body": "ignored"}
-    assert reviewer_specs_module.copilot_spec.classify_review(review_payload) == "clean"
-
-
-def test_claude_classify_returns_clean_when_state_is_approved() -> None:
-    review_payload = {"state": "APPROVED", "body": "lgtm"}
-    assert reviewer_specs_module.claude_spec.classify_review(review_payload) == "clean"
-
-
-def test_claude_classify_returns_dirty_when_state_is_changes_requested() -> None:
-    review_payload = {"state": "CHANGES_REQUESTED", "body": "fix this"}
-    assert reviewer_specs_module.claude_spec.classify_review(review_payload) == "dirty"
-
-
-def test_claude_classify_returns_dirty_when_state_is_commented_with_body() -> None:
-    review_payload = {"state": "COMMENTED", "body": "minor nit"}
-    assert reviewer_specs_module.claude_spec.classify_review(review_payload) == "dirty"
-
-
-def test_claude_classify_returns_clean_when_state_is_commented_with_empty_body() -> (
-    None
-):
-    review_payload = {"state": "COMMENTED", "body": ""}
-    assert reviewer_specs_module.claude_spec.classify_review(review_payload) == "clean"
-
-
-def test_claude_classify_returns_clean_when_state_is_unknown() -> None:
-    review_payload = {"state": "DISMISSED", "body": "ignored"}
-    assert reviewer_specs_module.claude_spec.classify_review(review_payload) == "clean"

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

@@ -1,139 +0,0 @@
-"""Tests for trigger_bugbot.
-
-Covers:
-- gh pr comment is invoked with --body-file (per gh-body-file rule)
-- the body file written contains the literal phrase "bugbot run\\n"
-- the comment URL emitted by gh is returned
-- the temp body file is cleaned up
-- subprocess errors propagate
-"""
-
-from __future__ import annotations
-
-import importlib.util
-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 / "trigger_bugbot.py"
-    spec = importlib.util.spec_from_file_location("trigger_bugbot", 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
-
-
-trigger_bugbot_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_comment_with_body_file_flag() -> None:
-    captured_body_paths: list[str] = []
-
-    def capture_body_file_contents(*subprocess_args, **_subprocess_kwargs):
-        invoked_argv = subprocess_args[0]
-        assert "--body-file" in invoked_argv
-        body_file_path = invoked_argv[invoked_argv.index("--body-file") + 1]
-        captured_body_paths.append(body_file_path)
-        return _completed("https://github.com/acme/widget/issues/42#issuecomment-99\n")
-
-    with patch("subprocess.run", side_effect=capture_body_file_contents) as mock_run:
-        trigger_bugbot_module.trigger_bugbot(owner="acme", repo="widget", number=42)
-    invoked_argv = mock_run.call_args[0][0]
-    assert invoked_argv[0:3] == ["gh", "pr", "comment"]
-    assert "42" in invoked_argv
-    assert "--repo" in invoked_argv
-    assert "acme/widget" in invoked_argv
-
-
-def test_should_write_literal_bugbot_run_phrase_into_body_file() -> None:
-    captured_body_contents: list[str] = []
-
-    def capture_body_file_contents(*subprocess_args, **_subprocess_kwargs):
-        invoked_argv = subprocess_args[0]
-        body_file_path = Path(invoked_argv[invoked_argv.index("--body-file") + 1])
-        captured_body_contents.append(body_file_path.read_text(encoding="utf-8"))
-        return _completed("https://example.com\n")
-
-    with patch("subprocess.run", side_effect=capture_body_file_contents):
-        trigger_bugbot_module.trigger_bugbot(owner="acme", repo="widget", number=42)
-    assert len(captured_body_contents) == 1
-    assert captured_body_contents[0] == "bugbot run\n"
-
-
-def test_should_return_comment_url_from_gh_stdout() -> None:
-    expected_url = "https://github.com/acme/widget/pull/42#issuecomment-12345"
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed(f"{expected_url}\n")
-        comment_url = trigger_bugbot_module.trigger_bugbot(
-            owner="acme", repo="widget", number=42
-        )
-    assert comment_url == expected_url
-
-
-def test_should_remove_temp_body_file_after_invocation() -> None:
-    captured_body_paths: list[Path] = []
-
-    def capture_body_file_contents(*subprocess_args, **_subprocess_kwargs):
-        invoked_argv = subprocess_args[0]
-        captured_body_paths.append(
-            Path(invoked_argv[invoked_argv.index("--body-file") + 1])
-        )
-        return _completed("https://example.com\n")
-
-    with patch("subprocess.run", side_effect=capture_body_file_contents):
-        trigger_bugbot_module.trigger_bugbot(owner="acme", repo="widget", number=42)
-    assert len(captured_body_paths) == 1
-    assert not captured_body_paths[0].exists()
-
-
-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):
-            trigger_bugbot_module.trigger_bugbot(owner="acme", repo="widget", number=42)
-
-
-def test_should_write_imported_constant_directly_without_local_alias() -> None:
-    captured_body_contents: list[str] = []
-
-    def capture_body_file_contents(*subprocess_args, **_subprocess_kwargs):
-        invoked_argv = subprocess_args[0]
-        body_file_path = Path(invoked_argv[invoked_argv.index("--body-file") + 1])
-        captured_body_contents.append(body_file_path.read_text(encoding="utf-8"))
-        return _completed("https://example.com\n")
-
-    with patch("subprocess.run", side_effect=capture_body_file_contents):
-        trigger_bugbot_module.trigger_bugbot(owner="acme", repo="widget", number=99)
-    assert len(captured_body_contents) == 1
-    assert (
-        captured_body_contents[0]
-        == trigger_bugbot_module.BUGBOT_RUN_TRIGGER_PHRASE
-    )
-
-
-def test_should_render_repo_arg_via_named_template_constant() -> None:
-    with patch("subprocess.run") as mock_run:
-        mock_run.return_value = _completed("https://example.com\n")
-        trigger_bugbot_module.trigger_bugbot(owner="acme", repo="widget", number=42)
-    invoked_argv = mock_run.call_args[0][0]
-    expected_repo_arg = trigger_bugbot_module.GH_REPO_ARG_TEMPLATE.format(
-        owner="acme", repo="widget"
-    )
-    assert expected_repo_arg == "acme/widget"
-    repo_flag_index = invoked_argv.index("--repo")
-    assert invoked_argv[repo_flag_index + 1] == expected_repo_arg