Fix quality violations by refactoring oversized function and feature envy

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: devill

src/dev/quality-checks-extractor.ts

@@ -5,6 +5,15 @@ interface QualityCheck {
   getGroupDefinition?: (_key: string) => { title: string; description: string } | undefined;
 }
 
+const buildPossibleKeysForCheck = (check: QualityCheck): string[] => {
+  return [
+    check.name,
+    `${check.name}Functions`,
+    `critical${check.name.charAt(0).toUpperCase() + check.name.slice(1)}`,
+    `large${check.name.charAt(0).toUpperCase() + check.name.slice(1)}`,
+  ];
+};
+
 export class QualityChecksExtractor {
   extractQualityChecksContent(): string {
     const qualityChecks = loadQualityChecks();
@@ -34,19 +43,10 @@ export class QualityChecksExtractor {
   }
 
   private getValidGroupKeys(check: QualityCheck): string[] {
-    const possibleKeys = this.buildPossibleKeys(check);
+    const possibleKeys = buildPossibleKeysForCheck(check);
     return possibleKeys.filter(key => this.isValidKey(check, key));
   }
 
-  private buildPossibleKeys(check: QualityCheck): string[] {
-    return [
-      check.name,
-      `${check.name}Functions`,
-      `critical${check.name.charAt(0).toUpperCase() + check.name.slice(1)}`,
-      `large${check.name.charAt(0).toUpperCase() + check.name.slice(1)}`,
-    ];
-  }
-
   private isValidKey(check: QualityCheck, key: string): boolean {
     try {
       return Boolean(check.getGroupDefinition?.(key));

src/dev/quality/checks/require-statements-check.ts

@@ -55,15 +55,18 @@ const isRequireCall = (node: CallExpression): boolean => {
          expression.getText() === 'require';
 };
 
+const isFunctionLikeSyntaxKind = (kind: SyntaxKind): boolean => {
+  return kind === SyntaxKind.FunctionDeclaration || 
+         kind === SyntaxKind.MethodDeclaration || 
+         kind === SyntaxKind.ArrowFunction ||
+         kind === SyntaxKind.FunctionExpression;
+};
+
 const isImportInsideFunction = (importDecl: ImportDeclaration): boolean => {
   let parent: Node | undefined = importDecl.getParent();
 
   while (parent) {
-    const kind = parent.getKind();
-    if (kind === SyntaxKind.FunctionDeclaration || 
-        kind === SyntaxKind.MethodDeclaration || 
-        kind === SyntaxKind.ArrowFunction ||
-        kind === SyntaxKind.FunctionExpression) {
+    if (isFunctionLikeSyntaxKind(parent.getKind())) {
       return true;
     }
     parent = parent.getParent();