[DeadCode] Skip RemoveAlwaysTrueIfConditionRector on inline HTML stmts

Unwrapping an always-true if whose body is inline HTML drops the close-tag transition and prints invalid PHP (inline HTML emitted inside PHP mode).

Author: TomasVotruba

rules-tests/DeadCode/Rector/If_/RemoveAlwaysTrueIfConditionRector/Fixture/skip_inline_html.php.inc

@@ -0,0 +1,3 @@
+<?php if (true) { ?>
+<div>always</div>
+<?php }

rules/DeadCode/Rector/If_/RemoveAlwaysTrueIfConditionRector.php

@@ -18,6 +18,7 @@
 use PhpParser\Node\Stmt;
 use PhpParser\Node\Stmt\Else_;
 use PhpParser\Node\Stmt\If_;
+use PhpParser\Node\Stmt\InlineHTML;
 use PhpParser\NodeVisitor;
 use PHPStan\Analyser\Scope;
 use PHPStan\Reflection\ClassReflection;
@@ -122,6 +123,13 @@ public function refactor(Node $node): int|null|array|If_
             return null;
         }
 
+        // unwrapping inline HTML stmts would drop the close-tag transition and break the file
+        foreach ($node->stmts as $stmt) {
+            if ($stmt instanceof InlineHTML) {
+                return null;
+            }
+        }
+
         if ($this->shouldSkipExpr($node->cond)) {
             return null;
         }