Feat auth single instance

[shimonewman]

Summary by CodeRabbit

• Chores

  • Tightened a backend dependency version constraint to a more specific release series.

• Refactor

  • Internal authorization handling was reworked to centralize skip behavior; no public APIs or method signatures changed and no user-facing behavior alterations.

[coderabbitai]

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)

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

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

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

Walkthrough

composer.json: updated the dependency constraint for utopia-php/database from "" to "4.".

src/Audit/Audit.php: replaced static Authorization::skip calls with $this->db->getAuthorization()->skip() throughout (log, logBatch, getters/counts, cleanup). Public method signatures are unchanged and the Validator import usage was removed.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Review composer.json change for intended version constraint semantics and compatibility with the codebase.
• Inspect src/Audit/Audit.php to ensure every Authorization::skip usage was correctly replaced and that the new getAuthorization() path is available on the db instance in all execution contexts.
• Verify removal of the Validator import doesn't leave any unused references or break static analysis.

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'Feat auth single instance' is vague and does not clearly summarize the main changes made in the pull request.	Consider using a more descriptive title that clearly explains the change, such as 'Replace Authorization::skip with database authorization handler' or 'Refactor auth handling to use single database instance'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

---

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.}