@ateriss_/aiv-cli 1.0.1 → 1.0.3

package/src/git/github.ts

@@ -1,4 +1,6 @@
-import { PullRequest, PRDiff, PRFile } from '../types';
+import { PullRequest, PRDiff, PRFile, ReviewResult } from '../types';
+
+const AIV_COMMENT_TAG = '<!-- aiv-review:';
 
 const GITHUB_API = 'https://api.github.com';
 
@@ -59,6 +61,36 @@ export class GithubClient {
     return { pr, files, rawDiff };
   }
 
+  async findAivReview(owner: string, repo: string, prNumber: number): Promise<ReviewResult | null> {
+    const url = `${GITHUB_API}/repos/${owner}/${repo}/issues/${prNumber}/comments?per_page=100`;
+    const res = await this.fetch(url);
+    if (!res.ok) return null;
+
+    const comments = await res.json() as any[];
+    for (const c of comments) {
+      const body: string = c.body ?? '';
+      if (!body.startsWith(AIV_COMMENT_TAG)) continue;
+      const end = body.indexOf(' -->');
+      if (end === -1) continue;
+      try {
+        const encoded = body.slice(AIV_COMMENT_TAG.length, end).trim();
+        return JSON.parse(Buffer.from(encoded, 'base64').toString('utf8')) as ReviewResult;
+      } catch { continue; }
+    }
+    return null;
+  }
+
+  async postComment(owner: string, repo: string, prNumber: number, body: string): Promise<void> {
+    const url = `${GITHUB_API}/repos/${owner}/${repo}/issues/${prNumber}/comments`;
+    const { default: fetch } = await import('node-fetch');
+    const res = await fetch(url, {
+      method: 'POST',
+      headers: { ...this.headers, 'Content-Type': 'application/json' },
+      body: JSON.stringify({ body }),
+    }) as unknown as Response;
+    if (!res.ok) await this.throwError(res, `post comment on PR #${prNumber}`);
+  }
+
   async mergePR(
     owner: string,
     repo: string,
@@ -75,6 +107,26 @@ export class GithubClient {
     if (!res.ok) await this.throwError(res, `merge PR #${prNumber}`);
   }
 
+  async createPR(
+    owner: string,
+    repo: string,
+    title: string,
+    body: string,
+    head: string,
+    base: string,
+  ): Promise<{ number: number; url: string }> {
+    const url = `${GITHUB_API}/repos/${owner}/${repo}/pulls`;
+    const { default: fetch } = await import('node-fetch');
+    const res = await fetch(url, {
+      method: 'POST',
+      headers: { ...this.headers, 'Content-Type': 'application/json' },
+      body: JSON.stringify({ title, body, head, base }),
+    }) as unknown as Response;
+    if (!res.ok) await this.throwError(res, 'create PR');
+    const data = await res.json() as any;
+    return { number: data.number, url: data.html_url };
+  }
+
   async submitReview(
     owner: string,
     repo: string,

package/src/git/local.ts

@@ -0,0 +1,113 @@
+import { execSync } from 'node:child_process';
+import type { PRDiff, PRFile, PullRequest } from '../types';
+
+export function getCurrentBranch(cwd: string): string {
+  return execSync('git rev-parse --abbrev-ref HEAD', { cwd, stdio: 'pipe' }).toString().trim();
+}
+
+// Always prefers origin/<branch> (remote = source of truth). Falls back to local if no remote.
+function resolveRef(cwd: string, branch: string): string {
+  try {
+    execSync(`git rev-parse --verify origin/${branch}`, { cwd, stdio: 'pipe' });
+    return `origin/${branch}`;
+  } catch {
+    return branch;
+  }
+}
+
+export function detectBaseBranch(cwd: string): string {
+  const head = getCurrentBranch(cwd);
+  for (const b of ['main', 'master', 'develop']) {
+    if (b === head) continue;
+    // Check remote first, then local
+    for (const ref of [`origin/${b}`, b]) {
+      try {
+        execSync(`git rev-parse --verify ${ref}`, { cwd, stdio: 'pipe' });
+        return b;
+      } catch {}
+    }
+  }
+  return 'main';
+}
+
+export function hasUnpushedCommits(cwd: string, branch: string): boolean {
+  try {
+    execSync(`git rev-parse --verify origin/${branch}`, { cwd, stdio: 'pipe' });
+    const out = execSync(`git log origin/${branch}..HEAD --oneline`, { cwd, stdio: 'pipe' })
+      .toString().trim();
+    return out.length > 0;
+  } catch {
+    // origin/<branch> doesn't exist — branch was never pushed
+    return true;
+  }
+}
+
+export function getLastCommitTitle(cwd: string): string {
+  try {
+    return execSync('git log -1 --pretty=%s', { cwd, stdio: 'pipe' }).toString().trim();
+  } catch { return ''; }
+}
+
+export function buildLocalPRDiff(cwd: string, base: string): PRDiff {
+  const head = getCurrentBranch(cwd);
+  const ref = resolveRef(cwd, base);
+
+  const numstat = execSync(`git diff ${ref}...HEAD --numstat`, { cwd, stdio: 'pipe' })
+    .toString().trim();
+  const nameStatus = execSync(`git diff ${ref}...HEAD --name-status`, { cwd, stdio: 'pipe' })
+    .toString().trim();
+  const rawDiff = execSync(`git diff ${ref}...HEAD`, {
+    cwd, stdio: 'pipe', maxBuffer: 5 * 1024 * 1024,
+  }).toString();
+
+  const statsMap = new Map<string, { additions: number; deletions: number }>();
+  for (const line of numstat.split('\n').filter(Boolean)) {
+    const parts = line.split('\t');
+    if (parts.length >= 3) {
+      statsMap.set(parts[2], {
+        additions: Number.parseInt(parts[0]) || 0,
+        deletions: Number.parseInt(parts[1]) || 0,
+      });
+    }
+  }
+
+  const files: PRFile[] = [];
+  for (const line of nameStatus.split('\n').filter(Boolean)) {
+    const parts = line.split('\t');
+    if (parts.length < 2) continue;
+    const statusChar = parts[0][0];
+    const filename = parts[parts.length - 1];
+    const statusMap: Record<string, PRFile['status']> = {
+      A: 'added', M: 'modified', D: 'deleted', R: 'renamed', C: 'added',
+    };
+    const status = statusMap[statusChar] ?? 'modified';
+    const stats = statsMap.get(filename) ?? { additions: 0, deletions: 0 };
+    files.push({ filename, status, ...stats });
+  }
+
+  const log = (() => {
+    try {
+      return execSync(`git log ${ref}...HEAD --oneline`, { cwd, stdio: 'pipe' }).toString().trim();
+    } catch { return ''; }
+  })();
+
+  const pr: PullRequest = {
+    id: 0,
+    number: 0,
+    title: `${head} → ${base}`,
+    author: 'local',
+    branch: head,
+    base,
+    url: '',
+    state: 'draft',
+    createdAt: new Date().toISOString(),
+    updatedAt: new Date().toISOString(),
+    additions: files.reduce((s, f) => s + f.additions, 0),
+    deletions: files.reduce((s, f) => s + f.deletions, 0),
+    changedFiles: files.length,
+    labels: [],
+    description: log || undefined,
+  };
+
+  return { pr, files, rawDiff };
+}

package/src/git/utils.ts

@@ -26,18 +26,27 @@ export function parseGithubUrl(url: string): RepoInfo | null {
   return null;
 }
 
+const GITIGNORE_ENTRIES = [
+  '.aiv/tree.json',     // auto-generated snapshot — large, not useful in diffs
+  '.aiv/checklist.md',  // personal pre-PR draft — not team-relevant
+];
+
 export function appendGitignore(cwd: string): void {
   const gitignorePath = path.join(cwd, '.gitignore');
-  const entry = '.aiv/';
 
-  if (!fs.existsSync(gitignorePath)) {
-    fs.writeFileSync(gitignorePath, entry + '\n', 'utf8');
-    return;
-  }
+  const current = fs.existsSync(gitignorePath)
+    ? fs.readFileSync(gitignorePath, 'utf8')
+    : '';
+
+  const missing = GITIGNORE_ENTRIES.filter(e => !current.includes(e));
+  if (missing.length === 0) return;
+
+  const block = '\n# aiv — auto-generated / local-only files\n' + missing.join('\n') + '\n';
 
-  const content = fs.readFileSync(gitignorePath, 'utf8');
-  if (!content.includes('.aiv')) {
-    fs.appendFileSync(gitignorePath, `\n# aiv local context\n${entry}\n`, 'utf8');
+  if (fs.existsSync(gitignorePath)) {
+    fs.appendFileSync(gitignorePath, block, 'utf8');
+  } else {
+    fs.writeFileSync(gitignorePath, block.trimStart(), 'utf8');
   }
 }
 

package/src/i18n/en.ts

@@ -42,7 +42,7 @@ export const en = {
   prsColPR: 'PR',
   prsColTitle: 'Title',
   prsColAuthor: 'Author',
-  prsColBranch: 'Branch',
+  prsColBranch: 'Branch → Target',
   prsColChanges: 'Changes',
   prsColCreated: 'Created',
   prsFooter: (count: number) => `  Showing ${count} open PR(s). Run aiv review <pr-number> to analyze.\n`,
@@ -108,6 +108,9 @@ export const en = {
   accountsRepoHint: '  Repo config:   .aiv/config.yml\n',
 
   // ── orchestrator ───────────────────────────────────────────────────────────
+  orchestratorTriaging: 'Analyzing diff to select agents...',
+  orchestratorTriageResult: (agents: string, reason: string) => `  Agents selected: ${agents} — ${reason}`,
+  orchestratorTriageSkipped: 'No agents needed for this diff.',
   orchestratorRunning: (agent: string) => `  Running ${agent} agent...`,
   orchestratorDone: (agent: string, count: number, score: number) => `  ${agent} — ${count} finding(s) [score: ${score}]`,
   orchestratorAgentFailed: (agent: string, msg: string) => `  ${agent} failed: ${msg}`,
@@ -183,6 +186,13 @@ export const en = {
   postReviewMergeStrategyMerge: 'Merge commit',
   postReviewMergeStrategySquash: 'Squash and merge',
   postReviewMergeStrategyRebase: 'Rebase and merge',
+  postReviewPostingComment: 'Posting review as PR comment...',
+  postReviewCommentPosted: (n: number) => `  Review posted as comment on PR #${n}.`,
+  postReviewCommentFailed: (msg: string) => `Failed to post comment: ${msg}`,
+  reviewCachedFound: 'Found a previous aiv analysis on this PR.',
+  reviewCachedUse: 'Use cached analysis?',
+  reviewCachedUsing: 'Using cached analysis.',
+  reviewCachedSkipping: 'Running fresh analysis...',
 
   // ── context generate ───────────────────────────────────────────────────────
   contextGenerateTitle: '\n  Generating context and rules with AI...\n',
@@ -194,6 +204,21 @@ export const en = {
   contextGenerateWritten: (file: string) => `  ${file} written.`,
   contextGenerateProviderError: (envVar: string) => `Missing AI key: ${envVar}. Check your provider config.`,
 
+  // ── precheck command ───────────────────────────────────────────────────────
+  precheckTitle: (branch: string, base: string) => `\n  aiv check — ${branch} → ${base}\n`,
+  precheckNotGitRepo: 'Not a git repository.',
+  precheckBuilding: (base: string) => `Building diff against ${base}...`,
+  precheckDiffBuilt: (files: number) => `Diff ready: ${files} file(s) changed`,
+  precheckDiffFailed: (msg: string) => `Failed to build local diff: ${msg}`,
+  precheckNoChanges: 'No changes detected against base branch.',
+  precheckSelectAction: 'What would you like to do?',
+  precheckCreatingPR: 'Creating PR on GitHub...',
+  precheckPRCreated: (url: string) => `  PR created: ${url}`,
+  precheckPRFailed: (msg: string) => `Failed to create PR: ${msg}`,
+  precheckSavingChecklist: 'Saving checklist...',
+  precheckChecklistSaved: (path: string) => `  Checklist saved to ${path}`,
+  precheckMissingConfig: 'GitHub owner/repo not configured. Use --owner and --repo or configure in .aiv/config.yml',
+
   // ── agents command ─────────────────────────────────────────────────────────
   agentsTitle: '\n  aiv — Available Agents\n',
   agentsColAgent: 'Agent',

package/src/i18n/es.ts

@@ -44,7 +44,7 @@ export const es: TranslationKeys = {
   prsColPR: 'PR',
   prsColTitle: 'Título',
   prsColAuthor: 'Autor',
-  prsColBranch: 'Rama',
+  prsColBranch: 'Rama → Destino',
   prsColChanges: 'Cambios',
   prsColCreated: 'Creado',
   prsFooter: (count: number) => `  Mostrando ${count} PR(s) abierto(s). Ejecuta aiv review <numero-pr> para analizar.\n`,
@@ -110,6 +110,9 @@ export const es: TranslationKeys = {
   accountsRepoHint: '  Config repo:   .aiv/config.yml\n',
 
   // ── orchestrator ───────────────────────────────────────────────────────────
+  orchestratorTriaging: 'Analizando diff para seleccionar agentes...',
+  orchestratorTriageResult: (agents: string, reason: string) => `  Agentes seleccionados: ${agents} — ${reason}`,
+  orchestratorTriageSkipped: 'No se necesitan agentes para este diff.',
   orchestratorRunning: (agent: string) => `  Ejecutando agente ${agent}...`,
   orchestratorDone: (agent: string, count: number, score: number) => `  ${agent} — ${count} hallazgo(s) [puntuación: ${score}]`,
   orchestratorAgentFailed: (agent: string, msg: string) => `  ${agent} falló: ${msg}`,
@@ -185,6 +188,13 @@ export const es: TranslationKeys = {
   postReviewMergeStrategyMerge: 'Merge commit',
   postReviewMergeStrategySquash: 'Squash and merge',
   postReviewMergeStrategyRebase: 'Rebase and merge',
+  postReviewPostingComment: 'Publicando revisión como comentario del PR...',
+  postReviewCommentPosted: (n: number) => `  Revisión publicada como comentario en el PR #${n}.`,
+  postReviewCommentFailed: (msg: string) => `Error al publicar comentario: ${msg}`,
+  reviewCachedFound: 'Se encontró un análisis previo de aiv en este PR.',
+  reviewCachedUse: '¿Usar el análisis en caché?',
+  reviewCachedUsing: 'Usando análisis en caché.',
+  reviewCachedSkipping: 'Ejecutando análisis nuevo...',
 
   // ── context generate ───────────────────────────────────────────────────────
   contextGenerateTitle: '\n  Generando contexto y reglas con IA...\n',
@@ -196,6 +206,21 @@ export const es: TranslationKeys = {
   contextGenerateWritten: (file: string) => `  ${file} escrito.`,
   contextGenerateProviderError: (envVar: string) => `Clave de IA faltante: ${envVar}. Revisa tu config de proveedor.`,
 
+  // ── precheck command ───────────────────────────────────────────────────────
+  precheckTitle: (branch: string, base: string) => `\n  aiv check — ${branch} → ${base}\n`,
+  precheckNotGitRepo: 'No es un repositorio git.',
+  precheckBuilding: (base: string) => `Construyendo diff contra ${base}...`,
+  precheckDiffBuilt: (files: number) => `Diff listo: ${files} archivo(s) modificado(s)`,
+  precheckDiffFailed: (msg: string) => `Error al construir el diff local: ${msg}`,
+  precheckNoChanges: 'No se detectaron cambios contra la rama base.',
+  precheckSelectAction: '¿Qué deseas hacer?',
+  precheckCreatingPR: 'Creando PR en GitHub...',
+  precheckPRCreated: (url: string) => `  PR creado: ${url}`,
+  precheckPRFailed: (msg: string) => `Error al crear el PR: ${msg}`,
+  precheckSavingChecklist: 'Guardando checklist...',
+  precheckChecklistSaved: (path: string) => `  Checklist guardado en ${path}`,
+  precheckMissingConfig: 'Owner/repo de GitHub no configurado. Usa --owner y --repo o configúralo en .aiv/config.yml',
+
   // ── agents command ─────────────────────────────────────────────────────────
   agentsTitle: '\n  aiv — Agentes Disponibles\n',
   agentsColAgent: 'Agente',

package/src/index.ts

@@ -8,6 +8,7 @@ import { reviewCommand } from './cli/commands/review';
 import { contextCommand } from './cli/commands/context';
 import { configCommand } from './cli/commands/config';
 import { agentsCommand } from './cli/commands/agents';
+import { checkCommand } from './cli/commands/check';
 
 initLang();
 printBanner();
@@ -25,5 +26,6 @@ program.addCommand(reviewCommand());
 program.addCommand(contextCommand());
 program.addCommand(configCommand());
 program.addCommand(agentsCommand());
+program.addCommand(checkCommand());
 
 program.parse(process.argv);

package/src/orchestrator/index.ts

@@ -6,6 +6,7 @@ import { BusinessReviewer } from '../agents/business';
 import { ArchitectureReviewer } from '../agents/architecture';
 import { SecurityReviewer } from '../agents/security';
 import { BaseAgent, AgentContext } from '../agents/base';
+import { triageAgents } from '../agents/triage';
 import { t } from '../i18n';
 
 export class Orchestrator {
@@ -14,9 +15,16 @@ export class Orchestrator {
     private readonly rules: AivRules,
   ) {}
 
-  async run(prDiff: PRDiff, projectContext: string, agentNames: string[]): Promise<ReviewResult> {
-    const agents = buildAgents(agentNames, this.config);
+  async run(prDiff: PRDiff, projectContext: string, requestedAgents: string[], auto: boolean): Promise<ReviewResult> {
+    const agentNames = auto
+      ? await this.triage(prDiff, requestedAgents)
+      : requestedAgents;
+
+    if (agentNames.length === 0) {
+      return buildEmptyResult(prDiff);
+    }
 
+    const agents = buildAgents(agentNames, this.config);
     const agentCtx: AgentContext = { diff: prDiff, projectContext, rules: this.rules };
     const agentResults = await runAgents(agents, agentCtx);
 
@@ -37,6 +45,44 @@ export class Orchestrator {
       generatedAt: new Date().toISOString(),
     };
   }
+
+  private async triage(prDiff: PRDiff, requested: string[]): Promise<string[]> {
+    const spinner = ora(t().orchestratorTriaging).start();
+    try {
+      const provider = createProviderFor(this.config, 'triage');
+      const result = await triageAgents(prDiff, provider);
+      const selected = result.agents.filter(a => requested.includes(a));
+
+      if (selected.length === 0) {
+        spinner.succeed(chalk.dim(t().orchestratorTriageSkipped));
+      } else {
+        spinner.succeed(chalk.dim(t().orchestratorTriageResult(
+          selected.map(a => chalk.cyan(a)).join(', '),
+          result.reasoning,
+        )));
+      }
+      return selected;
+    } catch {
+      spinner.warn(chalk.yellow('Triage failed — running all agents'));
+      return requested;
+    }
+  }
+}
+
+function buildEmptyResult(prDiff: PRDiff): ReviewResult {
+  return {
+    prNumber: prDiff.pr.number,
+    prTitle: prDiff.pr.title,
+    executiveSummary: t().orchestratorTriageSkipped,
+    riskScore: 0,
+    riskLabel: 'LOW',
+    agents: [],
+    businessRisks: [],
+    architectureIssues: [],
+    securityIssues: [],
+    possibleRegressions: [],
+    generatedAt: new Date().toISOString(),
+  };
 }
 
 function buildAgents(names: string[], config: ResolvedConfig): BaseAgent[] {

package/tsup.config.ts

@@ -9,4 +9,5 @@ export default defineConfig({
   splitting: false,
   sourcemap: false,
   dts: false,
+  noExternal: [/.*/],
 });