claude-dev-env 1.93.1 → 1.95.0

package/skills/logifix/scripts/logifix.ps1

@@ -1,205 +0,0 @@
-<#
-.SYNOPSIS
-Restores the Logitech Gaming Software (LCore) tray icon when it is missing on Windows.
-
-.DESCRIPTION
-Reproduces the verified recovery procedure from
-sessions/System Support/2. Logitech Tray Icon Fix Recurrence.md (2026-04-25):
-  1. Stops LCore in user context.
-  2. Single elevated UAC step:
-       - Starts LogiRegistryService (handles the case where it is Stopped).
-       - Stops explorer.exe and lets Windows shell auto-respawn rebuild it.
-     Deliberately omits Start-Process explorer from the elevated block to
-     avoid the duplicate admin-context explorer that blocked tray registration
-     during Session 2.
-  3. Waits for shell auto-respawn.
-  4. Verifies a single user-session explorer.
-  5. Performs LCoreRelaunchAttemptCount full stop-then-launch cycles of
-     LCore /minimized. Always runs the full count: a responsive LCore on the
-     first attempt does NOT imply Shell_NotifyIcon registered, per Session 2
-     gotcha #2 (responsive process, no tray icon).
-
-.PARAMETER ExplorerAutoRespawnWaitSeconds
-Seconds to wait after the elevated explorer kill before verifying respawn.
-
-.PARAMETER LCoreInitializationWaitSeconds
-Seconds to wait after each LCore relaunch before checking responsiveness.
-
-.PARAMETER LCoreRelaunchAttemptCount
-Total number of LCore relaunch cycles to perform (default 2). The full count
-always runs, because the documented failure mode is a responsive LCore that
-never registered a tray icon -- only a second stop+launch reliably triggers
-Shell_NotifyIcon registration.
-#>
-
-[CmdletBinding()]
-param(
-    [int] $ExplorerAutoRespawnWaitSeconds = 5,
-    [int] $LCoreInitializationWaitSeconds = 5,
-    [int] $LCoreRelaunchAttemptCount = 2
-)
-
-Set-StrictMode -Version Latest
-$ErrorActionPreference = 'Stop'
-
-$script:LCoreExecutablePath = 'C:\Program Files\Logitech Gaming Software\LCore.exe'
-$script:LCoreProcessName = 'LCore'
-
-function Get-CurrentInteractiveSessionId {
-    [OutputType([int])]
-    param()
-    return (Get-CimInstance Win32_Process -Filter "ProcessId=$PID").SessionId
-}
-
-function Get-ExplorerProcessesInSession {
-    [OutputType([object[]])]
-    param([Parameter(Mandatory)] [int] $TargetSessionId)
-    $allExplorers = Get-CimInstance Win32_Process -Filter "Name='explorer.exe'"
-    $matchingExplorers = @()
-    foreach ($eachExplorer in $allExplorers) {
-        if ($eachExplorer.SessionId -ne $TargetSessionId) { continue }
-        $ownerInfo = Invoke-CimMethod -InputObject $eachExplorer -MethodName GetOwner -ErrorAction SilentlyContinue
-        $matchingExplorers += [PSCustomObject]@{
-            ProcessId    = $eachExplorer.ProcessId
-            SessionId    = $eachExplorer.SessionId
-            OwnerUser    = if ($ownerInfo) { $ownerInfo.User } else { $null }
-            CreationDate = $eachExplorer.CreationDate
-        }
-    }
-    return ,$matchingExplorers
-}
-
-function Stop-LCoreIfRunning {
-    [OutputType([void])]
-    param()
-    $stopSettleMilliseconds = 500
-    Stop-Process -Name $script:LCoreProcessName -Force -ErrorAction SilentlyContinue
-    Start-Sleep -Milliseconds $stopSettleMilliseconds
-}
-
-function Invoke-ElevatedExplorerRebuild {
-    [OutputType([bool])]
-    param()
-    $elevatedScriptBlockText = @'
-Start-Service -Name LogiRegistryService -ErrorAction SilentlyContinue
-Stop-Process -Name explorer -Force -ErrorAction SilentlyContinue
-'@
-    $encodedElevatedCommand = [Convert]::ToBase64String([Text.Encoding]::Unicode.GetBytes($elevatedScriptBlockText))
-    try {
-        Start-Process -FilePath 'pwsh' `
-            -ArgumentList '-NoProfile', '-EncodedCommand', $encodedElevatedCommand `
-            -Verb RunAs `
-            -Wait `
-            -ErrorAction Stop
-        return $true
-    } catch {
-        Write-Warning "Elevated step did not run: $($_.Exception.Message)"
-        return $false
-    }
-}
-
-function Start-LCoreMinimized {
-    [OutputType([void])]
-    param()
-    $launchArguments = '/minimized'
-    Start-Process -FilePath $script:LCoreExecutablePath -ArgumentList $launchArguments
-}
-
-function Test-LCoreIsResponding {
-    [OutputType([bool])]
-    param([Parameter(Mandatory)] [int] $WaitSeconds)
-    Start-Sleep -Seconds $WaitSeconds
-    $lcoreProcesses = Get-Process -Name $script:LCoreProcessName -ErrorAction SilentlyContinue
-    if (-not $lcoreProcesses) { return $false }
-    foreach ($eachLCore in @($lcoreProcesses)) {
-        if (-not $eachLCore.Responding) { return $false }
-    }
-    return $true
-}
-
-function Write-StateSnapshot {
-    [OutputType([void])]
-    param([Parameter(Mandatory)] [string] $Label)
-    $logitechServiceNamesToVerify = @('LogiRegistryService', 'logi_lamparray_service')
-    Write-Host ""
-    Write-Host "=== $Label ==="
-    $sessionId = Get-CurrentInteractiveSessionId
-    $explorersInSession = Get-ExplorerProcessesInSession -TargetSessionId $sessionId
-    Write-Host "Explorer instances in session ${sessionId}: $($explorersInSession.Count)"
-    if ($explorersInSession.Count -gt 0) {
-        $explorersInSession | Format-Table ProcessId, OwnerUser, CreationDate -AutoSize | Out-String | Write-Host
-    }
-    $servicesObserved = Get-Service -Name $logitechServiceNamesToVerify -ErrorAction SilentlyContinue
-    if ($servicesObserved) {
-        $servicesObserved | Format-Table Name, Status -AutoSize | Out-String | Write-Host
-    }
-    $lcoreProcessesObserved = Get-Process -Name $script:LCoreProcessName -ErrorAction SilentlyContinue
-    if ($lcoreProcessesObserved) {
-        $lcoreProcessesObserved | Select-Object Id, Responding | Format-Table -AutoSize | Out-String | Write-Host
-    } else {
-        Write-Host "LCore not running."
-    }
-}
-
-function Invoke-LogifixRecovery {
-    [OutputType([void])]
-    param()
-
-    Write-StateSnapshot -Label 'Before'
-
-    if (-not (Test-Path -Path $script:LCoreExecutablePath)) {
-        Write-Error "LCore.exe not found at expected path: $script:LCoreExecutablePath"
-        return
-    }
-
-    Stop-LCoreIfRunning
-
-    $elevatedSucceeded = Invoke-ElevatedExplorerRebuild
-    if (-not $elevatedSucceeded) {
-        Write-Warning "UAC was canceled or the elevated step failed."
-        Write-Warning "Fallback: open Task Manager (Ctrl+Shift+Esc), find 'Windows Explorer', right-click, Restart. Then re-run /logifix."
-        return
-    }
-
-    Start-Sleep -Seconds $ExplorerAutoRespawnWaitSeconds
-
-    $sessionId = Get-CurrentInteractiveSessionId
-    $explorersAfterRespawn = Get-ExplorerProcessesInSession -TargetSessionId $sessionId
-    $explorerCountAfterRespawn = @($explorersAfterRespawn).Count
-    if ($explorerCountAfterRespawn -ne 1) {
-        if ($explorerCountAfterRespawn -eq 0) {
-            Write-Warning "No explorer.exe found in session $sessionId after $ExplorerAutoRespawnWaitSeconds-second wait."
-        } else {
-            Write-Warning "Expected exactly 1 explorer.exe in session $sessionId after shell auto-respawn, but found $explorerCountAfterRespawn."
-            Write-Warning "Multiple explorer.exe processes in the same session reproduce the Session 2 duplicate-explorer failure mode that blocks tray icon registration."
-            $explorersAfterRespawn | Format-Table ProcessId, OwnerUser, CreationDate -AutoSize | Out-String | Write-Host
-        }
-        Write-Warning "Restart Explorer manually via Task Manager (Ctrl+Shift+Esc, Windows Explorer, Restart), then re-run /logifix."
-        return
-    }
-
-    $lastAttemptResponded = $false
-    for ($attemptIndex = 1; $attemptIndex -le $LCoreRelaunchAttemptCount; $attemptIndex++) {
-        Stop-LCoreIfRunning
-        Start-LCoreMinimized
-        $lastAttemptResponded = Test-LCoreIsResponding -WaitSeconds $LCoreInitializationWaitSeconds
-        if ($lastAttemptResponded) {
-            Write-Host "LCore relaunch attempt ${attemptIndex}: process is responding."
-        } else {
-            Write-Warning "LCore relaunch attempt ${attemptIndex}: process not responding."
-        }
-    }
-
-    Write-StateSnapshot -Label 'After'
-
-    if (-not $lastAttemptResponded) {
-        Write-Warning "LCore did not respond after $LCoreRelaunchAttemptCount attempts."
-        Write-Warning "Use Task Manager to Restart Windows Explorer, then re-run /logifix."
-        return
-    }
-
-    Write-Host ""
-    Write-Host "Recovery complete ($LCoreRelaunchAttemptCount stop+launch cycles performed). If the tray icon is still not visible, use Task Manager to Restart Windows Explorer (guaranteed correct session/elevation), then re-run /logifix."
-}
-
-Invoke-LogifixRecovery

package/skills/monitor-open-prs/CLAUDE.md

@@ -1,34 +0,0 @@
-# monitor-open-prs skill
-
-Discovers every open pull request across the `jl-cmd/*` and `JonEcho/*` owner scopes, dispatches `/bugteam` on each with the `--bugbot-retrigger` flag, and polls for new Cursor bugbot replies after each bugteam run.
-
-**Trigger:** `/monitor-open-prs`, "sweep the open PRs", "audit the open PR backlog".
-
-## Key files
-
-| File | Purpose |
-|---|---|
-| `SKILL.md` | Full workflow: refusal cases, discovery, dispatch, post-convergence polling, final report |
-| `test_skill_contract.py` | Contract tests for the skill |
-| `packages/claude-dev-env/skills/monitor-open-prs/scripts/discover_open_prs.py` | Shells out to `gh search prs` per owner scope and flattens results to a uniform dict shape |
-| `packages/claude-dev-env/skills/monitor-open-prs/scripts/test_discover_open_prs.py` | Tests for the discovery helper |
-
-## Subdirectories
-
-| Directory | Role |
-|---|---|
-| `scripts/` | Discovery helper and its tests |
-
-## Workflow summary
-
-1. Call `discover_open_prs.discover_open_prs(all_owners=["jl-cmd", "JonEcho"])` to get the full open-PR list.
-2. For each PR, invoke `/bugteam <pr_number>`; include `--bugbot-retrigger` only when bugbot is enabled (`reviews_disabled.py --reviewer bugbot` exits 1).
-3. After each bugteam run, poll for new bugbot comments on a 60s → 120s → 240s → 480s → 960s backoff. Re-invoke bugteam if new findings appear.
-4. Emit a sweep summary when all PRs exit polling.
-
-## Refusals (checked first)
-
-- `CLAUDE_REVIEWS_DISABLED` has `bugteam` → stop with a message.
-- GitHub API not accessible → stop.
-- Uncommitted changes in the caller's repo → stop.
-- Required subagent types missing (`code-quality-agent`, `clean-coder`) → stop.

package/skills/monitor-open-prs/SKILL.md

@@ -1,88 +0,0 @@
----
-name: monitor-open-prs
-description: >-
-  Discover every open pull request across the jl-cmd/* and JonEcho/*
-  owner scopes, spawn /bugteam on each in parallel with the bugbot
-  re-trigger flag (--bugbot-retrigger), and poll Cursor's bugbot replies
-  after convergence so any post-bugteam findings loop back through
-  /bugteam. Triggers: '/monitor-open-prs', 'sweep the open PRs',
-  'audit the open PR backlog'.
----
-
-# Monitor Open PRs
-
-**Core principle:** One sweep covers every open PR across both owner scopes. Claude discovers PRs live via `scripts/discover_open_prs.py` which shells out to `gh search prs --owner <owner> --state open --json ...`, dispatches `/bugteam --bugbot-retrigger` per PR, then polls Cursor's bugbot replies until each PR is quiet for a full backoff cycle.
-
-## Contents
-
-- Transport check — gh presence and auth route the sweep local or through the cloud transport
-- When this skill applies — refusal cases and trigger conditions
-- Discovery — `scripts/discover_open_prs.py` queries via `gh search prs` across both owner scopes
-- Dispatch — `/bugteam --bugbot-retrigger <pr_numbers...>`
-- Post-convergence polling — bugbot replies and re-invocation
-- `scripts/discover_open_prs.py` — the discovery helper
-
-## Transport check (before any GitHub step)
-
-Run `command -v gh`; when it succeeds, run `gh auth status`; for each PR the sweep dispatches, `gh api repos/<owner>/<repo> --jq .permissions.push` must print `true`. When any check fails, run the `pr-loop-cloud-transport` skill first and route every `gh` operation in this skill through its substitution matrix, including the discovery query (`mcp__github__search_pull_requests` with `perPage` in place of `gh search prs`).
-
-## When this skill applies
-
-`/monitor-open-prs` authorizes one full sweep over all open PRs in both owner scopes.
-
-Refusals — first match wins; respond with the quoted line exactly and stop:
-
-- **Disabled via environment.** When `CLAUDE_REVIEWS_DISABLED` (comma-separated, case-insensitive, whitespace-tolerant) has the token `bugteam`: `/monitor-open-prs is a /bugteam dispatcher and /bugteam is disabled via CLAUDE_REVIEWS_DISABLED.` Gate semantics live in the `reviewer-gates` skill ([../reviewer-gates/SKILL.md](../reviewer-gates/SKILL.md)).
-- **GitHub API not accessible.** `get_me failed. /monitor-open-prs needs active GitHub MCP credentials.`
-- **Dirty tree on the caller's repo.** `Uncommitted changes detected. Stash, commit, or revert before /monitor-open-prs.`
-- **Required subagents missing.** Confirm `code-quality-agent` and `clean-coder` exist. Else: `Required subagent type <name> not installed.`
-
-## Discovery
-
-Call `scripts/discover_open_prs.discover_open_prs(all_owners=["jl-cmd", "JonEcho"])` to merge the live open-PR list across both scopes. The helper shells out to `gh search prs --owner <owner> --state open --json number,repository,url,headRefName,baseRefName` for each owner scope and flattens the result to a uniform dict shape with keys `number`, `owner`, `repo`, `head_ref`, `base_ref`, `url`. Empty scopes contribute empty lists; an entirely empty sweep returns `[]` and exits cleanly.
-
-## Dispatch
-
-For each discovered PR:
-
-1. Resolve the PR's repo checkout (existing worktree or fresh `git clone`).
-2. From that checkout, invoke `/bugteam <pr_number>`. Include `--bugbot-retrigger` only when bugbot is enabled for the run — when `python "$HOME/.claude/_shared/pr-loop/scripts/reviews_disabled.py" --reviewer bugbot` exits 1. On exit 0 (bugbot disabled — the default) omit `--bugbot-retrigger` so the bugbot leg sits out the run.
-3. The `--bugbot-retrigger` flag tells bugteam to post `bugbot run` as an issue comment after every successful FIX push so Cursor's bugbot re-evaluates the new commit.
-4. Bugteam runs its own 20-loop audit/fix cycle per PR; this skill waits for each bugteam invocation to return before dispatching the next (or fanning out — see below).
-
-**Fan-out (optional):** when the discovered list has more than one PR, the skill may spawn `/bugteam` dispatches in parallel by issuing multiple `Agent` calls in a single assistant message. Each dispatch operates in its own per-PR worktree (bugteam Step 1.1). Serialize when the caller sets an explicit `--serial` flag.
-
-## Post-Convergence Polling
-
-After a `/bugteam` invocation returns (converged, cap reached, stuck, or error), the PR may accumulate new Cursor bugbot comments within minutes. Poll for them:
-
-1. Baseline: capture `since_timestamp` as the PR's last commit timestamp.
-2. Every 60 seconds, call `pull_request_read(method="get", pullNumber=<pr_number>, owner=<owner>, repo=<repo>)` and filter the response's `comments` array for entries whose `.author.login` matches `"bugbot"` or `"cursor"` and `.createdAt` is after `<since_timestamp>`.
-3. Back off: 60s → 120s → 240s → 480s → 960s. If five successive polls return empty, exit polling for this PR.
-4. If bugbot posts a new finding in any poll, re-invoke `/bugteam <pr_number>` with the bugbot finding text seeded into the invocation's `bugs_to_fix` preamble. Reset the backoff.
-
-### Polling Cost and Cadence
-
-The five-step backoff sums to `60 + 120 + 240 + 480 + 960 = 1860` seconds (~31 minutes) of idle polling per PR before the skill declares bugbot quiet. A sweep over ten open PRs therefore retains up to ~5 wall-clock hours of bugbot-watch time beyond the active `/bugteam` work. Callers who need faster turnaround should pass `--serial` to disable fan-out (so polling starts only after the previous PR finishes) or accept the tradeoff: the backoff exists specifically to catch late bugbot analyses that can take several minutes to appear after a push.
-
-## Final Report
-
-After every PR has exited polling, emit:
-
-```
-/monitor-open-prs sweep summary
-PRs discovered: <N>
-  jl-cmd/*: <count>
-  JonEcho/*: <count>
-PRs converged clean: <count>
-PRs hit 20-loop cap: <count>
-PRs stuck: <count>
-PRs errored: <count>
-Bugbot re-triggers fired: <count>
-```
-
-## Non-Negotiable Guardrails
-
-- Never pass `--no-verify` or `--no-gpg-sign` to git in any dispatched bugteam run.
-- Never open a PR from this skill; only comment on existing ones.
-- Never merge or close PRs; the skill is read + audit + patch only.

package/skills/monitor-open-prs/scripts/CLAUDE.md

@@ -1,17 +0,0 @@
-# monitor-open-prs/scripts
-
-Discovery helper for the `monitor-open-prs` skill.
-
-## Key files
-
-| File | Purpose |
-|---|---|
-| `discover_open_prs.py` | Queries `gh search prs` for each owner scope and flattens results to a uniform list of dicts |
-| `test_discover_open_prs.py` | Tests for the discovery helper |
-
-## Conventions
-
-- `discover_open_prs(all_owners: list[str]) -> list[dict]` is the single entry point. It shells out to `gh search prs --owner <owner> --state open --json number,repository,url,headRefName,baseRefName` once per owner and merges the results.
-- Each returned dict has keys: `number`, `owner`, `repo`, `head_ref`, `base_ref`, `url`.
-- An empty scope or an empty sweep returns an empty list and exits cleanly with no errors.
-- The module is stateless and has no filesystem side effects.

package/skills/monitor-open-prs/scripts/discover_open_prs.py

@@ -1,69 +0,0 @@
-"""Discover every open pull request across a list of owner scopes.
-
-Shells out to ``gh search prs --owner <owner> --state open --json ...`` once per
-owner, parses the JSON output, and flattens each entry to a uniform dict shape.
-The module is stateless, takes no filesystem side effects, and exposes a single
-``discover_open_prs`` entry point consumed by the monitor-open-prs skill.
-"""
-
-from __future__ import annotations
-
-import json
-import subprocess
-
-
-def build_gh_search_argv(owner: str) -> list[str]:
-    gh_command_name = "gh"
-    search_subcommand = ("search", "prs")
-    owner_flag = "--owner"
-    state_flag = "--state"
-    state_open_value = "open"
-    json_fields_flag = "--json"
-    json_fields_value = "number,repository,url,headRefName,baseRefName"
-    return [
-        gh_command_name,
-        *search_subcommand,
-        owner_flag,
-        owner,
-        state_flag,
-        state_open_value,
-        json_fields_flag,
-        json_fields_value,
-    ]
-
-
-def split_repository_name(name_with_owner: str) -> tuple[str, str]:
-    repo_separator = "/"
-    if repo_separator not in name_with_owner:
-        return "", name_with_owner
-    owner_segment, repository_segment = name_with_owner.split(repo_separator, 1)
-    return owner_segment, repository_segment
-
-
-def flatten_pr_entry(raw_pr_entry: dict) -> dict:
-    name_with_owner = raw_pr_entry.get("repository", {}).get("nameWithOwner", "")
-    owner_segment, repository_segment = split_repository_name(name_with_owner)
-    return {
-        "number": raw_pr_entry.get("number"),
-        "owner": owner_segment,
-        "repo": repository_segment,
-        "head_ref": raw_pr_entry.get("headRefName", ""),
-        "base_ref": raw_pr_entry.get("baseRefName", ""),
-        "url": raw_pr_entry.get("url", ""),
-    }
-
-
-def fetch_open_prs_for_owner(owner: str) -> list[dict]:
-    search_argv = build_gh_search_argv(owner)
-    completed_process = subprocess.run(
-        search_argv, check=True, capture_output=True, text=True
-    )
-    raw_pr_entries = json.loads(completed_process.stdout)
-    return [flatten_pr_entry(each_entry) for each_entry in raw_pr_entries]
-
-
-def discover_open_prs(all_owners: list[str]) -> list[dict]:
-    all_discovered: list[dict] = []
-    for each_owner in all_owners:
-        all_discovered.extend(fetch_open_prs_for_owner(each_owner))
-    return all_discovered

package/skills/monitor-open-prs/scripts/test_discover_open_prs.py

@@ -1,149 +0,0 @@
-"""Tests for discover_open_prs.py — merges gh-search output across owners."""
-
-from __future__ import annotations
-
-import importlib.util
-import json
-import pathlib
-import sys
-
-import pytest
-
-
-def _load_discovery_module():
-    scripts_directory = pathlib.Path(__file__).parent
-    sys.path.insert(0, str(scripts_directory))
-    module_path = scripts_directory / "discover_open_prs.py"
-    module_spec = importlib.util.spec_from_file_location(
-        "discover_open_prs", module_path
-    )
-    loaded_module = importlib.util.module_from_spec(module_spec)
-    sys.modules["discover_open_prs"] = loaded_module
-    module_spec.loader.exec_module(loaded_module)
-    return loaded_module
-
-
-discover_open_prs = _load_discovery_module()
-
-
-def _make_fake_gh(responses_by_owner: dict):
-    """Return a callable whose behavior mocks subprocess.run for gh search."""
-
-    class FakeCompletedProcess:
-        def __init__(self, stdout_text: str):
-            self.stdout = stdout_text
-            self.returncode = 0
-
-    def fake_run(command_argv, check, capture_output, text):
-        owner = _extract_owner_argument(command_argv)
-        stdout_text = json.dumps(responses_by_owner.get(owner, []))
-        return FakeCompletedProcess(stdout_text)
-
-    return fake_run
-
-
-def _extract_owner_argument(command_argv) -> str:
-    for each_index, each_token in enumerate(command_argv):
-        if each_token == "--owner":
-            return command_argv[each_index + 1]
-    return ""
-
-
-class TestDiscoverOpenPrsBothOwnersReturnResults:
-    def test_merges_prs_across_both_owners(self, monkeypatch):
-        first_owner_prs = [
-            {
-                "number": 1,
-                "repository": {"nameWithOwner": "jl-cmd/alpha"},
-                "url": "https://github.com/jl-cmd/alpha/pull/1",
-                "headRefName": "feat-one",
-                "baseRefName": "main",
-            }
-        ]
-        second_owner_prs = [
-            {
-                "number": 2,
-                "repository": {"nameWithOwner": "JonEcho/beta"},
-                "url": "https://github.com/JonEcho/beta/pull/2",
-                "headRefName": "feat-two",
-                "baseRefName": "main",
-            }
-        ]
-        fake_run = _make_fake_gh(
-            {"jl-cmd": first_owner_prs, "JonEcho": second_owner_prs}
-        )
-        monkeypatch.setattr(discover_open_prs.subprocess, "run", fake_run)
-
-        all_discovered = discover_open_prs.discover_open_prs(
-            all_owners=["jl-cmd", "JonEcho"]
-        )
-
-        assert len(all_discovered) == 2
-        discovered_numbers = sorted(each["number"] for each in all_discovered)
-        assert discovered_numbers == [1, 2]
-
-
-class TestDiscoverOpenPrsOneOwnerEmpty:
-    def test_handles_single_owner_returning_empty_list(self, monkeypatch):
-        first_owner_prs = [
-            {
-                "number": 5,
-                "repository": {"nameWithOwner": "jl-cmd/gamma"},
-                "url": "https://github.com/jl-cmd/gamma/pull/5",
-                "headRefName": "feat-five",
-                "baseRefName": "main",
-            }
-        ]
-        fake_run = _make_fake_gh({"jl-cmd": first_owner_prs, "JonEcho": []})
-        monkeypatch.setattr(discover_open_prs.subprocess, "run", fake_run)
-
-        all_discovered = discover_open_prs.discover_open_prs(
-            all_owners=["jl-cmd", "JonEcho"]
-        )
-
-        assert len(all_discovered) == 1
-        assert all_discovered[0]["number"] == 5
-
-
-class TestDiscoverOpenPrsBothEmpty:
-    def test_returns_empty_list_when_no_owner_has_open_prs(self, monkeypatch):
-        fake_run = _make_fake_gh({"jl-cmd": [], "JonEcho": []})
-        monkeypatch.setattr(discover_open_prs.subprocess, "run", fake_run)
-
-        all_discovered = discover_open_prs.discover_open_prs(
-            all_owners=["jl-cmd", "JonEcho"]
-        )
-
-        assert all_discovered == []
-
-
-class TestDiscoverOpenPrsEntryShape:
-    def test_each_entry_exposes_number_owner_repo_refs_and_url(self, monkeypatch):
-        fake_run = _make_fake_gh(
-            {
-                "jl-cmd": [
-                    {
-                        "number": 42,
-                        "repository": {"nameWithOwner": "jl-cmd/demo"},
-                        "url": "https://github.com/jl-cmd/demo/pull/42",
-                        "headRefName": "feat-demo",
-                        "baseRefName": "main",
-                    }
-                ],
-                "JonEcho": [],
-            }
-        )
-        monkeypatch.setattr(discover_open_prs.subprocess, "run", fake_run)
-
-        all_discovered = discover_open_prs.discover_open_prs(
-            all_owners=["jl-cmd", "JonEcho"]
-        )
-
-        assert len(all_discovered) == 1
-        only_entry = all_discovered[0]
-        assert only_entry["number"] == 42
-        assert only_entry["owner"] == "jl-cmd"
-        assert only_entry["repo"] == "demo"
-        assert only_entry["head_ref"] == "feat-demo"
-        assert only_entry["base_ref"] == "main"
-        assert only_entry["url"].endswith("/pull/42")

package/skills/monitor-open-prs/test_skill_contract.py

@@ -1,27 +0,0 @@
-"""Markdown assertion tests for monitor-open-prs SKILL.md."""
-
-from __future__ import annotations
-
-import pathlib
-
-
-def _read_skill_text() -> str:
-    skill_path = pathlib.Path(__file__).parent / "SKILL.md"
-    return skill_path.read_text(encoding="utf-8")
-
-
-def test_skill_has_frontmatter_name():
-    skill_text = _read_skill_text()
-    assert skill_text.startswith("---\n")
-    assert "name: monitor-open-prs" in skill_text
-
-
-def test_skill_references_bugbot_retrigger_flag():
-    skill_text = _read_skill_text()
-    assert "--bugbot-retrigger" in skill_text
-
-
-def test_skill_enumerates_both_owner_scopes():
-    skill_text = _read_skill_text()
-    assert "jl-cmd" in skill_text
-    assert "JonEcho" in skill_text