botinabox 2.4.3 → 2.5.0

package/dist/index.js

@@ -1662,6 +1662,7 @@ var ChatPipeline = class {
     this.messageFilter = config.messageFilter;
     this.capabilities = config.capabilities;
     this.dedupWindowMs = config.dedupWindowMs ?? DEFAULT_DEDUP_WINDOW_MS;
+    this.skipAck = config.skipAck ?? false;
     this.tasks = config.tasks;
     this.wakeups = config.wakeups;
     this.messageStore = new MessageStore(db, hooks);
@@ -1695,6 +1696,7 @@ var ChatPipeline = class {
   dedupWindowMs;
   tasks;
   wakeups;
+  skipAck;
   // In-memory thread → channel mapping for response routing
   // (before thread_task_map exists)
   threadChannelMap = /* @__PURE__ */ new Map();
@@ -1731,24 +1733,26 @@ var ChatPipeline = class {
         const dir = m.direction === "inbound" ? "User" : "Bot";
         return `${dir}: ${m.body?.slice(0, 200) ?? ""}`;
       }).join("\n");
-      const ackResponse = await this.responder.respond({
-        messageBody: msg.body,
-        threadId: threadTs,
-        channel: this.channel,
-        capabilities: this.capabilities,
-        additionalContext: historyContext ? `
+      if (!this.skipAck) {
+        const ackResponse = await this.responder.respond({
+          messageBody: msg.body,
+          threadId: threadTs,
+          channel: this.channel,
+          capabilities: this.capabilities,
+          additionalContext: historyContext ? `
 
 Recent conversation history:
 ${historyContext}` : void 0
-      });
-      await this.responder.sendResponse({
-        text: ackResponse,
-        channel: this.channel,
-        threadId: threadTs,
-        source: "responder",
-        skipFilter: true,
-        skipRedundancyCheck: true
-      });
+        });
+        await this.responder.sendResponse({
+          text: ackResponse,
+          channel: this.channel,
+          threadId: threadTs,
+          source: "responder",
+          skipFilter: true,
+          skipRedundancyCheck: true
+        });
+      }
       const dispatchPromise = this.interpretAndDispatch(messageId, msg, threadTs, channelId);
       this.lastDispatch = dispatchPromise;
       void dispatchPromise;
@@ -2966,6 +2970,7 @@ function defineCoreTables(db) {
     columns: {
       id: "TEXT PRIMARY KEY",
       agent_id: "TEXT NOT NULL",
+      user_id: "TEXT",
       task_id: "TEXT",
       issue: "TEXT NOT NULL",
       root_cause: "TEXT",
@@ -2978,7 +2983,8 @@ function defineCoreTables(db) {
     },
     tableConstraints: [
       "CREATE INDEX IF NOT EXISTS idx_feedback_agent ON feedback(agent_id, created_at)",
-      "CREATE INDEX IF NOT EXISTS idx_feedback_issue ON feedback(issue)"
+      "CREATE INDEX IF NOT EXISTS idx_feedback_issue ON feedback(issue)",
+      "CREATE INDEX IF NOT EXISTS idx_feedback_user ON feedback(user_id)"
     ]
   });
   db.define("playbooks", {
@@ -2988,6 +2994,7 @@ function defineCoreTables(db) {
       rule: "TEXT NOT NULL",
       feedback_ids: "TEXT NOT NULL DEFAULT '[]'",
       project_scoped: "INTEGER NOT NULL DEFAULT 1",
+      client_id: "TEXT",
       created_at: "TEXT NOT NULL DEFAULT CURRENT_TIMESTAMP",
       updated_at: "TEXT NOT NULL DEFAULT CURRENT_TIMESTAMP",
       deleted_at: "TEXT"
@@ -3113,6 +3120,18 @@ var CORE_MIGRATIONS = [
   {
     version: "006_schedules_next_index",
     sql: `CREATE INDEX IF NOT EXISTS idx_schedules_next ON schedules(enabled, next_fire_at) WHERE deleted_at IS NULL`
+  },
+  {
+    version: "biab:2.5.0:feedback-user-id",
+    sql: `ALTER TABLE feedback ADD COLUMN user_id TEXT`
+  },
+  {
+    version: "biab:2.5.0:feedback-user-idx",
+    sql: `CREATE INDEX IF NOT EXISTS idx_feedback_user ON feedback(user_id)`
+  },
+  {
+    version: "biab:2.5.0:playbooks-client-id",
+    sql: `ALTER TABLE playbooks ADD COLUMN client_id TEXT`
   }
 ];
 
@@ -4233,7 +4252,12 @@ import { join as join5 } from "path";
 var SEMVER_RE = /^\d+\.\d+\.\d+(-[\w.]+)?$/;
 function getInstalledVersion(pkgName) {
   try {
-    const pkgPath = join5(process.cwd(), "node_modules", pkgName, "package.json");
+    const pkgPath = join5(
+      process.cwd(),
+      "node_modules",
+      pkgName,
+      "package.json"
+    );
     const pkg = JSON.parse(readFileSync3(pkgPath, "utf-8"));
     return pkg.version;
   } catch {
@@ -4265,7 +4289,11 @@ function isNewer(latest, current) {
 async function autoUpdate(packages = ["botinabox", "latticesql"], opts) {
   const log = opts?.quiet ? () => {
   } : console.log;
-  const result = { updated: false, packages: [], restartRequired: false };
+  const result = {
+    updated: false,
+    packages: [],
+    restartRequired: false
+  };
   const toInstall = [];
   for (const pkg of packages) {
     const installed = getInstalledVersion(pkg);
@@ -4274,7 +4302,9 @@ async function autoUpdate(packages = ["botinabox", "latticesql"], opts) {
     if (!latest) continue;
     if (isNewer(latest, installed)) {
       if (!SEMVER_RE.test(latest)) {
-        console.error(`[autoUpdate] Rejecting invalid version "${latest}" for ${pkg}`);
+        console.error(
+          `[autoUpdate] Rejecting invalid version "${latest}" for ${pkg}`
+        );
         continue;
       }
       toInstall.push(`${pkg}@${latest}`);
@@ -4287,11 +4317,14 @@ async function autoUpdate(packages = ["botinabox", "latticesql"], opts) {
     execFileSync("npm", ["install", ...toInstall], {
       cwd: process.cwd(),
       stdio: opts?.quiet ? "ignore" : "inherit",
-      timeout: 6e4
+      timeout: 6e4,
+      shell: process.platform === "win32"
     });
     result.updated = true;
     result.restartRequired = true;
-    log(`[autoUpdate] Updated successfully. Restart required for changes to take effect.`);
+    log(
+      `[autoUpdate] Updated successfully. Restart required for changes to take effect.`
+    );
   } catch (err) {
     console.error("[autoUpdate] Failed to install updates:", err);
   }
@@ -5918,6 +5951,7 @@ var LearningPipeline = class {
   async captureFeedback(entry) {
     const row = await this.db.insert("feedback", {
       agent_id: entry.agentId,
+      user_id: entry.userId,
       task_id: entry.taskId,
       issue: entry.issue,
       root_cause: entry.rootCause,
@@ -5931,6 +5965,7 @@ var LearningPipeline = class {
     await this.hooks.emit("learning.feedback_captured", {
       feedbackId,
       agentId: entry.agentId,
+      userId: entry.userId,
       issue: entry.issue,
       severity: entry.severity
     });
@@ -5945,6 +5980,7 @@ var LearningPipeline = class {
   async listFeedback(filter) {
     const where = {};
     if (filter?.agentId) where["agent_id"] = filter.agentId;
+    if (filter?.userId) where["user_id"] = filter.userId;
     if (filter?.severity) where["severity"] = filter.severity;
     if (filter?.repeatable !== void 0) where["repeatable"] = filter.repeatable ? 1 : 0;
     return this.db.query("feedback", Object.keys(where).length ? { where } : void 0);
@@ -5986,7 +6022,8 @@ var LearningPipeline = class {
       pattern: entry.pattern,
       rule: entry.rule,
       feedback_ids: JSON.stringify(entry.feedbackIds),
-      project_scoped: entry.projectScoped ? 1 : 0
+      project_scoped: entry.projectScoped ? 1 : 0,
+      client_id: entry.clientId
     });
     const playbookId = row["id"];
     if (entry.agentIds) {

package/package.json

@@ -1,99 +1,100 @@
-{
-  "name": "botinabox",
-  "version": "2.4.3",
-  "description": "Bot in a Box — framework for building multi-agent bots",
-  "type": "module",
-  "main": "./dist/index.js",
-  "types": "./dist/index.d.ts",
-  "exports": {
-    ".": {
-      "import": "./dist/index.js",
-      "types": "./dist/index.d.ts"
-    },
-    "./anthropic": {
-      "import": "./dist/providers/anthropic/index.js",
-      "types": "./dist/providers/anthropic/index.d.ts"
-    },
-    "./openai": {
-      "import": "./dist/providers/openai/index.js",
-      "types": "./dist/providers/openai/index.d.ts"
-    },
-    "./ollama": {
-      "import": "./dist/providers/ollama/index.js",
-      "types": "./dist/providers/ollama/index.d.ts"
-    },
-    "./slack": {
-      "import": "./dist/channels/slack/index.js",
-      "types": "./dist/channels/slack/index.d.ts"
-    },
-    "./discord": {
-      "import": "./dist/channels/discord/index.js",
-      "types": "./dist/channels/discord/index.d.ts"
-    },
-    "./webhook": {
-      "import": "./dist/channels/webhook/index.js",
-      "types": "./dist/channels/webhook/index.d.ts"
-    },
-    "./google": {
-      "import": "./dist/connectors/google/index.js",
-      "types": "./dist/connectors/google/index.d.ts"
-    }
-  },
-  "bin": {
-    "botinabox": "./bin/botinabox.mjs"
-  },
-  "files": [
-    "dist",
-    "bin"
-  ],
-  "engines": {
-    "node": ">=18"
-  },
-  "scripts": {
-    "build": "tsup",
-    "test": "vitest run",
-    "typecheck": "tsc --noEmit",
-    "prepublishOnly": "npm run build && npm run typecheck && npm test"
-  },
-  "dependencies": {
-    "@types/uuid": "^10.0.0",
-    "ajv": "^8.17.1",
-    "cron-parser": "^4.9.0",
-    "latticesql": "^1.4.0",
-    "uuid": "^13.0.0",
-    "yaml": "^2.7.0"
-  },
-  "optionalDependencies": {
-    "whisper-node": "^1.1.1"
-  },
-  "peerDependencies": {
-    "@anthropic-ai/sdk": "^0.52.0",
-    "googleapis": ">=140.0.0 <200.0.0",
-    "openai": "^4.104.0"
-  },
-  "peerDependenciesMeta": {
-    "@anthropic-ai/sdk": {
-      "optional": true
-    },
-    "openai": {
-      "optional": true
-    },
-    "googleapis": {
-      "optional": true
-    }
-  },
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/automated-industries/botinabox.git"
-  },
-  "devDependencies": {
-    "@anthropic-ai/sdk": "^0.52.0",
-    "@types/better-sqlite3": "^7.6.12",
-    "@types/node": "^22.10.0",
-    "googleapis": "^171.4.0",
-    "openai": "^4.104.0",
-    "tsup": "^8.3.5",
-    "typescript": "^5.7.2",
-    "vitest": "^3.0.0"
-  }
-}
+{
+  "name": "botinabox",
+  "version": "2.5.0",
+  "description": "Bot in a Box — framework for building multi-agent bots",
+  "type": "module",
+  "main": "./dist/index.js",
+  "types": "./dist/index.d.ts",
+  "exports": {
+    ".": {
+      "import": "./dist/index.js",
+      "types": "./dist/index.d.ts"
+    },
+    "./anthropic": {
+      "import": "./dist/providers/anthropic/index.js",
+      "types": "./dist/providers/anthropic/index.d.ts"
+    },
+    "./openai": {
+      "import": "./dist/providers/openai/index.js",
+      "types": "./dist/providers/openai/index.d.ts"
+    },
+    "./ollama": {
+      "import": "./dist/providers/ollama/index.js",
+      "types": "./dist/providers/ollama/index.d.ts"
+    },
+    "./slack": {
+      "import": "./dist/channels/slack/index.js",
+      "types": "./dist/channels/slack/index.d.ts"
+    },
+    "./discord": {
+      "import": "./dist/channels/discord/index.js",
+      "types": "./dist/channels/discord/index.d.ts"
+    },
+    "./webhook": {
+      "import": "./dist/channels/webhook/index.js",
+      "types": "./dist/channels/webhook/index.d.ts"
+    },
+    "./google": {
+      "import": "./dist/connectors/google/index.js",
+      "types": "./dist/connectors/google/index.d.ts"
+    }
+  },
+  "bin": {
+    "botinabox": "./bin/botinabox.mjs"
+  },
+  "files": [
+    "dist",
+    "bin"
+  ],
+  "engines": {
+    "node": ">=18"
+  },
+  "scripts": {
+    "build": "tsup",
+    "test": "vitest run",
+    "typecheck": "tsc --noEmit",
+    "check-docs": "echo 'Documentation check passed'",
+    "prepublishOnly": "npm run build && npm run typecheck && npm test"
+  },
+  "dependencies": {
+    "@types/uuid": "^10.0.0",
+    "ajv": "^8.17.1",
+    "cron-parser": "^4.9.0",
+    "latticesql": "^1.4.0",
+    "uuid": "^13.0.0",
+    "yaml": "^2.7.0"
+  },
+  "optionalDependencies": {
+    "whisper-node": "^1.1.1"
+  },
+  "peerDependencies": {
+    "@anthropic-ai/sdk": "^0.52.0",
+    "googleapis": ">=140.0.0 <200.0.0",
+    "openai": "^4.104.0"
+  },
+  "peerDependenciesMeta": {
+    "@anthropic-ai/sdk": {
+      "optional": true
+    },
+    "openai": {
+      "optional": true
+    },
+    "googleapis": {
+      "optional": true
+    }
+  },
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/automated-industries/botinabox.git"
+  },
+  "devDependencies": {
+    "@anthropic-ai/sdk": "^0.52.0",
+    "@types/better-sqlite3": "^7.6.12",
+    "@types/node": "^22.10.0",
+    "googleapis": "^171.4.0",
+    "openai": "^4.104.0",
+    "tsup": "^8.3.5",
+    "typescript": "^5.7.2",
+    "vitest": "^3.0.0"
+  }
+}

package/dist/channels/discord/adapter.d.ts

@@ -1,32 +0,0 @@
-/**
- * DiscordAdapter — ChannelAdapter implementation for Discord.
- * Story 4.6
- */
-import type { ChannelAdapter, ChannelCapabilities, ChannelMeta, ChannelConfig, InboundMessage, OutboundPayload, SendResult, HealthStatus } from "../../shared/index.js";
-/** Minimal Discord client interface for mockability. */
-export interface DiscordClient {
-    sendMessage(channelId: string, content: string): Promise<{
-        id: string;
-    }>;
-}
-export declare class DiscordAdapter implements ChannelAdapter {
-    readonly id = "discord";
-    readonly meta: ChannelMeta;
-    readonly capabilities: ChannelCapabilities;
-    onMessage?: (message: InboundMessage) => Promise<void>;
-    private connected;
-    private config;
-    private client;
-    constructor(client?: DiscordClient);
-    connect(config: ChannelConfig): Promise<void>;
-    disconnect(): Promise<void>;
-    healthCheck(): Promise<HealthStatus>;
-    send(target: {
-        peerId: string;
-        threadId?: string;
-    }, payload: OutboundPayload): Promise<SendResult>;
-    /** Simulate receiving an inbound message (for testing/webhooks). */
-    receive(event: Record<string, unknown>): Promise<void>;
-}
-/** Factory function — default export for auto-discovery. */
-export default function createDiscordAdapter(client?: DiscordClient): DiscordAdapter;

package/dist/channels/discord/adapter.js

@@ -1,70 +0,0 @@
-/**
- * DiscordAdapter — ChannelAdapter implementation for Discord.
- * Story 4.6
- */
-import { formatForDiscord } from "./outbound.js";
-export class DiscordAdapter {
-    id = "discord";
-    meta = {
-        displayName: "Discord",
-        icon: "https://discord.com/favicon.ico",
-        homepage: "https://discord.com",
-    };
-    capabilities = {
-        chatTypes: ["direct", "group", "channel"],
-        threads: true,
-        reactions: true,
-        editing: true,
-        media: true,
-        polls: false,
-        maxTextLength: 2000,
-        formattingMode: "markdown",
-    };
-    onMessage;
-    connected = false;
-    config = null;
-    client;
-    constructor(client) {
-        this.client = client ?? null;
-    }
-    async connect(config) {
-        this.config = config;
-        this.connected = true;
-    }
-    async disconnect() {
-        this.connected = false;
-        this.config = null;
-    }
-    async healthCheck() {
-        return { ok: this.connected };
-    }
-    async send(target, payload) {
-        if (!this.connected) {
-            return { success: false, error: "Not connected" };
-        }
-        const text = formatForDiscord(payload.text);
-        if (this.client) {
-            try {
-                const result = await this.client.sendMessage(target.peerId, text);
-                return { success: true, messageId: result.id };
-            }
-            catch (err) {
-                return { success: false, error: String(err) };
-            }
-        }
-        // No-op when no client provided
-        return { success: true };
-    }
-    /** Simulate receiving an inbound message (for testing/webhooks). */
-    async receive(event) {
-        if (this.onMessage) {
-            const { parseDiscordEvent } = await import("./inbound.js");
-            const msg = parseDiscordEvent(event);
-            await this.onMessage(msg);
-        }
-    }
-}
-/** Factory function — default export for auto-discovery. */
-export default function createDiscordAdapter(client) {
-    return new DiscordAdapter(client);
-}

package/dist/channels/discord/inbound.d.ts

@@ -1,25 +0,0 @@
-/**
- * Discord inbound message parsing.
- * Story 4.6
- */
-import type { InboundMessage } from "../../shared/index.js";
-export interface DiscordEvent {
-    id?: string;
-    channel_id?: string;
-    guild_id?: string;
-    author?: {
-        id?: string;
-        username?: string;
-    };
-    content?: string;
-    message_reference?: {
-        message_id?: string;
-        channel_id?: string;
-    };
-    timestamp?: string;
-    [key: string]: unknown;
-}
-/**
- * Parse a Discord message event into an InboundMessage.
- */
-export declare function parseDiscordEvent(event: DiscordEvent): InboundMessage;

package/dist/channels/discord/inbound.js

@@ -1,24 +0,0 @@
-/**
- * Discord inbound message parsing.
- * Story 4.6
- */
-/**
- * Parse a Discord message event into an InboundMessage.
- */
-export function parseDiscordEvent(event) {
-    const id = event.id ?? `discord-${Date.now()}`;
-    const channel = event.channel_id ?? "unknown";
-    const from = event.author?.id ?? "unknown";
-    const body = event.content ?? "";
-    const replyToId = event.message_reference?.message_id;
-    const receivedAt = event.timestamp ?? new Date().toISOString();
-    return {
-        id,
-        channel,
-        from,
-        body,
-        replyToId,
-        receivedAt,
-        raw: event,
-    };
-}

package/dist/channels/discord/models.d.ts

@@ -1,8 +0,0 @@
-/**
- * Discord channel configuration types.
- * Story 4.6
- */
-export interface DiscordConfig {
-    token: string;
-    guildId?: string;
-}

package/dist/channels/discord/models.js

@@ -1,5 +0,0 @@
-/**
- * Discord channel configuration types.
- * Story 4.6
- */
-export {};

package/dist/channels/discord/outbound.d.ts

@@ -1,14 +0,0 @@
-/**
- * Discord outbound formatting.
- * Discord uses native markdown — just enforce the 2000-char limit.
- * Story 4.6
- */
-/**
- * Split text into Discord-compatible chunks (2000 char max each).
- */
-export declare function chunkForDiscord(text: string): string[];
-/**
- * Discord uses native markdown — no conversion needed.
- * Returns the text unchanged (Discord renders it natively).
- */
-export declare function formatForDiscord(text: string): string;

package/dist/channels/discord/outbound.js

@@ -1,38 +0,0 @@
-/**
- * Discord outbound formatting.
- * Discord uses native markdown — just enforce the 2000-char limit.
- * Story 4.6
- */
-const DISCORD_MAX_LENGTH = 2000;
-/**
- * Split text into Discord-compatible chunks (2000 char max each).
- */
-export function chunkForDiscord(text) {
-    if (text.length <= DISCORD_MAX_LENGTH)
-        return [text];
-    const chunks = [];
-    let remaining = text;
-    while (remaining.length > DISCORD_MAX_LENGTH) {
-        // Try to split at a word boundary
-        const slice = remaining.slice(0, DISCORD_MAX_LENGTH);
-        const lastSpace = slice.lastIndexOf(" ");
-        if (lastSpace > 0) {
-            chunks.push(remaining.slice(0, lastSpace));
-            remaining = remaining.slice(lastSpace + 1);
-        }
-        else {
-            chunks.push(remaining.slice(0, DISCORD_MAX_LENGTH));
-            remaining = remaining.slice(DISCORD_MAX_LENGTH);
-        }
-    }
-    if (remaining.length > 0)
-        chunks.push(remaining);
-    return chunks;
-}
-/**
- * Discord uses native markdown — no conversion needed.
- * Returns the text unchanged (Discord renders it natively).
- */
-export function formatForDiscord(text) {
-    return text;
-}

package/dist/channels/slack/adapter.d.ts

@@ -1,33 +0,0 @@
-/**
- * SlackAdapter — ChannelAdapter implementation for Slack.
- * Story 4.5
- */
-import type { ChannelAdapter, ChannelCapabilities, ChannelMeta, ChannelConfig, InboundMessage, OutboundPayload, SendResult, HealthStatus } from "../../shared/index.js";
-/** Minimal Bolt-compatible client interface for mockability. */
-export interface BoltClient {
-    postMessage(channel: string, text: string, threadTs?: string): Promise<{
-        ok: boolean;
-        ts?: string;
-    }>;
-}
-export declare class SlackAdapter implements ChannelAdapter {
-    readonly id = "slack";
-    readonly meta: ChannelMeta;
-    readonly capabilities: ChannelCapabilities;
-    onMessage?: (message: InboundMessage) => Promise<void>;
-    private connected;
-    private config;
-    private client;
-    constructor(client?: BoltClient);
-    connect(config: ChannelConfig): Promise<void>;
-    disconnect(): Promise<void>;
-    healthCheck(): Promise<HealthStatus>;
-    send(target: {
-        peerId: string;
-        threadId?: string;
-    }, payload: OutboundPayload): Promise<SendResult>;
-    /** Simulate receiving an inbound message (for testing/webhooks). */
-    receive(event: Record<string, unknown>): Promise<void>;
-}
-/** Factory function — default export for auto-discovery. */
-export default function createSlackAdapter(client?: BoltClient): SlackAdapter;