unicorn/switch-case-braces

All the case clauses in a switch share one scope. A const declared in one clause is in scope in the next, and declaring the same name in two clauses is a syntax error. Braces give each clause its own scope.

A case without a body, which falls through to the next one, is never reported. Empty braces, as in case 1: {}, are always reported.

--fix adds or removes the braces.

Configure

Off by default. To turn it on:

[lint.rules]
"unicorn/switch-case-braces" = "error" # or "warn"

Every option, set to its default:

[lint.rules]
"unicorn/switch-case-braces" = ["error", "always"] # "always" or "avoid"

Examples

Reported:

switch (event.type) {
  case "open":
    show(event.target);
    break;
  default:
    ignore(event);
}

Not reported:

switch (event.type) {
  case "click":
  case "open": {
    show(event.target);
    break;
  }
  default: {
    ignore(event);
  }
}

Options

Option	What it does
"always" (default)	Require braces around the whole body of every case.
"avoid"	Report braces unless the block declares a variable or a function.

switch (event.type) {
  case "open": {
    show(event.target);
    break;
  }
}

switch (event.type) {
  case "open":
    show(event.target);
    break;
  case "close": {
    const panel = find(event.target);
    hide(panel);
    break;
  }
}