composer.json (1)

53-53: Use of dev branch alias for utopia-php/config

Bumping to "dev-feat-config-adapters as 1.0.0" makes sense to unlock adapters, but it ties you to a feature branch. Consider either:

• Planning a follow-up to move back to a tagged 1.x release once available, and/or
• Verifying minimum-stability / prefer-stable are configured as expected so this dev constraint doesn’t accidentally relax overall dependency stability.

app/init/configs.php (1)

3-45: Reuse the created config adapter instance

The new PHP adapter wiring looks correct, but you’re instantiating PHP for every Config::load while also creating $configAdapter once.

You could simplify and slightly reduce overhead by reusing the instance:

$configAdapter = new PHP();

Config::load('template-runtimes', __DIR__ . '/../config/template-runtimes.php', $configAdapter);
Config::load('events', __DIR__ . '/../config/events.php', $configAdapter);
// ...
Config::load('templates-site', __DIR__ . '/../config/templates/site.php', $configAdapter);

This keeps the file DRY and makes the adapter choice easier to adjust later.

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

app/init/configs.php (1)

8-43: Migration to utopia-php/config v1.0.0 is complete and correct.

All 36 Config::load calls have been properly updated to use the new adapter-based API. The imports are correct (use Utopia\Config\Adapters\PHP;), and the Config::load signature correctly accepts the adapter as the third parameter: public static function load(string $key, string $path, Adapter $adapter): void. No Config::load calls were missed elsewhere in the codebase.

Optional optimization: Consider reusing the adapter instance.

Creating 36 separate new PHP() instances can be optimized by creating one adapter and reusing it across all calls:

$phpAdapter = new PHP();

Config::load('template-runtimes', __DIR__ . '/../config/template-runtimes.php', $phpAdapter);
Config::load('events', __DIR__ . '/../config/events.php', $phpAdapter);
// ... (apply to all remaining Config::load calls)

This eliminates unnecessary object instantiation overhead. The optimization can be deferred if profiling shows negligible impact.

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro