@stigmer/cli 3.35.0 → 3.36.0

package/src/commands/install.test.ts

@@ -1,307 +0,0 @@
-// Command-level contract for `stigmer install`: a ref resolves through the
-// configured marketplaces to one entry, whose folder is pushed exactly as
-// `push plugin` would push it (the archive the backend receives is the one
-// the folder prepares to); --dry-run pushes nothing; a version pin, a path,
-// an unknown name and an ambiguous name refuse with their sentences. A real
-// Connect backend over h2c serves the plugin controllers; the config file
-// (HOME redirected) points a selfhost backend at it. The global fetch is
-// stubbed: the built-in vendor sources are asked through their marketplace
-// files (answered 404, so they declare nothing) and any download would fail
-// loudly, which pins that a bare install never fetches a vendor's zipball.
-
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
-import {
-  createServer as createHttp2Server,
-  type Http2Server,
-  type ServerHttp2Session,
-} from "node:http2";
-import type { AddressInfo } from "node:net";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { create } from "@bufbuild/protobuf";
-import type { ConnectRouter } from "@connectrpc/connect";
-import { connectNodeAdapter } from "@connectrpc/connect-node";
-import { PluginSchema } from "@stigmer/protos/ai/stigmer/agentic/plugin/v1/api_pb";
-import { PluginCommandController } from "@stigmer/protos/ai/stigmer/agentic/plugin/v1/command_pb";
-import {
-  ListPluginMembersResponseSchema,
-  type PushPluginRequest,
-} from "@stigmer/protos/ai/stigmer/agentic/plugin/v1/io_pb";
-import { PluginQueryController } from "@stigmer/protos/ai/stigmer/agentic/plugin/v1/query_pb";
-import { PluginState } from "@stigmer/protos/ai/stigmer/agentic/plugin/v1/status_pb";
-import { ApiResourceKind } from "@stigmer/protos/ai/stigmer/commons/apiresource/apiresourcekind/api_resource_kind_pb";
-import {
-  afterAll,
-  afterEach,
-  beforeAll,
-  beforeEach,
-  describe,
-  expect,
-  it,
-  vi,
-} from "vitest";
-import { classify, ExitCode } from "../errors/index.js";
-import { writeCursorMarketplace } from "../marketplace/__fixtures__/cursor-marketplace.js";
-import { preparePluginPush } from "../resources/plugin.js";
-import { buildProgram } from "../program.js";
-
-let backend: Http2Server;
-let port: number;
-const openSessions = new Set<ServerHttp2Session>();
-let pushes: PushPluginRequest[] = [];
-
-let home: string;
-let fixture: string;
-let originalHome: string | undefined;
-
-beforeAll(async () => {
-  const routes = (router: ConnectRouter) => {
-    router.service(PluginCommandController, {
-      push: (req) => {
-        pushes.push(req);
-        return create(PluginSchema, {
-          metadata: { id: "plg_1", slug: "warmer", org: "acme" },
-          spec: { name: "warmer", version: "1.0.0" },
-          status: {
-            digest: "a".repeat(64),
-            state: PluginState.READY,
-            materialized: { skills: 1, agents: 1 },
-          },
-        });
-      },
-    });
-    router.service(PluginQueryController, {
-      listMembers: () =>
-        create(ListPluginMembersResponseSchema, {
-          members: [
-            {
-              kind: ApiResourceKind.skill,
-              id: "skl_1",
-              slug: "warm",
-              name: "warm",
-            },
-            {
-              kind: ApiResourceKind.agent,
-              id: "agt_1",
-              slug: "warmer",
-              name: "warmer",
-            },
-          ],
-        }),
-    });
-  };
-  backend = createHttp2Server(connectNodeAdapter({ routes }));
-  backend.on("session", (session) => {
-    openSessions.add(session);
-    session.on("close", () => openSessions.delete(session));
-  });
-  await new Promise<void>((resolve) => backend.listen(0, "127.0.0.1", resolve));
-  port = (backend.address() as AddressInfo).port;
-});
-
-afterAll(async () => {
-  for (const session of openSessions) session.destroy();
-  await new Promise<void>((resolve) => backend.close(() => resolve()));
-});
-
-/** Every URL the stubbed fetch was asked; raw files answer 404, anything else refuses. */
-let fetched: string[] = [];
-
-beforeEach(() => {
-  pushes = [];
-  fetched = [];
-  vi.stubGlobal("fetch", async (input: RequestInfo | URL) => {
-    const url = String(input);
-    fetched.push(url);
-    if (url.startsWith("https://raw.githubusercontent.com/")) {
-      return new Response("Not Found", { status: 404 });
-    }
-    throw new Error(`unexpected network request in test: ${url}`);
-  });
-  originalHome = process.env.HOME;
-  home = mkdtempSync(join(tmpdir(), "stigmer-install-cmd-"));
-  fixture = mkdtempSync(join(tmpdir(), "stigmer-install-cmd-fixture-"));
-  writeCursorMarketplace(fixture);
-  process.env.HOME = home;
-  mkdirSync(join(home, ".stigmer"), { recursive: true });
-  writeFileSync(
-    join(home, ".stigmer", "config.yaml"),
-    [
-      "backend:",
-      "  type: cloud",
-      "backends:",
-      "  test:",
-      "    type: selfhost",
-      `    endpoint: 127.0.0.1:${port}`,
-      "current_backend: test",
-      "context:",
-      "  organization: acme",
-      "marketplaces:",
-      "  acme-plugins:",
-      "    type: local",
-      `    path: ${fixture}`,
-      "",
-    ].join("\n"),
-  );
-});
-
-afterEach(() => {
-  vi.unstubAllGlobals();
-  if (originalHome === undefined) delete process.env.HOME;
-  else process.env.HOME = originalHome;
-  rmSync(home, { recursive: true, force: true });
-  rmSync(fixture, { recursive: true, force: true });
-});
-
-interface RunOutcome {
-  readonly exitCode: number;
-  readonly message: string;
-  readonly stdout: string;
-}
-
-/** `stigmer [globals] install <args>`: the program's own flags (`--org`) go before the verb. */
-async function run(
-  args: string[] | string,
-  ...rest: string[]
-): Promise<RunOutcome> {
-  const [globals, local] = Array.isArray(args)
-    ? [args, rest]
-    : [[], [args, ...rest]];
-  const program = buildProgram();
-  program.exitOverride();
-  let stdout = "";
-  const outSpy = vi
-    .spyOn(process.stdout, "write")
-    .mockImplementation((chunk) => {
-      stdout += String(chunk);
-      return true;
-    });
-  const errSpy = vi.spyOn(process.stderr, "write").mockReturnValue(true);
-  try {
-    await program.parseAsync([
-      "node",
-      "stigmer",
-      ...globals,
-      "install",
-      ...local,
-    ]);
-    return { exitCode: ExitCode.Success, message: "", stdout };
-  } catch (err) {
-    return {
-      exitCode: classify(err)?.exitCode ?? -1,
-      message: err instanceof Error ? err.message : String(err),
-      stdout,
-    };
-  } finally {
-    outSpy.mockRestore();
-    errSpy.mockRestore();
-  }
-}
-
-describe("install", () => {
-  it("pushes the entry's folder as push plugin would, into the context org, and renders the install", async () => {
-    const outcome = await run("acme-plugins/warmer", "--json");
-    expect(outcome.exitCode).toBe(ExitCode.Success);
-    expect(pushes).toHaveLength(1);
-    const expected = await preparePluginPush(join(fixture, "warmer"));
-    expect(pushes[0]?.org).toBe("acme");
-    expect(
-      Buffer.from(pushes[0]?.artifact ?? []).equals(
-        Buffer.from(expected.archive),
-      ),
-    ).toBe(true);
-    expect(pushes[0]?.message).toBe(
-      "installed from marketplace 'acme-plugins'",
-    );
-
-    const payload = JSON.parse(outcome.stdout);
-    expect(payload.message).toMatch(
-      /^Installed plugin 'warmer' \(1 skill, 1 agent\)/,
-    );
-    const about = payload.sections.find(
-      (s: { title: string }) => s.title === "Plugin",
-    );
-    expect(about.fields).toContainEqual({
-      key: "Marketplace",
-      value: `acme-plugins (${fixture})`,
-    });
-    expect(payload.data.members).toHaveLength(2);
-  });
-
-  it("finds a bare name across the configured marketplaces and honours --org, --visibility and --message", async () => {
-    const outcome = await run(
-      ["--org", "other"],
-      "codeforge",
-      "--visibility",
-      "org",
-      "-m",
-      "hello",
-    );
-    expect(outcome.exitCode).toBe(ExitCode.Success);
-    expect(pushes[0]?.org).toBe("other");
-    expect(pushes[0]?.message).toBe("hello");
-    expect(pushes[0]?.visibility).toBeGreaterThan(0);
-  });
-
-  it("--dry-run reads the marketplace and describes the plugin without pushing", async () => {
-    const outcome = await run(
-      "acme-plugins/warmer@1.0.0",
-      "--dry-run",
-      "--json",
-    );
-    expect(outcome.exitCode).toBe(ExitCode.Success);
-    expect(pushes).toHaveLength(0);
-    const payload = JSON.parse(outcome.stdout);
-    expect(payload.message).toBe(
-      "Dry run: 'acme-plugins/warmer' would install plugin 'warmer'",
-    );
-    expect(payload.data.plugin.name).toBe("warmer");
-  });
-
-  it("refuses a version the marketplace does not offer, naming the one it does", async () => {
-    const outcome = await run("acme-plugins/warmer@9.9.9");
-    expect(outcome.exitCode).toBe(ExitCode.Usage);
-    expect(outcome.message).toMatch(
-      /pins version 9\.9\.9, but the marketplace offers 'warmer' at 1\.0\.0/,
-    );
-    expect(pushes).toHaveLength(0);
-  });
-
-  it("refuses a path toward push plugin, an unknown name, and an unknown marketplace", async () => {
-    expect((await run("./warmer")).message).toMatch(
-      /is a path.*stigmer push plugin \.\/warmer/s,
-    );
-    expect((await run("nope")).message).toMatch(
-      /no configured marketplace offers a plugin named 'nope'/,
-    );
-    expect((await run("nowhere/warmer")).message).toMatch(
-      /no marketplace named 'nowhere' is configured/,
-    );
-    expect(pushes).toHaveLength(0);
-  });
-
-  it("refuses a bare name two marketplaces offer, naming each qualified ref", async () => {
-    const second = mkdtempSync(join(tmpdir(), "stigmer-install-cmd-second-"));
-    try {
-      writeCursorMarketplace(second);
-      const { load, save } = await import("../config/index.js");
-      const config = load();
-      save({
-        ...config,
-        marketplaces: {
-          ...config.marketplaces,
-          second: { type: "local", path: second },
-        },
-      });
-      const outcome = await run("warmer");
-      expect(outcome.exitCode).toBe(ExitCode.Usage);
-      expect(outcome.message).toMatch(
-        /offered by more than one marketplace: acme-plugins, second\n\nName the one you mean: acme-plugins\/warmer or second\/warmer/,
-      );
-      // Only the official catalogue is built in, read from the checkout here; no GitHub host was asked.
-      expect(fetched.filter((url) => url.includes("github"))).toEqual([]);
-    } finally {
-      rmSync(second, { recursive: true, force: true });
-    }
-  });
-});

package/src/commands/marketplace.test.ts

@@ -1,205 +0,0 @@
-// Command-level contract for `stigmer marketplace`: add reads the source once
-// and records it under the file's own name (or --name), list is offline and
-// names an unreadable entry, show lists what a marketplace offers with each
-// entry's version, remove refuses the built-in name. Over the real config
-// file (HOME redirected) and a local marketplace fixture; no network.
-
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-import { load } from "../config/index.js";
-import { classify, ExitCode } from "../errors/index.js";
-import { writeCursorMarketplace } from "../marketplace/__fixtures__/cursor-marketplace.js";
-import { buildProgram } from "../program.js";
-
-let home: string;
-let fixture: string;
-let originalHome: string | undefined;
-
-beforeEach(() => {
-  originalHome = process.env.HOME;
-  home = mkdtempSync(join(tmpdir(), "stigmer-marketplace-cmd-"));
-  fixture = mkdtempSync(join(tmpdir(), "stigmer-marketplace-cmd-fixture-"));
-  writeCursorMarketplace(fixture);
-  process.env.HOME = home;
-});
-
-afterEach(() => {
-  if (originalHome === undefined) delete process.env.HOME;
-  else process.env.HOME = originalHome;
-  rmSync(home, { recursive: true, force: true });
-  rmSync(fixture, { recursive: true, force: true });
-});
-
-interface RunOutcome {
-  readonly exitCode: number;
-  readonly message: string;
-  readonly stdout: string;
-  readonly stderr: string;
-}
-
-async function run(...args: string[]): Promise<RunOutcome> {
-  const program = buildProgram();
-  program.exitOverride();
-  let stdout = "";
-  let stderr = "";
-  const outSpy = vi
-    .spyOn(process.stdout, "write")
-    .mockImplementation((chunk) => {
-      stdout += String(chunk);
-      return true;
-    });
-  const errSpy = vi
-    .spyOn(process.stderr, "write")
-    .mockImplementation((chunk) => {
-      stderr += String(chunk);
-      return true;
-    });
-  try {
-    await program.parseAsync(["node", "stigmer", "marketplace", ...args]);
-    return { exitCode: ExitCode.Success, message: "", stdout, stderr };
-  } catch (err) {
-    return {
-      exitCode: classify(err)?.exitCode ?? -1,
-      message: err instanceof Error ? err.message : String(err),
-      stdout,
-      stderr,
-    };
-  } finally {
-    outSpy.mockRestore();
-    errSpy.mockRestore();
-  }
-}
-
-describe("marketplace add", () => {
-  it("reads the source, records it under the file's own name, and says what it offers", async () => {
-    const outcome = await run("add", fixture, "--json");
-    expect(outcome.exitCode).toBe(ExitCode.Success);
-    const payload = JSON.parse(outcome.stdout);
-    expect(payload.message).toBe(
-      `Added source 'acme-plugins' (${fixture}), offering 2 plugins`,
-    );
-    expect(payload.data).toEqual({
-      name: "acme-plugins",
-      source: { type: "local", path: fixture },
-      plugins: 2,
-    });
-    expect(load().marketplaces?.["acme-plugins"]).toEqual({
-      type: "local",
-      path: fixture,
-    });
-  });
-
-  it("records it under --name instead, and refuses a name outside the grammar", async () => {
-    expect((await run("add", fixture, "--name", "cursor")).exitCode).toBe(
-      ExitCode.Success,
-    );
-    expect(Object.keys(load().marketplaces ?? {})).toEqual(["cursor"]);
-    const bad = await run("add", fixture, "--name", "Not Valid");
-    expect(bad.exitCode).toBe(ExitCode.Usage);
-    expect(bad.message).toMatch(/'Not Valid' is not a marketplace name/);
-  });
-
-  it("refuses a directory that is not a marketplace before recording anything", async () => {
-    const outcome = await run("add", home);
-    expect(outcome.exitCode).toBe(ExitCode.Usage);
-    expect(outcome.message).toMatch(/not a marketplace this CLI can read/);
-    expect(load().marketplaces).toBeUndefined();
-  });
-
-  it("refuses the built-in name toward --name, takes a vendor's name, and refuses a duplicate", async () => {
-    expect((await run("add", fixture, "--name", "stigmer")).message).toMatch(
-      /'stigmer' is a built-in source and cannot be added or replaced[\s\S]*--name/,
-    );
-    // A vendor's catalogue is a source the user adds; its name is his to take.
-    expect((await run("add", fixture, "--name", "cursor-plugins")).exitCode).toBe(
-      ExitCode.Success,
-    );
-    expect(load().marketplaces?.["cursor-plugins"]).toBeDefined();
-    await run("add", fixture);
-    expect((await run("add", fixture)).message).toMatch(/already configured/);
-  });
-});
-
-describe("marketplace list", () => {
-  it("is offline, the built-in first, and names an entry it cannot read", async () => {
-    await run("add", fixture);
-    const { save } = await import("../config/index.js");
-    const config = load();
-    save({
-      ...config,
-      marketplaces: { ...config.marketplaces, broken: { type: "gitlab" } },
-    });
-
-    const outcome = await run("list", "--json");
-    expect(outcome.exitCode).toBe(ExitCode.Success);
-    const payload = JSON.parse(outcome.stdout);
-    expect(payload.status).toBe("warning");
-    expect(payload.data.marketplaces).toEqual([
-      { name: "stigmer", source: { type: "official" } },
-      { name: "acme-plugins", source: { type: "local", path: fixture } },
-      {
-        name: "broken",
-        unreadable: "unknown type 'gitlab' (expected 'github' or 'local')",
-      },
-    ]);
-  });
-});
-
-describe("marketplace show", () => {
-  it("lists the offered plugins with their manifest versions and the entries it could not offer", async () => {
-    await run("add", fixture);
-    const outcome = await run("show", "acme-plugins", "--json");
-    expect(outcome.exitCode).toBe(ExitCode.Success);
-    const payload = JSON.parse(outcome.stdout);
-    expect(payload.message).toBe(
-      "Marketplace 'acme-plugins' offers 2 plugins, with 1 warning",
-    );
-    expect(payload.data.plugins).toEqual([
-      {
-        name: "warmer",
-        dir: "warmer",
-        version: "1.0.0",
-        description: "The warmer plugin.",
-      },
-      { name: "codeforge", dir: "third_party/codeforge", version: "2.1.0" },
-    ]);
-    expect(payload.data.warnings).toHaveLength(1);
-    expect(payload.data.warnings[0].subject).toBe("ghost");
-  });
-
-  it("shows the built-in marketplace", async () => {
-    const outcome = await run("show", "stigmer", "--json");
-    expect(outcome.exitCode).toBe(ExitCode.Success);
-    const payload = JSON.parse(outcome.stdout);
-    expect(payload.data.marketplace.name).toBe("stigmer");
-    expect(payload.data.plugins.map((p: { name: string }) => p.name)).toContain(
-      "linear",
-    );
-  });
-
-  it("refuses an unknown name toward list", async () => {
-    const outcome = await run("show", "nope");
-    expect(outcome.exitCode).toBe(ExitCode.Usage);
-    expect(outcome.message).toMatch(
-      /no marketplace named 'nope'.*stigmer marketplace list/s,
-    );
-  });
-});
-
-describe("marketplace remove", () => {
-  it("removes an added source and refuses a built-in and an unknown one", async () => {
-    await run("add", fixture);
-    expect((await run("remove", "acme-plugins")).exitCode).toBe(
-      ExitCode.Success,
-    );
-    expect(load().marketplaces).toBeUndefined();
-    expect((await run("remove", "stigmer")).message).toMatch(
-      /'stigmer' is a built-in source and cannot be removed/,
-    );
-    expect((await run("remove", "acme-plugins")).message).toMatch(
-      /no marketplace named 'acme-plugins'/,
-    );
-  });
-});

package/src/commands/mcp-server.test.ts

@@ -1,65 +0,0 @@
-import { describe, expect, it } from "vitest";
-import type { Config } from "../config/config.js";
-import { applyConfigEnv, applyFlagsEnv } from "./mcp-server.js";
-
-describe("applyConfigEnv", () => {
-  it("points a local backend at the local server", () => {
-    const env: NodeJS.ProcessEnv = {};
-    applyConfigEnv({ backend: { type: "local" } }, env);
-    expect(env.STIGMER_SERVER_ADDRESS).toBe("localhost:7234");
-    expect(env.STIGMER_API_KEY).toBeUndefined();
-  });
-
-  it("contributes the cloud endpoint and token", () => {
-    const env: NodeJS.ProcessEnv = {};
-    const config: Config = {
-      backend: { type: "cloud", cloud: { endpoint: "api.stigmer.ai:443", token: "tok-123" } },
-    };
-    applyConfigEnv(config, env);
-    expect(env.STIGMER_SERVER_ADDRESS).toBe("api.stigmer.ai:443");
-    expect(env.STIGMER_API_KEY).toBe("tok-123");
-  });
-
-  it("never overrides a value already present in the environment", () => {
-    const env: NodeJS.ProcessEnv = { STIGMER_SERVER_ADDRESS: "preset:7234" };
-    applyConfigEnv({ backend: { type: "local" } }, env);
-    expect(env.STIGMER_SERVER_ADDRESS).toBe("preset:7234");
-  });
-});
-
-describe("applyFlagsEnv", () => {
-  it("maps every set flag to its environment variable", () => {
-    const env: NodeJS.ProcessEnv = {};
-    applyFlagsEnv(
-      {
-        transport: "http",
-        port: "9090",
-        serverAddress: "host:7234",
-        apiKey: "key",
-        logFormat: "json",
-        logLevel: "debug",
-      },
-      env,
-    );
-    expect(env).toMatchObject({
-      STIGMER_MCP_TRANSPORT: "http",
-      STIGMER_MCP_HTTP_PORT: "9090",
-      STIGMER_SERVER_ADDRESS: "host:7234",
-      STIGMER_API_KEY: "key",
-      STIGMER_MCP_LOG_FORMAT: "json",
-      STIGMER_MCP_LOG_LEVEL: "debug",
-    });
-  });
-
-  it("overrides config-derived values (flags win)", () => {
-    const env: NodeJS.ProcessEnv = { STIGMER_SERVER_ADDRESS: "from-config:7234" };
-    applyFlagsEnv({ serverAddress: "from-flag:7234" }, env);
-    expect(env.STIGMER_SERVER_ADDRESS).toBe("from-flag:7234");
-  });
-
-  it("ignores unset flags", () => {
-    const env: NodeJS.ProcessEnv = {};
-    applyFlagsEnv({}, env);
-    expect(Object.keys(env)).toHaveLength(0);
-  });
-});