clean up/fix incorrect switch statements

[iisaduan]

Fixes redundant/incorrect break statements and broken fallthroughs (and some other statickcheck warnings)

[Copilot]

Pull Request Overview

This PR cleans up and fixes issues with switch statements throughout the codebase by removing redundant break statements, fixing broken fallthrough logic, and addressing other static analysis warnings.

• Removes unnecessary break statements from switch cases where they provide no value
• Converts if-else chains to more readable switch statements
• Fixes incorrect logic flow in switch statements and conditional expressions

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
internal/ls/utilities.go	Removed redundant break statements and fixed fallthrough logic in switch cases, converted if-else chains to switch statements
internal/ls/findallreferences.go	Converted if-else chain to switch statement and removed unnecessary break
internal/ls/completions.go	Converted if-else chain to switch statement
internal/checker/nodebuilderimpl.go	Removed redundant double negation and simplified conditional check
internal/checker/jsx.go	Removed redundant conditional wrapper around code block
internal/checker/emitresolver.go	Removed redundant double negation from boolean expression
internal/checker/checker.go	Fixed variable declaration scope and removed redundant conditional check
internal/ast/utilities.go	Fixed loop condition and control flow in GetSuperContainer function