From a857635396f7be44af6ad3349c4dea43cfa6b780 Mon Sep 17 00:00:00 2001
From: zglicz <michal.zgliczynski@sonarsource.com>
Date: Tue, 30 Apr 2024 15:48:11 +0200
Subject: [PATCH] Fix breaking no-one-iteration-loop after ESLint v9 breaking
 change (#467)

---
 src/rules/no-one-iteration-loop.ts | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/src/rules/no-one-iteration-loop.ts b/src/rules/no-one-iteration-loop.ts
index 1a35b338..932184fe 100644
--- a/src/rules/no-one-iteration-loop.ts
+++ b/src/rules/no-one-iteration-loop.ts
@@ -23,7 +23,6 @@ import type { TSESTree, TSESLint } from '@typescript-eslint/utils';
 import { Rule } from 'eslint';
 import { isContinueStatement } from '../utils/nodes';
 import docsUrl from '../utils/docs-url';
-import CodePath = Rule.CodePath;
 
 const rule: TSESLint.RuleModule<string, string[]> = {
   defaultOptions: [],
@@ -47,7 +46,8 @@ const rule: TSESLint.RuleModule<string, string[]> = {
       loop: TSESTree.WhileStatement | TSESTree.ForStatement;
       segments: Rule.CodePathSegment[];
     }> = [];
-    const currentCodePaths: Rule.CodePath[] = [];
+    const codePathSegments: Rule.CodePathSegment[][] = [];
+    let currentCodePathSegments: Rule.CodePathSegment[] = [];
 
     return {
       ForStatement(node: TSESTree.Node) {
@@ -59,11 +59,18 @@ const rule: TSESLint.RuleModule<string, string[]> = {
       DoWhileStatement(node: TSESTree.Node) {
         loops.add(node);
       },
-      onCodePathStart(codePath: CodePath) {
-        currentCodePaths.push(codePath);
+      onCodePathStart() {
+        codePathSegments.push(currentCodePathSegments);
+        currentCodePathSegments = [];
+      },
+      onCodePathSegmentStart(segment: Rule.CodePathSegment) {
+        currentCodePathSegments.push(segment);
+      },
+      onCodePathSegmentEnd() {
+        currentCodePathSegments.pop();
       },
       onCodePathEnd() {
-        currentCodePaths.pop();
+        currentCodePathSegments = codePathSegments.pop()!;
       },
       'WhileStatement > *'(node: TSESTree.Node) {
         visitLoopChild(node.parent as TSESTree.WhileStatement);
@@ -101,9 +108,8 @@ const rule: TSESLint.RuleModule<string, string[]> = {
     // we visit loop children and collect corresponding path segments as these segments are "toSegment"
     // in "onCodePathSegmentLoop" event.
     function visitLoopChild(parent: TSESTree.WhileStatement | TSESTree.ForStatement) {
-      if (currentCodePaths.length > 0) {
-        const currentCodePath = currentCodePaths[currentCodePaths.length - 1];
-        loopsAndTheirSegments.push({ segments: currentCodePath.currentSegments, loop: parent });
+      if (currentCodePathSegments.length > 0) {
+        loopsAndTheirSegments.push({ segments: [...currentCodePathSegments], loop: parent });
       }
     }
   },