noUnsafeFinally (since v11.0.0)

This rule is recommended by Rome.

Disallow control flow statements in finally blocks.

JavaScript suspends the control flow statements of try and catch blocks until the execution of finally block finishes. So, when return, throw, break or continue is used in finally, control flow statements inside try and catch are overwritten, which is considered as unexpected behavior.

Examples

Invalid

jsx

(() => {
    try {
        return 1; // 1 is returned but suspended until finally block ends
    } catch(err) {
        return 2;
    } finally {
        return 3; // 3 is returned before 1, which we did not expect
    }
})();

jsx

(() => {
    try {
        throw new Error("Try"); // error is thrown but suspended until finally block ends
    } finally {
        return 3; // 3 is returned before the error is thrown, which we did not expect
    }
})();

<pre class="language-text"><code class="language-text">correctness/noUnsafeFinally.js:5:9 <a href="https://docs.rome.tools/lint/rules/noUnsafeFinally">lint/correctness/noUnsafeFinally</a> ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ ✖ Unsafe usage of 'return'. 3 │ throw new Error("Try"); // error is thrown but suspended until finally block ends 4 │ } finally { > 5 │ return 3; // 3 is returned before the error is thrown, which we did not expect │ ^^^^^^^^^ 6 │ } 7 │ })(); ℹ 'return' in 'finally' overwrites the control flow statements inside 'try' and 'catch'. </code></pre>

jsx

(() => {
    try {
        throw new Error("Try")
    } catch(err) {
        throw err; // The error thrown from try block is caught and re-thrown
    } finally {
        throw new Error("Finally"); // Finally(...) is thrown, which we did not expect
    }
})();

<pre class="language-text"><code class="language-text">correctness/noUnsafeFinally.js:7:9 <a href="https://docs.rome.tools/lint/rules/noUnsafeFinally">lint/correctness/noUnsafeFinally</a> ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ ✖ Unsafe usage of 'throw'. 5 │ throw err; // The error thrown from try block is caught and re-thrown 6 │ } finally { > 7 │ throw new Error("Finally"); // Finally(...) is thrown, which we did not expect │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^ 8 │ } 9 │ })(); ℹ 'throw' in 'finally' overwrites the control flow statements inside 'try' and 'catch'. </code></pre>

jsx

(() => {
    label: try {
      return 0; // 0 is returned but suspended until finally block ends
    } finally {
      break label; // It breaks out the try-finally block, before 0 is returned.
    }
    return 1;
})();

<pre class="language-text"><code class="language-text">correctness/noUnsafeFinally.js:5:7 <a href="https://docs.rome.tools/lint/rules/noUnsafeFinally">lint/correctness/noUnsafeFinally</a> ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ ✖ Unsafe usage of 'break'. 3 │ return 0; // 0 is returned but suspended until finally block ends 4 │ } finally { > 5 │ break label; // It breaks out the try-finally block, before 0 is returned. │ ^^^^^^^^^^^^ 6 │ } 7 │ return 1; ℹ 'break' in 'finally' overwrites the control flow statements inside 'try' and 'catch'. </code></pre>

jsx

function a() {
  switch (condition) {
    case 'a': {
      try {
        console.log('a');
        return;
      } finally {
        break;
      }
    }
    case 'b': {
      console.log('b');
    }
  }
}

<pre class="language-text"><code class="language-text">correctness/noUnsafeFinally.js:8:9 <a href="https://docs.rome.tools/lint/rules/noUnsafeFinally">lint/correctness/noUnsafeFinally</a> ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ ✖ Unsafe usage of 'break'. 6 │ return; 7 │ } finally { > 8 │ break; │ ^^^^^^ 9 │ } 10 │ } ℹ 'break' in 'finally' overwrites the control flow statements inside 'try' and 'catch'. </code></pre>

Valid

jsx

let foo = function() {
    try {
        return 1;
    } catch(err) {
        return 2;
    } finally {
        console.log("hola!");
    }
};

jsx

let foo = function() {
    try {
        return 1;
    } catch(err) {
        return 2;
    } finally {
        let a = function() {
            return "hola!";
        }
    }
};

jsx

let foo = function(a) {
    try {
        return 1;
    } catch(err) {
        return 2;
    } finally {
        switch(a) {
            case 1: {
                console.log("hola!")
                break;
            }
        }
    }
};

Lint Rule noUnsafeFinally

noUnsafeFinally (since v11.0.0)

Examples

Invalid

Valid

Related links