@melaya/runner 1.0.103 → 1.0.105

package/dist/assistantHost.py

@@ -271,13 +271,25 @@ def _build_agent():
     # then ALWAYS require the in-chat approval card (fail-safe).
     if connector_services and not os.environ.get("MEL_ASSISTANT_CONNECTOR_HITL"):
         os.environ["MEL_ASSISTANT_CONNECTOR_HITL"] = "1"
-    # Core primitives (melaya_core): web_search / web_fetch, files, HTTP, data +
-    # office utilities, scraping, encoding, SQL, etc. — are ALWAYS discoverable via
-    # search_tools/activate_tool. The side-effecting ones run on the USER's OWN
-    # machine (this runner) and are HITL-gated by the assistant's safe/payments
-    # modes, so exposing them for discovery is safe. They stay in the LAZY pool
-    # (not pinned) so the ~73-tool category never explodes the active budget.
-    core_categories = ["melaya_core"]
+    # Core primitives: web_search / web_fetch, files, HTTP, data + office utilities,
+    # scraping, encoding, SQL, etc. — ALWAYS discoverable via search_tools/
+    # activate_tool. The side-effecting ones run on the USER's OWN machine (this
+    # runner) and are HITL-gated by the assistant's safe/payments modes, so exposing
+    # them for discovery is safe. They stay in the LAZY pool (not pinned) so the
+    # ~73-tool family never explodes the active budget.
+    #
+    # IMPORTANT: `melaya_core` is a CLIENT-SIDE DISPLAY bucket (toolServiceMap.ts),
+    # NOT a runtime registry category — the primitives actually live under `tools`,
+    # `scraping`, `msoffice`, `database`, … So we include the AUTHORITATIVE runtime
+    # category list (CORE_TOOL_CATEGORIES). Passing "melaya_core" matched NOTHING,
+    # which is why web_search/web_fetch never appeared in search_tools.
+    try:
+        from shared.runtime.registry import CORE_TOOL_CATEGORIES as core_categories
+    except Exception:
+        # Fallback if an older shared bundle predates the constant — hardcode the set.
+        core_categories = ["tools", "scraping", "data_utils", "msoffice", "aiml",
+                           "media", "knowledge", "netutil_tools", "qr_tools",
+                           "ics_tools"]  # excluded (need a connector): video_pipeline, database, devops
     try:
         from shared.orchestration.lazy_registry import build_lazy_toolkit
         # Base tools (melaya_agent + phone) stay active+pinned; core + any selected
@@ -339,6 +351,18 @@ def _build_agent():
         "data — read it from the connector.\n"
         if connector_services else ""
     )
+    # Core primitives are ALWAYS in the lazy pool now, so the model must be told
+    # they exist (they are not pinned/loaded upfront) — otherwise it concludes "no
+    # web-search tool is available" and refuses, exactly the reported failure.
+    core_rule = (
+        "- You have built-in CORE tools that are NOT loaded upfront: web_search "
+        "(live web search) and web_fetch (fetch a URL's content), plus file read/"
+        "write, HTTP requests, scraping, data/CSV/Excel utilities, SQL, QR and media. "
+        "To use any of them, call search_tools(query=...) (e.g. search_tools(\"web "
+        "search\")), then activate_tool(name=\"web_search\"), then call it. Whenever a "
+        "question needs live/current web data or a page's contents, use web_search / "
+        "web_fetch — do NOT claim you lack a web tool.\n"
+    )
     sys_prompt = (
         "You are the Melaya Assistant, the in-app copilot of the Melaya "
         "agent-orchestration platform. You have READ-ONLY tools over the data "
@@ -351,6 +375,7 @@ def _build_agent():
         "supports dimension='pipeline' to find which pipeline cost the most.\n"
         + phone_rule
         + connector_rule
+        + core_rule
         + "- Be concise and concrete; format small tables or bullet lists when comparing items.\n"
         "- If a question is outside the platform, answer normally without tools.\n"
         + (f"- Answer in the user's language: {language}.\n" if language and language != "en" else "")

package/dist/connection.js

@@ -14,7 +14,7 @@ import { io } from "socket.io-client";
 import chalk from "chalk";
 import ora from "ora";
 import { spawn } from "child_process";
-import { writeFileSync, mkdirSync, existsSync } from "fs";
+import { writeFileSync, mkdirSync, existsSync, readFileSync } from "fs";
 import { dirname, join } from "path";
 import { fileURLToPath } from "url";
 import { tmpdir, homedir } from "os";
@@ -28,6 +28,19 @@ import { ensureSharedModules, getSharedDir } from "./sharedVendor.js";
 import { startLumaBrowserBridge } from "./lumaBrowserBridge.js";
 const HEARTBEAT_INTERVAL = 30_000;
 const activeProcesses = new Map();
+// PR4/P1-5: the assistant-session wire protocol version. Bumped whenever the
+// handshake contract changes (this rev adds hostInstanceId + configHash + a
+// versioned ready/hello). The server enforces a MINIMUM: a runner advertising a
+// lower protocol never gets an assistant session routed to it (it gets a clear
+// "update your runner" instead of a silent stale-memory turn).
+const ASSISTANT_PROTOCOL_VERSION = 2;
+// Best-effort read of our own package version (mirrors cli.ts). Advertised in the
+// hello so the server can log / gate on a package floor too.
+let RUNNER_VERSION = "0.0.0";
+try {
+    RUNNER_VERSION = String(JSON.parse(readFileSync(join(__dirname, "..", "package.json"), "utf-8")).version || "0.0.0");
+}
+catch { /* dev tree layout differs — non-fatal */ }
 const activeAssistants = new Map();
 // PR4: sid-keyed boot reservation. The guard-then-spawn in runner:assistant_start
 // has an async gap (ensureSharedModules / ensurePythonEnv awaits) between the
@@ -133,7 +146,7 @@ export async function connect(opts) {
         // Advertise capabilities so the server only routes assistant chat sessions
         // to runners new enough to host them (older runners never get one → the
         // server falls back with a clear "update your runner" instead of hanging).
-        socket.emit("runner:hello", { capabilities: ["assistant_session"], package: "@melaya/runner" });
+        socket.emit("runner:hello", { capabilities: ["assistant_session"], package: "@melaya/runner", version: RUNNER_VERSION, assistantProtocol: ASSISTANT_PROTOCOL_VERSION });
         // Start local event relay
         if (!relay) {
             relay = await startLocalRelay(socket, opts.verbose, opts.serverUrl);
@@ -894,7 +907,7 @@ export async function connect(opts) {
         // that booted under an older generation and reboot it instead of serving the
         // next turn from stale memory/config. A bare ready hid this.
         if (live) {
-            emitEv({ kind: "ready", memoryWatermark: live.memoryWatermark || 0, generation: live.generation });
+            emitEv({ kind: "ready", memoryWatermark: live.memoryWatermark || 0, generation: live.generation, hostInstanceId: live.hostInstanceId, configHash: live.configHash, protocol: ASSISTANT_PROTOCOL_VERSION });
             return;
         }
         // A boot is already in flight for this sid (concurrent start / server restart
@@ -993,7 +1006,10 @@ export async function connect(opts) {
                 ...(payload.credentials || {}),
             };
             const proc = spawn(envResult.pythonPath, ["-u", stagedHost], { cwd: workDir, env, stdio: ["pipe", "pipe", "pipe"] });
-            const session = { proc, lastActivity: Date.now(), stdoutBuf: "", generation, memoryWatermark: 0 };
+            // hostInstanceId + configHash are filled from the host's OWN `ready` frame
+            // (assistantHost.py _HOST_ID / _config_hash) as it boots — authoritative and
+            // server-aligned. Start empty; the stdout parser captures them.
+            const session = { proc, lastActivity: Date.now(), stdoutBuf: "", generation, memoryWatermark: 0, hostInstanceId: "", configHash: "" };
             activeAssistants.set(sid, session);
             console.log(chalk.hex("#7c6ff0")(`  ◆ Assistant session ${sid.slice(0, 10)}… (${payload.provider})`));
             proc.stdout?.on("data", (data) => {
@@ -1011,6 +1027,16 @@ export async function connect(opts) {
                             // truthful watermark and the server skips a needless rehydrate.
                             if (typeof parsed?.memoryWatermark === "number")
                                 session.memoryWatermark = parsed.memoryWatermark;
+                            // CAPTURE the host's authoritative identity off its `ready` frame
+                            // (_HOST_ID / _config_hash) so the live-host ready path can re-report
+                            // them without the host re-emitting; ADD only the wire protocol.
+                            if (parsed?.kind === "ready") {
+                                if (typeof parsed.hostInstanceId === "string")
+                                    session.hostInstanceId = parsed.hostInstanceId;
+                                if (typeof parsed.configHash === "string")
+                                    session.configHash = parsed.configHash;
+                                parsed.protocol = ASSISTANT_PROTOCOL_VERSION;
+                            }
                             socket.emit("runner:assistant_event", { sessionId: sid, ...parsed });
                         }
                         catch { /* skip malformed */ }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@melaya/runner",
-  "version": "1.0.103",
+  "version": "1.0.105",
   "description": "Run Melaya AI pipelines locally with your own LM Studio or Ollama models",
   "license": "UNLICENSED",
   "private": false,