fix: 6 rules detail fix (#53)

* feat(#29): 중복 문제 해결 (#29)

* feat(#29): 중복 문제 해결

* Revert "Merge branch 'develop' into feature/#29"

* 🐛 fix: 삭제된 코드 revert

---------

Co-authored-by: jungsun <[email protected]>

* refactor(lint-manager): ESLint 설정 탐색 및 병합 로직 리팩토링 (#30)

* feat: 빠른 수정 메뉴에서 disable 안보이게 설정

* refactor: 5가지 핵심 논리 규칙 안정성 및 UX 개선 (#35)

LGTM

* feat: 퀵픽스 메뉴 한글 통일 및 AI 프롬프트 개선 (#36)

LGTM

* 🚀 deploy: tag 기반 release 작업 정리를 위해 workflow 수정

* 📝 docs: license 파일 수정 및 불필요한 의존성 삭제, 의존성 라이브러리 정리

* 📝 docs: 직접 의존성만 추가하도록 수정

* 📝 docs: LICENSE.txt > LICENSE

* 📝 docs: 오픈소스 라이선스 전문 추가 및 의존성 라이브러리 전부 추가

* fix: AI 파트 실행 오류 수정 (#40)

* 🔧 chore: setup semantic-release for VSCode extension deployment

* fix: publish.yml 수정

* fix: .vscodeignore에 env 추가

* Refactor/#48 filepath and config refactor (#49)

* refactor(lint): 파일 경로 처리 및 ESLint 설정 탐색 로직 리팩토링

* refactor(lint): 파일 경로 처리 및 ESLint 설정 탐색 로직 리팩토링

* Refactor/#51 (#52)

* refactor: 5가지 핵심 논리 규칙 안정성 및 UX 개선

* Refactor(a11y): 6개 로직 개선

---------

Co-authored-by: 최희수 <[email protected]>
Co-authored-by: hong-seo <[email protected]>
Co-authored-by: hansumin <[email protected]>

Author: 4 people

src/rules/logic/aria-activedescendant-has-tabindex.ts

@@ -18,7 +18,7 @@ export function ariaActivedescendantHasTabindexFix(
   fix.edit = new vscode.WorkspaceEdit();
 
   // 열림 태그에 tabIndex="0" 추가
-  const newCode = context.code.replace(/^<(\w+)/, `<$1 tabIndex="0"`);
+  const newCode = context.code.replace(/^<(\w+)/, `<$1 tabIndex={0}`);
 
   fix.edit.replace(context.document.uri, context.range, newCode);
   fix.diagnostics = [

src/rules/logic/click-events-have-key-events.ts

@@ -7,14 +7,19 @@ export function clickEventsHaveKeyEventsFix(
   const { code, range, document } = context;
 
   // 이미 onKeyDown 핸들러가 있으면 수정하지 않음
-  if (/\bonKeyDown\s*=/.test(code)) {
-    return [];
-  }
+  if (/\bonKeyDown\s*=/.test(code)) return [];
+
+  // onClick 핸들러 추출
+  const onClickMatch = code.match(/\bonClick\s*=\s*(\{[^}]+\}|\{[\s\S]*?\})/);
+  if (!onClickMatch) return [];
 
-  // onClick 핸들러와 그 내용을 추출
-  const onClickMatch = code.match(/\bonClick\s*=\s*({[\s\S]*?})/);
-  if (!onClickMatch) {
-    return [];
+  const onClickAttribute = onClickMatch[0];
+  const onClickValue = onClickMatch[1];
+
+  // onClick 내용 정제
+  let handlerCall = onClickValue.slice(1, -1).trim();
+  if (!handlerCall.endsWith(")")) {
+    handlerCall += "(event)";
   }
 
   const onClickAttribute = onClickMatch[0]; // ex: onClick={() => alert('hi')}
@@ -23,26 +28,27 @@ export function clickEventsHaveKeyEventsFix(
   // onKeyDown 핸들러를 새로 생성
   const onKeyDownAttribute = ` onKeyDown={(event) => { if (event.key === 'Enter' || event.key === ' ') { (${onClickValue.slice(1, -1)})(event); } }}`;
 
-  // 기존 onClick 속성 바로 뒤에 onKeyDown 속성을 추가
-  const newCode = code.replace(onClickAttribute, onClickAttribute + onKeyDownAttribute);
+  // onKeyDown 핸들러 생성
+  const onKeyDownAttribute = ` onKeyDown={(event) => { if (event.key === 'Enter' || event.key === ' ') { ${handlerCall}; } }}`;
 
-  if (newCode === code) {
-    return [];
-  }
+  // 새로운 코드 생성
+  const newCode = code.replace(onClickAttribute, onClickAttribute + onKeyDownAttribute);
 
+  // Quick Fix 생성
   const fix = new vscode.CodeAction(
     `키보드 이벤트(onKeyDown) 추가`,
     vscode.CodeActionKind.QuickFix
   );
+  fix.isPreferred = true;
   fix.edit = new vscode.WorkspaceEdit();
   fix.edit.replace(document.uri, range, newCode);
   fix.diagnostics = [
     new vscode.Diagnostic(
       range,
-      `클릭 가능한 요소에는 키보드 이벤트도 함께 제공해야 합니다.`,
+      `클릭 이벤트만 있는 요소는 키보드 접근성이 부족할 수 있습니다. onKeyDown 핸들러를 추가하여 Enter/Space 키 입력을 처리하세요.`,
       vscode.DiagnosticSeverity.Warning
     ),
   ];
-  
+
   return [fix];
-}
+}

src/rules/logic/interactive-supports-focus.ts

@@ -17,7 +17,7 @@ export function interactiveSupportsFocusFix(
   );
   fix.edit = new vscode.WorkspaceEdit();
 
-  const newCode = context.code.replace(/^<(\w+)/, `<$1 tabIndex="0"`);
+  const newCode = context.code.replace(/^<(\w+)/, `<$1 tabIndex={0}`);
 
   fix.edit.replace(context.document.uri, context.range, newCode);
   fix.diagnostics = [

src/rules/logic/label-has-associated-control.ts

@@ -7,42 +7,89 @@ export const labelHasAssociatedControlFix: RuleFixer = (
   const { code, range, document, fullLine } = context;
   const fixes: vscode.CodeAction[] = [];
 
-  // 1. 같은 줄에 있는 input의 id를 찾아 for(htmlFor) 속성으로 연결하는 것이 가장 안정적입니다.
-  // fullLine에서 id="some-id" 패턴을 찾습니다.
-  const inputIdMatch = fullLine.match(/id="([^"]+)"/);
-
-  // 만약 연결할 input의 id를 찾았다면, for 속성을 추가하는 수정 제안을 생성합니다.
-  if (inputIdMatch) {
-    const inputId = inputIdMatch[1];
-    
-    // <label> 태그에 for="some-id"를 추가합니다.
-    const fixedCode = code.replace(
-      /<label(.*?)>/i,
-      `<label htmlFor="${inputId}"$1>`
-    );
-
-    const fixFor = new vscode.CodeAction(
-      `input id와 연결 (htmlFor="${inputId}")`,
-      vscode.CodeActionKind.QuickFix
-    );
-    fixFor.edit = new vscode.WorkspaceEdit();
-    fixFor.edit.replace(document.uri, range, fixedCode);
-    fixFor.isPreferred = true; // 이 수정을 가장 우선적으로 추천합니다.
-
-    fixFor.diagnostics = [
-      {
-        message: `label의 htmlFor 속성을 input의 id와 연결하여 접근성을 향상시킵니다.`,
-        range,
-        severity: vscode.DiagnosticSeverity.Warning,
-        source: "a11y-fixer",
-      },
-    ];
-    fixes.push(fixFor);
+  // 1) 현재 라벨 태그(열리는 태그) 가져오기
+  const labelMatch = code.match(/<label\b([^>]*)>/i);
+  if (!labelMatch) {
+    // 라벨 패턴을 찾지 못하면 종료
+    return [];
   }
+  const labelOpenTag = labelMatch[0]; // e.g. <label> or <label class="...">
+  const labelAttrs = labelMatch[1] || ""; // 속성 문자열
 
-  // 2. 오류가 있던 '중첩' 로직은 제거하여 혼동을 방지합니다.
-  // 만약 for 속성으로 연결할 id를 찾지 못했다면, AI 기반 수정(control-has-associated-label)이
-  // 더 복잡한 문맥을 이해하고 해결책을 제시할 수 있으므로, 여기서는 추가적인 제안을 하지 않습니다.
+  // 2) 이미 htmlFor / for 속성이 있으면 수정 제안 불필요
+  if (/\bhtmlFor\s*=|\bfor\s*=/.test(labelAttrs)) {
+    return [];
+  }
+
+  // 3) 라벨과 연결된 input id를 찾기 — 현재 줄을 기준으로 아래/위로 탐색
+  const startLine = range.start.line;
+  const maxLookahead = 10; // 검색할 라인 수(요구에 따라 늘릴수 있음)
+  let foundId: string | null = null;
+
+  // 우선 현재 라인(또는 fullLine)에 id가 있는지 확인
+  const idInFullLine = fullLine.match(/id\s*=\s*["']([^"']+)["']/);
+  if (idInFullLine) {
+    foundId = idInFullLine[1];
+  }
+
+  // 아래쪽으로 탐색
+  if (!foundId) {
+    for (let i = startLine; i <= Math.min(document.lineCount - 1, startLine + maxLookahead); i++) {
+      const text = document.lineAt(i).text;
+      const m = text.match(/id\s*=\s*["']([^"']+)["']/);
+      if (m) {
+        foundId = m[1];
+        break;
+      }
+    }
+  }
+
+  // 위쪽으로 탐색 (아래에서 못 찾았을 때)
+  if (!foundId) {
+    for (let i = startLine - 1; i >= Math.max(0, startLine - maxLookahead); i--) {
+      const text = document.lineAt(i).text;
+      const m = text.match(/id\s*=\s*["']([^"']+)["']/);
+      if (m) {
+        foundId = m[1];
+        break;
+      }
+    }
+  }
+
+  if (!foundId) {
+    // 가까운 라인에서 id를 찾지 못하면 제안하지 않음
+    return [];
+  }
+
+  // 4) 치환: 기존 attrs를 보존하고 htmlFor 속성 추가
+  // labelOpenTag 예시: "<label>" or "<label class=\"x\">"
+  // 새 오프닝 태그를 안전하게 생성
+  // 공백 처리: 만약 attrs가 비어있으면 한 칸 띄우지 않음
+  const attrsPart = labelAttrs.trim();
+  const newLabelOpenTag = attrsPart.length > 0
+    ? `<label ${attrsPart} htmlFor="${foundId}">`
+    : `<label htmlFor="${foundId}">`;
+
+  // code 문자열 내에서 첫 번째 라벨 오프닝 태그를 대체
+  const fixedCode = code.replace(labelOpenTag, newLabelOpenTag);
+
+  // 5) CodeAction 생성
+  const fixFor = new vscode.CodeAction(
+    `input id와 연결 (htmlFor="${foundId}")`,
+    vscode.CodeActionKind.QuickFix
+  );
+  fixFor.edit = new vscode.WorkspaceEdit();
+  fixFor.edit.replace(document.uri, range, fixedCode);
+  fixFor.isPreferred = true;
+
+  fixFor.diagnostics = [
+    new vscode.Diagnostic(
+      range,
+      `label의 htmlFor 속성을 input의 id("${foundId}")와 연결하여 접근성을 향상시킵니다.`,
+      vscode.DiagnosticSeverity.Warning
+    ),
+  ];
 
+  fixes.push(fixFor);
   return fixes;
-};
+};

src/rules/logic/no-noninteractive-tabindex.ts

@@ -9,23 +9,34 @@ export const noNoninteractiveTabindexFix: RuleFixer = (
 ): vscode.CodeAction[] => {
   const { code, range, document } = context;
 
-  // tabIndex 속성과 값만 정확히 일치시키는 정규식으로 수정
-  // 공백을 포함하여 'tabIndex' 속성만 정확히 제거하도록 개선
-  const fixed = code.replace(/\s+tabIndex\s*=\s*(?:".*?"|'.*?'|{[^}]*})/, "");
+  // ✅ 정확히 tabIndex={0} 또는 "0" 등을 매칭
+  const tabIndexRegex = /\s*tabIndex\s*=\s*(?:"[^"]*"|'[^']*'|\{[^}]*\})/;
+  const match = code.match(tabIndexRegex);
 
-  if (fixed === code) {
+  if (!match) {
+    console.warn(`[DEBUG] tabIndex not found in code: ${code}`);
     return [];
   }
 
+  const matchText = match[0];
+  const startOffset = code.indexOf(matchText);
+  const endOffset = startOffset + matchText.length;
+
+  // 실제로 교체할 범위 계산 (range 기준 상대 위치로)
+  const tabIndexRange = new vscode.Range(
+    document.positionAt(document.offsetAt(range.start) + startOffset),
+    document.positionAt(document.offsetAt(range.start) + endOffset)
+  );
+
   const fix = new vscode.CodeAction(
     `tabIndex 속성 제거`,
     vscode.CodeActionKind.QuickFix
   );
 
   fix.edit = new vscode.WorkspaceEdit();
-  fix.edit.replace(document.uri, range, fixed);
-  fix.isPreferred = true;
+  fix.edit.replace(document.uri, tabIndexRange, ""); 
 
+  fix.isPreferred = true;
   fix.diagnostics = [
     {
       message: `tabIndex는 비인터랙티브 요소에 사용될 수 없습니다.`,
@@ -36,4 +47,4 @@ export const noNoninteractiveTabindexFix: RuleFixer = (
   ];
 
   return [fix];
-};
+};

src/rules/logic/tabindex-no-positive.ts

@@ -6,54 +6,59 @@ export function tabindexNoPositiveFix(
 ): vscode.CodeAction[] {
   const fixes: vscode.CodeAction[] = [];
 
-  const regex = /(tabIndex\s*=\s*)(["']?|\{?)(\d+)(["']?|\}?)/;
+  // 캡처 그룹 1: 값 부분 (예: "{2}", "'2'", '"2"')
+  const regex = /tabIndex\s*=\s*(\{?\s*['"]?\s*\d+\s*['"]?\s*\}?)/;
   const match = context.code.match(regex);
 
-  if (!match || match.length < 5) {
+  // [수정됨]
+  // match.length는 2여야 합니다 (match[0]: 전체, match[1]: 캡처 그룹).
+  if (!match || match.length < 2) { 
     console.warn(
       `[DEBUG - tabindexFix] Could not parse tabIndex value from context.code: '${context.code}'. Cannot provide fix.`
     );
     return [];
   }
 
-  const [, prefix, opener, originalValue, closer] = match;
+  // 값 부분(match[1])에서 숫자만 추출합니다.
+  const valueMatch = match[1].match(/\d+/);
+  if (!valueMatch || parseInt(valueMatch[0], 10) <= 0) {
+    // 숫자를 찾지 못했거나 양수가 아니면 종료
+    return [];
+  }
 
-  if (parseInt(originalValue, 10) <= 0) {
-   return []; // 양수가 아니면 수정 제안을 만들지 않고 종료합니다.
- }
-  // 매개변수 'targetValue'에 string 타입을 명시적으로 지정
-  const createReplacementCode = (targetValue: string) => {
-    return context.code.replace(match[0], `${prefix}"${targetValue}"`);
+  // targetValue는 0 또는 -1 숫자입니다.
+  const createReplacementCode = (targetValue: number) => {
+    // [수정됨]
+    // context.code에서 값 부분(match[1])을 새로운 JSX 표현식(예: {0})으로 교체합니다.
+    // (이전 로직은 if문이 중복되어 하나로 통일했습니다.)
+    return context.code.replace(match[1], `{${targetValue}}`);
   };
 
-  // 1. tabIndex를 "0"으로 변경하는 제안
+  // 1. tabIndex를 {0}으로 변경하는 제안
   const fixToZero = new vscode.CodeAction(
-    `tabIndex를 "0"으로 변경 (기본 포커스 순서 포함)`,
+    `tabIndex를 {0}으로 변경 (기본 포커스 순서 포함)`,
     vscode.CodeActionKind.QuickFix
   );
   fixToZero.edit = new vscode.WorkspaceEdit();
   fixToZero.isPreferred = true;
-  const newCodeZero = createReplacementCode("0");
-
+  const newCodeZero = createReplacementCode(0);
   fixToZero.edit.replace(context.document.uri, context.range, newCodeZero);
   fixToZero.diagnostics = [
     new vscode.Diagnostic(
       context.range,
-      `tabIndex에 양수 값 사용은 접근성을 저해할 수 있습니다. (권장: tabIndex="0")`,
+      `tabIndex에 양수 값 사용은 접근성을 저해할 수 있습니다. (권장: tabIndex={0})`,
       vscode.DiagnosticSeverity.Warning
     ),
   ];
   fixes.push(fixToZero);
 
-  // 2. tabIndex를 "-1"로 변경하는 제안
+  // 2. tabIndex를 {-1}로 변경하는 제안
   const fixToMinusOne = new vscode.CodeAction(
-    `tabIndex를 "-1"로 변경 (프로그래밍 방식 포커스만 가능)`,
+    `tabIndex를 {-1}로 변경 (프로그래밍 방식 포커스만 가능)`,
     vscode.CodeActionKind.QuickFix
   );
   fixToMinusOne.edit = new vscode.WorkspaceEdit();
-
-  const newCodeMinusOne = createReplacementCode("-1");
-
+  const newCodeMinusOne = createReplacementCode(-1);
   fixToMinusOne.edit.replace(
     context.document.uri,
     context.range,
@@ -62,7 +67,7 @@ export function tabindexNoPositiveFix(
   fixToMinusOne.diagnostics = [
     new vscode.Diagnostic(
       context.range,
-      `tabIndex에 양수 값 사용은 접근성을 저해할 수 있습니다. (대안: tabIndex="-1")`,
+      `tabIndex에 양수 값 사용은 접근성을 저해할 수 있습니다. (대안: tabIndex={-1})`,
       vscode.DiagnosticSeverity.Warning
     ),
   ];