diff --git a/scripts/validate_architecture.py b/scripts/validate_architecture.py index 55125df3..956251be 100755 --- a/scripts/validate_architecture.py +++ b/scripts/validate_architecture.py @@ -469,22 +469,20 @@ class ArchitectureValidator: # JS-001: Check for window.apiClient for i, line in enumerate(lines, 1): - if ( - "window.apiClient" in line - and "//" not in line[: line.find("window.apiClient")] - if "window.apiClient" in line - else True - ): - self._add_violation( - rule_id="JS-001", - rule_name="Use apiClient directly", - severity=Severity.WARNING, - file_path=file_path, - line_number=i, - message="Use apiClient directly instead of window.apiClient", - context=line.strip(), - suggestion="Replace window.apiClient with apiClient", - ) + if "window.apiClient" in line: + # Check if it's not in a comment + before_occurrence = line[: line.find("window.apiClient")] + if "//" not in before_occurrence: + self._add_violation( + rule_id="JS-001", + rule_name="Use apiClient directly", + severity=Severity.WARNING, + file_path=file_path, + line_number=i, + message="Use apiClient directly instead of window.apiClient", + context=line.strip(), + suggestion="Replace window.apiClient with apiClient", + ) # JS-002: Check for console usage for i, line in enumerate(lines, 1):