[Bug]: Quickly opening and merging a PR crashes deploys and leaves ghosts

[ity-alex]

Description

When connecting a resource via Github App and opening a PR, a preview deployment gets queued. If the PR is merged right after it is created (few seconds) the deployment stalls (I assume because there is nothing to pull from?), blocks the queue and the new docker container will not get cleaned up.

We are using the hosted interface at app.coolify.com, love this project so far! This is only a minor issue, not urgent for us at all!

All the best!

Minimal Reproduction (if possible, example repository)

Connect source as Github App, open Pull Request, wait only 2-3 seconds, close/merge the PR.

Exception or Error

No response

Version

v4.0.0-beta.297

[ity-alex]

I found these two possibly relevant code sections:

• Deleting a PR container by webhook when the PR closes (here something seems to be missing):
  

  coolify/app/Http/Controllers/Webhook/Github.php

  Lines 422 to 428 in de7380f

  	$found = ApplicationPreview::where('application_id', $application->id)->where('pull_request_id', $pull_request_id)->first();
  	if ($found) {
  	$container_name = generateApplicationContainerName($application, $pull_request_id);
  	instant_remote_process(["docker rm -f $container_name"], $application->destination->server);
  	ApplicationPullRequestUpdateJob::dispatchSync(application: $application, preview: $found, status: ProcessStatus::CLOSED);
  	$found->delete();

• Cancelling a running deploy, which seems to remove the container correctly:
  

  coolify/app/Livewire/Project/Application/DeploymentNavbar.php

  Line 58 in de7380f

  $kill_command = "docker rm -f {$this->application_deployment_queue->deployment_uuid}";

Maybe the deletion by webhook only removes the final container, while the cancelling of the deploy removes the currently building container. Possible solution would be to check for currently active deploys in the webhook handler?