claude-dev-env 1.34.1 → 1.36.0

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

@@ -0,0 +1,69 @@
+"""Tests for mark_pr_ready.
+
+Covers:
+- gh pr ready is invoked with the PR number and --repo flag
+- 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 / "mark_pr_ready.py"
+    spec = importlib.util.spec_from_file_location("mark_pr_ready", 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
+
+
+mark_pr_ready_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_ready_with_number_and_repo() -> None:
+    with patch("subprocess.run") as mock_run:
+        mock_run.return_value = _completed('Pull request "#42" is marked as ready')
+        mark_pr_ready_module.mark_pr_ready(owner="acme", repo="widget", number=42)
+    invoked_argv = mock_run.call_args[0][0]
+    assert invoked_argv[0:3] == ["gh", "pr", "ready"]
+    assert "42" in invoked_argv
+    assert "--repo" in invoked_argv
+    assert "acme/widget" in invoked_argv
+
+
+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):
+            mark_pr_ready_module.mark_pr_ready(owner="acme", repo="widget", number=42)
+
+
+def test_should_render_repo_arg_via_named_template_constant() -> None:
+    with patch("subprocess.run") as mock_run:
+        mock_run.return_value = _completed("ok\n")
+        mark_pr_ready_module.mark_pr_ready(owner="acme", repo="widget", number=42)
+    invoked_argv = mock_run.call_args[0][0]
+    expected_repo_arg = mark_pr_ready_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

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

@@ -0,0 +1,236 @@
+"""Tests for open_followup_copilot_pr.
+
+Covers:
+- branch name follows COPILOT_FOLLOWUP_BRANCH_TEMPLATE with the short SHA
+- subprocess sequence: gh pr view (base ref) -> git fetch -> git switch -c -> git push -> gh pr create
+- gh pr create uses --draft, --base, --head, --title, --body-file (per gh-body-file rule)
+- the returned PR URL is the trimmed stdout from gh pr create
+- 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 / "open_followup_copilot_pr.py"
+    spec = importlib.util.spec_from_file_location(
+        "open_followup_copilot_pr", 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
+
+
+open_followup_copilot_pr_module = _load_module()
+
+
+def _completed(stdout: str) -> subprocess.CompletedProcess:
+    process = MagicMock(spec=subprocess.CompletedProcess)
+    process.stdout = stdout
+    process.returncode = 0
+    return process
+
+
+def _scripted_subprocess_runs(
+    *,
+    base_ref_payload: str,
+    new_pr_url: str,
+) -> list[subprocess.CompletedProcess]:
+    return [
+        _completed(base_ref_payload),
+        _completed(""),
+        _completed(""),
+        _completed(""),
+        _completed(new_pr_url),
+    ]
+
+
+def test_should_build_branch_name_from_parent_number_and_short_sha(
+    tmp_path: Path,
+) -> None:
+    findings_file = tmp_path / "findings.md"
+    findings_file.write_text("- Item 1\n", encoding="utf-8")
+    payload_sequence = _scripted_subprocess_runs(
+        base_ref_payload=json.dumps({"baseRefName": "main"}),
+        new_pr_url="https://github.com/acme/widget/pull/313\n",
+    )
+    with patch("subprocess.run", side_effect=payload_sequence) as mock_run:
+        open_followup_copilot_pr_module.open_followup_copilot_pr(
+            owner="acme",
+            repo="widget",
+            parent_number=312,
+            head="abc12345deadbeefcafe",
+            findings_file=findings_file,
+        )
+    git_switch_argv = mock_run.call_args_list[2][0][0]
+    expected_branch = "chore/copilot-followup-312-abc12345"
+    assert expected_branch in git_switch_argv
+
+
+def test_should_invoke_subprocess_call_sequence_in_documented_order(
+    tmp_path: Path,
+) -> None:
+    findings_file = tmp_path / "findings.md"
+    findings_file.write_text("- Item\n", encoding="utf-8")
+    payload_sequence = _scripted_subprocess_runs(
+        base_ref_payload=json.dumps({"baseRefName": "main"}),
+        new_pr_url="https://github.com/acme/widget/pull/313\n",
+    )
+    with patch("subprocess.run", side_effect=payload_sequence) as mock_run:
+        open_followup_copilot_pr_module.open_followup_copilot_pr(
+            owner="acme",
+            repo="widget",
+            parent_number=312,
+            head="abc12345deadbeefcafe",
+            findings_file=findings_file,
+        )
+    invoked_command_sequence = [
+        each_call[0][0] for each_call in mock_run.call_args_list
+    ]
+    assert invoked_command_sequence[0][0:3] == ["gh", "pr", "view"]
+    assert invoked_command_sequence[1][0:2] == ["git", "fetch"]
+    assert invoked_command_sequence[2][0:3] == ["git", "switch", "-c"]
+    assert invoked_command_sequence[3][0:2] == ["git", "push"]
+    assert invoked_command_sequence[4][0:3] == ["gh", "pr", "create"]
+
+
+def test_should_invoke_gh_pr_create_with_draft_and_body_file_flags(
+    tmp_path: Path,
+) -> None:
+    findings_file = tmp_path / "findings.md"
+    findings_file.write_text("- Finding A\n", encoding="utf-8")
+    payload_sequence = _scripted_subprocess_runs(
+        base_ref_payload=json.dumps({"baseRefName": "develop"}),
+        new_pr_url="https://github.com/acme/widget/pull/444\n",
+    )
+    with patch("subprocess.run", side_effect=payload_sequence) as mock_run:
+        open_followup_copilot_pr_module.open_followup_copilot_pr(
+            owner="acme",
+            repo="widget",
+            parent_number=312,
+            head="abc12345deadbeef",
+            findings_file=findings_file,
+        )
+    pr_create_argv = mock_run.call_args_list[4][0][0]
+    assert pr_create_argv[0:3] == ["gh", "pr", "create"]
+    assert "--draft" in pr_create_argv
+    assert "--base" in pr_create_argv
+    assert "develop" in pr_create_argv
+    assert "--head" in pr_create_argv
+    assert "--title" in pr_create_argv
+    assert "--body-file" in pr_create_argv
+    body_file_argv = pr_create_argv[pr_create_argv.index("--body-file") + 1]
+    assert body_file_argv == str(findings_file)
+
+
+def test_should_render_pr_title_via_named_template_constant(tmp_path: Path) -> None:
+    findings_file = tmp_path / "findings.md"
+    findings_file.write_text("- Finding\n", encoding="utf-8")
+    payload_sequence = _scripted_subprocess_runs(
+        base_ref_payload=json.dumps({"baseRefName": "main"}),
+        new_pr_url="https://github.com/acme/widget/pull/513\n",
+    )
+    with patch("subprocess.run", side_effect=payload_sequence) as mock_run:
+        open_followup_copilot_pr_module.open_followup_copilot_pr(
+            owner="acme",
+            repo="widget",
+            parent_number=312,
+            head="abc12345deadbeef",
+            findings_file=findings_file,
+        )
+    pr_create_argv = mock_run.call_args_list[4][0][0]
+    title_index = pr_create_argv.index("--title")
+    title_value = pr_create_argv[title_index + 1]
+    assert title_value == "chore: address Copilot findings from PR #312"
+
+
+def test_should_return_trimmed_pr_url(tmp_path: Path) -> None:
+    findings_file = tmp_path / "findings.md"
+    findings_file.write_text("- Finding\n", encoding="utf-8")
+    payload_sequence = _scripted_subprocess_runs(
+        base_ref_payload=json.dumps({"baseRefName": "main"}),
+        new_pr_url="  https://github.com/acme/widget/pull/313  \n",
+    )
+    with patch("subprocess.run", side_effect=payload_sequence):
+        new_pr_url = open_followup_copilot_pr_module.open_followup_copilot_pr(
+            owner="acme",
+            repo="widget",
+            parent_number=312,
+            head="abc12345deadbeef",
+            findings_file=findings_file,
+        )
+    assert new_pr_url == "https://github.com/acme/widget/pull/313"
+
+
+def test_should_pass_repo_arg_to_gh_pr_view_for_base_ref(tmp_path: Path) -> None:
+    findings_file = tmp_path / "findings.md"
+    findings_file.write_text("- Finding\n", encoding="utf-8")
+    payload_sequence = _scripted_subprocess_runs(
+        base_ref_payload=json.dumps({"baseRefName": "main"}),
+        new_pr_url="https://github.com/acme/widget/pull/313\n",
+    )
+    with patch("subprocess.run", side_effect=payload_sequence) as mock_run:
+        open_followup_copilot_pr_module.open_followup_copilot_pr(
+            owner="acme",
+            repo="widget",
+            parent_number=312,
+            head="abc12345deadbeef",
+            findings_file=findings_file,
+        )
+    pr_view_argv = mock_run.call_args_list[0][0][0]
+    assert pr_view_argv[0:3] == ["gh", "pr", "view"]
+    assert "--repo" in pr_view_argv
+    repo_arg_value = pr_view_argv[pr_view_argv.index("--repo") + 1]
+    assert repo_arg_value == "acme/widget"
+
+
+def test_should_pass_repo_arg_to_gh_pr_create_for_followup_pr(
+    tmp_path: Path,
+) -> None:
+    findings_file = tmp_path / "findings.md"
+    findings_file.write_text("- Finding\n", encoding="utf-8")
+    payload_sequence = _scripted_subprocess_runs(
+        base_ref_payload=json.dumps({"baseRefName": "main"}),
+        new_pr_url="https://github.com/acme/widget/pull/313\n",
+    )
+    with patch("subprocess.run", side_effect=payload_sequence) as mock_run:
+        open_followup_copilot_pr_module.open_followup_copilot_pr(
+            owner="acme",
+            repo="widget",
+            parent_number=312,
+            head="abc12345deadbeef",
+            findings_file=findings_file,
+        )
+    pr_create_argv = mock_run.call_args_list[4][0][0]
+    assert pr_create_argv[0:3] == ["gh", "pr", "create"]
+    assert "--repo" in pr_create_argv
+    repo_arg_value = pr_create_argv[pr_create_argv.index("--repo") + 1]
+    assert repo_arg_value == "acme/widget"
+
+
+def test_should_raise_when_subprocess_fails(tmp_path: Path) -> None:
+    findings_file = tmp_path / "findings.md"
+    findings_file.write_text("- Finding\n", encoding="utf-8")
+    failure = subprocess.CalledProcessError(
+        returncode=1, cmd=["gh"], stderr="auth failure"
+    )
+    with patch("subprocess.run", side_effect=failure):
+        with pytest.raises(subprocess.CalledProcessError):
+            open_followup_copilot_pr_module.open_followup_copilot_pr(
+                owner="acme",
+                repo="widget",
+                parent_number=312,
+                head="abc12345",
+                findings_file=findings_file,
+            )

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

@@ -0,0 +1,195 @@
+"""Tests for post-bugbot-run PowerShell helpers.
+
+Covers Resolve-InvocationMode / Build-GhArgumentList for URL, owner/repo#N, and
+-Repository/-Number forms (Windows Bugbot re-trigger argv construction).
+"""
+
+from __future__ import annotations
+
+import json
+import os
+import subprocess
+import sys
+from pathlib import Path
+
+import pytest
+
+
+def _run_powershell(*, expression: str) -> str:
+    helpers = Path(__file__).resolve().parent / "post-bugbot-run.helpers.ps1"
+    command = (
+        f". '{helpers}'; "
+        + expression
+        + " | ConvertTo-Json -Compress -Depth 5"
+    )
+    completed = subprocess.run(
+        ["pwsh", "-NoProfile", "-NonInteractive", "-ExecutionPolicy", "Bypass", "-Command", command],
+        capture_output=True,
+        text=True,
+        encoding="utf-8",
+        errors="replace",
+        check=False,
+    )
+    if completed.returncode != 0:
+        raise AssertionError(
+            f"pwsh failed ({completed.returncode}): stderr={completed.stderr!r} stdout={completed.stdout!r}"
+        )
+    return completed.stdout.strip()
+
+
+def _argv_for(*, pull: str, repository: str, number: int, body: str) -> list[str]:
+    pull_esc = pull.replace("'", "''")
+    repository_esc = repository.replace("'", "''")
+    body_esc = body.replace("'", "''")
+    expression = (
+        f"$i = Resolve-InvocationMode -PullRequestInput '{pull_esc}' "
+        f"-RepositoryInput '{repository_esc}' -NumberInput {int(number)}; "
+        f"@(Build-GhArgumentList -Invocation $i -BodyFilePath '{body_esc}')"
+    )
+    raw = _run_powershell(expression=expression)
+    return json.loads(raw)
+
+
+def test_should_build_arguments_for_https_pull_url() -> None:
+    argv = _argv_for(
+        pull="https://github.com/acme/widget/pull/42",
+        repository="",
+        number=0,
+        body=r"C:\\temp\\body.md",
+    )
+    assert argv == [
+        "pr",
+        "comment",
+        "https://github.com/acme/widget/pull/42",
+        "--body-file",
+        r"C:\\temp\\body.md",
+    ]
+
+
+def test_should_build_arguments_for_owner_repo_hash_form() -> None:
+    argv = _argv_for(
+        pull="acme/widget#7",
+        repository="",
+        number=0,
+        body=r"D:\\x\\f.md",
+    )
+    assert argv == ["pr", "comment", "7", "-R", "acme/widget", "--body-file", r"D:\\x\\f.md"]
+
+
+def test_should_build_arguments_for_repository_and_number_parameters() -> None:
+    argv = _argv_for(
+        pull="",
+        repository="jl-cmd/claude-code-config",
+        number=331,
+        body=r"E:\\y\\z.md",
+    )
+    assert argv == [
+        "pr",
+        "comment",
+        "331",
+        "-R",
+        "jl-cmd/claude-code-config",
+        "--body-file",
+        r"E:\\y\\z.md",
+    ]
+
+
+def test_should_fail_when_number_without_repository() -> None:
+    helpers = Path(__file__).resolve().parent / "post-bugbot-run.helpers.ps1"
+    command = (
+        f". '{helpers}'; "
+        "try { Resolve-InvocationMode -PullRequestInput '' -RepositoryInput '' -NumberInput 3 } "
+        "catch { $_.Exception.Message }"
+    )
+    completed = subprocess.run(
+        ["pwsh", "-NoProfile", "-NonInteractive", "-ExecutionPolicy", "Bypass", "-Command", command],
+        capture_output=True,
+        text=True,
+        encoding="utf-8",
+        errors="replace",
+        check=False,
+    )
+    assert completed.returncode == 0
+    message = completed.stdout.strip()
+    assert "Repository" in message
+
+
+def test_should_reject_pull_url_with_trailing_junk() -> None:
+    helpers = Path(__file__).resolve().parent / "post-bugbot-run.helpers.ps1"
+    pull = "https://github.com/acme/widget/pull/42extra"
+    command = (
+        f". '{helpers}'; "
+        f"$i = Resolve-InvocationMode -PullRequestInput '{pull}' -RepositoryInput '' -NumberInput 0; "
+        r"try { Build-GhArgumentList -Invocation $i -BodyFilePath 'C:\\t\\b.md' } "
+        "catch { $_.Exception.Message }"
+    )
+    completed = subprocess.run(
+        ["pwsh", "-NoProfile", "-NonInteractive", "-ExecutionPolicy", "Bypass", "-Command", command],
+        capture_output=True,
+        text=True,
+        encoding="utf-8",
+        errors="replace",
+        check=False,
+    )
+    assert completed.returncode == 0
+    assert "Unrecognized PullRequest" in completed.stdout
+
+
+def test_should_fail_for_unrecognized_pull_string() -> None:
+    helpers = Path(__file__).resolve().parent / "post-bugbot-run.helpers.ps1"
+    pull = "not-a-valid-pr-reference"
+    command = (
+        f". '{helpers}'; "
+        f"$i = Resolve-InvocationMode -PullRequestInput '{pull}' -RepositoryInput '' -NumberInput 0; "
+        r"try { Build-GhArgumentList -Invocation $i -BodyFilePath 'C:\\t\\b.md' } "
+        "catch { $_.Exception.Message }"
+    )
+    completed = subprocess.run(
+        ["pwsh", "-NoProfile", "-NonInteractive", "-ExecutionPolicy", "Bypass", "-Command", command],
+        capture_output=True,
+        text=True,
+        encoding="utf-8",
+        errors="replace",
+        check=False,
+    )
+    assert completed.returncode == 0
+    assert "Unrecognized PullRequest" in completed.stdout
+
+
+def test_full_script_removes_temp_files_when_gh_stub_succeeds(tmp_path: Path) -> None:
+    scripts_dir = Path(__file__).resolve().parent
+    script_path = scripts_dir / "post-bugbot-run.ps1"
+    stub_bin_dir = tmp_path / "gh_stub_bin"
+    stub_bin_dir.mkdir()
+    gh_cmd = stub_bin_dir / "gh.cmd"
+    gh_cmd.write_text("@echo off\r\nexit /b 0\r\n", encoding="utf-8")
+    env = dict(os.environ)
+    env["PATH"] = str(stub_bin_dir) + os.pathsep + env.get("PATH", "")
+    completed = subprocess.run(
+        [
+            "pwsh",
+            "-NoProfile",
+            "-NonInteractive",
+            "-ExecutionPolicy",
+            "Bypass",
+            "-File",
+            str(script_path),
+            "acme/widget#9",
+        ],
+        capture_output=True,
+        text=True,
+        encoding="utf-8",
+        errors="replace",
+        env=env,
+        check=False,
+    )
+    assert completed.returncode == 0, (completed.stdout, completed.stderr)
+
+def test_post_bugbot_run_script_finally_removes_temp_paths() -> None:
+    script_text = (Path(__file__).resolve().parent / "post-bugbot-run.ps1").read_text(
+        encoding="utf-8"
+    )
+    assert "finally" in script_text
+    assert "Remove-Item" in script_text
+    assert "body_file_path" in script_text
+    assert "scratch_temp_path" in script_text

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

@@ -0,0 +1,159 @@
+"""Tests for reply_to_inline_comment.
+
+Covers:
+- gh api -X POST is invoked against the inline-comments replies endpoint
+- the reply body comes from a file via gh's `body=@<path>` form (per gh-body-file rule)
+- the reply id from gh's JSON output is returned
+- 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 / "reply_to_inline_comment.py"
+    spec = importlib.util.spec_from_file_location(
+        "reply_to_inline_comment", 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
+
+
+reply_to_inline_comment_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_api_post_against_replies_endpoint(tmp_path: Path) -> None:
+    body_file = tmp_path / "reply.md"
+    body_file.write_text("Confirmed and fixed.\n", encoding="utf-8")
+    with patch("subprocess.run") as mock_run:
+        mock_run.return_value = _completed(json.dumps({"id": 7777}))
+        reply_to_inline_comment_module.reply_to_inline_comment(
+            owner="acme",
+            repo="widget",
+            number=42,
+            comment_id=12345,
+            body_file=body_file,
+        )
+    invoked_argv = mock_run.call_args[0][0]
+    assert invoked_argv[0:2] == ["gh", "api"]
+    assert "-X" in invoked_argv
+    assert "POST" in invoked_argv
+    assert "repos/acme/widget/pulls/42/comments/12345/replies" in invoked_argv
+
+
+def test_should_pass_body_via_field_at_path_form(tmp_path: Path) -> None:
+    body_file = tmp_path / "reply.md"
+    body_file.write_text("Confirmed and fixed.\n", encoding="utf-8")
+    with patch("subprocess.run") as mock_run:
+        mock_run.return_value = _completed(json.dumps({"id": 7777}))
+        reply_to_inline_comment_module.reply_to_inline_comment(
+            owner="acme",
+            repo="widget",
+            number=42,
+            comment_id=12345,
+            body_file=body_file,
+        )
+    invoked_argv = mock_run.call_args[0][0]
+    assert "-F" in invoked_argv
+    field_value = invoked_argv[invoked_argv.index("-F") + 1]
+    assert field_value.startswith("body=@")
+    assert str(body_file) in field_value
+
+
+def test_should_return_reply_id_from_gh_response(tmp_path: Path) -> None:
+    body_file = tmp_path / "reply.md"
+    body_file.write_text("Reply text\n", encoding="utf-8")
+    with patch("subprocess.run") as mock_run:
+        mock_run.return_value = _completed(json.dumps({"id": 7777, "body": "..."}))
+        reply_id = reply_to_inline_comment_module.reply_to_inline_comment(
+            owner="acme",
+            repo="widget",
+            number=42,
+            comment_id=12345,
+            body_file=body_file,
+        )
+    assert reply_id == 7777
+
+
+def test_should_raise_when_gh_subprocess_fails(tmp_path: Path) -> None:
+    body_file = tmp_path / "reply.md"
+    body_file.write_text("Reply text\n", encoding="utf-8")
+    failure = subprocess.CalledProcessError(
+        returncode=1, cmd=["gh"], stderr="auth failure"
+    )
+    with patch("subprocess.run", side_effect=failure):
+        with pytest.raises(subprocess.CalledProcessError):
+            reply_to_inline_comment_module.reply_to_inline_comment(
+                owner="acme",
+                repo="widget",
+                number=42,
+                comment_id=12345,
+                body_file=body_file,
+            )
+
+
+def test_should_build_body_field_value_from_named_prefix_constant(
+    tmp_path: Path,
+) -> None:
+    body_file = tmp_path / "reply.md"
+    body_file.write_text("Reply text\n", encoding="utf-8")
+    with patch("subprocess.run") as mock_run:
+        mock_run.return_value = _completed(json.dumps({"id": 7777}))
+        reply_to_inline_comment_module.reply_to_inline_comment(
+            owner="acme",
+            repo="widget",
+            number=42,
+            comment_id=12345,
+            body_file=body_file,
+        )
+    invoked_argv = mock_run.call_args[0][0]
+    field_value = invoked_argv[invoked_argv.index("-F") + 1]
+    expected_prefix = (
+        reply_to_inline_comment_module.GH_FIELD_BODY_AT_PREFIX
+    )
+    assert expected_prefix == "body=@"
+    assert field_value == f"{expected_prefix}{body_file}"
+
+
+def test_should_extract_int_id_from_mixed_typed_response_payload(
+    tmp_path: Path,
+) -> None:
+    body_file = tmp_path / "reply.md"
+    body_file.write_text("Reply text\n", encoding="utf-8")
+    response_with_string_and_object_fields = json.dumps(
+        {
+            "id": 7777,
+            "body": "Reply body text",
+            "user": {"login": "octocat", "id": 1},
+            "created_at": "2026-05-02T12:00:00Z",
+        }
+    )
+    with patch("subprocess.run") as mock_run:
+        mock_run.return_value = _completed(response_with_string_and_object_fields)
+        reply_id = reply_to_inline_comment_module.reply_to_inline_comment(
+            owner="acme",
+            repo="widget",
+            number=42,
+            comment_id=12345,
+            body_file=body_file,
+        )
+    assert reply_id == 7777