@boyingliu01/xp-gate 0.26.0 → 0.27.0

package/plugins/dsh/src/skills.test.ts

@@ -14,10 +14,20 @@ const REPO_SKILLS_DIR = resolve(PLUGIN_ROOT, "..", "..", "skills")
 /**
  * The expected set is the REPO's skill directories, discovered rather than
  * hardcoded. A hardcoded list silently drifts: it previously named 12 skills and
- * omitted clipboard-vision, while prepack shipped 12 of 13 (#448).
+ * omitted one that prepack shipped, breaking parity (#448).
  */
 function repoSkills(): string[] {
-  return readdirSync(REPO_SKILLS_DIR, { withFileTypes: true })
+  // Guarded IO: a missing/unreadable repo skills dir is a test-environment
+  // failure we surface loudly, not a silent empty set (clean-code.missing-error-handling).
+  let entries;
+  try {
+    entries = readdirSync(REPO_SKILLS_DIR, { withFileTypes: true })
+  } catch (err) {
+    throw new Error(
+      `Cannot read repo skills dir ${REPO_SKILLS_DIR}: ${err instanceof Error ? err.message : String(err)}`
+    )
+  }
+  return entries
     .filter((d) => d.isDirectory())
     .map((d) => d.name)
     .filter((name) => existsSync(join(REPO_SKILLS_DIR, name, "SKILL.md")))
@@ -25,7 +35,14 @@ function repoSkills(): string[] {
 }
 
 function readFrontmatter(file: string): Record<string, unknown> {
-  const raw = readFileSync(file, "utf8")
+  // Guarded IO: re-throw with context so a broken SKILL.md names the file
+  // (clean-code.missing-error-handling).
+  let raw;
+  try {
+    raw = readFileSync(file, "utf8")
+  } catch (err) {
+    throw new Error(`${file}: cannot read file (${err instanceof Error ? err.message : String(err)})`)
+  }
   const m = raw.match(/^---\s*\r?\n([\s\S]*?)\r?\n---\s*\r?\n/)
   if (!m) throw new Error(`${file}: missing YAML frontmatter`)
   return parse(m[1]) as Record<string, unknown>
@@ -58,8 +75,4 @@ describe("bundled skills", () => {
       expect((fm.description as string).trim().length, `${name}: non-empty description`).toBeGreaterThan(0)
     }
   })
-
-  it("bundles clipboard-vision, which the old hardcoded list omitted (#448)", () => {
-    expect(existsSync(join(SKILLS_DIR, "clipboard-vision", "SKILL.md"))).toBe(true)
-  })
 })

package/plugins/opencode/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/opencode-plugin",
-  "version": "0.26.0",
+  "version": "0.27.0",
   "type": "module",
   "main": "index.ts",
   "description": "XP-Gate quality gates + AI workflow skills + Sprint Flow TUI sidebar for OpenCode",

package/plugins/opencode/scripts/prepack.cjs

@@ -16,7 +16,6 @@ const REPO_ROOT = path.resolve(PLUGIN_ROOT, '..', '..');
 
 const CORE_SKILLS = [
   'admin-template-guidelines',
-  'clipboard-vision',
   'delphi-review',
   'improve-codebase-architecture',
   'ralph-loop',

package/plugins/opencode/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-16
-**Commit:** 98096abe
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.26.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/plugins/opencode/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
 **Generated:** 2026-10-16
-**Commit:** 98096abe
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.26.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/opencode/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-16
-**Commit:** 98096abe
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.26.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/qoder/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.26.0",
+  "version": "0.27.0",
   "displayName": "XP-Gate",
   "description": "Extreme Programming quality gates + AI workflow skills for Qoder. Includes 12 pre-commit gates (Gate 0-11), 8 pre-push gates, Sprint Flow (6 phases), and Delphi multi-expert review (>=90% consensus).",
   "author": {

package/plugins/qoder/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-16
-**Commit:** 98096abe
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.26.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/plugins/qoder/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
 **Generated:** 2026-10-16
-**Commit:** 98096abe
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.26.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/qoder/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-16
-**Commit:** 98096abe
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.26.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/principles/AGENTS.md

@@ -1,9 +1,9 @@
 # PRINCIPLES CHECKER MODULE
 
 **Generated:** 2026-10-16
-**Commit:** 98096abe
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.26.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Clean Code & SOLID principles checker — **Gate 4** of pre-commit. 14 rules × 9 language adapters, SARIF 2.1.0 output. Houses the **Boy Scout Rule** enforcement engine (Gate 6) and warning-baseline storage.

package/principles/rules/__tests__/clean-code/code-duplication.test.ts

@@ -56,15 +56,16 @@ describe('code-duplication.ts - Code Duplication Rule', () => {
     expect(violations).toEqual([]);
   });
 
-  it('should return empty array when adapter throws', () => {
-    const mockAdapterThrowing = {
-      ...mockAdapter,
-      duplicationPercentage: () => { throw new Error('jscpd error'); }
-    };
+  it('should propagate adapter errors (no empty catch, #392)', () => {
+    // #392-#4: the rule no longer swallows adapter errors in an empty catch, so
+    // a throwing adapter getter must bubble up to analyzer.runRuleOnFile().
+    const mockAdapterThrowing = Object.defineProperty(
+      { ...mockAdapter },
+      'duplicationPercentage',
+      { get: () => { throw new Error('jscpd error'); } }
+    );
     
-    const violations = codeDuplicationRule.check('test.ts', mockAdapterThrowing as never);
-    
-    expect(violations).toEqual([]);
+    expect(() => codeDuplicationRule.check('test.ts', mockAdapterThrowing as never)).toThrow('jscpd error');
   });
 
   it('should handle multiple files with duplication', () => {

package/principles/rules/__tests__/clean-code/deep-nesting.test.ts

@@ -61,14 +61,13 @@ function deeplyNested() {
     expect(deepNestingRule.severity).toBe('warning');
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractFunctions: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = deepNestingRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations.length).toBe(0);
+    // Rule errors must bubble up to analyzer.runRuleOnFile(), not be swallowed.
+    expect(() => deepNestingRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 });

package/principles/rules/__tests__/clean-code/god-class.test.ts

@@ -98,14 +98,12 @@ class DataClass {
     expect(godClassRule.severity).toBe('warning');
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractClasses: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = godClassRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations.length).toBe(0);
+    expect(() => godClassRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 });

package/principles/rules/__tests__/clean-code/large-file.test.ts

@@ -61,14 +61,12 @@ describe('largeFileRule', () => {
     expect(largeFileRule.threshold).toEqual(1150);
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       countLines: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = largeFileRule.check('test.ts', mockAdapterThatThrows);
-    
-    expect(violations).toHaveLength(0);
+    expect(() => largeFileRule.check('test.ts', mockAdapterThatThrows)).toThrow('Adapter failed');
   });
 });

package/principles/rules/__tests__/clean-code/long-function.test.ts

@@ -70,14 +70,12 @@ describe('longFunctionRule', () => {
     expect(longFunctionRule.threshold).toEqual(50);
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractFunctions: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = longFunctionRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations).toHaveLength(0);
+    expect(() => longFunctionRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 });

package/principles/rules/__tests__/clean-code/many-exports.test.ts

@@ -111,14 +111,13 @@ describe('manyExportsRule', () => {
     expect(violations).toHaveLength(0);
   });
 
-  it('returns empty violations when adapter throws (catch branch)', () => {
+  it('propagates adapter errors (no empty catch, #392)', () => {
     const adapter: never = {
       ...baseAdapter,
       extractExports: () => {
         throw new Error('boom');
       },
     } as never;
-    const violations = manyExportsRule.check('file.ts', adapter);
-    expect(violations).toHaveLength(0);
+    expect(() => manyExportsRule.check('file.ts', adapter)).toThrow('boom');
   });
 });

package/principles/rules/__tests__/clean-code/missing-error-handling.test.ts

@@ -134,15 +134,13 @@ async function fetchUser(id: string) {
     expect(missingErrorHandlingRule.severity).toBe('warning');
   });
 
-  it('should return empty array when adapter.extractFunctions throws', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThrowing = {
       ...mockAdapter,
       extractFunctions: () => { throw new Error('Parse error'); }
     };
     
-    const violations = missingErrorHandlingRule.check('test.ts', mockAdapterThrowing as never);
-    
-    expect(violations).toEqual([]);
+    expect(() => missingErrorHandlingRule.check('test.ts', mockAdapterThrowing as never)).toThrow('Parse error');
   });
 
   it('should detect multiple IO operations in one function', () => {

package/principles/rules/__tests__/clean-code/too-many-params.test.ts

@@ -47,14 +47,12 @@ describe('too-many-params.ts - Too Many Parameters Rule', () => {
     expect(tooManyParamsRule.severity).toBe('info');
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractFunctions: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = tooManyParamsRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations.length).toBe(0);
+    expect(() => tooManyParamsRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 });

package/principles/rules/__tests__/clean-code/unused-imports.test.ts

@@ -79,15 +79,15 @@ describe('unused-imports.ts - Unused Imports Rule', () => {
     expect(violations).toEqual([]);
   });
 
-  it('should return empty array when adapter throws', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThrowing = {
       ...mockAdapter,
+      // Violate the adapter contract: `imports` must be an array. With the old
+      // empty catch this was swallowed; now it must bubble up to the analyzer.
       imports: () => { throw new Error('Parse error'); }
     };
     
-    const violations = unusedImportsRule.check('test.ts', mockAdapterThrowing as never);
-    
-    expect(violations).toEqual([]);
+    expect(() => unusedImportsRule.check('test.ts', mockAdapterThrowing as never)).toThrow();
   });
 
   it('should handle namespace imports', () => {

package/principles/rules/__tests__/solid/dip.test.ts

@@ -82,15 +82,13 @@ class DateService {
     expect(dipRule.severity).toBe('warning');
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractClasses: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = dipRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations.length).toBe(0);
+    expect(() => dipRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 
   it('should exclude built-in types (Date, Map, Set, etc.)', () => {

package/principles/rules/__tests__/solid/isp.test.ts

@@ -47,14 +47,12 @@ describe('isp.ts - Interface Segregation Principle Rule', () => {
     expect(ispRule.severity).toBe('info');
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractInterfaces: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = ispRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations.length).toBe(0);
+    expect(() => ispRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 });

package/principles/rules/__tests__/solid/lsp.test.ts

@@ -49,15 +49,13 @@ class DerivedRepo extends BaseRepo {
     expect(lspRule.severity).toBe('info');
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractClasses: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = lspRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations.length).toBe(0);
+    expect(() => lspRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 
   /**

package/principles/rules/__tests__/solid/ocp.test.ts

@@ -56,14 +56,12 @@ class DerivedClass extends BaseClass {
     expect(ocpRule.severity).toBe('info');
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractClasses: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = ocpRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations.length).toBe(0);
+    expect(() => ocpRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 });

package/principles/rules/__tests__/solid/srp.test.ts

@@ -76,14 +76,12 @@ describe('srp.ts - Single Responsibility Principle Rule', () => {
     expect(srpRule.severity).toBe('warning');
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractClasses: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = srpRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations.length).toBe(0);
+    expect(() => srpRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 });

package/principles/rules/clean-code/code-duplication.ts

@@ -11,20 +11,18 @@ export const codeDuplicationRule: Rule = {
     // Read the ACTIVE config per invocation; a module-load snapshot was #457.
     const settings = getActiveConfig().rules['clean-code']['code-duplication'];
     
-    try {
-      const typedAdapter = adapter as { duplicationPercentage?: number };
-      const duplicationPercentage = typedAdapter.duplicationPercentage;
-      
-      if (duplicationPercentage && duplicationPercentage > (settings.threshold ?? 0)) {
-        violations.push({
-          file,
-          line: 1,
-          ruleId: 'clean-code.code-duplication',
-          message: `Code duplication detected: ${duplicationPercentage}% (threshold: ${settings.threshold}%). Consider refactoring duplicated code.`,
-          severity: (settings.severity as Severity) ?? 'warning'
-        });
-      }
-    } catch { }
+    const typedAdapter = adapter as { duplicationPercentage?: number };
+    const duplicationPercentage = typedAdapter.duplicationPercentage;
+    
+    if (duplicationPercentage && duplicationPercentage > (settings.threshold ?? 0)) {
+      violations.push({
+        file,
+        line: 1,
+        ruleId: 'clean-code.code-duplication',
+        message: `Code duplication detected: ${duplicationPercentage}% (threshold: ${settings.threshold}%). Consider refactoring duplicated code.`,
+        severity: (settings.severity as Severity) ?? 'warning'
+      });
+    }
     
     return violations;
   }

package/principles/rules/clean-code/deep-nesting.ts

@@ -11,27 +11,25 @@ export const deepNestingRule: Rule = {
     // Read the ACTIVE config per invocation; a module-load snapshot was #457.
     const settings = getActiveConfig().rules['clean-code']['deep-nesting'];
     
-    try {
-      interface TypedAdapter {
-        extractFunctions?: () => Array<{name?: string, nestingDepth?: number, startLine?: number, line?: number}> | undefined;
-      }
-      const typedAdapter = adapter as TypedAdapter;
-      const functions = typedAdapter.extractFunctions?.() || [];
-      
-      for (const func of functions) {
-        if (func.nestingDepth && func.nestingDepth > (settings.threshold ?? 0)) {
-          violations.push({
-            file,
-            line: func.startLine ?? func.line ?? 1,
-            ruleId: 'clean-code.deep-nesting',
-            message: `Function "${func.name}" has deep nesting: ${func.nestingDepth} levels (maximum: ${
-              settings.threshold
-            })`,
-            severity: (settings.severity as Severity) ?? 'warning'
-          });
-        }
+    interface TypedAdapter {
+      extractFunctions?: () => Array<{name?: string, nestingDepth?: number, startLine?: number, line?: number}> | undefined;
+    }
+    const typedAdapter = adapter as TypedAdapter;
+    const functions = typedAdapter.extractFunctions?.() || [];
+    
+    for (const func of functions) {
+      if (func.nestingDepth && func.nestingDepth > (settings.threshold ?? 0)) {
+        violations.push({
+          file,
+          line: func.startLine ?? func.line ?? 1,
+          ruleId: 'clean-code.deep-nesting',
+          message: `Function "${func.name}" has deep nesting: ${func.nestingDepth} levels (maximum: ${
+            settings.threshold
+          })`,
+          severity: (settings.severity as Severity) ?? 'warning'
+        });
       }
-    } catch { }
+    }
     
     return violations;
   }

package/principles/rules/clean-code/god-class.ts

@@ -11,43 +11,41 @@ export const godClassRule: Rule = {
     // Read the ACTIVE config per invocation; a module-load snapshot was #457.
     const settings = getActiveConfig().rules['clean-code']['god-class'];
     
-    try {
-      interface ClassObject {
-        code?: string;
-        line: number;
-        name: string;
-      }
-      
-      interface TypedAdapter {
-        extractClasses?: () => ClassObject[] | undefined;
-      }
-      
-      const typedAdapter = adapter as TypedAdapter;
-      const classes = typedAdapter.extractClasses?.() || [];
+    interface ClassObject {
+      code?: string;
+      line: number;
+      name: string;
+    }
+    
+    interface TypedAdapter {
+      extractClasses?: () => ClassObject[] | undefined;
+    }
+    
+    const typedAdapter = adapter as TypedAdapter;
+    const classes = typedAdapter.extractClasses?.() || [];
+    
+    for (const cls of classes) {
+      const methodMatches = cls.code?.match(/(get|set)\s+\w+\s*\(|\w+\s*\([^)]*\)\s*{/g) || [];
       
-      for (const cls of classes) {
-        const methodMatches = cls.code?.match(/(get|set)\s+\w+\s*\(|\w+\s*\([^)]*\)\s*{/g) || [];
-        
-        const methodCount = methodMatches.filter((m: string) => {
-          if (m.startsWith('get ') || m.startsWith('set ')) {
-            return false;
-          }
-          return true;
-        }).length;
-        
-        if (methodCount > (settings.threshold ?? 0)) {
-          violations.push({
-            file,
-            line: cls.line,
-            ruleId: 'clean-code.god-class',
-            message: `Class "${cls.name}" has too many methods: ${methodCount} (maximum: ${
-              settings.threshold
-            })`,
-            severity: (settings.severity as Severity) ?? 'warning'
-          });
+      const methodCount = methodMatches.filter((m: string) => {
+        if (m.startsWith('get ') || m.startsWith('set ')) {
+          return false;
         }
+        return true;
+      }).length;
+      
+      if (methodCount > (settings.threshold ?? 0)) {
+        violations.push({
+          file,
+          line: cls.line,
+          ruleId: 'clean-code.god-class',
+          message: `Class "${cls.name}" has too many methods: ${methodCount} (maximum: ${
+            settings.threshold
+          })`,
+          severity: (settings.severity as Severity) ?? 'warning'
+        });
       }
-    } catch { }
+    }
     
     return violations;
   }

package/principles/rules/clean-code/large-file.ts

@@ -15,23 +15,21 @@ export const largeFileRule: Rule = {
     const settings = getActiveConfig().rules['clean-code']['large-file'];
     const threshold = settings.threshold ?? 500;
 
-    try {
-      interface TypedAdapter {
-        countLines?: (file: string) => number | undefined;
-      }
-      const typedAdapter = adapter as TypedAdapter;
-      const lineCount = typedAdapter.countLines?.(file) ?? 0;
+    interface TypedAdapter {
+      countLines?: (file: string) => number | undefined;
+    }
+    const typedAdapter = adapter as TypedAdapter;
+    const lineCount = typedAdapter.countLines?.(file) ?? 0;
 
-      if (lineCount > threshold) {
-        violations.push({
-          file,
-          line: 1,
-          ruleId: 'clean-code.large-file',
-          message: `File is too large: ${lineCount} lines (maximum: ${threshold})`,
-          severity: (settings.severity as Severity) ?? 'warning'
-        });
-      }
-    } catch { }
+    if (lineCount > threshold) {
+      violations.push({
+        file,
+        line: 1,
+        ruleId: 'clean-code.large-file',
+        message: `File is too large: ${lineCount} lines (maximum: ${threshold})`,
+        severity: (settings.severity as Severity) ?? 'warning'
+      });
+    }
 
     return violations;
   }