flowlint 0.1.2 → 0.3.0

package/dist/packages/observability/metrics.js

@@ -1,194 +0,0 @@
-"use strict";
-/**
- * Prometheus Metrics Definitions for FlowLint
- *
- * This module defines all custom Prometheus metrics for FlowLint observability.
- * Metrics follow Prometheus naming conventions and are designed for low cardinality.
- *
- * @module packages/observability/metrics
- */
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.httpRequestDuration = exports.redisOpsCounter = exports.findingsGeneratedCounter = exports.githubApiCallsCounter = exports.queueDepthGauge = exports.jobDurationHistogram = exports.jobsCompletedCounter = exports.jobsQueuedCounter = exports.webhookCounter = exports.register = void 0;
-exports.getMetrics = getMetrics;
-exports.getContentType = getContentType;
-exports.clearMetrics = clearMetrics;
-exports.getMetric = getMetric;
-const prom_client_1 = require("prom-client");
-// Create dedicated registry for FlowLint metrics
-exports.register = new prom_client_1.Registry();
-// Enable default process and Node.js metrics
-(0, prom_client_1.collectDefaultMetrics)({ register: exports.register });
-// ============================================================================
-// Custom Application Metrics
-// ============================================================================
-/**
- * Metric 1: Webhook Counter
- *
- * Tracks total webhooks received from GitHub, labeled by event type and action.
- *
- * Example usage:
- *   webhookCounter.labels('pull_request', 'opened').inc();
- */
-exports.webhookCounter = new prom_client_1.Counter({
-    name: 'flowlint_webhook_received_total',
-    help: 'Total number of webhooks received from GitHub',
-    labelNames: ['event_type', 'action'],
-    registers: [exports.register]
-});
-/**
- * Metric 2: Jobs Queued Counter
- *
- * Tracks total review jobs added to BullMQ queue.
- *
- * Example usage:
- *   jobsQueuedCounter.labels('acme-corp/api-gateway').inc();
- */
-exports.jobsQueuedCounter = new prom_client_1.Counter({
-    name: 'flowlint_jobs_queued_total',
-    help: 'Total number of review jobs queued for processing',
-    labelNames: ['repo'],
-    registers: [exports.register]
-});
-/**
- * Metric 3: Jobs Completed Counter
- *
- * Tracks total review jobs completed (success or failure).
- *
- * Example usage:
- *   jobsCompletedCounter.labels('success', 'acme-corp/api-gateway').inc();
- */
-exports.jobsCompletedCounter = new prom_client_1.Counter({
-    name: 'flowlint_jobs_completed_total',
-    help: 'Total number of review jobs completed',
-    labelNames: ['status', 'repo'],
-    registers: [exports.register]
-});
-/**
- * Metric 4: Job Duration Histogram
- *
- * Distribution of job processing time in seconds.
- * Buckets optimized for typical workflow review times (0.1s to 120s).
- *
- * Example usage:
- *   const timer = jobDurationHistogram.startTimer({ repo: 'acme-corp/api-gateway' });
- *   // ... process job
- *   timer(); // Records duration
- */
-exports.jobDurationHistogram = new prom_client_1.Histogram({
-    name: 'flowlint_job_duration_seconds',
-    help: 'Duration of job processing in seconds',
-    labelNames: ['repo'],
-    buckets: [0.1, 0.5, 1, 2, 5, 10, 30, 60, 120],
-    registers: [exports.register]
-});
-/**
- * Metric 5: Queue Depth Gauge
- *
- * Current number of jobs in queue by state.
- * Updated periodically by collector (every 10 seconds).
- *
- * Example usage:
- *   queueDepthGauge.set({ state: 'waiting' }, 10);
- */
-exports.queueDepthGauge = new prom_client_1.Gauge({
-    name: 'flowlint_queue_depth',
-    help: 'Current number of jobs in queue by state',
-    labelNames: ['state'],
-    registers: [exports.register]
-});
-/**
- * Metric 6: GitHub API Calls Counter
- *
- * Tracks GitHub API calls made by FlowLint, labeled by method and status.
- *
- * Example usage:
- *   githubApiCallsCounter.labels('GET', '200', '/repos/:owner/:repo/pulls/:number/files').inc();
- */
-exports.githubApiCallsCounter = new prom_client_1.Counter({
-    name: 'flowlint_github_api_calls_total',
-    help: 'Total GitHub API calls made by FlowLint',
-    labelNames: ['method', 'status', 'endpoint'],
-    registers: [exports.register]
-});
-/**
- * Metric 7: Findings Generated Counter
- *
- * Tracks lint findings generated, labeled by rule and severity.
- *
- * Example usage:
- *   findingsGeneratedCounter.labels('rate_limit_retry', 'must').inc();
- */
-exports.findingsGeneratedCounter = new prom_client_1.Counter({
-    name: 'flowlint_findings_generated_total',
-    help: 'Total lint findings generated by rule and severity',
-    labelNames: ['rule', 'severity'],
-    registers: [exports.register]
-});
-/**
- * Metric 8: Redis Operations Counter (Optional)
- *
- * Tracks Redis operations performed by BullMQ.
- *
- * Example usage:
- *   redisOpsCounter.labels('zadd', 'success').inc();
- */
-exports.redisOpsCounter = new prom_client_1.Counter({
-    name: 'flowlint_redis_operations_total',
-    help: 'Total Redis operations performed',
-    labelNames: ['operation', 'status'],
-    registers: [exports.register]
-});
-// ============================================================================
-// HTTP Metrics (Automatic via middleware)
-// ============================================================================
-/**
- * HTTP Request Duration Histogram
- *
- * Automatically populated by metricsMiddleware.
- * Tracks HTTP request latency by method, route, and status.
- */
-exports.httpRequestDuration = new prom_client_1.Histogram({
-    name: 'http_request_duration_seconds',
-    help: 'Duration of HTTP requests in seconds',
-    labelNames: ['method', 'route', 'status'],
-    buckets: [0.001, 0.005, 0.01, 0.05, 0.1, 0.5, 1, 5],
-    registers: [exports.register]
-});
-// ============================================================================
-// Utility Functions
-// ============================================================================
-/**
- * Get all metrics in Prometheus exposition format
- *
- * @returns Prometheus-formatted metrics string
- */
-async function getMetrics() {
-    return exports.register.metrics();
-}
-/**
- * Get metrics content type header
- *
- * @returns Prometheus content type string
- */
-function getContentType() {
-    return exports.register.contentType;
-}
-/**
- * Clear all metrics (useful for testing)
- *
- * WARNING: This removes all metrics from the registry.
- * Only use in test environments.
- */
-function clearMetrics() {
-    exports.register.clear();
-}
-/**
- * Get single metric by name (for testing)
- *
- * @param name - Metric name
- * @returns Metric instance or undefined
- */
-function getMetric(name) {
-    return exports.register.getSingleMetric(name);
-}
-//# sourceMappingURL=metrics.js.map

package/dist/packages/observability/metrics.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"metrics.js","sourceRoot":"","sources":["../../../packages/observability/metrics.ts"],"names":[],"mappings":";AAAA;;;;;;;GAOG;;;AAsKH,gCAEC;AAOD,wCAEC;AAQD,oCAEC;AAQD,8BAEC;AAnMD,6CAAyF;AAEzF,iDAAiD;AACpC,QAAA,QAAQ,GAAG,IAAI,sBAAQ,EAAE,CAAC;AAEvC,6CAA6C;AAC7C,IAAA,mCAAqB,EAAC,EAAE,QAAQ,EAAR,gBAAQ,EAAE,CAAC,CAAC;AAEpC,+EAA+E;AAC/E,6BAA6B;AAC7B,+EAA+E;AAE/E;;;;;;;GAOG;AACU,QAAA,cAAc,GAAG,IAAI,qBAAO,CAAC;IACxC,IAAI,EAAE,iCAAiC;IACvC,IAAI,EAAE,+CAA+C;IACrD,UAAU,EAAE,CAAC,YAAY,EAAE,QAAQ,CAAC;IACpC,SAAS,EAAE,CAAC,gBAAQ,CAAC;CACtB,CAAC,CAAC;AAEH;;;;;;;GAOG;AACU,QAAA,iBAAiB,GAAG,IAAI,qBAAO,CAAC;IAC3C,IAAI,EAAE,4BAA4B;IAClC,IAAI,EAAE,mDAAmD;IACzD,UAAU,EAAE,CAAC,MAAM,CAAC;IACpB,SAAS,EAAE,CAAC,gBAAQ,CAAC;CACtB,CAAC,CAAC;AAEH;;;;;;;GAOG;AACU,QAAA,oBAAoB,GAAG,IAAI,qBAAO,CAAC;IAC9C,IAAI,EAAE,+BAA+B;IACrC,IAAI,EAAE,uCAAuC;IAC7C,UAAU,EAAE,CAAC,QAAQ,EAAE,MAAM,CAAC;IAC9B,SAAS,EAAE,CAAC,gBAAQ,CAAC;CACtB,CAAC,CAAC;AAEH;;;;;;;;;;GAUG;AACU,QAAA,oBAAoB,GAAG,IAAI,uBAAS,CAAC;IAChD,IAAI,EAAE,+BAA+B;IACrC,IAAI,EAAE,uCAAuC;IAC7C,UAAU,EAAE,CAAC,MAAM,CAAC;IACpB,OAAO,EAAE,CAAC,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,GAAG,CAAC;IAC7C,SAAS,EAAE,CAAC,gBAAQ,CAAC;CACtB,CAAC,CAAC;AAEH;;;;;;;;GAQG;AACU,QAAA,eAAe,GAAG,IAAI,mBAAK,CAAC;IACvC,IAAI,EAAE,sBAAsB;IAC5B,IAAI,EAAE,0CAA0C;IAChD,UAAU,EAAE,CAAC,OAAO,CAAC;IACrB,SAAS,EAAE,CAAC,gBAAQ,CAAC;CACtB,CAAC,CAAC;AAEH;;;;;;;GAOG;AACU,QAAA,qBAAqB,GAAG,IAAI,qBAAO,CAAC;IAC/C,IAAI,EAAE,iCAAiC;IACvC,IAAI,EAAE,yCAAyC;IAC/C,UAAU,EAAE,CAAC,QAAQ,EAAE,QAAQ,EAAE,UAAU,CAAC;IAC5C,SAAS,EAAE,CAAC,gBAAQ,CAAC;CACtB,CAAC,CAAC;AAEH;;;;;;;GAOG;AACU,QAAA,wBAAwB,GAAG,IAAI,qBAAO,CAAC;IAClD,IAAI,EAAE,mCAAmC;IACzC,IAAI,EAAE,oDAAoD;IAC1D,UAAU,EAAE,CAAC,MAAM,EAAE,UAAU,CAAC;IAChC,SAAS,EAAE,CAAC,gBAAQ,CAAC;CACtB,CAAC,CAAC;AAEH;;;;;;;GAOG;AACU,QAAA,eAAe,GAAG,IAAI,qBAAO,CAAC;IACzC,IAAI,EAAE,iCAAiC;IACvC,IAAI,EAAE,kCAAkC;IACxC,UAAU,EAAE,CAAC,WAAW,EAAE,QAAQ,CAAC;IACnC,SAAS,EAAE,CAAC,gBAAQ,CAAC;CACtB,CAAC,CAAC;AAEH,+EAA+E;AAC/E,0CAA0C;AAC1C,+EAA+E;AAE/E;;;;;GAKG;AACU,QAAA,mBAAmB,GAAG,IAAI,uBAAS,CAAC;IAC/C,IAAI,EAAE,+BAA+B;IACrC,IAAI,EAAE,sCAAsC;IAC5C,UAAU,EAAE,CAAC,QAAQ,EAAE,OAAO,EAAE,QAAQ,CAAC;IACzC,OAAO,EAAE,CAAC,KAAK,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,CAAC,CAAC;IACnD,SAAS,EAAE,CAAC,gBAAQ,CAAC;CACtB,CAAC,CAAC;AAEH,+EAA+E;AAC/E,oBAAoB;AACpB,+EAA+E;AAE/E;;;;GAIG;AACI,KAAK,UAAU,UAAU;IAC9B,OAAO,gBAAQ,CAAC,OAAO,EAAE,CAAC;AAC5B,CAAC;AAED;;;;GAIG;AACH,SAAgB,cAAc;IAC5B,OAAO,gBAAQ,CAAC,WAAW,CAAC;AAC9B,CAAC;AAED;;;;;GAKG;AACH,SAAgB,YAAY;IAC1B,gBAAQ,CAAC,KAAK,EAAE,CAAC;AACnB,CAAC;AAED;;;;;GAKG;AACH,SAAgB,SAAS,CAAC,IAAY;IACpC,OAAO,gBAAQ,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;AACxC,CAAC"}

package/dist/packages/observability/middleware.d.ts

@@ -1,32 +0,0 @@
-/**
- * Express Middleware for HTTP Metrics Collection
- *
- * This middleware automatically tracks HTTP request metrics including:
- * - Request duration (latency)
- * - Request count by method, route, status
- *
- * @module packages/observability/middleware
- */
-import { Request, Response, NextFunction } from 'express';
-/**
- * Express middleware to collect HTTP request metrics
- *
- * Automatically instruments all HTTP requests with Prometheus metrics.
- * Metrics are labeled by HTTP method, route path, and response status code.
- *
- * Usage:
- *   import { metricsMiddleware } from './packages/observability/middleware';
- *   app.use(metricsMiddleware);
- *
- * @param req - Express request object
- * @param res - Express response object
- * @param next - Express next function
- */
-export declare function metricsMiddleware(req: Request, res: Response, next: NextFunction): void;
-/**
- * Middleware error handler
- *
- * Ensures metrics failures don't crash the application.
- * Logs errors and continues processing.
- */
-export declare function metricsErrorHandler(err: Error, req: Request, res: Response, next: NextFunction): void;

package/dist/packages/observability/middleware.js

@@ -1,58 +0,0 @@
-"use strict";
-/**
- * Express Middleware for HTTP Metrics Collection
- *
- * This middleware automatically tracks HTTP request metrics including:
- * - Request duration (latency)
- * - Request count by method, route, status
- *
- * @module packages/observability/middleware
- */
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.metricsMiddleware = metricsMiddleware;
-exports.metricsErrorHandler = metricsErrorHandler;
-const metrics_1 = require("./metrics");
-/**
- * Express middleware to collect HTTP request metrics
- *
- * Automatically instruments all HTTP requests with Prometheus metrics.
- * Metrics are labeled by HTTP method, route path, and response status code.
- *
- * Usage:
- *   import { metricsMiddleware } from './packages/observability/middleware';
- *   app.use(metricsMiddleware);
- *
- * @param req - Express request object
- * @param res - Express response object
- * @param next - Express next function
- */
-function metricsMiddleware(req, res, next) {
-    // Start timer for this request
-    const timer = metrics_1.httpRequestDuration.startTimer({
-        method: req.method,
-        route: req.route?.path || req.path,
-        status: '0' // Will be updated on response finish
-    });
-    // Record duration when response finishes
-    res.on('finish', () => {
-        try {
-            timer({ status: res.statusCode.toString() });
-        }
-        catch (error) {
-            // Don't let metrics errors crash the application
-            console.error('Failed to record HTTP metrics:', error);
-        }
-    });
-    next();
-}
-/**
- * Middleware error handler
- *
- * Ensures metrics failures don't crash the application.
- * Logs errors and continues processing.
- */
-function metricsErrorHandler(err, req, res, next) {
-    console.error('Metrics middleware error:', err);
-    next(err); // Pass error to next error handler
-}
-//# sourceMappingURL=middleware.js.map

package/dist/packages/observability/middleware.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"middleware.js","sourceRoot":"","sources":["../../../packages/observability/middleware.ts"],"names":[],"mappings":";AAAA;;;;;;;;GAQG;;AAmBH,8CAmBC;AAQD,kDAGC;AA9CD,uCAAgD;AAEhD;;;;;;;;;;;;;GAaG;AACH,SAAgB,iBAAiB,CAAC,GAAY,EAAE,GAAa,EAAE,IAAkB;IAC/E,+BAA+B;IAC/B,MAAM,KAAK,GAAG,6BAAmB,CAAC,UAAU,CAAC;QAC3C,MAAM,EAAE,GAAG,CAAC,MAAM;QAClB,KAAK,EAAE,GAAG,CAAC,KAAK,EAAE,IAAI,IAAI,GAAG,CAAC,IAAI;QAClC,MAAM,EAAE,GAAG,CAAC,qCAAqC;KAClD,CAAC,CAAC;IAEH,yCAAyC;IACzC,GAAG,CAAC,EAAE,CAAC,QAAQ,EAAE,GAAG,EAAE;QACpB,IAAI,CAAC;YACH,KAAK,CAAC,EAAE,MAAM,EAAE,GAAG,CAAC,UAAU,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC;QAC/C,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,iDAAiD;YACjD,OAAO,CAAC,KAAK,CAAC,gCAAgC,EAAE,KAAK,CAAC,CAAC;QACzD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,IAAI,EAAE,CAAC;AACT,CAAC;AAED;;;;;GAKG;AACH,SAAgB,mBAAmB,CAAC,GAAU,EAAE,GAAY,EAAE,GAAa,EAAE,IAAkB;IAC7F,OAAO,CAAC,KAAK,CAAC,2BAA2B,EAAE,GAAG,CAAC,CAAC;IAChD,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,mCAAmC;AAChD,CAAC"}

package/dist/packages/review/analysis-engine.d.ts

@@ -1,19 +0,0 @@
-/**
- * FlowLint Analysis Engine
- *
- * Orchestrates the linting workflow:
- * 1. Load configuration
- * 2. Fetch files to analyze
- * 3. Parse each file into a graph
- * 4. Run rules against the graph
- * 5. Report results
- */
-import type { AnalysisEngine, AnalysisSummary, ConfigProvider, FileSource, Reporter } from './providers';
-export declare class DefaultAnalysisEngine implements AnalysisEngine {
-    private fileSource;
-    private configProvider;
-    private reporter;
-    constructor(fileSource: FileSource, configProvider: ConfigProvider, reporter: Reporter);
-    analyze(): Promise<AnalysisSummary>;
-    private formatParseError;
-}

package/dist/packages/review/analysis-engine.js

@@ -1,106 +0,0 @@
-"use strict";
-/**
- * FlowLint Analysis Engine
- *
- * Orchestrates the linting workflow:
- * 1. Load configuration
- * 2. Fetch files to analyze
- * 3. Parse each file into a graph
- * 4. Run rules against the graph
- * 5. Report results
- */
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.DefaultAnalysisEngine = void 0;
-const parser_n8n_1 = require("./parser-n8n");
-const rules_1 = require("./rules");
-const schemas_1 = require("./schemas");
-const logger_1 = require("../logger");
-const findings_1 = require("./utils/findings");
-class DefaultAnalysisEngine {
-    constructor(fileSource, configProvider, reporter) {
-        this.fileSource = fileSource;
-        this.configProvider = configProvider;
-        this.reporter = reporter;
-    }
-    async analyze() {
-        // 1. Load configuration
-        const cfg = await this.configProvider.load();
-        logger_1.logger.debug({ rules: Object.keys(cfg.rules) }, 'loaded configuration');
-        // 2. Fetch files to analyze
-        const files = await this.fileSource.getFiles(cfg.files);
-        logger_1.logger.info({ count: files.length }, 'fetched files for analysis');
-        // 3. Analyze each file
-        const results = [];
-        let errorCount = 0;
-        for (const file of files) {
-            try {
-                // Parse file to graph
-                const graph = (0, parser_n8n_1.parseN8n)(file.content);
-                logger_1.logger.debug({ file: file.path, nodes: graph.nodes.length }, 'parsed file');
-                // Run all rules
-                const findings = (0, rules_1.runAllRules)(graph, {
-                    path: file.path,
-                    cfg,
-                    nodeLines: graph.meta.nodeLines,
-                });
-                results.push({
-                    file,
-                    graph,
-                    findings,
-                });
-            }
-            catch (error) {
-                errorCount++;
-                const message = error instanceof Error ? error.message : String(error);
-                const raw_details = this.formatParseError(error);
-                logger_1.logger.warn({ file: file.path, error: message }, 'failed to parse file');
-                // Add error finding
-                const errorFinding = {
-                    rule: 'PARSE',
-                    severity: 'must',
-                    path: file.path,
-                    message,
-                    raw_details,
-                    line: 1,
-                };
-                results.push({
-                    file,
-                    graph: { nodes: [], edges: [], meta: {} },
-                    findings: [errorFinding],
-                    errors: [{ error: message, details: raw_details }],
-                });
-            }
-        }
-        // 4. Calculate summary
-        const allFindings = results.flatMap((r) => r.findings);
-        const summary = (0, findings_1.countFindingsBySeverity)(allFindings);
-        // 5. Report results
-        logger_1.logger.info({ totalResults: results.length, totalFindings: summary.total }, 'reporting results');
-        await this.reporter.report(results);
-        // 6. Return summary
-        return {
-            totalFiles: files.length,
-            totalFindings: summary.total,
-            findingsBySeverity: { must: summary.must, should: summary.should, nit: summary.nit },
-            errors: errorCount,
-            hasBlockingIssues: summary.must > 0,
-        };
-    }
-    formatParseError(error) {
-        if (error instanceof schemas_1.ValidationError && Array.isArray(error.errors) && error.errors.length > 0) {
-            return error.errors
-                .map((err) => {
-                const suggestion = err.suggestion ? ` (suggestion: ${err.suggestion})` : '';
-                return `- ${err.path}: ${err.message}${suggestion}`;
-            })
-                .join('\n')
-                .slice(0, 64000);
-        }
-        if (error instanceof Error && error.stack) {
-            return error.stack.slice(0, 64000);
-        }
-        return undefined;
-    }
-}
-exports.DefaultAnalysisEngine = DefaultAnalysisEngine;
-//# sourceMappingURL=analysis-engine.js.map

package/dist/packages/review/analysis-engine.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"analysis-engine.js","sourceRoot":"","sources":["../../../packages/review/analysis-engine.ts"],"names":[],"mappings":";AAAA;;;;;;;;;GASG;;;AAGH,6CAAwC;AACxC,mCAAsC;AACtC,uCAA4C;AAE5C,sCAAmC;AACnC,+CAA2D;AAE3D,MAAa,qBAAqB;IAChC,YACU,UAAsB,EACtB,cAA8B,EAC9B,QAAkB;QAFlB,eAAU,GAAV,UAAU,CAAY;QACtB,mBAAc,GAAd,cAAc,CAAgB;QAC9B,aAAQ,GAAR,QAAQ,CAAU;IACzB,CAAC;IAEJ,KAAK,CAAC,OAAO;QACX,wBAAwB;QACxB,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,cAAc,CAAC,IAAI,EAAE,CAAC;QAC7C,eAAM,CAAC,KAAK,CAAC,EAAE,KAAK,EAAE,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,EAAE,sBAAsB,CAAC,CAAC;QAExE,4BAA4B;QAC5B,MAAM,KAAK,GAAG,MAAM,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;QACxD,eAAM,CAAC,IAAI,CAAC,EAAE,KAAK,EAAE,KAAK,CAAC,MAAM,EAAE,EAAE,4BAA4B,CAAC,CAAC;QAEnE,uBAAuB;QACvB,MAAM,OAAO,GAAqB,EAAE,CAAC;QACrC,IAAI,UAAU,GAAG,CAAC,CAAC;QAEnB,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,IAAI,CAAC;gBACH,sBAAsB;gBACtB,MAAM,KAAK,GAAG,IAAA,qBAAQ,EAAC,IAAI,CAAC,OAAO,CAAC,CAAC;gBACrC,eAAM,CAAC,KAAK,CAAC,EAAE,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE,KAAK,EAAE,KAAK,CAAC,KAAK,CAAC,MAAM,EAAE,EAAE,aAAa,CAAC,CAAC;gBAE5E,gBAAgB;gBAChB,MAAM,QAAQ,GAAG,IAAA,mBAAW,EAAC,KAAK,EAAE;oBAClC,IAAI,EAAE,IAAI,CAAC,IAAI;oBACf,GAAG;oBACH,SAAS,EAAE,KAAK,CAAC,IAAI,CAAC,SAA+C;iBACtE,CAAC,CAAC;gBAEH,OAAO,CAAC,IAAI,CAAC;oBACX,IAAI;oBACJ,KAAK;oBACL,QAAQ;iBACT,CAAC,CAAC;YACL,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,UAAU,EAAE,CAAC;gBACb,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;gBACvE,MAAM,WAAW,GAAG,IAAI,CAAC,gBAAgB,CAAC,KAAK,CAAC,CAAC;gBAEjD,eAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE,KAAK,EAAE,OAAO,EAAE,EAAE,sBAAsB,CAAC,CAAC;gBAEzE,oBAAoB;gBACpB,MAAM,YAAY,GAAY;oBAC5B,IAAI,EAAE,OAAO;oBACb,QAAQ,EAAE,MAAM;oBAChB,IAAI,EAAE,IAAI,CAAC,IAAI;oBACf,OAAO;oBACP,WAAW;oBACX,IAAI,EAAE,CAAC;iBACR,CAAC;gBAEF,OAAO,CAAC,IAAI,CAAC;oBACX,IAAI;oBACJ,KAAK,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE,IAAI,EAAE,EAAE,EAAE;oBACzC,QAAQ,EAAE,CAAC,YAAY,CAAC;oBACxB,MAAM,EAAE,CAAC,EAAE,KAAK,EAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC;iBACnD,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,uBAAuB;QACvB,MAAM,WAAW,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC;QACvD,MAAM,OAAO,GAAG,IAAA,kCAAuB,EAAC,WAAW,CAAC,CAAC;QAErD,oBAAoB;QACpB,eAAM,CAAC,IAAI,CAAC,EAAE,YAAY,EAAE,OAAO,CAAC,MAAM,EAAE,aAAa,EAAE,OAAO,CAAC,KAAK,EAAE,EAAE,mBAAmB,CAAC,CAAC;QACjG,MAAM,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QAEpC,oBAAoB;QACpB,OAAO;YACL,UAAU,EAAE,KAAK,CAAC,MAAM;YACxB,aAAa,EAAE,OAAO,CAAC,KAAK;YAC5B,kBAAkB,EAAE,EAAE,IAAI,EAAE,OAAO,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,CAAC,MAAM,EAAE,GAAG,EAAE,OAAO,CAAC,GAAG,EAAE;YACpF,MAAM,EAAE,UAAU;YAClB,iBAAiB,EAAE,OAAO,CAAC,IAAI,GAAG,CAAC;SACpC,CAAC;IACJ,CAAC;IAEO,gBAAgB,CAAC,KAAc;QACrC,IAAI,KAAK,YAAY,yBAAe,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC/F,OAAO,KAAK,CAAC,MAAM;iBAChB,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE;gBACX,MAAM,UAAU,GAAG,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,iBAAiB,GAAG,CAAC,UAAU,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;gBAC5E,OAAO,KAAK,GAAG,CAAC,IAAI,KAAK,GAAG,CAAC,OAAO,GAAG,UAAU,EAAE,CAAC;YACtD,CAAC,CAAC;iBACD,IAAI,CAAC,IAAI,CAAC;iBACV,KAAK,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC;QACrB,CAAC;QAED,IAAI,KAAK,YAAY,KAAK,IAAI,KAAK,CAAC,KAAK,EAAE,CAAC;YAC1C,OAAO,KAAK,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC;QACrC,CAAC;QAED,OAAO,SAAS,CAAC;IACnB,CAAC;CACF;AAnGD,sDAmGC"}

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

@@ -1,12 +0,0 @@
-/**
- * Main exports from packages/review
- */
-export * from './types';
-export * from './providers';
-export * from './analysis-engine';
-export * from './parser-n8n';
-export * from './rules';
-export * from './reporter';
-export * from './sniffer';
-export * from './schemas';
-export * from './providers/github';

package/dist/packages/review/index.js

@@ -1,29 +0,0 @@
-"use strict";
-/**
- * Main exports from packages/review
- */
-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 __exportStar = (this && this.__exportStar) || function(m, exports) {
-    for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-__exportStar(require("./types"), exports);
-__exportStar(require("./providers"), exports);
-__exportStar(require("./analysis-engine"), exports);
-__exportStar(require("./parser-n8n"), exports);
-__exportStar(require("./rules"), exports);
-__exportStar(require("./reporter"), exports);
-__exportStar(require("./sniffer"), exports);
-__exportStar(require("./schemas"), exports);
-__exportStar(require("./providers/github"), exports);
-//# sourceMappingURL=index.js.map

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

@@ -1 +0,0 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../packages/review/index.ts"],"names":[],"mappings":";AAAA;;GAEG;;;;;;;;;;;;;;;;AAEH,0CAAwB;AACxB,8CAA4B;AAC5B,oDAAkC;AAClC,+CAA6B;AAC7B,0CAAwB;AACxB,6CAA2B;AAC3B,4CAA0B;AAC1B,4CAA0B;AAC1B,qDAAmC"}

package/dist/packages/review/parser-n8n.d.ts

@@ -1,2 +0,0 @@
-import type { Graph } from './types';
-export declare function parseN8n(doc: string): Graph;

package/dist/packages/review/parser-n8n.js

@@ -1,118 +0,0 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.parseN8n = parseN8n;
-const yaml_1 = __importDefault(require("yaml"));
-const schemas_1 = require("./schemas");
-const utils_1 = require("./utils");
-function parseN8n(doc) {
-    let parsed;
-    try {
-        parsed = JSON.parse(doc);
-    }
-    catch {
-        parsed = yaml_1.default.parse(doc);
-    }
-    // Validate workflow structure before parsing
-    (0, schemas_1.validateN8nWorkflow)(parsed);
-    const nodes = parsed.nodes.map((node, idx) => {
-        const nodeId = node.id || node.name || `node-${idx}`;
-        const flags = {
-            continueOnFail: node.continueOnFail,
-            retryOnFail: node.retryOnFail ?? node.settings?.retryOnFail,
-        };
-        const hasFlags = flags.continueOnFail !== undefined || flags.retryOnFail !== undefined;
-        return {
-            id: nodeId,
-            type: node.type,
-            name: node.name,
-            params: node.parameters,
-            cred: node.credentials,
-            flags: hasFlags ? flags : undefined,
-        };
-    });
-    const nameToId = new Map();
-    for (const node of nodes) {
-        if (node.id)
-            nameToId.set(node.id, node.id);
-        if (node.name)
-            nameToId.set(node.name, node.id);
-    }
-    const lines = doc.split(/\r?\n/);
-    const idLine = new Map();
-    const nameLine = new Map();
-    lines.forEach((line, idx) => {
-        const idMatch = line.match(/"id":\s*"([^"]+)"/);
-        if (idMatch)
-            idLine.set(idMatch[1], idx + 1);
-        const nameMatch = line.match(/"name":\s*"([^"]+)"/);
-        if (nameMatch)
-            nameLine.set(nameMatch[1], idx + 1);
-    });
-    const nodeLines = new Map();
-    for (const node of nodes) {
-        const lineNumber = (node.name && nameLine.get(node.name)) || idLine.get(node.id);
-        if (lineNumber) {
-            nodeLines.set(node.id, lineNumber);
-        }
-    }
-    const nodeById = new Map();
-    for (const node of nodes) {
-        nodeById.set(node.id, node);
-    }
-    const resolveEdgeType = (connectionType, outputIndex, sourceType) => {
-        if (connectionType === 'error')
-            return 'error';
-        if (connectionType === 'timeout')
-            return 'timeout';
-        if (connectionType === 'main') {
-            if (typeof outputIndex === 'number' &&
-                outputIndex > 0 &&
-                sourceType &&
-                (0, utils_1.isErrorProneNode)(sourceType)) {
-                return 'error';
-            }
-            return 'success';
-        }
-        return 'success';
-    };
-    const edges = [];
-    Object.entries(parsed.connections || {}).forEach(([from, exits]) => {
-        if (!exits) {
-            return;
-        }
-        const exitChannels = exits;
-        Object.entries(exitChannels).forEach(([exitType, conn]) => {
-            const sourceId = nameToId.get(from) ?? from;
-            const sourceNode = nodeById.get(sourceId);
-            const enqueueEdges = (value, outputIndex) => {
-                (0, utils_1.flattenConnections)(value).forEach((link) => {
-                    if (!link || typeof link !== 'object')
-                        return;
-                    const targetId = nameToId.get(link.node) ?? link.node;
-                    if (!targetId)
-                        return;
-                    const edgeType = resolveEdgeType(exitType, outputIndex, sourceNode?.type);
-                    edges.push({ from: sourceId, to: targetId, on: edgeType });
-                });
-            };
-            if (Array.isArray(conn)) {
-                conn.forEach((entry, index) => enqueueEdges(entry, index));
-            }
-            else {
-                enqueueEdges(conn);
-            }
-        });
-    });
-    return {
-        nodes,
-        edges,
-        meta: {
-            credentials: !!parsed.credentials,
-            nodeLines: Object.fromEntries(nodeLines),
-        },
-    };
-}
-//# sourceMappingURL=parser-n8n.js.map

package/dist/packages/review/parser-n8n.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"parser-n8n.js","sourceRoot":"","sources":["../../../packages/review/parser-n8n.ts"],"names":[],"mappings":";;;;;AAKA,4BAsHC;AA3HD,gDAAwB;AAExB,uCAAgD;AAChD,mCAA+D;AAE/D,SAAgB,QAAQ,CAAC,GAAW;IAClC,IAAI,MAAW,CAAC;IAChB,IAAI,CAAC;QACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IAC3B,CAAC;IAAC,MAAM,CAAC;QACP,MAAM,GAAG,cAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IAC3B,CAAC;IAED,6CAA6C;IAC7C,IAAA,6BAAmB,EAAC,MAAM,CAAC,CAAC;IAE5B,MAAM,KAAK,GAAc,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,IAAS,EAAE,GAAW,EAAE,EAAE;QACnE,MAAM,MAAM,GAAG,IAAI,CAAC,EAAE,IAAI,IAAI,CAAC,IAAI,IAAI,QAAQ,GAAG,EAAE,CAAC;QACrD,MAAM,KAAK,GAAqB;YAC9B,cAAc,EAAE,IAAI,CAAC,cAAc;YACnC,WAAW,EAAE,IAAI,CAAC,WAAW,IAAI,IAAI,CAAC,QAAQ,EAAE,WAAW;SAC5D,CAAC;QACF,MAAM,QAAQ,GAAG,KAAK,CAAC,cAAc,KAAK,SAAS,IAAI,KAAK,CAAC,WAAW,KAAK,SAAS,CAAC;QAEvF,OAAO;YACL,EAAE,EAAE,MAAM;YACV,IAAI,EAAE,IAAI,CAAC,IAAI;YACf,IAAI,EAAE,IAAI,CAAC,IAAI;YACf,MAAM,EAAE,IAAI,CAAC,UAAU;YACvB,IAAI,EAAE,IAAI,CAAC,WAAW;YACtB,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS;SACpC,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAkB,CAAC;IAC3C,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,IAAI,CAAC,EAAE;YAAE,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,EAAE,IAAI,CAAC,EAAE,CAAC,CAAC;QAC5C,IAAI,IAAI,CAAC,IAAI;YAAE,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,CAAC,CAAC;IAClD,CAAC;IAED,MAAM,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IACjC,MAAM,MAAM,GAAG,IAAI,GAAG,EAAkB,CAAC;IACzC,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAkB,CAAC;IAC3C,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,GAAG,EAAE,EAAE;QAC1B,MAAM,OAAO,GAAG,IAAI,CAAC,KAAK,CAAC,mBAAmB,CAAC,CAAC;QAChD,IAAI,OAAO;YAAE,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,GAAG,GAAG,CAAC,CAAC,CAAC;QAC7C,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,qBAAqB,CAAC,CAAC;QACpD,IAAI,SAAS;YAAE,QAAQ,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,GAAG,GAAG,CAAC,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,MAAM,SAAS,GAAG,IAAI,GAAG,EAAkB,CAAC;IAC5C,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,UAAU,GAAG,CAAC,IAAI,CAAC,IAAI,IAAI,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,IAAI,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACjF,IAAI,UAAU,EAAE,CAAC;YACf,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,EAAE,UAAU,CAAC,CAAC;QACrC,CAAC;IACH,CAAC;IAED,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAmB,CAAC;IAC5C,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,EAAE,IAAI,CAAC,CAAC;IAC9B,CAAC;IAED,MAAM,eAAe,GAAG,CACtB,cAAsB,EACtB,WAA+B,EAC/B,UAAmB,EACP,EAAE;QACd,IAAI,cAAc,KAAK,OAAO;YAAE,OAAO,OAAO,CAAC;QAC/C,IAAI,cAAc,KAAK,SAAS;YAAE,OAAO,SAAS,CAAC;QAEnD,IAAI,cAAc,KAAK,MAAM,EAAE,CAAC;YAC9B,IACE,OAAO,WAAW,KAAK,QAAQ;gBAC/B,WAAW,GAAG,CAAC;gBACf,UAAU;gBACV,IAAA,wBAAgB,EAAC,UAAU,CAAC,EAC5B,CAAC;gBACD,OAAO,OAAO,CAAC;YACjB,CAAC;YACD,OAAO,SAAS,CAAC;QACnB,CAAC;QAED,OAAO,SAAS,CAAC;IACnB,CAAC,CAAC;IAEF,MAAM,KAAK,GAAW,EAAE,CAAC;IACzB,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,WAAW,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,EAAE,KAAK,CAAC,EAAE,EAAE;QACjE,IAAI,CAAC,KAAK,EAAE,CAAC;YACX,OAAO;QACT,CAAC;QACD,MAAM,YAAY,GAAG,KAA4B,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,EAAE,IAAI,CAAC,EAAE,EAAE;YACxD,MAAM,QAAQ,GAAG,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC;YAC5C,MAAM,UAAU,GAAG,QAAQ,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;YAE1C,MAAM,YAAY,GAAG,CAAC,KAAU,EAAE,WAAoB,EAAE,EAAE;gBACxD,IAAA,0BAAkB,EAAC,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE;oBACzC,IAAI,CAAC,IAAI,IAAI,OAAO,IAAI,KAAK,QAAQ;wBAAE,OAAO;oBAC9C,MAAM,QAAQ,GAAG,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,IAAI,CAAC;oBACtD,IAAI,CAAC,QAAQ;wBAAE,OAAO;oBAEtB,MAAM,QAAQ,GAAG,eAAe,CAAC,QAAQ,EAAE,WAAW,EAAE,UAAU,EAAE,IAAI,CAAC,CAAC;oBAC1E,KAAK,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,EAAE,QAAQ,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC,CAAC;gBAC7D,CAAC,CAAC,CAAC;YACL,CAAC,CAAC;YAEF,IAAI,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC;gBACxB,IAAI,CAAC,OAAO,CAAC,CAAC,KAAK,EAAE,KAAK,EAAE,EAAE,CAAC,YAAY,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC,CAAC;YAC7D,CAAC;iBAAM,CAAC;gBACN,YAAY,CAAC,IAAI,CAAC,CAAC;YACrB,CAAC;QACH,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,OAAO;QACL,KAAK;QACL,KAAK;QACL,IAAI,EAAE;YACJ,WAAW,EAAE,CAAC,CAAC,MAAM,CAAC,WAAW;YACjC,SAAS,EAAE,MAAM,CAAC,WAAW,CAAC,SAAS,CAAC;SACzC;KACF,CAAC;AACJ,CAAC"}

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

@@ -1,62 +0,0 @@
-/**
- * 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;
-}