abelworkflow 0.9.2 → 1.0.0-rc.1

package/skills/dev-browser/src/runtime.test.ts

@@ -1,60 +0,0 @@
-import { join } from "node:path";
-
-import { describe, expect, it } from "vitest";
-
-import { getPlaywrightBrowserRoots, isPlaywrightChromiumInstalled } from "./runtime.js";
-
-describe("getPlaywrightBrowserRoots", () => {
-  const skillDir = "/tmp/dev-browser";
-
-  it("uses the hermetic local browser directory when PLAYWRIGHT_BROWSERS_PATH is 0", () => {
-    expect(
-      getPlaywrightBrowserRoots({
-        skillDir,
-        platform: "win32",
-        env: { PLAYWRIGHT_BROWSERS_PATH: "0" },
-      })
-    ).toEqual([join(skillDir, "node_modules", "playwright-core", ".local-browsers")]);
-  });
-
-  it("prefers LOCALAPPDATA on Windows before USERPROFILE fallback", () => {
-    expect(
-      getPlaywrightBrowserRoots({
-        skillDir,
-        platform: "win32",
-        env: {
-          LOCALAPPDATA: "C:\\Users\\Alice\\AppData\\Local",
-          USERPROFILE: "C:\\Users\\Alice",
-        },
-      })
-    ).toEqual([join("C:\\Users\\Alice\\AppData\\Local", "ms-playwright")]);
-  });
-
-  it("falls back to USERPROFILE when LOCALAPPDATA is unavailable", () => {
-    expect(
-      getPlaywrightBrowserRoots({
-        skillDir,
-        platform: "win32",
-        env: { USERPROFILE: "C:\\Users\\Alice" },
-      })
-    ).toEqual([join("C:\\Users\\Alice", "AppData", "Local", "ms-playwright")]);
-  });
-});
-
-describe("isPlaywrightChromiumInstalled", () => {
-  const skillDir = "/tmp/dev-browser";
-
-  it("detects chromium in the hermetic local browser directory", () => {
-    const hermeticRoot = join(skillDir, "node_modules", "playwright-core", ".local-browsers");
-
-    expect(
-      isPlaywrightChromiumInstalled({
-        skillDir,
-        platform: "win32",
-        env: { PLAYWRIGHT_BROWSERS_PATH: "0" },
-        exists: (path) => path === hermeticRoot,
-        readDir: () => ["chromium-1162"],
-      })
-    ).toBe(true);
-  });
-});

package/skills/dev-browser/src/runtime.ts

@@ -1,171 +0,0 @@
-import { dirname, join } from "node:path";
-import { fileURLToPath } from "node:url";
-
-export interface CommandStatusResult {
-  status: number | null;
-}
-
-export interface PackageManager {
-  name: "bun" | "pnpm" | "npm";
-  command: string;
-  args: string[];
-}
-
-export interface ChromiumInstallCheckOptions {
-  skillDir: string;
-  platform: NodeJS.Platform;
-  env: NodeJS.ProcessEnv;
-  exists: (path: string) => boolean;
-  readDir: (path: string) => string[];
-}
-
-export interface MissingPackageDependenciesOptions {
-  skillDir: string;
-  dependencies: string[];
-  exists: (path: string) => boolean;
-}
-
-export function commandExists(
-  command: string,
-  runCheck: (command: string) => CommandStatusResult
-): boolean {
-  return runCheck(command).status === 0;
-}
-
-export function findAvailablePackageManager(
-  hasCommand: (command: string) => boolean
-): PackageManager | null {
-  const candidates: PackageManager[] = [
-    {
-      name: "bun",
-      command: "bunx",
-      args: ["playwright", "install", "chromium"],
-    },
-    {
-      name: "pnpm",
-      command: "pnpm",
-      args: ["exec", "playwright", "install", "chromium"],
-    },
-    {
-      name: "npm",
-      command: "npx",
-      args: ["playwright", "install", "chromium"],
-    },
-  ];
-
-  for (const candidate of candidates) {
-    const probe = candidate.name === "npm" ? "npm" : candidate.name;
-    if (hasCommand(probe)) {
-      return candidate;
-    }
-  }
-
-  return null;
-}
-
-export function getPlaywrightInstallCommand(manager: PackageManager): string {
-  return [manager.command, ...manager.args].join(" ");
-}
-
-function normalizePathForComparison(path: string): string {
-  return path.replaceAll("\\", "/").replace(/\/+/g, "/").toLowerCase();
-}
-
-export function getPlaywrightBrowserRoots({
-  skillDir,
-  platform,
-  env,
-}: {
-  skillDir: string;
-  platform: NodeJS.Platform;
-  env: NodeJS.ProcessEnv;
-}): string[] {
-  const explicitPath = env.PLAYWRIGHT_BROWSERS_PATH?.trim();
-  if (explicitPath) {
-    if (explicitPath === "0") {
-      return [join(skillDir, "node_modules", "playwright-core", ".local-browsers")];
-    }
-    return [explicitPath];
-  }
-
-  if (platform === "win32") {
-    const roots: string[] = [];
-    const localAppData = env.LOCALAPPDATA?.trim();
-    if (localAppData) {
-      roots.push(join(localAppData, "ms-playwright"));
-    }
-
-    const userProfile = env.USERPROFILE?.trim() ?? env.HOME?.trim();
-    if (userProfile) {
-      const fallbackRoot = join(userProfile, "AppData", "Local", "ms-playwright");
-      if (!roots.some((root) => normalizePathForComparison(root) === normalizePathForComparison(fallbackRoot))) {
-        roots.push(fallbackRoot);
-      }
-    }
-
-    return roots;
-  }
-
-  const home = env.HOME ?? env.USERPROFILE;
-  return home ? [join(home, ".cache", "ms-playwright")] : [];
-}
-
-export function isPlaywrightChromiumInstalled({
-  skillDir,
-  platform,
-  env,
-  exists,
-  readDir,
-}: ChromiumInstallCheckOptions): boolean {
-  for (const root of getPlaywrightBrowserRoots({ skillDir, platform, env })) {
-    if (!exists(root)) {
-      continue;
-    }
-
-    try {
-      const entries = readDir(root);
-      if (entries.some((entry) => entry.startsWith("chromium"))) {
-        return true;
-      }
-    } catch {
-      // Ignore unreadable directories and continue probing.
-    }
-  }
-
-  return false;
-}
-
-export function getMissingPackageDependencies({
-  skillDir,
-  dependencies,
-  exists,
-}: MissingPackageDependenciesOptions): string[] {
-  return dependencies.filter(
-    (dependency) =>
-      !exists(join(skillDir, "node_modules", ...dependency.split("/"), "package.json"))
-  );
-}
-
-export function resolveRuntimePaths(skillDir: string) {
-  const tmpDir = join(skillDir, "tmp");
-  const profileDir = join(skillDir, "profiles");
-
-  return {
-    skillDir,
-    tmpDir,
-    profileDir,
-    browserDataDir: join(profileDir, "browser-data"),
-  };
-}
-
-export function resolveImportMetaDir(moduleUrl: string): string {
-  return dirname(fileURLToPath(moduleUrl));
-}
-
-export function resolveSkillDirFromEntrypoint(moduleUrl: string): string {
-  return dirname(resolveImportMetaDir(moduleUrl));
-}
-
-export function shouldUseShellForPackageCommands(platform: NodeJS.Platform): boolean {
-  return platform === "win32";
-}

package/skills/dev-browser/src/startup.test.ts

@@ -1,95 +0,0 @@
-import { describe, expect, it, vi } from "vitest";
-
-import {
-  DEFAULT_HOST,
-  formatHttpUrl,
-  formatReadinessLines,
-  parseEntrypointArgs,
-  resolveHostForProbe,
-} from "./entrypoint.js";
-import { preflightStandaloneStartup } from "./startup.js";
-
-describe("entrypoint host handling", () => {
-  it("defaults standalone host to localhost", () => {
-    const args = parseEntrypointArgs([], {});
-
-    expect(args.host).toBe(DEFAULT_HOST);
-    expect(args.host).toBe("localhost");
-  });
-
-  it("formats IPv6 hosts in readiness output", () => {
-    expect(formatHttpUrl("::1", 9222)).toBe("http://[::1]:9222");
-
-    expect(
-      formatReadinessLines({
-        mode: "standalone",
-        host: "::1",
-        port: 9222,
-        wsEndpoint: "ws://127.0.0.1:9223/devtools/browser/test",
-        tmpDir: "/tmp/dev-browser",
-        profileDir: "/tmp/dev-browser/profile",
-      })
-    ).toContain("  HTTP: http://[::1]:9222");
-  });
-
-  it("normalizes wildcard hosts to a reachable probe target", () => {
-    expect(resolveHostForProbe("0.0.0.0")).toBe("127.0.0.1");
-    expect(resolveHostForProbe("::")).toBe("::1");
-    expect(resolveHostForProbe("[::1]")).toBe("::1");
-    expect(resolveHostForProbe("localhost")).toBe("localhost");
-  });
-});
-
-describe("preflightStandaloneStartup", () => {
-  it("checks the configured host before starting", async () => {
-    const checkServer = vi.fn().mockResolvedValue({ ok: false });
-    const isPortInUse = vi.fn().mockResolvedValue(false);
-    const log = vi.fn();
-
-    await expect(
-      preflightStandaloneStartup(
-        {
-          mode: "standalone",
-          host: "::1",
-          port: 9222,
-          cdpPort: 9223,
-          headless: false,
-        },
-        {
-          checkServer,
-          isPortInUse,
-          log,
-        }
-      )
-    ).resolves.toBe(true);
-
-    expect(checkServer).toHaveBeenCalledWith("::1", 9222);
-  });
-
-  it("short-circuits when the configured host already has a server", async () => {
-    const checkServer = vi.fn().mockResolvedValue({ ok: true, info: { wsEndpoint: "ws://test" } });
-    const isPortInUse = vi.fn();
-    const log = vi.fn();
-
-    await expect(
-      preflightStandaloneStartup(
-        {
-          mode: "standalone",
-          host: "localhost",
-          port: 9222,
-          cdpPort: 9223,
-          headless: false,
-        },
-        {
-          checkServer,
-          isPortInUse,
-          log,
-        }
-      )
-    ).resolves.toBe(false);
-
-    expect(checkServer).toHaveBeenCalledWith("localhost", 9222);
-    expect(isPortInUse).not.toHaveBeenCalled();
-    expect(log).toHaveBeenCalledWith("Server already running on port 9222");
-  });
-});

package/skills/dev-browser/src/startup.ts

@@ -1,153 +0,0 @@
-import { formatReadinessLines, type EntrypointArgs } from "./entrypoint.js";
-import type { PackageManager } from "./runtime.js";
-
-export interface RuntimePaths {
-  skillDir: string;
-  tmpDir: string;
-  profileDir: string;
-  browserDataDir: string;
-}
-
-interface StandaloneServer {
-  port: number;
-  wsEndpoint: string;
-  stop: () => Promise<void>;
-}
-
-interface ExtensionServer {
-  port: number;
-  wsEndpoint: string;
-  stop: () => Promise<void>;
-}
-
-export interface RunEntrypointDeps {
-  runtimePaths: RuntimePaths;
-  mkdir: (path: string, options: { recursive: true }) => void;
-  serveStandalone: (options: {
-    port: number;
-    host: string;
-    headless: boolean;
-    cdpPort: number;
-    profileDir: string;
-  }) => Promise<StandaloneServer>;
-  serveExtension: (options: { port: number; host: string }) => Promise<ExtensionServer>;
-  registerShutdown: (stop: () => Promise<void>) => void;
-  keepAlive: () => Promise<void>;
-  log: (line: string) => void;
-  ensureBrowser?: () => Promise<void>;
-  preflightStandalone?: () => Promise<boolean>;
-}
-
-export interface EnsurePlaywrightChromiumDeps {
-  isInstalled: () => boolean;
-  findPackageManager: () => PackageManager | null;
-  runCommand: (command: string, args: string[]) => Promise<void>;
-  log: (line: string) => void;
-}
-
-export interface PreflightStandaloneStartupDeps {
-  checkServer: (host: string, port: number) => Promise<{ ok: boolean; info?: { wsEndpoint?: string } }>;
-  isPortInUse: (port: number) => Promise<boolean>;
-  browserDataDir?: string;
-  recoverStaleBrowser?: (options: { cdpPort: number; browserDataDir: string }) => Promise<boolean>;
-  log: (line: string) => void;
-}
-
-export async function preflightStandaloneStartup(
-  args: EntrypointArgs,
-  deps: PreflightStandaloneStartupDeps
-): Promise<boolean> {
-  const serverCheck = await deps.checkServer(args.host, args.port);
-  if (serverCheck.ok) {
-    deps.log(`Server already running on port ${args.port}`);
-    return false;
-  }
-
-  if (await deps.isPortInUse(args.cdpPort)) {
-    const recovered =
-      deps.browserDataDir && deps.recoverStaleBrowser
-        ? await deps.recoverStaleBrowser({
-            cdpPort: args.cdpPort,
-            browserDataDir: deps.browserDataDir,
-          })
-        : false;
-
-    if (!recovered && (await deps.isPortInUse(args.cdpPort))) {
-      throw new Error(`CDP port ${args.cdpPort} is already in use by another process`);
-    }
-  }
-
-  return true;
-}
-
-export async function ensurePlaywrightChromium(deps: EnsurePlaywrightChromiumDeps) {
-  if (deps.isInstalled()) {
-    deps.log("Playwright Chromium already installed.");
-    return;
-  }
-
-  deps.log("Playwright Chromium not found. Installing (this may take a minute)...");
-  const manager = deps.findPackageManager();
-  if (!manager) {
-    throw new Error("No package manager found (tried bun, pnpm, npm)");
-  }
-
-  deps.log(`Using ${manager.name} to install Playwright...`);
-  await deps.runCommand(manager.command, manager.args);
-  deps.log("Chromium installed successfully.");
-}
-
-export async function runEntrypoint(args: EntrypointArgs, deps: RunEntrypointDeps) {
-  deps.mkdir(deps.runtimePaths.tmpDir, { recursive: true });
-
-  if (args.mode === "extension") {
-    const server = await deps.serveExtension({
-      port: args.port,
-      host: args.host,
-    });
-
-    for (const line of formatReadinessLines({
-      mode: "extension",
-      host: args.host,
-      port: args.port,
-      wsEndpoint: server.wsEndpoint,
-    })) {
-      deps.log(line);
-    }
-
-    deps.registerShutdown(() => server.stop());
-    await deps.keepAlive();
-    return;
-  }
-
-  deps.mkdir(deps.runtimePaths.profileDir, { recursive: true });
-
-  const shouldStart = await deps.preflightStandalone?.();
-  if (shouldStart === false) {
-    return;
-  }
-
-  await deps.ensureBrowser?.();
-
-  const server = await deps.serveStandalone({
-    port: args.port,
-    host: args.host,
-    headless: args.headless,
-    cdpPort: args.cdpPort,
-    profileDir: deps.runtimePaths.profileDir,
-  });
-
-  for (const line of formatReadinessLines({
-    mode: "standalone",
-    host: args.host,
-    port: args.port,
-    wsEndpoint: server.wsEndpoint,
-    tmpDir: deps.runtimePaths.tmpDir,
-    profileDir: deps.runtimePaths.profileDir,
-  })) {
-    deps.log(line);
-  }
-
-  deps.registerShutdown(() => server.stop());
-  await deps.keepAlive();
-}

package/skills/dev-browser/src/types.ts

@@ -1,35 +0,0 @@
-// API request/response types - shared between client and server
-
-export interface ServeOptions {
-  port?: number;
-  host?: string;
-  headless?: boolean;
-  cdpPort?: number;
-  /** Directory to store persistent browser profiles (cookies, localStorage, etc.) */
-  profileDir?: string;
-}
-
-export interface ViewportSize {
-  width: number;
-  height: number;
-}
-
-export interface GetPageRequest {
-  name: string;
-  /** Optional viewport size for new pages */
-  viewport?: ViewportSize;
-}
-
-export interface GetPageResponse {
-  wsEndpoint: string;
-  name: string;
-  targetId: string; // CDP target ID for reliable page matching
-}
-
-export interface ListPagesResponse {
-  pages: string[];
-}
-
-export interface ServerInfoResponse {
-  wsEndpoint: string;
-}

package/skills/dev-browser/tsconfig.json

@@ -1,36 +0,0 @@
-{
-  "compilerOptions": {
-    // Environment setup & latest features
-    "lib": ["ESNext"],
-    "target": "ESNext",
-    "module": "Preserve",
-    "moduleDetection": "force",
-    "jsx": "react-jsx",
-    "allowJs": true,
-
-    // Bundler mode
-    "moduleResolution": "bundler",
-    "allowImportingTsExtensions": true,
-    "verbatimModuleSyntax": true,
-    "noEmit": true,
-
-    // Path aliases
-    "baseUrl": ".",
-    "paths": {
-      "@/*": ["./src/*"]
-    },
-
-    // Best practices
-    "strict": true,
-    "skipLibCheck": true,
-    "noFallthroughCasesInSwitch": true,
-    "noUncheckedIndexedAccess": true,
-    "noImplicitOverride": true,
-
-    // Some stricter flags (disabled by default)
-    "noUnusedLocals": false,
-    "noUnusedParameters": false,
-    "noPropertyAccessFromIndexSignature": false
-  },
-  "include": ["src/**/*", "scripts/**/*"]
-}

package/skills/dev-browser/vitest.config.ts

@@ -1,12 +0,0 @@
-import { defineConfig } from "vitest/config";
-
-export default defineConfig({
-  test: {
-    globals: true,
-    environment: "node",
-    include: ["src/**/*.test.ts"],
-    testTimeout: 60000, // Playwright tests can be slow
-    hookTimeout: 60000,
-    teardownTimeout: 60000,
-  },
-});