Simplify the prompt

Author: ogorzalka

src/Services/OpenAITranslator.php

@@ -24,7 +24,7 @@ final class OpenAITranslator implements Translator
 
     public function __construct(
         string $apiKey,
-        private readonly string $model = 'gpt-4o',
+        private readonly string $model = 'gpt-4',
     ) {
         $this->client = (new Factory())->withApiKey($apiKey)->make();
         $this->languageCodeService = new LanguageCodeService();
@@ -82,7 +82,7 @@ public function translate(array $groups, string $targetLanguage, ?string $source
                 throw new RuntimeException('Invalid JSON in response');
             }
 
-            // Verify translations are valid
+            // Verify that translations are different from the original text
             foreach ($translations as $key => $translation) {
                 if (! isset($strings[$key])) {
                     throw new RuntimeException(sprintf(
@@ -95,6 +95,10 @@ public function translate(array $groups, string $targetLanguage, ?string $source
                 if (! is_string($translation)) {
                     throw new RuntimeException("Invalid translation for key '{$key}': expected string, got ".gettype($translation));
                 }
+
+                if (mb_strtolower($translation) === mb_strtolower($strings[$key]['text'])) {
+                    throw new RuntimeException("Translation for '{$key}' is the same as the original text");
+                }
             }
 
             // Verify all strings are translated
@@ -164,12 +168,13 @@ private function getSystemPrompt(): string
 Your task is to translate text while preserving meaning and context.
 
 Important rules to follow:
-1. Translate the text to the target language, keeping the original text if it's already appropriate in the target language
+1. Always translate the text to the target language, never return it unchanged
 2. Preserve the meaning and context of each string
 3. Use appropriate translations based on context
 4. Return ONLY a valid JSON object with translations, nothing else
 5. Each key in the JSON must be exactly as provided in the input
-6. Your response must be a valid JSON object, starting with { and ending with }
+6. Never return the original text unchanged
+7. Your response must be a valid JSON object, starting with { and ending with }
 
 Example request:
 ```

tests/Feature/TranslationTest.php

@@ -126,38 +126,3 @@ public function translate(array $groups, string $targetLanguage, ?string $source
 
     expect($translation)->toBe('translated_Welcome');
 });
-
-test('it allows translations that are similar to the original text', function () {
-    $similarTranslator = new class implements Translator {
-        public function translate(array $groups, string $targetLanguage, ?string $sourceLanguage = null, ?string $globalContext = null): array
-        {
-            $result = [];
-            foreach ($groups as $groupName => $group) {
-                $translatedStrings = [];
-                foreach ($group->getStrings() as $key => $string) {
-                    // For some words like "OK" or "Menu", the translation might be the same in many languages
-                    $translatedStrings[$key] = new TranslationString(
-                        text: $string->getText(),
-                        translatedText: $string->getText(), // Same as original
-                        context: $string->getContext(),
-                    );
-                }
-                $result[$groupName] = new TranslationGroup($groupName, $translatedStrings);
-            }
-
-            return $result;
-        }
-    };
-
-    $pollingo = Pollingo::make(translator: $similarTranslator);
-    $translation = $pollingo
-        ->to('fr')
-        ->group('ui', [
-            'ok' => 'OK',
-            'menu' => 'Menu',
-        ])
-        ->translate();
-
-    expect($translation['ui']['ok'])->toBe('OK');
-    expect($translation['ui']['menu'])->toBe('Menu');
-});