@rankcli/agent-runtime 0.0.14 → 0.0.16

package/src/git/pr-helper.test.ts

@@ -1,284 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import {
-  formatPRTitle,
-  formatPRBody,
-  generatePRDescription,
-  createPullRequest,
-  type PRConfig,
-  type SEOFixSummary,
-  type PRResult,
-} from './pr-helper.js';
-
-describe('pr-helper', () => {
-  describe('formatPRTitle', () => {
-    it('formats title for single category fix', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING'], filesCount: 2 },
-      ];
-      expect(formatPRTitle(fixes)).toBe('fix(meta): SEO improvements - 1 issue fixed');
-    });
-
-    it('formats title for multiple issues in same category', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING', 'META_DESC_MISSING'], filesCount: 2 },
-      ];
-      expect(formatPRTitle(fixes)).toBe('fix(meta): SEO improvements - 2 issues fixed');
-    });
-
-    it('formats title for multiple categories', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING'], filesCount: 1 },
-        { category: 'social-meta', issues: ['OG_TITLE_MISSING', 'OG_IMAGE_MISSING'], filesCount: 1 },
-      ];
-      expect(formatPRTitle(fixes)).toBe('fix(seo): SEO improvements - 3 issues fixed');
-    });
-
-    it('handles empty fixes array', () => {
-      expect(formatPRTitle([])).toBe('fix(seo): SEO audit fixes');
-    });
-
-    it('uses feat for new features like schema', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'structured-data', issues: ['SCHEMA_MISSING'], filesCount: 1 },
-      ];
-      expect(formatPRTitle(fixes)).toBe('feat(schema): SEO improvements - 1 issue fixed');
-    });
-  });
-
-  describe('formatPRBody', () => {
-    it('generates body with summary section', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING'], filesCount: 2 },
-      ];
-      const body = formatPRBody(fixes);
-
-      expect(body).toContain('## Summary');
-      expect(body).toContain('1 SEO issue');
-      expect(body).toContain('2 files');
-    });
-
-    it('includes issue codes in body', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING', 'META_DESC_MISSING'], filesCount: 2 },
-      ];
-      const body = formatPRBody(fixes);
-
-      expect(body).toContain('TITLE_MISSING');
-      expect(body).toContain('META_DESC_MISSING');
-    });
-
-    it('includes category breakdown', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING'], filesCount: 1 },
-        { category: 'social-meta', issues: ['OG_TITLE_MISSING'], filesCount: 1 },
-      ];
-      const body = formatPRBody(fixes);
-
-      expect(body).toContain('On-Page');
-      expect(body).toContain('Social Meta');
-    });
-
-    it('includes test plan section', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING'], filesCount: 1 },
-      ];
-      const body = formatPRBody(fixes);
-
-      expect(body).toContain('## Test plan');
-      expect(body).toContain('- [ ]');
-    });
-
-    it('includes generated by footer', () => {
-      const fixes: SEOFixSummary[] = [];
-      const body = formatPRBody(fixes);
-
-      expect(body).toContain('Generated with');
-      expect(body).toContain('SEO Autopilot');
-    });
-
-    it('includes score improvement if provided', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING'], filesCount: 1 },
-      ];
-      const body = formatPRBody(fixes, { scoreBefore: 65, scoreAfter: 82 });
-
-      expect(body).toContain('65');
-      expect(body).toContain('82');
-      expect(body).toContain('+17');
-    });
-  });
-
-  describe('generatePRDescription', () => {
-    it('generates full PR description with title and body', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING'], filesCount: 1 },
-      ];
-      const description = generatePRDescription(fixes);
-
-      expect(description.title).toBeDefined();
-      expect(description.body).toBeDefined();
-      expect(description.title).toContain('fix');
-      expect(description.body).toContain('## Summary');
-    });
-
-    it('includes labels suggestion', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING'], filesCount: 1 },
-      ];
-      const description = generatePRDescription(fixes);
-
-      expect(description.labels).toContain('seo');
-      expect(description.labels).toContain('automated');
-    });
-
-    it('suggests reviewers if configured', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'on-page', issues: ['TITLE_MISSING'], filesCount: 1 },
-      ];
-      const config: PRConfig = {
-        reviewers: ['alice', 'bob'],
-      };
-      const description = generatePRDescription(fixes, config);
-
-      expect(description.reviewers).toEqual(['alice', 'bob']);
-    });
-  });
-
-  describe('createPullRequest', () => {
-    beforeEach(() => {
-      vi.resetAllMocks();
-    });
-
-    it('returns PR URL on success', async () => {
-      const mockExec = vi.fn().mockResolvedValue({
-        stdout: 'https://github.com/owner/repo/pull/123\n',
-        stderr: '',
-      });
-
-      const result = await createPullRequest({
-        title: 'Test PR',
-        body: 'Test body',
-        baseBranch: 'main',
-        headBranch: 'seo-fixes',
-      }, { execAsync: mockExec });
-
-      expect(result.success).toBe(true);
-      expect(result.url).toBe('https://github.com/owner/repo/pull/123');
-      expect(result.number).toBe(123);
-    });
-
-    it('handles gh cli errors', async () => {
-      const mockExec = vi.fn().mockRejectedValue(new Error('gh: not logged in'));
-
-      const result = await createPullRequest({
-        title: 'Test PR',
-        body: 'Test body',
-        baseBranch: 'main',
-        headBranch: 'seo-fixes',
-      }, { execAsync: mockExec });
-
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('not logged in');
-    });
-
-    it('creates branch if needed', async () => {
-      const commands: string[] = [];
-      const mockExec = vi.fn().mockImplementation((cmd: string) => {
-        commands.push(cmd);
-        if (cmd.includes('gh pr create')) {
-          return Promise.resolve({ stdout: 'https://github.com/owner/repo/pull/456\n', stderr: '' });
-        }
-        return Promise.resolve({ stdout: '', stderr: '' });
-      });
-
-      await createPullRequest({
-        title: 'Test PR',
-        body: 'Test body',
-        baseBranch: 'main',
-        headBranch: 'seo-fixes',
-        createBranch: true,
-      }, { execAsync: mockExec });
-
-      expect(commands.some(c => c.includes('checkout -b'))).toBe(true);
-    });
-
-    it('supports dry run mode', async () => {
-      const mockExec = vi.fn();
-
-      const result = await createPullRequest({
-        title: 'Test PR',
-        body: 'Test body',
-        baseBranch: 'main',
-        headBranch: 'seo-fixes',
-        dryRun: true,
-      }, { execAsync: mockExec });
-
-      expect(result.success).toBe(true);
-      expect(result.dryRun).toBe(true);
-      expect(mockExec).not.toHaveBeenCalledWith(expect.stringContaining('gh pr create'));
-    });
-
-    it('includes labels in gh command', async () => {
-      let capturedCommand = '';
-      const mockExec = vi.fn().mockImplementation((cmd: string) => {
-        capturedCommand = cmd;
-        return Promise.resolve({ stdout: 'https://github.com/owner/repo/pull/789\n', stderr: '' });
-      });
-
-      await createPullRequest({
-        title: 'Test PR',
-        body: 'Test body',
-        baseBranch: 'main',
-        headBranch: 'seo-fixes',
-        labels: ['seo', 'automated'],
-      }, { execAsync: mockExec });
-
-      expect(capturedCommand).toContain('--label');
-      expect(capturedCommand).toContain('seo');
-    });
-  });
-
-  describe('integration scenarios', () => {
-    it('generates complete PR for typical SEO audit', () => {
-      const fixes: SEOFixSummary[] = [
-        {
-          category: 'on-page',
-          issues: ['TITLE_MISSING', 'META_DESC_MISSING', 'H1_MISSING'],
-          filesCount: 5
-        },
-        {
-          category: 'social-meta',
-          issues: ['OG_TITLE_MISSING', 'OG_IMAGE_MISSING', 'TWITTER_CARD_MISSING'],
-          filesCount: 5
-        },
-        {
-          category: 'crawlability',
-          issues: ['ROBOTS_TXT_MISSING', 'SITEMAP_MISSING'],
-          filesCount: 2
-        },
-      ];
-
-      const description = generatePRDescription(fixes, {
-        scoreBefore: 45,
-        scoreAfter: 78,
-      });
-
-      expect(description.title).toContain('8 issues fixed');
-      expect(description.body).toContain('+33');
-      expect(description.body).toContain('ROBOTS_TXT_MISSING');
-      expect(description.body).toContain('OG_TITLE_MISSING');
-      expect(description.labels).toContain('seo');
-    });
-
-    it('handles single-file schema addition', () => {
-      const fixes: SEOFixSummary[] = [
-        { category: 'structured-data', issues: ['SCHEMA_MISSING'], filesCount: 1 },
-      ];
-
-      const description = generatePRDescription(fixes);
-
-      expect(description.title).toContain('feat(schema)');
-      expect(description.body).toContain('SCHEMA_MISSING');
-    });
-  });
-});

package/src/git/pr-helper.ts

@@ -1,307 +0,0 @@
-/**
- * PR Helper
- *
- * Creates pull requests for SEO fixes using GitHub CLI.
- * Generates formatted PR titles, bodies, and handles branch management.
- */
-
-import { exec } from 'child_process';
-import { promisify } from 'util';
-
-const execAsync = promisify(exec);
-
-export interface SEOFixSummary {
-  category: string;
-  issues: string[];
-  filesCount: number;
-}
-
-export interface PRConfig {
-  scoreBefore?: number;
-  scoreAfter?: number;
-  reviewers?: string[];
-  assignees?: string[];
-  draft?: boolean;
-}
-
-export interface PRDescription {
-  title: string;
-  body: string;
-  labels: string[];
-  reviewers?: string[];
-  assignees?: string[];
-}
-
-export interface CreatePROptions {
-  title: string;
-  body: string;
-  baseBranch: string;
-  headBranch: string;
-  createBranch?: boolean;
-  labels?: string[];
-  reviewers?: string[];
-  assignees?: string[];
-  draft?: boolean;
-  dryRun?: boolean;
-}
-
-export interface PRResult {
-  success: boolean;
-  url?: string;
-  number?: number;
-  error?: string;
-  dryRun?: boolean;
-}
-
-export interface ExecOptions {
-  execAsync?: typeof execAsync;
-}
-
-/**
- * Maps categories to commit types and scopes
- */
-const CATEGORY_MAPPING: Record<string, { type: string; scope: string; displayName: string }> = {
-  'crawlability': { type: 'fix', scope: 'seo', displayName: 'Crawlability' },
-  'indexability': { type: 'fix', scope: 'seo', displayName: 'Indexability' },
-  'on-page': { type: 'fix', scope: 'meta', displayName: 'On-Page' },
-  'content': { type: 'fix', scope: 'content', displayName: 'Content' },
-  'links': { type: 'fix', scope: 'links', displayName: 'Links' },
-  'images': { type: 'fix', scope: 'images', displayName: 'Images' },
-  'structured-data': { type: 'feat', scope: 'schema', displayName: 'Structured Data' },
-  'performance': { type: 'perf', scope: 'web', displayName: 'Performance' },
-  'security': { type: 'fix', scope: 'security', displayName: 'Security' },
-  'mobile': { type: 'fix', scope: 'mobile', displayName: 'Mobile' },
-  'international': { type: 'fix', scope: 'i18n', displayName: 'International' },
-  'social-meta': { type: 'feat', scope: 'og', displayName: 'Social Meta' },
-};
-
-/**
- * Formats a PR title based on the fixes
- */
-export function formatPRTitle(fixes: SEOFixSummary[]): string {
-  if (fixes.length === 0) {
-    return 'fix(seo): SEO audit fixes';
-  }
-
-  const totalIssues = fixes.reduce((sum, f) => sum + f.issues.length, 0);
-  const categories = [...new Set(fixes.map(f => f.category))];
-
-  // Determine type and scope
-  let type = 'fix';
-  let scope = 'seo';
-
-  if (categories.length === 1) {
-    const mapping = CATEGORY_MAPPING[categories[0]];
-    if (mapping) {
-      type = mapping.type;
-      scope = mapping.scope;
-    }
-  }
-
-  const issueText = totalIssues === 1 ? '1 issue fixed' : `${totalIssues} issues fixed`;
-  return `${type}(${scope}): SEO improvements - ${issueText}`;
-}
-
-/**
- * Formats the PR body with detailed information
- */
-export function formatPRBody(
-  fixes: SEOFixSummary[],
-  options?: { scoreBefore?: number; scoreAfter?: number }
-): string {
-  const totalIssues = fixes.reduce((sum, f) => sum + f.issues.length, 0);
-  const totalFiles = fixes.reduce((sum, f) => sum + f.filesCount, 0);
-  const uniqueFiles = new Set(fixes.flatMap(f => Array(f.filesCount).fill(0))).size || totalFiles;
-
-  let body = '## Summary\n';
-  body += `This PR fixes ${totalIssues} SEO issue${totalIssues === 1 ? '' : 's'} across ${totalFiles} file${totalFiles === 1 ? '' : 's'}.\n\n`;
-
-  // Score improvement if provided
-  if (options?.scoreBefore !== undefined && options?.scoreAfter !== undefined) {
-    const improvement = options.scoreAfter - options.scoreBefore;
-    const sign = improvement >= 0 ? '+' : '';
-    body += `### Score Improvement\n`;
-    body += `- Before: ${options.scoreBefore}/100\n`;
-    body += `- After: ${options.scoreAfter}/100\n`;
-    body += `- Change: **${sign}${improvement}** points\n\n`;
-  }
-
-  // Category breakdown
-  if (fixes.length > 0) {
-    body += '### Issues Fixed\n\n';
-
-    for (const fix of fixes) {
-      const mapping = CATEGORY_MAPPING[fix.category] || { displayName: fix.category };
-      body += `**${mapping.displayName}**\n`;
-      for (const issue of fix.issues) {
-        body += `- \`${issue}\`\n`;
-      }
-      body += '\n';
-    }
-  }
-
-  // Test plan
-  body += '## Test plan\n';
-  body += '- [ ] Run SEO audit to verify fixes\n';
-  body += '- [ ] Check pages render correctly\n';
-  body += '- [ ] Validate structured data (if applicable)\n';
-  body += '- [ ] Test social share previews (if OG tags changed)\n';
-  body += '\n';
-
-  // Footer
-  body += '---\n';
-  body += '🤖 Generated with [SEO Autopilot](https://github.com/seo-autopilot/seo-autopilot)\n';
-
-  return body;
-}
-
-/**
- * Generates complete PR description including title, body, and labels
- */
-export function generatePRDescription(
-  fixes: SEOFixSummary[],
-  config?: PRConfig
-): PRDescription {
-  const title = formatPRTitle(fixes);
-  const body = formatPRBody(fixes, {
-    scoreBefore: config?.scoreBefore,
-    scoreAfter: config?.scoreAfter,
-  });
-
-  const labels = ['seo', 'automated'];
-
-  // Add category-specific labels
-  const categories = [...new Set(fixes.map(f => f.category))];
-  for (const category of categories) {
-    const mapping = CATEGORY_MAPPING[category];
-    if (mapping && !labels.includes(mapping.scope)) {
-      labels.push(mapping.scope);
-    }
-  }
-
-  return {
-    title,
-    body,
-    labels,
-    reviewers: config?.reviewers,
-    assignees: config?.assignees,
-  };
-}
-
-/**
- * Creates a pull request using GitHub CLI
- */
-export async function createPullRequest(
-  options: CreatePROptions,
-  deps?: ExecOptions
-): Promise<PRResult> {
-  const run = deps?.execAsync || execAsync;
-
-  // Dry run mode
-  if (options.dryRun) {
-    console.log('DRY RUN - Would create PR:');
-    console.log(`Title: ${options.title}`);
-    console.log(`Base: ${options.baseBranch}`);
-    console.log(`Head: ${options.headBranch}`);
-    console.log(`Labels: ${options.labels?.join(', ') || 'none'}`);
-    console.log('---');
-    console.log(options.body);
-    console.log('---');
-    return { success: true, dryRun: true };
-  }
-
-  try {
-    // Create branch if needed
-    if (options.createBranch) {
-      await run(`git checkout -b ${options.headBranch}`);
-    }
-
-    // Build gh pr create command
-    let cmd = `gh pr create --title "${options.title.replace(/"/g, '\\"')}"`;
-    cmd += ` --base "${options.baseBranch}"`;
-    cmd += ` --head "${options.headBranch}"`;
-
-    // Add labels
-    if (options.labels && options.labels.length > 0) {
-      cmd += ` --label "${options.labels.join(',')}"`;
-    }
-
-    // Add reviewers
-    if (options.reviewers && options.reviewers.length > 0) {
-      cmd += ` --reviewer "${options.reviewers.join(',')}"`;
-    }
-
-    // Add assignees
-    if (options.assignees && options.assignees.length > 0) {
-      cmd += ` --assignee "${options.assignees.join(',')}"`;
-    }
-
-    // Draft PR
-    if (options.draft) {
-      cmd += ' --draft';
-    }
-
-    // Add body using heredoc
-    cmd += ` --body "$(cat <<'EOF'\n${options.body}\nEOF\n)"`;
-
-    const { stdout } = await run(cmd);
-    const url = stdout.trim();
-
-    // Extract PR number from URL
-    const numberMatch = url.match(/\/pull\/(\d+)/);
-    const number = numberMatch ? parseInt(numberMatch[1], 10) : undefined;
-
-    return {
-      success: true,
-      url,
-      number,
-    };
-  } catch (error) {
-    return {
-      success: false,
-      error: (error as Error).message,
-    };
-  }
-}
-
-/**
- * Generates a branch name for SEO fixes
- */
-export function generateBranchName(prefix = 'seo-fixes'): string {
-  const timestamp = new Date().toISOString().slice(0, 10).replace(/-/g, '');
-  const random = Math.random().toString(36).slice(2, 6);
-  return `${prefix}-${timestamp}-${random}`;
-}
-
-/**
- * Checks if gh CLI is available and authenticated
- */
-export async function checkGitHubCLI(deps?: ExecOptions): Promise<{
-  available: boolean;
-  authenticated: boolean;
-  error?: string;
-}> {
-  const run = deps?.execAsync || execAsync;
-
-  try {
-    // Check if gh is installed
-    await run('gh --version');
-
-    // Check if authenticated
-    await run('gh auth status');
-
-    return { available: true, authenticated: true };
-  } catch (error) {
-    const message = (error as Error).message;
-
-    if (message.includes('command not found') || message.includes('not recognized')) {
-      return { available: false, authenticated: false, error: 'GitHub CLI (gh) is not installed' };
-    }
-
-    if (message.includes('not logged') || message.includes('auth')) {
-      return { available: true, authenticated: false, error: 'GitHub CLI is not authenticated' };
-    }
-
-    return { available: false, authenticated: false, error: message };
-  }
-}

package/src/index.ts

@@ -1,76 +0,0 @@
-// Agent runtime for SEO Autopilot
-
-// Types
-export * from './types.js';
-
-// Comprehensive Audit Module (Ahrefs-level checks)
-export * from './audit/index.js';
-
-// Keyword Research Module
-export * from './keywords/index.js';
-
-// Content Analysis & Optimization Module
-export * from './content/index.js';
-
-// Tracking & Monitoring Module
-export * from './tracking/index.js';
-
-// Loader
-export { loadAgent, loadAgentByName, interpolatePrompt } from './loader.js';
-
-// Executor
-export { executeAgent, runDirectAnalysis } from './executor.js';
-export type { ExecuteOptions, ExecutionResult } from './executor.js';
-
-// Fix generator
-export { generateFixes, applyFixes } from './fixer.js';
-export type { FixGeneratorOptions, GeneratedFix } from './fixer.js';
-export * from './fixer/framework-fixes.js';
-// Schemas are exported from ./fixer/schemas.js - import directly to avoid conflicts
-// with generateFAQJsonLd and other duplicates in framework-fixes
-export { Schemas } from './fixer/schemas.js';
-
-// Tools
-export { tools } from './tools/index.js';
-export {
-  crawlUrl,
-  extractMeta,
-  analyzeHeadings,
-  extractImages,
-  extractLinks,
-  extractSchema,
-  checkRobots,
-  checkSitemap,
-  readFile,
-  writeFile,
-  listFiles,
-  detectFramework,
-  findHtmlEntry,
-  findPageFiles,
-  analyzeUrl,
-} from './tools/index.js';
-
-// Git integration (conventional commits, co-authoring)
-export * from './git/index.js';
-
-// Scheduler (scheduled audits, alerts)
-export * from './scheduler/index.js';
-
-// Reports (HTML/PDF generation, white-label)
-export * from './reports/index.js';
-
-// GEO (Generative Engine Optimization) tracking
-export * from './geo/index.js';
-
-// Keyword Rank Tracking
-export * from './ranking/index.js';
-
-// Advanced Analyzers (GEO, CWV, Security, Schema, Images, Links, Mobile)
-export * as analyzers from './analyzers/index.js';
-
-// Framework Detection & SEO Recipes
-// Note: Export as namespace to avoid conflict with tools/detectFramework
-export * as frameworks from './frameworks/index.js';
-
-// WASM Adapter (sync audit for embedded execution)
-export { runSyncAudit } from './wasm-adapter.js';