@stigmer/cli 3.35.0 → 3.36.0

package/src/local/temporal/manager.test.ts

@@ -1,64 +0,0 @@
-import { existsSync, mkdirSync, mkdtempSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { describe, expect, it } from "vitest";
-import { temporalPidFile } from "../paths.js";
-import { writePidFile } from "../state/pidfile.js";
-import { TemporalManager } from "./manager.js";
-
-function tempHome(): string {
-  return mkdtempSync(join(tmpdir(), "stigmer-home-"));
-}
-
-describe("TemporalManager", () => {
-  it("exposes the frontend address", () => {
-    const manager = TemporalManager.forHome(tempHome());
-    expect(manager.address).toBe("127.0.0.1:7233");
-  });
-
-  it("reads the PID from the ~/.stigmer/temporal.pid location", () => {
-    const home = tempHome();
-    const manager = TemporalManager.forHome(home);
-    expect(manager.getPid()).toBeNull();
-    writePidFile(temporalPidFile(home), 5151);
-    expect(manager.getPid()).toBe(5151);
-  });
-
-  it("reports not-running when there is no PID file", async () => {
-    const manager = TemporalManager.forHome(tempHome());
-    expect(await manager.isRunning()).toBe(false);
-  });
-
-  // A stale PID file naming THIS process (the restarted container's fresh PID
-  // namespace reused it) can never be a running Temporal.
-  it("reports not-running when the PID file names the current process", async () => {
-    const home = tempHome();
-    writePidFile(temporalPidFile(home), process.pid);
-    expect(await TemporalManager.forHome(home).isRunning()).toBe(false);
-  });
-
-  describe("STIGMER_TEMPORAL_BIN", () => {
-    it("relocates only the binary; data, log, PID and lock stay under the home", async () => {
-      const home = tempHome();
-      const binPath = join(tempHome(), "opt", "temporal");
-      mkdirSync(join(binPath, ".."), { recursive: true });
-      writeFileSync(binPath, "#!/bin/sh\n");
-
-      const manager = TemporalManager.forHome(home, { STIGMER_TEMPORAL_BIN: binPath });
-      expect(manager.binPath).toBe(binPath);
-
-      // Nothing to download: the pre-installed binary is what ensureInstalled checks for.
-      await manager.ensureInstalled();
-      expect(existsSync(join(home, ".stigmer", "bin", "temporal"))).toBe(false);
-      // State still lives in the home layout.
-      writePidFile(temporalPidFile(home), 5151);
-      expect(manager.getPid()).toBe(5151);
-    });
-
-    it("is ignored when empty, keeping the ~/.stigmer/bin default", () => {
-      const home = tempHome();
-      const manager = TemporalManager.forHome(home, { STIGMER_TEMPORAL_BIN: "" });
-      expect(manager.binPath).toBe(join(home, ".stigmer", "bin", "temporal"));
-    });
-  });
-});

package/src/local/temporal/supervisor.test.ts

@@ -1,77 +0,0 @@
-import { describe, expect, it, vi } from "vitest";
-import { type Logger } from "../../logger.js";
-import { type SupervisedTarget, TemporalSupervisor } from "./supervisor.js";
-
-const silentLog: Logger = { debug: vi.fn(), info: vi.fn(), warn: vi.fn(), error: vi.fn() };
-
-function makeTarget(overrides: Partial<SupervisedTarget> = {}): SupervisedTarget & {
-  isRunning: ReturnType<typeof vi.fn>;
-  start: ReturnType<typeof vi.fn>;
-} {
-  return {
-    isRunning: vi.fn(async () => true),
-    start: vi.fn(async () => {}),
-    ...overrides,
-  } as never;
-}
-
-function makeSupervisor(target: SupervisedTarget) {
-  return new TemporalSupervisor(target, { backoffMs: 0, sleep: async () => {}, log: silentLog });
-}
-
-describe("TemporalSupervisor.checkHealthAndRestart", () => {
-  it("does nothing while the target is healthy", async () => {
-    const target = makeTarget({ isRunning: vi.fn(async () => true) });
-    await makeSupervisor(target).checkHealthAndRestart();
-    expect(target.start).not.toHaveBeenCalled();
-  });
-
-  it("restarts when the target is unhealthy", async () => {
-    const target = makeTarget({ isRunning: vi.fn(async () => false) });
-    await makeSupervisor(target).checkHealthAndRestart();
-    expect(target.start).toHaveBeenCalledTimes(1);
-  });
-
-  it("swallows a restart failure (retried on the next tick)", async () => {
-    const target = makeTarget({
-      isRunning: vi.fn(async () => false),
-      start: vi.fn(async () => {
-        throw new Error("boom");
-      }),
-    });
-    await expect(makeSupervisor(target).checkHealthAndRestart()).resolves.toBeUndefined();
-    expect(target.start).toHaveBeenCalledTimes(1);
-  });
-
-  it("does not restart if stopped during the backoff", async () => {
-    const target = makeTarget({ isRunning: vi.fn(async () => false) });
-    const supervisor = new TemporalSupervisor(target, {
-      backoffMs: 0,
-      // Stop the supervisor while it is "sleeping" through the backoff.
-      sleep: async () => {
-        supervisor.stop();
-      },
-      log: silentLog,
-    });
-    await supervisor.checkHealthAndRestart();
-    expect(target.start).not.toHaveBeenCalled();
-  });
-});
-
-describe("TemporalSupervisor start/stop", () => {
-  it("schedules ticks and stops cleanly", async () => {
-    vi.useFakeTimers();
-    try {
-      const target = makeTarget({ isRunning: vi.fn(async () => true) });
-      const supervisor = new TemporalSupervisor(target, { intervalMs: 1000, log: silentLog });
-      supervisor.start();
-      await vi.advanceTimersByTimeAsync(1000);
-      expect(target.isRunning).toHaveBeenCalledTimes(1);
-      supervisor.stop();
-      await vi.advanceTimersByTimeAsync(5000);
-      expect(target.isRunning).toHaveBeenCalledTimes(1); // no further ticks after stop
-    } finally {
-      vi.useRealTimers();
-    }
-  });
-});

package/src/marketplace/__fixtures__/cursor-marketplace.ts

@@ -1,69 +0,0 @@
-// A small Cursor-dialect marketplace written to a directory for the
-// marketplace suites: two plugins offered (one with a skill, one MCP-only,
-// the second under a nested source directory) and one entry whose directory
-// is missing, so every suite sees an offered list and a dropped entry. It
-// names itself `acme-plugins`: the vendors' own names are built into every
-// client, so a fixture under one of them would meet the reserved-name
-// refusal instead of being added. Its plugins carry names no real catalogue
-// offers (`warmer`, `codeforge`): a bare-name install searches the official
-// catalogue too, which in a checkout is the live `plugins/` tree, and a
-// fixture named after a real plugin would collide with it as the catalogue
-// grows.
-
-import { mkdirSync, writeFileSync } from "node:fs";
-import { dirname, join } from "node:path";
-
-export const CURSOR_MARKETPLACE_NAME = "acme-plugins";
-
-export function writeCursorMarketplace(root: string): void {
-  const write = (rel: string, content: string): void => {
-    const full = join(root, rel);
-    mkdirSync(dirname(full), { recursive: true });
-    writeFileSync(full, content);
-  };
-  write(
-    ".cursor-plugin/marketplace.json",
-    JSON.stringify({
-      name: CURSOR_MARKETPLACE_NAME,
-      metadata: { description: "A test catalogue" },
-      plugins: [
-        {
-          name: "warmer",
-          source: "warmer",
-          description: "Keeps things warm.",
-        },
-        { name: "codeforge", source: "third_party/codeforge" },
-        { name: "ghost", source: "ghost", description: "Listed, not present." },
-      ],
-    }),
-  );
-  write(
-    "warmer/.cursor-plugin/plugin.json",
-    JSON.stringify({
-      name: "warmer",
-      version: "1.0.0",
-      description: "The warmer plugin.",
-      skills: "./skills/",
-    }),
-  );
-  write(
-    "warmer/skills/warm/SKILL.md",
-    "---\nname: warm\ndescription: Warms.\n---\nKeep it warm.\n",
-  );
-  write(
-    "third_party/codeforge/.cursor-plugin/plugin.json",
-    JSON.stringify({
-      name: "codeforge",
-      version: "2.1.0",
-      mcpServers: "./mcp.json",
-    }),
-  );
-  write(
-    "third_party/codeforge/mcp.json",
-    JSON.stringify({
-      mcpServers: {
-        codeforge: { type: "http", url: "https://api.githubcopilot.com/mcp/" },
-      },
-    }),
-  );
-}

package/src/marketplace/config.test.ts

@@ -1,254 +0,0 @@
-// Pins the marketplace list over the real config file (HOME redirected):
-// the official one is always first and never stored; add and remove round-
-// trip through the file's own dialect (`type`); a hand-edited entry the CLI
-// cannot read is listed with its reason, kept on save, and named when a
-// command asks for it; the reserved name and --standalone refuse.
-
-import {
-  mkdirSync,
-  mkdtempSync,
-  readFileSync,
-  rmSync,
-  writeFileSync,
-} from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, it } from "vitest";
-import { configPath, load } from "../config/index.js";
-import { setStandalone } from "../runtime.js";
-import {
-  OFFICIAL_MARKETPLACE_NAME,
-  addMarketplace,
-  describeSource,
-  findMarketplace,
-  isOwnerRepo,
-  listMarketplaces,
-  narrowEntry,
-  removeMarketplace,
-} from "./config.js";
-
-let home: string;
-let originalHome: string | undefined;
-
-beforeEach(() => {
-  originalHome = process.env.HOME;
-  home = mkdtempSync(join(tmpdir(), "stigmer-marketplace-config-"));
-  process.env.HOME = home;
-});
-
-afterEach(() => {
-  setStandalone(false);
-  if (originalHome === undefined) delete process.env.HOME;
-  else process.env.HOME = originalHome;
-  rmSync(home, { recursive: true, force: true });
-});
-
-function writeConfig(yaml: string): void {
-  mkdirSync(join(home, ".stigmer"), { recursive: true });
-  writeFileSync(configPath(), yaml);
-}
-
-const BUILT_IN_NAMES = [OFFICIAL_MARKETPLACE_NAME];
-
-describe("listMarketplaces", () => {
-  it("is the built-in source alone on a fresh machine, the official catalogue", () => {
-    const listing = listMarketplaces();
-    expect(listing.known.map((m) => m.name)).toEqual(BUILT_IN_NAMES);
-    expect(listing.known[0]?.source).toEqual({ type: "official" });
-    expect(listing.known).toHaveLength(1);
-    expect(listing.unreadable).toEqual([]);
-  });
-
-  it("lists configured entries after the built-in, in file order, names the unreadable ones, and lets the built-in win over a same-named entry", () => {
-    writeConfig(
-      [
-        "backend:",
-        "  type: local",
-        "marketplaces:",
-        "  acme:",
-        "    type: github",
-        "    repo: acme/plugins",
-        // A hand edit under the built-in's name: the built-in wins, the entry stays in the file.
-        "  stigmer:",
-        "    type: github",
-        "    repo: someone/fork",
-        "  pinned:",
-        "    type: github",
-        "    repo: anthropics/claude-code",
-        "    ref: v1.2.3",
-        "  team:",
-        "    type: local",
-        "    path: /srv/plugins",
-        "  broken:",
-        "    type: gitlab",
-        "    repo: a/b",
-        "  typeless:",
-        "    repo: a/b",
-        "",
-      ].join("\n"),
-    );
-    const listing = listMarketplaces();
-    expect(listing.known.map((m) => m.name)).toEqual([
-      ...BUILT_IN_NAMES,
-      "acme",
-      "pinned",
-      "team",
-    ]);
-    expect(listing.known[0]?.source).toEqual({ type: "official" });
-    expect(listing.known[1]?.source).toEqual({
-      type: "github",
-      repo: "acme/plugins",
-    });
-    expect(listing.known[2]?.source).toEqual({
-      type: "github",
-      repo: "anthropics/claude-code",
-      ref: "v1.2.3",
-    });
-    expect(listing.known[3]?.source).toEqual({
-      type: "local",
-      path: "/srv/plugins",
-    });
-    expect(listing.unreadable).toEqual([
-      {
-        name: "broken",
-        reason: "unknown type 'gitlab' (expected 'github' or 'local')",
-      },
-      { name: "typeless", reason: "no 'type' (expected 'github' or 'local')" },
-    ]);
-  });
-});
-
-describe("addMarketplace and removeMarketplace", () => {
-  it("round-trips a github source and a local source through the file", () => {
-    addMarketplace("acme", {
-      type: "github",
-      repo: "acme/plugins",
-    });
-    addMarketplace("team", { type: "local", path: "/srv/plugins" });
-    const text = readFileSync(configPath(), "utf8");
-    expect(text).toContain("marketplaces:");
-    expect(text).toContain("type: github");
-    expect(text).toContain("repo: acme/plugins");
-    expect(text).not.toContain("ref:");
-    expect(text).toContain("path: /srv/plugins");
-    expect(listMarketplaces().known.map((m) => m.name)).toEqual([
-      ...BUILT_IN_NAMES,
-      "acme",
-      "team",
-    ]);
-
-    removeMarketplace("acme");
-    expect(listMarketplaces().known.map((m) => m.name)).toEqual([
-      ...BUILT_IN_NAMES,
-      "team",
-    ]);
-    removeMarketplace("team");
-    expect(load().marketplaces).toBeUndefined();
-    expect(readFileSync(configPath(), "utf8")).not.toContain("marketplaces");
-  });
-
-  it("keeps the rest of the config and an unreadable entry across a save", () => {
-    writeConfig(
-      "backend:\n  type: cloud\nbackends:\n  cloud:\n    type: cloud\n    token: t\ncurrent_backend: cloud\nmarketplaces:\n  broken:\n    type: gitlab\n",
-    );
-    addMarketplace("team", { type: "local", path: "/srv/plugins" });
-    const after = load();
-    expect(after.current_backend).toBe("cloud");
-    expect(after.backends?.["cloud"]?.token).toBe("t");
-    expect(after.marketplaces?.["broken"]).toEqual({ type: "gitlab" });
-    expect(after.marketplaces?.["team"]).toEqual({
-      type: "local",
-      path: "/srv/plugins",
-    });
-  });
-
-  it("refuses the built-in name, a duplicate, an unknown removal, and the built-in's removal; a vendor's name is free", () => {
-    expect(() =>
-      addMarketplace("stigmer", { type: "local", path: "/x" }),
-    ).toThrow(/'stigmer' is a built-in source and cannot be added or replaced/);
-    addMarketplace("cursor-plugins", { type: "github", repo: "cursor/plugins" });
-    addMarketplace("team", { type: "local", path: "/srv/plugins" });
-    expect(() =>
-      addMarketplace("team", { type: "local", path: "/other" }),
-    ).toThrow(/already configured/);
-    expect(() => removeMarketplace("nope")).toThrow(
-      /no marketplace named 'nope'/,
-    );
-    expect(() => removeMarketplace("stigmer")).toThrow(
-      /'stigmer' is a built-in source and cannot be removed/,
-    );
-  });
-
-  it("refuses under --standalone instead of writing a default file over the user's", () => {
-    writeConfig(
-      "backend:\n  type: cloud\nbackends:\n  cloud:\n    type: cloud\n    token: t\ncurrent_backend: cloud\n",
-    );
-    setStandalone(true);
-    expect(() => addMarketplace("team", { type: "local", path: "/x" })).toThrow(
-      /--standalone/,
-    );
-    expect(() => removeMarketplace("team")).toThrow(/--standalone/);
-    setStandalone(false);
-    expect(load().backends?.["cloud"]?.token).toBe("t");
-  });
-});
-
-describe("findMarketplace", () => {
-  it("finds the official and configured ones, returns undefined for an unknown name, and names an unreadable entry", () => {
-    writeConfig(
-      "backend:\n  type: local\nmarketplaces:\n  team:\n    type: local\n    path: /srv\n  broken:\n    type: gitlab\n",
-    );
-    expect(findMarketplace("stigmer")?.source).toEqual({ type: "official" });
-    expect(findMarketplace("team")?.source).toEqual({
-      type: "local",
-      path: "/srv",
-    });
-    expect(findMarketplace("nope")).toBeUndefined();
-    expect(() => findMarketplace("broken")).toThrow(
-      /configured but cannot be read: unknown type 'gitlab'/,
-    );
-  });
-});
-
-describe("narrowEntry, describeSource, isOwnerRepo", () => {
-  it("refuses a github entry without a repo and a local entry without a path", () => {
-    expect(narrowEntry({ type: "github" })).toEqual({
-      ok: false,
-      reason: "type 'github' needs 'repo: owner/repo'",
-    });
-    expect(narrowEntry({ type: "github", repo: "not-a-repo" })).toEqual({
-      ok: false,
-      reason: "type 'github' needs 'repo: owner/repo'",
-    });
-    expect(narrowEntry({ type: "github", repo: "a/b", ref: "" })).toEqual({
-      ok: false,
-      reason: "'ref' must be a non-empty branch, tag or commit",
-    });
-    expect(narrowEntry({ type: "local" })).toEqual({
-      ok: false,
-      reason: "type 'local' needs 'path: <directory>'",
-    });
-  });
-
-  it("describes each source in one phrase", () => {
-    expect(describeSource({ type: "official" })).toBe("built in");
-    expect(describeSource({ type: "github", repo: "cursor/plugins" })).toBe(
-      "github.com/cursor/plugins",
-    );
-    expect(
-      describeSource({ type: "github", repo: "cursor/plugins", ref: "main" }),
-    ).toBe("github.com/cursor/plugins@main");
-    expect(describeSource({ type: "local", path: "/srv/plugins" })).toBe(
-      "/srv/plugins",
-    );
-  });
-
-  it("accepts owner/repo as GitHub spells it and nothing else", () => {
-    expect(isOwnerRepo("cursor/plugins")).toBe(true);
-    expect(isOwnerRepo("my-org/my.repo_1")).toBe(true);
-    expect(isOwnerRepo("cursor")).toBe(false);
-    expect(isOwnerRepo("a/b/c")).toBe(false);
-    expect(isOwnerRepo("../x")).toBe(false);
-    expect(isOwnerRepo("a/")).toBe(false);
-  });
-});