Skip to content

Conversation

@cleptric
Copy link
Member

Updates PHPUnit versions in the CI workflow matrix to the latest releases:

  • PHPUnit 8.5: ^8.5.40^8.5.52 (for PHP 7.2)
  • PHPUnit 9.6: ^9.6.21/^9.6.25^9.6.33 (for PHP 7.3-8.5)

This ensures we're testing against the latest patch releases which include
bug fixes and compatibility improvements.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@cleptric cleptric marked this pull request as ready for review January 29, 2026 16:40
@cleptric cleptric requested a review from stayallive January 29, 2026 16:40
@cleptric cleptric self-assigned this Jan 29, 2026
@cleptric cleptric added the github_actions Pull requests that update GitHub Actions code label Jan 29, 2026
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

* @see https://www.php.net/manual/en/language.oop5.magic.php#object.sleep
*/
public function __sleep()
public function __serialize()
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Serialization protection broken for PHP 7.2 and 7.3

High Severity

The __sleep() and __wakeup() methods were replaced with __serialize() and __unserialize(), but these magic methods were only introduced in PHP 7.4. Since the project supports PHP 7.2+ (per composer.json), on PHP 7.2 and 7.3 these methods won't be called during serialization, breaking the singleton pattern protection. Users on those PHP versions could now serialize/unserialize the HubAdapter singleton.

Fix in Cursor Fix in Web

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants