skyloom 1.22.0 → 1.24.0

package/tests/channel_setup.test.ts

@@ -0,0 +1,88 @@
+import { describe, it, expect, beforeEach, afterEach } from "vitest";
+import * as fs from "fs";
+import * as os from "os";
+import * as path from "path";
+import {
+  CHANNEL_SETUP,
+  SETUP_CHANNEL_IDS,
+  callbackUrl,
+  saveChannelConfig,
+  missingRequired,
+} from "../src/gateway/setup";
+import { renderQR } from "../src/gateway/qr";
+
+describe("channel setup · metadata", () => {
+  it("covers the three channels with console URLs and webhook paths", () => {
+    expect(SETUP_CHANNEL_IDS.sort()).toEqual(["feishu", "qq", "wecom"]);
+    for (const id of SETUP_CHANNEL_IDS) {
+      const s = CHANNEL_SETUP[id];
+      expect(s.consoleUrl).toMatch(/^https:\/\//);
+      expect(s.webhookPath).toBe(`/webhook/${id}`);
+      expect(s.fields.length).toBeGreaterThan(0);
+      expect(s.steps.length).toBeGreaterThan(0);
+    }
+  });
+
+  it("each channel marks its credential fields with env fallbacks", () => {
+    const feishu = CHANNEL_SETUP.feishu;
+    const appId = feishu.fields.find((f) => f.key === "appId")!;
+    expect(appId.required).toBe(true);
+    expect(appId.env).toBe("FEISHU_APP_ID");
+    const secret = feishu.fields.find((f) => f.key === "appSecret")!;
+    expect(secret.secret).toBe(true);
+  });
+});
+
+describe("channel setup · callbackUrl", () => {
+  it("joins base + webhook path, trimming trailing slash", () => {
+    expect(callbackUrl("https://bot.example.com", "feishu")).toBe("https://bot.example.com/webhook/feishu");
+    expect(callbackUrl("https://bot.example.com/", "wecom")).toBe("https://bot.example.com/webhook/wecom");
+    expect(callbackUrl("http://localhost:8848", "qq")).toBe("http://localhost:8848/webhook/qq");
+  });
+});
+
+describe("channel setup · missingRequired", () => {
+  it("reports unfilled required fields, ignores optional", () => {
+    expect(missingRequired("feishu", { appId: "a" })).toEqual(["appSecret"]);
+    expect(missingRequired("feishu", { appId: "a", appSecret: "s" })).toEqual([]);
+    // optional fields (verificationToken/encryptKey) never reported
+    expect(missingRequired("feishu", { appId: "a", appSecret: "s" })).not.toContain("encryptKey");
+  });
+});
+
+describe("channel setup · saveChannelConfig", () => {
+  let cfgPath: string;
+  beforeEach(() => { cfgPath = path.join(fs.mkdtempSync(path.join(os.tmpdir(), "sky-chcfg-")), "config.yaml"); });
+  afterEach(() => { try { fs.rmSync(path.dirname(cfgPath), { recursive: true, force: true }); } catch {} });
+
+  it("writes channels.<id> and merges on re-save", () => {
+    saveChannelConfig("feishu", { appId: "a", appSecret: "s" }, { configPath: cfgPath });
+    const yaml = require("yaml");
+    let cfg = yaml.parse(fs.readFileSync(cfgPath, "utf8"));
+    expect(cfg.channels.feishu).toMatchObject({ appId: "a", appSecret: "s", enabled: true });
+
+    // re-save adds a field without dropping the old ones
+    saveChannelConfig("feishu", { encryptKey: "k" }, { configPath: cfgPath });
+    cfg = yaml.parse(fs.readFileSync(cfgPath, "utf8"));
+    expect(cfg.channels.feishu).toMatchObject({ appId: "a", appSecret: "s", encryptKey: "k" });
+  });
+
+  it("preserves other top-level config when writing a channel", () => {
+    const yaml = require("yaml");
+    fs.mkdirSync(path.dirname(cfgPath), { recursive: true });
+    fs.writeFileSync(cfgPath, yaml.stringify({ default_model: "gpt-4o", api_keys: { openai: "sk" } }));
+    saveChannelConfig("qq", { appId: "1", secret: "x" }, { configPath: cfgPath });
+    const cfg = yaml.parse(fs.readFileSync(cfgPath, "utf8"));
+    expect(cfg.default_model).toBe("gpt-4o");
+    expect(cfg.api_keys.openai).toBe("sk");
+    expect(cfg.channels.qq.appId).toBe("1");
+  });
+});
+
+describe("channel setup · QR rendering", () => {
+  it("renders a non-empty scannable block for a URL", () => {
+    const qr = renderQR("https://open.feishu.cn/app");
+    expect(qr.length).toBeGreaterThan(50);
+    expect(qr).toMatch(/[█▀▄ ]/); // block-drawing characters
+  });
+});

package/tests/gateway.test.ts

@@ -1,7 +1,9 @@
 import { describe, it, expect } from "vitest";
 import * as crypto from "crypto";
 import { resolveSecret, TokenCache } from "../src/gateway/helpers";
-import { describeMedia } from "../src/gateway/types";
+import { describeMedia, parseReply } from "../src/gateway/types";
+import { isSendableSrc } from "../src/gateway/helpers";
+import { describeImages } from "../src/gateway/vision";
 import { buildAdapters, SUPPORTED_CHANNELS } from "../src/gateway/registry";
 import { decryptFeishu, createFeishuAdapter } from "../src/gateway/channels/feishu";
 import { wecomSignature, decryptWecom, createWecomAdapter } from "../src/gateway/channels/wecom";
@@ -81,6 +83,87 @@ describe("gateway · media", () => {
   });
 });
 
+describe("gateway · parseReply (outbound media)", () => {
+  it("extracts a markdown image and strips it from the text", () => {
+    const r = parseReply("看这张图 ![猫](https://x.com/cat.png) 好看吧");
+    expect(r.media).toEqual([{ kind: "image", src: "https://x.com/cat.png", alt: "猫" }]);
+    expect(r.text).toContain("看这张图");
+    expect(r.text).not.toContain("![");
+  });
+
+  it("extracts [[image:...]] and [[file:...|alt]] directives", () => {
+    const r = parseReply("结果:\n[[image:/tmp/out.png]]\n[[file:/tmp/report.pdf|季度报告]]");
+    expect(r.media).toEqual([
+      { kind: "image", src: "/tmp/out.png", alt: undefined },
+      { kind: "file", src: "/tmp/report.pdf", alt: "季度报告" },
+    ]);
+    expect(r.text).toBe("结果:");
+  });
+
+  it("leaves text without media untouched", () => {
+    const r = parseReply("就是一段普通文字");
+    expect(r.media).toHaveLength(0);
+    expect(r.text).toBe("就是一段普通文字");
+  });
+
+  it("handles multiple images in one reply", () => {
+    const r = parseReply("![a](http://h/1.png) 和 ![b](http://h/2.png)");
+    expect(r.media.map((m) => m.src)).toEqual(["http://h/1.png", "http://h/2.png"]);
+  });
+});
+
+describe("gateway · isSendableSrc", () => {
+  it("accepts http(s) URLs, rejects bare non-existent paths", () => {
+    expect(isSendableSrc("https://x.com/a.png")).toBe(true);
+    expect(isSendableSrc("http://x.com/a.png")).toBe(true);
+    expect(isSendableSrc("/no/such/file/xyz.png")).toBe(false);
+    expect(isSendableSrc("not a path")).toBe(false);
+  });
+});
+
+describe("gateway · sendMedia capability", () => {
+  it("all three adapters expose sendMedia", () => {
+    const f = createFeishuAdapter({ appId: "a", appSecret: "s" }, {})!;
+    const w = createWecomAdapter({ corpId: "c", corpSecret: "s", token: "t", encodingAesKey: "k".repeat(43), agentId: 1 }, {})!;
+    const q = createQQAdapter({ appId: "1", secret: "supersecretseedvalue" }, {})!;
+    expect(typeof f.sendMedia).toBe("function");
+    expect(typeof w.sendMedia).toBe("function");
+    expect(typeof q.sendMedia).toBe("function");
+  });
+
+  it("qq sendMedia rejects a non-URL source", async () => {
+    const q = createQQAdapter({ appId: "1", secret: "supersecretseedvalue" }, {})!;
+    await expect(q.sendMedia!({ channel: "qq", kind: "group", groupOpenid: "g" }, { kind: "image", src: "/local/file.png" }))
+      .rejects.toThrow(/http\(s\) URL/);
+  });
+});
+
+describe("gateway · vision (multimodal read)", () => {
+  it("describeImages returns null with no images", async () => {
+    expect(await describeImages([], { model: "gpt-4o-mini", env: {} })).toBeNull();
+  });
+  it("returns null when no API key is available (skips silently)", async () => {
+    const img = { data: Buffer.from("x"), filename: "a.png", contentType: "image/png" };
+    expect(await describeImages([img], { model: "gpt-4o-mini", env: {} })).toBeNull();
+  });
+  it("skips Anthropic models (not OpenAI-chat-shaped here)", async () => {
+    const img = { data: Buffer.from("x"), filename: "a.png" };
+    expect(await describeImages([img], { model: "claude-sonnet-4-6", env: { ANTHROPIC_API_KEY: "k" } })).toBeNull();
+  });
+  it("all three adapters expose fetchMedia", () => {
+    const f = createFeishuAdapter({ appId: "a", appSecret: "s" }, {})!;
+    const w = createWecomAdapter({ corpId: "c", corpSecret: "s", token: "t", encodingAesKey: "k".repeat(43), agentId: 1 }, {})!;
+    const q = createQQAdapter({ appId: "1", secret: "supersecretseedvalue" }, {})!;
+    expect(typeof f.fetchMedia).toBe("function");
+    expect(typeof w.fetchMedia).toBe("function");
+    expect(typeof q.fetchMedia).toBe("function");
+  });
+  it("qq fetchMedia returns null without a url", async () => {
+    const q = createQQAdapter({ appId: "1", secret: "supersecretseedvalue" }, {})!;
+    expect(await q.fetchMedia!({ kind: "image" } as any, {} as any)).toBeNull();
+  });
+});
+
 describe("gateway · registry", () => {
   it("lists the three supported channels", () => {
     expect(SUPPORTED_CHANNELS.sort()).toEqual(["feishu", "qq", "wecom"]);