Skip to content

fix(api-platform): remove leftover bundle registration and annotations - #475

Merged
Neophytis merged 1 commit into
feature/docker-betafrom
neophytis/fix-api-platform-cleanup
Aug 8, 2026
Merged

fix(api-platform): remove leftover bundle registration and annotations#475
Neophytis merged 1 commit into
feature/docker-betafrom
neophytis/fix-api-platform-cleanup

Conversation

@Neophytis

Copy link
Copy Markdown
Contributor

Summary

The previous chore: remove api-platform/core commit removed the package from composer but missed three files that still referenced it:

  • config/bundles.php still registered ApiPlatformBundle — this crashes the Symfony kernel on every boot with Class "ApiPlatform\Core\Bridge\Symfony\Bundle\ApiPlatformBundle" not found, making the PHP-FPM container exit 255 immediately
  • config/routes/api_platform.yaml still declared an api_platform route type
  • src/Entity/Member.php still had @ApiResource, @ApiProperty, and @ApiFilter annotations

Test plan

  • CI builds the beta image successfully (no fatal on kernel boot)
  • Beta deploy workflow completes without health check failure
  • docker run ghcr.io/bewelcome/rox:docker-beta php-fpm starts PHP-FPM and reaches ready to handle connections

The previous removal of api-platform/core (composer.json + core classes)
missed three files that still referenced the bundle:
- config/bundles.php: ApiPlatformBundle registration (crashes kernel on boot)
- config/routes/api_platform.yaml: route type api_platform (no-op without bundle)
- src/Entity/Member.php: @ApiResource, @ApiProperty, @APIFilter annotations

These leftovers cause a PHP fatal on every container start, making the
beta deploy fail immediately with exit 255.
@Neophytis
Neophytis merged commit 22279d1 into feature/docker-beta Aug 8, 2026
1 of 11 checks passed
@Neophytis
Neophytis deleted the neophytis/fix-api-platform-cleanup branch August 8, 2026 21:11
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.

1 participant