flowlint 0.3.1 → 0.3.3

package/dist/packages/review/providers/github.d.ts

@@ -0,0 +1,62 @@
+/**
+ * GitHub-specific provider implementations.
+ * These are used by the existing GitHub App workflow.
+ */
+import type { Octokit } from 'octokit';
+import type { FileSource, LintableFile, ConfigProvider, Reporter, AnalysisResult } from '../providers';
+import type { FlowLintConfig } from '../../config/flowlint-config';
+/**
+ * GitHub PR FileSource
+ * Fetches changed files from a GitHub Pull Request
+ */
+export declare class GitHubPrFileSource implements FileSource {
+    private gh;
+    private repo;
+    private prNumber;
+    constructor(gh: Octokit, repo: string, prNumber: number);
+    getFiles(patterns: {
+        include: string[];
+        ignore: string[];
+    }): Promise<LintableFile[]>;
+    /**
+     * Filter PR files based on glob patterns
+     */
+    private pickTargets;
+    /**
+     * Fetch raw file content from GitHub
+     */
+    private fetchRawFiles;
+}
+/**
+ * GitHub Remote ConfigProvider
+ * Loads .flowlint.yml from a GitHub repository
+ */
+export declare class GitHubConfigProvider implements ConfigProvider {
+    private gh;
+    private repo;
+    private sha;
+    constructor(gh: Octokit, repo: string, sha: string);
+    load(): Promise<FlowLintConfig>;
+    /**
+     * Deep merge default config with overrides
+     */
+    private deepMerge;
+    private mergeInto;
+}
+/**
+ * GitHub Check Run Reporter
+ * Sends results to GitHub Checks API
+ */
+export declare class GitHubCheckReporter implements Reporter {
+    private gh;
+    private repo;
+    private checkRunId;
+    private cfg;
+    private readonly BATCH_SIZE;
+    constructor(gh: Octokit, repo: string, checkRunId: number, cfg: FlowLintConfig);
+    report(results: AnalysisResult[]): Promise<void>;
+    private buildAnnotations;
+    private inferConclusion;
+    private summarize;
+    private mapSeverity;
+}

package/dist/packages/review/providers/github.js

@@ -0,0 +1,275 @@
+"use strict";
+/**
+ * GitHub-specific provider implementations.
+ * These are used by the existing GitHub App workflow.
+ */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.GitHubCheckReporter = exports.GitHubConfigProvider = exports.GitHubPrFileSource = void 0;
+const logger_1 = require("../../logger");
+const micromatch_1 = __importDefault(require("micromatch"));
+/**
+ * GitHub PR FileSource
+ * Fetches changed files from a GitHub Pull Request
+ */
+class GitHubPrFileSource {
+    constructor(gh, repo, prNumber) {
+        this.gh = gh;
+        this.repo = repo;
+        this.prNumber = prNumber;
+    }
+    async getFiles(patterns) {
+        const [owner, repoName] = this.repo.split('/');
+        const paginate = this.gh.paginate;
+        if (typeof paginate !== 'function') {
+            throw new Error('Octokit client is missing paginate(); ensure @octokit/plugin-paginate-rest is registered.');
+        }
+        // 1. List all files in the PR
+        const allFiles = await paginate.call(this.gh, 'GET /repos/{owner}/{repo}/pulls/{pull_number}/files', {
+            owner,
+            repo: repoName,
+            pull_number: this.prNumber,
+            per_page: 100,
+        });
+        // 2. Filter for target files using globs
+        const targets = this.pickTargets(allFiles, patterns);
+        // 3. Fetch raw content
+        const { contents } = await this.fetchRawFiles(targets);
+        return Array.from(contents.entries()).map(([path, content]) => ({
+            path,
+            content,
+        }));
+    }
+    /**
+     * Filter PR files based on glob patterns
+     */
+    pickTargets(files, globs) {
+        return files
+            .filter((file) => file.status !== 'removed')
+            .filter((file) => {
+            const normalized = file.filename.replace(/\\/g, '/');
+            const included = micromatch_1.default.isMatch(normalized, globs.include, { dot: true });
+            const ignored = micromatch_1.default.isMatch(normalized, globs.ignore, { dot: true });
+            return included && !ignored;
+        });
+    }
+    /**
+     * Fetch raw file content from GitHub
+     */
+    async fetchRawFiles(targets) {
+        const [owner, repoName] = this.repo.split('/');
+        const contents = new Map();
+        const errors = [];
+        for (const file of targets) {
+            if (!file.sha) {
+                errors.push({ filename: file.filename, error: 'Missing SHA (file may be removed)' });
+                continue;
+            }
+            try {
+                const { data: blob } = await this.gh.request('GET /repos/{owner}/{repo}/git/blobs/{file_sha}', {
+                    owner,
+                    repo: repoName,
+                    file_sha: file.sha,
+                });
+                const raw = Buffer.from(blob.content, 'base64').toString('utf8');
+                contents.set(file.filename, raw);
+            }
+            catch (error) {
+                const errorMsg = error instanceof Error ? error.message : String(error);
+                logger_1.logger.warn({ filename: file.filename, error: errorMsg }, 'failed to fetch file');
+                errors.push({ filename: file.filename, error: errorMsg });
+            }
+        }
+        return { contents, errors };
+    }
+}
+exports.GitHubPrFileSource = GitHubPrFileSource;
+/**
+ * GitHub Remote ConfigProvider
+ * Loads .flowlint.yml from a GitHub repository
+ */
+class GitHubConfigProvider {
+    constructor(gh, repo, sha) {
+        this.gh = gh;
+        this.repo = repo;
+        this.sha = sha;
+    }
+    async load() {
+        const { defaultConfig } = await Promise.resolve().then(() => __importStar(require('../../config/flowlint-config')));
+        const YAML = (await Promise.resolve().then(() => __importStar(require('yaml')))).default;
+        const [owner, repoName] = this.repo.split('/');
+        try {
+            const { data } = await this.gh.request('GET /repos/{owner}/{repo}/contents/{path}', {
+                owner,
+                repo: repoName,
+                path: '.flowlint.yml',
+                ref: this.sha,
+            });
+            if (!('content' in data))
+                return defaultConfig;
+            const content = Buffer.from(data.content, 'base64').toString('utf8');
+            const parsed = YAML.parse(content) || {};
+            return this.deepMerge(defaultConfig, parsed);
+        }
+        catch {
+            return defaultConfig;
+        }
+    }
+    /**
+     * Deep merge default config with overrides
+     */
+    deepMerge(base, override) {
+        const baseCopy = JSON.parse(JSON.stringify(base));
+        if (!override)
+            return baseCopy;
+        return this.mergeInto(baseCopy, override);
+    }
+    mergeInto(target, source) {
+        for (const [key, value] of Object.entries(source)) {
+            if (value === undefined || value === null)
+                continue;
+            if (Array.isArray(value)) {
+                target[key] = value;
+            }
+            else if (typeof value === 'object') {
+                if (typeof target[key] !== 'object' || target[key] === null) {
+                    target[key] = {};
+                }
+                this.mergeInto(target[key], value);
+            }
+            else {
+                target[key] = value;
+            }
+        }
+        return target;
+    }
+}
+exports.GitHubConfigProvider = GitHubConfigProvider;
+/**
+ * GitHub Check Run Reporter
+ * Sends results to GitHub Checks API
+ */
+class GitHubCheckReporter {
+    constructor(gh, repo, checkRunId, cfg) {
+        this.gh = gh;
+        this.repo = repo;
+        this.checkRunId = checkRunId;
+        this.cfg = cfg;
+        this.BATCH_SIZE = 50;
+    }
+    async report(results) {
+        const [owner, repoName] = this.repo.split('/');
+        // Collect all findings
+        const allFindings = results.flatMap((r) => r.findings);
+        // Build output
+        const summary = this.summarize(allFindings);
+        const conclusion = this.inferConclusion(allFindings);
+        // Send annotations if enabled
+        if (this.cfg.report.annotations) {
+            const annotations = this.buildAnnotations(allFindings);
+            const limitedAnnotations = this.cfg.report.summary_limit > 0 && annotations.length > this.cfg.report.summary_limit
+                ? annotations.slice(0, this.cfg.report.summary_limit)
+                : annotations;
+            for (let i = 0; i < limitedAnnotations.length; i += this.BATCH_SIZE) {
+                const batch = limitedAnnotations.slice(i, i + this.BATCH_SIZE);
+                await this.gh.request('PATCH /repos/{owner}/{repo}/check-runs/{check_run_id}', {
+                    owner,
+                    repo: repoName,
+                    check_run_id: this.checkRunId,
+                    output: {
+                        title: process.env.CHECK_TITLE || 'FlowLint findings',
+                        summary,
+                        annotations: batch,
+                    },
+                });
+            }
+        }
+        // Finalize check run
+        await this.gh.request('PATCH /repos/{owner}/{repo}/check-runs/{check_run_id}', {
+            owner,
+            repo: repoName,
+            check_run_id: this.checkRunId,
+            status: 'completed',
+            conclusion,
+            completed_at: new Date().toISOString(),
+            output: {
+                title: process.env.CHECK_TITLE || 'FlowLint findings',
+                summary,
+            },
+        });
+    }
+    buildAnnotations(findings) {
+        const severityOrder = ['must', 'should', 'nit'];
+        const ordered = [...findings].sort((a, b) => severityOrder.indexOf(a.severity) - severityOrder.indexOf(b.severity));
+        return ordered.map((finding) => {
+            const line = finding.line ?? 1;
+            return {
+                path: finding.path,
+                start_line: line,
+                end_line: line,
+                annotation_level: this.mapSeverity(finding.severity),
+                message: `${finding.rule}: ${finding.message}`,
+                raw_details: finding.raw_details?.slice(0, 64000),
+            };
+        });
+    }
+    inferConclusion(findings) {
+        if (findings.some((f) => f.severity === 'must'))
+            return 'failure';
+        if (findings.some((f) => f.severity === 'should'))
+            return 'neutral';
+        return 'success';
+    }
+    summarize(findings) {
+        if (findings.length === 0)
+            return 'No issues found.';
+        const must = findings.filter((f) => f.severity === 'must').length;
+        const should = findings.filter((f) => f.severity === 'should').length;
+        const nit = findings.filter((f) => f.severity === 'nit').length;
+        return `${must} must-fix, ${should} should-fix, ${nit} nit.`;
+    }
+    mapSeverity(severity) {
+        if (severity === 'must')
+            return 'failure';
+        if (severity === 'should')
+            return 'warning';
+        return 'notice';
+    }
+}
+exports.GitHubCheckReporter = GitHubCheckReporter;
+//# sourceMappingURL=github.js.map

package/dist/packages/review/providers/github.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"github.js","sourceRoot":"","sources":["../../../../packages/review/providers/github.ts"],"names":[],"mappings":";AAAA;;;GAGG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAMH,yCAAsC;AACtC,4DAAoC;AAEpC;;;GAGG;AACH,MAAa,kBAAkB;IAC7B,YACU,EAAW,EACX,IAAY,EACZ,QAAgB;QAFhB,OAAE,GAAF,EAAE,CAAS;QACX,SAAI,GAAJ,IAAI,CAAQ;QACZ,aAAQ,GAAR,QAAQ,CAAQ;IACvB,CAAC;IAEJ,KAAK,CAAC,QAAQ,CAAC,QAAiD;QAC9D,MAAM,CAAC,KAAK,EAAE,QAAQ,CAAC,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAE/C,MAAM,QAAQ,GAAI,IAAI,CAAC,EAAmD,CAAC,QAAQ,CAAC;QACpF,IAAI,OAAO,QAAQ,KAAK,UAAU,EAAE,CAAC;YACnC,MAAM,IAAI,KAAK,CAAC,2FAA2F,CAAC,CAAC;QAC/G,CAAC;QAED,8BAA8B;QAC9B,MAAM,QAAQ,GAAG,MAAM,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,EAAE,qDAAqD,EAAE;YACnG,KAAK;YACL,IAAI,EAAE,QAAQ;YACd,WAAW,EAAE,IAAI,CAAC,QAAQ;YAC1B,QAAQ,EAAE,GAAG;SACd,CAAC,CAAC;QAEH,yCAAyC;QACzC,MAAM,OAAO,GAAG,IAAI,CAAC,WAAW,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;QAErD,uBAAuB;QACvB,MAAM,EAAE,QAAQ,EAAE,GAAG,MAAM,IAAI,CAAC,aAAa,CAAC,OAAO,CAAC,CAAC;QAEvD,OAAO,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,OAAO,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,OAAO,CAAC,EAAE,EAAE,CAAC,CAAC;YAC9D,IAAI;YACJ,OAAO;SACR,CAAC,CAAC,CAAC;IACN,CAAC;IAED;;OAEG;IACK,WAAW,CAAC,KAAe,EAAE,KAA8C;QACjF,OAAO,KAAK;aACT,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,MAAM,KAAK,SAAS,CAAC;aAC3C,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE;YACf,MAAM,UAAU,GAAG,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;YACrD,MAAM,QAAQ,GAAG,oBAAU,CAAC,OAAO,CAAC,UAAU,EAAE,KAAK,CAAC,OAAO,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,CAAC,CAAC;YAC9E,MAAM,OAAO,GAAG,oBAAU,CAAC,OAAO,CAAC,UAAU,EAAE,KAAK,CAAC,MAAM,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,CAAC,CAAC;YAC5E,OAAO,QAAQ,IAAI,CAAC,OAAO,CAAC;QAC9B,CAAC,CAAC,CAAC;IACP,CAAC;IAED;;OAEG;IACK,KAAK,CAAC,aAAa,CAAC,OAAiB;QAC3C,MAAM,CAAC,KAAK,EAAE,QAAQ,CAAC,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC/C,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAkB,CAAC;QAC3C,MAAM,MAAM,GAA+C,EAAE,CAAC;QAE9D,KAAK,MAAM,IAAI,IAAI,OAAO,EAAE,CAAC;YAC3B,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC;gBACd,MAAM,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,QAAQ,EAAE,KAAK,EAAE,mCAAmC,EAAE,CAAC,CAAC;gBACrF,SAAS;YACX,CAAC;YAED,IAAI,CAAC;gBACH,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,GAAG,MAAM,IAAI,CAAC,EAAE,CAAC,OAAO,CAAC,gDAAgD,EAAE;oBAC7F,KAAK;oBACL,IAAI,EAAE,QAAQ;oBACd,QAAQ,EAAE,IAAI,CAAC,GAAG;iBACnB,CAAC,CAAC;gBACH,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,QAAQ,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;gBACjE,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC;YACnC,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,QAAQ,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;gBACxE,eAAM,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,QAAQ,EAAE,KAAK,EAAE,QAAQ,EAAE,EAAE,sBAAsB,CAAC,CAAC;gBAClF,MAAM,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,QAAQ,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,CAAC;YAC5D,CAAC;QACH,CAAC;QAED,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,CAAC;IAC9B,CAAC;CACF;AAhFD,gDAgFC;AAED;;;GAGG;AACH,MAAa,oBAAoB;IAC/B,YACU,EAAW,EACX,IAAY,EACZ,GAAW;QAFX,OAAE,GAAF,EAAE,CAAS;QACX,SAAI,GAAJ,IAAI,CAAQ;QACZ,QAAG,GAAH,GAAG,CAAQ;IAClB,CAAC;IAEJ,KAAK,CAAC,IAAI;QACR,MAAM,EAAE,aAAa,EAAE,GAAG,wDAAa,8BAA8B,GAAC,CAAC;QACvE,MAAM,IAAI,GAAG,CAAC,wDAAa,MAAM,GAAC,CAAC,CAAC,OAAO,CAAC;QAE5C,MAAM,CAAC,KAAK,EAAE,QAAQ,CAAC,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC/C,IAAI,CAAC;YACH,MAAM,EAAE,IAAI,EAAE,GAAG,MAAM,IAAI,CAAC,EAAE,CAAC,OAAO,CAAC,2CAA2C,EAAE;gBAClF,KAAK;gBACL,IAAI,EAAE,QAAQ;gBACd,IAAI,EAAE,eAAe;gBACrB,GAAG,EAAE,IAAI,CAAC,GAAG;aACd,CAAC,CAAC;YACH,IAAI,CAAC,CAAC,SAAS,IAAI,IAAI,CAAC;gBAAE,OAAO,aAAa,CAAC;YAC/C,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAE,IAAY,CAAC,OAAO,EAAE,QAAQ,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;YAC9E,MAAM,MAAM,GAAI,IAAI,CAAC,KAAK,CAAC,OAAO,CAA6B,IAAI,EAAE,CAAC;YACtE,OAAO,IAAI,CAAC,SAAS,CAAC,aAAa,EAAE,MAAM,CAAC,CAAC;QAC/C,CAAC;QAAC,MAAM,CAAC;YACP,OAAO,aAAa,CAAC;QACvB,CAAC;IACH,CAAC;IAED;;OAEG;IACK,SAAS,CAAI,IAAO,EAAE,QAAiC;QAC7D,MAAM,QAAQ,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC;QAClD,IAAI,CAAC,QAAQ;YAAE,OAAO,QAAQ,CAAC;QAC/B,OAAO,IAAI,CAAC,SAAS,CAAC,QAAe,EAAE,QAAQ,CAAM,CAAC;IACxD,CAAC;IAEO,SAAS,CAAC,MAA+B,EAAE,MAA+B;QAChF,KAAK,MAAM,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,CAAC;YAClD,IAAI,KAAK,KAAK,SAAS,IAAI,KAAK,KAAK,IAAI;gBAAE,SAAS;YACpD,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;gBACzB,MAAM,CAAC,GAAG,CAAC,GAAG,KAAK,CAAC;YACtB,CAAC;iBAAM,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;gBACrC,IAAI,OAAO,MAAM,CAAC,GAAG,CAAC,KAAK,QAAQ,IAAI,MAAM,CAAC,GAAG,CAAC,KAAK,IAAI,EAAE,CAAC;oBAC5D,MAAM,CAAC,GAAG,CAAC,GAAG,EAAE,CAAC;gBACnB,CAAC;gBACD,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,GAAG,CAA4B,EAAE,KAAgC,CAAC,CAAC;YAC3F,CAAC;iBAAM,CAAC;gBACN,MAAM,CAAC,GAAG,CAAC,GAAG,KAAK,CAAC;YACtB,CAAC;QACH,CAAC;QACD,OAAO,MAAM,CAAC;IAChB,CAAC;CACF;AArDD,oDAqDC;AAED;;;GAGG;AACH,MAAa,mBAAmB;IAG9B,YACU,EAAW,EACX,IAAY,EACZ,UAAkB,EAClB,GAAmB;QAHnB,OAAE,GAAF,EAAE,CAAS;QACX,SAAI,GAAJ,IAAI,CAAQ;QACZ,eAAU,GAAV,UAAU,CAAQ;QAClB,QAAG,GAAH,GAAG,CAAgB;QANZ,eAAU,GAAG,EAAE,CAAC;IAO9B,CAAC;IAEJ,KAAK,CAAC,MAAM,CAAC,OAAyB;QACpC,MAAM,CAAC,KAAK,EAAE,QAAQ,CAAC,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAE/C,uBAAuB;QACvB,MAAM,WAAW,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC;QAEvD,eAAe;QACf,MAAM,OAAO,GAAG,IAAI,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC;QAC5C,MAAM,UAAU,GAAG,IAAI,CAAC,eAAe,CAAC,WAAW,CAAC,CAAC;QAErD,8BAA8B;QAC9B,IAAI,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC;YAChC,MAAM,WAAW,GAAG,IAAI,CAAC,gBAAgB,CAAC,WAAW,CAAC,CAAC;YACvD,MAAM,kBAAkB,GACtB,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,aAAa,GAAG,CAAC,IAAI,WAAW,CAAC,MAAM,GAAG,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,aAAa;gBACrF,CAAC,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC,EAAE,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,aAAa,CAAC;gBACrD,CAAC,CAAC,WAAW,CAAC;YAElB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,kBAAkB,CAAC,MAAM,EAAE,CAAC,IAAI,IAAI,CAAC,UAAU,EAAE,CAAC;gBACpE,MAAM,KAAK,GAAG,kBAAkB,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,GAAG,IAAI,CAAC,UAAU,CAAC,CAAC;gBAC/D,MAAM,IAAI,CAAC,EAAE,CAAC,OAAO,CAAC,uDAAuD,EAAE;oBAC7E,KAAK;oBACL,IAAI,EAAE,QAAQ;oBACd,YAAY,EAAE,IAAI,CAAC,UAAU;oBAC7B,MAAM,EAAE;wBACN,KAAK,EAAE,OAAO,CAAC,GAAG,CAAC,WAAW,IAAI,mBAAmB;wBACrD,OAAO;wBACP,WAAW,EAAE,KAAK;qBACnB;iBACF,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,qBAAqB;QACrB,MAAM,IAAI,CAAC,EAAE,CAAC,OAAO,CAAC,uDAAuD,EAAE;YAC7E,KAAK;YACL,IAAI,EAAE,QAAQ;YACd,YAAY,EAAE,IAAI,CAAC,UAAU;YAC7B,MAAM,EAAE,WAAW;YACnB,UAAU;YACV,YAAY,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;YACtC,MAAM,EAAE;gBACN,KAAK,EAAE,OAAO,CAAC,GAAG,CAAC,WAAW,IAAI,mBAAmB;gBACrD,OAAO;aACR;SACF,CAAC,CAAC;IACL,CAAC;IAEO,gBAAgB,CAAC,QAAe;QACtC,MAAM,aAAa,GAAG,CAAC,MAAM,EAAE,QAAQ,EAAE,KAAK,CAAC,CAAC;QAChD,MAAM,OAAO,GAAG,CAAC,GAAG,QAAQ,CAAC,CAAC,IAAI,CAChC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,aAAa,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC,GAAG,aAAa,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC,CAChF,CAAC;QAEF,OAAO,OAAO,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE;YAC7B,MAAM,IAAI,GAAG,OAAO,CAAC,IAAI,IAAI,CAAC,CAAC;YAC/B,OAAO;gBACL,IAAI,EAAE,OAAO,CAAC,IAAI;gBAClB,UAAU,EAAE,IAAI;gBAChB,QAAQ,EAAE,IAAI;gBACd,gBAAgB,EAAE,IAAI,CAAC,WAAW,CAAC,OAAO,CAAC,QAAQ,CAAC;gBACpD,OAAO,EAAE,GAAG,OAAO,CAAC,IAAI,KAAK,OAAO,CAAC,OAAO,EAAE;gBAC9C,WAAW,EAAE,OAAO,CAAC,WAAW,EAAE,KAAK,CAAC,CAAC,EAAE,KAAK,CAAC;aAClD,CAAC;QACJ,CAAC,CAAC,CAAC;IACL,CAAC;IAEO,eAAe,CAAC,QAAe;QACrC,IAAI,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,MAAM,CAAC;YAAE,OAAO,SAAS,CAAC;QAClE,IAAI,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,QAAQ,CAAC;YAAE,OAAO,SAAS,CAAC;QACpE,OAAO,SAAS,CAAC;IACnB,CAAC;IAEO,SAAS,CAAC,QAAe;QAC/B,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC;YAAE,OAAO,kBAAkB,CAAC;QACrD,MAAM,IAAI,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,MAAM,CAAC,CAAC,MAAM,CAAC;QAClE,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,QAAQ,CAAC,CAAC,MAAM,CAAC;QACtE,MAAM,GAAG,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,KAAK,CAAC,CAAC,MAAM,CAAC;QAChE,OAAO,GAAG,IAAI,cAAc,MAAM,gBAAgB,GAAG,OAAO,CAAC;IAC/D,CAAC;IAEO,WAAW,CAAC,QAAgB;QAClC,IAAI,QAAQ,KAAK,MAAM;YAAE,OAAO,SAAS,CAAC;QAC1C,IAAI,QAAQ,KAAK,QAAQ;YAAE,OAAO,SAAS,CAAC;QAC5C,OAAO,QAAQ,CAAC;IAClB,CAAC;CACF;AAhGD,kDAgGC"}

package/dist/packages/review/providers.d.ts

@@ -0,0 +1,106 @@
+/**
+ * Provider Interfaces for FlowLint
+ *
+ * These abstractions decouple the core linting engine from specific data sources
+ * and output formats, allowing FlowLint to be used in multiple contexts:
+ * - GitHub App (existing)
+ * - CLI (local file analysis)
+ * - Future: Web Dashboard, CI/CD integrations
+ */
+import type { Finding, Graph } from './types';
+import type { FlowLintConfig } from '../config/flowlint-config';
+export type { Finding } from './types';
+/**
+ * LintableFile represents a single file that can be analyzed.
+ * This replaces the GitHub-specific PRFile type for core linting operations.
+ */
+export interface LintableFile {
+    path: string;
+    content: string;
+}
+/**
+ * FileSource provides files for analysis.
+ * Implementations can read from:
+ * - Local filesystem (for CLI)
+ * - GitHub PR (for App)
+ * - Any other source
+ */
+export interface FileSource {
+    /**
+     * Get files matching the provided glob patterns.
+     * @param patterns Include/ignore patterns
+     * @returns Array of lintable files
+     */
+    getFiles(patterns: {
+        include: string[];
+        ignore: string[];
+    }): Promise<LintableFile[]>;
+}
+/**
+ * ConfigProvider loads FlowLint configuration.
+ * Implementations can load from:
+ * - Local .flowlint.yml file (for CLI)
+ * - Remote GitHub repository (for App)
+ * - Default/hardcoded configuration
+ */
+export interface ConfigProvider {
+    /**
+     * Load the FlowLint configuration.
+     * Should return merged config (defaults + overrides).
+     * @returns Loaded configuration
+     */
+    load(): Promise<FlowLintConfig>;
+}
+/**
+ * Reporter generates output from analysis results.
+ * Implementations can output to:
+ * - GitHub Checks API (for App)
+ * - Console/table (for CLI)
+ * - JSON file (for integrations/Web Dashboard)
+ */
+export interface Reporter {
+    /**
+     * Report analysis results.
+     * @param results Array of findings from all files
+     * @returns Promise that resolves when reporting is complete
+     */
+    report(results: AnalysisResult[]): Promise<void>;
+}
+/**
+ * AnalysisResult represents the findings for a single analyzed file.
+ * This is the data structure passed to reporters.
+ */
+export interface AnalysisResult {
+    file: LintableFile;
+    graph: Graph;
+    findings: Finding[];
+    errors?: Array<{
+        error: string;
+        details?: string;
+    }>;
+}
+/**
+ * AnalysisEngine orchestrates the linting process.
+ * It coordinates file source, config loading, rule execution, and reporting.
+ */
+export interface AnalysisEngine {
+    /**
+     * Run a complete analysis on the provided scope.
+     * @returns Summary of the analysis (e.g., total findings, errors)
+     */
+    analyze(): Promise<AnalysisSummary>;
+}
+/**
+ * Summary of a completed analysis.
+ */
+export interface AnalysisSummary {
+    totalFiles: number;
+    totalFindings: number;
+    findingsBySeverity: {
+        must: number;
+        should: number;
+        nit: number;
+    };
+    errors: number;
+    hasBlockingIssues: boolean;
+}

package/dist/packages/review/providers.js

@@ -0,0 +1,12 @@
+"use strict";
+/**
+ * Provider Interfaces for FlowLint
+ *
+ * These abstractions decouple the core linting engine from specific data sources
+ * and output formats, allowing FlowLint to be used in multiple contexts:
+ * - GitHub App (existing)
+ * - CLI (local file analysis)
+ * - Future: Web Dashboard, CI/CD integrations
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+//# sourceMappingURL=providers.js.map

package/dist/packages/review/providers.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"providers.js","sourceRoot":"","sources":["../../../packages/review/providers.ts"],"names":[],"mappings":";AAAA;;;;;;;;GAQG"}

package/dist/packages/review/reporter.d.ts

@@ -0,0 +1,17 @@
+import type { Finding } from './types';
+import type { FlowLintConfig } from '../config/flowlint-config';
+type Conclusion = 'action_required' | 'neutral' | 'success' | 'failure';
+export declare function buildCheckOutput({ findings, cfg, summaryOverride, conclusionOverride, }: {
+    findings: Finding[];
+    cfg: FlowLintConfig;
+    summaryOverride?: string;
+    conclusionOverride?: Conclusion;
+}): {
+    conclusion: Conclusion;
+    output: {
+        title: string;
+        summary: string;
+    };
+};
+export declare function buildAnnotations(findings: Finding[]): any[];
+export {};

package/dist/packages/review/reporter.js

@@ -0,0 +1,62 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.buildCheckOutput = buildCheckOutput;
+exports.buildAnnotations = buildAnnotations;
+function buildCheckOutput({ findings, cfg, summaryOverride, conclusionOverride, }) {
+    const summary = summaryOverride ?? summarize(findings);
+    const conclusion = conclusionOverride ?? inferConclusion(findings);
+    return {
+        conclusion,
+        output: {
+            title: process.env.CHECK_TITLE || 'FlowLint findings',
+            summary,
+        },
+    };
+}
+function buildAnnotations(findings) {
+    const severityOrder = ['must', 'should', 'nit'];
+    const ordered = [...findings].sort((a, b) => severityOrder.indexOf(a.severity) - severityOrder.indexOf(b.severity));
+    return ordered.map((finding) => {
+        const line = finding.line ?? 1;
+        // Build details with documentation link if available
+        const details = [];
+        if (finding.documentationUrl) {
+            details.push(`📚 See examples: ${finding.documentationUrl}`);
+        }
+        if (finding.raw_details) {
+            details.push(finding.raw_details);
+        }
+        const raw_details = details.join('\n').slice(0, 64000) || undefined;
+        return {
+            path: finding.path,
+            start_line: line,
+            end_line: line,
+            annotation_level: mapSeverity(finding.severity),
+            message: `${finding.rule}: ${finding.message}`,
+            raw_details,
+        };
+    });
+}
+function inferConclusion(findings) {
+    if (findings.some((f) => f.severity === 'must'))
+        return 'failure';
+    if (findings.some((f) => f.severity === 'should'))
+        return 'neutral';
+    return 'success';
+}
+function summarize(findings) {
+    if (findings.length === 0)
+        return 'No issues found.';
+    const must = findings.filter((f) => f.severity === 'must').length;
+    const should = findings.filter((f) => f.severity === 'should').length;
+    const nit = findings.filter((f) => f.severity === 'nit').length;
+    return `${must} must-fix, ${should} should-fix, ${nit} nit.`;
+}
+function mapSeverity(severity) {
+    if (severity === 'must')
+        return 'failure';
+    if (severity === 'should')
+        return 'warning';
+    return 'notice';
+}
+//# sourceMappingURL=reporter.js.map

package/dist/packages/review/reporter.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reporter.js","sourceRoot":"","sources":["../../../packages/review/reporter.ts"],"names":[],"mappings":";;AAKA,4CAqBC;AAED,4CAyBC;AAhDD,SAAgB,gBAAgB,CAAC,EAC/B,QAAQ,EACR,GAAG,EACH,eAAe,EACf,kBAAkB,GAMnB;IACC,MAAM,OAAO,GAAG,eAAe,IAAI,SAAS,CAAC,QAAQ,CAAC,CAAC;IACvD,MAAM,UAAU,GAAG,kBAAkB,IAAI,eAAe,CAAC,QAAQ,CAAC,CAAC;IAEnE,OAAO;QACL,UAAU;QACV,MAAM,EAAE;YACN,KAAK,EAAE,OAAO,CAAC,GAAG,CAAC,WAAW,IAAI,mBAAmB;YACrD,OAAO;SACR;KACF,CAAC;AACJ,CAAC;AAED,SAAgB,gBAAgB,CAAC,QAAmB;IAClD,MAAM,aAAa,GAA0B,CAAC,MAAM,EAAE,QAAQ,EAAE,KAAK,CAAC,CAAC;IACvE,MAAM,OAAO,GAAG,CAAC,GAAG,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,aAAa,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC,GAAG,aAAa,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC;IAEpH,OAAO,OAAO,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE;QAC7B,MAAM,IAAI,GAAG,OAAO,CAAC,IAAI,IAAI,CAAC,CAAC;QAC/B,qDAAqD;QACrD,MAAM,OAAO,GAAG,EAAE,CAAC;QACnB,IAAI,OAAO,CAAC,gBAAgB,EAAE,CAAC;YAC7B,OAAO,CAAC,IAAI,CAAC,oBAAoB,OAAO,CAAC,gBAAgB,EAAE,CAAC,CAAC;QAC/D,CAAC;QACD,IAAI,OAAO,CAAC,WAAW,EAAE,CAAC;YACxB,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC;QACpC,CAAC;QACD,MAAM,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,KAAK,CAAC,IAAI,SAAS,CAAC;QAEpE,OAAO;YACL,IAAI,EAAE,OAAO,CAAC,IAAI;YAClB,UAAU,EAAE,IAAI;YAChB,QAAQ,EAAE,IAAI;YACd,gBAAgB,EAAE,WAAW,CAAC,OAAO,CAAC,QAAQ,CAAC;YAC/C,OAAO,EAAE,GAAG,OAAO,CAAC,IAAI,KAAK,OAAO,CAAC,OAAO,EAAE;YAC9C,WAAW;SACZ,CAAC;IACJ,CAAC,CAAC,CAAC;AACL,CAAC;AAED,SAAS,eAAe,CAAC,QAAmB;IAC1C,IAAI,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,MAAM,CAAC;QAAE,OAAO,SAAS,CAAC;IAClE,IAAI,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,QAAQ,CAAC;QAAE,OAAO,SAAS,CAAC;IACpE,OAAO,SAAS,CAAC;AACnB,CAAC;AAED,SAAS,SAAS,CAAC,QAAmB;IACpC,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,kBAAkB,CAAC;IACrD,MAAM,IAAI,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,MAAM,CAAC,CAAC,MAAM,CAAC;IAClE,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,QAAQ,CAAC,CAAC,MAAM,CAAC;IACtE,MAAM,GAAG,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,KAAK,CAAC,CAAC,MAAM,CAAC;IAChE,OAAO,GAAG,IAAI,cAAc,MAAM,gBAAgB,GAAG,OAAO,CAAC;AAC/D,CAAC;AAED,SAAS,WAAW,CAAC,QAA6B;IAChD,IAAI,QAAQ,KAAK,MAAM;QAAE,OAAO,SAAS,CAAC;IAC1C,IAAI,QAAQ,KAAK,QAAQ;QAAE,OAAO,SAAS,CAAC;IAC5C,OAAO,QAAQ,CAAC;AAClB,CAAC"}

package/dist/packages/review/rules/index.d.ts

@@ -0,0 +1,9 @@
+import type { Graph, Finding } from '../types';
+import type { FlowLintConfig } from '../../config/flowlint-config';
+type RuleContext = {
+    path: string;
+    cfg: FlowLintConfig;
+    nodeLines?: Record<string, number>;
+};
+export declare function runAllRules(graph: Graph, ctx: RuleContext): Finding[];
+export {};