@creativeintelligence/abbie 0.1.3 → 0.1.5

package/src/lib/doctor/repos.ts

@@ -1,381 +0,0 @@
-/**
- * Per-repo contract validation for agents doctor
- *
- * Checks:
- * - .abbie/config.json or .loop.json exists
- * - Config is valid JSON
- * - extends references resolve
- * - Required fields present
- * - AGENTS.md exists
- * - CLAUDE.md → AGENTS.md symlink
- */
-
-import { existsSync, lstatSync, readdirSync, readFileSync, readlinkSync } from "node:fs";
-import { homedir } from "node:os";
-import { basename, join } from "node:path";
-import type { Check } from "./index.js";
-import { resolveExtends } from "./templates.js";
-
-// ============================================================================
-// Types
-// ============================================================================
-
-export interface RepoConfig {
-  path: string;
-  name: string;
-  configType: "agents-config" | "loop-json" | "none";
-  configPath?: string;
-  config?: Record<string, unknown>;
-  hasAgentsMd: boolean;
-  hasClaudeMd: boolean;
-  agentsIsSymlink: boolean;
-  claudeIsSymlink: boolean;
-  agentsSymlinkTarget?: string;
-  claudeSymlinkTarget?: string;
-  extendsResolved?: boolean;
-}
-
-export interface RepoCheckResult {
-  repo: string;
-  checks: Check[];
-}
-
-// ============================================================================
-// Constants
-// ============================================================================
-
-const DEFAULT_REPOS_PATH = join(homedir(), "Developer", "apps");
-
-// ============================================================================
-// Repo discovery
-// ============================================================================
-
-export function discoverRepos(basePath: string = DEFAULT_REPOS_PATH): string[] {
-  const repos: string[] = [];
-
-  if (!existsSync(basePath)) {
-    return repos;
-  }
-
-  try {
-    // Look for directories that contain a package.json or .git
-    const entries = readdirSync(basePath, { withFileTypes: true });
-
-    for (const entry of entries) {
-      if (!entry.isDirectory()) continue;
-
-      const projectPath = join(basePath, entry.name);
-
-      // Check for nested project structure (e.g., arbor/arbor-xyz)
-      const nestedEntries = readdirSync(projectPath, { withFileTypes: true });
-      for (const nested of nestedEntries) {
-        if (!nested.isDirectory()) continue;
-
-        const nestedPath = join(projectPath, nested.name);
-        if (existsSync(join(nestedPath, "package.json")) || existsSync(join(nestedPath, ".git"))) {
-          repos.push(nestedPath);
-        }
-      }
-
-      // Also check if the top-level is a project
-      if (existsSync(join(projectPath, "package.json")) || existsSync(join(projectPath, ".git"))) {
-        repos.push(projectPath);
-      }
-    }
-  } catch {
-    // Ignore errors reading directory
-  }
-
-  return repos;
-}
-
-// ============================================================================
-// Single repo analysis
-// ============================================================================
-
-export function analyzeRepo(repoPath: string): RepoConfig {
-  const name = basename(repoPath);
-  const result: RepoConfig = {
-    path: repoPath,
-    name,
-    configType: "none",
-    hasAgentsMd: false,
-    hasClaudeMd: false,
-    agentsIsSymlink: false,
-    claudeIsSymlink: false,
-  };
-
-  // Check for .abbie/config.json (preferred)
-  const agentsConfigPath = join(repoPath, ".abbie", "config.json");
-  if (existsSync(agentsConfigPath)) {
-    result.configType = "agents-config";
-    result.configPath = agentsConfigPath;
-    try {
-      result.config = JSON.parse(readFileSync(agentsConfigPath, "utf-8"));
-    } catch {
-      // Invalid JSON, will be caught in checks
-    }
-  }
-
-  // Check for .loop.json (legacy)
-  const loopJsonPath = join(repoPath, ".loop.json");
-  if (result.configType === "none" && existsSync(loopJsonPath)) {
-    result.configType = "loop-json";
-    result.configPath = loopJsonPath;
-    try {
-      result.config = JSON.parse(readFileSync(loopJsonPath, "utf-8"));
-    } catch {
-      // Invalid JSON, will be caught in checks
-    }
-  }
-
-  // Check extends resolution
-  if (result.config && typeof result.config.extends === "string") {
-    const resolved = resolveExtends(result.config.extends);
-    result.extendsResolved = resolved.exists;
-  }
-
-  // Check for AGENTS.md
-  const agentsPath = join(repoPath, "AGENTS.md");
-  result.hasAgentsMd = existsSync(agentsPath);
-
-  // Check for CLAUDE.md
-  const claudePath = join(repoPath, "CLAUDE.md");
-  result.hasClaudeMd = existsSync(claudePath);
-
-  // Check symlink direction
-  if (result.hasAgentsMd) {
-    try {
-      result.agentsIsSymlink = lstatSync(agentsPath).isSymbolicLink();
-      if (result.agentsIsSymlink) {
-        result.agentsSymlinkTarget = readlinkSync(agentsPath);
-      }
-    } catch {
-      // Ignore stat errors
-    }
-  }
-
-  if (result.hasClaudeMd) {
-    try {
-      result.claudeIsSymlink = lstatSync(claudePath).isSymbolicLink();
-      if (result.claudeIsSymlink) {
-        result.claudeSymlinkTarget = readlinkSync(claudePath);
-      }
-    } catch {
-      // Ignore stat errors
-    }
-  }
-
-  return result;
-}
-
-// ============================================================================
-// Check generation
-// ============================================================================
-
-export function checkRepo(repo: RepoConfig): Check[] {
-  const checks: Check[] = [];
-  const shortPath = repo.path.replace(homedir(), "~");
-
-  // Config existence
-  if (repo.configType === "agents-config") {
-    checks.push({
-      name: `repo-${repo.name}-config`,
-      status: "ok",
-      detail: `${shortPath}/.abbie/config.json exists`,
-    });
-  } else if (repo.configType === "loop-json") {
-    checks.push({
-      name: `repo-${repo.name}-config`,
-      status: "warn",
-      detail: `${shortPath}/.loop.json exists (migrate to .abbie/config.json)`,
-    });
-  } else {
-    checks.push({
-      name: `repo-${repo.name}-config`,
-      status: "warn",
-      detail: `${shortPath} has no agent config`,
-    });
-  }
-
-  // Config validity
-  if (repo.configPath && !repo.config) {
-    checks.push({
-      name: `repo-${repo.name}-config-valid`,
-      status: "missing",
-      detail: `${basename(repo.configPath)} is invalid JSON`,
-    });
-  }
-
-  // Extends resolution
-  if (repo.config && typeof repo.config.extends === "string") {
-    if (repo.extendsResolved) {
-      checks.push({
-        name: `repo-${repo.name}-extends`,
-        status: "ok",
-        detail: `extends resolved: ${repo.config.extends}`,
-      });
-    } else {
-      checks.push({
-        name: `repo-${repo.name}-extends`,
-        status: "missing",
-        detail: `extends not found: ${repo.config.extends}`,
-      });
-    }
-  }
-
-  // Project fields
-  if (repo.config) {
-    const project = repo.config.project as Record<string, unknown> | undefined;
-    if (project?.name && project.team) {
-      checks.push({
-        name: `repo-${repo.name}-project`,
-        status: "ok",
-        detail: `project: ${project.name} (${project.team})`,
-      });
-    } else if (typeof repo.config.project === "string") {
-      // Old format: project is just a string
-      checks.push({
-        name: `repo-${repo.name}-project`,
-        status: "warn",
-        detail: `project is string, migrate to object with name/team`,
-      });
-    } else if (!project) {
-      checks.push({
-        name: `repo-${repo.name}-project`,
-        status: "warn",
-        detail: `missing project field`,
-      });
-    }
-  }
-
-  // AGENTS.md / CLAUDE.md
-  if (repo.hasAgentsMd && repo.hasClaudeMd) {
-    const claudeTarget = repo.claudeSymlinkTarget?.replace(/^[.][/]/, "");
-    const agentsTarget = repo.agentsSymlinkTarget?.replace(/^[.][/]/, "");
-
-    if (!repo.agentsIsSymlink && repo.claudeIsSymlink && claudeTarget === "AGENTS.md") {
-      checks.push({
-        name: `repo-${repo.name}-docs`,
-        status: "ok",
-        detail: `AGENTS.md is canonical, CLAUDE.md → AGENTS.md`,
-      });
-    } else if (repo.agentsIsSymlink && agentsTarget === "CLAUDE.md") {
-      checks.push({
-        name: `repo-${repo.name}-docs`,
-        status: "missing",
-        detail: `Invalid docs: AGENTS.md → CLAUDE.md (invert symlink direction)`,
-      });
-    } else if (repo.agentsIsSymlink && !repo.claudeIsSymlink) {
-      checks.push({
-        name: `repo-${repo.name}-docs`,
-        status: "warn",
-        detail: `AGENTS.md is a symlink; expected AGENTS.md to be the real file`,
-      });
-    } else if (!repo.claudeIsSymlink) {
-      checks.push({
-        name: `repo-${repo.name}-docs`,
-        status: "warn",
-        detail: `CLAUDE.md is not a symlink; expected CLAUDE.md → AGENTS.md`,
-      });
-    } else {
-      checks.push({
-        name: `repo-${repo.name}-docs`,
-        status: "warn",
-        detail: `Docs present but symlink is unexpected (CLAUDE.md → ${repo.claudeSymlinkTarget ?? "?"})`,
-      });
-    }
-  } else if (repo.hasAgentsMd || repo.hasClaudeMd) {
-    checks.push({
-      name: `repo-${repo.name}-docs`,
-      status: "warn",
-      detail: repo.hasAgentsMd
-        ? `AGENTS.md exists (missing CLAUDE.md symlink)`
-        : `CLAUDE.md exists (missing AGENTS.md canonical file)`,
-    });
-  } else {
-    checks.push({
-      name: `repo-${repo.name}-docs`,
-      status: "warn",
-      detail: `No AGENTS.md or CLAUDE.md`,
-    });
-  }
-
-  return checks;
-}
-
-// ============================================================================
-// Main repo checking function
-// ============================================================================
-
-export function checkRepos(basePath?: string): RepoCheckResult[] {
-  const reposPath = basePath || DEFAULT_REPOS_PATH;
-  const repoPaths = discoverRepos(reposPath);
-  const results: RepoCheckResult[] = [];
-
-  for (const repoPath of repoPaths) {
-    const repo = analyzeRepo(repoPath);
-    const checks = checkRepo(repo);
-    results.push({
-      repo: repo.name,
-      checks,
-    });
-  }
-
-  return results;
-}
-
-// ============================================================================
-// Summary helpers
-// ============================================================================
-
-export function summarizeRepoChecks(results: RepoCheckResult[]): {
-  total: number;
-  ok: number;
-  warn: number;
-  missing: number;
-  repos: {
-    withConfig: number;
-    withAgentsConfig: number;
-    withLoopJson: number;
-    noConfig: number;
-  };
-} {
-  const allChecks = results.flatMap((r) => r.checks);
-
-  let withAgentsConfig = 0;
-  let withLoopJson = 0;
-  let noConfig = 0;
-
-  for (const result of results) {
-    const configCheck = result.checks.find((c) => c.name.endsWith("-config"));
-    // Check for .loop.json first since the detail might contain both strings
-    // e.g., ".loop.json exists (migrate to .abbie/config.json)"
-    if (configCheck?.detail.includes(".loop.json exists")) {
-      withLoopJson++;
-    } else if (configCheck?.detail.includes(".abbie/config.json exists")) {
-      withAgentsConfig++;
-    } else {
-      noConfig++;
-    }
-  }
-
-  return {
-    total: allChecks.length,
-    ok: allChecks.filter((c) => c.status === "ok").length,
-    warn: allChecks.filter((c) => c.status === "warn").length,
-    missing: allChecks.filter((c) => c.status === "missing").length,
-    repos: {
-      withConfig: withAgentsConfig + withLoopJson,
-      withAgentsConfig,
-      withLoopJson,
-      noConfig,
-    },
-  };
-}
-
-// ============================================================================
-// Exports
-// ============================================================================
-
-export { DEFAULT_REPOS_PATH };

package/src/lib/doctor/templates.ts

@@ -1,191 +0,0 @@
-/**
- * Template validation for agents doctor
- *
- * Checks:
- * - ~/.abbie/templates/ directory exists
- * - Required templates exist (convex-loop.json, library-loop.json)
- * - Template JSON is valid
- * - Schema file exists
- */
-
-import { existsSync, readdirSync, readFileSync } from "node:fs";
-import { homedir } from "node:os";
-import { join } from "node:path";
-import type { Check } from "./index.js";
-
-// ============================================================================
-// Constants
-// ============================================================================
-
-const REQUIRED_TEMPLATES = ["loop/convex.json", "loop/library.json"] as const;
-
-function getTemplatesRoot(): string {
-  return join(homedir(), ".abbie", "templates");
-}
-
-function getLoopTemplatesRoot(): string {
-  return join(getTemplatesRoot(), "loop");
-}
-
-function getSchemasRoot(): string {
-  return join(getTemplatesRoot(), "schemas");
-}
-
-// ============================================================================
-// Template checks
-// ============================================================================
-
-export function checkTemplates(): Check[] {
-  const checks: Check[] = [];
-  const templatesRoot = getTemplatesRoot();
-  const schemasRoot = getSchemasRoot();
-
-  // Check templates directory exists
-  if (existsSync(templatesRoot)) {
-    checks.push({
-      name: "templates-root",
-      status: "ok",
-      detail: "~/.abbie/templates exists",
-    });
-  } else {
-    checks.push({
-      name: "templates-root",
-      status: "missing",
-      detail: "~/.abbie/templates not found",
-    });
-    // If templates root doesn't exist, skip other checks
-    return checks;
-  }
-
-  // Check schemas directory exists
-  if (existsSync(schemasRoot)) {
-    checks.push({
-      name: "templates-schemas",
-      status: "ok",
-      detail: "~/.abbie/templates/schemas exists",
-    });
-  } else {
-    checks.push({
-      name: "templates-schemas",
-      status: "warn",
-      detail: "~/.abbie/templates/schemas not found",
-    });
-  }
-
-  // Check schema file exists
-  const schemaPath = join(schemasRoot, "config.schema.json");
-  if (existsSync(schemaPath)) {
-    try {
-      JSON.parse(readFileSync(schemaPath, "utf-8"));
-      checks.push({
-        name: "templates-schema-valid",
-        status: "ok",
-        detail: "config.schema.json is valid JSON",
-      });
-    } catch (e) {
-      checks.push({
-        name: "templates-schema-valid",
-        status: "missing",
-        detail: `config.schema.json is invalid: ${e instanceof Error ? e.message : "parse error"}`,
-      });
-    }
-  } else {
-    checks.push({
-      name: "templates-schema-valid",
-      status: "warn",
-      detail: "config.schema.json not found",
-    });
-  }
-
-  // Check loop templates directory exists
-  const loopTemplatesRoot = getLoopTemplatesRoot();
-  if (existsSync(loopTemplatesRoot)) {
-    checks.push({
-      name: "templates-loop",
-      status: "ok",
-      detail: "~/.abbie/templates/loop exists",
-    });
-  } else {
-    checks.push({
-      name: "templates-loop",
-      status: "warn",
-      detail: "~/.abbie/templates/loop not found",
-    });
-  }
-
-  // Check required templates exist and are valid
-  for (const template of REQUIRED_TEMPLATES) {
-    const templatePath = join(templatesRoot, template);
-    const templateName = template.replace("loop/", "");
-    if (existsSync(templatePath)) {
-      try {
-        const content = JSON.parse(readFileSync(templatePath, "utf-8"));
-        // Basic validation - check for required fields
-        if (content.project || content.$id) {
-          checks.push({
-            name: `template-${templateName}`,
-            status: "ok",
-            detail: `loop/${templateName} is valid`,
-          });
-        } else {
-          checks.push({
-            name: `template-${templateName}`,
-            status: "warn",
-            detail: `loop/${templateName} missing required fields`,
-          });
-        }
-      } catch (e) {
-        checks.push({
-          name: `template-${templateName}`,
-          status: "missing",
-          detail: `loop/${templateName} is invalid JSON: ${e instanceof Error ? e.message : "parse error"}`,
-        });
-      }
-    } else {
-      checks.push({
-        name: `template-${templateName}`,
-        status: "warn",
-        detail: `loop/${templateName} not found`,
-      });
-    }
-  }
-
-  // List additional loop templates (informational)
-  try {
-    if (existsSync(loopTemplatesRoot)) {
-      const allFiles = readdirSync(loopTemplatesRoot).filter((f) => f.endsWith(".json"));
-      const requiredNames = REQUIRED_TEMPLATES.map((t) => t.replace("loop/", ""));
-      const extraTemplates = allFiles.filter((f) => !requiredNames.includes(f));
-      if (extraTemplates.length > 0) {
-        checks.push({
-          name: "templates-extra",
-          status: "ok",
-          detail: `Additional loop templates: ${extraTemplates.join(", ")}`,
-        });
-      }
-    }
-  } catch {
-    // Ignore errors listing directory
-  }
-
-  return checks;
-}
-
-// ============================================================================
-// Resolve extends reference
-// ============================================================================
-
-export function resolveExtends(extendsPath: string): { exists: boolean; path: string } {
-  // Replace ~ with home directory
-  const resolvedPath = extendsPath.replace(/^~/, homedir());
-  return {
-    exists: existsSync(resolvedPath),
-    path: resolvedPath,
-  };
-}
-
-// ============================================================================
-// Exports
-// ============================================================================
-
-export { REQUIRED_TEMPLATES, getTemplatesRoot, getSchemasRoot };

package/src/lib/errors.ts

@@ -1,111 +0,0 @@
-/**
- * Base error class for abbie CLI operations.
- * Includes a code for programmatic handling and optional context.
- */
-export class AgentsError extends Error {
-  constructor(
-    message: string,
-    public code: string,
-    public context?: Record<string, unknown>,
-  ) {
-    super(message);
-    this.name = this.constructor.name;
-    Error.captureStackTrace?.(this, this.constructor);
-  }
-}
-
-export type JsonError = {
-  error: true;
-  code: number;
-  type: string;
-  message: string;
-  context?: Record<string, unknown>;
-};
-
-/**
- * Error thrown when a session already exists.
- */
-export class SessionExistsError extends AgentsError {
-  constructor(path: string) {
-    super(`Session already exists: ${path}`, "SESSION_EXISTS", { path });
-  }
-}
-
-/**
- * Error thrown when a session is not found.
- */
-export class SessionNotFoundError extends AgentsError {
-  constructor(project: string, slug: string) {
-    super(`Session not found: ${project}/${slug}`, "SESSION_NOT_FOUND", { project, slug });
-  }
-}
-
-/**
- * Error thrown for invalid project names.
- */
-export class InvalidProjectError extends AgentsError {
-  constructor(project: string) {
-    super(`Invalid project name: ${project}`, "INVALID_PROJECT", { project });
-  }
-}
-
-/**
- * Error thrown for invalid session data.
- */
-export class InvalidSessionError extends AgentsError {
-  constructor(message: string, context?: Record<string, unknown>) {
-    super(message, "INVALID_SESSION", context);
-  }
-}
-
-/**
- * Map errors to exit codes for consistent CLI behavior.
- */
-export function errorToExitCode(err: unknown): number {
-  if (err instanceof SessionExistsError) return 2;
-  if (err instanceof SessionNotFoundError) return 3;
-  if (err instanceof InvalidProjectError) return 4;
-  if (err instanceof InvalidSessionError) return 5;
-  if (err instanceof AgentsError) return 6; // Generic AgentsError
-  return 1; // Generic error
-}
-
-export function toJsonError(err: unknown, exitCode: number): JsonError {
-  let message = "An unknown error occurred";
-  let type = "Error";
-  let context: Record<string, unknown> | undefined;
-
-  if (err instanceof Error) {
-    message = err.message || message;
-    type = err.name || type;
-  }
-
-  if (typeof err === "string" || typeof err === "number") {
-    message = String(err);
-  }
-
-  if (err && typeof err === "object") {
-    const record = err as Record<string, unknown>;
-    const recordMessage = record.message;
-    if (typeof recordMessage === "string" && recordMessage.length > 0) {
-      message = recordMessage;
-    }
-    const recordType = record.type;
-    if (typeof recordType === "string" && recordType.length > 0) {
-      type = recordType;
-    } else {
-      const recordCode = record.code;
-      if (typeof recordCode === "string" && recordCode.length > 0) {
-        type = recordCode;
-      }
-    }
-    const recordContext = record.context;
-    if (recordContext && typeof recordContext === "object" && !Array.isArray(recordContext)) {
-      context = recordContext as Record<string, unknown>;
-    }
-  }
-
-  return context
-    ? { error: true, code: exitCode, type, message, context }
-    : { error: true, code: exitCode, type, message };
-}