@misterhuydo/sentinel 1.2.8 → 1.2.9

package/.cairn/session.json

@@ -1,6 +1,6 @@
 {
-  "message": "Auto-checkpoint at 2026-03-23T11:46:43.946Z",
-  "checkpoint_at": "2026-03-23T11:46:43.948Z",
+  "message": "Auto-checkpoint at 2026-03-23T12:00:29.548Z",
+  "checkpoint_at": "2026-03-23T12:00:29.550Z",
   "active_files": [],
   "notes": [],
   "mtime_snapshot": {}

package/package.json

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

package/python/sentinel/fix_engine.py

@@ -155,6 +155,7 @@ def generate_fix(
     repo: RepoConfig,
     cfg: SentinelConfig,
     patches_dir: Path,
+    store=None,
 ) -> tuple[str, Path | None, str]:
     """
     Generate a fix for the given error event.

package/python/sentinel/main.py

@@ -90,27 +90,31 @@ async def _handle_error(event: ErrorEvent, cfg_loader: ConfigLoader, store: Stat
     if status != "patch" or patch_path is None:
         outcome = "skipped" if status in ("skip", "needs_human") else "failed"
         store.record_fix(event.fingerprint, outcome, repo_name=repo.repo_name)
-        submitter_uid = getattr(event, "submitter_user_id", "")
+        # For log-detected errors: NEEDS_HUMAN -> DM/channel; SKIP -> email only (not spam)
         if status == "needs_human":
-            # marker holds the reason string for needs_human
             notify_fix_blocked(sentinel, event.source, event.message,
                                reason=marker, repo_name=repo.repo_name,
-                               submitter_user_id=submitter_uid)
+                               submitter_user_id="")
         else:
-            notify_fix_blocked(sentinel, event.source, event.message,
-                               reason=f"Claude Code returned {status.upper()}",
-                               repo_name=repo.repo_name,
-                               submitter_user_id=submitter_uid)
+            send_failure_notification(sentinel, {
+                "source":    event.source,
+                "message":   event.message,
+                "repo_name": repo.repo_name,
+                "reason":    f"Claude Code returned {status.upper()}",
+                "body":      event.full_text()[:500],
+            })
         return
 
     commit_status, commit_hash = apply_and_commit(event, patch_path, repo, sentinel)
     if commit_status != "committed":
         store.record_fix(event.fingerprint, "failed", repo_name=repo.repo_name)
-        submitter_uid = getattr(event, "submitter_user_id", "")
-        notify_fix_blocked(sentinel, event.source, event.message,
-                           reason="Patch was generated but commit/tests failed",
-                           repo_name=repo.repo_name,
-                           submitter_user_id=submitter_uid)
+        send_failure_notification(sentinel, {
+            "source":    event.source,
+            "message":   event.message,
+            "repo_name": repo.repo_name,
+            "reason":    "Patch was generated but commit/tests failed",
+            "body":      event.full_text()[:500],
+        })
         return
 
     branch, pr_url = publish(event, repo, sentinel, commit_hash)
@@ -184,16 +188,12 @@ async def _handle_issue(event: IssueEvent, cfg_loader: ConfigLoader, store: Stat
     if status != "patch" or patch_path is None:
         store.record_fix(event.fingerprint, "skipped" if status in ("skip", "needs_human") else "failed",
                          repo_name=repo.repo_name)
+        # For user-submitted issues: always notify (person is waiting)
         submitter_uid = getattr(event, "submitter_user_id", "")
-        if status == "needs_human":
-            notify_fix_blocked(sentinel, event.source, event.message,
-                               reason=marker, repo_name=repo.repo_name,
-                               submitter_user_id=submitter_uid)
-        else:
-            notify_fix_blocked(sentinel, event.source, event.message,
-                               reason=f"Claude Code returned {status.upper()}",
-                               repo_name=repo.repo_name,
-                               submitter_user_id=submitter_uid)
+        reason_text = marker if status == "needs_human" else f"Claude Code returned {status.upper()}"
+        notify_fix_blocked(sentinel, event.source, event.message,
+                           reason=reason_text, repo_name=repo.repo_name,
+                           submitter_user_id=submitter_uid)
         mark_done(event.issue_file)
         return
 
@@ -307,21 +307,31 @@ async def poll_cycle(cfg_loader: ConfigLoader, store: StateStore):
 
     # -- Health URL checks -------------------------------------------------------
     if cfg_loader.repos:
-        health_results = evaluate_repos(
-            cfg_loader.repos, cfg_loader.log_sources, cfg_loader.sentinel.workspace_dir,
-            store=store,
+        import asyncio as _asyncio
+        _loop = _asyncio.get_event_loop()
+        health_results = await _loop.run_in_executor(
+            None,
+            lambda: evaluate_repos(
+                cfg_loader.repos, cfg_loader.log_sources,
+                cfg_loader.sentinel.workspace_dir, store=store,
+            )
         )
         for hr in health_results:
             if hr["action"] == "fix":
                 fp = f"health-{hr['repo_name']}"
                 store.record_error(fp, f"health_checker/{hr['repo_name']}", hr["message"])
                 if not store.fix_attempted_recently(fp, hours=6):
-                    synth = ErrorEvent(
+                    from .log_parser import ErrorEvent as _EE
+                    from datetime import datetime, timezone as _tz
+                    synth = _EE(
                         source=f"health_checker/{hr['repo_name']}",
-                        severity="ERROR",
-                        message=f"App startup failure: {hr['message']}",
-                        raw_lines=[hr["startup_failure_line"]],
-                        timestamp=None,
+                        log_file="",
+                        timestamp=datetime.now(_tz.utc).isoformat(),
+                        level="ERROR",
+                        thread="health_checker",
+                        logger_name="health_checker",
+                        message=f"App startup failure detected: {hr['message']}",
+                        stack_trace=[hr["startup_failure_line"]] if hr["startup_failure_line"] else [],
                     )
                     synth.fingerprint = fp
                     await _handle_error(synth, cfg_loader, store)