Sync with 3.x

[abnegate]

Summary by CodeRabbit

• Chores

  • Updated core PHP dependencies to latest compatible versions.

• Bug Fixes

  • Improved timeout handling for database operations to enhance reliability during extended queries.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The pull request updates two PHP package dependencies in composer.json (utopia-php/fetch from ^0.4.2 to 0.5.* and utopia-php/validators from ^0.1.0 to 0.1.*) and adjusts the HTTP client timeout value in the ClickHouse adapter from 30 to 30_000 with a clarifying comment indicating "30 seconds". The timeout adjustment appears to convert from seconds to milliseconds.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'Sync with 3.x' is vague and generic, using non-descriptive language that doesn't clearly convey the specific changes made in the PR.	Consider using a more specific title that describes the actual changes, such as 'Update PHP dependencies and adjust timeout configuration' or 'Upgrade utopia packages and convert timeout to milliseconds'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom Pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ccadf20 and 4049b13.

⛔ Files ignored due to path filters (1)

• composer.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

• composer.json
• src/Audit/Adapter/ClickHouse.php

🔇 Additional comments (2)

composer.json (1)

32-33: No action required—breaking changes have been addressed.

The fetch library upgrade from ^0.4.2 to 0.5.* and the timeout API breaking change have already been implemented. The code correctly uses setTimeout(30_000) in milliseconds (30 seconds) on line 75 of src/Audit/Adapter/ClickHouse.php, which aligns with the new API. The validators constraint was loosened from ^0.1.0 to 0.1.* and is locked to 0.1.0. Note that fetch 0.5.1 requires PHP >=8.1 (increased from >=8.0 in 0.4.2).

src/Audit/Adapter/ClickHouse.php (1)

75-75: No changes needed—the timeout API change is correct.

The utopia-php/fetch 0.5.1 setTimeout() method expects milliseconds, not seconds. The parameter documentation explicitly states "Timeout in milliseconds" and the default value is set to 15000 (15 seconds in milliseconds). The code correctly sets the timeout to 30_000 milliseconds (30 seconds), and the inline comment accurately documents this.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}