Skip to content

Conversation

@ChiragAgg5k
Copy link
Member

@ChiragAgg5k ChiragAgg5k commented Jan 22, 2026

Summary

  • add Hook unit tests for action/groups/params/injections and ordering
  • document Hook usage and correct package metadata in README
  • include Hook class file

Testing

  • not run (not requested)

Summary by CodeRabbit

  • New Features

    • Introduced Hooks functionality for managing lifecycle events in the Dependency Injection system
  • Documentation

    • Added Hooks usage examples to documentation
    • Updated PHP requirement to 8.2+
    • Refreshed library references throughout
  • Tests

    • Added comprehensive test coverage for the new Hooks feature

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

@coderabbitai
Copy link

coderabbitai bot commented Jan 22, 2026

Warning

Rate limit exceeded

@ChiragAgg5k has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 18 minutes and 17 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

Walkthrough

This pull request introduces the Utopia DI package Hook class, a new lifecycle hook management system for dependency injection. The Hook class provides chainable methods to configure actions, groups, injections, and parameters with metadata including validators, defaults, and ordering. Documentation is updated to reflect the package name change from Utopia HTTP to Utopia DI, PHP version requirement updated to 8.2+, and comprehensive test coverage is added for the new Hook functionality.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 64.29% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: adding Hook tests and updating documentation/README.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

Choose a reason for hiding this comment

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

are we moving hook from https://github.com/utopia-php/servers/blob/main/src/Servers/Hook.php to DI or are we making duplicate implementation? Where should the hook be?

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