flowlint 0.5.1 → 0.6.0

package/dist/packages/review/rules/rule-utils.d.ts

@@ -1,36 +0,0 @@
-import type { Graph, Finding, NodeRef, FindingSeverity } from '../types';
-import type { FlowLintConfig } from '../../config/flowlint-config';
-type Rule = string;
-type RuleContext = {
-    path: string;
-    cfg: FlowLintConfig;
-    nodeLines?: Record<string, number>;
-};
-type RuleRunner = (graph: Graph, ctx: RuleContext) => Finding[];
-type NodeRuleLogic = (node: NodeRef, graph: Graph, ctx: RuleContext) => Finding | Finding[] | null;
-/**
- * A higher-order function to create a rule that iterates over each node in the graph.
- * It abstracts the boilerplate of checking if the rule is enabled and iterating through nodes.
- *
- * @param ruleId - The ID of the rule (e.g., 'R1').
- * @param configKey - The key in the FlowLintConfig rules object.
- * @param logic - The function containing the core logic to be executed for each node.
- * @returns A RuleRunner function.
- */
-export declare function createNodeRule(ruleId: Rule, configKey: keyof FlowLintConfig['rules'], logic: NodeRuleLogic): RuleRunner;
-type HardcodedStringRuleOptions = {
-    ruleId: Rule;
-    severity: FindingSeverity;
-    configKey: 'secrets' | 'config_literals';
-    messageFn: (node: NodeRef, value: string) => string;
-    details: string;
-};
-/**
- * Creates a rule that checks for hardcoded strings in node parameters based on a denylist of regex patterns.
- * This is used to create R4 (Secrets) and R9 (Config Literals).
- *
- * @param options - The configuration for the hardcoded string rule.
- * @returns A RuleRunner function.
- */
-export declare function createHardcodedStringRule({ ruleId, severity, configKey, messageFn, details, }: HardcodedStringRuleOptions): RuleRunner;
-export {};

package/dist/packages/review/rules/rule-utils.js

@@ -1,75 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.createNodeRule = createNodeRule;
-exports.createHardcodedStringRule = createHardcodedStringRule;
-const utils_1 = require("../utils");
-/**
- * A higher-order function to create a rule that iterates over each node in the graph.
- * It abstracts the boilerplate of checking if the rule is enabled and iterating through nodes.
- *
- * @param ruleId - The ID of the rule (e.g., 'R1').
- * @param configKey - The key in the FlowLintConfig rules object.
- * @param logic - The function containing the core logic to be executed for each node.
- * @returns A RuleRunner function.
- */
-function createNodeRule(ruleId, configKey, logic) {
-    return (graph, ctx) => {
-        const ruleConfig = ctx.cfg.rules[configKey];
-        if (!ruleConfig?.enabled) {
-            return [];
-        }
-        const findings = [];
-        for (const node of graph.nodes) {
-            const result = logic(node, graph, ctx);
-            if (result) {
-                if (Array.isArray(result)) {
-                    findings.push(...result);
-                }
-                else {
-                    findings.push(result);
-                }
-            }
-        }
-        return findings;
-    };
-}
-/**
- * Creates a rule that checks for hardcoded strings in node parameters based on a denylist of regex patterns.
- * This is used to create R4 (Secrets) and R9 (Config Literals).
- *
- * @param options - The configuration for the hardcoded string rule.
- * @returns A RuleRunner function.
- */
-function createHardcodedStringRule({ ruleId, severity, configKey, messageFn, details, }) {
-    const logic = (node, graph, ctx) => {
-        const cfg = ctx.cfg.rules[configKey];
-        if (!cfg.denylist_regex?.length) {
-            return null;
-        }
-        const regexes = cfg.denylist_regex.map((pattern) => (0, utils_1.toRegex)(pattern));
-        const findings = [];
-        const strings = (0, utils_1.collectStrings)(node.params);
-        for (const value of strings) {
-            // Ignore expressions and empty strings
-            if (!value || value.includes('{{')) {
-                continue;
-            }
-            if (regexes.some((regex) => regex.test(value))) {
-                findings.push({
-                    rule: ruleId,
-                    severity,
-                    path: ctx.path,
-                    message: messageFn(node, value),
-                    nodeId: node.id,
-                    line: ctx.nodeLines?.[node.id],
-                    raw_details: details,
-                });
-                // Only report one finding per node to avoid noise
-                break;
-            }
-        }
-        return findings;
-    };
-    return createNodeRule(ruleId, configKey, logic);
-}
-//# sourceMappingURL=rule-utils.js.map

package/dist/packages/review/rules/rule-utils.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"rule-utils.js","sourceRoot":"","sources":["../../../../packages/review/rules/rule-utils.ts"],"names":[],"mappings":";;AAkBA,wCAwBC;AAiBD,8DAyCC;AAlGD,oCAAmD;AAOnD;;;;;;;;GAQG;AACH,SAAgB,cAAc,CAC5B,MAAY,EACZ,SAAwC,EACxC,KAAoB;IAEpB,OAAO,CAAC,KAAY,EAAE,GAAgB,EAAa,EAAE;QACnD,MAAM,UAAU,GAAG,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAA0B,CAAC;QACrE,IAAI,CAAC,UAAU,EAAE,OAAO,EAAE,CAAC;YACzB,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,MAAM,QAAQ,GAAc,EAAE,CAAC;QAC/B,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,KAAK,EAAE,CAAC;YAC/B,MAAM,MAAM,GAAG,KAAK,CAAC,IAAI,EAAE,KAAK,EAAE,GAAG,CAAC,CAAC;YACvC,IAAI,MAAM,EAAE,CAAC;gBACX,IAAI,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,CAAC;oBAC1B,QAAQ,CAAC,IAAI,CAAC,GAAG,MAAM,CAAC,CAAC;gBAC3B,CAAC;qBAAM,CAAC;oBACN,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;gBACxB,CAAC;YACH,CAAC;QACH,CAAC;QACD,OAAO,QAAQ,CAAC;IAClB,CAAC,CAAC;AACJ,CAAC;AAUD;;;;;;GAMG;AACH,SAAgB,yBAAyB,CAAC,EACxC,MAAM,EACN,QAAQ,EACR,SAAS,EACT,SAAS,EACT,OAAO,GACoB;IAC3B,MAAM,KAAK,GAAkB,CAAC,IAAI,EAAE,KAAK,EAAE,GAAG,EAAE,EAAE;QAChD,MAAM,GAAG,GAAG,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;QACrC,IAAI,CAAC,GAAG,CAAC,cAAc,EAAE,MAAM,EAAE,CAAC;YAChC,OAAO,IAAI,CAAC;QACd,CAAC;QACD,MAAM,OAAO,GAAG,GAAG,CAAC,cAAc,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,IAAA,eAAO,EAAC,OAAO,CAAC,CAAC,CAAC;QAEtE,MAAM,QAAQ,GAAc,EAAE,CAAC;QAC/B,MAAM,OAAO,GAAG,IAAA,sBAAc,EAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QAE5C,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;YAC5B,uCAAuC;YACvC,IAAI,CAAC,KAAK,IAAI,KAAK,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;gBACnC,SAAS;YACX,CAAC;YAED,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC;gBAC/C,QAAQ,CAAC,IAAI,CAAC;oBACZ,IAAI,EAAE,MAAM;oBACZ,QAAQ;oBACR,IAAI,EAAE,GAAG,CAAC,IAAI;oBACd,OAAO,EAAE,SAAS,CAAC,IAAI,EAAE,KAAK,CAAC;oBAC/B,MAAM,EAAE,IAAI,CAAC,EAAE;oBACf,IAAI,EAAE,GAAG,CAAC,SAAS,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC9B,WAAW,EAAE,OAAO;iBACrB,CAAC,CAAC;gBACH,kDAAkD;gBAClD,MAAM;YACR,CAAC;QACH,CAAC;QACD,OAAO,QAAQ,CAAC;IAClB,CAAC,CAAC;IAEF,OAAO,cAAc,CAAC,MAAM,EAAE,SAAS,EAAE,KAAK,CAAC,CAAC;AAClD,CAAC"}

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

@@ -1,17 +0,0 @@
-export declare class ValidationError extends Error {
-    errors: Array<{
-        path: string;
-        message: string;
-        suggestion?: string;
-    }>;
-    constructor(errors: Array<{
-        path: string;
-        message: string;
-        suggestion?: string;
-    }>);
-}
-/**
- * Validate n8n workflow structure
- * Throws ValidationError with detailed messages if validation fails
- */
-export declare function validateN8nWorkflow(data: any): void;

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

@@ -1,167 +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.ValidationError = void 0;
-exports.validateN8nWorkflow = validateN8nWorkflow;
-const ajv_1 = __importDefault(require("ajv"));
-const ajv_formats_1 = __importDefault(require("ajv-formats"));
-const n8n_workflow_schema_json_1 = __importDefault(require("./n8n-workflow.schema.json"));
-const utils_1 = require("../utils");
-// Custom error class for validation failures
-class ValidationError extends Error {
-    constructor(errors) {
-        super(`Workflow validation failed: ${errors.length} error(s)`);
-        this.errors = errors;
-        this.name = 'ValidationError';
-    }
-}
-exports.ValidationError = ValidationError;
-// Dummy validator that always passes
-const createDummyValidator = () => {
-    const v = () => true;
-    v.errors = [];
-    return v;
-};
-// Singleton instance
-let validatorInstance = null;
-function getValidator() {
-    if (validatorInstance)
-        return validatorInstance;
-    // Detect Node.js environment safely
-    // Use optional chaining to satisfy SonarQube
-    const isNode = typeof process !== 'undefined' && process?.versions?.node != null;
-    if (isNode) {
-        try {
-            const ajv = new ajv_1.default({
-                allErrors: true,
-                strict: false,
-                verbose: true,
-                code: { source: true, es5: true }
-            });
-            (0, ajv_formats_1.default)(ajv);
-            validatorInstance = ajv.compile(n8n_workflow_schema_json_1.default);
-        }
-        catch (error) {
-            // Fallback to dummy validator if compilation fails (e.g. due to strict CSP in some environments)
-            console.warn('Failed to compile JSON schema validator, falling back to dummy validator:', error);
-            validatorInstance = createDummyValidator();
-        }
-    }
-    else {
-        validatorInstance = createDummyValidator();
-    }
-    return validatorInstance;
-}
-/**
- * Throws a ValidationError if the provided set contains items.
- * Centralizes the pattern of checking validation results and throwing errors.
- *
- * @param items - Set of items that represent validation failures
- * @param config - Configuration for building error messages
- * @throws ValidationError if items set is not empty
- */
-function throwIfInvalid(items, config) {
-    if (items.size > 0) {
-        const errors = (0, utils_1.buildValidationErrors)(items, config);
-        throw new ValidationError(errors);
-    }
-}
-/**
- * Check for duplicate node IDs in the workflow
- */
-function checkDuplicateNodeIds(data) {
-    if (!Array.isArray(data.nodes))
-        return;
-    const seen = new Set();
-    const duplicates = new Set();
-    for (const node of data.nodes) {
-        if (node.id && seen.has(node.id)) {
-            duplicates.add(node.id);
-        }
-        if (node.id) {
-            seen.add(node.id);
-        }
-    }
-    throwIfInvalid(duplicates, {
-        path: 'nodes[].id',
-        messageTemplate: (id) => `Duplicate node ID: "${id}"`,
-        suggestionTemplate: (id) => `Each node must have a unique ID. Remove or rename the duplicate node with ID "${id}".`,
-    });
-}
-/**
- * Check for orphaned connections (references to non-existent nodes)
- */
-function checkOrphanedConnections(data) {
-    if (!data.connections || !Array.isArray(data.nodes))
-        return;
-    const nodeIds = new Set();
-    const nodeNames = new Set();
-    // Collect all node IDs and names
-    for (const node of data.nodes) {
-        if (node.id)
-            nodeIds.add(node.id);
-        if (node.name)
-            nodeNames.add(node.name);
-    }
-    const orphanedRefs = new Set();
-    // Check all connection targets
-    Object.entries(data.connections).forEach(([sourceId, channels]) => {
-        // Check if source exists
-        if (!nodeIds.has(sourceId) && !nodeNames.has(sourceId)) {
-            orphanedRefs.add(sourceId);
-        }
-        // Check targets
-        if (typeof channels === 'object' && channels !== null) {
-            Object.values(channels).forEach((connArray) => {
-                const flatConnections = (0, utils_1.flattenConnections)(connArray);
-                flatConnections.forEach((conn) => {
-                    if (conn?.node) {
-                        if (!nodeIds.has(conn.node) && !nodeNames.has(conn.node)) {
-                            orphanedRefs.add(conn.node);
-                        }
-                    }
-                });
-            });
-        }
-    });
-    throwIfInvalid(orphanedRefs, {
-        path: 'connections',
-        messageTemplate: (ref) => `Orphaned connection reference: "${ref}"`,
-        suggestionTemplate: (ref) => `Connection references node "${ref}" which does not exist. Add the missing node or remove the invalid connection.`,
-    });
-}
-/**
- * Validate n8n workflow structure
- * Throws ValidationError with detailed messages if validation fails
- */
-function validateN8nWorkflow(data) {
-    const validate = getValidator();
-    // Basic schema validation
-    if (!validate(data)) {
-        const errors = (validate.errors || []).map((err) => {
-            const path = err.instancePath || err.schemaPath;
-            const message = err.message || 'Validation error';
-            let suggestion = '';
-            // Provide helpful suggestions based on error type
-            if (err.keyword === 'required') {
-                const missing = err.params?.missingProperty;
-                suggestion = `Add the required field "${missing}" to the workflow.`;
-            }
-            else if (err.keyword === 'type') {
-                const expected = err.params?.type;
-                suggestion = `The field should be of type "${expected}".`;
-            }
-            else if (err.keyword === 'minLength') {
-                suggestion = 'This field cannot be empty.';
-            }
-            return { path, message, suggestion };
-        });
-        throw new ValidationError(errors);
-    }
-    // Additional custom validations
-    checkDuplicateNodeIds(data);
-    checkOrphanedConnections(data);
-}
-//# sourceMappingURL=index.js.map

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

@@ -1 +0,0 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../packages/review/schemas/index.ts"],"names":[],"mappings":";;;;;;AA2JA,kDA8BC;AAzLD,8CAAiD;AACjD,8DAAqC;AACrC,0FAAwD;AACxD,oCAAqE;AAErE,6CAA6C;AAC7C,MAAa,eAAgB,SAAQ,KAAK;IACxC,YACS,MAIL;QAEF,KAAK,CAAC,+BAA+B,MAAM,CAAC,MAAM,WAAW,CAAC,CAAC;QANxD,WAAM,GAAN,MAAM,CAIX;QAGF,IAAI,CAAC,IAAI,GAAG,iBAAiB,CAAC;IAChC,CAAC;CACF;AAXD,0CAWC;AAED,qCAAqC;AACrC,MAAM,oBAAoB,GAAG,GAAqB,EAAE;IAClD,MAAM,CAAC,GAAQ,GAAG,EAAE,CAAC,IAAI,CAAC;IAC1B,CAAC,CAAC,MAAM,GAAG,EAAE,CAAC;IACd,OAAO,CAAqB,CAAC;AAC/B,CAAC,CAAC;AAEF,qBAAqB;AACrB,IAAI,iBAAiB,GAA4B,IAAI,CAAC;AAEtD,SAAS,YAAY;IACnB,IAAI,iBAAiB;QAAE,OAAO,iBAAiB,CAAC;IAEhD,oCAAoC;IACpC,6CAA6C;IAC7C,MAAM,MAAM,GAAG,OAAO,OAAO,KAAK,WAAW,IAAI,OAAO,EAAE,QAAQ,EAAE,IAAI,IAAI,IAAI,CAAC;IAEjF,IAAI,MAAM,EAAE,CAAC;QACX,IAAI,CAAC;YACH,MAAM,GAAG,GAAG,IAAI,aAAG,CAAC;gBAClB,SAAS,EAAE,IAAI;gBACf,MAAM,EAAE,KAAK;gBACb,OAAO,EAAE,IAAI;gBACb,IAAI,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,EAAE,IAAI,EAAE;aAClC,CAAC,CAAC;YACH,IAAA,qBAAU,EAAC,GAAG,CAAC,CAAC;YAChB,iBAAiB,GAAG,GAAG,CAAC,OAAO,CAAC,kCAAc,CAAC,CAAC;QAClD,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,iGAAiG;YACjG,OAAO,CAAC,IAAI,CAAC,2EAA2E,EAAE,KAAK,CAAC,CAAC;YACjG,iBAAiB,GAAG,oBAAoB,EAAE,CAAC;QAC7C,CAAC;IACH,CAAC;SAAM,CAAC;QACN,iBAAiB,GAAG,oBAAoB,EAAE,CAAC;IAC7C,CAAC;IAED,OAAO,iBAAiB,CAAC;AAC3B,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,cAAc,CACrB,KAAa,EACb,MAIC;IAED,IAAI,KAAK,CAAC,IAAI,GAAG,CAAC,EAAE,CAAC;QACnB,MAAM,MAAM,GAAG,IAAA,6BAAqB,EAAC,KAAK,EAAE,MAAM,CAAC,CAAC;QACpD,MAAM,IAAI,eAAe,CAAC,MAAM,CAAC,CAAC;IACpC,CAAC;AACH,CAAC;AAED;;GAEG;AACH,SAAS,qBAAqB,CAAC,IAAS;IACtC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC;QAAE,OAAO;IAEvC,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;IAC/B,MAAM,UAAU,GAAG,IAAI,GAAG,EAAU,CAAC;IAErC,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,KAAK,EAAE,CAAC;QAC9B,IAAI,IAAI,CAAC,EAAE,IAAI,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,CAAC;YACjC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAC1B,CAAC;QACD,IAAI,IAAI,CAAC,EAAE,EAAE,CAAC;YACZ,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACpB,CAAC;IACH,CAAC;IAED,cAAc,CAAC,UAAU,EAAE;QACzB,IAAI,EAAE,YAAY;QAClB,eAAe,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,uBAAuB,EAAE,GAAG;QACrD,kBAAkB,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,iFAAiF,EAAE,IAAI;KACpH,CAAC,CAAC;AACL,CAAC;AAED;;GAEG;AACH,SAAS,wBAAwB,CAAC,IAAS;IACzC,IAAI,CAAC,IAAI,CAAC,WAAW,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC;QAAE,OAAO;IAE5D,MAAM,OAAO,GAAG,IAAI,GAAG,EAAU,CAAC;IAClC,MAAM,SAAS,GAAG,IAAI,GAAG,EAAU,CAAC;IAEpC,iCAAiC;IACjC,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,KAAK,EAAE,CAAC;QAC9B,IAAI,IAAI,CAAC,EAAE;YAAE,OAAO,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAClC,IAAI,IAAI,CAAC,IAAI;YAAE,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC1C,CAAC;IAED,MAAM,YAAY,GAAG,IAAI,GAAG,EAAU,CAAC;IAEvC,+BAA+B;IAC/B,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,EAAE,QAAQ,CAAC,EAAE,EAAE;QAChE,yBAAyB;QACzB,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,QAAQ,CAAC,EAAE,CAAC;YACvD,YAAY,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;QAC7B,CAAC;QAED,gBAAgB;QAChB,IAAI,OAAO,QAAQ,KAAK,QAAQ,IAAI,QAAQ,KAAK,IAAI,EAAE,CAAC;YACtD,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,OAAO,CAAC,CAAC,SAAc,EAAE,EAAE;gBACjD,MAAM,eAAe,GAAG,IAAA,0BAAkB,EAAC,SAAS,CAAC,CAAC;gBACtD,eAAe,CAAC,OAAO,CAAC,CAAC,IAAS,EAAE,EAAE;oBACpC,IAAI,IAAI,EAAE,IAAI,EAAE,CAAC;wBACf,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;4BACzD,YAAY,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;wBAC9B,CAAC;oBACH,CAAC;gBACH,CAAC,CAAC,CAAC;YACL,CAAC,CAAC,CAAC;QACL,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,cAAc,CAAC,YAAY,EAAE;QAC3B,IAAI,EAAE,aAAa;QACnB,eAAe,EAAE,CAAC,GAAG,EAAE,EAAE,CAAC,mCAAmC,GAAG,GAAG;QACnE,kBAAkB,EAAE,CAAC,GAAG,EAAE,EAAE,CAAC,+BAA+B,GAAG,gFAAgF;KAChJ,CAAC,CAAC;AACL,CAAC;AAED;;;GAGG;AACH,SAAgB,mBAAmB,CAAC,IAAS;IAC3C,MAAM,QAAQ,GAAG,YAAY,EAAE,CAAC;IAEhC,0BAA0B;IAC1B,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;QACpB,MAAM,MAAM,GAAG,CAAC,QAAQ,CAAC,MAAM,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,GAAQ,EAAE,EAAE;YACtD,MAAM,IAAI,GAAG,GAAG,CAAC,YAAY,IAAI,GAAG,CAAC,UAAU,CAAC;YAChD,MAAM,OAAO,GAAG,GAAG,CAAC,OAAO,IAAI,kBAAkB,CAAC;YAClD,IAAI,UAAU,GAAG,EAAE,CAAC;YAEpB,kDAAkD;YAClD,IAAI,GAAG,CAAC,OAAO,KAAK,UAAU,EAAE,CAAC;gBAC/B,MAAM,OAAO,GAAG,GAAG,CAAC,MAAM,EAAE,eAAe,CAAC;gBAC5C,UAAU,GAAG,2BAA2B,OAAO,oBAAoB,CAAC;YACtE,CAAC;iBAAM,IAAI,GAAG,CAAC,OAAO,KAAK,MAAM,EAAE,CAAC;gBAClC,MAAM,QAAQ,GAAG,GAAG,CAAC,MAAM,EAAE,IAAI,CAAC;gBAClC,UAAU,GAAG,gCAAgC,QAAQ,IAAI,CAAC;YAC5D,CAAC;iBAAM,IAAI,GAAG,CAAC,OAAO,KAAK,WAAW,EAAE,CAAC;gBACvC,UAAU,GAAG,6BAA6B,CAAC;YAC7C,CAAC;YAED,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,MAAM,IAAI,eAAe,CAAC,MAAM,CAAC,CAAC;IACpC,CAAC;IAED,gCAAgC;IAChC,qBAAqB,CAAC,IAAI,CAAC,CAAC;IAC5B,wBAAwB,CAAC,IAAI,CAAC,CAAC;AACjC,CAAC"}

package/dist/packages/review/schemas/n8n-workflow.schema.json

@@ -1,177 +0,0 @@
-{
-    "$schema": "http://json-schema.org/draft-07/schema#",
-    "$id": "https://flowlint.dev/schemas/n8n-workflow.json",
-    "title": "n8n Workflow Schema",
-    "description": "JSON Schema for n8n workflow files (v1.x)",
-    "type": "object",
-    "required": ["nodes", "connections"],
-    "properties": {
-        "name": {
-            "type": "string",
-            "description": "Workflow name"
-        },
-        "nodes": {
-            "type": "array",
-            "description": "Array of workflow nodes",
-            "items": {
-                "type": "object",
-                "required": ["type", "name"],
-                "properties": {
-                    "id": {
-                        "type": "string",
-                        "minLength": 1,
-                        "description": "Unique node identifier"
-                    },
-                    "type": {
-                        "type": "string",
-                        "minLength": 1,
-                        "description": "Node type (e.g., n8n-nodes-base.httpRequest)"
-                    },
-                    "name": {
-                        "type": "string",
-                        "minLength": 1,
-                        "description": "Human-readable node name"
-                    },
-                    "parameters": {
-                        "type": "object",
-                        "description": "Node-specific configuration parameters"
-                    },
-                    "credentials": {
-                        "type": "object",
-                        "description": "Credential references for this node"
-                    },
-                    "position": {
-                        "type": "array",
-                        "description": "X,Y coordinates for UI placement",
-                        "items": {
-                            "type": "number"
-                        },
-                        "minItems": 2,
-                        "maxItems": 2
-                    },
-                    "continueOnFail": {
-                        "type": "boolean",
-                        "description": "Whether to continue execution on node failure"
-                    },
-                    "disabled": {
-                        "type": "boolean",
-                        "description": "Whether the node is disabled"
-                    },
-                    "notesInFlow": {
-                        "type": "boolean"
-                    },
-                    "notes": {
-                        "type": "string"
-                    },
-                    "typeVersion": {
-                        "type": "number",
-                        "description": "Version of the node type"
-                    }
-                },
-                "additionalProperties": true
-            }
-        },
-        "connections": {
-            "type": "object",
-            "description": "Map of node connections (source node ID -> connection details)",
-            "patternProperties": {
-                "^.*$": {
-                    "type": "object",
-                    "description": "Connection channels for a source node",
-                    "patternProperties": {
-                        "^(main|error|timeout|.*?)$": {
-                            "description": "Connection array for this channel",
-                            "oneOf": [
-                                {
-                                    "type": "array",
-                                    "items": {
-                                        "type": "object",
-                                        "required": ["node"],
-                                        "properties": {
-                                            "node": {
-                                                "type": "string",
-                                                "description": "Target node ID or name"
-                                            },
-                                            "type": {
-                                                "type": "string",
-                                                "description": "Connection type"
-                                            },
-                                            "index": {
-                                                "type": "number",
-                                                "description": "Output index"
-                                            }
-                                        }
-                                    }
-                                },
-                                {
-                                    "type": "array",
-                                    "items": {
-                                        "type": "array",
-                                        "items": {
-                                            "type": "object",
-                                            "required": ["node"],
-                                            "properties": {
-                                                "node": {
-                                                    "type": "string"
-                                                },
-                                                "type": {
-                                                    "type": "string"
-                                                },
-                                                "index": {
-                                                    "type": "number"
-                                                }
-                                            }
-                                        }
-                                    }
-                                }
-                            ]
-                        }
-                    }
-                }
-            }
-        },
-        "active": {
-            "type": "boolean",
-            "description": "Whether the workflow is active"
-        },
-        "settings": {
-            "type": "object",
-            "description": "Workflow settings"
-        },
-        "tags": {
-            "type": "array",
-            "description": "Workflow tags",
-            "items": {
-                "oneOf": [
-                    { "type": "string" },
-                    {
-                        "type": "object",
-                        "required": ["name"],
-                        "properties": {
-                            "id": { "type": "string" },
-                            "name": { "type": "string" }
-                        },
-                        "additionalProperties": true
-                    }
-                ]
-            }
-        },
-        "pinData": {
-            "type": "object",
-            "description": "Pinned execution data for testing"
-        },
-        "versionId": {
-            "type": "string",
-            "description": "Workflow version identifier"
-        },
-        "id": {
-            "type": ["string", "number"],
-            "description": "Workflow ID"
-        },
-        "meta": {
-            "type": "object",
-            "description": "Metadata"
-        }
-    },
-    "additionalProperties": true
-}

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

@@ -1,15 +0,0 @@
-import type { Octokit } from 'octokit';
-import type { PRFile } from './types';
-export type GlobSet = {
-    include: string[];
-    ignore: string[];
-};
-export declare function pickTargets(files: PRFile[], globs: GlobSet): PRFile[];
-export type FetchResult = {
-    contents: Map<string, string>;
-    errors: Array<{
-        filename: string;
-        error: string;
-    }>;
-};
-export declare function fetchRawFiles(gh: Octokit, repoFull: string, targets: PRFile[]): Promise<FetchResult>;

package/dist/packages/review/sniffer.js

@@ -1,47 +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.pickTargets = pickTargets;
-exports.fetchRawFiles = fetchRawFiles;
-const logger_1 = require("../logger");
-const micromatch_1 = __importDefault(require("micromatch"));
-function 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;
-    });
-}
-async function fetchRawFiles(gh, repoFull, targets) {
-    const [owner, repo] = repoFull.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 gh.request('GET /repos/{owner}/{repo}/git/blobs/{file_sha}', {
-                owner,
-                repo,
-                file_sha: file.sha,
-            });
-            const raw = Buffer.from(blob.content, 'base64').toString('utf8');
-            contents.set(file.filename, raw);
-        }
-        catch (error) {
-            // Handle transient errors (404, timeouts, rate limits) gracefully
-            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 };
-}
-//# sourceMappingURL=sniffer.js.map

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

@@ -1 +0,0 @@
-{"version":3,"file":"sniffer.js","sourceRoot":"","sources":["../../../packages/review/sniffer.ts"],"names":[],"mappings":";;;;;AAOA,kCASC;AAOD,sCAgCC;AArDD,sCAAmC;AACnC,4DAAoC;AAIpC,SAAgB,WAAW,CAAC,KAAe,EAAE,KAAc;IACzD,OAAO,KAAK;SACT,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,MAAM,KAAK,SAAS,CAAC;SAC3C,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE;QACf,MAAM,UAAU,GAAG,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;QACrD,MAAM,QAAQ,GAAG,oBAAU,CAAC,OAAO,CAAC,UAAU,EAAE,KAAK,CAAC,OAAO,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,CAAC,CAAC;QAC9E,MAAM,OAAO,GAAG,oBAAU,CAAC,OAAO,CAAC,UAAU,EAAE,KAAK,CAAC,MAAM,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5E,OAAO,QAAQ,IAAI,CAAC,OAAO,CAAC;IAC9B,CAAC,CAAC,CAAC;AACP,CAAC;AAOM,KAAK,UAAU,aAAa,CACjC,EAAW,EACX,QAAgB,EAChB,OAAiB;IAEjB,MAAM,CAAC,KAAK,EAAE,IAAI,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IAC1C,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAkB,CAAC;IAC3C,MAAM,MAAM,GAA+C,EAAE,CAAC;IAE9D,KAAK,MAAM,IAAI,IAAI,OAAO,EAAE,CAAC;QAC3B,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC;YACd,MAAM,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,QAAQ,EAAE,KAAK,EAAE,mCAAmC,EAAE,CAAC,CAAC;YACrF,SAAS;QACX,CAAC;QAED,IAAI,CAAC;YACH,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,GAAG,MAAM,EAAE,CAAC,OAAO,CAAC,gDAAgD,EAAE;gBACxF,KAAK;gBACL,IAAI;gBACJ,QAAQ,EAAE,IAAI,CAAC,GAAG;aACnB,CAAC,CAAC;YACH,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,QAAQ,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;YACjE,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC;QACnC,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,kEAAkE;YAClE,MAAM,QAAQ,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;YACxE,eAAM,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,QAAQ,EAAE,KAAK,EAAE,QAAQ,EAAE,EAAE,sBAAsB,CAAC,CAAC;YAClF,MAAM,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,QAAQ,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,CAAC;QAC5D,CAAC;IACH,CAAC;IAED,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,CAAC;AAC9B,CAAC"}

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

@@ -1,40 +0,0 @@
-export type PRFile = {
-    filename: string;
-    status: string;
-    sha?: string;
-    patch?: string;
-};
-export type FindingSeverity = 'must' | 'should' | 'nit';
-export type Finding = {
-    rule: string;
-    severity: FindingSeverity;
-    path: string;
-    message: string;
-    raw_details?: string;
-    nodeId?: string;
-    line?: number;
-    documentationUrl?: string;
-};
-export type NodeRef = {
-    id: string;
-    type: string;
-    name?: string;
-    params?: Record<string, unknown>;
-    cred?: Record<string, unknown>;
-    flags?: {
-        continueOnFail?: boolean;
-        retryOnFail?: boolean | string;
-        waitBetweenTries?: number | string;
-        maxTries?: number | string;
-    };
-};
-export type Edge = {
-    from: string;
-    to: string;
-    on?: 'success' | 'error' | 'timeout';
-};
-export type Graph = {
-    nodes: NodeRef[];
-    edges: Edge[];
-    meta: Record<string, unknown>;
-};

package/dist/packages/review/types.js

@@ -1,3 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-//# sourceMappingURL=types.js.map

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

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

package/dist/packages/review/utils/findings.d.ts

@@ -1,23 +0,0 @@
-/**
- * Findings utilities
- * Shared logic for processing and analyzing findings across both review engine and CLI
- */
-import type { Finding } from '../types';
-export interface FindingsSummary {
-    must: number;
-    should: number;
-    nit: number;
-    total: number;
-}
-/**
- * Count findings by severity level
- */
-export declare function countFindingsBySeverity(findings: Finding[]): FindingsSummary;
-/**
- * Get severity order for sorting
- */
-export declare function getSeverityOrder(): Record<string, number>;
-/**
- * Sort findings by severity
- */
-export declare function sortFindingsBySeverity(findings: Finding[]): Finding[];

package/dist/packages/review/utils/findings.js

@@ -1,34 +0,0 @@
-"use strict";
-/**
- * Findings utilities
- * Shared logic for processing and analyzing findings across both review engine and CLI
- */
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.countFindingsBySeverity = countFindingsBySeverity;
-exports.getSeverityOrder = getSeverityOrder;
-exports.sortFindingsBySeverity = sortFindingsBySeverity;
-/**
- * Count findings by severity level
- */
-function countFindingsBySeverity(findings) {
-    return {
-        must: findings.filter((f) => f.severity === 'must').length,
-        should: findings.filter((f) => f.severity === 'should').length,
-        nit: findings.filter((f) => f.severity === 'nit').length,
-        total: findings.length,
-    };
-}
-/**
- * Get severity order for sorting
- */
-function getSeverityOrder() {
-    return { must: 0, should: 1, nit: 2 };
-}
-/**
- * Sort findings by severity
- */
-function sortFindingsBySeverity(findings) {
-    const order = getSeverityOrder();
-    return [...findings].sort((a, b) => order[a.severity] - order[b.severity]);
-}
-//# sourceMappingURL=findings.js.map