@misterhuydo/sentinel 1.2.0 → 1.2.2

package/.cairn/session.json

@@ -1,6 +1,6 @@
 {
-  "message": "Auto-checkpoint at 2026-03-23T09:22:14.601Z",
-  "checkpoint_at": "2026-03-23T09:22:14.603Z",
+  "message": "Auto-checkpoint at 2026-03-23T09:42:32.547Z",
+  "checkpoint_at": "2026-03-23T09:42:32.548Z",
   "active_files": [],
   "notes": [],
   "mtime_snapshot": {}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@misterhuydo/sentinel",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "description": "Sentinel — Autonomous DevOps Agent installer and manager",
   "bin": {
     "sentinel": "./bin/sentinel.js"

package/python/sentinel/sentinel_boss.py

@@ -187,6 +187,13 @@ Session context — critical rules:
 - When handling a new request, call the tools fresh. Do not assume any prior tool result is still current or that any prior step "counts" toward the current task.
 - The only exception: if the user explicitly asks about something from the history ("what did you find earlier?"), you may reference it — but note it is from a prior session.
 
+Trust your tool results — never contradict them:
+- If any search_logs call in this response returned total_matches > 0, you HAVE results. Report them.
+- Never say "no results found" or "nothing was found" when a tool result shows total_matches > 0.
+- If one source-specific call returns 0 but a broader call returned matches, use the broader results.
+- A cached result with files_searched=0 is a source-name lookup failure, NOT an absence of log data.
+  Treat it as "source not recognised" and fall back to the broad search results you already have.
+
 Avoid redundant tool calls (within a single response only — always run tools fresh for new requests):
 - If a broad search (e.g. search_logs with no source filter) already returned results in THIS response, do NOT repeat the same search with a source filter to "refine" — use what you already fetched.
 - If a tool call fails in THIS response, do NOT retry the entire search from scratch. Continue with what succeeded and note the failure.
@@ -201,12 +208,12 @@ Issue identification — before calling create_issue:
    - Attachments: summarise any files/screenshots the user shared.
    - Support URL: note any ticket/doc/link the user mentioned.
    - Identity: always captured automatically from the Slack session.
-3. ALWAYS populate the `findings` field with evidence from this session:
+3. Populate `findings` with curated evidence — only when relevant and concise:
    - If you ran search_logs, tail_log, ask_codebase, or get_status before creating the issue,
-     include the relevant results (log excerpts, error counts, timestamps, stack traces, etc.).
-   - The fix engine reads only the issue file — it has no access to the conversation.
-     Everything it needs to understand and fix the problem must be in the file.
-   - A `description` with no evidence forces the fix engine to guess. Always attach what you found.
+     summarise only the findings directly related to this specific issue.
+   - Do NOT paste raw tool output. Summarise: which services, how often, key pattern, 1-3 example lines.
+   - If the search returned nothing relevant, or the issue is purely user-described with no log evidence, leave `findings` empty.
+   - The fix engine reads only the issue file. Give it signal, not noise — 500 words max.
 4. Before calling the tool, confirm with the user in natural language:
    e.g. "I'll create an issue for project *1881* — here's what I have: [summary]. Look right?"
    Wait for their confirmation before proceeding.
@@ -276,11 +283,11 @@ _TOOLS = [
                 "findings": {
                     "type": "string",
                     "description": (
-                        "Evidence gathered during this session: log excerpts, search results, "
-                        "stack traces, error counts, timestamps — anything from tool calls that "
-                        "is relevant to the issue. Include this whenever you ran search_logs, "
-                        "tail_log, ask_codebase, or get_status before creating the issue. "
-                        "The fix engine reads the issue file and needs this context to act."
+                        "A concise, curated summary of evidence directly relevant to this issue — "
+                        "NOT raw tool output. Include only what the fix engine needs: "
+                        "key error patterns, affected services, approximate frequency/timestamps, "
+                        "and 1-3 representative log lines. Omit unrelated results. "
+                        "Keep under 500 words. Leave empty if no tool results are relevant."
                     ),
                 },
             },
@@ -1111,9 +1118,15 @@ async def _run_tool(name: str, inputs: dict, cfg_loader, store, slack_client=Non
                 })
 
         # ── Fallback: search locally-cached log files ──────────────────────────
+        # Reaching here means: live script unavailable OR source filter matched no config files.
+        # A result with files_searched=0 means the source name wasn't recognised — NOT that
+        # there are no log entries. Do not interpret this as "no results found".
         fetched_dir = Path("workspace/fetched")
         if not fetched_dir.exists():
-            return json.dumps({"error": "No fetched logs found and fetch_log.sh unavailable"})
+            return json.dumps({
+                "error": "No fetched logs found and fetch_log.sh unavailable",
+                "note": "This is a config/setup problem, not a 'no results' answer.",
+            })
         try:
             pattern = re.compile(query, re.IGNORECASE)
         except re.error as e:
@@ -1134,13 +1147,21 @@ async def _run_tool(name: str, inputs: dict, cfg_loader, store, slack_client=Non
             except Exception:
                 pass
         total = sum(len(r["matches"]) for r in results)
-        return json.dumps({
+        files_searched = len(list(fetched_dir.glob("*.log")))
+        result = {
             "query": query,
             "mode": "cached",
             "total_matches": total,
-            "files_searched": len(list(fetched_dir.glob("*.log"))),
+            "files_searched": files_searched,
             "results": results,
-        })
+        }
+        if files_searched == 0:
+            result["warning"] = (
+                "No cached log files found for this source filter. "
+                "This means the source name was not recognised — not that there are no log entries. "
+                "Try search_logs without a source filter to search all sources."
+            )
+        return json.dumps(result)
 
     if name == "trigger_poll":
         Path("SENTINEL_POLL_NOW").touch()