@rankcli/agent-runtime 0.0.14 → 0.0.16

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

@@ -1,261 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import {
-  formatConventionalCommit,
-  formatSEOCommitMessage,
-  isGitRepo,
-  detectGitHubPages,
-  generateCommitSummary,
-  COMMIT_TYPES,
-  SEO_SCOPES,
-  type ConventionalCommit,
-  type SEOFixCommit,
-  type CommitConfig,
-} from './commit-helper.js';
-
-describe('commit-helper', () => {
-  describe('formatConventionalCommit', () => {
-    it('formats a basic commit message', () => {
-      const commit: ConventionalCommit = {
-        type: 'fix',
-        description: 'resolve meta tag issue',
-      };
-      expect(formatConventionalCommit(commit)).toBe('fix: resolve meta tag issue');
-    });
-
-    it('includes scope when provided', () => {
-      const commit: ConventionalCommit = {
-        type: 'fix',
-        scope: 'seo',
-        description: 'add missing title tag',
-      };
-      expect(formatConventionalCommit(commit)).toBe('fix(seo): add missing title tag');
-    });
-
-    it('adds breaking change indicator', () => {
-      const commit: ConventionalCommit = {
-        type: 'feat',
-        scope: 'schema',
-        description: 'change JSON-LD format',
-        breaking: true,
-      };
-      expect(formatConventionalCommit(commit)).toBe('feat(schema)!: change JSON-LD format');
-    });
-
-    it('includes body when provided', () => {
-      const commit: ConventionalCommit = {
-        type: 'fix',
-        scope: 'meta',
-        description: 'add meta description',
-        body: 'Added meta descriptions to all pages.\nThis improves SEO score.',
-      };
-      const result = formatConventionalCommit(commit);
-      expect(result).toContain('fix(meta): add meta description');
-      expect(result).toContain('Added meta descriptions to all pages.');
-      expect(result).toContain('This improves SEO score.');
-    });
-
-    it('includes issue codes when provided', () => {
-      const commit: ConventionalCommit = {
-        type: 'fix',
-        scope: 'seo',
-        description: 'fix crawlability issues',
-        issues: ['ROBOTS_TXT_MISSING', 'SITEMAP_MISSING'],
-      };
-      const result = formatConventionalCommit(commit);
-      expect(result).toContain('Issue codes: ROBOTS_TXT_MISSING, SITEMAP_MISSING');
-    });
-  });
-
-  describe('formatSEOCommitMessage', () => {
-    it('formats SEO fix with co-author', () => {
-      const fix: SEOFixCommit = {
-        category: 'on-page',
-        issues: ['TITLE_MISSING', 'META_DESC_MISSING'],
-        filesChanged: ['index.html'],
-        description: 'add on page improvements',
-      };
-      const config: CommitConfig = {
-        systemName: 'SEO Autopilot',
-        systemEmail: 'bot@seo-autopilot.dev',
-      };
-
-      const result = formatSEOCommitMessage(fix, config);
-
-      expect(result).toContain('fix(meta): add on page improvements');
-      expect(result).toContain('Files changed:');
-      expect(result).toContain('- index.html');
-      expect(result).toContain('Issue codes: TITLE_MISSING, META_DESC_MISSING');
-      expect(result).toContain('Co-Authored-By: SEO Autopilot <bot@seo-autopilot.dev>');
-    });
-
-    it('maps categories to correct commit types', () => {
-      const testCases = [
-        { category: 'crawlability', expectedType: 'fix', expectedScope: 'seo' },
-        { category: 'on-page', expectedType: 'fix', expectedScope: 'meta' },
-        { category: 'structured-data', expectedType: 'feat', expectedScope: 'schema' },
-        { category: 'performance', expectedType: 'perf', expectedScope: 'web' },
-        { category: 'social-meta', expectedType: 'feat', expectedScope: 'og' },
-        { category: 'images', expectedType: 'fix', expectedScope: 'images' },
-        { category: 'links', expectedType: 'fix', expectedScope: 'links' },
-        { category: 'security', expectedType: 'fix', expectedScope: 'security' },
-      ];
-
-      testCases.forEach(({ category, expectedType, expectedScope }) => {
-        const fix: SEOFixCommit = {
-          category,
-          issues: ['TEST_ISSUE'],
-          filesChanged: ['test.html'],
-          description: 'test fix',
-        };
-        const result = formatSEOCommitMessage(fix);
-        expect(result).toContain(`${expectedType}(${expectedScope}): test fix`);
-      });
-    });
-
-    it('uses default scope for unknown categories', () => {
-      const fix: SEOFixCommit = {
-        category: 'unknown-category',
-        issues: ['TEST_ISSUE'],
-        filesChanged: ['test.html'],
-        description: 'test fix',
-      };
-      const result = formatSEOCommitMessage(fix);
-      expect(result).toContain('fix(seo): test fix');
-    });
-
-    it('handles multiple files', () => {
-      const fix: SEOFixCommit = {
-        category: 'on-page',
-        issues: ['TEST'],
-        filesChanged: ['index.html', 'about.html', 'contact.html'],
-        description: 'fix meta tags',
-      };
-      const result = formatSEOCommitMessage(fix);
-      expect(result).toContain('- index.html');
-      expect(result).toContain('- about.html');
-      expect(result).toContain('- contact.html');
-    });
-
-    it('uses custom system name and email', () => {
-      const fix: SEOFixCommit = {
-        category: 'on-page',
-        issues: ['TEST'],
-        filesChanged: ['index.html'],
-        description: 'test',
-      };
-      const config: CommitConfig = {
-        systemName: 'Custom Bot',
-        systemEmail: 'custom@example.com',
-      };
-      const result = formatSEOCommitMessage(fix, config);
-      expect(result).toContain('Co-Authored-By: Custom Bot <custom@example.com>');
-    });
-  });
-
-  describe('generateCommitSummary', () => {
-    it('generates summary for successful commits', () => {
-      const commits = [
-        {
-          fix: {
-            category: 'on-page',
-            issues: ['TITLE_MISSING'],
-            filesChanged: ['index.html'],
-            description: 'add title tags',
-          },
-          result: { success: true, hash: 'abc1234' },
-        },
-        {
-          fix: {
-            category: 'social-meta',
-            issues: ['OG_TITLE_MISSING'],
-            filesChanged: ['index.html'],
-            description: 'add og tags',
-          },
-          result: { success: true, hash: 'def5678' },
-        },
-      ];
-
-      const summary = generateCommitSummary(commits);
-
-      expect(summary).toContain('## SEO Fix Commits Summary');
-      expect(summary).toContain('2 commits created');
-      expect(summary).toContain('abc1234');
-      expect(summary).toContain('def5678');
-      expect(summary).toContain('add title tags');
-      expect(summary).toContain('add og tags');
-    });
-
-    it('includes failed commits in summary', () => {
-      const commits = [
-        {
-          fix: {
-            category: 'on-page',
-            issues: ['TITLE_MISSING'],
-            filesChanged: ['index.html'],
-            description: 'add title tags',
-          },
-          result: { success: true, hash: 'abc1234' },
-        },
-        {
-          fix: {
-            category: 'crawlability',
-            issues: ['ROBOTS_TXT_MISSING'],
-            filesChanged: ['robots.txt'],
-            description: 'add robots.txt',
-          },
-          result: { success: false, error: 'Permission denied' },
-        },
-      ];
-
-      const summary = generateCommitSummary(commits);
-
-      expect(summary).toContain('1 commits created');
-      expect(summary).toContain('1 commits failed');
-      expect(summary).toContain('### Failed Commits');
-      expect(summary).toContain('Permission denied');
-    });
-  });
-
-  describe('COMMIT_TYPES and SEO_SCOPES', () => {
-    it('has all expected commit types', () => {
-      const types = COMMIT_TYPES.map((t) => t.type);
-      expect(types).toContain('fix');
-      expect(types).toContain('feat');
-      expect(types).toContain('docs');
-      expect(types).toContain('perf');
-      expect(types).toContain('chore');
-    });
-
-    it('has all expected SEO scopes', () => {
-      expect(SEO_SCOPES).toContain('seo');
-      expect(SEO_SCOPES).toContain('meta');
-      expect(SEO_SCOPES).toContain('og');
-      expect(SEO_SCOPES).toContain('schema');
-      expect(SEO_SCOPES).toContain('images');
-      expect(SEO_SCOPES).toContain('links');
-      expect(SEO_SCOPES).toContain('security');
-    });
-  });
-});
-
-describe('detectGitHubPages', () => {
-  // These tests would need filesystem mocking
-  // For now, we test the function exists and returns expected shape
-  it('returns expected structure', () => {
-    // Test with a non-existent path returns false
-    const result = detectGitHubPages('/non/existent/path');
-    expect(result).toHaveProperty('isGitHubPages');
-    expect(typeof result.isGitHubPages).toBe('boolean');
-  });
-});
-
-describe('isGitRepo', () => {
-  it('returns false for non-git directory', () => {
-    expect(isGitRepo('/tmp')).toBe(false);
-  });
-
-  it('returns boolean', () => {
-    const result = isGitRepo();
-    expect(typeof result).toBe('boolean');
-  });
-});

package/src/git/commit-helper.ts

@@ -1,329 +0,0 @@
-/**
- * Git Commit Helper
- *
- * Creates conventional commits for SEO fixes with proper co-authoring.
- * Supports GitHub Pages and various static site generators.
- */
-
-import { execSync, exec } from 'child_process';
-import { promisify } from 'util';
-
-const execAsync = promisify(exec);
-
-// System name is configurable (TBD - SEO Autopilot is temporary)
-const SYSTEM_NAME = process.env.SEO_SYSTEM_NAME || 'SEO Autopilot';
-const SYSTEM_EMAIL = process.env.SEO_SYSTEM_EMAIL || 'bot@seo-autopilot.dev';
-
-export interface CommitConfig {
-  systemName?: string;
-  systemEmail?: string;
-  dryRun?: boolean;
-}
-
-export interface ConventionalCommit {
-  type: 'fix' | 'feat' | 'docs' | 'style' | 'refactor' | 'perf' | 'chore';
-  scope?: string;
-  description: string;
-  body?: string;
-  breaking?: boolean;
-  issues?: string[]; // Issue codes like TITLE_MISSING
-}
-
-export interface SEOFixCommit {
-  category: string;
-  issues: string[];
-  filesChanged: string[];
-  description: string;
-}
-
-/**
- * Maps SEO issue categories to conventional commit types and scopes
- */
-const CATEGORY_MAPPING: Record<string, { type: ConventionalCommit['type']; scope: string }> = {
-  'crawlability': { type: 'fix', scope: 'seo' },
-  'indexability': { type: 'fix', scope: 'seo' },
-  'on-page': { type: 'fix', scope: 'meta' },
-  'content': { type: 'fix', scope: 'content' },
-  'links': { type: 'fix', scope: 'links' },
-  'images': { type: 'fix', scope: 'images' },
-  'structured-data': { type: 'feat', scope: 'schema' },
-  'performance': { type: 'perf', scope: 'web' },
-  'security': { type: 'fix', scope: 'security' },
-  'mobile': { type: 'fix', scope: 'mobile' },
-  'international': { type: 'fix', scope: 'i18n' },
-  'social-meta': { type: 'feat', scope: 'og' },
-};
-
-/**
- * Formats a conventional commit message
- */
-export function formatConventionalCommit(commit: ConventionalCommit): string {
-  let message = commit.type;
-
-  if (commit.scope) {
-    message += `(${commit.scope})`;
-  }
-
-  if (commit.breaking) {
-    message += '!';
-  }
-
-  message += `: ${commit.description}`;
-
-  if (commit.body) {
-    message += `\n\n${commit.body}`;
-  }
-
-  if (commit.issues && commit.issues.length > 0) {
-    message += `\n\nIssue codes: ${commit.issues.join(', ')}`;
-  }
-
-  return message;
-}
-
-/**
- * Formats a commit message for SEO fixes
- */
-export function formatSEOCommitMessage(fix: SEOFixCommit, config?: CommitConfig): string {
-  const mapping = CATEGORY_MAPPING[fix.category] || { type: 'fix', scope: 'seo' };
-  const systemName = config?.systemName || SYSTEM_NAME;
-  const systemEmail = config?.systemEmail || SYSTEM_EMAIL;
-
-  const commit: ConventionalCommit = {
-    type: mapping.type,
-    scope: mapping.scope,
-    description: fix.description,
-    body: `Files changed:\n${fix.filesChanged.map(f => `  - ${f}`).join('\n')}`,
-    issues: fix.issues,
-  };
-
-  let message = formatConventionalCommit(commit);
-
-  // Add co-author trailer
-  message += `\n\nCo-Authored-By: ${systemName} <${systemEmail}>`;
-
-  return message;
-}
-
-/**
- * Creates a git commit with the SEO fix
- */
-export async function createSEOCommit(
-  fix: SEOFixCommit,
-  config?: CommitConfig
-): Promise<{ success: boolean; hash?: string; error?: string }> {
-  const message = formatSEOCommitMessage(fix, config);
-
-  if (config?.dryRun) {
-    console.log('DRY RUN - Would create commit:');
-    console.log('---');
-    console.log(message);
-    console.log('---');
-    return { success: true, hash: 'dry-run' };
-  }
-
-  try {
-    // Stage the files
-    for (const file of fix.filesChanged) {
-      await execAsync(`git add "${file}"`);
-    }
-
-    // Create the commit using heredoc for proper message formatting
-    const { stdout } = await execAsync(`git commit -m "$(cat <<'EOF'
-${message}
-EOF
-)"`);
-
-    // Extract commit hash
-    const hashMatch = stdout.match(/\[[\w-]+ ([a-f0-9]+)\]/);
-    const hash = hashMatch ? hashMatch[1] : undefined;
-
-    return { success: true, hash };
-  } catch (error) {
-    return { success: false, error: (error as Error).message };
-  }
-}
-
-/**
- * Creates multiple commits, one per category
- */
-export async function createSEOCommits(
-  fixes: SEOFixCommit[],
-  config?: CommitConfig
-): Promise<{ commits: Array<{ fix: SEOFixCommit; result: Awaited<ReturnType<typeof createSEOCommit>> }> }> {
-  const commits = [];
-
-  for (const fix of fixes) {
-    const result = await createSEOCommit(fix, config);
-    commits.push({ fix, result });
-  }
-
-  return { commits };
-}
-
-/**
- * Checks if the current directory is a git repository
- */
-export function isGitRepo(dir?: string): boolean {
-  try {
-    const cwd = dir ? { cwd: dir } : undefined;
-    execSync('git rev-parse --is-inside-work-tree', { ...cwd, stdio: 'ignore' });
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Initializes a git repository if not already initialized
- */
-export async function ensureGitRepo(dir?: string): Promise<void> {
-  if (!isGitRepo(dir)) {
-    const cwd = dir ? { cwd: dir } : undefined;
-    await execAsync('git init', cwd);
-  }
-}
-
-/**
- * Gets the current git user config
- */
-export function getGitUser(): { name?: string; email?: string } {
-  try {
-    const name = execSync('git config user.name', { encoding: 'utf8' }).trim();
-    const email = execSync('git config user.email', { encoding: 'utf8' }).trim();
-    return { name, email };
-  } catch {
-    return {};
-  }
-}
-
-/**
- * Detects if this is a GitHub Pages repository
- */
-export function detectGitHubPages(dir: string): {
-  isGitHubPages: boolean;
-  type?: 'jekyll' | 'static' | 'docs' | 'custom';
-  baseUrl?: string;
-} {
-  const fs = require('fs');
-  const path = require('path');
-
-  // Check for common GitHub Pages indicators
-  const checks = [
-    { file: '_config.yml', type: 'jekyll' as const },
-    { file: 'docs/index.html', type: 'docs' as const },
-    { file: 'CNAME', type: 'custom' as const },
-    { file: '.nojekyll', type: 'static' as const },
-    { file: 'index.html', type: 'static' as const },
-  ];
-
-  for (const check of checks) {
-    const filePath = path.join(dir, check.file);
-    if (fs.existsSync(filePath)) {
-      // Try to extract base URL from _config.yml
-      if (check.file === '_config.yml') {
-        try {
-          const content = fs.readFileSync(filePath, 'utf8');
-          const baseUrlMatch = content.match(/baseurl:\s*["']?([^"'\n]+)/);
-          const urlMatch = content.match(/url:\s*["']?([^"'\n]+)/);
-          return {
-            isGitHubPages: true,
-            type: check.type,
-            baseUrl: urlMatch ? urlMatch[1] + (baseUrlMatch?.[1] || '') : undefined,
-          };
-        } catch {
-          // Continue
-        }
-      }
-
-      // Check for CNAME (custom domain)
-      if (check.file === 'CNAME') {
-        try {
-          const cname = fs.readFileSync(path.join(dir, 'CNAME'), 'utf8').trim();
-          return {
-            isGitHubPages: true,
-            type: check.type,
-            baseUrl: `https://${cname}`,
-          };
-        } catch {
-          // Continue
-        }
-      }
-
-      return { isGitHubPages: true, type: check.type };
-    }
-  }
-
-  // Check if remote is github.io
-  try {
-    const remote = execSync('git remote get-url origin', { cwd: dir, encoding: 'utf8' }).trim();
-    if (remote.includes('github.io') || remote.includes('.github.')) {
-      return { isGitHubPages: true, type: 'static' };
-    }
-  } catch {
-    // Not a git repo or no remote
-  }
-
-  return { isGitHubPages: false };
-}
-
-/**
- * Generates a summary of commits made
- */
-export function generateCommitSummary(
-  commits: Array<{ fix: SEOFixCommit; result: { success: boolean; hash?: string; error?: string } }>
-): string {
-  const successful = commits.filter(c => c.result.success);
-  const failed = commits.filter(c => !c.result.success);
-
-  let summary = `## SEO Fix Commits Summary\n\n`;
-  summary += `✅ ${successful.length} commits created\n`;
-  if (failed.length > 0) {
-    summary += `❌ ${failed.length} commits failed\n`;
-  }
-  summary += '\n';
-
-  if (successful.length > 0) {
-    summary += `### Commits Created\n\n`;
-    for (const { fix, result } of successful) {
-      const mapping = CATEGORY_MAPPING[fix.category] || { type: 'fix', scope: 'seo' };
-      summary += `- \`${result.hash?.slice(0, 7) || 'N/A'}\` ${mapping.type}(${mapping.scope}): ${fix.description}\n`;
-      summary += `  - Files: ${fix.filesChanged.join(', ')}\n`;
-      summary += `  - Issues: ${fix.issues.join(', ')}\n`;
-    }
-  }
-
-  if (failed.length > 0) {
-    summary += `\n### Failed Commits\n\n`;
-    for (const { fix, result } of failed) {
-      summary += `- ❌ ${fix.description}: ${result.error}\n`;
-    }
-  }
-
-  return summary;
-}
-
-// Export types for conventional commit categories
-export const COMMIT_TYPES = [
-  { type: 'fix', description: 'Bug fix (SEO issue correction)' },
-  { type: 'feat', description: 'New feature (schema, OG tags, etc.)' },
-  { type: 'docs', description: 'Documentation update' },
-  { type: 'style', description: 'Code style changes (formatting)' },
-  { type: 'refactor', description: 'Code restructuring' },
-  { type: 'perf', description: 'Performance improvement' },
-  { type: 'chore', description: 'Maintenance task' },
-] as const;
-
-export const SEO_SCOPES = [
-  'seo',      // General SEO
-  'meta',     // Meta tags (title, description)
-  'og',       // Open Graph / Twitter Cards
-  'schema',   // Structured data / JSON-LD
-  'images',   // Image optimization
-  'links',    // Link fixes
-  'content',  // Content optimization
-  'security', // Security headers
-  'mobile',   // Mobile optimization
-  'i18n',     // Internationalization
-  'perf',     // Performance
-] as const;

package/src/git/index.ts

@@ -1,12 +0,0 @@
-/**
- * Git integration module
- *
- * Provides git operations for SEO fixes including:
- * - Conventional commit formatting
- * - Co-author support
- * - GitHub Pages detection
- * - Pull request generation
- */
-
-export * from './commit-helper.js';
-export * from './pr-helper.js';