fix: address review feedback (docs update for batch-size flag and whitespace cleanup)

Author: Hellnight2005

packages/cli/src/cli/cmd/run/index.ts

@@ -125,7 +125,7 @@ export default new Command()
   )
   .option(
     "--batch-size <number>",
-    "Number of translations to process in a single batch",
+    "Number of translations to process in a single batch (not applicable when using lingo.dev provider)",
     (val: string) => parseInt(val),
   )
   .action(async (args) => {

packages/cli/src/cli/localizer/explicit.ts

@@ -279,10 +279,16 @@ function createAiSdkLocalizer(params: {
         try {
           result = parseModelResponse(response.text);
         } catch (e2) {
+          const snippet =
+            response.text.length > 500
+              ? `${response.text.slice(0, 500)}…`
+              : response.text;
           console.error(
-            `Failed to parse response from Lingo.dev. Response: ${response.text}`,
+            `Failed to parse response from Lingo.dev. Response snippet: ${snippet}`,
+          );
+          throw new Error(
+            `Failed to parse response from Lingo.dev: ${e2} (Snippet: ${snippet})`,
           );
-          throw new Error(`Failed to parse response from Lingo.dev: ${e2}`);
         }
         let finalResult: Record<string, any> = {};
 
@@ -321,7 +327,7 @@ function createAiSdkLocalizer(params: {
  */
 function extractPayloadChunks(
   payload: Record<string, string>,
-  batchSize: number = 25,
+  batchSize?: number,
 ): Record<string, string>[] {
   const idealBatchItemSize = 250;
   const result: Record<string, string>[] = [];
@@ -335,9 +341,11 @@ function extractPayloadChunks(
     currentChunkItemCount++;
 
     const currentChunkSize = countWordsInRecord(currentChunk);
+    const effectiveBatchSize =
+      batchSize && batchSize > 0 ? batchSize : payloadEntries.length || 1;
     if (
       currentChunkSize > idealBatchItemSize ||
-      currentChunkItemCount >= batchSize ||
+      currentChunkItemCount >= effectiveBatchSize ||
       i === payloadEntries.length - 1
     ) {
       result.push(currentChunk);

packages/cli/src/cli/processor/basic.spec.ts

@@ -59,6 +59,31 @@ describe("createBasicTranslator", () => {
         );
     });
 
+    it("should process >25 keys in a single batch by default (infinite batch size)", async () => {
+        const inputData: Record<string, string> = {};
+        for (let i = 0; i < 30; i++) {
+            inputData[`key${i}`] = `value${i}`;
+        }
+
+        const input = {
+            sourceLocale: "en",
+            targetLocale: "fr",
+            processableData: inputData,
+        };
+
+        (generateText as any).mockResolvedValue({
+            text: JSON.stringify({ data: {} }),
+        });
+
+        const onProgress = vi.fn();
+        const translator = createBasicTranslator(mockModel, mockSystemPrompt);
+
+        await translator(input, onProgress);
+
+        // Should be 1 call, not 2 (which would happen if default was 25)
+        expect(generateText).toHaveBeenCalledTimes(1);
+    });
+
     it("should respect batchSize parameter", async () => {
         const input = {
             sourceLocale: "en",

packages/cli/src/cli/processor/basic.ts

@@ -97,7 +97,7 @@ export function createBasicTranslator(
  */
 function extractPayloadChunks(
   payload: Record<string, string>,
-  batchSize: number = 25,
+  batchSize?: number,
 ): Record<string, string>[] {
   const idealBatchItemSize = 250;
   const result: Record<string, string>[] = [];
@@ -111,9 +111,11 @@ function extractPayloadChunks(
     currentChunkItemCount++;
 
     const currentChunkSize = countWordsInRecord(currentChunk);
+    const effectiveBatchSize =
+      batchSize && batchSize > 0 ? batchSize : payloadEntries.length || 1;
     if (
       currentChunkSize > idealBatchItemSize ||
-      currentChunkItemCount >= batchSize ||
+      currentChunkItemCount >= effectiveBatchSize ||
       i === payloadEntries.length - 1
     ) {
       result.push(currentChunk);