wave-agent-sdk 1.0.5 → 1.0.7

package/src/utils/shellResolver.ts

@@ -1,190 +0,0 @@
-import fs from "fs";
-import path from "path";
-import { execFileSync } from "child_process";
-
-export const WINDOWS_GIT_BASH_PATHS = [
-  "C:\\Program Files\\Git\\bin\\bash.exe",
-  "C:\\Program Files (x86)\\Git\\bin\\bash.exe",
-];
-
-// Common git.exe locations (checked before falling back to `where.exe`).
-const WINDOWS_GIT_EXE_PATHS = [
-  "C:\\Program Files\\Git\\cmd\\git.exe",
-  "C:\\Program Files (x86)\\Git\\cmd\\git.exe",
-];
-
-// Common directories where bash/zsh may live, used as fallback after PATH lookup.
-const FIXED_SHELL_DIRS = [
-  "/bin",
-  "/usr/bin",
-  "/usr/local/bin",
-  "/opt/homebrew/bin",
-];
-
-function isExecutable(shellPath: string): boolean {
-  try {
-    fs.accessSync(shellPath, fs.constants.X_OK);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-/** Resolve a command name (e.g. "bash") to an absolute path by searching $PATH. */
-function which(command: string): string | undefined {
-  const pathEnv = process.env.PATH;
-  if (!pathEnv) return undefined;
-  for (const dir of pathEnv.split(":")) {
-    if (!dir) continue;
-    const candidate = path.join(dir, command);
-    if (isExecutable(candidate)) {
-      return candidate;
-    }
-  }
-  return undefined;
-}
-
-function isSupportedShell(shellPath: string): boolean {
-  return shellPath.includes("bash") || shellPath.includes("zsh");
-}
-
-/**
- * Locate the `git` executable on Windows, then infer bash.exe from it.
- * Checks common Git for Windows install locations first, then falls back
- * to `where.exe git` (aligned with Claude Code's findExecutable('git')).
- * Returns the inferred bash.exe path, or undefined if not found.
- */
-function inferGitBashFromGit(): string | undefined {
-  // 1. Common git.exe locations
-  let gitExe: string | undefined;
-  for (const candidate of WINDOWS_GIT_EXE_PATHS) {
-    if (fs.existsSync(candidate)) {
-      gitExe = candidate;
-      break;
-    }
-  }
-
-  // 2. Fallback: where.exe git
-  if (!gitExe) {
-    try {
-      const result = execFileSync("where", ["git"], {
-        stdio: "pipe",
-        encoding: "utf8",
-        timeout: 3000,
-      }).trim();
-      // where.exe may return multiple lines; take the first non-empty one.
-      gitExe = result.split(/\r?\n/).find((line) => line.trim()) || undefined;
-    } catch {
-      gitExe = undefined;
-    }
-  }
-
-  if (!gitExe) return undefined;
-
-  // git.exe is at <install>/cmd/git.exe → bash.exe is at <install>/bin/bash.exe
-  // join treats git.exe as a segment, so ".." cancels it (→ cmd dir),
-  // the second ".." cancels cmd (→ <install>), then bin/bash.exe.
-  const bashPath = path.win32.join(gitExe, "..", "..", "bin", "bash.exe");
-  if (fs.existsSync(bashPath)) {
-    return bashPath;
-  }
-  return undefined;
-}
-
-/**
- * Resolve a bash or zsh binary on macOS/Linux.
- *
- * Priority (aligned with Claude Code's findSuitableShell):
- *   1. WAVE_SHELL env var (only if it points to an executable bash/zsh)
- *   2. $SHELL (only if it is bash/zsh)
- *   3. `which bash` / `which zsh` discovered on $PATH
- *   4. Common fixed locations (/bin, /usr/bin, /usr/local/bin, /opt/homebrew/bin)
- *
- * When $SHELL is bash, bash is preferred; otherwise zsh is preferred. This
- * matters because /bin/sh may be dash (Debian/Ubuntu) or POSIX-mode bash
- * (macOS), neither of which supports bashisms like process substitution
- * `<()`, causing `eval '<()...'` to fail at parse time.
- */
-function resolveUnixShell(): string | undefined {
-  // 1. WAVE_SHELL env override
-  const waveShell = process.env.WAVE_SHELL;
-  if (waveShell && isSupportedShell(waveShell) && isExecutable(waveShell)) {
-    return waveShell;
-  }
-
-  // 2. $SHELL (only bash/zsh)
-  const envShell = process.env.SHELL;
-  const isEnvShellSupported = !!envShell && isSupportedShell(envShell);
-  const preferBash = envShell?.includes("bash") ?? false;
-
-  // 3. Locate via PATH
-  const bashPath = which("bash");
-  const zshPath = which("zsh");
-
-  // 4. Fixed common locations, ordered by user preference
-  const shellOrder = preferBash ? ["bash", "zsh"] : ["zsh", "bash"];
-  const candidates = shellOrder.flatMap((shell) =>
-    FIXED_SHELL_DIRS.map((dir) => `${dir}/${shell}`),
-  );
-
-  // Discovered PATH paths: preferred type first, the other as fallback
-  if (preferBash) {
-    if (bashPath) candidates.unshift(bashPath);
-    if (zshPath) candidates.push(zshPath);
-  } else {
-    if (zshPath) candidates.unshift(zshPath);
-    if (bashPath) candidates.push(bashPath);
-  }
-
-  // Always prioritize $SHELL if it is a supported, executable shell
-  if (isEnvShellSupported && envShell && isExecutable(envShell)) {
-    candidates.unshift(envShell);
-  }
-
-  return candidates.find((candidate) => isExecutable(candidate));
-}
-
-/**
- * Resolve the Git Bash path on Windows.
- *
- * Priority (aligned with Claude Code's findGitBashPath):
- *   1. WAVE_GIT_BASH_PATH env var
- *   2. Infer from `git` executable location (<git dir>/../../bin/bash.exe)
- *   3. Common install paths (C:\Program Files\Git\bin\bash.exe, etc.)
- */
-function resolveWindowsShell(): string | undefined {
-  // 1. Env var override
-  if (process.env.WAVE_GIT_BASH_PATH) {
-    return process.env.WAVE_GIT_BASH_PATH;
-  }
-
-  // 2. Infer from git executable location
-  const inferred = inferGitBashFromGit();
-  if (inferred) {
-    return inferred;
-  }
-
-  // 4. Common install paths
-  const paths = [
-    ...WINDOWS_GIT_BASH_PATHS,
-    process.env.LOCALAPPDATA
-      ? `${process.env.LOCALAPPDATA}\\Programs\\Git\\bin\\bash.exe`
-      : null,
-  ].filter(Boolean) as string[];
-
-  for (const p of paths) {
-    if (fs.existsSync(p)) {
-      return p;
-    }
-  }
-
-  return undefined;
-}
-
-export function resolveShellPath(): string | undefined {
-  if (process.platform === "win32") {
-    return resolveWindowsShell();
-  }
-
-  return resolveUnixShell();
-}

package/src/utils/skillParser.ts

@@ -1,246 +0,0 @@
-import { readFileSync } from "fs";
-import { dirname } from "path";
-import type {
-  ParsedSkillFile,
-  SkillParseOptions,
-  SkillFrontmatter,
-  SkillMetadata,
-} from "../types/index.js";
-
-/**
- * Parse a SKILL.md file and validate its contents
- */
-export function parseSkillFile(
-  filePath: string,
-  options: SkillParseOptions = {},
-): ParsedSkillFile {
-  const { validateMetadata = true, basePath } = options;
-
-  const result: ParsedSkillFile = {
-    frontmatter: { name: "", description: "" },
-    content: "",
-    skillMetadata: {
-      name: "",
-      description: "",
-      type: "personal",
-      skillPath: "",
-    },
-    validationErrors: [],
-    isValid: false,
-  };
-
-  try {
-    // Read file content
-    const content = readFileSync(filePath, "utf-8");
-    result.content = content;
-
-    // Parse YAML frontmatter
-    const frontmatterMatch = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
-    if (!frontmatterMatch) {
-      result.validationErrors.push("Missing YAML frontmatter");
-      return result;
-    }
-
-    const yamlContent = frontmatterMatch[1];
-    result.frontmatter = parseYamlFrontmatter(yamlContent);
-
-    if (!result.frontmatter.name || !result.frontmatter.description) {
-      result.validationErrors.push(
-        "Missing required fields: name and description",
-      );
-      return result;
-    }
-
-    // Determine skill type and path
-    const skillPath = basePath || dirname(filePath);
-    const skillType =
-      skillPath.includes("/.wave/skills") ||
-      skillPath.includes("\\.wave\\skills") ||
-      skillPath.includes("/.claude/skills") ||
-      skillPath.includes("\\.claude\\skills") ||
-      skillPath.includes("/.agents/skills") ||
-      skillPath.includes("\\.agents\\skills")
-        ? "project"
-        : "personal";
-
-    // Extract allowed tools
-    let allowedTools: string[] | undefined;
-    const rawAllowedTools = result.frontmatter["allowed-tools"];
-    if (Array.isArray(rawAllowedTools)) {
-      allowedTools = rawAllowedTools.map((t) => String(t).trim());
-    } else if (typeof rawAllowedTools === "string") {
-      allowedTools = rawAllowedTools
-        .split(",")
-        .map((t) => t.trim())
-        .filter((t) => t.length > 0);
-    }
-
-    // Extract disable-model-invocation
-    const disableModelInvocation =
-      result.frontmatter["disable-model-invocation"] === true ||
-      result.frontmatter["disable-model-invocation"] === "true";
-
-    // Extract user-invocable (default to true)
-    const userInvocable =
-      result.frontmatter["user-invocable"] === undefined ||
-      result.frontmatter["user-invocable"] === true ||
-      result.frontmatter["user-invocable"] === "true";
-
-    result.skillMetadata = {
-      name: result.frontmatter.name,
-      description: result.frontmatter.description,
-      type: skillType,
-      skillPath,
-      allowedTools,
-      context: result.frontmatter.context,
-      agent: result.frontmatter.agent,
-      model: result.frontmatter.model as string | undefined,
-      disableModelInvocation,
-      userInvocable,
-    };
-
-    // Validate metadata if requested
-    if (validateMetadata) {
-      const validationErrors = validateSkillMetadata(result.skillMetadata);
-      result.validationErrors.push(...validationErrors);
-    }
-
-    result.isValid = result.validationErrors.length === 0;
-    return result;
-  } catch (error) {
-    result.validationErrors.push(
-      `Failed to read skill file: ${error instanceof Error ? error.message : String(error)}`,
-    );
-    return result;
-  }
-}
-
-/**
- * Simple YAML frontmatter parser for skill files
- */
-function parseYamlFrontmatter(yamlContent: string): SkillFrontmatter {
-  const frontmatter: SkillFrontmatter = { name: "", description: "" };
-
-  try {
-    const lines = yamlContent.split("\n");
-    let currentKey: string | null = null;
-
-    for (const line of lines) {
-      const trimmed = line.trim();
-      if (!trimmed || trimmed.startsWith("#")) continue;
-
-      // Check for list item
-      if (trimmed.startsWith("-") && currentKey) {
-        const value = trimmed
-          .substring(1)
-          .trim()
-          .replace(/^["']|["']$/g, "");
-        if (value) {
-          const existing = frontmatter[currentKey];
-          if (Array.isArray(existing)) {
-            (existing as string[]).push(value);
-          } else {
-            frontmatter[currentKey] = [value];
-          }
-        }
-        continue;
-      }
-
-      const colonIndex = trimmed.indexOf(":");
-      if (colonIndex === -1) continue;
-
-      const key = trimmed.substring(0, colonIndex).trim();
-      const value = trimmed
-        .substring(colonIndex + 1)
-        .trim()
-        .replace(/^["']|["']$/g, "");
-
-      currentKey = key;
-      if (key && value) {
-        frontmatter[key] = value;
-      } else if (key) {
-        // Key with no value on the same line, might be a list
-        frontmatter[key] = [];
-      }
-    }
-  } catch {
-    // Return empty frontmatter on parse error - validation will catch missing fields
-  }
-
-  return frontmatter;
-}
-
-/**
- * Validate skill metadata according to requirements
- */
-export function validateSkillMetadata(metadata: SkillMetadata): string[] {
-  const errors: string[] = [];
-
-  // Import SKILL_DEFAULTS dynamically to avoid circular imports
-  const NAME_PATTERN = /^[a-z0-9-]+$/;
-  const MAX_NAME_LENGTH = 64;
-  const MAX_DESCRIPTION_LENGTH = 1024;
-  const MIN_DESCRIPTION_LENGTH = 1;
-
-  // Validate name
-  if (!metadata.name) {
-    errors.push("Skill name is required");
-  } else {
-    if (metadata.name.length > MAX_NAME_LENGTH) {
-      errors.push(`Skill name must be ${MAX_NAME_LENGTH} characters or less`);
-    }
-    if (!NAME_PATTERN.test(metadata.name)) {
-      errors.push(
-        "Skill name must contain only lowercase letters, numbers, and hyphens",
-      );
-    }
-  }
-
-  // Validate description
-  if (!metadata.description) {
-    errors.push("Skill description is required");
-  } else {
-    if (metadata.description.length < MIN_DESCRIPTION_LENGTH) {
-      errors.push(
-        `Skill description must be at least ${MIN_DESCRIPTION_LENGTH} character`,
-      );
-    }
-    if (metadata.description.length > MAX_DESCRIPTION_LENGTH) {
-      errors.push(
-        `Skill description must be ${MAX_DESCRIPTION_LENGTH} characters or less`,
-      );
-    }
-  }
-
-  return errors;
-}
-
-/**
- * Check if a skill name is valid format
- */
-export function isValidSkillName(name: string): boolean {
-  const NAME_PATTERN = /^[a-z0-9-]+$/;
-  const MAX_NAME_LENGTH = 64;
-
-  return (
-    name.length > 0 && name.length <= MAX_NAME_LENGTH && NAME_PATTERN.test(name)
-  );
-}
-
-/**
- * Generate user-friendly error messages for skill validation
- */
-export function formatSkillError(skillPath: string, errors: string[]): string {
-  const header = `Skill validation failed for ${skillPath}:`;
-  const errorList = errors.map((error) => `  • ${error}`).join("\n");
-
-  const suggestions = [
-    "To fix this skill:",
-    "  1. Ensure SKILL.md has valid YAML frontmatter (---...---)",
-    "  2. Include required fields: name and description",
-    "  3. Use lowercase letters, numbers, and hyphens only for name",
-    "  4. Keep name under 64 characters and description under 1024 characters",
-  ].join("\n");
-
-  return `${header}\n${errorList}\n\n${suggestions}`;
-}

package/src/utils/stringUtils.ts

@@ -1,156 +0,0 @@
-/**
- * Remove code block wrappers
- * @param content Content that may contain code block wrappers
- * @returns Content after removing wrappers
- */
-export function removeCodeBlockWrappers(content: string): string {
-  // Remove code block wrappers from beginning and end
-  // Supports the following formats:
-  // ```language
-  // code content
-  // ```
-  //
-  // ```
-  // code content
-  // ```
-
-  const lines = content.split("\n");
-  let startIndex = 0;
-  let endIndex = lines.length - 1;
-
-  // Check if there is a code block marker at the beginning
-  if (lines[startIndex]?.trim().startsWith("```")) {
-    startIndex = 1;
-  }
-
-  // Check if there is a code block marker at the end
-  if (lines[endIndex]?.trim() === "```") {
-    endIndex = endIndex - 1;
-  }
-
-  // If no complete code block wrapper is found, return original content
-  if (startIndex === 0 && endIndex === lines.length - 1) {
-    return content;
-  }
-
-  // Return content after removing wrappers
-  return lines.slice(startIndex, endIndex + 1).join("\n");
-}
-
-/**
- * Parse custom headers from a string (typically from environment variables)
- * Format: Key: Value, separated by newlines
- * @param headersString String containing headers
- * @returns Record of headers
- */
-export function parseCustomHeaders(
-  headersString: string,
-): Record<string, string> {
-  if (!headersString || typeof headersString !== "string") {
-    return {};
-  }
-
-  const headers: Record<string, string> = {};
-  const lines = headersString.split(/\r?\n/);
-
-  for (const line of lines) {
-    const trimmedLine = line.trim();
-    if (!trimmedLine) continue;
-
-    const colonIndex = trimmedLine.indexOf(":");
-    if (colonIndex === -1) continue;
-
-    const key = trimmedLine.slice(0, colonIndex).trim();
-    const value = trimmedLine.slice(colonIndex + 1).trim();
-
-    if (key) {
-      headers[key] = value;
-    }
-  }
-
-  return headers;
-}
-
-/**
- * Function to remove ANSI color codes
- * @param text Text containing ANSI color codes
- * @returns Plain text with color codes removed
- */
-export const stripAnsiColors = (text: string): string => {
-  // Create the escape character dynamically to avoid control character detection
-  const escapeChar = String.fromCharCode(27); // ESC character
-  const ansiEscapeRegex = new RegExp(`${escapeChar}\\[[0-9;]*[a-zA-Z]`, "g");
-  return text.replace(ansiEscapeRegex, "");
-};
-
-/**
- * Format a line number prefix in cat -n style (padStart(6) + tab)
- * @param lineNumber The line number
- * @returns Formatted line number prefix
- */
-export function formatLineNumberPrefix(lineNumber: number): string {
-  return `${lineNumber.toString().padStart(6)}\t`;
-}
-
-/**
- * Attempt to recover truncated JSON (e.g., missing closing braces due to max tokens).
- * Tracks brace depth and only recovers if there are unclosed `{` braces.
- * Will NOT recover if there are unclosed `[` brackets (can't guess the content).
- * @param jsonStr Potentially truncated JSON string
- * @returns Recovered JSON string, or the original if unrecoverable
- */
-export function recoverTruncatedJson(jsonStr: string): string {
-  let braceDepth = 0;
-  let bracketDepth = 0;
-  let inString = false;
-  let escaped = false;
-
-  for (const ch of jsonStr) {
-    if (escaped) {
-      escaped = false;
-      continue;
-    }
-    if (ch === "\\" && inString) {
-      escaped = true;
-      continue;
-    }
-    if (ch === '"') {
-      inString = !inString;
-      continue;
-    }
-    if (!inString) {
-      if (ch === "{") braceDepth++;
-      if (ch === "}") braceDepth--;
-      if (ch === "[") bracketDepth++;
-      if (ch === "]") bracketDepth--;
-    }
-  }
-
-  // Build recovery suffix
-  let suffix = "";
-  if (inString) suffix += '"'; // Close unclosed string
-  if (braceDepth > 0 && bracketDepth === 0) {
-    suffix += "}".repeat(braceDepth);
-  }
-  return suffix ? jsonStr + suffix : jsonStr;
-}
-
-/**
- * Efficiently get the last N lines of a string without splitting the whole string.
- */
-export function getLastLines(text: string, count: number): string {
-  if (!text || count <= 0) return "";
-  let pos = text.length;
-  let found = 0;
-  while (pos > 0 && found < count) {
-    const nextNewline = text.lastIndexOf("\n", pos - 1);
-    if (nextNewline === -1) {
-      pos = 0;
-      break;
-    } else {
-      pos = nextNewline;
-      found++;
-    }
-  }
-  return text.substring(pos === 0 && found < count ? 0 : pos + 1);
-}