Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

always combine the result of expansions #9562

Merged
merged 1 commit into from Mar 23, 2023
Merged

always combine the result of expansions #9562

merged 1 commit into from Mar 23, 2023

Conversation

orklah
Copy link
Collaborator

@orklah orklah commented Mar 23, 2023

This fixes #9305

What happened is that we had two array types in $new_return_type_parts at the end of the loop. However, an Union can only take a single array type without having to combine them

There was a specific handling for split_values but I couldn't get why Psalm thinks TTypeAlias or a single type that is extended into two need to be combined but two types would not need to be combined.

This may result in cases where performance are degraded but I'm not sure how to check the need to make a combination without reimplementing half the checks made in TypeCombiner...

@orklah orklah added the release:fix The PR will be included in 'Fixes' section of the release notes label Mar 23, 2023
@orklah orklah requested a review from weirdan March 23, 2023 20:08
@weirdan
Copy link
Collaborator

weirdan commented Mar 23, 2023

There was a specific handling for split_values but I couldn't get why Psalm thinks TTypeAlias or a single type that is extended into two need to be combined but two types would not need to be combined.

I think the special case comes from the imperfect fix for #3929 that failed to consider that generics may need to be combined as well.

This may result in cases where performance are degraded but I'm not sure how to check the need to make a combination without reimplementing half the checks made in TypeCombiner...

Looks good to me. In case we face performance issues, I think it would be better to address them in TypeCombiner.

@orklah orklah merged commit 7bdb2d6 into vimeo:master Mar 23, 2023
22 of 23 checks passed
@provokateurin
Copy link

May I ask when this will be in a release? I don't need it right now, just some rough timeline when the next release is expected would be nice.

@orklah
Copy link
Collaborator Author

orklah commented Mar 30, 2023

released :)

@provokateurin
Copy link

Thanks so much! ❤️

tcarrio pushed a commit to open-feature/php-sdk that referenced this pull request Aug 16, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [vimeo/psalm](https://togithub.com/vimeo/psalm) | require-dev | minor
| `~5.8.0` -> `~5.14.0` |

---

### Release Notes

<details>
<summary>vimeo/psalm (vimeo/psalm)</summary>

### [`v5.14.0`](https://togithub.com/vimeo/psalm/releases/tag/5.14.0)

[Compare
Source](https://togithub.com/vimeo/psalm/compare/5.13.1...5.14.0)

<!-- Release notes generated using configuration in .github/release.yml
at 5.x -->

#### What's Changed

##### Features

- Infer output format from GITHUB_ACTIONS env by
[@&#8203;ro0NL](https://togithub.com/ro0NL) in
[vimeo/psalm#9982
- fix mixed replace return types for arrays by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9981
- make stubbed constant types configurable with comment by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9992
- Allow space before array shape opening brace and added unit tests by
[@&#8203;rhertogh](https://togithub.com/rhertogh) in
[vimeo/psalm#10000
- Add generic stubs for MongoDB BSON classes by
[@&#8203;alcaeus](https://togithub.com/alcaeus) in
[vimeo/psalm#10016
- Allow comments in types (e.g. in array shapes) by
[@&#8203;rhertogh](https://togithub.com/rhertogh) in
[vimeo/psalm#10004
- [#&#8203;10030](https://togithub.com/vimeo/psalm/issues/10030) Fix
missing TaintedSql by [@&#8203;cgocast](https://togithub.com/cgocast) in
[vimeo/psalm#10032
- Automatic LSP container path mapping by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#10033
- added GitHub CI for unit tests with PHP 8.2 and 8.3 by
[@&#8203;thbley](https://togithub.com/thbley) in
[vimeo/psalm#10028
- Modernize LSP progress reporting by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#10050
- Add type detection for PDOStatement::fetchAll(PDO::FETCH_CLASS,
SomeClass::class) by [@&#8203;thbley](https://togithub.com/thbley) in
[vimeo/psalm#10055

##### Fixes

- Fix sprintf single arg splat array by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9975
- Fixed exhuming enums as enums, not as classes by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9988
- Missing composer lock should not invalidate cache created with missing
composer lock by [@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9970
- no error for printf with only 1 arg by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9993
- Fix "days" falsable return not ignored consistently by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9998
- offsetExists/Get/Set/Unset types were too generic by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9999
- Fix inference high order function when closure param is omitted by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#10014
- Fix incorrect positions set in code action edits by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#10034
- Always report UnusedBaseline issues by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#10012
- Fix scanning of intersection types by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#10043
- Fix combination of non-falsy-string and falsy literal by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#10039
- Fix variadic callable type check by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#10066

##### Internal changes

- Fixed psalm self test by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9990
- Fix/phpparser deprecation name parts by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9964
- Simplify conditions with deep object access by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#10037
- Map special RPC paths by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#10046

#### New Contributors

- [@&#8203;rhertogh](https://togithub.com/rhertogh) made their first
contribution in
[vimeo/psalm#10000
- [@&#8203;cgocast](https://togithub.com/cgocast) made their first
contribution in
[vimeo/psalm#10032

**Full Changelog**:
vimeo/psalm@5.13.1...5.14.0

### [`v5.13.1`](https://togithub.com/vimeo/psalm/releases/tag/5.13.1)

[Compare
Source](https://togithub.com/vimeo/psalm/compare/5.13.0...5.13.1)

<!-- Release notes generated using configuration in .github/release.yml
at 5.13.1 -->

#### What's Changed

##### Fixes

- Fix glob CallMap and stub to prevent crash on alpine by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9942

##### Docs

- Fix the link reference in the doc by
[@&#8203;stof](https://togithub.com/stof) in
[vimeo/psalm#9967

**Full Changelog**:
vimeo/psalm@5.13.0...5.13.1

### [`v5.13.0`](https://togithub.com/vimeo/psalm/releases/tag/5.13.0)

[Compare
Source](https://togithub.com/vimeo/psalm/compare/5.12.0...5.13.0)

<!-- Release notes generated using configuration in .github/release.yml
at master -->

#### What's Changed

##### Features

- Add a support for multilevel glob wildcards by
[@&#8203;2e3s](https://togithub.com/2e3s) in
[vimeo/psalm#9813
- Mapping closed inheritance to union during assertion by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9829
- Use PHPStorm output format by default on JetBrains terminals by
[@&#8203;lptn](https://togithub.com/lptn) in
[vimeo/psalm#9848
- sprintf basic non-empty-string return type provider by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9841
- Extend Shepherd payload by information about package versions by
[@&#8203;lptn](https://togithub.com/lptn) in
[vimeo/psalm#9856
- Allow if/false assert for same variable to allow array/list
distinction by [@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9853
- readonly does not have write access, therefore is safe as long as the…
by [@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9887
- Sprintf improve return param type validation by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9877
- feat: cache.gz by [@&#8203;dkarlovi](https://togithub.com/dkarlovi) in
[vimeo/psalm#9889
- make compressor configurable v2 by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9924
- Invalidate cached methods when referenced files are deleted by
[@&#8203;tscni](https://togithub.com/tscni) in
[vimeo/psalm#9931
- Invalidate cached methods when referenced class property types change
by [@&#8203;tscni](https://togithub.com/tscni) in
[vimeo/psalm#9955

##### Fixes

- glob: allow empty string for pattern by
[@&#8203;Hanmac](https://togithub.com/Hanmac) in
[vimeo/psalm#9814
- [GH-9825](https://togithub.com/vimeo/psalm/issues/9825) by
[@&#8203;jack-worman](https://togithub.com/jack-worman) in
[vimeo/psalm#9828
- Fix [#&#8203;9824](https://togithub.com/vimeo/psalm/issues/9824) const
enum self reference by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9833
- base64\_decode returns false only in strict mode by
[@&#8203;kamil-tekiela](https://togithub.com/kamil-tekiela) in
[vimeo/psalm#9832
- Dirname return type non empty string by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9837
- uniqid return type non-empty-string inconsistent by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9842
- PropertyNotSetInConstructor should not report for abstract
constructors by [@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9844
- Intersect template types during inheritance check by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9865
- Fix generic type params mapping by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9866
- Better intersection of template types during inheritance check by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9883
- Update the signature of debug_backtrace by
[@&#8203;stof](https://togithub.com/stof) in
[vimeo/psalm#9868
- Glob return type stub by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9878
- Improve the return types for getimagesize and getimagesizefromstring
by [@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9875
- Unmark DateInterval::$invert as read-only by
[@&#8203;morozov](https://togithub.com/morozov) in
[vimeo/psalm#9895
- Sprintf additional validations and bugfix by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9904
- Fixes [#&#8203;9827](https://togithub.com/vimeo/psalm/issues/9827) by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9903
- set type of **LINE** to positive int by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9921
- Add a stub file for Attributes defined by PHP by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9920
- Don't update types of the outer scope when in an elseif conditional by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9922
- Update callmap for ext-mongodb 1.16 by
[@&#8203;alcaeus](https://togithub.com/alcaeus) in
[vimeo/psalm#9933
- Add call map entries for MongoDB functions by
[@&#8203;alcaeus](https://togithub.com/alcaeus) in
[vimeo/psalm#9934
- fix reconciliation between a list and iterable by
[@&#8203;orklah](https://togithub.com/orklah) in
[vimeo/psalm#9950
- `non-empty-list` assertion for iterable by
[@&#8203;boesing](https://togithub.com/boesing) in
[vimeo/psalm#9953
- fix PHP 7 sprintf too many arguments false positive by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9943

##### Docs

- Document classAndDescendants configuration tag by
[@&#8203;janopae](https://togithub.com/janopae) in
[vimeo/psalm#9867

##### Internal changes

- Introduce and use `IssueData` constants for severity levels by
[@&#8203;lptn](https://togithub.com/lptn) in
[vimeo/psalm#9846
- Shepherd: send a list of issues (instead of array with int keys) by
[@&#8203;lptn](https://togithub.com/lptn) in
[vimeo/psalm#9845
- fix failing mock tests by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9880
- Bump mheap/github-action-required-labels from 4 to 5 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[vimeo/psalm#9898
- fix CI warnings for findUnusedCode in psalm 6 by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9915
- docblock parser minor performance gain by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9916
- improve perfs for switch by not creating reverse assertions against
constants by [@&#8203;orklah](https://togithub.com/orklah) in
[vimeo/psalm#9929
- remove StementsVolatileCache for perf reasons by
[@&#8203;orklah](https://togithub.com/orklah) in
[vimeo/psalm#9930
- put config to true to prevent deprecations for new config files by
[@&#8203;orklah](https://togithub.com/orklah) in
[vimeo/psalm#9857
- Make config warn about missing cache related functions by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9935
- Fix/cache fail silent by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9932

##### Typos

- Fixed typo in ConstantTest by
[@&#8203;staabm](https://togithub.com/staabm) in
[vimeo/psalm#9835

#### New Contributors

- [@&#8203;Hanmac](https://togithub.com/Hanmac) made their first
contribution in
[vimeo/psalm#9814
- [@&#8203;stof](https://togithub.com/stof) made their first
contribution in
[vimeo/psalm#9868

**Full Changelog**:
vimeo/psalm@5.12.0...5.13.0

### [`v5.12.0`](https://togithub.com/vimeo/psalm/releases/tag/5.12.0)

[Compare
Source](https://togithub.com/vimeo/psalm/compare/5.11.0...5.12.0)

<!-- Release notes generated using configuration in .github/release.yml
at master -->

#### What's Changed

##### Features

- WIP - Limit inheritance to a subset of classes
[#&#8203;1450](https://togithub.com/vimeo/psalm/issues/1450) by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9687
- UnsupportedPropertyReferenceUsage by
[@&#8203;jack-worman](https://togithub.com/jack-worman) in
[vimeo/psalm#9769
- Improve return types of pow() by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9795
- Narrow the type of $haystack when strpos(...) !== false by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9788
- allow using more than 8G of memory in psalter by
[@&#8203;orklah](https://togithub.com/orklah) in
[vimeo/psalm#9805

##### Fixes

- Fix missing global class type when scanning cached files by
[@&#8203;tscni](https://togithub.com/tscni) in
[vimeo/psalm#9735
- Fix union argument type check against templated param by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9750
- Allow dynamic properties from PHPDoc by
[@&#8203;fluffycondor](https://togithub.com/fluffycondor) in
[vimeo/psalm#9742
- improve glob stubs by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9752
- Removed return type false from date and gmdate by
[@&#8203;Alkisum](https://togithub.com/Alkisum) in
[vimeo/psalm#9761
- methods and vars cannot be falsy by
[@&#8203;kkmuffme](https://togithub.com/kkmuffme) in
[vimeo/psalm#9796
- fix offset for type param changes by
[@&#8203;orklah](https://togithub.com/orklah) in
[vimeo/psalm#9803

##### Docs

- Documented suggestions for HTML user input by
[@&#8203;mmcev106](https://togithub.com/mmcev106) in
[vimeo/psalm#9780
- Document array shape unsealing syntax by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9784

##### Internal changes

- Enable opcache if it is installed by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#9779

#### New Contributors

- [@&#8203;tscni](https://togithub.com/tscni) made their first
contribution in
[vimeo/psalm#9735
- [@&#8203;Alkisum](https://togithub.com/Alkisum) made their first
contribution in
[vimeo/psalm#9761

**Full Changelog**:
vimeo/psalm@5.11.0...5.12.0

### [`v5.11.0`](https://togithub.com/vimeo/psalm/releases/tag/5.11.0)

[Compare
Source](https://togithub.com/vimeo/psalm/compare/5.10.0...5.11.0)

<!-- Release notes generated using configuration in .github/release.yml
at master -->

#### What's Changed

##### Fixes

- SplDoublyLinkedList's index is always int by
[@&#8203;kamil-tekiela](https://togithub.com/kamil-tekiela) in
[vimeo/psalm#9736
- Fix string negation to never by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9743
- Fix ParadoxicalCondition with complex match expression by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9739
- Fix invalid cast for templated class-string by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9738
- Fix OverriddenPropertyAccess error message by
[@&#8203;edsrzf](https://togithub.com/edsrzf) in
[vimeo/psalm#9740
- Ignore nullable return form SimpleXMLElement::addChild() by
[@&#8203;greg0ire](https://togithub.com/greg0ire) in
[vimeo/psalm#9745

**Full Changelog**:
vimeo/psalm@5.10.0...5.11.0

### [`v5.10.0`](https://togithub.com/vimeo/psalm/releases/tag/5.10.0)

[Compare
Source](https://togithub.com/vimeo/psalm/compare/5.9.0...5.10.0)

<!-- Release notes generated using configuration in .github/release.yml
at master -->

#### What's Changed

##### Features

- Introduce `value-of` with backed enum cases in assertions by
[@&#8203;boesing](https://togithub.com/boesing) in
[vimeo/psalm#9586
- Introduce callable object intersection by
[@&#8203;boesing](https://togithub.com/boesing) in
[vimeo/psalm#9599
- Contextual inference for const callable by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9623
- Allow to intersect type alias with non-type-aliases by
[@&#8203;boesing](https://togithub.com/boesing) in
[vimeo/psalm#9638
- Provide support for templated `value-of` enum values by
[@&#8203;boesing](https://togithub.com/boesing) in
[vimeo/psalm#9655
- (re-)implement object-shape assertions by
[@&#8203;boesing](https://togithub.com/boesing) in
[vimeo/psalm#9656
- Support anonymous object template replacement by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9664
- Implement unsealed array generic syntax by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#9694
- Introduce BeforeExpressionAnalysisEvent by
[@&#8203;tuqqu](https://togithub.com/tuqqu) in
[vimeo/psalm#9725
- Add support for
[@&#8203;psalm-no-seal-properties](https://togithub.com/psalm-no-seal-properties)
and
[@&#8203;psalm-no-seal-methods](https://togithub.com/psalm-no-seal-methods)
by [@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9681

##### Fixes

- Fix remaining intl, reflection. xml and zip callmaps by
[@&#8203;othercorey](https://togithub.com/othercorey) in
[vimeo/psalm#9595
- fix [#&#8203;9049](https://togithub.com/vimeo/psalm/issues/9049) by
[@&#8203;orklah](https://togithub.com/orklah) in
[vimeo/psalm#9598
- fix coercion detection between two keyed arrays by
[@&#8203;orklah](https://togithub.com/orklah) in
[vimeo/psalm#9606
- fix: treat includes starting with '.' and '..' correct by
[@&#8203;jjjb03](https://togithub.com/jjjb03) in
[vimeo/psalm#9608
- Allow enabling mixed issues reporting for error levels > 2. by
[@&#8203;ADmad](https://togithub.com/ADmad) in
[vimeo/psalm#9626
- Preserve `non-empty-string` type when combining literal and numeric
strings by [@&#8203;boesing](https://togithub.com/boesing) in
[vimeo/psalm#9630
- Do not report required closure arguments as unused by
[@&#8203;boesing](https://togithub.com/boesing) in
[vimeo/psalm#9629
- Fix various db callmaps by
[@&#8203;othercorey](https://togithub.com/othercorey) in
[vimeo/psalm#9632
- fix: add missing constructor parameter to IteratorIterator by
[@&#8203;SenseException](https://togithub.com/SenseException) in
[vimeo/psalm#9646
- Flatten `match` arm conditions to check conditions independently by
[@&#8203;tuqqu](https://togithub.com/tuqqu) in
[vimeo/psalm#9647
- Fixed non empty arg issue in array_splice function by
[@&#8203;whizsid](https://togithub.com/whizsid) in
[vimeo/psalm#9643
- Fix `array_flip` to preserve non-empty array type by
[@&#8203;tuqqu](https://togithub.com/tuqqu) in
[vimeo/psalm#9651
- Fix list<T> template replacement by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9658
- Fix iterable template replacement by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9660
- Fix/9373 using enum property as const array key by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9393
- Specify array share for `stream_context_get_params` return value by
[@&#8203;lptn](https://togithub.com/lptn) in
[vimeo/psalm#9663
- Fixed crash issue when using multiple literals for array_splice by
[@&#8203;whizsid](https://togithub.com/whizsid) in
[vimeo/psalm#9673
- Fix comparison of unions of multiple callables by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#9676
- Add stub for DateInterval by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9682
- Return a non-empty-(lowercase-)string from mb_strtolower by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9678
- Don't throw UnnecesseryVarAnnotation when hinting a mixed template var
by [@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9679
- Fix is_object assertions on final classes by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#9686
- Add return type provider for date/gmdate by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9691
- Avoid wrong assertions when working with objects returned by methods
by [@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#9685
- Handle included directory paths without crashing by
[@&#8203;edsrzf](https://togithub.com/edsrzf) in
[vimeo/psalm#9701
- Contextual closure arg inference for class methods by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9700
- Fix [#&#8203;9692](https://togithub.com/vimeo/psalm/issues/9692) by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#9693
- Allow running new on template objects by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#9704
- Fix date return type provider by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#9711
- Fix UnevaluatedCode false positive at declare(strict_types=1) by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9719
- Fix ambiguous constant inheritance false positive by
[@&#8203;Jean85](https://togithub.com/Jean85) in
[vimeo/psalm#9720

##### Docs

- Add CLI help for the error-level option by
[@&#8203;dantleech](https://togithub.com/dantleech) in
[vimeo/psalm#9590
- Replace `@template T as X` with `@template T of X` in documentation by
[@&#8203;janopae](https://togithub.com/janopae) in
[vimeo/psalm#9690
- Make it easier to find how to promote Trace by
[@&#8203;greg0ire](https://togithub.com/greg0ire) in
[vimeo/psalm#9717
- Document [@&#8203;mixin](https://togithub.com/mixin) by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9706

##### Internal changes

- Bump mheap/github-action-required-labels from 3 to 4 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[vimeo/psalm#9601
- Avoid verbose opcache logging by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#9670
- Valid array access on a non-empty-string yields a non-empty-string by
[@&#8203;robchett](https://togithub.com/robchett) in
[vimeo/psalm#9675
- Remove clone leftovers from immutable refactoring by
[@&#8203;danog](https://togithub.com/danog) in
[vimeo/psalm#9709

#### New Contributors

- [@&#8203;dantleech](https://togithub.com/dantleech) made their first
contribution in
[vimeo/psalm#9590
- [@&#8203;jjjb03](https://togithub.com/jjjb03) made their first
contribution in
[vimeo/psalm#9608
- [@&#8203;SenseException](https://togithub.com/SenseException) made
their first contribution in
[vimeo/psalm#9646
- [@&#8203;tuqqu](https://togithub.com/tuqqu) made their first
contribution in
[vimeo/psalm#9647
- [@&#8203;whizsid](https://togithub.com/whizsid) made their first
contribution in
[vimeo/psalm#9643
- [@&#8203;janopae](https://togithub.com/janopae) made their first
contribution in
[vimeo/psalm#9690

**Full Changelog**:
vimeo/psalm@5.9.0...5.10.0

### [`v5.9.0`](https://togithub.com/vimeo/psalm/releases/tag/5.9.0)

[Compare Source](https://togithub.com/vimeo/psalm/compare/5.8.0...5.9.0)

<!-- Release notes generated using configuration in .github/release.yml
at master -->

#### What's Changed

##### Features

- `json_encode()` returns a non-empty-string with `JSON_THROW_ON_ERROR |
JSON_UNESCAPED_UNICODE` by
[@&#8203;fluffycondor](https://togithub.com/fluffycondor) in
[vimeo/psalm#9479
- WIP: Performance/Feature Improvements to the Language Server by
[@&#8203;tm1000](https://togithub.com/tm1000) in
[vimeo/psalm#8960
- Support int separators in docblocks by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#9491
- [@&#8203;property](https://togithub.com/property) annotations: allow
*not* implying
[@&#8203;psalm-seal-properties](https://togithub.com/psalm-seal-properties)
by [@&#8203;danielbeardsley](https://togithub.com/danielbeardsley) in
[vimeo/psalm#9568
- Improve PHP8 str_\* function signatures by
[@&#8203;fluffycondor](https://togithub.com/fluffycondor) in
[vimeo/psalm#9534
- Contextual inference for first-class-callable by
[@&#8203;klimick](https://togithub.com/klimick) in
[vimeo/psalm#9570

##### Fixes

- Compare list generic params by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#9487
- Add `truthy-string` to reserved word list. by
[@&#8203;ZebulanStanphill](https://togithub.com/ZebulanStanphill) in
[vimeo/psalm#9486
- Fix [#&#8203;9433](https://togithub.com/vimeo/psalm/issues/9433) by
[@&#8203;ptomulik](https://togithub.com/ptomulik) in
[vimeo/psalm#9439
- Add missing array, float, int and bool types to callmaps by
[@&#8203;othercorey](https://togithub.com/othercorey) in
[vimeo/psalm#9474
- Fix asserted property mangling by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#9507
- Fix return type of str_split() by
[@&#8203;othercorey](https://togithub.com/othercorey) in
[vimeo/psalm#9504
- Fix integer overflow in array keys by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#9499
- Set inside_call for eval expressions by
[@&#8203;edsrzf](https://togithub.com/edsrzf) in
[vimeo/psalm#9519
- Improve json_encode/json_decode functions signatures by
[@&#8203;fluffycondor](https://togithub.com/fluffycondor) in
[vimeo/psalm#9525
- Fix MissingThrowsDocblock when documented interface is extended by
thrown exception interface by
[@&#8203;paoloconi96](https://togithub.com/paoloconi96) in
[vimeo/psalm#9528
- Fixed [#&#8203;9496](https://togithub.com/vimeo/psalm/issues/9496) by
[@&#8203;ptomulik](https://togithub.com/ptomulik) in
[vimeo/psalm#9497
- Prevent assertion leaks from `else` by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#9538
- Fixed coercion of mixed into non-empty-mixed by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#9541
- always combine the result of expansions by
[@&#8203;orklah](https://togithub.com/orklah) in
[vimeo/psalm#9562
- Clean up unreflectable callmap methods by
[@&#8203;othercorey](https://togithub.com/othercorey) in
[vimeo/psalm#9526
- Add better return type to mysqli fetch_field\* by
[@&#8203;MoonE](https://togithub.com/MoonE) in
[vimeo/psalm#9555
- \[mysql] Add stubs for `mysqli`, `mysqli_result` and `mysqli_stmt`
classes by [@&#8203;phansys](https://togithub.com/phansys) in
[vimeo/psalm#9547
- Fix array_unique callmap by
[@&#8203;othercorey](https://togithub.com/othercorey) in
[vimeo/psalm#9323
- Improve types for range() by
[@&#8203;kamil-tekiela](https://togithub.com/kamil-tekiela) in
[vimeo/psalm#9577
- Fixed [#&#8203;9506](https://togithub.com/vimeo/psalm/issues/9506) by
[@&#8203;ptomulik](https://togithub.com/ptomulik) in
[vimeo/psalm#9509
- Handle different line endings from baseline. by
[@&#8203;DnwK98](https://togithub.com/DnwK98) in
[vimeo/psalm#9566

##### Docs

- Doc/psalm api on methods by
[@&#8203;ygottschalk](https://togithub.com/ygottschalk) in
[vimeo/psalm#9544

##### Internal changes

- Optimize composer caching by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#9493
- Update phar-build to large resource class in config.yml by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#9494
- Refuse to use PHP versions we did not request by
[@&#8203;weirdan](https://togithub.com/weirdan) in
[vimeo/psalm#9492
- Make Atomic subclasses consistently call their parent constructors by
[@&#8203;edsrzf](https://togithub.com/edsrzf) in
[vimeo/psalm#9406
- Remove always-empty ClassAnalyzer leftover_stmts array by
[@&#8203;edsrzf](https://togithub.com/edsrzf) in
[vimeo/psalm#9522
- Skip gnupg extension in callmap tests by
[@&#8203;othercorey](https://togithub.com/othercorey) in
[vimeo/psalm#9576

#### New Contributors

- [@&#8203;ZebulanStanphill](https://togithub.com/ZebulanStanphill) made
their first contribution in
[vimeo/psalm#9486
- [@&#8203;paoloconi96](https://togithub.com/paoloconi96) made their
first contribution in
[vimeo/psalm#9528
- [@&#8203;MoonE](https://togithub.com/MoonE) made their first
contribution in
[vimeo/psalm#9555
- [@&#8203;danielbeardsley](https://togithub.com/danielbeardsley) made
their first contribution in
[vimeo/psalm#9568
- [@&#8203;DnwK98](https://togithub.com/DnwK98) made their first
contribution in
[vimeo/psalm#9566

**Full Changelog**: vimeo/psalm@5.8.0...5.9.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-feature/php-sdk).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4yMi4xIiwidXBkYXRlZEluVmVyIjoiMzYuMjQuMiIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:fix The PR will be included in 'Fixes' section of the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Imported type alias as template parameter inside union type of the same classes does not work
3 participants