@rcrsr/rill 0.4.5 → 0.6.0

package/dist/check/fixer.js

@@ -1,119 +0,0 @@
-/**
- * Fix Applier
- * Apply automatic fixes to source code with collision detection.
- */
-import { parse } from '../parser/index.js';
-// ============================================================
-// FIX APPLICATION
-// ============================================================
-/**
- * Apply automatic fixes to source code.
- *
- * Constraints:
- * - Applies fixes in reverse position order (end to start) to avoid offset shifts
- * - Skips fixes where applicable === false
- * - Detects collisions (overlapping ranges) and skips with reason
- * - Verifies modified source parses successfully
- * - Throws if any applied fix creates invalid syntax
- *
- * @param source - Original source code
- * @param diagnostics - Diagnostics with potential fixes
- * @param context - Validation context (unused but required by spec)
- * @returns ApplyResult with modified source and counts
- * @throws Error if applied fixes create invalid syntax [EC-6]
- */
-export function applyFixes(source, diagnostics, _context) {
-    // Filter to diagnostics with fixes
-    const fixableDiagnostics = diagnostics.filter((d) => d.fix !== null && d.fix.applicable);
-    // If no applicable fixes, return original
-    if (fixableDiagnostics.length === 0) {
-        return {
-            modified: source,
-            applied: 0,
-            skipped: 0,
-            skippedReasons: [],
-        };
-    }
-    // Convert to ApplicableFix with positions
-    const fixes = fixableDiagnostics.map((d) => ({
-        code: d.code,
-        start: d.fix.range.start.offset,
-        end: d.fix.range.end.offset,
-        replacement: d.fix.replacement,
-    }));
-    // Sort fixes by end position (descending) to apply from end to start
-    // This avoids offset shifts when applying multiple fixes
-    const sortedFixes = fixes.slice().sort((a, b) => b.end - a.end);
-    // Detect collisions and filter to non-overlapping fixes
-    const { validFixes, skippedReasons } = filterCollisions(sortedFixes);
-    // Apply fixes to source
-    let modified = source;
-    for (const fix of validFixes) {
-        const before = modified.slice(0, fix.start);
-        const after = modified.slice(fix.end);
-        modified = before + fix.replacement + after;
-    }
-    // Verify modified source parses successfully [EC-6]
-    try {
-        parse(modified);
-    }
-    catch {
-        throw new Error('Fix would create invalid syntax');
-    }
-    const applied = validFixes.length;
-    const skipped = sortedFixes.length - applied;
-    return {
-        modified,
-        applied,
-        skipped,
-        skippedReasons,
-    };
-}
-// ============================================================
-// COLLISION DETECTION
-// ============================================================
-/**
- * Filter fixes to remove overlapping ranges.
- * Detects collisions where fix ranges overlap [EC-5].
- *
- * Strategy: Keep first fix in sorted order (end to start),
- * skip subsequent fixes that overlap with any kept fix.
- *
- * @param sortedFixes - Fixes sorted by end position (descending)
- * @returns Valid fixes and reasons for skipped fixes
- */
-function filterCollisions(sortedFixes) {
-    const validFixes = [];
-    const skippedReasons = [];
-    for (const fix of sortedFixes) {
-        // Check if this fix overlaps with any already-kept fix
-        const hasCollision = validFixes.some((kept) => rangesOverlap(fix, kept));
-        if (hasCollision) {
-            // Skip this fix due to collision [EC-5]
-            skippedReasons.push({
-                code: fix.code,
-                reason: 'Fix range overlaps with another fix',
-            });
-        }
-        else {
-            // No collision, keep this fix
-            validFixes.push(fix);
-        }
-    }
-    return { validFixes, skippedReasons };
-}
-/**
- * Check if two fix ranges overlap.
- *
- * Ranges overlap if:
- * - One starts before the other ends, AND
- * - One ends after the other starts
- *
- * @param a - First fix range
- * @param b - Second fix range
- * @returns true if ranges overlap
- */
-function rangesOverlap(a, b) {
-    return a.start < b.end && a.end > b.start;
-}
-//# sourceMappingURL=fixer.js.map

package/dist/check/fixer.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"fixer.js","sourceRoot":"","sources":["../../src/check/fixer.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,KAAK,EAAE,MAAM,oBAAoB,CAAC;AA+B3C,+DAA+D;AAC/D,kBAAkB;AAClB,+DAA+D;AAE/D;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,UAAU,CACxB,MAAc,EACd,WAAyB,EACzB,QAA2B;IAE3B,mCAAmC;IACnC,MAAM,kBAAkB,GAAG,WAAW,CAAC,MAAM,CAC3C,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,GAAG,KAAK,IAAI,IAAI,CAAC,CAAC,GAAG,CAAC,UAAU,CAC1C,CAAC;IAEF,0CAA0C;IAC1C,IAAI,kBAAkB,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACpC,OAAO;YACL,QAAQ,EAAE,MAAM;YAChB,OAAO,EAAE,CAAC;YACV,OAAO,EAAE,CAAC;YACV,cAAc,EAAE,EAAE;SACnB,CAAC;IACJ,CAAC;IAED,0CAA0C;IAC1C,MAAM,KAAK,GAAoB,kBAAkB,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;QAC5D,IAAI,EAAE,CAAC,CAAC,IAAI;QACZ,KAAK,EAAE,CAAC,CAAC,GAAI,CAAC,KAAK,CAAC,KAAK,CAAC,MAAM;QAChC,GAAG,EAAE,CAAC,CAAC,GAAI,CAAC,KAAK,CAAC,GAAG,CAAC,MAAM;QAC5B,WAAW,EAAE,CAAC,CAAC,GAAI,CAAC,WAAW;KAChC,CAAC,CAAC,CAAC;IAEJ,qEAAqE;IACrE,yDAAyD;IACzD,MAAM,WAAW,GAAG,KAAK,CAAC,KAAK,EAAE,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,GAAG,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC;IAEhE,wDAAwD;IACxD,MAAM,EAAE,UAAU,EAAE,cAAc,EAAE,GAAG,gBAAgB,CAAC,WAAW,CAAC,CAAC;IAErE,wBAAwB;IACxB,IAAI,QAAQ,GAAG,MAAM,CAAC;IACtB,KAAK,MAAM,GAAG,IAAI,UAAU,EAAE,CAAC;QAC7B,MAAM,MAAM,GAAG,QAAQ,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,KAAK,CAAC,CAAC;QAC5C,MAAM,KAAK,GAAG,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QACtC,QAAQ,GAAG,MAAM,GAAG,GAAG,CAAC,WAAW,GAAG,KAAK,CAAC;IAC9C,CAAC;IAED,oDAAoD;IACpD,IAAI,CAAC;QACH,KAAK,CAAC,QAAQ,CAAC,CAAC;IAClB,CAAC;IAAC,MAAM,CAAC;QACP,MAAM,IAAI,KAAK,CAAC,iCAAiC,CAAC,CAAC;IACrD,CAAC;IAED,MAAM,OAAO,GAAG,UAAU,CAAC,MAAM,CAAC;IAClC,MAAM,OAAO,GAAG,WAAW,CAAC,MAAM,GAAG,OAAO,CAAC;IAE7C,OAAO;QACL,QAAQ;QACR,OAAO;QACP,OAAO;QACP,cAAc;KACf,CAAC;AACJ,CAAC;AAED,+DAA+D;AAC/D,sBAAsB;AACtB,+DAA+D;AAE/D;;;;;;;;;GASG;AACH,SAAS,gBAAgB,CAAC,WAA4B;IAIpD,MAAM,UAAU,GAAoB,EAAE,CAAC;IACvC,MAAM,cAAc,GAA4C,EAAE,CAAC;IAEnE,KAAK,MAAM,GAAG,IAAI,WAAW,EAAE,CAAC;QAC9B,uDAAuD;QACvD,MAAM,YAAY,GAAG,UAAU,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,aAAa,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC,CAAC;QAEzE,IAAI,YAAY,EAAE,CAAC;YACjB,wCAAwC;YACxC,cAAc,CAAC,IAAI,CAAC;gBAClB,IAAI,EAAE,GAAG,CAAC,IAAI;gBACd,MAAM,EAAE,qCAAqC;aAC9C,CAAC,CAAC;QACL,CAAC;aAAM,CAAC;YACN,8BAA8B;YAC9B,UAAU,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACvB,CAAC;IACH,CAAC;IAED,OAAO,EAAE,UAAU,EAAE,cAAc,EAAE,CAAC;AACxC,CAAC;AAED;;;;;;;;;;GAUG;AACH,SAAS,aAAa,CACpB,CAAiC,EACjC,CAAiC;IAEjC,OAAO,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,GAAG,IAAI,CAAC,CAAC,GAAG,GAAG,CAAC,CAAC,KAAK,CAAC;AAC5C,CAAC"}

package/dist/check/index.d.ts

@@ -1,10 +0,0 @@
-/**
- * Check Module - Static Analysis for rill
- * Public API for rill-check tool.
- */
-export type { ValidationRule, RuleCategory, Severity, RuleState, Diagnostic, Fix, CheckConfig, ValidationContext, FixContext, } from './types.js';
-export { VALIDATION_RULES } from './rules/index.js';
-export { loadConfig, createDefaultConfig } from './config.js';
-export { validateScript } from './validator.js';
-export { applyFixes, type ApplyResult } from './fixer.js';
-//# sourceMappingURL=index.d.ts.map

package/dist/check/index.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/check/index.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAKH,YAAY,EACV,cAAc,EACd,YAAY,EACZ,QAAQ,EACR,SAAS,EACT,UAAU,EACV,GAAG,EACH,WAAW,EACX,iBAAiB,EACjB,UAAU,GACX,MAAM,YAAY,CAAC;AAKpB,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAC;AAKpD,OAAO,EAAE,UAAU,EAAE,mBAAmB,EAAE,MAAM,aAAa,CAAC;AAK9D,OAAO,EAAE,cAAc,EAAE,MAAM,gBAAgB,CAAC;AAKhD,OAAO,EAAE,UAAU,EAAE,KAAK,WAAW,EAAE,MAAM,YAAY,CAAC"}

package/dist/check/index.js

@@ -1,21 +0,0 @@
-/**
- * Check Module - Static Analysis for rill
- * Public API for rill-check tool.
- */
-// ============================================================
-// RULE REGISTRY
-// ============================================================
-export { VALIDATION_RULES } from './rules/index.js';
-// ============================================================
-// CONFIGURATION
-// ============================================================
-export { loadConfig, createDefaultConfig } from './config.js';
-// ============================================================
-// VALIDATION
-// ============================================================
-export { validateScript } from './validator.js';
-// ============================================================
-// FIX APPLICATION
-// ============================================================
-export { applyFixes } from './fixer.js';
-//# sourceMappingURL=index.js.map

package/dist/check/index.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/check/index.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAiBH,+DAA+D;AAC/D,gBAAgB;AAChB,+DAA+D;AAC/D,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAC;AAEpD,+DAA+D;AAC/D,gBAAgB;AAChB,+DAA+D;AAC/D,OAAO,EAAE,UAAU,EAAE,mBAAmB,EAAE,MAAM,aAAa,CAAC;AAE9D,+DAA+D;AAC/D,aAAa;AACb,+DAA+D;AAC/D,OAAO,EAAE,cAAc,EAAE,MAAM,gBAAgB,CAAC;AAEhD,+DAA+D;AAC/D,kBAAkB;AAClB,+DAA+D;AAC/D,OAAO,EAAE,UAAU,EAAoB,MAAM,YAAY,CAAC"}

package/dist/check/rules/anti-patterns.d.ts

@@ -1,65 +0,0 @@
-/**
- * Anti-Pattern Rules
- * Enforces best practices from docs/16_conventions.md:411-462.
- */
-import type { ValidationRule } from '../types.js';
-/**
- * Warns on variable reassignment patterns.
- * Variables lock to their first type, and reassignment suggests confusing
- * flow control. Prefer functional style or new variables.
- *
- * Detection:
- * - Capture node (:> $var) where $var already exists in validation context
- * - Tracks variables seen during validation pass
- *
- * Valid alternatives:
- * - Use new variable: $result1, $result2
- * - Functional chains: value -> op1 -> op2
- *
- * References:
- * - docs/16_conventions.md:413-424
- */
-export declare const AVOID_REASSIGNMENT: ValidationRule;
-/**
- * Warns on complex nested boolean conditions.
- * Complex conditions with multiple nested operators are hard to read.
- * Extract to named variables for clarity.
- *
- * Detection:
- * - Conditional nodes with conditions containing 3+ boolean operators (&&, ||)
- * - Nesting depth > 2 for boolean expressions
- *
- * Valid alternatives:
- * - Extract sub-conditions to named variables
- * - Split complex checks into multiple smaller checks
- *
- * References:
- * - docs/16_conventions.md:451-461
- */
-export declare const COMPLEX_CONDITION: ValidationRule;
-/**
- * Detects attempts to modify outer-scope variables from inside loops.
- * This is a common LLM-generated anti-pattern that never works in Rill.
- *
- * Rill's scoping rules mean that captures inside loop bodies create LOCAL
- * variables that don't affect outer scope. This is a fundamental language
- * constraint, not a style preference.
- *
- * WRONG - this pattern NEVER works:
- *   0 :> $count
- *   [1, 2, 3] -> each { $count + 1 :> $count }  # creates LOCAL $count
- *   $count                                       # still 0!
- *
- * RIGHT - use accumulators:
- *   [1, 2, 3] -> fold(0) { $@ + 1 }             # returns 3
- *   [1, 2, 3] -> each(0) { $@ + 1 }             # returns [1, 2, 3]
- *
- * This rule catches captures inside loop/collection bodies where the
- * variable name matches an outer-scope variable.
- *
- * References:
- * - docs/99_llm-reference.txt (LOOP STATE PATTERNS)
- * - docs/03_variables.md (Scope Rules)
- */
-export declare const LOOP_OUTER_CAPTURE: ValidationRule;
-//# sourceMappingURL=anti-patterns.d.ts.map

package/dist/check/rules/anti-patterns.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"anti-patterns.d.ts","sourceRoot":"","sources":["../../../src/check/rules/anti-patterns.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,KAAK,EACV,cAAc,EAGf,MAAM,aAAa,CAAC;AAuBrB;;;;;;;;;;;;;;;GAeG;AACH,eAAO,MAAM,kBAAkB,EAAE,cAgDhC,CAAC;AAMF;;;;;;;;;;;;;;;GAeG;AACH,eAAO,MAAM,iBAAiB,EAAE,cA6C/B,CAAC;AAmIF;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AACH,eAAO,MAAM,kBAAkB,EAAE,cAwFhC,CAAC"}