fix: resolve Docker Compose service name collisions with dots

Services like "api.test" and "api-test" now create unique Traefik labels
by normalizing dots to hyphens and adding a 4-char hash suffix. Update
docker_compose_domains storage to use original service names instead of
transformed names to prevent key collisions.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: andrasbacsai

bootstrap/helpers/docker.php

@@ -11,6 +11,16 @@
 use Symfony\Component\Yaml\Yaml;
 use Visus\Cuid2\Cuid2;
 
+/**
+ * Generate a stable 4-character hash from a service name for uniqueness
+ * This is used to differentiate services like "api.test" and "api-test"
+ * which would otherwise collide when normalized.
+ */
+function serviceNameHash(string $serviceName): string
+{
+    return substr(md5($serviceName), 0, 4);
+}
+
 function getCurrentApplicationContainerStatus(Server $server, int $id, ?int $pullRequestId = null, ?bool $includePullrequests = false): Collection
 {
     $containers = collect([]);
@@ -437,8 +447,13 @@ function fqdnLabelsForTraefik(string $uuid, Collection $domains, bool $is_force_
             $http_label = "http-{$loop}-{$uuid}";
             $https_label = "https-{$loop}-{$uuid}";
             if ($service_name) {
-                $http_label = "http-{$loop}-{$uuid}-{$service_name}";
-                $https_label = "https-{$loop}-{$uuid}-{$service_name}";
+                // Normalize service name for Traefik labels by replacing dots with hyphens
+                // This prevents label parsing issues with service names like "api.test"
+                // Add a 4-char hash to ensure uniqueness for services like "api.test" vs "api-test"
+                $normalized_service_name = str($service_name)->replace('.', '-')->value();
+                $hash = serviceNameHash($service_name);
+                $http_label = "http-{$loop}-{$uuid}-{$normalized_service_name}-{$hash}";
+                $https_label = "https-{$loop}-{$uuid}-{$normalized_service_name}-{$hash}";
             }
             if (str($image)->contains('ghost')) {
                 $labels->push("traefik.http.middlewares.redir-ghost-{$uuid}.redirectregex.regex=^{$path}/(.*)");

bootstrap/helpers/parsers.php

@@ -534,27 +534,30 @@ function applicationParser(Application $resource, int $pull_request_id = 0, ?int
                     if ($resource->build_pack === 'dockercompose') {
                         // Check if a service with this name actually exists
                         $serviceExists = false;
+                        $actualServiceName = null;
                         foreach ($services as $serviceName => $service) {
                             $transformedServiceName = str($serviceName)->replace('-', '_')->replace('.', '_')->value();
                             if ($transformedServiceName === $fqdnFor) {
                                 $serviceExists = true;
+                                $actualServiceName = $serviceName; // Store the ORIGINAL service name
                                 break;
                             }
                         }
 
                         // Only add domain if the service exists
-                        if ($serviceExists) {
+                        if ($serviceExists && $actualServiceName) {
                             $domains = collect(json_decode(data_get($resource, 'docker_compose_domains'))) ?? collect([]);
-                            $domainExists = data_get($domains->get($fqdnFor), 'domain');
+                            // Use the ORIGINAL service name as the key to avoid collisions
+                            $domainExists = data_get($domains->get($actualServiceName), 'domain');
                             $envExists = $resource->environment_variables()->where('key', $key->value())->first();
                             if (str($domainExists)->replace('http://', '')->replace('https://', '')->value() !== $envExists->value) {
                                 $envExists->update([
                                     'value' => $url,
                                 ]);
                             }
                             if (is_null($domainExists)) {
-                                // Put URL in the domains array instead of FQDN
-                                $domains->put((string) $fqdnFor, [
+                                // Put URL in the domains array using ORIGINAL service name
+                                $domains->put((string) $actualServiceName, [
                                     'domain' => $url,
                                 ]);
                                 $resource->docker_compose_domains = $domains->toJson();
@@ -589,26 +592,30 @@ function applicationParser(Application $resource, int $pull_request_id = 0, ?int
                     if ($resource->build_pack === 'dockercompose') {
                         // Check if a service with this name actually exists
                         $serviceExists = false;
+                        $actualServiceName = null;
                         foreach ($services as $serviceName => $service) {
                             $transformedServiceName = str($serviceName)->replace('-', '_')->replace('.', '_')->value();
                             if ($transformedServiceName === $urlFor) {
                                 $serviceExists = true;
+                                $actualServiceName = $serviceName; // Store the ORIGINAL service name
                                 break;
                             }
                         }
 
                         // Only add domain if the service exists
-                        if ($serviceExists) {
+                        if ($serviceExists && $actualServiceName) {
                             $domains = collect(json_decode(data_get($resource, 'docker_compose_domains'))) ?? collect([]);
-                            $domainExists = data_get($domains->get($urlFor), 'domain');
+                            // Use the ORIGINAL service name as the key to avoid collisions
+                            $domainExists = data_get($domains->get($actualServiceName), 'domain');
                             $envExists = $resource->environment_variables()->where('key', $key->value())->first();
                             if ($domainExists !== $envExists->value) {
                                 $envExists->update([
                                     'value' => $urlWithPort,
                                 ]);
                             }
                             if (is_null($domainExists)) {
-                                $domains->put((string) $urlFor, [
+                                // Put URL in the domains array using ORIGINAL service name
+                                $domains->put((string) $actualServiceName, [
                                     'domain' => $urlWithPort,
                                 ]);
                                 $resource->docker_compose_domains = $domains->toJson();
@@ -1067,8 +1074,8 @@ function applicationParser(Application $resource, int $pull_request_id = 0, ?int
         if ($resource->build_pack !== 'dockercompose') {
             $domains = collect([]);
         }
-        $changedServiceName = str($serviceName)->replace('-', '_')->replace('.', '_')->value();
-        $fqdns = data_get($domains, "$changedServiceName.domain");
+        // Use the original service name for lookup (no transformation needed)
+        $fqdns = data_get($domains, "$serviceName.domain");
         // Generate SERVICE_FQDN & SERVICE_URL for dockercompose
         if ($resource->build_pack === 'dockercompose') {
             foreach ($domains as $forServiceName => $domain) {