npm - claude-dev-env - Versions diffs - 1.37.0 → 1.38.0 - Mend

claude-dev-env 1.37.0 → 1.38.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (95) hide show

package/CLAUDE.md +3 -0
package/_shared/pr-loop/audit-contract.md +4 -3
package/_shared/pr-loop/fix-protocol.md +2 -0
package/_shared/pr-loop/gh-payloads.md +38 -37
package/_shared/pr-loop/scripts/README.md +0 -1
package/_shared/pr-loop/scripts/preflight.py +2 -1
package/_shared/pr-loop/scripts/tests/test_code_rules_gate.py +2 -2
package/_shared/pr-loop/scripts/tests/test_preflight.py +22 -0
package/_shared/pr-loop/state-schema.md +10 -10
package/agents/clean-coder.md +4 -0
package/agents/code-quality-agent.md +23 -85
package/agents/groq-coder.md +8 -6
package/hooks/blocking/__init__.py +0 -0
package/hooks/blocking/hedging_language_blocker.py +2 -2
package/hooks/blocking/state_description_blocker.py +243 -0
package/hooks/blocking/tdd_enforcer.py +94 -0
package/hooks/blocking/test_hedging_language_blocker.py +1 -1
package/hooks/blocking/test_state_description_blocker.py +618 -0
package/hooks/blocking/test_tdd_enforcer.py +152 -0
package/hooks/config/state_description_blocker_constants.py +130 -0
package/hooks/hooks.json +10 -0
package/package.json +1 -1
package/rules/gh-paginate.md +4 -50
package/rules/no-historical-clutter.md +57 -0
package/scripts/config/groq_bugteam_config.py +13 -5
package/skills/bugteam/CONSTRAINTS.md +20 -27
package/skills/bugteam/EXAMPLES.md +1 -1
package/skills/bugteam/PROMPTS.md +78 -42
package/skills/bugteam/SKILL.md +76 -63
package/skills/bugteam/SKILL_EVALS.md +12 -12
package/skills/bugteam/reference/audit-and-teammates.md +21 -48
package/skills/bugteam/reference/audit-contract.md +7 -7
package/skills/bugteam/reference/github-pr-reviews.md +31 -31
package/skills/bugteam/reference/team-setup.md +1 -1
package/skills/bugteam/reference/teardown-publish-permissions.md +4 -4
package/skills/copilot-review/SKILL.md +7 -14
package/skills/findbugs/SKILL.md +2 -2
package/skills/fixbugs/SKILL.md +1 -1
package/skills/monitor-open-prs/SKILL.md +6 -6
package/skills/pr-converge/SKILL.md +7 -6
package/skills/pr-converge/reference/convergence-gates.md +46 -44
package/skills/pr-converge/reference/examples.md +4 -4
package/skills/pr-converge/reference/fix-protocol.md +8 -8
package/skills/pr-converge/reference/multi-pr-orchestration.md +10 -10
package/skills/pr-converge/reference/per-tick.md +24 -36
package/skills/pr-converge/reference/stop-conditions.md +7 -7
package/skills/pr-converge/scripts/README.md +65 -117
package/skills/pr-review-responder/EXAMPLES.md +2 -2
package/skills/pr-review-responder/PRINCIPLES.md +2 -8
package/skills/pr-review-responder/README.md +7 -48
package/skills/pr-review-responder/SKILL.md +2 -3
package/skills/pr-review-responder/TESTING.md +8 -65
package/skills/qbug/SKILL.md +10 -16
package/_shared/pr-loop/scripts/config/gh_util_constants.py +0 -31
package/_shared/pr-loop/scripts/gh_util.py +0 -193
package/_shared/pr-loop/scripts/tests/test_gh_util.py +0 -257
package/_shared/pr-loop/scripts/tests/test_gh_util_constants.py +0 -61
package/skills/pr-converge/scripts/check_pr_mergeability.py +0 -78
package/skills/pr-converge/scripts/config/pr_converge_constants.py +0 -118
package/skills/pr-converge/scripts/config/test_pr_converge_constants.py +0 -152
package/skills/pr-converge/scripts/fetch_bugbot_inline_comments.py +0 -70
package/skills/pr-converge/scripts/fetch_bugbot_reviews.py +0 -57
package/skills/pr-converge/scripts/fetch_claude_inline_comments.py +0 -70
package/skills/pr-converge/scripts/fetch_claude_reviews.py +0 -61
package/skills/pr-converge/scripts/fetch_copilot_inline_comments.py +0 -70
package/skills/pr-converge/scripts/fetch_copilot_reviews.py +0 -61
package/skills/pr-converge/scripts/mark_pr_ready.py +0 -54
package/skills/pr-converge/scripts/post-bugbot-run.helpers.ps1 +0 -49
package/skills/pr-converge/scripts/post-bugbot-run.ps1 +0 -33
package/skills/pr-converge/scripts/reply_to_inline_comment.py +0 -84
package/skills/pr-converge/scripts/request_copilot_review.py +0 -71
package/skills/pr-converge/scripts/resolve_pr_head.py +0 -58
package/skills/pr-converge/scripts/review_field_helpers.py +0 -43
package/skills/pr-converge/scripts/reviewer_fetch_core.py +0 -153
package/skills/pr-converge/scripts/reviewer_specs.py +0 -98
package/skills/pr-converge/scripts/test_check_pr_mergeability.py +0 -126
package/skills/pr-converge/scripts/test_fetch_bugbot_inline_comments.py +0 -443
package/skills/pr-converge/scripts/test_fetch_bugbot_reviews.py +0 -299
package/skills/pr-converge/scripts/test_fetch_claude_inline_comments.py +0 -485
package/skills/pr-converge/scripts/test_fetch_claude_reviews.py +0 -368
package/skills/pr-converge/scripts/test_fetch_copilot_inline_comments.py +0 -440
package/skills/pr-converge/scripts/test_fetch_copilot_reviews.py +0 -366
package/skills/pr-converge/scripts/test_mark_pr_ready.py +0 -69
package/skills/pr-converge/scripts/test_post_bugbot_run.py +0 -195
package/skills/pr-converge/scripts/test_reply_to_inline_comment.py +0 -159
package/skills/pr-converge/scripts/test_request_copilot_review.py +0 -101
package/skills/pr-converge/scripts/test_resolve_pr_head.py +0 -79
package/skills/pr-converge/scripts/test_review_field_helpers.py +0 -80
package/skills/pr-converge/scripts/test_reviewer_fetch_core.py +0 -448
package/skills/pr-converge/scripts/test_reviewer_specs.py +0 -107
package/skills/pr-converge/scripts/test_trigger_bugbot.py +0 -139
package/skills/pr-converge/scripts/test_view_pr_context.py +0 -111
package/skills/pr-converge/scripts/trigger_bugbot.py +0 -77
package/skills/pr-converge/scripts/view_pr_context.py +0 -47
package/skills/pr-review-responder/scripts/respond_to_reviews.py +0 -376

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

@@ -1,118 +0,0 @@
-"""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.
-"""
-import re
-from pathlib import Path
-CURSOR_BOT_LOGIN: str = "cursor[bot]"
-CURSOR_LOGIN_FILTER_SUBSTRING: str = "cursor"
-COPILOT_REVIEWER_LOGIN: str = "copilot-pull-request-reviewer[bot]"
-COPILOT_REVIEWER_REQUEST_ID: str = COPILOT_REVIEWER_LOGIN
-COPILOT_LOGIN_FILTER_SUBSTRING: str = "copilot"
-COPILOT_CLEAN_REVIEW_STATE: str = "APPROVED"
-ALL_COPILOT_DIRTY_REVIEW_STATES: tuple[str, ...] = ("CHANGES_REQUESTED", "COMMENTED")
-COPILOT_SOFT_DIRTY_REVIEW_STATE: str = "COMMENTED"
-CLAUDE_REVIEWER_LOGIN: str = "claude[bot]"
-CLAUDE_REVIEWER_REQUEST_ID: str = CLAUDE_REVIEWER_LOGIN
-CLAUDE_LOGIN_FILTER_SUBSTRING: str = "claude"
-CLAUDE_CLEAN_REVIEW_STATE: str = "APPROVED"
-ALL_CLAUDE_DIRTY_REVIEW_STATES: tuple[str, ...] = ("CHANGES_REQUESTED", "COMMENTED")
-CLAUDE_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}"
-SKILL_REFLOW_MAXIMUM_WIDTH: int = 80
-PR_CONVERGE_SKILL_PATH: Path = Path(__file__).resolve().parent.parent.parent / "SKILL.md"
-MARKDOWN_CODE_FENCE_MARKER: str = "```"
-YAML_FRONT_MATTER_DELIMITER: str = "---"
-YAML_DESCRIPTION_PREFIX: str = "description: >-"
-EXAMPLE_OPEN_TAG: str = "<example>"
-EXAMPLE_CLOSE_TAG: str = "</example>"
-BASH_FENCE_LANGUAGE: str = "bash"
-BASH_LINE_CONTINUATION_SUFFIX: str = " \\"
-BASH_CONTINUATION_INDENT: str = "  "
-REFLOW_FRONT_MATTER_ERROR: str = "expected YAML front matter starting with ---"
-ORDERED_MARKDOWN_LIST_PATTERN: re.Pattern[str] = re.compile(
-    r"^(?P<leading_whitespace>\s*)(?P<marker>\d+\.\s)(?P<body>.*)$"
-)
-BULLET_MARKDOWN_LIST_PATTERN: re.Pattern[str] = re.compile(
-    r"^(?P<leading_whitespace>\s*)(?P<marker>[-*]\s)(?P<body>.*)$"
-)
-UNFINISHED_MARKDOWN_LINK_TARGET_PATTERN: re.Pattern[str] = re.compile(r"\]\([^)]*$")
-MARKDOWN_HEADING_PATTERN: re.Pattern[str] = re.compile(r"^#{1,6}\s+.+$")
-MARKDOWN_REFERENCE_DEFINITION_PATTERN: re.Pattern[str] = re.compile(r"^\[[^\]]+\]:\s+\S+")
-BASH_LINE_CONTINUATION_MARKER_WIDTH: int = 2
-CODE_FENCE_MARKER_LENGTH: int = 3
-BASH_MINIMUM_SEGMENT_WIDTH: int = 1
-LONG_ROW_PREVIEW_LIMIT: int = 20

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

@@ -1,152 +0,0 @@
-"""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]"

package/skills/pr-converge/scripts/fetch_bugbot_inline_comments.py DELETED Viewed

@@ -1,70 +0,0 @@
-"""Fetch unaddressed Cursor Bugbot inline comments for the latest Bugbot review on a commit.
-Thin wrapper around ``reviewer_fetch_core.fetch_reviewer_inline_comments``
-parameterised by ``bugbot_spec``. The ``fetch_bugbot_reviews`` call lives here
-(rather than inside the core) so tests can patch it on this module to exercise
-the inline-comments fetch in isolation.
-Wraps the gh CLI invocation required by the gh-paginate rule for the comments
-list: ``gh api`` on ``repos/{owner}/{repo}/pulls/{number}/comments`` with
-``--paginate --slurp`` and external JSON handling.
-"""
-from __future__ import annotations
-import argparse
-import json
-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 fetch_bugbot_reviews import fetch_bugbot_reviews
-from reviewer_fetch_core import fetch_reviewer_inline_comments
-from reviewer_specs import bugbot_spec
-def fetch_bugbot_inline_comments(
-    *,
-    owner: str,
-    repo: str,
-    number: int,
-    current_head: str,
-) -> list[dict[str, object]]:
-    """Return Bugbot inline comments for the latest Bugbot review on ``current_head``."""
-    all_bugbot_reviews = fetch_bugbot_reviews(owner=owner, repo=repo, number=number)
-    return fetch_reviewer_inline_comments(
-        bugbot_spec,
-        owner=owner,
-        repo=repo,
-        number=number,
-        current_head=current_head,
-        all_reviews=all_bugbot_reviews,
-    )
-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)
-    parser.add_argument("--commit", required=True, dest="current_head")
-    parsed_arguments = parser.parse_args()
-    all_comments = fetch_bugbot_inline_comments(
-        owner=parsed_arguments.owner,
-        repo=parsed_arguments.repo,
-        number=parsed_arguments.number,
-        current_head=parsed_arguments.current_head,
-    )
-    json.dump(all_comments, sys.stdout)
-    sys.stdout.write("\n")
-    return 0
-if __name__ == "__main__":
-    sys.exit(main())

package/skills/pr-converge/scripts/fetch_bugbot_reviews.py DELETED Viewed

@@ -1,57 +0,0 @@
-"""Fetch Cursor Bugbot reviews newest-first, classified as dirty or clean.
-Thin wrapper around ``reviewer_fetch_core.fetch_reviewer_reviews`` parameterised
-by ``bugbot_spec``. Wraps the gh CLI invocation required by the gh-paginate
-rule: ``gh api '...?per_page=100' --paginate --slurp`` piped through external
-Python JSON handling (instead of ``gh --jq``, which runs per-page and breaks
-cross-page operations like sort/reverse - see GitHub CLI issue 10459).
-"""
-from __future__ import annotations
-import argparse
-import json
-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 reviewer_fetch_core import fetch_reviewer_reviews
-from reviewer_specs import bugbot_spec
-def fetch_bugbot_reviews(
-    *,
-    owner: str,
-    repo: str,
-    number: int,
-) -> list[dict[str, object]]:
-    """Return Cursor Bugbot reviews newest-first, each with a classification."""
-    return fetch_reviewer_reviews(
-        bugbot_spec, owner=owner, repo=repo, number=number
-    )
-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()
-    all_reviews = fetch_bugbot_reviews(
-        owner=parsed_arguments.owner,
-        repo=parsed_arguments.repo,
-        number=parsed_arguments.number,
-    )
-    json.dump(all_reviews, sys.stdout)
-    sys.stdout.write("\n")
-    return 0
-if __name__ == "__main__":
-    sys.exit(main())

package/skills/pr-converge/scripts/fetch_claude_inline_comments.py DELETED Viewed

@@ -1,70 +0,0 @@
-"""Fetch unaddressed Claude inline comments for the latest Claude review on a commit.
-Thin wrapper around ``reviewer_fetch_core.fetch_reviewer_inline_comments``
-parameterised by ``claude_spec``. The ``fetch_claude_reviews`` call lives here
-(rather than inside the core) so tests can patch it on this module to exercise
-the inline-comments fetch in isolation.
-Wraps the gh CLI invocation required by the gh-paginate rule for the comments
-list: ``gh api`` on ``repos/{owner}/{repo}/pulls/{number}/comments`` with
-``--paginate --slurp`` and external JSON handling.
-"""
-from __future__ import annotations
-import argparse
-import json
-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 fetch_claude_reviews import fetch_claude_reviews
-from reviewer_fetch_core import fetch_reviewer_inline_comments
-from reviewer_specs import claude_spec
-def fetch_claude_inline_comments(
-    *,
-    owner: str,
-    repo: str,
-    number: int,
-    current_head: str,
-) -> list[dict[str, object]]:
-    """Return Claude inline comments for the latest Claude review on ``current_head``."""
-    all_claude_reviews = fetch_claude_reviews(owner=owner, repo=repo, number=number)
-    return fetch_reviewer_inline_comments(
-        claude_spec,
-        owner=owner,
-        repo=repo,
-        number=number,
-        current_head=current_head,
-        all_reviews=all_claude_reviews,
-    )
-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)
-    parser.add_argument("--commit", required=True, dest="current_head")
-    parsed_arguments = parser.parse_args()
-    all_comments = fetch_claude_inline_comments(
-        owner=parsed_arguments.owner,
-        repo=parsed_arguments.repo,
-        number=parsed_arguments.number,
-        current_head=parsed_arguments.current_head,
-    )
-    json.dump(all_comments, sys.stdout)
-    sys.stdout.write("\n")
-    return 0
-if __name__ == "__main__":
-    sys.exit(main())

package/skills/pr-converge/scripts/fetch_claude_reviews.py DELETED Viewed

@@ -1,61 +0,0 @@
-"""Fetch Claude reviewer-bot reviews newest-first, classified as dirty or clean.
-Thin wrapper around ``reviewer_fetch_core.fetch_reviewer_reviews`` parameterised
-by ``claude_spec``. Classification follows the review's ``state`` field
-(``APPROVED`` -> clean; ``CHANGES_REQUESTED`` -> dirty; ``COMMENTED`` with
-non-empty body -> dirty; everything else -> clean) - see ``reviewer_specs``.
-Wraps the gh CLI invocation required by the gh-paginate rule:
-``gh api '...?per_page=100' --paginate --slurp`` piped through external Python
-JSON handling (instead of ``gh --jq``, which runs per-page and breaks
-cross-page operations like sort/reverse - see GitHub CLI issue 10459).
-"""
-from __future__ import annotations
-import argparse
-import json
-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 reviewer_fetch_core import fetch_reviewer_reviews
-from reviewer_specs import claude_spec
-def fetch_claude_reviews(
-    *,
-    owner: str,
-    repo: str,
-    number: int,
-) -> list[dict[str, object]]:
-    """Return Claude reviews newest-first, each with a classification."""
-    return fetch_reviewer_reviews(
-        claude_spec, owner=owner, repo=repo, number=number
-    )
-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()
-    all_reviews = fetch_claude_reviews(
-        owner=parsed_arguments.owner,
-        repo=parsed_arguments.repo,
-        number=parsed_arguments.number,
-    )
-    json.dump(all_reviews, sys.stdout)
-    sys.stdout.write("\n")
-    return 0
-if __name__ == "__main__":
-    sys.exit(main())

package/skills/pr-converge/scripts/fetch_copilot_inline_comments.py DELETED Viewed

@@ -1,70 +0,0 @@
-"""Fetch unaddressed Copilot inline comments for the latest Copilot review on a commit.
-Thin wrapper around ``reviewer_fetch_core.fetch_reviewer_inline_comments``
-parameterised by ``copilot_spec``. The ``fetch_copilot_reviews`` call lives
-here (rather than inside the core) so tests can patch it on this module to
-exercise the inline-comments fetch in isolation.
-Wraps the gh CLI invocation required by the gh-paginate rule for the comments
-list: ``gh api`` on ``repos/{owner}/{repo}/pulls/{number}/comments`` with
-``--paginate --slurp`` and external JSON handling.
-"""
-from __future__ import annotations
-import argparse
-import json
-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 fetch_copilot_reviews import fetch_copilot_reviews
-from reviewer_fetch_core import fetch_reviewer_inline_comments
-from reviewer_specs import copilot_spec
-def fetch_copilot_inline_comments(
-    *,
-    owner: str,
-    repo: str,
-    number: int,
-    current_head: str,
-) -> list[dict[str, object]]:
-    """Return Copilot inline comments for the latest Copilot review on ``current_head``."""
-    all_copilot_reviews = fetch_copilot_reviews(owner=owner, repo=repo, number=number)
-    return fetch_reviewer_inline_comments(
-        copilot_spec,
-        owner=owner,
-        repo=repo,
-        number=number,
-        current_head=current_head,
-        all_reviews=all_copilot_reviews,
-    )
-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)
-    parser.add_argument("--commit", required=True, dest="current_head")
-    parsed_arguments = parser.parse_args()
-    all_comments = fetch_copilot_inline_comments(
-        owner=parsed_arguments.owner,
-        repo=parsed_arguments.repo,
-        number=parsed_arguments.number,
-        current_head=parsed_arguments.current_head,
-    )
-    json.dump(all_comments, sys.stdout)
-    sys.stdout.write("\n")
-    return 0
-if __name__ == "__main__":
-    sys.exit(main())

package/skills/pr-converge/scripts/fetch_copilot_reviews.py DELETED Viewed

@@ -1,61 +0,0 @@
-"""Fetch GitHub Copilot reviewer reviews newest-first, classified as dirty or clean.
-Thin wrapper around ``reviewer_fetch_core.fetch_reviewer_reviews`` parameterised
-by ``copilot_spec``. Classification follows the review's ``state`` field
-(``APPROVED`` -> clean; ``CHANGES_REQUESTED`` -> dirty; ``COMMENTED`` with
-non-empty body -> dirty; everything else -> clean) - see ``reviewer_specs``.
-Wraps the gh CLI invocation required by the gh-paginate rule:
-``gh api '...?per_page=100' --paginate --slurp`` piped through external Python
-JSON handling (instead of ``gh --jq``, which runs per-page and breaks
-cross-page operations like sort/reverse - see GitHub CLI issue 10459).
-"""
-from __future__ import annotations
-import argparse
-import json
-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 reviewer_fetch_core import fetch_reviewer_reviews
-from reviewer_specs import copilot_spec
-def fetch_copilot_reviews(
-    *,
-    owner: str,
-    repo: str,
-    number: int,
-) -> list[dict[str, object]]:
-    """Return Copilot reviews newest-first, each with a classification."""
-    return fetch_reviewer_reviews(
-        copilot_spec, owner=owner, repo=repo, number=number
-    )
-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()
-    all_reviews = fetch_copilot_reviews(
-        owner=parsed_arguments.owner,
-        repo=parsed_arguments.repo,
-        number=parsed_arguments.number,
-    )
-    json.dump(all_reviews, sys.stdout)
-    sys.stdout.write("\n")
-    return 0
-if __name__ == "__main__":
-    sys.exit(main())