claude-dev-env 1.42.0 → 1.44.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"
-    )

package/skills/pre-compact/SKILL.md

@@ -0,0 +1,114 @@
+---
+name: pre-compact
+description: >-
+  Composes a focus directive for `/compact [instructions]` and copies the full
+  `/compact <directive>` string to the operator's clipboard so the next prompt
+  is a single paste. The directive pins the session's load-bearing identifiers
+  (branch, PR, HEAD, worktree, in-flight work, decisions, blockers, files in
+  play, follow-ups) and lists the redundant tool outputs the summarizer should
+  drop. Use when the user says `/pre-compact`, asks to prep for compaction, or
+  asks to compose a focus directive for `/compact`.
+disable-model-invocation: true
+---
+
+# Pre-Compact
+
+`/compact [instructions]` accepts a focus directive that steers the
+compaction-summary LLM toward high-signal content. This skill writes that
+directive from the live session and copies the full `/compact <directive>`
+string to the operator's clipboard.
+
+**Announce at start:** "I'm composing your compact focus directive."
+
+## Step 1 — Read the live session
+
+Pull the load-bearing identifiers from the current conversation. Run
+`git status`, `git rev-parse --short HEAD`, or `gh pr view` when values are
+not already in context.
+
+| Field | What to capture | Source |
+|---|---|---|
+| `branch` | Active branch name | `git branch --show-current` |
+| `pr` | Active PR number, when one exists | `gh pr view --json number` |
+| `head` | Short HEAD SHA (whatever `git rev-parse --short` outputs) | `git rev-parse --short HEAD` |
+| `worktree` | Absolute path to the working directory | `pwd` |
+| `in_flight` | One sentence describing what is being worked on right now | conversation |
+| `decisions` | Architectural choices, library picks, tradeoffs settled this session | conversation |
+| `blockers` | Failures observed, root causes identified, fixes pending | conversation |
+| `files` | Paths the operator is iterating on (edited or read more than once) | conversation |
+| `follow_ups` | What the user asked to be remembered or revisited | conversation |
+
+A field whose value cannot be stated as a concrete identifier is omitted
+from the directive.
+
+## Step 2 — Render the directive
+
+Render this exact shape, populating only the fields with concrete values:
+
+```
+Preserve:
+- Branch: <name>
+- PR: #<number>
+- HEAD: <short-sha>
+- Worktree: <path>
+- In-flight: <one sentence>
+- Decisions: <bullet per decision>
+- Blockers: <bullet per blocker>
+- Files: <path>, <path>, <path>
+- Follow-ups: <bullet per follow-up>
+
+Drop:
+- Tool outputs already applied to files
+- Per-tick progress narration
+- Resolved findings and superseded SHAs
+- Listing/grep output whose conclusion appears above
+```
+
+The `Preserve:` block leads so the summarizer maximizes recall first. The
+`Drop:` block lists the lightest-touch removals — raw tool outputs are the
+safest content to drop because the work they produced lives in the files
+and commits.
+
+Source: [Effective context engineering for AI agents](https://www.anthropic.com/engineering/effective-context-engineering-for-ai-agents)
+— "start by maximizing recall to ensure your compaction prompt captures
+every relevant piece of information from the trace, then iterate to improve
+precision by eliminating superfluous content."
+
+## Step 3 — Copy `/compact <directive>` to the clipboard
+
+Write the full `/compact <directive>` string to a temporary file via the
+Write tool, then copy the file contents to the clipboard with PowerShell:
+
+```
+pwsh -NoProfile -Command "Get-Content -LiteralPath '<temp file path>' -Raw | Set-Clipboard"
+```
+
+`Get-Content -LiteralPath … -Raw` reads the file as a single string with
+no wildcard expansion, and `Set-Clipboard` writes it verbatim. The intermediate file keeps the directive content out
+of any shell-parsing path: session text passes through `Get-Content`
+unmodified regardless of which characters it contains.
+
+A reasonable temp path under `$env:TEMP` (Windows) or `$TMPDIR` (POSIX)
+works; clean it up after the `Set-Clipboard` call returns.
+
+## Step 4 — Hand off
+
+Print this confirmation line to the operator:
+
+> Copied `/compact …` to your clipboard. Paste it as your next prompt to
+> compact this conversation with focus.
+
+Then list up to the first three `Preserve:` bullets (or fewer when the
+directive omits fields) and the first `Drop:` bullet inline so the
+operator can spot-check before pasting.
+
+---
+
+## References
+
+- `/compact [instructions]` — [Claude Code commands](https://code.claude.com/docs/en/commands)
+- Compaction strategy — [Effective context engineering for AI agents](https://www.anthropic.com/engineering/effective-context-engineering-for-ai-agents)
+
+## Folder map
+
+- `SKILL.md` — hub. No companions.