claude-dev-env 1.42.0 → 1.43.0

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

@@ -33,7 +33,7 @@ _pr_converge_dir = Path(__file__).absolute().parent.parent
 if str(_pr_converge_dir) not in sys.path:
     sys.path.insert(0, str(_pr_converge_dir))
 
-from config.constants import (
+from pr_converge_skill_constants.constants import (
     ALL_BUGBOT_CHECK_RUN_ACTIVE_STATUSES,
     ALL_BUGBOT_CHECK_RUN_COMPLETE_CONCLUSIONS,
     BUGBOT_CHECK_RUN_COMPLETED_STATUS,

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

@@ -23,7 +23,7 @@ _pr_converge_dir = Path(__file__).absolute().parent.parent
 if str(_pr_converge_dir) not in sys.path:
     sys.path.insert(0, str(_pr_converge_dir))
 
-from config.constants import (
+from pr_converge_skill_constants.constants import (
     ALL_COPILOT_DIRTY_REVIEW_STATES,
     ALL_BUGBOT_CHECK_RUN_COMPLETE_CONCLUSIONS,
     BUGBOT_CHECK_RUN_NAME_SUBSTRING,
@@ -495,9 +495,16 @@ def check_all(*, owner: str, repo: str, number: int, bugbot_down: bool) -> int:
 
     Returns:
         ``0`` when every gate reports PASS, ``1`` when at least one gate
-        reports FAIL. The function never raises for gate-level failures;
-        gh-api transport failures surface as gate FAILs in the printed
-        output and contribute to the ``1`` exit code.
+        reports FAIL. Per-gate ``gh api`` transport failures surface as
+        gate FAIL lines in the printed output and contribute to the ``1``
+        exit code.
+
+    Raises:
+        SystemExit: Propagated by the initial ``_get_pr_head_sha`` call
+            with ``EXIT_CODE_GH_ERROR`` when the PR-head-SHA fetch fails
+            before any gate runs. The function does not catch this
+            exception; the caller is responsible for converting it into
+            an exit code.
     """
     head_sha = _get_pr_head_sha(owner=owner, repo=repo, number=number)
     print(f"HEAD: {head_sha[:7]}\n")

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

@@ -21,7 +21,7 @@ _pr_converge_dir = Path(__file__).absolute().parent.parent
 if str(_pr_converge_dir) not in sys.path:
     sys.path.insert(0, str(_pr_converge_dir))
 
-from config.constants import (
+from pr_converge_skill_constants.constants import (
     EXIT_CODE_GH_ERROR,
     GH_REVIEWS_PATH_TEMPLATE,
     REVIEWS_PER_PAGE,

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

@@ -19,7 +19,7 @@ _pr_converge_dir = Path(__file__).absolute().parent.parent
 if str(_pr_converge_dir) not in sys.path:
     sys.path.insert(0, str(_pr_converge_dir))
 
-from config.constants import (
+from pr_converge_skill_constants.constants import (
     COPILOT_LOGIN_FILTER_SUBSTRING,
     GH_REVIEWS_PATH_TEMPLATE,
     REVIEWS_PER_PAGE,

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

@@ -24,7 +24,7 @@ _pr_converge_dir = Path(__file__).absolute().parent.parent
 if str(_pr_converge_dir) not in sys.path:
     sys.path.insert(0, str(_pr_converge_dir))
 
-from config.constants import (
+from pr_converge_skill_constants.constants import (
     EXIT_CODE_GH_ERROR,
     GH_INLINE_COMMENT_REPLY_PATH_TEMPLATE,
     GH_ISSUE_COMMENT_CREATE_PATH_TEMPLATE,

package/skills/pr-converge/scripts/{config → pr_converge_scripts_constants}/pr_converge_constants.py

@@ -11,7 +11,7 @@ settings) live here.
 import re
 from pathlib import Path
 
-from config.constants import (  # noqa: F401
+from pr_converge_skill_constants.constants import (  # noqa: F401
     ALL_BUGBOT_CHECK_RUN_ACTIVE_STATUSES,
     ALL_CLAUDE_DIRTY_REVIEW_STATES,
     ALL_COPILOT_DIRTY_REVIEW_STATES,

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

@@ -9,22 +9,9 @@ Run: python3 packages/claude-dev-env/skills/pr-converge/scripts/reflow_skill_md.
 
 from __future__ import annotations
 
-import sys
 import textwrap
-from pathlib import Path
 
-script_directory = str(Path(__file__).resolve().parent)
-
-while script_directory in sys.path:
-    sys.path.remove(script_directory)
-if script_directory not in sys.path:
-    sys.path.insert(0, script_directory)
-
-from evict_cached_config_modules import evict_cached_config_modules
-
-evict_cached_config_modules()
-
-from config.reflow_skill_md_constants import (
+from pr_converge_scripts_constants.reflow_skill_md_constants import (
     BASH_CONTINUATION_MARKER_WIDTH,
     BULLET_LIST_ITEM_PATTERN as BULLET_RE,
     MARKDOWN_REFERENCE_DEFINITION_PATTERN as REF_DEF_RE,
@@ -36,6 +23,14 @@ from config.reflow_skill_md_constants import (
 
 
 def wrap_paragraph_plain(text: str) -> list[str]:
+    """Wrap a plain paragraph to MAX_WIDTH after collapsing whitespace.
+
+    Args:
+        text: Paragraph text with internal whitespace runs to collapse.
+
+    Returns:
+        Wrapped lines; empty list when the input collapses to nothing.
+    """
     collapsed = " ".join(text.split())
     if not collapsed:
         return []
@@ -48,6 +43,17 @@ def wrap_paragraph_plain(text: str) -> list[str]:
 
 
 def wrap_list_item(lead_ws: str, marker: str, body: str) -> list[str]:
+    """Wrap a list item, preserving the leading marker and indentation.
+
+    Args:
+        lead_ws: Leading whitespace before the list marker.
+        marker: List marker such as a hyphen or numeric prefix.
+        body: Item body text to wrap.
+
+    Returns:
+        Wrapped lines with the marker on the first line and matching
+        indentation on subsequent lines.
+    """
     collapsed = " ".join(body.split())
     if not collapsed:
         return [lead_ws + marker.rstrip()]
@@ -67,6 +73,16 @@ def reflow_yaml_description_block(
     all_lines: list[str],
     body_start: int,
 ) -> tuple[list[str], int]:
+    """Reflow the YAML description block until the closing fence.
+
+    Args:
+        all_lines: Full SKILL.md lines.
+        body_start: Index of the first description body line.
+
+    Returns:
+        Tuple of wrapped description lines and the index just past the
+        closing fence.
+    """
     body_parts: list[str] = []
     index = body_start
     while index < len(all_lines):
@@ -95,6 +111,16 @@ def is_table_line(line: str) -> bool:
 
 
 def is_new_logical_line(stripped: str) -> bool:
+    """Decide whether ``stripped`` starts a new logical line.
+
+    Args:
+        stripped: Candidate line with leading whitespace already removed.
+
+    Returns:
+        True when the line begins a new markdown construct (fence, heading,
+        table row, list item, reference definition, or example tag) and
+        therefore must not be merged into the prior buffer.
+    """
     if not stripped:
         return False
     if stripped.startswith("```"):
@@ -115,7 +141,16 @@ def is_new_logical_line(stripped: str) -> bool:
 
 
 def merge_without_space(buffer: str, continuation: str) -> bool:
-    """Join without space only for split markdown link URL paths."""
+    """Join without space only for split markdown link URL paths.
+
+    Args:
+        buffer: Accumulated line preceding the candidate continuation.
+        continuation: Next line to evaluate for joining.
+
+    Returns:
+        True when continuation is the tail of a split markdown link target
+        and buffer ends inside an unfinished link target.
+    """
     base = buffer.rstrip()
     stripped = continuation.lstrip()
     if not base or not stripped:
@@ -126,6 +161,15 @@ def merge_without_space(buffer: str, continuation: str) -> bool:
 
 
 def merge_soft_breaks(all_lines: list[str]) -> list[str]:
+    """Merge soft line breaks across non-fence markdown paragraphs.
+
+    Args:
+        all_lines: Raw SKILL.md lines.
+
+    Returns:
+        Lines with each paragraph collapsed to a single buffer line; fences
+        and blank lines are preserved verbatim.
+    """
     reflowed_lines: list[str] = []
     index = 0
     is_inside_fence = False
@@ -166,6 +210,15 @@ def merge_soft_breaks(all_lines: list[str]) -> list[str]:
 
 
 def reflow_merged_line(line: str) -> list[str]:
+    """Reflow a single merged buffer line into MAX_WIDTH-bounded lines.
+
+    Args:
+        line: Buffer line produced by merge_soft_breaks.
+
+    Returns:
+        Wrapped lines; structural constructs (fences, tables, separators)
+        are returned unchanged.
+    """
     stripped = line.strip()
     if stripped == "":
         return [""]
@@ -221,6 +274,14 @@ def reflow_merged_line(line: str) -> list[str]:
 
 
 def reflow_markdown_body(all_lines: list[str]) -> list[str]:
+    """Merge soft breaks then reflow every line of the SKILL.md body.
+
+    Args:
+        all_lines: Raw SKILL.md body lines following the YAML front matter.
+
+    Returns:
+        Reflowed body lines bounded by MAX_WIDTH.
+    """
     merged = merge_soft_breaks(all_lines)
     reflowed_lines: list[str] = []
     for each_line in merged:
@@ -232,7 +293,15 @@ def reflow_markdown_body(all_lines: list[str]) -> list[str]:
 
 
 def wrap_long_bash_fence_lines(all_lines: list[str]) -> list[str]:
-    """Hard-wrap only ```bash fence bodies that still exceed MAX_WIDTH."""
+    """Hard-wrap bash fence bodies that still exceed MAX_WIDTH.
+
+    Args:
+        all_lines: SKILL.md body lines after paragraph reflow.
+
+    Returns:
+        Lines with overlong bash-fence bodies split on whitespace with a
+        trailing backslash continuation marker.
+    """
     wrapped_lines: list[str] = []
     is_inside_bash_fence = False
     for each_line in all_lines:
@@ -272,6 +341,11 @@ def wrap_long_bash_fence_lines(all_lines: list[str]) -> list[str]:
 
 
 def main() -> None:
+    """Read SKILL.md, reflow it to MAX_WIDTH, and write the result back.
+
+    Raises:
+        SystemExit: When the file does not start with YAML front matter.
+    """
     raw = SKILL_PATH.read_text(encoding="utf-8")
     lines = raw.splitlines()
     if not lines or lines[0].strip() != "---":

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

@@ -304,3 +304,21 @@ def should_bypass_bugbot_gates_when_bugbot_down_is_true(
     assert "_check_bugbot_not_dirty" not in all_invocation_names
     assert "bypassed (bugbot_down)" in captured_stdout
     assert exit_code == 0
+
+
+def should_propagate_systemexit_from_get_pr_head_sha(
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    def stub_get_pr_head_sha_raising_systemexit(
+        *, owner: str, repo: str, number: int
+    ) -> str:
+        raise SystemExit(check_convergence.EXIT_CODE_GH_ERROR)
+
+    monkeypatch.setattr(
+        check_convergence, "_get_pr_head_sha", stub_get_pr_head_sha_raising_systemexit
+    )
+
+    with pytest.raises(SystemExit) as exc_info:
+        check_convergence.check_all(owner="o", repo="r", number=1, bugbot_down=False)
+
+    assert exc_info.value.code == check_convergence.EXIT_CODE_GH_ERROR

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

@@ -108,27 +108,6 @@ def test_reflow_merged_line_preserves_long_markdown_reference_definition() -> No
     assert reflow_module.reflow_merged_line(line) == [stripped_line]
 
 
-def test_reflow_bootstrap_moves_script_directory_ahead_of_shadow_config(
-    tmp_path: Path,
-) -> None:
-    """sys.path bootstrap must move the script directory ahead of shadow config packages."""
-    shadow_config_directory = tmp_path / "shadow" / "config"
-    shadow_config_directory.mkdir(parents=True)
-    (shadow_config_directory / "__init__.py").write_text("", encoding="utf-8")
-    (shadow_config_directory / "pr_converge_constants.py").write_text(
-        "BROKEN = True\n", encoding="utf-8"
-    )
-    original_sys_path = list(sys.path)
-    try:
-        sys.path.insert(0, str(tmp_path / "shadow"))
-        loaded_module = _load_module()
-        assert loaded_module.MAX_WIDTH == 80
-        assert sys.path[0] == str(_SCRIPTS_DIRECTORY)
-        assert sys.path.count(str(_SCRIPTS_DIRECTORY)) == 1
-    finally:
-        sys.path[:] = original_sys_path
-
-
 def test_wrap_long_bash_fence_lines_uses_continuation_marker_for_long_lines() -> None:
     """Wrapped continuation lines use the bash continuation marker."""
     long_line = "echo " + "word " * 20
@@ -150,13 +129,3 @@ def test_reflow_uses_config_constant_for_continuation_marker_width() -> None:
         "reflow_skill_md.py must import BASH_CONTINUATION_MARKER_WIDTH from config"
     )
 
-def test_reflow_bootstrap_matches_code_rules_sys_path_pattern() -> None:
-    """Bootstrap must guard insert with a membership check."""
-    module_path = _SCRIPTS_DIRECTORY / "reflow_skill_md.py"
-    source = module_path.read_text(encoding="utf-8")
-    assert "while script_directory in sys.path:" in source, (
-        "Bootstrap must dedup script_directory entries before insert"
-    )
-    assert "sys.path.insert(0, script_directory)" in source, (
-        "Bootstrap must insert script_directory at index 0"
-    )