feat: improve template sync and API test UI

Template Sync Service improvements:
- Add duplicate detection before creating templates in Sinch
- Skip templates that already exist (based on description matching)
- Track skipped count in sync results
- Better logging with existing template detection
- Prevents duplicate templates when syncing multiple times

Settings Template improvements:
- Display app configuration details after successful API test
- Show display name, channels, and channel states
- Warn if no channels are configured
- Better formatting with pre-wrap for multi-line output
- Clarify that API config can be in Globals, messaging config only here

Benefits:
- Prevents creating duplicate templates in Sinch dashboard
- Better user feedback about Sinch configuration status
- Helps diagnose channel configuration issues

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

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

Author: kojiromike

src/Module/Service/TemplateSyncService.php

@@ -44,17 +44,59 @@ public function syncAllTemplates(): array
             'total' => count($templateDefinitions),
             'created' => 0,
             'updated' => 0,
+            'skipped' => 0,
             'failed' => 0,
             'errors' => [],
         ];
 
+        // Get existing templates from Sinch to check for duplicates
+        try {
+            $existingTemplates = $this->apiClient->listTemplates();
+            $existingByDescription = [];
+            foreach ($existingTemplates as $template) {
+                $desc = $template['description'] ?? '';
+                if (!empty($desc)) {
+                    $existingByDescription[$desc] = $template;
+                }
+            }
+            $this->logger->debug(
+                "Found " . count($existingTemplates) . " existing templates in Sinch"
+            );
+        } catch (\Throwable $e) {
+            $this->logger->warning(
+                "Could not list existing templates, will attempt to create all: " . $e->getMessage()
+            );
+            $existingByDescription = [];
+        }
+
         foreach ($templateDefinitions as $template) {
             try {
+                $description = $template['description'] ?? $template['template_name'];
+
+                // Check if template already exists in Sinch
+                if (isset($existingByDescription[$description])) {
+                    $sinchTemplate = $existingByDescription[$description];
+                    $sinchTemplateId = $sinchTemplate['id'] ?? null;
+
+                    $this->logger->debug(
+                        "Template already exists in Sinch: {$template['template_key']}",
+                        ['sinch_id' => $sinchTemplateId]
+                    );
+
+                    // Save/update locally with existing Sinch ID
+                    if ($sinchTemplateId) {
+                        $this->saveTemplateLocally($template, $sinchTemplateId);
+                        $results['skipped']++;
+                        continue;
+                    }
+                }
+
+                // Template doesn't exist, sync it
                 $this->syncTemplate($template);
 
-                // Check if template already exists locally
+                // Check if template already existed locally
                 $existing = $this->getLocalTemplate($template['template_key']);
-                if ($existing) {
+                if ($existing && !empty($existing['sinch_template_id'])) {
                     $results['updated']++;
                 } else {
                     $results['created']++;
@@ -69,6 +111,10 @@ public function syncAllTemplates(): array
                     "Failed to sync template: {$template['template_key']}",
                     ['error' => $e->getMessage()]
                 );
+
+                // Stop on first failure
+                $this->logger->warning("Stopping template sync due to failure");
+                break;
             }
         }
 

templates/settings/config.html.twig

@@ -60,7 +60,7 @@
 
         <div class="alert alert-info" role="alert">
             <strong>{{ 'Note:'|xlt }}</strong>
-            {{ 'You can also configure these settings via Administration > Globals > OpenCoreEMR Sinch Conversations'|xlt }}
+            {{ 'The Sinch API Configuration (Project ID, App ID, API Key, API Secret, Region) can also be configured via Administration > Globals > OpenCoreEMR Sinch Conversations. The Messaging Configuration settings are only available here.'|xlt }}
         </div>
 
         <form method="post" action="?action=save">
@@ -271,7 +271,30 @@
                 .then(data => {
                     resultDiv.style.display = 'block';
                     alertDiv.className = 'alert ' + (data.success ? 'alert-success' : 'alert-danger');
-                    alertDiv.textContent = data.message;
+
+                    let message = data.message;
+
+                    // Show app configuration if available
+                    if (data.success && data.app_config) {
+                        message += '\n\n{{ "App Configuration:"|xlt }}\n';
+                        message += '{{ "Display Name:"|xlt }} ' + (data.app_config.display_name || 'N/A') + '\n';
+
+                        if (data.app_config.channel_credentials && data.app_config.channel_credentials.length > 0) {
+                            message += '\n{{ "Channels:"|xlt }}\n';
+                            data.app_config.channel_credentials.forEach(channel => {
+                                message += '  - ' + (channel.channel || 'Unknown') + ': ' +
+                                          (channel.state || 'Unknown state') + '\n';
+                                if (channel.static_bearer) {
+                                    message += '    {{ "Bearer:"|xlt }} ' + channel.static_bearer.claimed_identity + '\n';
+                                }
+                            });
+                        } else {
+                            message += '\n{{ "⚠️ No channels configured!"|xlt }}\n';
+                        }
+                    }
+
+                    alertDiv.style.whiteSpace = 'pre-wrap';
+                    alertDiv.textContent = message;
                 })
                 .catch(error => {
                     resultDiv.style.display = 'block';