Skip to content

Conversation

@ChiragAgg5k
Copy link
Member

No description provided.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 18, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat-remove-validator

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.

@ChiragAgg5k ChiragAgg5k changed the title Fix array to string conversion warning in Request::getSize() feat: remove validators and use utopia validators lib Dec 18, 2025
@ChiragAgg5k ChiragAgg5k force-pushed the feat-remove-validator branch from f269c4b to 2ed3ff4 Compare December 18, 2025 08:10
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR migrates validator classes from the Utopia\Http package to the external utopia-php/validators library, reducing code duplication and promoting reusability across Utopia projects.

Key Changes:

  • Removed all validator classes and tests from this repository
  • Updated import statements to use Utopia\Validator instead of Utopia\Http\Validator
  • Added utopia-php/validators 0.1.* as a new dependency

Reviewed changes

Copilot reviewed 50 out of 51 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/Http/Validator.php Removed base Validator abstract class
src/Http/Validator/*.php Removed all validator implementations (AllOf, AnyOf, ArrayList, Assoc, Boolean, Domain, FloatValidator, HexColor, Host, Hostname, IP, Integer, JSON, NoneOf, Nullable, Numeric, Range, Text, URL, WhiteList, Wildcard)
src/Http/Http.php Added import for Utopia\Validator to support validator type checking
src/Http/Hook.php Added import for Utopia\Validator to support param validation
tests/Validator/*.php Removed all validator test files
tests/e2e/init.php Updated import from Utopia\Http\Validator\Text to Utopia\Validator\Text
tests/RouteTest.php Updated import from Utopia\Http\Validator\Text to Utopia\Validator\Text
tests/HttpTest.php Updated import from Utopia\Http\Validator\Text to Utopia\Validator\Text
tests/HookTest.php Updated imports from Utopia\Http\Validator to Utopia\Validator
example/src/server.php Updated import from Utopia\Http\Validator\Text to Utopia\Validator\Text
docs/Getting-Starting-Guide.md Updated import from Utopia\Http\Validator\Wildcard to Utopia\Validator\Wildcard
composer.json Added utopia-php/validators 0.1.* as a dependency
composer.lock Updated with new dependency and dev dependency version updates

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@abnegate abnegate merged commit 3571c49 into 0.34.x Dec 18, 2025
11 checks 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.

3 participants