@allansantos-dev/smart-tool 0.9.23 → 0.9.25

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Changelog
 
+## 0.9.25 - beta
+
+- A redirected search made while the first index of a folder is still being built no longer waits in the folder's queue
+  behind the build (22 s measured, past the hook's 15 s, so the agent got the native result with no notice): it is
+  answered at once by the keyword search of the disk, with the redirect notice and the index-loading warning.
+- The first search of a cold folder also gets the index-loading sentence in the notice (it was read from a job field that
+  the answer text does not carry; it is now read from the answer's own warning).
+
+## 0.9.24 - beta
+
+- A redirected search now tells the agent what happened: the answer opens with a line saying the search was redirected to
+  `smart_search` (index, related code, dependency map), that the index of the project is still loading when the answer is
+  a keyword search of the disk (the next searches will have it ready), and to keep using `smart_search`. Before, the
+  answer carried no word of the redirect.
+- The deny text of a search that cannot be redirected no longer says that reading or searching inside one known file is
+  never blocked: no message the agent receives offers a way around the redirect (file by file, narrower scope).
+  `NoWayAroundTheRedirectTest` fails if any of those texts (deny, advice, redirect notice, cold-index and stale-index
+  warnings) suggests one.
+
 ## 0.9.23 - beta
 
 - The hook no longer blocks. A content search over a project folder (the Grep tool on a folder, `grep -r`, `rg`,

package/README.md

@@ -15,7 +15,7 @@ registered as the `playwright` server where this machine lets a browser be drive
 An optional hook routes the agent's native Grep/Glob/Read/Bash/WebSearch/WebFetch calls to these tools when they
 are cheaper, or just tells the agent what Smart Tool would do.
 
-**Status:** 0.9.23 beta. Windows only. What changed in each version: [CHANGELOG.md](CHANGELOG.md).
+**Status:** 0.9.25 beta. Windows only. What changed in each version: [CHANGELOG.md](CHANGELOG.md).
 
 ## How it works
 

package/hook_decision.py

@@ -138,14 +138,14 @@ def _load_hint(client, tool):
 
 def redirect_message(client, tool_name, target, reason, cwd):
     """Deny text for a broad raw search: the exact tool and arguments to call (project_root from the folder the search
-    reads), and that rewriting the same search in Bash, python, node or PowerShell is the bypass the rule stops."""
+    reads), and that rewriting the same search in Bash, python, node or PowerShell is the bypass the rule stops. It
+    never suggests a way around the rule (narrower or per-file searches): the only way offered is smart_search."""
     root = _search_root(target, cwd).replace("\\", "/")
     return (f"Blocked by Smart Tool, a routing rule set by the user, not a failure ({tool_name}: {clean_reason(reason)}). "
             f'Run this search with {_tool_name(client, "smart_search")}: project_root="{root}", query_identifiers = what '
             f"you are looking for in identifier terms (plus query_comments when the project has two languages)."
             f'{_load_hint(client, "smart_search")} Do not repeat the same search through Bash, python, node or '
-            f"PowerShell: that bypasses the rule. Reading one known file, or searching inside one known file, is never "
-            f"blocked.")
+            f"PowerShell: that bypasses the rule.")
 
 
 def web_message(client, tool_name, url, why):

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@allansantos-dev/smart-tool",
-  "version": "0.9.23",
+  "version": "0.9.25",
   "description": "Local MCP server that gives coding agents (Claude Code, Codex) cheaper, sharper tools than their built-in search. Windows.",
   "license": "Apache-2.0",
   "author": "Allan Santos",

package/smart_tool_daemon.py

@@ -3654,12 +3654,27 @@ def _decide_within_budget(payload, client):
     return output
 
 
+def _cold_index_building(root):
+    """Whether the folder has no index yet and the job that builds it is queued or running. A search job of that folder
+    would wait behind it in the per-folder queue (21.9 s measured, past the hook's wait), so a redirected search is
+    answered at once by `_perform_smart_search`'s cold path: keyword matches read from disk, with the notice."""
+    if indexer.info(root).get("chunks") or not _cold_index(root, config.load_config(), _gateway_problem()):
+        return False
+    project_id = project_identity.project_id(root)
+    with _JOBS_LOCK:
+        return any(job.get("kind") == "project_index" and job.get("status") == "pending"
+                   and job.get("project_id") == project_id for job in _jobs.values())
+
+
 def _redirected_search(arguments):
     """smart_search text for a redirected native search, waiting up to SMART_SEARCH_SYNC_WAIT_S; raises when the search
-    is rejected, fails or is still running then. The hook's single_language_reason only passes the language check: the
-    answer stands for the native call's output and does not repeat it as a warning."""
+    is rejected, fails or is still running then. The hook's single_language_reason only passes the language check, it is
+    not repeated as a warning. The answer opens with `tool_redirect.search_notice`: the agent is told its search was
+    redirected, whether the index is still loading, and to keep using smart_search."""
     checked = _smart_search_arguments(arguments)
     checked.pop("single_language_reason", None)
+    if _cold_index_building(checked["project_root"]):
+        return f"{tool_redirect.search_notice(True)}\n{_perform_smart_search(checked, '')}"
     job_id, _future = _submit_project_job("smart_search", checked)
     deadline = time.monotonic() + SMART_SEARCH_SYNC_WAIT_S
     payload = _smart_search_job_payload(job_id)
@@ -3668,7 +3683,8 @@ def _redirected_search(arguments):
         payload = _smart_search_job_payload(job_id)
     if payload["status"] != "done":
         raise RuntimeError(f"smart_search job {job_id} is {payload['status']}: {payload.get('error') or ''}"[:300])
-    return _agent_search_answer(payload)
+    answer = _agent_search_answer(payload)
+    return f"{tool_redirect.search_notice('Indexing in progress' in answer.partition(chr(10) + 'code:')[0])}\n{answer}"
 
 
 def _redirected_text(handler):

package/tool_redirect.py

@@ -60,6 +60,15 @@ class Jobs:
             self._futures.pop(key, None)
 
 
+def search_notice(indexing):
+    """First line of a redirected search's answer: the agent made a native search, Smart Tool answered it. While the
+    index loads the answer is a keyword search of the disk; either way the agent is told to keep using smart_search."""
+    state = ("The index of this project is still loading, so this answer is a keyword search of the files on disk; "
+             "the next searches will have the index ready. " if indexing else "")
+    return ("Smart Tool: your search was redirected to smart_search, which searches the project index and returns "
+            f"related code with its dependency map. {state}Keep using smart_search for searches of the project.")
+
+
 def output_for(tool_name, response, text):
     """`updatedToolOutput` for a native tool: its own response with the Smart Tool text in the field the model reads,
     or None when the response has an unknown shape."""

package/version.py

@@ -1,7 +1,7 @@
 """Single source of the Smart Tool version and of the User-Agent sent to public APIs."""
 import re
 
-VERSION = "0.9.23"
+VERSION = "0.9.25"
 _CONTACT_RE = re.compile(r"^(?:[^\s@()<>;]+@[^\s@()<>;]+\.[^\s@()<>;]+|https?://[^\s()<>;]+)$")