lingui-po-translate 1.0.2 → 1.0.4

package/dist/core/invoke-translation-service.js

@@ -66,6 +66,7 @@ async function runTranslationService(rawInputs, args) {
         serviceConfig: args.serviceConfig,
         prompt: args.prompt,
         baseUrl: args.baseUrl,
+        debug: args.debug,
     };
     console.info(`Invoke '${args.service}' from '${args.srcLng}' to '${args.targetLng}' with ${serviceArgs.strings.length} inputs...`);
     const translationService = await (0, service_definitions_1.instantiateTService)(args.service);

package/dist/core/translate-cli.js

@@ -32,7 +32,7 @@ function formatCliOptions(options) {
 }
 exports.formatCliOptions = formatCliOptions;
 async function translateCli(cliArgs) {
-    var _a, _b, _c;
+    var _a, _b, _c, _d;
     checkForEmptyStringOptions(cliArgs);
     const fileFormats = (0, file_format_definitions_1.getTFileFormatList)();
     const services = (0, service_definitions_1.getTServiceList)();
@@ -73,6 +73,7 @@ async function translateCli(cliArgs) {
         prompt: (_b = cliArgs.prompt) !== null && _b !== void 0 ? _b : "",
         sourceOverride: (0, core_definitions_1.parseSourceOverride)(cliArgs.sourceOverride),
         baseUrl: (_c = cliArgs.baseUrl) !== null && _c !== void 0 ? _c : null,
+        debug: (_d = cliArgs.debug) !== null && _d !== void 0 ? _d : false,
     };
     const result = await (0, translate_core_1.translateCore)(coreArgs);
     const flushTarget = !oldTarget || !(0, tset_ops_1.areEqual)(oldTarget, result.newTarget);

package/dist/core/translate-core.js

@@ -21,6 +21,7 @@ function filterByManualMarking(inputs, args) {
     const toCopyOriginal = new Map();
     const toTranslateFromOverride = new Map();
     inputs.forEach((value, key) => {
+        var _a;
         const parsed = parsedComments.get(key);
         if (!parsed || !(0, comment_parser_1.hasManualMarking)(parsed)) {
             // No @manual marking, translate normally
@@ -30,6 +31,12 @@ function filterByManualMarking(inputs, args) {
         if ((0, comment_parser_1.shouldSkipForManual)(parsed, args.targetLng)) {
             // Target language is in @manual list, skip
             toSkip.set(key, value);
+            // Log @manual skip info
+            const targetTranslation = (_a = args.oldTarget) === null || _a === void 0 ? void 0 : _a.get(key);
+            const translationStatus = targetTranslation
+                ? `already translated: "${targetTranslation}"`
+                : "not yet translated";
+            console.info(`@manual:${args.targetLng} - skip "${key}" (${translationStatus})`);
             return;
         }
         // Has @manual but target not in list

package/dist/file-formats/po/comment-parser.js

@@ -8,11 +8,13 @@ exports.hasManualMarking = exports.shouldSkipForManual = exports.parseExtractedC
  * - @manual:zh-Hans,zh-Hant - languages requiring manual translation
  * - @context:text - context for AI translation
  * - Plain text (without @) - also treated as context
+ * - Multiple directives can be separated by ; or newline
  *
  * Example:
+ * #. @manual:zh-Hans; @context:This is a save button
+ * Or:
  * #. @manual:zh-Hans
  * #. @context:This is a save button
- * #. Some additional context
  *
  * @param extracted - The extracted comment string from PO file
  * @returns ParsedComment with manual languages and context
@@ -25,30 +27,38 @@ function parseExtractedComment(extracted) {
     if (!extracted) {
         return result;
     }
-    const lines = extracted.split("\n");
+    // Split by newline first, then by ; for each line
+    const parts = [];
+    for (const line of extracted.split("\n")) {
+        for (const part of line.split(";")) {
+            const trimmed = part.trim();
+            if (trimmed) {
+                parts.push(trimmed);
+            }
+        }
+    }
     const contextParts = [];
-    for (const line of lines) {
-        const trimmedLine = line.trim();
-        if (trimmedLine.startsWith("@manual:")) {
+    for (const part of parts) {
+        if (part.startsWith("@manual:")) {
             // Parse @manual:zh-Hans,zh-Hant
-            const langList = trimmedLine.substring("@manual:".length).trim();
+            const langList = part.substring("@manual:".length).trim();
             if (langList) {
                 result.manual = langList.split(",").map((lang) => lang.trim()).filter(Boolean);
             }
         }
-        else if (trimmedLine.startsWith("@context:")) {
+        else if (part.startsWith("@context:")) {
             // Parse @context:text
-            const contextText = trimmedLine.substring("@context:".length).trim();
+            const contextText = part.substring("@context:".length).trim();
             if (contextText) {
                 contextParts.push(contextText);
             }
         }
-        else if (trimmedLine && !trimmedLine.startsWith("@")) {
+        else if (!part.startsWith("@")) {
             // Plain text without @ prefix is also context
-            contextParts.push(trimmedLine);
+            contextParts.push(part);
         }
     }
-    result.context = contextParts.join("\n");
+    result.context = contextParts.join(" ");
     return result;
 }
 exports.parseExtractedComment = parseExtractedComment;

package/dist/index.js

@@ -32,6 +32,7 @@ function run(process, cliBinDir) {
         .option("--prompt <prompt>", "supply a prompt for the AI translation service")
         .option("--sourceOverride <mapping>", "override source language for specific targets (e.g., 'zh-Hant:zh-Hans,pt-BR:pt-PT')")
         .option("--baseUrl <url>", "custom API base URL for OpenAI-compatible services (e.g., 'https://api.openai.com/v1')")
+        .option("--debug", "print debug information including API requests")
         .version((0, extract_version_1.extractVersion)({ cliBinDir }), "-v, --version")
         .parse(process.argv);
     if ((_a = commander_1.default.args) === null || _a === void 0 ? void 0 : _a.length) {
@@ -51,6 +52,7 @@ function run(process, cliBinDir) {
         prompt: commander_1.default.opts().prompt,
         sourceOverride: commander_1.default.opts().sourceOverride,
         baseUrl: commander_1.default.opts().baseUrl,
+        debug: commander_1.default.opts().debug,
     };
     (0, translate_cli_1.translateCli)(args)
         .then(() => {

package/dist/services/openai-translate.js

@@ -15,13 +15,23 @@ async function translateSingleString(tString, args) {
         basePath: (_b = (_a = args.baseUrl) !== null && _a !== void 0 ? _a : process.env.OPENAI_BASE_URL) !== null && _b !== void 0 ? _b : undefined,
     });
     const openai = new openai_1.OpenAIApi(configuration);
-    const prompt = generatePrompt(tString, args);
+    const { systemPrompt, userPrompt } = generatePrompt(tString, args);
     const messages = [
+        {
+            role: "system",
+            content: systemPrompt,
+        },
         {
             role: "user",
-            content: prompt,
+            content: userPrompt,
         },
     ];
+    if (args.debug) {
+        console.log("\n[DEBUG] OpenAI Request:");
+        console.log("  Model: gpt-4o-mini-2024-07-18");
+        console.log("  System:", systemPrompt);
+        console.log("  User:", userPrompt);
+    }
     /**
      * https://platform.openai.com/docs/api-reference/completions/create
      * What sampling temperature to use, between 0 and 2. Higher values like 0.8 will make the output more random, while lower values like 0.2 will make it more focused and deterministic.
@@ -34,6 +44,9 @@ async function translateSingleString(tString, args) {
             temperature: 0,
             max_tokens: 2048,
         });
+        if (args.debug) {
+            console.log("\n[DEBUG] OpenAI Full Response:", JSON.stringify(completion.data, null, 2));
+        }
         const text = (_c = completion.data.choices[0].message) === null || _c === void 0 ? void 0 : _c.content;
         if (text == undefined) {
             (0, util_1.logFatal)("OpenAI returned undefined for prompt " + prompt);
@@ -53,16 +66,22 @@ async function translateSingleString(tString, args) {
     }
 }
 function generatePrompt(tString, args) {
-    const capitalizedText = tString.value;
-    const initialPrompt = `only translate my software string from ${args.srcLng} to ${args.targetLng}. don't chat or explain. Using the correct terms for computer software in the target language, only show target language never repeat string. if you don't find something to translate, don't respond`;
-    let contextInfo = `\n\nkey (used for context): ${tString.key}`;
+    // System: Role, language pair, rules, context, and custom instructions
+    let systemPrompt = `You are a software UI translator.
+
+Task: Translate from ${args.srcLng} to ${args.targetLng}.
+Input: User provides text in <source> tags.
+Output: Return only the translated text, without any tags or explanations.
+Rules: Keep all placeholders ({name}, %s, %d), HTML tags, and formatting unchanged.`;
     if (tString.context) {
-        contextInfo += `\n\ncontext: ${tString.context}`;
+        systemPrompt += `\nContext: ${tString.context}`;
+    }
+    if (args.prompt) {
+        systemPrompt += `\nNote: ${args.prompt}`;
     }
-    return (initialPrompt +
-        (args.prompt ? `\n\n${args.prompt}` : "") +
-        contextInfo +
-        `\n\nstring to translate: ${capitalizedText}`);
+    // User: Text wrapped in XML tags
+    const userPrompt = `<source>${tString.value}</source>`;
+    return { systemPrompt, userPrompt };
 }
 async function translateBatch(batch, args) {
     console.log("Translate a batch of " + batch.length + " strings with OpenAI...");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lingui-po-translate",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "AI-powered translation tool for Lingui PO files with context-aware translations",
   "repository": {
     "type": "git",