Skip to content

Conversation

@loks0n
Copy link
Contributor

@loks0n loks0n commented Jan 1, 2026

Summary by CodeRabbit

  • New Features

    • Added AWS S3 storage device support with configurable regions
    • Introduced telemetry monitoring for storage operations
    • Enhanced multi-part upload with automatic retry on failure
    • Added ZIP file type validation support
    • Expanded filename validation to allow dots, hyphens, and underscores
  • Improvements

    • Improved error handling with specific exceptions for missing files
  • Technical Updates

    • Upgraded PHP requirement to version 8.1

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

@loks0n loks0n changed the base branch from main to 0.18.x January 1, 2026 18:34
@coderabbitai
Copy link

coderabbitai bot commented Jan 1, 2026

Caution

Review failed

Failed to post review comments

Walkthrough

This pull request modernizes the storage device infrastructure by updating the PHP runtime requirement to 8.1 (Docker, Composer) and refactoring storage device architecture. It introduces telemetry integration at the Device base class level, adds a new AWS device class extending S3, and substantially refactors S3 to support configurable hosts and operation-based telemetry tracking with built-in retry logic. Related device classes (Backblaze, DOSpaces, Linode, Wasabi) are updated to pass computed hosts to parent constructors. A new NotFoundException exception type, a Telemetry wrapper device class, and new dependencies (utopia-php/telemetry, utopia-php/validators) are added. Validators are enhanced to support hyphens, underscores, and dots in filenames, and ZIP file extension support. CI/CD testing switches from S3 to AWS device, with comprehensive test additions for exception handling and partial upload scenarios.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title 'chore: bump validators' is misleading and does not accurately reflect the main changes in the changeset, which include major version updates (PHP 8.0→8.1), new AWS S3 device implementation, telemetry integration, and exception handling improvements. Revise the title to better summarize the primary changes, such as 'feat: add AWS S3 device, bump PHP to 8.1, and integrate telemetry' or 'chore: upgrade PHP 8.1, add AWS device, and add telemetry integration'.
Docstring Coverage ⚠️ Warning Docstring coverage is 44.19% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings

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

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

@loks0n loks0n merged commit cabf77f into 0.18.x Jan 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants