@misterhuydo/sentinel 1.5.56 → 1.5.58

package/.cairn/.hint-lock

@@ -1 +1 @@
-2026-04-21T07:25:38.460Z
+2026-04-21T07:58:09.595Z

package/.cairn/session.json

@@ -1,6 +1,6 @@
 {
-  "message": "Auto-checkpoint at 2026-04-21T07:26:06.811Z",
-  "checkpoint_at": "2026-04-21T07:26:06.813Z",
+  "message": "Auto-checkpoint at 2026-04-21T08:02:23.465Z",
+  "checkpoint_at": "2026-04-21T08:02:23.467Z",
   "active_files": [
     "J:\\Projects\\Sentinel\\cli\\bin\\sentinel.js",
     "J:\\Projects\\Sentinel\\cli\\lib\\test.js",

package/package.json

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

package/python/sentinel/__init__.py

@@ -1 +1 @@
-__version__ = "1.5.56"
+__version__ = "1.5.58"

package/python/sentinel/main.py

@@ -1848,7 +1848,11 @@ async def _execute_monitor(monitor: dict, cfg_loader: ConfigLoader, store: State
         combined = "\n".join(formatted_parts)
         MAX_LEN  = 3800
         if len(combined) > MAX_LEN:
-            combined = combined[:MAX_LEN] + f"\n_…truncated ({len(combined)} chars total)_"
+            tail = combined[:MAX_LEN]
+            # Close any unclosed code block before the truncation note
+            if tail.count("```") % 2 == 1:
+                tail += "\n```"
+            combined = tail + f"\n_…truncated ({len(combined)} chars total)_"
         header = f":repeat: *Monitor `{mon_id}`* ({mon_name}) — run #{runs_after}"
         if done:
             header += " _(final)_"

package/python/sentinel/sentinel_boss.py

@@ -377,8 +377,8 @@ reply with a grouped summary like this:
   Minimum interval: 60 seconds. Allowed tools: fetch_logs, filter_logs, get_status,
   ask_logs, list_recent_commits, check_health.
   Always confirm to the user with the monitor ID and stop condition before creating.
-• `stop_monitor` — cancel a monitor by ID, or pass "all" to cancel all in this channel
-• `list_monitors` — show all active monitors
+• `stop_monitor` — delete a monitor by ID (stops it if active); pass "all" to delete all in this channel
+• `list_monitors` — show active monitors plus completed/cancelled ones from the last 24 hours
 
 *File sharing*
 • `post_file` — upload any output as a Slack file (logs, diffs, reports)
@@ -1394,8 +1394,12 @@ _TOOLS = [
     {
         "name": "stop_monitor",
         "description": (
-            "Cancel a running monitor by ID. Pass 'all' to cancel every active monitor in this channel. "
-            "Use for: 'stop monitor m-abc123', 'stop all monitors', 'cancel the log watch'."
+            "Delete a monitor by ID — removes it from the list immediately. "
+            "Works on any monitor regardless of status (active, done, or cancelled). "
+            "For active monitors this also stops it from running. "
+            "Pass 'all' to delete every monitor in this channel. "
+            "Use for: 'delete monitor m-abc123', 'stop monitor m-abc123', 'delete all monitors', "
+            "'clear monitors', 'cancel the log watch'."
         ),
         "input_schema": {
             "type": "object",
@@ -1411,8 +1415,9 @@ _TOOLS = [
     {
         "name": "list_monitors",
         "description": (
-            "List all active scheduled monitors. "
-            "Use for: 'what monitors are running?', 'show scheduled tasks', 'list active monitors'."
+            "List all monitors — active ones plus completed/cancelled ones from the last 24 hours. "
+            "Use for: 'what monitors are running?', 'show scheduled tasks', 'list monitors', "
+            "'show recent monitors'. Each entry includes status (active/done/cancelled)."
         ),
         "input_schema": {"type": "object", "properties": {}},
     },
@@ -3589,32 +3594,33 @@ async def _run_tool(name: str, inputs: dict, cfg_loader, store, slack_client=Non
         if not mon_id:
             return json.dumps({"error": "monitor_id is required"})
         if mon_id.lower() == "all":
-            count = store.cancel_all_monitors(channel=channel)
-            return json.dumps({"cancelled": count, "message": f"Cancelled {count} active monitor(s)."})
-        ok = store.cancel_monitor(mon_id)
+            count = store.delete_all_monitors(channel=channel)
+            return json.dumps({"deleted": count, "message": f"Deleted {count} monitor(s)."})
+        ok = store.delete_monitor(mon_id)
         if ok:
-            return json.dumps({"status": "cancelled", "monitor_id": mon_id})
-        return json.dumps({"error": f"Monitor '{mon_id}' not found or already stopped."})
+            return json.dumps({"status": "deleted", "monitor_id": mon_id})
+        return json.dumps({"error": f"Monitor '{mon_id}' not found."})
 
     if name == "list_monitors":
         monitors = store.list_active_monitors()
         if not monitors:
-            return json.dumps({"monitors": [], "message": "No active monitors."})
+            return json.dumps({"monitors": [], "message": "No monitors in the last 24 hours."})
         result = []
         for _m in monitors:
             _runs_left = None
             if _m.get("max_runs"):
                 _runs_left = _m["max_runs"] - _m["runs_so_far"]
             result.append({
-                "id":          _m["id"],
-                "name":        _m.get("name") or "",
-                "status":      _m["status"],
-                "interval":    _format_duration(_m["interval_seconds"]),
-                "runs_so_far": _m["runs_so_far"],
-                "runs_left":   _runs_left,
-                "next_run_at": _m.get("next_run_at") or "",
-                "stop_at":     _m.get("stop_at") or "",
-                "steps":       json.loads(_m.get("steps_json") or "[]"),
+                "id":           _m["id"],
+                "name":         _m.get("name") or "",
+                "status":       _m["status"],
+                "interval":     _format_duration(_m["interval_seconds"]),
+                "runs_so_far":  _m["runs_so_far"],
+                "runs_left":    _runs_left,
+                "next_run_at":  _m.get("next_run_at") or "",
+                "stop_at":      _m.get("stop_at") or "",
+                "completed_at": _m.get("completed_at") or "",
+                "steps":        json.loads(_m.get("steps_json") or "[]"),
             })
         return json.dumps({"monitors": result})
 

package/python/sentinel/state_store.py

@@ -450,8 +450,14 @@ class StateStore:
             "channel TEXT, "
             "user_id TEXT, "
             "status TEXT DEFAULT 'active', "
+            "completed_at TEXT, "
             "created_at TEXT)"
         )
+        # Migration: add completed_at to existing tables
+        try:
+            conn.execute("ALTER TABLE monitors ADD COLUMN completed_at TEXT")
+        except Exception:
+            pass
 
     def create_monitor(self, id: str, name: str, steps_json: str,
                        interval_seconds: int, stop_at, max_runs,
@@ -486,10 +492,21 @@ class StateStore:
             return dict(row) if row else None
 
     def list_active_monitors(self) -> list[dict]:
+        from datetime import datetime, timezone, timedelta
+        now = datetime.now(timezone.utc)
+        cutoff = (now - timedelta(hours=24)).isoformat()
         with self._conn() as conn:
             self._ensure_monitors_table(conn)
+            # Auto-clean completed/done monitors older than 24 h
+            conn.execute(
+                "DELETE FROM monitors WHERE status != 'active' AND completed_at < ?",
+                (cutoff,),
+            )
             rows = conn.execute(
-                "SELECT * FROM monitors WHERE status = 'active' ORDER BY created_at DESC"
+                "SELECT * FROM monitors "
+                "WHERE status = 'active' OR (status != 'active' AND completed_at >= ?) "
+                "ORDER BY created_at DESC",
+                (cutoff,),
             ).fetchall()
             return [dict(r) for r in rows]
 
@@ -505,7 +522,12 @@ class StateStore:
         with self._conn() as conn:
             self._ensure_monitors_table(conn)
             if done:
-                conn.execute("DELETE FROM monitors WHERE id = ?", (id,))
+                # Keep done monitors visible for 24 h; user can delete explicitly via stop_monitor
+                conn.execute(
+                    "UPDATE monitors SET runs_so_far = runs_so_far + 1, last_run_at = ?, "
+                    "status = 'done', completed_at = ? WHERE id = ?",
+                    (_now(), _now(), id),
+                )
             else:
                 conn.execute(
                     "UPDATE monitors SET runs_so_far = runs_so_far + 1, last_run_at = ?, "
@@ -513,26 +535,31 @@ class StateStore:
                     (_now(), next_run_at, id),
                 )
 
-    def cancel_monitor(self, id: str) -> bool:
+    def delete_monitor(self, id: str) -> bool:
+        """Delete any monitor regardless of status (cancel + remove for active ones)."""
         with self._conn() as conn:
             self._ensure_monitors_table(conn)
-            cur = conn.execute(
-                "DELETE FROM monitors WHERE id = ? AND status = 'active'", (id,)
-            )
+            cur = conn.execute("DELETE FROM monitors WHERE id = ?", (id,))
             return cur.rowcount > 0
 
-    def cancel_all_monitors(self, channel: str = "") -> int:
+    def delete_all_monitors(self, channel: str = "") -> int:
+        """Delete all monitors in a channel (or globally if channel is empty)."""
         with self._conn() as conn:
             self._ensure_monitors_table(conn)
             if channel:
-                cur = conn.execute(
-                    "DELETE FROM monitors WHERE status = 'active' AND channel = ?",
-                    (channel,),
-                )
+                cur = conn.execute("DELETE FROM monitors WHERE channel = ?", (channel,))
             else:
-                cur = conn.execute("DELETE FROM monitors WHERE status = 'active'")
+                cur = conn.execute("DELETE FROM monitors")
             return cur.rowcount
 
+    def cancel_monitor(self, id: str) -> bool:
+        """Alias for delete_monitor — kept for compatibility."""
+        return self.delete_monitor(id)
+
+    def cancel_all_monitors(self, channel: str = "") -> int:
+        """Alias for delete_all_monitors — kept for compatibility."""
+        return self.delete_all_monitors(channel)
+
     # ── Pending bot-message routing questions ─────────────────────────────────
 
     def _ensure_pending_routings_table(self, conn):