@dobby.ai/dobby 0.1.1 → 0.1.2

package/src/cli/tests/discord-mapper.test.ts

@@ -1,128 +0,0 @@
-import assert from "node:assert/strict";
-import { access, mkdtemp, readFile } from "node:fs/promises";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import test from "node:test";
-import pino from "pino";
-import { mapDiscordMessage } from "../../../plugins/connector-discord/src/mapper.js";
-
-function createMessage(overrides?: {
-  id?: string;
-  content?: string;
-  attachments?: Map<string, unknown>;
-}): unknown {
-  return {
-    id: overrides?.id ?? "msg-1",
-    content: overrides?.content ?? "hello",
-    author: {
-      id: "user-1",
-      username: "alice",
-      bot: false,
-    },
-    attachments: overrides?.attachments ?? new Map(),
-    mentions: {
-      users: {
-        has: () => false,
-      },
-    },
-    guildId: "guild-1",
-    channelId: "channel-1",
-    channel: {
-      isThread: () => false,
-    },
-    createdTimestamp: 1_700_000_000_000,
-    toJSON: () => ({ ok: true }),
-  };
-}
-
-async function pathExists(path: string): Promise<boolean> {
-  try {
-    await access(path);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-const logger = pino({ enabled: false });
-
-test("mapDiscordMessage does not create attachment directory when message has no attachments", async () => {
-  const root = await mkdtemp(join(tmpdir(), "dobby-discord-mapper-empty-"));
-  const message = createMessage();
-
-  const envelope = await mapDiscordMessage(
-    message as never,
-    "discord.main",
-    "bot-1",
-    "source-1",
-    root,
-    logger,
-  );
-
-  assert.ok(envelope);
-  assert.deepEqual(envelope.attachments, []);
-  assert.equal(await pathExists(join(root, "source-1", "msg-1")), false);
-});
-
-test("mapDiscordMessage only creates attachment directory when a download succeeds", async (t) => {
-  t.mock.method(globalThis, "fetch", async () => new Response("file-body", { status: 200 }));
-
-  const root = await mkdtemp(join(tmpdir(), "dobby-discord-mapper-file-"));
-  const message = createMessage({
-    attachments: new Map([
-      ["att-1", {
-        id: "att-1",
-        name: "hello.png",
-        contentType: "image/png",
-        size: 9,
-        url: "https://example.test/hello.png",
-      }],
-    ]),
-  });
-
-  const envelope = await mapDiscordMessage(
-    message as never,
-    "discord.main",
-    "bot-1",
-    "source-1",
-    root,
-    logger,
-  );
-
-  assert.ok(envelope);
-  assert.equal(envelope.attachments.length, 1);
-  assert.equal(await pathExists(join(root, "source-1", "msg-1")), true);
-  assert.equal(envelope.attachments[0]?.localPath, join(root, "source-1", "msg-1", "hello.png"));
-  assert.equal(await readFile(join(root, "source-1", "msg-1", "hello.png"), "utf-8"), "file-body");
-});
-
-test("mapDiscordMessage does not leave an empty attachment directory when download fails", async (t) => {
-  t.mock.method(globalThis, "fetch", async () => new Response("nope", { status: 500 }));
-
-  const root = await mkdtemp(join(tmpdir(), "dobby-discord-mapper-fail-"));
-  const message = createMessage({
-    attachments: new Map([
-      ["att-1", {
-        id: "att-1",
-        name: "broken.png",
-        contentType: "image/png",
-        size: 9,
-        url: "https://example.test/broken.png",
-      }],
-    ]),
-  });
-
-  const envelope = await mapDiscordMessage(
-    message as never,
-    "discord.main",
-    "bot-1",
-    "source-1",
-    root,
-    logger,
-  );
-
-  assert.ok(envelope);
-  assert.equal(envelope.attachments.length, 1);
-  assert.equal(envelope.attachments[0]?.localPath, undefined);
-  assert.equal(await pathExists(join(root, "source-1", "msg-1")), false);
-});

package/src/cli/tests/doctor.test.ts

@@ -1,269 +0,0 @@
-import assert from "node:assert/strict";
-import { spawn } from "node:child_process";
-import { mkdtemp, mkdir, rm, writeFile } from "node:fs/promises";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import test from "node:test";
-
-/**
- * Writes a temporary default config under HOME/.dobby/gateway.json.
- */
-async function writeTempHomeConfig(homeDir: string, payload: unknown): Promise<string> {
-  const dobbyDir = join(homeDir, ".dobby");
-  await mkdir(dobbyDir, { recursive: true });
-  const configPath = join(dobbyDir, "gateway.json");
-  await writeFile(configPath, `${JSON.stringify(payload, null, 2)}\n`, "utf-8");
-  return configPath;
-}
-
-/**
- * Runs `dobby doctor` in a child process with an isolated HOME directory.
- */
-async function runDoctorWithHome(homeDir: string, configPath: string): Promise<{ code: number | null; output: string }> {
-  return await new Promise((resolve, reject) => {
-    const child = spawn(
-      process.execPath,
-      ["--import", "tsx", "src/main.ts", "doctor"],
-      {
-        cwd: process.cwd(),
-        env: {
-          ...process.env,
-          HOME: homeDir,
-          DOBBY_CONFIG_PATH: configPath,
-        },
-        stdio: ["ignore", "pipe", "pipe"],
-      },
-    );
-
-    let output = "";
-    child.stdout.on("data", (chunk) => {
-      output += String(chunk);
-    });
-    child.stderr.on("data", (chunk) => {
-      output += String(chunk);
-    });
-
-    child.once("error", (error) => reject(error));
-    child.once("close", (code) => {
-      resolve({ code, output });
-    });
-  });
-}
-
-test("doctor reports invalid binding route references", async () => {
-  const homeDir = await mkdtemp(join(tmpdir(), "dobby-doctor-home-"));
-
-  try {
-    const configPath = await writeTempHomeConfig(homeDir, {
-      extensions: { allowList: [] },
-      providers: {
-        default: "pi.main",
-        items: {
-          "pi.main": {
-            type: "provider.pi",
-          },
-        },
-      },
-      connectors: {
-        items: {
-          "discord.main": {
-            type: "connector.discord",
-            botName: "dobby-main",
-            botToken: "token",
-          },
-        },
-      },
-      sandboxes: {
-        default: "host.builtin",
-        items: {},
-      },
-      routes: {
-        defaults: {
-          provider: "pi.main",
-          sandbox: "host.builtin",
-          tools: "full",
-          mentions: "required",
-        },
-        items: {
-          main: {
-            projectRoot: process.cwd(),
-          },
-        },
-      },
-      bindings: {
-        items: {
-          "discord.main.123": {
-            connector: "discord.main",
-            source: {
-              type: "channel",
-              id: "123",
-            },
-            route: "missing-route",
-          },
-        },
-      },
-      data: {
-        rootDir: "./data",
-        dedupTtlMs: 604800000,
-      },
-    });
-
-    const result = await runDoctorWithHome(homeDir, configPath);
-    assert.equal(result.code, 1);
-    assert.equal(
-      result.output.includes("bindings.items['discord.main.123'].route") && result.output.includes("missing-route"),
-      true,
-    );
-  } finally {
-    await rm(homeDir, { recursive: true, force: true });
-  }
-});
-
-test("doctor reports invalid default binding route references", async () => {
-  const homeDir = await mkdtemp(join(tmpdir(), "dobby-doctor-default-binding-"));
-
-  try {
-    const configPath = await writeTempHomeConfig(homeDir, {
-      extensions: { allowList: [] },
-      providers: {
-        default: "pi.main",
-        items: {
-          "pi.main": {
-            type: "provider.pi",
-          },
-        },
-      },
-      connectors: {
-        items: {
-          "discord.main": {
-            type: "connector.discord",
-            botName: "dobby-main",
-            botToken: "token",
-          },
-        },
-      },
-      sandboxes: {
-        default: "host.builtin",
-        items: {},
-      },
-      routes: {
-        defaults: {
-          projectRoot: process.cwd(),
-          provider: "pi.main",
-          sandbox: "host.builtin",
-          tools: "full",
-          mentions: "required",
-        },
-        items: {
-          main: {},
-        },
-      },
-      bindings: {
-        default: {
-          route: "missing-route",
-        },
-        items: {},
-      },
-      data: {
-        rootDir: "./data",
-        dedupTtlMs: 604800000,
-      },
-    });
-
-    const result = await runDoctorWithHome(homeDir, configPath);
-    assert.equal(result.code, 1);
-    assert.equal(result.output.includes("bindings.default.route") && result.output.includes("missing-route"), true);
-  } finally {
-    await rm(homeDir, { recursive: true, force: true });
-  }
-});
-
-test("doctor reports init template placeholders as errors and warnings", async () => {
-  const homeDir = await mkdtemp(join(tmpdir(), "dobby-doctor-placeholders-"));
-
-  try {
-    const configPath = await writeTempHomeConfig(homeDir, {
-      extensions: { allowList: [] },
-      providers: {
-        default: "pi.main",
-        items: {
-          "pi.main": {
-            type: "provider.pi",
-            model: "REPLACE_WITH_PROVIDER_MODEL_ID",
-            baseUrl: "REPLACE_WITH_PROVIDER_BASE_URL",
-            apiKey: "REPLACE_WITH_PROVIDER_API_KEY_OR_ENV",
-          },
-        },
-      },
-      connectors: {
-        items: {
-          "discord.main": {
-            type: "connector.discord",
-            botName: "dobby-main",
-            botToken: "REPLACE_WITH_DISCORD_BOT_TOKEN",
-          },
-          "feishu.main": {
-            type: "connector.feishu",
-            appId: "REPLACE_WITH_FEISHU_APP_ID",
-            appSecret: "REPLACE_WITH_FEISHU_APP_SECRET",
-          },
-        },
-      },
-      sandboxes: {
-        default: "host.builtin",
-        items: {},
-      },
-      routes: {
-        defaults: {
-          provider: "pi.main",
-          sandbox: "host.builtin",
-          tools: "full",
-          mentions: "required",
-        },
-        items: {
-          main: {
-            projectRoot: "./REPLACE_WITH_PROJECT_ROOT",
-          },
-        },
-      },
-      bindings: {
-        items: {
-          "discord.main.main": {
-            connector: "discord.main",
-            source: {
-              type: "channel",
-              id: "YOUR_DISCORD_CHANNEL_ID",
-            },
-            route: "main",
-          },
-          "feishu.main.main": {
-            connector: "feishu.main",
-            source: {
-              type: "chat",
-              id: "YOUR_FEISHU_CHAT_ID",
-            },
-            route: "main",
-          },
-        },
-      },
-      data: {
-        rootDir: "./data",
-        dedupTtlMs: 604800000,
-      },
-    });
-
-    const result = await runDoctorWithHome(homeDir, configPath);
-    assert.equal(result.code, 1);
-    assert.equal(result.output.includes("providers.items['pi.main'].model still uses placeholder value"), true);
-    assert.equal(result.output.includes("providers.items['pi.main'].baseUrl still uses placeholder value"), true);
-    assert.equal(result.output.includes("providers.items['pi.main'].apiKey still uses placeholder value"), true);
-    assert.equal(result.output.includes("connectors.items['discord.main'].botToken still uses placeholder value"), true);
-    assert.equal(result.output.includes("connectors.items['feishu.main'].appId still uses placeholder value"), true);
-    assert.equal(result.output.includes("connectors.items['feishu.main'].appSecret still uses placeholder value"), true);
-    assert.equal(result.output.includes("routes.items['main'].projectRoot still uses placeholder value"), true);
-    assert.equal(result.output.includes("bindings.items['discord.main.main'].source.id still uses placeholder value"), true);
-    assert.equal(result.output.includes("bindings.items['feishu.main.main'].source.id still uses placeholder value"), true);
-  } finally {
-    await rm(homeDir, { recursive: true, force: true });
-  }
-});

package/src/cli/tests/init-catalog.test.ts

@@ -1,144 +0,0 @@
-import assert from "node:assert/strict";
-import test from "node:test";
-import { createInitSelectionConfig } from "../shared/init-catalog.js";
-
-test("createInitSelectionConfig writes Discord starter template for provider.pi", () => {
-  const selected = createInitSelectionConfig(["provider.pi"], ["connector.discord"], {
-    routeProviderChoiceId: "provider.pi",
-    defaultProjectRoot: "/Users/oolong/workspace/dobby",
-  });
-
-  assert.deepEqual(selected.providerChoiceIds, ["provider.pi"]);
-  assert.deepEqual(selected.connectorChoiceIds, ["connector.discord"]);
-  assert.equal(selected.routeId, "main");
-  assert.equal(selected.providerInstanceId, "pi.main");
-  assert.deepEqual(selected.providerInstances, [{
-    choiceId: "provider.pi",
-    instanceId: "pi.main",
-    contributionId: "provider.pi",
-    config: {
-      model: "REPLACE_WITH_PROVIDER_MODEL_ID",
-      baseUrl: "REPLACE_WITH_PROVIDER_BASE_URL",
-      apiKey: "REPLACE_WITH_PROVIDER_API_KEY_OR_ENV",
-    },
-  }]);
-  assert.deepEqual(selected.connectorInstances, [{
-    choiceId: "connector.discord",
-    instanceId: "discord.main",
-    contributionId: "connector.discord",
-    config: {
-      botName: "dobby-main",
-      botToken: "REPLACE_WITH_DISCORD_BOT_TOKEN",
-      reconnectStaleMs: 60_000,
-      reconnectCheckIntervalMs: 10_000,
-    },
-  }]);
-  assert.deepEqual(selected.routeDefaults, {
-    projectRoot: "/Users/oolong/workspace/dobby",
-    tools: "full",
-    mentions: "required",
-    provider: "pi.main",
-    sandbox: "host.builtin",
-  });
-  assert.deepEqual(selected.routeProfile, {});
-  assert.deepEqual(selected.defaultBinding, {
-    route: "main",
-  });
-  assert.deepEqual(selected.bindings, [{
-    id: "discord.main.main",
-    config: {
-      connector: "discord.main",
-      source: {
-        type: "channel",
-        id: "YOUR_DISCORD_CHANNEL_ID",
-      },
-      route: "main",
-    },
-  }]);
-});
-
-test("createInitSelectionConfig writes Feishu starter template for provider.claude-cli", () => {
-  const selected = createInitSelectionConfig(["provider.claude-cli"], ["connector.feishu"], {
-    routeProviderChoiceId: "provider.claude-cli",
-    defaultProjectRoot: "/Users/oolong/workspace/dobby",
-  });
-
-  assert.deepEqual(selected.providerChoiceIds, ["provider.claude-cli"]);
-  assert.deepEqual(selected.connectorChoiceIds, ["connector.feishu"]);
-  assert.equal(selected.routeId, "main");
-  assert.equal(selected.providerInstanceId, "claude-cli.main");
-  assert.deepEqual(selected.connectorInstances, [{
-    choiceId: "connector.feishu",
-    instanceId: "feishu.main",
-    contributionId: "connector.feishu",
-    config: {
-      appId: "REPLACE_WITH_FEISHU_APP_ID",
-      appSecret: "REPLACE_WITH_FEISHU_APP_SECRET",
-      domain: "feishu",
-      messageFormat: "card_markdown",
-      replyMode: "direct",
-      downloadAttachments: true,
-    },
-  }]);
-  assert.deepEqual(selected.bindings, [{
-    id: "feishu.main.main",
-    config: {
-      connector: "feishu.main",
-      source: {
-        type: "chat",
-        id: "YOUR_FEISHU_CHAT_ID",
-      },
-      route: "main",
-    },
-  }]);
-});
-
-test("createInitSelectionConfig supports multiple providers and connectors with one default provider", () => {
-  const selected = createInitSelectionConfig(
-    ["provider.pi", "provider.claude-cli"],
-    ["connector.discord", "connector.feishu"],
-    {
-      routeProviderChoiceId: "provider.claude-cli",
-      defaultProjectRoot: "/Users/oolong/workspace/dobby",
-    },
-  );
-
-  assert.deepEqual(selected.providerChoiceIds, ["provider.pi", "provider.claude-cli"]);
-  assert.deepEqual(selected.connectorChoiceIds, ["connector.discord", "connector.feishu"]);
-  assert.deepEqual(selected.extensionPackages, [
-    "@dobby.ai/provider-pi",
-    "@dobby.ai/provider-claude-cli",
-    "@dobby.ai/connector-discord",
-    "@dobby.ai/connector-feishu",
-  ]);
-  assert.equal(selected.providerInstanceId, "claude-cli.main");
-  assert.equal(selected.routeDefaults.provider, "claude-cli.main");
-  assert.equal(selected.routeProviderChoiceId, "provider.claude-cli");
-  assert.deepEqual(selected.defaultBinding, {
-    route: "main",
-  });
-  assert.deepEqual(selected.bindings, [
-    {
-      id: "discord.main.main",
-      config: {
-        connector: "discord.main",
-        source: {
-          type: "channel",
-          id: "YOUR_DISCORD_CHANNEL_ID",
-        },
-        route: "main",
-      },
-    },
-    {
-      id: "feishu.main.main",
-      config: {
-        connector: "feishu.main",
-        source: {
-          type: "chat",
-          id: "YOUR_FEISHU_CHAT_ID",
-        },
-        route: "main",
-      },
-    },
-  ]);
-});

package/src/cli/tests/program-options.test.ts

@@ -1,95 +0,0 @@
-import assert from "node:assert/strict";
-import test from "node:test";
-import { CommanderError } from "commander";
-import { buildProgram } from "../program.js";
-
-test("CLI rejects --config option", async () => {
-  const program = buildProgram();
-  program.configureOutput({
-    writeErr: () => {},
-    writeOut: () => {},
-  });
-  program.exitOverride();
-
-  await assert.rejects(
-    program.parseAsync(["node", "dobby", "--config", "./config/gateway.json"]),
-    (error) => {
-      assert.equal(error instanceof CommanderError, true);
-      assert.equal((error as CommanderError).code, "commander.unknownOption");
-      assert.match(String((error as CommanderError).message), /unknown option '--config'/i);
-      return true;
-    },
-  );
-});
-
-test("init help has no merge/overwrite flags", () => {
-  const program = buildProgram();
-  const initCommand = program.commands.find((command) => command.name() === "init");
-  assert.ok(initCommand);
-
-  const help = initCommand.helpInformation();
-  assert.equal(help.includes("--merge"), false);
-  assert.equal(help.includes("--merge-strategy"), false);
-  assert.equal(help.includes("--overwrite"), false);
-
-  assert.equal(help.includes("--preset"), false);
-  assert.equal(help.includes("--non-interactive"), false);
-  assert.equal(help.includes("--yes"), false);
-  assert.equal(help.includes("--config"), false);
-});
-
-test("config help shows read-only inspect commands and schema", () => {
-  const program = buildProgram();
-  const configCommand = program.commands.find((command) => command.name() === "config");
-  assert.ok(configCommand);
-
-  const help = configCommand.helpInformation();
-  assert.match(help, /show \[options\] \[section\]/);
-  assert.match(help, /list \[options\] \[section\]/);
-  assert.match(help, /schema/);
-  assert.equal(help.includes("edit"), false);
-
-  assert.equal(help.includes("get"), false);
-  assert.equal(help.includes("set"), false);
-  assert.equal(help.includes("unset"), false);
-});
-
-test("config schema help shows list/show subcommands", () => {
-  const program = buildProgram();
-  const configCommand = program.commands.find((command) => command.name() === "config");
-  assert.ok(configCommand);
-
-  const schemaCommand = configCommand.commands.find((command) => command.name() === "schema");
-  assert.ok(schemaCommand);
-
-  const help = schemaCommand.helpInformation();
-  assert.match(help, /list \[options\]/);
-  assert.match(help, /show \[options\] <contributionId>/);
-});
-
-test("cron help shows core subcommands", () => {
-  const program = buildProgram();
-  const cronCommand = program.commands.find((command) => command.name() === "cron");
-  assert.ok(cronCommand);
-
-  const help = cronCommand.helpInformation();
-  assert.match(help, /add \[options\] <name>/);
-  assert.match(help, /list \[options\]/);
-  assert.match(help, /run \[options\] <jobId>/);
-  assert.match(help, /remove \[options\] <jobId>/);
-});
-
-test("top-level help keeps bootstrap, inspect, install, validate, and ops commands only", () => {
-  const program = buildProgram();
-  const help = program.helpInformation();
-  assert.match(help, /start/);
-  assert.match(help, /init/);
-  assert.match(help, /config/);
-  assert.match(help, /extension/);
-  assert.match(help, /doctor/);
-  assert.match(help, /cron/);
-  assert.equal(help.includes("configure"), false);
-  assert.equal(help.includes("bot"), false);
-  assert.equal(help.includes("binding"), false);
-  assert.equal(help.includes("route"), false);
-});