Fix phpstan/phpstan#14455: missing has-offset via conditional type

- Added processBooleanNotSureSureConditionalTypes method to TypeSpecifier
  that cross-pairs sureNotType conditions with sureType results
- Registered the new method in both BooleanAnd (false context) and
  BooleanOr (true context) conditional holder generation
- New regression test in tests/PHPStan/Analyser/nsrt/bug-14455.php
- Root cause: existing methods only paired sureTypes with sureTypes and
  sureNotTypes with sureNotTypes, missing the cross-pairing needed when
  one side of a compound condition produces sureNotTypes and the other
  produces sureTypes (e.g. empty($arr['key']) && $type === 'filter')

Author: staabm

src/Analyser/TypeSpecifier.php

@@ -741,6 +741,8 @@ public function specifyTypesInCondition(
 					$this->processBooleanNotSureConditionalTypes($scope, $rightTypesForHolders, $leftTypesForHolders),
 					$this->processBooleanSureConditionalTypes($scope, $leftTypesForHolders, $rightTypesForHolders),
 					$this->processBooleanSureConditionalTypes($scope, $rightTypesForHolders, $leftTypesForHolders),
+					$this->processBooleanNotSureSureConditionalTypes($scope, $leftTypesForHolders, $rightTypesForHolders),
+					$this->processBooleanNotSureSureConditionalTypes($scope, $rightTypesForHolders, $leftTypesForHolders),
 				))->setRootExpr($expr);
 			}
 
@@ -790,6 +792,8 @@ public function specifyTypesInCondition(
 					$this->processBooleanNotSureConditionalTypes($scope, $rightTypes, $leftTypes),
 					$this->processBooleanSureConditionalTypes($scope, $leftTypes, $rightTypes),
 					$this->processBooleanSureConditionalTypes($scope, $rightTypes, $leftTypes),
+					$this->processBooleanNotSureSureConditionalTypes($scope, $leftTypes, $rightTypes),
+					$this->processBooleanNotSureSureConditionalTypes($scope, $rightTypes, $leftTypes),
 				))->setRootExpr($expr);
 			}
 
@@ -2119,6 +2123,73 @@ private function processBooleanNotSureConditionalTypes(Scope $scope, SpecifiedTy
 		return [];
 	}
 
+	/**
+	 * @return array<string, ConditionalExpressionHolder[]>
+	 */
+	private function processBooleanNotSureSureConditionalTypes(Scope $scope, SpecifiedTypes $conditionTypes, SpecifiedTypes $resultTypes): array
+	{
+		$conditionExpressionTypes = [];
+		foreach ($conditionTypes->getSureNotTypes() as $exprString => [$expr, $type]) {
+			if (!$expr instanceof Expr\Variable) {
+				continue;
+			}
+			if (!is_string($expr->name)) {
+				continue;
+			}
+
+			$conditionExpressionTypes[$exprString] = ExpressionTypeHolder::createYes(
+				$expr,
+				TypeCombinator::intersect($scope->getType($expr), $type),
+			);
+		}
+
+		if (count($conditionExpressionTypes) > 0) {
+			$holders = [];
+			foreach ($resultTypes->getSureTypes() as $exprString => [$expr, $type]) {
+				if (!$expr instanceof Expr\Variable) {
+					continue;
+				}
+				if (!is_string($expr->name)) {
+					continue;
+				}
+
+				if (!isset($holders[$exprString])) {
+					$holders[$exprString] = [];
+				}
+
+				$conditions = $conditionExpressionTypes;
+				foreach ($conditions as $conditionExprString => $conditionExprTypeHolder) {
+					$conditionExpr = $conditionExprTypeHolder->getExpr();
+					if (!$conditionExpr instanceof Expr\Variable) {
+						continue;
+					}
+					if (!is_string($conditionExpr->name)) {
+						continue;
+					}
+					if ($conditionExpr->name !== $expr->name) {
+						continue;
+					}
+
+					unset($conditions[$conditionExprString]);
+				}
+
+				if (count($conditions) === 0) {
+					continue;
+				}
+
+				$holder = new ConditionalExpressionHolder(
+					$conditions,
+					ExpressionTypeHolder::createYes($expr, TypeCombinator::intersect($scope->getType($expr), $type)),
+				);
+				$holders[$exprString][$holder->getKey()] = $holder;
+			}
+
+			return $holders;
+		}
+
+		return [];
+	}
+
 	/**
 	 * @return array{Expr, ConstantScalarType, Type}|null
 	 */

tests/PHPStan/Analyser/nsrt/bug-14455.php

@@ -0,0 +1,27 @@
+<?php
+
+namespace Bug14455;
+
+use function PHPStan\Testing\assertType;
+
+/**
+ * @param array<string, mixed> $aggregation
+ * @param non-falsy-string $type
+ */
+function testTriviallyTrueConditionSkipped(array $aggregation, string $type): void
+{
+	if (empty($aggregation['field']) && $type === 'filter') {
+		return;
+	}
+
+	assertType("array<string, mixed>", $aggregation);
+	assertType('non-falsy-string', $type);
+
+	if ($type === 'filter') {
+		assertType("non-empty-array<string, mixed>&hasOffset('field')", $aggregation);
+	} else {
+		assertType("array<string, mixed>", $aggregation);
+	}
+
+	assertType('non-falsy-string', $type);
+}