@bike4mind/cli 1.2.0 → 2.0.0

package/dist/{buildAgent-C-C-VGff.mjs → buildAgent-DOvEp92a.mjs}

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
-import { I as generateCliTools, J as setWebSocketToolExecutor, L as wrapTools, N as ReActAgent, X as buildSystemPrompt, Y as getPlanModeFilePath, a as loadProjectContext, c as createSseBackend, i as buildProjectAgentStore, it as Logger, l as ServerLlmBackend, n as BackgroundAgentManager, o as McpManager, r as SubagentOrchestrator, t as AgentHistoryStore } from "./AgentHistoryStore-B2NEOvSW.mjs";
-import { D as MODEL_INFO_FIELD_GROUP_OF, H as escapeThinkMarkers, P as PermissionDeniedError, S as CONTEXT_WINDOW_SAFETY_BUFFER_TOKENS, V as createThinkMarkerEscaper, W as getQuestErrorCode, Y as isUserInitiatedAbort, k as ModelBackend, n as logger, w as ChatModels } from "./ConfigStore-qV7NrCgZ.mjs";
+import { I as generateCliTools, J as setWebSocketToolExecutor, L as wrapTools, N as ReActAgent, X as buildSystemPrompt, Y as getPlanModeFilePath, a as loadProjectContext, c as createSseBackend, i as buildProjectAgentStore, it as Logger, l as ServerLlmBackend, n as BackgroundAgentManager, o as McpManager, r as SubagentOrchestrator, t as AgentHistoryStore } from "./AgentHistoryStore-DIMzvf-y.mjs";
+import { $ as hasDeliverablePinnedArtifact, C as ARTIFACT_REMOVED_PLACEHOLDER, E as CONTEXT_WINDOW_SAFETY_BUFFER_TOKENS, J as filterToolArtifactMarkup, K as createThinkMarkerEscaper, M as MODEL_INFO_FIELD_GROUP_OF, O as ChatModels, P as ModelBackend, S as ARTIFACT_DELIVERED_PLACEHOLDER, Z as getQuestErrorCode, ct as stripDeliveredArtifactBlocks, lt as stripToolArtifactMarkup, n as logger, q as escapeThinkMarkers, rt as isUserInitiatedAbort, z as PermissionDeniedError } from "./ConfigStore-DIs-K9oD.mjs";
 import "crypto";
 import { v4 } from "uuid";
 import { z as z$1 } from "zod";
@@ -135,6 +135,96 @@ function attachFullToolResult(entry, observation) {
 		writable: true
 	});
 }
+/** The `artifactType` the named tool declared at registration, for the helpers below. */
+function declaredArtifactType(tools, toolName) {
+	return tools?.find((t) => t.toolSchema.name === toolName)?.artifactType;
+}
+/**
+* Helper function to handle tool result streaming for artifact-generating tools
+* This ensures tools like recharts that generate artifacts are streamed immediately
+* rather than waiting for recursive completion calls.
+*
+* Only the emitters in TOOL_ARTIFACT_EMITTERS (or a tool that declared `artifactType`, see
+* declaredArtifactType) stream, and only their pinned artifact type: streamed text is parsed
+* into reply artifacts, so any other tool's markup would render.
+*
+* What does stream is still a raw tool artifact rather than reply prose, so `streamCallback`
+* receives the channel tag and a public surface drops the text on that tag alone.
+*/
+async function handleToolResultStreaming(toolName, toolResult, streamCallback, artifactType) {
+	const filtered = filterToolArtifactMarkup(toolName, String(toolResult), artifactType);
+	if (filtered !== null) await streamCallback([filtered], { channel: "tool-artifact" });
+}
+/**
+* Strips artifact markup from a tool result before it re-enters history, so the model cannot echo
+* it into a second card. Live-emitting backends pass `delivered` = whether their tool-artifact
+* emit fired. The `hasDeliverablePinnedArtifact` fallback is for the OpenAI Responses path, whose
+* artifact reaches the client through sharedToolBuilder extraction instead.
+*
+* The delivered/removed choice is made once per result and stamped on every block it contains.
+* That holds because each emitter emits one pinned block; a result mixing a foreign-type block
+* with a pinned one would label both DELIVERED.
+*/
+function stripUnstreamedToolResult(toolName, result, delivered) {
+	return stripToolArtifactMarkup(result, delivered ?? hasDeliverablePinnedArtifact(toolName, result) ? ARTIFACT_DELIVERED_PLACEHOLDER : ARTIFACT_REMOVED_PLACEHOLDER);
+}
+/**
+* Buffers a recursive completion's text instead of streaming it live, so an <artifact> tag the
+* model echoes back (after seeing its own tool result, even sanitized) can be stripped before the
+* client ever renders a second, empty card for it. Only removes blocks whose `identifier` matches
+* one already delivered this turn (stripDeliveredArtifactBlocks) - a stray "<artifact" the model's
+* prose merely mentions, or a genuinely NEW artifact the model composes with a different
+* identifier, must not cost the rest of a legitimate reply, unlike the tool-output-facing
+* stripToolArtifactMarkup.
+*
+* One instance is created lazily by whichever level first streams an artifact in a recursive
+* chain, and reused UNCHANGED by every deeper level (never recreated per level) - see each
+* backend's `_internal.artifactGuard`. A chained tool call's artifact goes through `emitArtifact`
+* on this same shared instance, which flushes any text already buffered ahead of it before
+* sending the artifact through - so ordering survives even when the chained call happens several
+* recursion levels below where the guard was created.
+*/
+function createRecursiveArtifactGuard(cb) {
+	let buffer = "";
+	let deliveredMarkup = "";
+	let meta = {};
+	const callback = (async (text, info) => {
+		for (const chunk of text) if (chunk != null) buffer += chunk;
+		if (info?.inputTokens || info?.outputTokens) meta = { ...info };
+	});
+	const emitArtifact = async (results, info) => {
+		if (buffer) {
+			const pending = buffer;
+			buffer = "";
+			const stripped = stripDeliveredArtifactBlocks(pending, deliveredMarkup);
+			if (stripped.trim()) await cb([stripped], {});
+		}
+		deliveredMarkup += results.join("");
+		await cb(results, info);
+	};
+	const markDelivered = (markup) => {
+		deliveredMarkup += markup;
+	};
+	let flushPromise = null;
+	const flush = () => {
+		if (flushPromise) return flushPromise;
+		const cleaned = stripDeliveredArtifactBlocks(buffer, deliveredMarkup).trim();
+		let resolve;
+		let reject;
+		flushPromise = new Promise((res, rej) => {
+			resolve = res;
+			reject = rej;
+		});
+		(async () => cb(cleaned ? [cleaned] : [], meta))().then(() => resolve(), reject);
+		return flushPromise;
+	};
+	return {
+		callback,
+		emitArtifact,
+		markDelivered,
+		flush
+	};
+}
 /**
 * Drops the messages whose instructions assume a tool is attached (see `IMessage.requiresTool`).
 *
@@ -850,20 +940,32 @@ var OllamaBackend = class OllamaBackend {
 				parallel: options.parallelToolExecution !== false,
 				maxConcurrency: options.maxParallelTools
 			});
+			const inheritedArtifactGuard = options._internal?.artifactGuard;
+			let artifactGuard = inheritedArtifactGuard;
 			const observations = [];
-			outcomes.forEach((outcome, i) => {
+			for (let i = 0; i < outcomes.length; i++) {
+				const outcome = outcomes[i];
 				const { tc } = resolved[i];
 				const params = tc.arguments || "{}";
 				if (outcome.ok) {
-					observations[i] = outcome.result;
+					let emitted = false;
+					await handleToolResultStreaming(tc.name, outcome.result, async (results, artifactInfo) => {
+						emitted = true;
+						if (!artifactGuard) artifactGuard = createRecursiveArtifactGuard(callback);
+						await artifactGuard.emitArtifact(results, {
+							toolsUsed,
+							...artifactInfo
+						});
+					}, declaredArtifactType(options.tools, tc.name));
+					observations[i] = stripUnstreamedToolResult(tc.name, outcome.result, emitted);
 					this.pushToolMessages(messages, {
 						id: tc.id,
 						name: tc.name,
 						parameters: params
-					}, outcome.result);
+					}, observations[i]);
 				} else {
 					if (outcome.error instanceof PermissionDeniedError) throw outcome.error;
-					const errorMsg = `Error running ${tc.name}: ${outcome.error instanceof Error ? outcome.error.message : "Unknown error"}`;
+					const errorMsg = stripToolArtifactMarkup(`Error running ${tc.name}: ${outcome.error instanceof Error ? outcome.error.message : "Unknown error"}`, ARTIFACT_REMOVED_PLACEHOLDER);
 					observations[i] = errorMsg;
 					this.pushToolMessages(messages, {
 						id: tc.id,
@@ -871,7 +973,7 @@ var OllamaBackend = class OllamaBackend {
 						parameters: params
 					}, errorMsg);
 				}
-			});
+			}
 			const executedToolsUsed = [...priorToolsUsed, ...resolved.map(({ tc }, i) => {
 				const entry = {
 					name: tc.name,
@@ -884,11 +986,12 @@ var OllamaBackend = class OllamaBackend {
 				return entry;
 			})];
 			if (options.abortSignal?.aborted) {
-				await callback([""], {
+				await (artifactGuard?.callback ?? callback)([""], {
 					inputTokens,
 					outputTokens,
 					toolsUsed: executedToolsUsed
 				});
+				if (!inheritedArtifactGuard && artifactGuard) await artifactGuard.flush();
 				return;
 			}
 			await this.complete(model, messages, {
@@ -898,9 +1001,11 @@ var OllamaBackend = class OllamaBackend {
 					toolCallCount: toolCallCount + 1,
 					accumToolsUsed: executedToolsUsed,
 					accumInputTokens: inputTokens,
-					accumOutputTokens: outputTokens
+					accumOutputTokens: outputTokens,
+					artifactGuard
 				}
-			}, callback);
+			}, artifactGuard?.callback ?? callback);
+			if (!inheritedArtifactGuard && artifactGuard) await artifactGuard.flush();
 		} catch (error) {
 			if (error instanceof Error && isUserInitiatedAbort(error, options.abortSignal)) this._logger.debug("[OllamaBackend] Ollama request cancelled by the caller");
 			else this._logger.error("[OllamaBackend] Error during Ollama API call:", error);

package/dist/commands/acpCommand.mjs

@@ -1,8 +1,8 @@
 #!/usr/bin/env node
-import { $ as CustomCommandStore, A as createAgentDelegateTool, C as createTodoStore, D as createCoordinateTaskTool, F as PermissionManager, J as setWebSocketToolExecutor, L as wrapTools, O as createResumeAgentTool, Q as RemoteSkillSource, S as createFindDefinitionTool, T as createSkillTool, b as createGetFileStructureTool, et as CheckpointStore, j as FallbackLlmBackend, k as createBackgroundAgentTools, s as buildSandbox, tt as SessionStore, w as createWriteTodosTool } from "../AgentHistoryStore-B2NEOvSW.mjs";
-import { h as requireApiUrl, n as logger, s as isReadOnlyTool, t as ConfigStore } from "../ConfigStore-qV7NrCgZ.mjs";
-import { a as deferredToolRegistry, i as createToolSearchTool, n as buildSupportingStores, o as NotifyingLlmBackend, r as buildLlmBackend, t as buildAgent } from "../buildAgent-C-C-VGff.mjs";
-import { t as ApiClient } from "../ApiClient-D0fQ2FT6.mjs";
+import { $ as CustomCommandStore, A as createAgentDelegateTool, C as createTodoStore, D as createCoordinateTaskTool, F as PermissionManager, J as setWebSocketToolExecutor, L as wrapTools, O as createResumeAgentTool, Q as RemoteSkillSource, S as createFindDefinitionTool, T as createSkillTool, b as createGetFileStructureTool, et as CheckpointStore, j as FallbackLlmBackend, k as createBackgroundAgentTools, s as buildSandbox, tt as SessionStore, w as createWriteTodosTool } from "../AgentHistoryStore-DIMzvf-y.mjs";
+import { _ as requireApiUrl, d as isReadOnlyTool, n as logger, t as ConfigStore } from "../ConfigStore-DIs-K9oD.mjs";
+import { a as deferredToolRegistry, i as createToolSearchTool, n as buildSupportingStores, o as NotifyingLlmBackend, r as buildLlmBackend, t as buildAgent } from "../buildAgent-DOvEp92a.mjs";
+import { t as ApiClient } from "../ApiClient-BTI0-m0g.mjs";
 import { randomUUID } from "crypto";
 import { existsSync, realpathSync, statSync } from "fs";
 import { isAbsolute } from "path";
@@ -575,6 +575,7 @@ var AcpServer = class {
 	}
 	async buildStack() {
 		const config = await this.configStore.load();
+		this.configStore.warnPendingMcpApprovals();
 		await this.loadCustomCommands();
 		const authTokens = await this.configStore.getAuthTokens();
 		if (!authTokens) throw RequestError.authRequired(void 0, "Not authenticated. Run `b4m /login` first.");

package/dist/commands/apiCommand.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { g as resolveApiEndpoint, m as parseApiUrl, t as ConfigStore } from "../ConfigStore-qV7NrCgZ.mjs";
+import { g as parseApiUrl, t as ConfigStore, v as resolveApiEndpoint } from "../ConfigStore-DIs-K9oD.mjs";
 //#region src/commands/apiCommand.ts
 /**
 * External API config command (--api-url / --reset-api)

package/dist/commands/doctorCommand.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { t as version } from "../package-CGZIoxcs.mjs";
+import { t as version } from "../package-Dl_F2Fah.mjs";
 import { a as fetchLatestVersion, c as isNpmPrefixWritable, i as compareSemver } from "../updateChecker-CQW8bxo6.mjs";
 import { t as checkRipgrep } from "../ripgrepCheck-BmkyTK2i.mjs";
 import { execSync } from "child_process";

package/dist/commands/envCommand.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { t as ConfigStore } from "../ConfigStore-qV7NrCgZ.mjs";
+import { t as ConfigStore } from "../ConfigStore-DIs-K9oD.mjs";
 //#region src/commands/envCommand.ts
 /**
 * Environment switching for the `--dev` / `--prod` launch flags.

package/dist/commands/headlessCommand.mjs

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
-import { $ as CustomCommandStore, A as createAgentDelegateTool, C as createTodoStore, D as createCoordinateTaskTool, F as PermissionManager, G as classifyCommandRisk, I as generateCliTools, J as setWebSocketToolExecutor, L as wrapTools, N as ReActAgent, O as createResumeAgentTool, Q as RemoteSkillSource, S as createFindDefinitionTool, T as createSkillTool, X as buildSystemPrompt, a as loadProjectContext, b as createGetFileStructureTool, c as createSseBackend, et as CheckpointStore, i as buildProjectAgentStore, j as FallbackLlmBackend, k as createBackgroundAgentTools, n as BackgroundAgentManager, o as McpManager, r as SubagentOrchestrator, s as buildSandbox, t as AgentHistoryStore, tt as SessionStore, w as createWriteTodosTool, x as createWorkItemTools, z as SHELL_LIKE_TOOL_COMMAND_FIELDS } from "../AgentHistoryStore-B2NEOvSW.mjs";
-import { h as requireApiUrl, n as logger, s as isReadOnlyTool, t as ConfigStore } from "../ConfigStore-qV7NrCgZ.mjs";
-import { t as ApiClient } from "../ApiClient-D0fQ2FT6.mjs";
+import { $ as CustomCommandStore, A as createAgentDelegateTool, C as createTodoStore, D as createCoordinateTaskTool, F as PermissionManager, G as classifyCommandRisk, I as generateCliTools, J as setWebSocketToolExecutor, L as wrapTools, N as ReActAgent, O as createResumeAgentTool, Q as RemoteSkillSource, S as createFindDefinitionTool, T as createSkillTool, X as buildSystemPrompt, a as loadProjectContext, b as createGetFileStructureTool, c as createSseBackend, et as CheckpointStore, i as buildProjectAgentStore, j as FallbackLlmBackend, k as createBackgroundAgentTools, n as BackgroundAgentManager, o as McpManager, r as SubagentOrchestrator, s as buildSandbox, t as AgentHistoryStore, tt as SessionStore, w as createWriteTodosTool, x as createWorkItemTools, z as SHELL_LIKE_TOOL_COMMAND_FIELDS } from "../AgentHistoryStore-DIMzvf-y.mjs";
+import { _ as requireApiUrl, d as isReadOnlyTool, n as logger, t as ConfigStore } from "../ConfigStore-DIs-K9oD.mjs";
+import { t as ApiClient } from "../ApiClient-BTI0-m0g.mjs";
 import { r as reconstructTurnBlocks, t as WorkItemsClient } from "../WorkItemsClient-Cow6nXx7.mjs";
 import { randomBytes } from "crypto";
 import { v4 } from "uuid";
@@ -250,6 +250,7 @@ async function handleHeadlessCommand(options) {
 	const runId = v4();
 	try {
 		const config = await configStore.load();
+		configStore.warnPendingMcpApprovals();
 		const configDirs = await configStore.getAdditionalDirectories();
 		const flagDirs = process.env.B4M_ADDITIONAL_DIRS ? parseStringArray(process.env.B4M_ADDITIONAL_DIRS, "B4M_ADDITIONAL_DIRS") : [];
 		const additionalDirectories = [.../* @__PURE__ */ new Set([

package/dist/commands/mcpCommand.mjs

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
-import { t as ConfigStore } from "../ConfigStore-qV7NrCgZ.mjs";
-import { t as version } from "../package-CGZIoxcs.mjs";
+import { t as ConfigStore } from "../ConfigStore-DIs-K9oD.mjs";
+import { t as version } from "../package-Dl_F2Fah.mjs";
 //#region src/commands/mcpCommand.ts
 /**
 * External MCP commands (b4m mcp list, b4m mcp add, etc.)
@@ -70,7 +70,7 @@ async function handleMcpCommand(subcommand, argv) {
 			await handleDisable(config, argv.name, configStore);
 			break;
 		case "serve": {
-			const { handleMcpServeCommand } = await import("../serve-CPXqcEZr.mjs");
+			const { handleMcpServeCommand } = await import("../serve-Y2zMbfGs.mjs");
 			await handleMcpServeCommand({
 				http: Boolean(argv.http),
 				port: typeof argv.port === "number" ? argv.port : void 0,

package/dist/commands/pluginCommand.mjs

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import { n as getDefaultPluginsDir, r as isFeatureEnabled, t as PluginStore } from "../PluginStore-DwvOJ-G3.mjs";
-import { t as ConfigStore } from "../ConfigStore-qV7NrCgZ.mjs";
+import { t as ConfigStore } from "../ConfigStore-DIs-K9oD.mjs";
 import { execFileSync } from "child_process";
 import { promises } from "fs";
 import path from "path";

package/dist/commands/updateCommand.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { t as version } from "../package-CGZIoxcs.mjs";
+import { t as version } from "../package-Dl_F2Fah.mjs";
 import { c as isNpmPrefixWritable, l as setAutoUpdatePreference, n as REEXEC_GUARD_ENV, o as forceCheckForUpdate, r as checkForUpdate, s as getAutoUpdatePreference, t as INSTALL_CMD, u as shouldAttemptAutoUpdate } from "../updateChecker-CQW8bxo6.mjs";
 import { t as checkRipgrep } from "../ripgrepCheck-BmkyTK2i.mjs";
 import { execSync, spawnSync } from "child_process";