chore: remove withinDeferredBoundary dead code in require-execsync-try-catch rule

[Copilot]

withinDeferredBoundary was computed in the CallExpression visitor but immediately discarded via void withinDeferredBoundary — the rule always reported regardless of its value. Dead code with misleading comments.

Changes

• eslint-factory/src/rules/require-execsync-try-catch.ts: Remove the ancestors traversal, withinDeferredBoundary variable, its loop, and the surrounding comments (old lines 148–160)
• Remove now-unused isDeferredCallback from the import

-        // Ignore execSync inside deferred callbacks — the parent try block does not protect them.
-        const ancestors = sourceCode.getAncestors(node);
-        let withinDeferredBoundary = false;
-        for (let i = ancestors.length - 1; i >= 0; i--) {
-          if (isDeferredCallback(ancestors[i])) {
-            withinDeferredBoundary = true;
-            break;
-          }
-        }
-        // Still flag it even in deferred callbacks — execSync in async callbacks is still risky.
-        void withinDeferredBoundary;
-
         const argText = node.arguments.length > 0 ? sourceCode.getText(node.arguments[0]) : "";

No behavior change. isInsideTryBlock already handles the deferred-boundary logic correctly via its own crossedDeferredBoundary traversal.

• Fixes [Code Quality] Remove dead code withinDeferredBoundary in require-execsync-try-catch rule (lines 148-160) #47227

[Copilot]

Pull request overview

Removes misleading dead code from the require-execsync-try-catch ESLint rule without changing behavior.

Changes:

• Removed the unused deferred-boundary traversal.
• Removed the now-unused isDeferredCallback import.

Show a summary per file

File	Description
eslint-factory/src/rules/require-execsync-try-catch.ts	Simplifies the rule by removing redundant logic and import.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 1/1 changed files
• Comments generated: 0
• Review effort level: Medium

[github-actions]

🎉 This pull request is included in a new release.

Release: v0.83.0