claude-dev-env 1.34.1 → 1.36.0

package/skills/pr-converge/scripts/README.md

@@ -0,0 +1,145 @@
+# pr-converge scripts
+
+Thin Python wrappers around the gh CLI calls the skill makes per tick. Centralizing them lets the skill body reference one script path per action and keeps the gh-paginate and gh-body-file rules enforced in one place.
+
+Scripts that target a specific repository are invoked as `python "${CLAUDE_SKILL_DIR}/scripts/<name>.py" --owner OWNER --repo REPO --number NUMBER ...`. `view_pr_context.py` relies on `gh`'s default repository context and takes no `--owner` / `--repo` / `--number` flags. Scripts return non-zero on subprocess failure and surface gh's stderr through `subprocess.CalledProcessError`.
+
+## Scripts
+
+### `view_pr_context.py`
+
+Returns the per-tick PR context as JSON.
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/view_pr_context.py"
+```
+
+Output: `{"number", "url", "headRefOid", "baseRefName", "headRefName", "isDraft"}`. Wraps `gh pr view --json number,url,headRefOid,baseRefName,headRefName,isDraft`.
+
+### `fetch_bugbot_reviews.py`
+
+Fetches every Cursor Bugbot review on the PR newest-first, classified per body content.
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/fetch_bugbot_reviews.py" \
+  --owner <OWNER> --repo <REPO> --number <NUMBER>
+```
+
+Output: JSON array of `{review_id, commit_id, submitted_at, body, classification}` where `classification` is `"dirty"` (body matches `Cursor Bugbot has reviewed your changes and found <N> potential issue`) or `"clean"`. Uses `--paginate --slurp` and flattens pages in Python — required by `../../../rules/gh-paginate.md` because `gh --paginate --jq` runs the filter per-page (gh CLI #10459).
+
+### `fetch_bugbot_inline_comments.py`
+
+Fetches unaddressed Cursor Bugbot inline comments for the **newest submitted Bugbot review** on the requested ``--commit`` SHA (matches ``pull_request_review_id`` to the review returned by ``fetch_bugbot_reviews.py`` so stale inline threads from an older Bugbot review on the same SHA are ignored).
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/fetch_bugbot_inline_comments.py" \
+  --owner <OWNER> --repo <REPO> --number <NUMBER> --commit <CURRENT_HEAD>
+```
+
+Output: JSON array of `{comment_id, commit_id, path, line, body}`. Uses the same `--paginate --slurp` pattern as `fetch_bugbot_reviews.py`.
+
+### `resolve_pr_head.py`
+
+Returns the current HEAD SHA of the PR. Wraps the single-object endpoint `repos/<owner>/<repo>/pulls/<number>` which is not paginated, so `gh`'s built-in `--jq .head.sha` is safe here (see "Single-object endpoints" in `../../../rules/gh-paginate.md`).
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/resolve_pr_head.py" \
+  --owner <OWNER> --repo <REPO> --number <NUMBER>
+```
+
+Output: the SHA on stdout, trailing newline.
+
+### `trigger_bugbot.py`
+
+Posts the literal `bugbot run` re-trigger comment via `gh pr comment --body-file` (per `../../../rules/gh-body-file.md` — passing the body inline can corrupt backticks). Writes and removes the temp body file internally.
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/trigger_bugbot.py" \
+  --owner <OWNER> --repo <REPO> --number <NUMBER>
+```
+
+Output: the comment URL from gh on stdout.
+
+### `mark_pr_ready.py`
+
+Marks a draft PR as ready for review. Convergence action invoked when both bugbot and bugteam are clean against the same HEAD.
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/mark_pr_ready.py" \
+  --owner <OWNER> --repo <REPO> --number <NUMBER>
+```
+
+### `reply_to_inline_comment.py`
+
+Posts an inline reply to a PR review comment. Reply body is sourced from a caller-supplied file via `gh api ... -F body=@<path>` (per `../../../rules/gh-body-file.md`).
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/reply_to_inline_comment.py" \
+  --owner <OWNER> --repo <REPO> --number <NUMBER> \
+  --comment-id <COMMENT_ID> --body-file <PATH_TO_REPLY_MD>
+```
+
+Output: the new reply id from gh's JSON response, on stdout.
+
+### `check_pr_mergeability.py`
+
+Returns the mergeability state of the current PR as JSON. Wraps `gh pr view --json mergeable,mergeStateStatus,headRefOid` (single-object endpoint — no pagination needed). Used by the convergence gate to detect base-branch conflicts (`mergeStateStatus == "DIRTY"` / `mergeable == "CONFLICTING"`) before flipping the PR ready.
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/check_pr_mergeability.py"
+```
+
+Output: `{"mergeable", "mergeStateStatus", "headRefOid"}`.
+
+### `fetch_copilot_reviews.py`
+
+Fetches every Copilot reviewer (`copilot-pull-request-reviewer[bot]`) review on the PR newest-first, classified per the review's `state` field.
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/fetch_copilot_reviews.py" \
+  --owner <OWNER> --repo <REPO> --number <NUMBER>
+```
+
+Output: JSON array of `{review_id, commit_id, submitted_at, state, body, classification}` where `classification` is `"clean"` for `state == "APPROVED"`, `"dirty"` for `state == "CHANGES_REQUESTED"`, and `"dirty"` for `state == "COMMENTED"` with a non-empty body. Uses `--paginate --slurp` and flattens pages in Python — required by `../../../rules/gh-paginate.md`.
+
+### `fetch_copilot_inline_comments.py`
+
+Fetches unaddressed Copilot inline comments for the **newest submitted Copilot review** on the requested ``--commit`` SHA (matches ``pull_request_review_id`` to the review returned by ``fetch_copilot_reviews.py`` so stale inline threads from an older Copilot review on the same SHA are ignored).
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/fetch_copilot_inline_comments.py" \
+  --owner <OWNER> --repo <REPO> --number <NUMBER> --commit <CURRENT_HEAD>
+```
+
+Output: JSON array of `{comment_id, commit_id, path, line, body}`. Uses the same `--paginate --slurp` pattern as `fetch_copilot_reviews.py`.
+
+### `request_copilot_review.py`
+
+Requests a Copilot review on the current PR via `gh api -X POST repos/{owner}/{repo}/pulls/{number}/requested_reviewers -f 'reviewers[]=copilot-pull-request-reviewer[bot]'`. The `[bot]` suffix is load-bearing per `../../copilot-review/SKILL.md` — `Copilot`, `copilot`, and `github-copilot` all silently no-op.
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/request_copilot_review.py" \
+  --owner <OWNER> --repo <REPO> --number <NUMBER>
+```
+
+Output: none on success (gh's stdout is suppressed); `subprocess.CalledProcessError` on failure.
+
+### `open_followup_copilot_pr.py`
+
+Opens a follow-up draft PR addressing Copilot findings from the parent PR. Subprocess sequence: resolve parent's `baseRefName` → `git fetch origin <head_sha>` → `git switch -c <new_branch> <head_sha>` → `git push -u origin <new_branch>` → `gh pr create --draft --base <base_ref> --head <new_branch> --title <...> --body-file <findings_file>` (per `../../../rules/gh-body-file.md`). Branch name format: `chore/copilot-followup-{parent_number}-{short_sha}`.
+
+```bash
+python "${CLAUDE_SKILL_DIR}/scripts/open_followup_copilot_pr.py" \
+  --owner <OWNER> --repo <REPO> --parent-number <PARENT_NUMBER> \
+  --head <HEAD_SHA> --findings-file <PATH_TO_FINDINGS_MD>
+```
+
+Output: the new PR URL on stdout, trimmed.
+
+## Tests
+
+Each script has a sibling `test_<name>.py`. Run them all with:
+
+```bash
+python -m pytest packages/claude-dev-env/skills/pr-converge/scripts/ -v
+```

package/skills/pr-converge/scripts/caller-window-pid.ps1

@@ -0,0 +1,86 @@
+[CmdletBinding()]
+param()
+
+$DESKTOP_SHELL_TERMINATOR_NAME = 'explorer'
+$MAXIMUM_PARENT_WALK_DEPTH     = 24
+
+if (-not ('CallerWindowPid.Win32ForegroundWindowQuery' -as [type])) {
+    Add-Type -Namespace 'CallerWindowPid' -Name 'Win32ForegroundWindowQuery' -MemberDefinition @'
+        [System.Runtime.InteropServices.DllImport("user32.dll")]
+        public static extern System.IntPtr GetForegroundWindow();
+        [System.Runtime.InteropServices.DllImport("user32.dll")]
+        public static extern uint GetWindowThreadProcessId(System.IntPtr hWnd, out uint processId);
+'@
+}
+
+function Get-ParentProcessId {
+    param([int]$ChildProcessId)
+    try {
+        return (Get-CimInstance -ClassName Win32_Process -Filter "ProcessId=$ChildProcessId" -ErrorAction Stop).ParentProcessId
+    } catch {
+        return 0
+    }
+}
+
+function Resolve-NearestGuiAncestorPid {
+    param([int]$StartingProcessId)
+
+    $visited_process_ids = @{}
+    $current_process_id  = $StartingProcessId
+    $walk_depth          = 0
+
+    while ($walk_depth -lt $MAXIMUM_PARENT_WALK_DEPTH) {
+        $walk_depth++
+        if ($visited_process_ids.ContainsKey($current_process_id)) {
+            return $null
+        }
+        $visited_process_ids[$current_process_id] = $true
+
+        $parent_process_id = Get-ParentProcessId -ChildProcessId $current_process_id
+        if (-not $parent_process_id -or $parent_process_id -eq 0) {
+            return $null
+        }
+
+        try {
+            $parent_process = Get-Process -Id $parent_process_id -ErrorAction Stop
+        } catch {
+            return $null
+        }
+
+        if ($parent_process.ProcessName -eq $DESKTOP_SHELL_TERMINATOR_NAME) {
+            return $null
+        }
+
+        if ($parent_process.MainWindowHandle -ne [IntPtr]::Zero) {
+            return $parent_process.Id
+        }
+
+        $current_process_id = $parent_process_id
+    }
+
+    return $null
+}
+
+function Resolve-ForegroundWindowPid {
+    $foreground_window_handle = [CallerWindowPid.Win32ForegroundWindowQuery]::GetForegroundWindow()
+    if ($foreground_window_handle -eq [IntPtr]::Zero) {
+        return $null
+    }
+    $foreground_window_pid = [uint32]0
+    [void][CallerWindowPid.Win32ForegroundWindowQuery]::GetWindowThreadProcessId($foreground_window_handle, [ref]$foreground_window_pid)
+    if ($foreground_window_pid -eq 0) {
+        return $null
+    }
+    return [int]$foreground_window_pid
+}
+
+$resolved_pid = Resolve-NearestGuiAncestorPid -StartingProcessId $PID
+if ($null -eq $resolved_pid) {
+    $resolved_pid = Resolve-ForegroundWindowPid
+}
+if ($null -eq $resolved_pid) {
+    Write-Error "Could not resolve a GUI process from PID $PID or the foreground window."
+    exit 1
+}
+
+Write-Output $resolved_pid

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

@@ -0,0 +1,79 @@
+"""Resolve the per-tick mergeability state of an explicitly-targeted PR.
+
+Wraps ``gh pr view <number> --repo <owner>/<repo> --json mergeable,mergeStateStatus,headRefOid``
+so the skill body emits one script invocation. Single-object endpoint - no
+pagination. Explicit ``--owner``/``--repo``/``--number`` targeting matches every
+sibling convergence-gate script (``fetch_*_reviews.py``,
+``fetch_*_inline_comments.py``, ``request_copilot_review.py``,
+``mark_pr_ready.py``); under multi-PR orchestration or after
+``open_followup_copilot_pr.py`` switches the checkout, the gate is guaranteed
+to query the intended PR rather than whichever PR the current git context
+points at.
+
+The returned dict gates pr-converge's mark-ready step against PRs whose base
+branch state is DIRTY (conflicts) or otherwise non-CLEAN.
+"""
+
+import argparse
+import json
+import subprocess
+import sys
+from pathlib import Path
+
+if str(Path(__file__).resolve().parent) not in sys.path:
+    sys.path.insert(0, str(Path(__file__).resolve().parent))
+
+from evict_cached_config_modules import evict_cached_config_modules
+
+evict_cached_config_modules()
+
+from config.pr_converge_constants import GH_REPO_ARG_TEMPLATE, MERGEABILITY_FIELDS
+
+
+def check_pr_mergeability(
+    *,
+    owner: str,
+    repo: str,
+    number: int,
+) -> dict[str, object]:
+    """Return ``{mergeable, mergeStateStatus, headRefOid}`` from ``gh pr view`` for the targeted PR."""
+    repo_arg = GH_REPO_ARG_TEMPLATE.format(owner=owner, repo=repo)
+    gh_command: list[str] = [
+        "gh",
+        "pr",
+        "view",
+        str(number),
+        "--repo",
+        repo_arg,
+        "--json",
+        MERGEABILITY_FIELDS,
+    ]
+    completed = subprocess.run(
+        gh_command,
+        capture_output=True,
+        check=True,
+        text=True,
+        encoding="utf-8",
+        errors="replace",
+    )
+    return json.loads(completed.stdout)
+
+
+def main() -> int:
+    parser = argparse.ArgumentParser(description=__doc__)
+    parser.add_argument("--owner", required=True)
+    parser.add_argument("--repo", required=True)
+    parser.add_argument("--number", required=True, type=int)
+    parsed_arguments = parser.parse_args()
+    mergeability_state = check_pr_mergeability(
+        owner=parsed_arguments.owner,
+        repo=parsed_arguments.repo,
+        number=parsed_arguments.number,
+    )
+    json.dump(mergeability_state, sys.stdout)
+    sys.stdout.write("\n")
+    return 0
+
+
+if __name__ == "__main__":
+    sys.exit(main())

package/skills/pr-converge/scripts/config/pr_converge_constants.py

@@ -0,0 +1,65 @@
+"""Configuration constants for the pr-converge skill scripts.
+
+Path templates accept ``str.format(**kwargs)`` substitution; bugbot strings
+match the literal phrasing the Cursor Bugbot reviewer emits.
+"""
+
+CURSOR_BOT_LOGIN: str = "cursor[bot]"
+
+COPILOT_REVIEWER_LOGIN: str = "copilot-pull-request-reviewer[bot]"
+
+COPILOT_REVIEWER_REQUEST_ID: str = COPILOT_REVIEWER_LOGIN
+
+COPILOT_CLEAN_REVIEW_STATE: str = "APPROVED"
+
+ALL_COPILOT_DIRTY_REVIEW_STATES: tuple[str, ...] = ("CHANGES_REQUESTED", "COMMENTED")
+
+COPILOT_SOFT_DIRTY_REVIEW_STATE: str = "COMMENTED"
+
+BUGBOT_DIRTY_BODY_REGEX: str = (
+    r"Cursor Bugbot has reviewed your changes and found \d+ potential issue"
+)
+
+GH_REVIEWS_PATH_TEMPLATE: str = (
+    "repos/{owner}/{repo}/pulls/{number}/reviews?per_page=100"
+)
+
+GH_INLINE_COMMENTS_PATH_TEMPLATE: str = (
+    "repos/{owner}/{repo}/pulls/{number}/comments?per_page=100"
+)
+
+GH_PR_OBJECT_PATH_TEMPLATE: str = "repos/{owner}/{repo}/pulls/{number}"
+
+GH_INLINE_COMMENT_REPLY_PATH_TEMPLATE: str = (
+    "repos/{owner}/{repo}/pulls/{number}/comments/{comment_id}/replies"
+)
+
+GH_REQUESTED_REVIEWERS_PATH_TEMPLATE: str = (
+    "repos/{owner}/{repo}/pulls/{number}/requested_reviewers"
+)
+
+GH_REQUESTED_REVIEWERS_FIELD_TEMPLATE: str = "reviewers[]={reviewer_id}"
+
+BUGBOT_RUN_TRIGGER_PHRASE: str = "bugbot run\n"
+
+BUGBOT_RUN_TEMPFILE_SUFFIX: str = ".md"
+
+BUGBOT_RUN_TEMPFILE_PREFIX: str = "pr-converge-bugbot-run-"
+
+PR_CONTEXT_FIELDS: str = "number,url,headRefOid,baseRefName,headRefName,isDraft"
+
+MERGEABILITY_FIELDS: str = "mergeable,mergeStateStatus,headRefOid"
+
+GH_FIELD_BODY_AT_PREFIX: str = "body=@"
+
+GH_REPO_ARG_TEMPLATE: str = "{owner}/{repo}"
+
+PR_BASE_REF_FIELDS: str = "baseRefName"
+
+COPILOT_FOLLOWUP_BRANCH_TEMPLATE: str = "chore/copilot-followup-{parent_number}-{short_sha}"
+
+COPILOT_FOLLOWUP_PR_TITLE_TEMPLATE: str = (
+    "chore: address Copilot findings from PR #{parent_number}"
+)
+
+COPILOT_FOLLOWUP_SHORT_SHA_LENGTH: int = 8

package/skills/pr-converge/scripts/config/test_pr_converge_constants.py

@@ -0,0 +1,176 @@
+"""Tests for pr_converge_constants.
+
+Verifies that path templates accept the documented format substitutions
+(owner, repo, number, comment_id) and the bugbot regex matches dirty review
+bodies but not clean ones.
+"""
+
+from __future__ import annotations
+
+import importlib.util
+import re
+from pathlib import Path
+from types import ModuleType
+
+
+def _load_module() -> ModuleType:
+    module_path = Path(__file__).parent / "pr_converge_constants.py"
+    spec = importlib.util.spec_from_file_location("pr_converge_constants", 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
+
+
+pr_converge_constants_module = _load_module()
+
+
+def test_reviews_path_template_accepts_owner_repo_number() -> None:
+    rendered = pr_converge_constants_module.GH_REVIEWS_PATH_TEMPLATE.format(
+        owner="acme", repo="widget", number=42
+    )
+    assert rendered == "repos/acme/widget/pulls/42/reviews?per_page=100"
+
+
+def test_inline_comments_path_template_accepts_owner_repo_number() -> None:
+    rendered = pr_converge_constants_module.GH_INLINE_COMMENTS_PATH_TEMPLATE.format(
+        owner="acme", repo="widget", number=42
+    )
+    assert rendered == "repos/acme/widget/pulls/42/comments?per_page=100"
+
+
+def test_pr_object_path_template_accepts_owner_repo_number() -> None:
+    rendered = pr_converge_constants_module.GH_PR_OBJECT_PATH_TEMPLATE.format(
+        owner="acme", repo="widget", number=42
+    )
+    assert rendered == "repos/acme/widget/pulls/42"
+
+
+def test_inline_comment_reply_path_template_accepts_all_substitutions() -> None:
+    rendered = (
+        pr_converge_constants_module.GH_INLINE_COMMENT_REPLY_PATH_TEMPLATE.format(
+            owner="acme", repo="widget", number=42, comment_id=12345
+        )
+    )
+    assert rendered == "repos/acme/widget/pulls/42/comments/12345/replies"
+
+
+def test_bugbot_dirty_body_regex_distinguishes_findings_from_clean_bodies() -> None:
+    dirty_body = "Cursor Bugbot has reviewed your changes and found 3 potential issues."
+    clean_body = "Bugbot reviewed your changes and found no new issues!"
+    compiled_pattern = re.compile(pr_converge_constants_module.BUGBOT_DIRTY_BODY_REGEX)
+    dirty_match = compiled_pattern.search(dirty_body)
+    assert dirty_match is not None
+    assert "found 3 potential issue" in dirty_match.group(0)
+    assert compiled_pattern.search(clean_body) is None
+
+
+def test_cursor_bot_login_matches_github_login_string() -> None:
+    assert pr_converge_constants_module.CURSOR_BOT_LOGIN == "cursor[bot]"
+
+
+def test_bugbot_run_trigger_phrase_ends_with_newline() -> None:
+    assert pr_converge_constants_module.BUGBOT_RUN_TRIGGER_PHRASE == "bugbot run\n"
+
+
+def test_pr_context_fields_lists_documented_field_names() -> None:
+    fields_arg = pr_converge_constants_module.PR_CONTEXT_FIELDS
+    for required_field in (
+        "number",
+        "url",
+        "headRefOid",
+        "baseRefName",
+        "headRefName",
+        "isDraft",
+    ):
+        assert required_field in fields_arg
+
+
+def test_gh_field_body_at_prefix_matches_gh_field_from_file_form() -> None:
+    assert pr_converge_constants_module.GH_FIELD_BODY_AT_PREFIX == "body=@"
+
+
+def test_gh_repo_arg_template_renders_owner_slash_repo() -> None:
+    rendered = pr_converge_constants_module.GH_REPO_ARG_TEMPLATE.format(
+        owner="acme", repo="widget"
+    )
+    assert rendered == "acme/widget"
+
+
+def test_copilot_reviewer_login_carries_bot_suffix() -> None:
+    assert (
+        pr_converge_constants_module.COPILOT_REVIEWER_LOGIN
+        == "copilot-pull-request-reviewer[bot]"
+    )
+
+
+def test_copilot_reviewer_request_id_reuses_login_constant() -> None:
+    request_id = pr_converge_constants_module.COPILOT_REVIEWER_REQUEST_ID
+    login = pr_converge_constants_module.COPILOT_REVIEWER_LOGIN
+    assert request_id == login
+    assert request_id is login
+
+
+def test_copilot_clean_review_state_is_approved() -> None:
+    assert pr_converge_constants_module.COPILOT_CLEAN_REVIEW_STATE == "APPROVED"
+
+
+def test_copilot_dirty_review_states_lists_changes_requested_and_commented() -> None:
+    dirty_states = pr_converge_constants_module.ALL_COPILOT_DIRTY_REVIEW_STATES
+    assert "CHANGES_REQUESTED" in dirty_states
+    assert "COMMENTED" in dirty_states
+
+
+def test_copilot_soft_dirty_review_state_is_commented() -> None:
+    assert pr_converge_constants_module.COPILOT_SOFT_DIRTY_REVIEW_STATE == "COMMENTED"
+
+
+def test_mergeability_fields_lists_required_field_names() -> None:
+    fields_arg = pr_converge_constants_module.MERGEABILITY_FIELDS
+    for required_field in ("mergeable", "mergeStateStatus", "headRefOid"):
+        assert required_field in fields_arg
+
+
+def test_requested_reviewers_path_template_accepts_owner_repo_number() -> None:
+    rendered = (
+        pr_converge_constants_module.GH_REQUESTED_REVIEWERS_PATH_TEMPLATE.format(
+            owner="acme", repo="widget", number=42
+        )
+    )
+    assert rendered == "repos/acme/widget/pulls/42/requested_reviewers"
+
+
+def test_requested_reviewers_field_template_accepts_reviewer_id() -> None:
+    rendered = (
+        pr_converge_constants_module.GH_REQUESTED_REVIEWERS_FIELD_TEMPLATE.format(
+            reviewer_id="copilot-pull-request-reviewer[bot]"
+        )
+    )
+    assert rendered == "reviewers[]=copilot-pull-request-reviewer[bot]"
+
+
+def test_pr_base_ref_fields_lists_base_ref_name() -> None:
+    assert "baseRefName" in pr_converge_constants_module.PR_BASE_REF_FIELDS
+
+
+def test_copilot_followup_branch_template_renders_parent_number_and_sha() -> None:
+    rendered = (
+        pr_converge_constants_module.COPILOT_FOLLOWUP_BRANCH_TEMPLATE.format(
+            parent_number=312, short_sha="abc12345"
+        )
+    )
+    assert rendered == "chore/copilot-followup-312-abc12345"
+
+
+def test_copilot_followup_pr_title_template_renders_parent_number() -> None:
+    rendered = (
+        pr_converge_constants_module.COPILOT_FOLLOWUP_PR_TITLE_TEMPLATE.format(
+            parent_number=312
+        )
+    )
+    assert rendered == "chore: address Copilot findings from PR #312"
+
+
+def test_copilot_followup_short_sha_length_is_eight() -> None:
+    assert pr_converge_constants_module.COPILOT_FOLLOWUP_SHORT_SHA_LENGTH == 8

package/skills/pr-converge/scripts/cursor-agents-continue-caller.cmd

@@ -0,0 +1,9 @@
+@echo off
+setlocal
+for /f "delims=" %%P in ('pwsh -NoProfile -ExecutionPolicy Bypass -File "%~dp0caller-window-pid.ps1"') do set CALLER_PID=%%P
+if "%CALLER_PID%"=="" (
+    echo [cursor-agents-continue-caller] Failed to resolve caller PID.
+    exit /b 1
+)
+call "%~dp0cursor-agents-continue.cmd" %CALLER_PID% --start-on
+endlocal

package/skills/pr-converge/scripts/cursor-agents-continue-stop-others.ps1

@@ -0,0 +1,16 @@
+[CmdletBinding()]
+param(
+    [Parameter(HelpMessage = 'Process id to keep (0 = kill every matching AutoHotkey instance).')]
+    [int] $KeepProcessId = 0
+)
+
+$scriptMarker = 'cursor-agents-continue.ahk'
+$all_processes = Get-CimInstance Win32_Process -Filter "Name='AutoHotkey64.exe'" |
+    Where-Object { $_.CommandLine -like "*$scriptMarker*" }
+
+foreach ($eachProcess in $all_processes) {
+    if ($KeepProcessId -ne 0 -and $eachProcess.ProcessId -eq $KeepProcessId) {
+        continue
+    }
+    Stop-Process -Id $eachProcess.ProcessId -Force -ErrorAction SilentlyContinue
+}