@misterhuydo/sentinel 1.5.31 → 1.5.33

package/package.json

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

package/python/sentinel/__init__.py

@@ -1 +1 @@
-__version__ = "1.5.31"
+__version__ = "1.5.33"

package/python/sentinel/repo_task_engine.py

@@ -34,7 +34,7 @@ from .git_manager import _git_env
 
 logger = logging.getLogger(__name__)
 
-_META_PREFIXES = ("REPO:", "TYPE:", "SUBMITTED_BY:", "SUBMITTED_AT:", "RUN_AT:", "NOTIFY:", "ORIGIN_CHANNEL:")
+_META_PREFIXES = ("REPO:", "TYPE:", "SUBMITTED_BY:", "SUBMITTED_AT:", "RUN_AT:", "NOTIFY:", "ORIGIN_CHANNEL:", "SKIP_TESTS:")
 _TASK_TIMEOUT = 900  # 15 minutes
 
 
@@ -51,6 +51,7 @@ class RepoTask:
     timestamp: str = ""
     run_at: datetime | None = None  # UTC; task is held until this time if set
     origin_channel: str = ""       # Slack channel ID where task was requested (DM or group)
+    skip_tests: bool = False        # skip test suite (safe for trivial/non-logic changes)
 
     def __post_init__(self):
         if not self.fingerprint:
@@ -62,6 +63,20 @@ class RepoTask:
 
 def _build_repo_prompt(task: RepoTask, repo: RepoConfig) -> str:
     submitted = f"Requested by: <@{task.submitter_user_id}>" if task.submitter_user_id else ""
+    if task.skip_tests:
+        test_instruction = (
+            "4. The user explicitly requested to skip tests — syntax check only, "
+            "do NOT run the test suite."
+        )
+    else:
+        test_instruction = (
+            "4. After making your changes, run: git diff HEAD  to review the diff, then decide:\n"
+            "   - If ALL changes are non-logic (log statements, comments, string literals,\n"
+            "     config values, whitespace/formatting): syntax check only — skip the test suite.\n"
+            "   - If ANY logic changed (conditionals, data flow, new/removed code paths,\n"
+            "     method behaviour): run the full test suite (mvn test / npm test / gradlew test)\n"
+            "     and commit only if it passes."
+        )
     return (
         f"You are implementing a requested change in the repository at {repo.local_path}.\n"
         f"Repository: {repo.repo_name}\n"
@@ -76,7 +91,7 @@ def _build_repo_prompt(task: RepoTask, repo: RepoConfig) -> str:
         f"   before making any changes.\n"
         f"2. Implement the requested change following the repo's existing code style.\n"
         f"3. Syntax/compile check modified files.\n"
-        f"4. Run tests if available (Maven, Gradle, npm test) — commit only if passing.\n"
+        f"{test_instruction}\n"
         f"5. Commit all changes:\n"
         f"   git add -A\n"
         f"   git commit -m \"{task.task_type}(<scope>): <concise summary> [sentinel-task]\"\n"
@@ -347,6 +362,7 @@ def drop_repo_task(
     notify_user_ids: list | None = None,
     run_at: datetime | None = None,
     origin_channel: str = "",
+    skip_tests: bool = False,
 ) -> Path:
     """Drop a repo task file into <project_dir>/repo-tasks/.
 
@@ -372,6 +388,8 @@ def drop_repo_task(
         lines.append(f"NOTIFY: {','.join(notify_user_ids)}")
     if origin_channel:
         lines.append(f"ORIGIN_CHANNEL: {origin_channel}")
+    if skip_tests:
+        lines.append("SKIP_TESTS: true")
     lines += ["", description]
     fpath.write_text("\n".join(lines), encoding="utf-8")
     if run_at:
@@ -404,6 +422,7 @@ def scan_repo_tasks(project_dir: Path) -> list[RepoTask]:
         notify_user_ids: list = []
         run_at: datetime | None = None
         origin_channel = ""
+        skip_tests = False
         body_start = 0
 
         for i, line in enumerate(lines):
@@ -434,6 +453,9 @@ def scan_repo_tasks(project_dir: Path) -> list[RepoTask]:
             elif upper.startswith("ORIGIN_CHANNEL:"):
                 origin_channel = stripped[15:].strip()
                 body_start = i + 1
+            elif upper.startswith("SKIP_TESTS:"):
+                skip_tests = stripped[11:].strip().lower() in ("true", "yes", "1")
+                body_start = i + 1
             elif any(upper.startswith(p) for p in _META_PREFIXES) or not stripped:
                 body_start = i + 1
             else:
@@ -469,6 +491,7 @@ def scan_repo_tasks(project_dir: Path) -> list[RepoTask]:
             notify_user_ids=notify_user_ids,
             run_at=run_at,
             origin_channel=origin_channel,
+            skip_tests=skip_tests,
         ))
         logger.info("Found repo task: %s → %s (type=%s)", f.name, repo_name, task_type)
 

package/python/sentinel/sentinel_boss.py

@@ -116,6 +116,9 @@ BEFORE CALLING dev_task OR repo_task — GATHER A COMPLETE SPEC:
 Ask follow-up questions until you have enough to write an unambiguous task description.
 Typical questions to resolve:
 - For repo_task: which repo exactly? (confirm if multiple match)
+- For repo_task: Claude Code auto-judges whether to run tests by inspecting its own diff.
+  Only set skip_tests=true when the user explicitly says "skip tests", "don't run tests",
+  or similar — never set it on your own judgment.
 - What exactly should happen? (specific behaviour, not vague intent)
 - Any config values, credentials, or external dependencies involved?
 - How should it be triggered? (schedule, event, API call, etc.)
@@ -532,6 +535,10 @@ When to act vs. when to ask:
 - Explaining a tool ("what does X do?") → explain naturally, then offer to run it if relevant.
 - NEVER gate investigation on user approval. If diagnosing a problem, run all relevant read tools
   first, then present findings. Asking "Want me to look?" wastes a round trip.
+- NEVER answer from session memory alone when the question is about current state (commits,
+  tasks, fixes, releases). Always call get_status or list_recent_commits first to verify live
+  state. Session memory is a snapshot — tasks complete, commits land, queues drain between turns.
+  If you remember "task X was in-flight", check whether it finished before telling the user to wait.
 - Prefer filter_logs over search_logs when synced logs are available — it's instant and never causes session timeout.
   Use search_logs only when the user explicitly wants live/real-time data or synced logs are not yet available.
 - If a tool call will take a moment (search, fetch, pull), prefix your reply with a brief "working" line ending in "..." before the results, e.g. "Searching SSOLWA for TryDig activity..." then the actual output.
@@ -916,6 +923,15 @@ _TOOLS = [
                         "If omitted the task runs immediately on the next poll cycle."
                     ),
                 },
+                "skip_tests": {
+                    "type": "boolean",
+                    "description": (
+                        "Only set to true when the user explicitly requests skipping tests "
+                        "(e.g. 'skip tests', 'don\\'t run tests', 'just commit it'). "
+                        "When false (default), Claude Code automatically decides whether to run "
+                        "tests by inspecting its own diff — non-logic changes skip tests on their own."
+                    ),
+                },
             },
             "required": ["repo_name", "description"],
         },
@@ -1165,6 +1181,15 @@ _TOOLS = [
                     "type": "string",
                     "description": "Project short name this bot's issues should be routed to (e.g. '1881', 'elprint'). Infer from context or ask user before calling.",
                 },
+                "target_repo": {
+                    "type": "string",
+                    "description": (
+                        "Optional. Repo name to route this bot's issues to (e.g. '1881-SSOLoginWebApp'). "
+                        "Required when the project has multiple repos — without it, routing will fail. "
+                        "If the project has only one repo, omit this and it will be auto-selected. "
+                        "Ask the user which repo if unclear."
+                    ),
+                },
             },
             "required": ["user_ids"],
         },
@@ -2473,6 +2498,8 @@ async def _run_tool(name: str, inputs: dict, cfg_loader, store, slack_client=Non
         if not _project_dirs:
             return json.dumps({"error": "No project directory found."})
 
+        skip_tests = bool(inputs.get("skip_tests", False))
+
         from .repo_task_engine import drop_repo_task as _drop_repo_task
         task_file = _drop_repo_task(
             _project_dirs[0],
@@ -2483,6 +2510,7 @@ async def _run_tool(name: str, inputs: dict, cfg_loader, store, slack_client=Non
             notify_user_ids=notify_ids,
             run_at=run_at_dt,
             origin_channel=channel,
+            skip_tests=skip_tests,
         )
         logger.info(
             "Boss repo_task: dropped %s for user %s (repo=%s, run_at=%s)",
@@ -3076,6 +3104,18 @@ async def _run_tool(name: str, inputs: dict, cfg_loader, store, slack_client=Non
                     "action_needed": "Ask the user to specify the project, then retry with project filled in.",
                 })
 
+        target_repo_arg = (inputs.get("target_repo") or "").strip()
+        # Fuzzy-match target_repo against known repos
+        resolved_target_repo = ""
+        if target_repo_arg and hasattr(cfg_loader, "repos"):
+            if target_repo_arg in cfg_loader.repos:
+                resolved_target_repo = target_repo_arg
+            else:
+                for rname in cfg_loader.repos:
+                    if target_repo_arg.lower() in rname.lower() or rname.lower() in target_repo_arg.lower():
+                        resolved_target_repo = rname
+                        break
+
         results = []
         for uid in user_ids:
             if not slack_client:
@@ -3088,9 +3128,9 @@ async def _run_tool(name: str, inputs: dict, cfg_loader, store, slack_client=Non
                     results.append({"user_id": uid, "status": "skipped", "reason": "not a bot — only bots can be watched passively"})
                     continue
                 bot_name = user.get("real_name") or user.get("name") or uid
-                store.add_watched_bot(uid, bot_name, added_by="boss", project_name=resolved_project)
-                logger.info("Boss: now watching bot %s (%s) → project '%s'", bot_name, uid, resolved_project or "unset")
-                results.append({"user_id": uid, "bot_name": bot_name, "project": resolved_project, "status": "watching"})
+                store.add_watched_bot(uid, bot_name, added_by="boss", project_name=resolved_project, target_repo=resolved_target_repo)
+                logger.info("Boss: now watching bot %s (%s) → project '%s', repo '%s'", bot_name, uid, resolved_project or "unset", resolved_target_repo or "auto")
+                results.append({"user_id": uid, "bot_name": bot_name, "project": resolved_project, "target_repo": resolved_target_repo or "auto", "status": "watching"})
             except Exception as e:
                 results.append({"user_id": uid, "status": "error", "reason": str(e)})
         return json.dumps({"results": results})

package/python/sentinel/slack_bot.py

@@ -214,6 +214,7 @@ async def _handle_bot_message(event: dict, client, cfg_loader, store) -> None:
         None,
     )
     project_name = (bot_info or {}).get("project_name") or ""
+    target_repo  = (bot_info or {}).get("target_repo") or ""
 
     # Resolve the project issues directory
     workspace = _Path(cfg_loader.sentinel.workspace_dir).parent
@@ -238,9 +239,11 @@ async def _handle_bot_message(event: dict, client, cfg_loader, store) -> None:
 
     uid = _uuid.uuid4().hex[:8]
     fname = f"bot-{project_name or 'unknown'}-{uid}.txt"
+    target_line = f"TARGET_REPO: {target_repo}\n" if target_repo else ""
     content = (
         f"SOURCE: Slack bot {bot_id} in channel {channel}\n"
-        f"SLACK_TS: {ts}\n\n"
+        f"SLACK_TS: {ts}\n"
+        f"{target_line}\n"
         f"{text}"
     )
     (issues_dir / fname).write_text(content, encoding="utf-8")

package/python/sentinel/state_store.py

@@ -99,6 +99,7 @@ class StateStore:
             ("add_fix_outcome",           "ALTER TABLE fixes ADD COLUMN fix_outcome TEXT"),
             ("add_marker_seen_at",        "ALTER TABLE fixes ADD COLUMN marker_seen_at TEXT"),
             ("add_watched_bots_project",  "ALTER TABLE watched_bots ADD COLUMN project_name TEXT"),
+            ("add_watched_bots_target_repo", "ALTER TABLE watched_bots ADD COLUMN target_repo TEXT"),
             ("add_alert_thread_ts",       "ALTER TABLE errors ADD COLUMN alert_thread_ts TEXT"),
             ("add_alert_channel",         "ALTER TABLE errors ADD COLUMN alert_channel TEXT"),
             # knowledge_cache was originally created with a 'cached_at' column; rebuilt with 'expires_at'
@@ -408,13 +409,13 @@ class StateStore:
             "(bot_id TEXT PRIMARY KEY, bot_name TEXT, added_by TEXT, added_at TEXT)"
         )
 
-    def add_watched_bot(self, bot_id: str, bot_name: str, added_by: str = "config", project_name: str = ""):
+    def add_watched_bot(self, bot_id: str, bot_name: str, added_by: str = "config", project_name: str = "", target_repo: str = ""):
         with self._conn() as conn:
             self._ensure_watched_bots_table(conn)
             conn.execute(
-                "INSERT OR REPLACE INTO watched_bots (bot_id, bot_name, added_by, added_at, project_name) "
-                "VALUES (?, ?, ?, ?, ?)",
-                (bot_id, bot_name, added_by, _now(), project_name or None),
+                "INSERT OR REPLACE INTO watched_bots (bot_id, bot_name, added_by, added_at, project_name, target_repo) "
+                "VALUES (?, ?, ?, ?, ?, ?)",
+                (bot_id, bot_name, added_by, _now(), project_name or None, target_repo or None),
             )
 
     def remove_watched_bot(self, bot_id: str) -> bool: