drydock-cli 2.9.54__tar.gz → 2.9.55__tar.gz

{drydock_cli-2.9.54 → drydock_cli-2.9.55}/PKG-INFO

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

{drydock_cli-2.9.54 → drydock_cli-2.9.55}/drydock/admiral/llm_analyzer.py

@@ -143,8 +143,13 @@ async def analyze(agent_loop: AgentLoop, finding: Finding) -> str | None:
     # Strip reasoning tokens that some models leak.
     if text.startswith("STUMPED"):
         return None
-    if "DIRECTIVE:" in text:
-        proposal = text.split("DIRECTIVE:", 1)[1].strip()
+    # The expected format is `DIRECTIVE [Bx,Cy]: <text>` — the colon comes
+    # after the bracket group, not immediately after "DIRECTIVE". A plain
+    # `"DIRECTIVE:" in text` check fails for this format, so use a regex.
+    import re as _re
+    m = _re.search(r"DIRECTIVE(?:\s*\[[^\]]*\])?\s*:(.*)", text, _re.DOTALL)
+    if m:
+        proposal = m.group(1).strip()
         # Take only the first line/paragraph — discard chain-of-thought.
         proposal = proposal.split("\n\n")[0].strip()
         return proposal or None

{drydock_cli-2.9.54 → drydock_cli-2.9.55}/drydock/curiosity/gap_detector.py

@@ -34,6 +34,11 @@ _STOPWORDS: frozenset[str] = frozenset({
     # Common imperative openers for drydock tasks.
     "Build", "Fix", "Add", "Remove", "Update", "Refactor", "Test",
     "Run", "Check", "Review", "Show", "List", "Explain", "Find", "Look",
+    # Sentence-starting verbs that appear after periods in numbered lists
+    # ("scratch.Step 1", "fail.Step 2") — dotted-identifier regex treats
+    # these as module.attr but they're sentence boundaries.
+    "Step", "Stop", "Start", "Read", "Write", "Open", "Use", "See",
+    "Try", "Note", "Save", "Load", "Copy", "Move", "Call", "Pass",
     # HLE/exam prose openers — flagged in 2026-05-14 queue audit
     # because they always lead a Title-Case phrase ("Consider the X").
     "Consider", "Suppose", "Given", "Let", "Define", "Compute",
@@ -85,6 +90,19 @@ _TEMPLATE_NOISE: frozenset[str] = frozenset(
         "CI", "CD", "PR", "MR", "WIP",
         "SSH", "SSL", "TLS", "JWT", "OAUTH", "SAML",
         "CPU", "GPU", "RAM", "SSD", "HDD", "NFS",
+        # Date/time format placeholders — appear in PRD prompts as template
+        # strings, never a meaningful GraphRAG ingest target.
+        "YYYY", "YY", "MM", "DD", "HH", "SS", "UTC", "GMT",
+        # Open-source license / protocol names — always known in coding context
+        "APACHE", "APACHE2", "MIT", "BSD", "GPL", "LGPL", "MPL", "ISC",
+        # Generic status/signal words that match _RE_ACRONYM but carry no
+        # knowledge gap — 2026-06-04 queue audit top false positives.
+        "ERROR", "WARNING", "INFO", "DEBUG", "CRITICAL", "FATAL",
+        "TRACE", "VERBOSE", "NOTICE",
+        "STATUS", "STATE", "MODE", "TYPE", "KIND", "FLAG", "OPTION",
+        "FINDINGS", "PATTERNS", "REGEXES", "FIELDS", "METHODS",
+        "VERSION", "NAME", "PATH", "DATA", "OUTPUT", "INPUT",
+        "CONFIG", "SCHEMA", "FORMAT", "TEMPLATE", "LAYOUT",
     }
 )
 
@@ -125,9 +143,50 @@ def _is_template_noise(candidate: str) -> bool:
         ".py", ".md", ".txt", ".csv", ".json", ".yaml", ".yml",
         ".toml", ".log", ".html", ".xml", ".sql", ".sh", ".rs",
         ".go", ".ts", ".js", ".jsx", ".tsx", ".css", ".scss",
+        ".bak", ".tmp", ".lock", ".pid", ".gz", ".zip", ".tar",
     )
     if "/" not in norm and any(norm.endswith(ext) for ext in _CODE_EXTS):
         return True
+    # Dotted identifier where the post-dot component is a common English
+    # word → sentence boundary artifact ("scratch.Step", "fail.Stop").
+    if "." in norm:
+        parts = norm.split(".")
+        _sw_lower = {w.lower() for w in _STOPWORDS}
+        if any(p in _sw_lower for p in parts[1:]):
+            return True
+    # LaTeX math expressions — false positives from HLE/academic prompts.
+    # A term starting with $ or a LaTeX command backslash is math notation,
+    # not a GraphRAG-retrievable identifier.
+    if norm.startswith("$") or norm.startswith("\\"):
+        return True
+    # Pure numeric/punctuation strings — quoted answer-choice values like
+    # "33,1" or "0.5" from HLE prompts; never a meaningful ingest target.
+    if re.match(r"^[\d\s,./\-+%()]+$", norm):
+        return True
+    # Python dunder attributes (__all__, __init__.__all__, etc.) are always
+    # known in a coding context — never a GraphRAG gap.
+    if "__" in norm:
+        return True
+    # Test file / function names (test_cli, test_routes) are project artifacts
+    # produced by the agent, not external knowledge worth retrieving.
+    if norm.startswith("test_") or norm.startswith("tests_"):
+        return True
+    # Type-conversion function names (to_int, to_roman, to_snake_case) —
+    # generic utility patterns, not external library gaps.
+    if norm.startswith("to_"):
+        return True
+    # All-lowercase multi-word phrases (e.g. "buy milk", "sample logs") —
+    # quoted prose from task prompts, not identifiers worth retrieving.
+    # Only filter if the *original* candidate is already lowercase — Title Case
+    # phrases like "Attention Is All You Need" must still be detected.
+    orig = candidate.strip(" :.,;")
+    if " " in norm and orig == orig.lower() and re.match(r"^[a-z][a-z0-9 _-]+$", norm):
+        return True
+    # Date format strings like "YYYY-MM", "YYYY-MM-DD", "/DD/YYYY" — these
+    # are template tokens that contain YYYY/MM/DD components (now in
+    # TEMPLATE_NOISE) separated by punctuation but the combined string isn't.
+    if re.match(r"^[/\-]?(?:yyyy|mm|dd|hh|ss)(?:[/\-](?:yyyy|mm|dd|hh|ss))+$", norm):
+        return True
     return False
 
 # Acronyms shorter than this are too noisy to chase ("ID", "OK", "OS").

{drydock_cli-2.9.54 → drydock_cli-2.9.55}/pyproject.toml

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

{drydock_cli-2.9.54 → drydock_cli-2.9.55}/tests/test_curiosity.py

@@ -112,6 +112,40 @@ class TestGapDetector:
         for noise in ("Consider", "Suppose", "Consider the"):
             assert noise not in gaps, f"detected noise: {noise!r} in {gaps}"
 
+    def test_drops_sentence_boundary_dotted_idents(self):
+        """Numbered-step prompts produce 'scratch.Step', 'fail.Step', etc.
+        when the gap-detector sees period+uppercase-word at a sentence
+        boundary.  These must not be queued as unknown terms (2026-06-04
+        curiosity queue filled with 9988 junk entries from one session)."""
+        text = (
+            "DO NOT scaffold from scratch.Step 1: cd here and run pytest -q."
+            " You'll see 24 tests pass and 3 fail.Step 2: Read the output."
+        )
+        gaps = detect_gaps(text)
+        for noise in ("scratch.Step", "fail.Step", "green.Stop"):
+            assert noise not in gaps, f"detected noise: {noise!r} in {gaps}"
+        # Real dotted identifiers (module.attr) must still be detected.
+        real = detect_gaps("Use sklearn.metrics.f1_score here.")
+        assert any("sklearn" in g for g in real), real
+
+    def test_drops_to_x_converter_names(self):
+        """to_int, to_roman, to_snake_case etc. are project-defined converters,
+        not external library gaps. The to_ prefix alone is sufficient."""
+        for term in ("to_int", "to_roman", "to_str", "to_bool", "to_snake_case"):
+            gaps = detect_gaps(f"Implement `{term}` function.")
+            assert term not in gaps, f"to_X converter leaked into gaps: {term!r} in {gaps}"
+
+    def test_drops_lowercase_prose_phrases(self):
+        """Quoted lowercase multi-word phrases like 'buy milk' or 'sample logs'
+        are prose noise from task prompts, not retrievable identifiers."""
+        for phrase in ("buy milk", "sample logs", "backend disk"):
+            gaps = detect_gaps(f'Remember to "{phrase}" today.')
+            assert phrase not in gaps, f"prose phrase leaked into gaps: {phrase!r} in {gaps}"
+        # Real snake_case identifiers with spaces should still be detected
+        # via the snake_case pattern (no space version).
+        real = detect_gaps("Call backend_memory.flush() to clear.")
+        assert any("backend_memory" in g for g in real), real
+
     def test_keeps_real_terms_amid_hle_noise(self):
         """Validates the fix isn't over-broad: real entity names must
         survive alongside the boilerplate."""

{drydock_cli-2.9.54 → drydock_cli-2.9.55}/uv.lock

@@ -326,7 +326,7 @@ wheels = [
 
 [[package]]
 name = "drydock-cli"
-version = "2.9.49"
+version = "2.9.54"
 source = { editable = "." }
 dependencies = [
     { name = "agent-client-protocol" },