Phpstan Rules Laravel Package

Fixes:

• forbidVariableTypeOverwriting: fix false positive with fluent $this returns (#350)
• forbidUnusedException & forbidUnusedMatchResult support remaining edge cases like string concat (#353)
• forbidUselessNullableReturn: fix false positive with property hooks (https://github.com/shipmonk-rnd/phpstan-rules/pull/355)

Fixes:

• forbidCheckedExceptionInYieldingMethod: support functions and (immediately-invoked) closures (#345)
• Add composer.lock to export-ignore (#344)

Fixes:

• forbidCheckedExceptionInCallable: fix reported file for traits (#342)

Fixes:

• No crash or false-positives when running with fibers-based PHPStan 2.1.34+ (#340, #337)

Fixes:

• forbidCast: avoid failure on (void) cast (#336, @dragosprotung)

Fixes:

• enforceReadonlyPublicProperty: skip untyped props + add excludePropertyWithDefaultValue option (#334, @devfrey)

Fixes:

enforceReadonlyPublicProperty: skip static properties (#333, @devfrey)

Improvements:

• enforceReadonlyPublicProperty support asymmetric visibility (#331, @JanTvrdik)
• forbidArithmeticOperationOnNonNumber add support for BcMath\Number (#321, @JokubasR)

Fixes:

• enforceNativeReturnTypehint fix false positive for unions with template type (#325, @mspirkov)
• forbidPhpDocNullabilityMismatchWithNativeTypehint: ignore void methods (#328, @mspirkov)

Dependencies:

• phpstan/phpstan now requires ^2.1.32 (#331)

Fixes:

• forbidUselessNullableReturn: no error for overridable methods (#314 by @vrana)

New features:

• forbidUnusedClosureParameters: report unused trailing params for closures and array functions (#313)

Fixes:

• enforceNativeReturnTypehint: fix property hook false positive (#309)

Fixes:

• forbidUnusedMatchResult: fix false positive for usage in arrow function (#305, @maryo)

Fixes:

• enforceReadonlyPublicProperty: fix false positive with hooked properties (#303)

Fixes:

• forbidUnusedException: fix ->getException()::class false positive (#299)

Fixes:

• forbidCast: fix custom blacklist configuration (#297, @forrest79)

Dependencies

• phpstan/phpstan requires ^2.1.8 (#296)

New features:

• forbidCustomFunctions now detects even fist class callables, anonymous classes and array/string callables in arguments (#288)
  • e.g. new class extends DeniedParent {}
  • e.g. $this->deniedMethod(...)
  • e.g. array_map([$this, 'deniedMethod'], $a)
  • e.g. array_map('denied_function', $a)

Breaking changes:

• Dropped uselessPrivatePropertyNullability as it got implemented in native PHPStan (#283)
• Shrunk function list in forbidEnumInFunctionArguments as most of them are detected by native PHPStan (#278)

Dependencies

• phpstan/phpstan now requires 2.0.0 (#274)

Fixes:

• Fix reflection error on unknown symbols (#273)

Dependencies:

• phpstan/phpstan now requires 1.12.5 (was 1.11.0) (#272)

New features:

• Ease enabling rules one-by-one (#260)

parameters:
    shipmonkRules:
        enableAllRules: false

        allowComparingOnlyComparableTypes:
            enabled: true

New features:

• forbidUnsafeArrayKey: allow only int|string in array keys (deny null, bool, float etc) (#254)

Breaking changes:

• forbidAssignmentNotMatchingVarDoc: removed as there is better native replacement when bleedingEdge is enabled (#238)
• allowNamedArgumentOnlyInAttributes: removed as it was highly opinionated and did not provide any extra strictness (#238)
• forbidCheckedExceptionInCallable: removed config immediatelyCalledCallables (we now respect native [@param-immediately-invoked-callable](https://github.com/param-immediately-invoked-callable), see docs) (#240)
• forbidReturnValueInYieldingMethod: option reportRegardlessOfReturnType is now true by default

Improvements

• All rules accepting classnames in its config checks class existence

Dependencies

• phpstan/phpstan now requires at least 1.11.0 (was 1.10.51)
  • 1.11.0 comes with error identifiers (which we support since 2.10.0)

New features

• forbidNotNormalizedType: check invalid Parent|Child even in multi-catch and [@throws](https://github.com/throws) (#231, #232)

Fixes

• forbidCheckedExceptionInCallable: call_user_func & forward_static_call is immediatelly called (#212)

Dependencies

• remove nikic/php-parser requirement (#222)
  • it is bundled inside PHPStan's phar anyway

Fixes

• Fix compatibility with PHPStan >=1.10.51 (#206)

Dependencies

• phpstan/phpstan now requires at least 1.10.51 (was 1.10.30) (#206)

Fixes:

• forbidEnumInFunctionArguments:
  • allow array_unique($enums, SORT_REGULAR) (#196)
  • deny also natcasesort (#197)

New features:

• forbidNotNormalizedType: deny Child|Parent within unions or intersections; there is no mixed|false, just mixed (#185)
• enforceClosureParamNativeTypehint: enforce closures and array function to have typehinted parameters; configurable to omit when inferred (#192)

Fixes:

• forbidUnusedMatchResult: fix void false positive in latest PHPStan (>= 1.10.49) (https://github.com/shipmonk-rnd/phpstan-rules/pull/189)

Fixes

• allowComparingOnlyComparableTypes: allow safe tuple comparison (lists of equal size & item types) (#182)
  • example: usort($items, fn ($a, $b) => [$a->updatedAt, $a->id] <=> [$b->updatedAt, $b->id])

New features

• forbidArithmeticOperationOnNonNumber: deny arithmetic operations like $a + $b over non numbers (#180)
• forbidIncrementDecrementOnNonInteger: deny $i++, $i--, ++$i, --$i with any non-integer (#179)
• Add error identifiers to all rules: usable in upcoming PHPStan 1.11 (#140)

Fixes

• forbidUnusedMatchResult: fix false positive with assign operators, += etc (#171)

Fixes

• ImmediatelyCalledCallableThrowTypeExtension: ignore implicit throw points https://github.com/shipmonk-rnd/phpstan-rules/pull/163