SONARPHP-1889 Add null checks to prevent NPE and suppress warning

[roberto-orlandi-sonarsource]

Part of

---

Summary by Gitar

• Null safety improvements:
  • Added @Nullable annotations and explicit null checks to TokenVisitor methods prevToken and nextToken.
  • Updated ControlStructureSpacingCheck helper methods to handle null tokens gracefully.
• Code maintenance:
  • Suppressed unnecessary recursion warning in PHPGrammar with @SuppressWarnings.
• Testing:
  • Added TokenVisitorTest to verify token navigation logic and edge cases including null inputs.

_{This will update automatically on new commits.}

[hashicorp-vault-sonar-prod]

SONARPHP-1889

[sonarqube-next]

Quality Gate passed

Issues
0 New issues
4 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
70.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

[karim-ouerghemmi-sonarsource]

LGTM!

[gitar-bot]

Code Review ✅ Approved

Adds null checks and nullable annotations to TokenVisitor and ControlStructureSpacingCheck to prevent NPEs, alongside a recursion warning suppression in PHPGrammar. New unit tests confirm the updated token navigation logic.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply	Compact
gitar auto-apply:on	gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}