drydock-cli 2.7.39__tar.gz → 2.7.41__tar.gz

drydock_cli-2.7.41/.auto_release.lock

@@ -0,0 +1 @@
+2874397

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 2.7.39
+Version: 2.7.41
 Summary: Local-first CLI coding agent — tested with Gemma 4 26B via vLLM
 Project-URL: Homepage, https://github.com/fbobe321/drydock
 Project-URL: Repository, https://github.com/fbobe321/drydock

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/drydock/cli/textual_ui/widgets/messages.py

@@ -118,6 +118,16 @@ _SENTENCE_BOUNDARY_RE = re.compile(r"(?<=[.!?]) (?=[A-Z])")
 _INLINE_CAPWORD_HEADING_RE = re.compile(
     r"(?<=[.!?]) (?=[A-Z][A-Za-z]{1,15}:\s)"
 )
+# Issue #16: quantized Gemma 4 (Q3_K_M) strips whitespace tokens, jamming
+# `### Header text body` and `* item * item * item` runs onto one line.
+# These patterns insert paragraph breaks before inline markdown headers
+# and asterisk bullets that weren't preceded by a real newline.
+_INLINE_ATX_HEADING_RE = re.compile(r"(?<=[.!?\)]) (?=#{1,4} [A-Z])")
+# Asterisk-bullet rescue. Accepts either case for the first bullet word
+# (file names start lowercase, sentences start uppercase). Requires the
+# bullet word to be ≥3 alpha chars + dot/space — guards against math
+# (`2 * 3`) and inline emphasis (`a * b`).
+_INLINE_ASTERISK_BULLET_RE = re.compile(r"(?<=[a-zA-Z.!?\)]) (?=\* [A-Za-z][A-Za-z0-9_]{2,})")
 
 
 def _sentence_chunks(text: str, group: int = 2) -> str:
@@ -164,6 +174,10 @@ def _break_walls_of_text(text: str) -> str:
     out = _INLINE_ENUM_RE.sub("\n", out)
     out = _INLINE_DASH_BULLET_RE.sub("\n", out)
     out = _INLINE_CAPWORD_HEADING_RE.sub("\n\n", out)
+    # Issue #16: rescue inline ATX headers (`### Foo`) and asterisk bullets
+    # (`* item`) that quantized Gemma 4 emits without surrounding newlines.
+    out = _INLINE_ATX_HEADING_RE.sub("\n\n", out)
+    out = _INLINE_ASTERISK_BULLET_RE.sub("\n", out)
     # If inline markers didn't produce any breaks AND the text is long
     # enough that one paragraph hurts readability, split at sentence
     # boundaries. group=1 = each sentence on its own paragraph (most

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/drydock/core/agent_loop.py

@@ -995,7 +995,16 @@ class AgentLoop:
 
             # Empty response — try to recover inline.
             if _stall_attempt >= MAX_STALL_RETRIES:
-                _dbg(f"[STALL-DEBUG] max retries ({MAX_STALL_RETRIES}) exhausted; leaving empty")
+                _dbg(f"[STALL-DEBUG] max retries ({MAX_STALL_RETRIES}) exhausted; injecting fallback")
+                # Replace the silent empty message with visible text so the
+                # harness and the user both see a clean end-of-turn rather
+                # than a frozen TUI waiting for content that never arrives.
+                last_message.content = (
+                    "[Drydock: model returned an empty response after "
+                    f"{MAX_STALL_RETRIES} retries. Please rephrase your "
+                    "request or use /clear to reset context.]"
+                )
+                yield AssistantEvent(content=last_message.content)
                 break
             prev_role = self.messages[-2].role if len(self.messages) >= 2 else None
             if prev_role not in (Role.tool, Role.user):
@@ -1918,9 +1927,18 @@ class AgentLoop:
                     and not (msg.content or "").strip()
                     and not msg.tool_calls):
                 # Skip empty assistant — also drop any immediately preceding
-                # tool result that would be left dangling without a matching
-                # assistant turn.
+                # tool result that would be truly orphaned (no matching
+                # assistant.tool_calls entry in the preceding messages).
                 while cleaned2 and cleaned2[-1].role == Role.tool:
+                    preceding_tool = cleaned2[-1]
+                    tcid = getattr(preceding_tool, "tool_call_id", None)
+                    if tcid and any(
+                        m.role == Role.assistant
+                        and m.tool_calls
+                        and any(tc.id == tcid for tc in m.tool_calls)
+                        for m in cleaned2[:-1]
+                    ):
+                        break  # tool result has a valid match; keep it
                     cleaned2.pop()
             else:
                 cleaned2.append(msg)
@@ -2074,22 +2092,37 @@ class AgentLoop:
             temp = active_model.temperature
             extra_sampling: dict | None = None
 
+            # Per-model `extra_params` (top_k, top_p, frequency_penalty,
+            # max_tokens, etc.) declared in config.toml flow through
+            # extra_sampling. This is the seam llama.cpp users need to
+            # pass top_k=40, top_p=0.95, frequency_penalty=1.1 per the
+            # Gemma 4 loop-fix recipe.
+            cfg_extra = getattr(active_model, "extra_params", None) or {}
+            if cfg_extra:
+                extra_sampling = dict(cfg_extra)
+
             # Token-level loop-breaker: when repetition is detected, bump
             # temperature and add frequency_penalty + a fresh seed so the
             # model's next completion is mechanically likely to diverge.
             # Mistral/OpenAI-compat backends pass these straight through
-            # to vLLM's SamplingParams.
+            # to vLLM's SamplingParams. These OVERRIDE config.extra_params
+            # for the duration of the loop-break.
             if getattr(self, "_loop_detected", False):
                 signal = getattr(self, "_loop_signal", "") or ""
                 # Heavier bump if we've already hit the FORCE_STOP signal
                 # (=8 repeats) vs a WARNING (=3-5 repeats).
                 heavy = signal == "FORCE_STOP"
                 temp = min(1.0, temp + (0.5 if heavy else 0.3))
-                extra_sampling = {
+                # Merge loop-breaker overrides on top of any cfg_extra so
+                # config-declared sampling params survive when a loop is
+                # NOT detected, and get overridden when one IS detected.
+                if extra_sampling is None:
+                    extra_sampling = {}
+                extra_sampling.update({
                     "frequency_penalty": 0.7 if heavy else 0.4,
                     "presence_penalty": 0.3,
                     "seed": int(time.time() * 1000) & 0x7FFFFFFF,
-                }
+                })
                 logger.info(
                     "[LOOP-BREAK] %s → temp %.2f, freq_pen %.2f, seed %d",
                     signal, temp, extra_sampling["frequency_penalty"],

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/drydock/core/config/_settings.py

@@ -238,6 +238,18 @@ class ModelConfig(BaseModel):
     output_price: float = 0.0  # Price per million output tokens
     thinking: Literal["off", "low", "medium", "high"] = "off"
     auto_compact_threshold: int = 200_000
+    # Free-form sampling overrides forwarded to the LLM backend's
+    # extra_sampling (top_k, top_p, frequency_penalty, repeat_penalty,
+    # min_p, max_tokens, etc). Anything supported by the OpenAI-compat
+    # server. For llama.cpp + Gemma 4 the article-recommended config is:
+    #   [models.extra_params]
+    #   top_k = 40
+    #   top_p = 0.95
+    #   frequency_penalty = 1.1
+    #   max_tokens = 2048
+    # Set temperature via the top-level `temperature` field; do not
+    # duplicate it here.
+    extra_params: dict[str, Any] = Field(default_factory=dict)
 
     @model_validator(mode="before")
     @classmethod

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/drydock/core/config/local_detect.py

@@ -117,4 +117,17 @@ def patch_config_for_local(
         for model in models:
             if isinstance(model, dict) and model.get("alias") == "local":
                 model["name"] = info.model_name
+                # When the detected backend is llama.cpp, bake in the
+                # Gemma 4 anti-loop sampling recipe (article: i-built-a-
+                # gemma-4-ai-agent-it-kept-looping). Temperature must be
+                # 1.0; lower values reinforce loops on quantized GGUF.
+                # Don't clobber user-defined values if config was already
+                # touched.
+                if info.label == "llama.cpp":
+                    model.setdefault("temperature", 1.0)
+                    extra = model.setdefault("extra_params", {})
+                    extra.setdefault("top_k", 40)
+                    extra.setdefault("top_p", 0.95)
+                    extra.setdefault("frequency_penalty", 1.1)
+                    extra.setdefault("max_tokens", 2048)
                 break

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/drydock/core/prompts/gemma4.md

@@ -140,3 +140,12 @@ When you're asked to BUILD a package: always also write a
 `strong_tests.sh` that exercises the PRD's specific examples and
 roundtrip/state properties. A package with only `functional_tests.sh`
 is not finished.
+
+FORMATTING — when answering with prose (not tool calls):
+- Always put a BLANK LINE before any `###`/`##`/`#` header.
+- Always put a BLANK LINE between paragraphs.
+- Always put a BLANK LINE before the FIRST item of a bullet list and a
+  newline between items. Do NOT pack `* a * b * c` onto one line.
+- Use `\n\n` between sections. Quantized models drop whitespace tokens
+  to save length; the renderer cannot always recover the original
+  structure if you do.

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/drydock/core/tools/builtins/bash.py

@@ -636,11 +636,14 @@ class Bash(
             # command text is run 5+ times regardless of output or exit code.
             # Catches benchmark/timeit/import-check loops where each run has
             # varying output (timings, object ids) that defeats the hash check.
-            # Excludes commands that write files (heredoc, redirect, sed -i)
-            # since those legitimately run the same form multiple times.
+            # Excludes redirect/heredoc writes since those legitimately append
+            # the same content in loops. Does NOT exclude `sed -i`: when the
+            # sed pattern is wrong (e.g. escape mismatch) it exits 0 but makes
+            # no change, so the model loops forever — the repetition check
+            # must fire to break it (harness:bash:escape_loop pattern).
             _is_write_cmd = bool(
                 __import__("re").search(
-                    r"(>|>>|\bsed\s+-i\b|\btee\b|\bcat\s+<<|\bprintf\b.*>)",
+                    r"(>|>>|\btee\b|\bcat\s+<<|\bprintf\b.*>)",
                     args.command,
                 )
             )
@@ -700,7 +703,7 @@ class Bash(
                     # "EDIT SOURCE CODE" confuses it — give a targeted hint.
                     import re as _re
                     _is_heredoc_write = bool(_re.search(
-                        r"cat\s+<<\s*['\"]?EOF['\"]?\s+>", args.command
+                        r"cat\s+<<\s*['\"]?[A-Za-z_]+['\"]?\s+>+", args.command
                     ))
                     # Detect ls/grep/find search that returned empty output.
                     # The model runs `ls -F | grep "test_cli"` repeatedly when

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/drydock/core/tools/builtins/search_replace.py

@@ -1069,6 +1069,17 @@ class SearchReplace(
 
                 if fuzzy_context:
                     error_msg += f"\n{fuzzy_context}"
+                elif "not found anywhere" in context:
+                    # First search line not found at all — embed file head so the
+                    # model can craft a correct SEARCH block without calling read_file.
+                    head_lines = current_content.split("\n")[:30]
+                    head = "\n".join(head_lines)
+                    error_msg += (
+                        f"\nFile head (first {len(head_lines)} lines of actual content):\n"
+                        f"```\n{head}\n```\n"
+                        "Read the file head above and craft your SEARCH block using "
+                        "text that actually appears in the file."
+                    )
 
                 error_msg += (
                     "\nDebugging tips:\n"

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/drydock/core/tools/builtins/write_file.py

@@ -470,39 +470,37 @@ class WriteFile(
 
                     body = "File already has this exact content (no-op). "
                     if repeat_n >= 2:
-                        try:
-                            siblings = sorted(
-                                p.name for p in file_path.parent.iterdir()
-                                if p.is_file() and not p.name.startswith("__pycache__")
-                            )
-                            sibling_str = ", ".join(siblings) if siblings else "(none)"
-                            body += (
-                                f"You've written this exact file {repeat_n} times. "
-                                f"Files currently in {file_path.parent.name}/: "
-                                f"{sibling_str}. Move on — write a DIFFERENT file "
-                                f"from your plan, or run `bash` with `python3 -m "
-                                f"{file_path.parent.name} --help` to verify."
-                            )
-                            # Also surface missing imports so the model knows
-                            # which file to write next instead of re-looping.
-                            if file_path.suffix == ".py":
-                                try:
-                                    import ast as _ast
-                                    _tree = _ast.parse(args.content)
-                                    missing = _check_missing_sibling_imports(
-                                        _tree, file_path
+                        body += f"You've written this exact file {repeat_n} times. "
+                    try:
+                        siblings = sorted(
+                            p.name for p in file_path.parent.iterdir()
+                            if p.is_file() and not p.name.startswith("__pycache__")
+                        )
+                        sibling_str = ", ".join(siblings) if siblings else "(none)"
+                        body += (
+                            f"Files currently in {file_path.parent.name}/: "
+                            f"{sibling_str}. Move on — write a DIFFERENT file "
+                            f"from your plan, or run `bash` with `python3 -m "
+                            f"{file_path.parent.name} --help` to verify."
+                        )
+                        # Surface missing imports so the model knows which file
+                        # to write next instead of re-looping.
+                        if file_path.suffix == ".py":
+                            try:
+                                import ast as _ast
+                                _tree = _ast.parse(args.content)
+                                missing = _check_missing_sibling_imports(
+                                    _tree, file_path
+                                )
+                                if missing:
+                                    body += (
+                                        f" This file imports modules that don't "
+                                        f"exist yet: {sorted(missing)}. "
+                                        f"Write those files next."
                                     )
-                                    if missing:
-                                        body += (
-                                            f" This file imports modules that don't "
-                                            f"exist yet: {sorted(missing)}. "
-                                            f"Write those files next."
-                                        )
-                                except Exception:
-                                    pass
-                        except Exception:
-                            body += "Move to the NEXT file in your plan."
-                    else:
+                            except Exception:
+                                pass
+                    except Exception:
                         body += "Move to the NEXT file in your plan."
                     msg = f"<system-reminder>\n{body}\n</system-reminder>"
 

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "drydock-cli"
-version = "2.7.39"
+version = "2.7.41"
 description = "Local-first CLI coding agent — tested with Gemma 4 26B via vLLM"
 readme = "README.md"
 requires-python = ">=3.12"

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/scripts/hle_eval.py

@@ -210,10 +210,15 @@ def score_answer(q: dict, pred: str) -> dict:
 def notify_telegram(tag: str, message: str) -> None:
     """Fire-and-forget Telegram ping via the existing notify_release.py.
 
-    All HLE runs notify on start, every MILESTONE_EVERY completions, and
-    on final completion (or crash). Failures are silent — a missed
-    Telegram is never worth crashing the eval.
+    Default OFF: HLE eval is too noisy for routine notifications and the
+    user explicitly asked to stop. Opt in via HLE_TELEGRAM=1 if you want
+    start/milestone/final/crash pings for a specific run.
+
+    Failures are silent — a missed Telegram is never worth crashing
+    the eval.
     """
+    if os.environ.get("HLE_TELEGRAM", "").strip().lower() not in ("1", "true", "yes"):
+        return
     if not NOTIFY.exists():
         return
     import subprocess

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/scripts/notify_release.py

@@ -51,6 +51,17 @@ if __name__ == "__main__":
         version = sys.argv[1] if len(sys.argv) > 1 else "unknown"
         summary = "New release"
 
+    # User explicitly asked to stop HLE telegram spam (2026-05-04).
+    # Drop HLE-prefixed tags by default; override with HLE_TELEGRAM=1.
+    # This silences the in-flight overnight HLE process too — it shells
+    # out to this script per ping, so the filter applies retroactively.
+    import os as _os
+    if (version.startswith("hle-")
+            and _os.environ.get("HLE_TELEGRAM", "").strip().lower()
+            not in ("1", "true", "yes")):
+        print(f"[notify_release] dropped HLE-tagged ping: {version}")
+        sys.exit(0)
+
     if version == "status":
         # Hourly status — plain text, no Markdown parsing issues
         msg = f"⚓ {summary}"

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/tests/test_local_detect.py

@@ -164,5 +164,79 @@ def test_patch_handles_missing_keys_gracefully():
     assert cfg["active_model"] == "local"
 
 
+def test_patch_bakes_llama_cpp_anti_loop_recipe():
+    """When the detected backend is llama.cpp, the local model should
+    inherit the article-recommended Gemma 4 anti-loop recipe — temp 1.0
+    and the extra_params block. (issue #15)"""
+    cfg = {
+        "providers": [
+            {"name": "llamacpp", "api_base": "http://127.0.0.1:8080/v1"},
+        ],
+        "models": [
+            {"alias": "local", "name": "placeholder", "provider": "llamacpp"},
+        ],
+    }
+    info = LocalServerInfo(
+        label="llama.cpp",
+        api_base="http://192.168.50.21:8000/v1",
+        model_name="gemma-4-26B-A4B-it-Q3_K_M",
+    )
+    patch_config_for_local(cfg, info)
+    local = next(m for m in cfg["models"] if m["alias"] == "local")
+    assert local["temperature"] == 1.0
+    assert local["extra_params"]["top_k"] == 40
+    assert local["extra_params"]["top_p"] == 0.95
+    assert local["extra_params"]["frequency_penalty"] == 1.1
+    assert local["extra_params"]["max_tokens"] == 2048
+
+
+def test_patch_does_not_overwrite_user_extra_params():
+    """If the user already configured extra_params, the llama.cpp
+    detector adds missing keys but keeps user values."""
+    cfg = {
+        "providers": [{"name": "llamacpp", "api_base": "http://127.0.0.1:8080/v1"}],
+        "models": [
+            {"alias": "local", "name": "p", "provider": "llamacpp",
+             "temperature": 0.5,
+             "extra_params": {"top_k": 64, "min_p": 0.05}},
+        ],
+    }
+    info = LocalServerInfo(
+        label="llama.cpp",
+        api_base="http://127.0.0.1:8080/v1",
+        model_name="gemma-4-26B-A4B-it-Q3_K_M",
+    )
+    patch_config_for_local(cfg, info)
+    local = next(m for m in cfg["models"] if m["alias"] == "local")
+    # User-set values preserved
+    assert local["temperature"] == 0.5
+    assert local["extra_params"]["top_k"] == 64
+    assert local["extra_params"]["min_p"] == 0.05
+    # Missing keys filled in with article defaults
+    assert local["extra_params"]["top_p"] == 0.95
+    assert local["extra_params"]["frequency_penalty"] == 1.1
+    assert local["extra_params"]["max_tokens"] == 2048
+
+
+def test_patch_does_not_inject_for_non_llama_backends():
+    """vLLM detection should NOT bake in the llama.cpp-specific recipe."""
+    cfg = {
+        "providers": [{"name": "llamacpp", "api_base": "http://127.0.0.1:8080/v1"}],
+        "models": [
+            {"alias": "local", "name": "p", "provider": "llamacpp"},
+        ],
+    }
+    info = LocalServerInfo(
+        label="vLLM",
+        api_base="http://127.0.0.1:8000/v1",
+        model_name="gemma4",
+    )
+    patch_config_for_local(cfg, info)
+    local = next(m for m in cfg["models"] if m["alias"] == "local")
+    # No temperature override, no extra_params injection
+    assert "temperature" not in local
+    assert local.get("extra_params", {}) == {} or "extra_params" not in local
+
+
 if __name__ == "__main__":
     raise SystemExit(pytest.main([__file__, "-v"]))

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/tests/test_wall_of_text_rescue.py

@@ -75,6 +75,36 @@ class TestBreakWallsOfText:
         assert out.count("\n") > 0
         assert out != wall
 
+    def test_inline_atx_headers_get_breaks(self):
+        """Issue #16: Q3_K_M Gemma 4 jams `### Header` markers inline."""
+        wall = (
+            "I reviewed the codebase and there are several layers worth "
+            "describing. ### Architecture The system uses a three-tier "
+            "design with clear separation. ### Components The indexer "
+            "reads markdown files into the database. ### Tests Each "
+            "module has its own pytest file with shared fixtures."
+        )
+        out = _break_walls_of_text(wall)
+        assert "\n\n### Architecture" in out
+        assert "\n\n### Components" in out
+        assert "\n\n### Tests" in out
+
+    def test_inline_asterisk_bullets_get_breaks(self):
+        """Issue #16: `* item * item * item` runs onto one line on quantized
+        Gemma 4. Insert newlines so the markdown renderer treats them as
+        a real list."""
+        wall = (
+            "Here is the project structure overview that captures the "
+            "main building blocks of the system as currently designed. "
+            "* indexer.py reads markdown * searcher.py runs BM25 "
+            "* cli.py exposes the REPL * tests/ is the pytest suite"
+        )
+        out = _break_walls_of_text(wall)
+        # Each `* item` should now be on its own line (or at least
+        # separated by a newline).
+        bullets_with_newlines = sum(1 for line in out.split("\n") if line.strip().startswith("* "))
+        assert bullets_with_newlines >= 3
+
     def test_existing_newlines_preserved(self):
         text = "Already\nhas\nbreaks. " * 30
         # nl >= 3 disables wall-rescue.

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/tests/tools/test_bash_exact_cmd_loop_breaker.py

@@ -57,3 +57,20 @@ async def test_distinct_commands_no_breaker(bash, tmp_path):
     for i in range(6):
         result = await collect_result(bash.run(BashArgs(command=f"echo cmd_{i}")))
     assert "LOOP-BREAKER" not in result.stdout
+
+
+@pytest.mark.asyncio
+async def test_sed_i_escape_loop_triggers_breaker(bash, tmp_path):
+    """sed -i with exact same command fires the loop-breaker (harness:bash:escape_loop).
+
+    When the sed pattern is malformed (e.g. escape mismatch), sed exits 0 but
+    makes no change. The model retries the same command indefinitely. The loop-
+    breaker must fire because sed -i is no longer exempt from the repetition check.
+    """
+    target = tmp_path / "demo.py"
+    target.write_text('print("n")\n')
+    # Exact same sed -i command repeated 5 times (same escaping mistake each time)
+    cmd = f"sed -i 's/print(\"n/print(\"\\\\n/g' {target}"
+    for _ in range(5):
+        result = await collect_result(bash.run(BashArgs(command=cmd)))
+    assert "LOOP-BREAKER" in result.stdout

{drydock_cli-2.7.39 → drydock_cli-2.7.41}/tests/tools/test_bash_heredoc_confirmation.py

@@ -74,3 +74,27 @@ async def test_non_heredoc_command_unaffected(bash):
     cmd = "true"
     result = await collect_result(bash.run(BashArgs(command=cmd)))
     assert "File written" not in result.stdout
+
+
+@pytest.mark.asyncio
+async def test_heredoc_loop_breaker_non_eof_delimiter(bash, tmp_path):
+    """Loop-breaker targets heredoc message for non-EOF delimiters (CONTENT, PYTHON, etc).
+
+    Before fix: regex only matched 'EOF'; non-EOF delimiters fell through to the
+    generic 'EDIT SOURCE CODE' message, which confused the model.
+    After fix: any alpha delimiter triggers the targeted heredoc hint.
+    """
+    target = tmp_path / "plugin.py"
+    cmd = f"cat <<CONTENT > {target}\nprint('hello')\nCONTENT"
+
+    # Run 3 times to trigger the hash-based loop breaker
+    for _ in range(3):
+        result = await collect_result(bash.run(BashArgs(command=cmd)))
+
+    # The 3rd run must produce the heredoc-targeted notice, not the generic one
+    assert "cat command" in result.stdout or "re-run this cat" in result.stdout, (
+        "Expected heredoc-targeted loop notice, got: " + result.stdout[:200]
+    )
+    assert "EDIT SOURCE CODE" not in result.stdout, (
+        "Got generic loop notice instead of heredoc-targeted one"
+    )