fix[smartling]: duplicate content entries handling (#4169)

## Description
If we make changes to a content entry and re-add to existing job, it was
adding it twice

_Screenshot_
If relevant, add a screenshot or two of the changes you made.

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Prevents duplicate content when re-adding to Smartling jobs, adds
localization support for `Core:Button` text, updates Smartling strings
URL, and bumps versions.
> 
> - **Smartling plugin**:
> - **Duplicate handling**: `updateLocalJob` filters out content already
in the job and only processes symbols for newly added items.
> - **Smartling strings link**: Updates URL to filter by file URI
(`content.id`) to show strings across jobs.
> - **Versions/Deps**: Bump to `0.0.23-9`; pin `@builder.io/utils` to
`1.1.25`.
> - **Utils (`translation-helpers`)**:
> - **Core:Button localization**: Treats `Core:Button` `options.text`
like `Text`—exposes in `getTranslateableFields` and writes localized
values in `applyTranslation`.
> - **Tests**: Add cases for buttons with plain and pre-localized text.
> - **Versions/Config**: Bump to `1.1.25`; tsconfig adds empty `types`
array.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
4c03217. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: anaghav2023

packages/utils/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@builder.io/utils",
-  "version": "1.1.24",
+  "version": "1.1.25",
   "description": "Utils for working with Builder.io content",
   "main": "./dist/index.js",
   "scripts": {

packages/utils/package.json

@@ -54,6 +54,38 @@ test('getTranslateableFields from content to match snapshot', async () => {
             },
           },
         },
+        {
+          meta: {
+            instructions: 'Button with plain text',
+          },
+          id: 'button-plain-text-id',
+          '@type': '@builder.io/sdk:Element',
+          component: {
+            name: 'Core:Button',
+            options: {
+              text: 'Cute Baby',
+              openLinkInNewTab: false,
+            },
+          },
+        },
+        {
+          meta: {
+            instructions: 'Button with pre-localized text',
+          },
+          id: 'button-localized-text-id',
+          '@type': '@builder.io/sdk:Element',
+          component: {
+            name: 'Core:Button',
+            options: {
+              text: {
+                '@type': localizedType,
+                'en-US': 'Click Me!',
+                Default: 'Click Here',
+              },
+              openLinkInNewTab: true,
+            },
+          },
+        },
         {
           '@type': '@builder.io/sdk:Element',
           id: 'builder-custom-component-id',

packages/utils/src/translation-helpers.test.ts

@@ -190,6 +190,21 @@ export function getTranslateableFields(
         };
       }
 
+      if (el && el.id && el.component?.name === 'Core:Button' && !el.meta?.excludeFromTranslation) {
+        const componentText = el.component.options?.text;
+        if (componentText) {
+          const textValue = typeof componentText === 'string'
+            ? componentText
+            : componentText?.[sourceLocaleId] || componentText?.Default;
+          if (textValue) {
+            results[`blocks.${el.id}#text`] = {
+              value: textValue,
+              instructions: el.meta?.instructions || defaultInstructions,
+            };
+          }
+        }
+      }
+
       if (el && el.id && el.component?.name === 'Symbol') {
         const symbolInputs = Object.entries(el.component?.options?.symbol?.data) || [];
         if (symbolInputs.length) {
@@ -325,6 +340,45 @@ export function applyTranslation(
         });
       }
 
+      // Core:Button special handling - similar to Text component
+      if (
+        el &&
+        el.id &&
+        el.component?.name === 'Core:Button' &&
+        !el.meta?.excludeFromTranslation &&
+        translation[`blocks.${el.id}#text`]
+      ) {
+        const localizedValues =
+          typeof el.component.options?.text === 'string'
+            ? {
+                Default: el.component.options.text,
+              }
+            : el.component.options.text;
+
+        const updatedElement = {
+          ...el,
+          meta: {
+            ...el.meta,
+            translated: true,
+            // this tells the editor that this is a forced localized input similar to clicking the globe icon
+            'transformed.text': 'localized',
+          },
+          component: {
+            ...el.component,
+            options: {
+              ...el.component.options,
+              text: {
+                '@type': localizedType,
+                ...localizedValues,
+                [locale]: unescapeStringOrObject(translation[`blocks.${el.id}#text`].value),
+              },
+            },
+          },
+        };
+        
+        this.update(updatedElement);
+      }
+
       // custom components
       if (el && el.id && el.meta?.localizedTextInputs) {
         // there's a localized input

packages/utils/src/translation-helpers.ts

@@ -7,7 +7,8 @@
     "esModuleInterop": true,
     "skipLibCheck": true,
     "strict": true,
-    "declaration": true
+    "declaration": true,
+    "types": []
   },
   "include": ["./src"],
   "exclude": ["node_modules", "**/*.test.ts"]

packages/utils/tsconfig.json

@@ -1,6 +1,6 @@
 {
   "name": "@builder.io/plugin-smartling",
-  "version": "0.0.23-8",
+  "version": "0.0.23-9",
   "description": "",
   "keywords": [],
   "main": "dist/plugin.system.js",
@@ -125,7 +125,7 @@
     "typescript": "^3.0.3"
   },
   "dependencies": {
-    "@builder.io/utils": "^1.1.23",
+    "@builder.io/utils": "1.1.25",
     "fast-json-stable-stringify": "^2.1.0",
     "lodash": "^4.17.21",
     "object-hash": "^3.0.0",

plugins/smartling/package-lock.json

@@ -692,8 +692,8 @@ const initializeSmartlingPlugin = async () => {
       },
       async onClick(content) {
         const translationBatch = fastClone(content.meta).translationBatch;
-        // https://dashboard.smartling.com/app/projects/0e6193784/strings/jobs/schqxtpcnxix
-        const smartlingFile = `https://dashboard.smartling.com/app/projects/${translationBatch.projectId}/strings/jobs/${translationBatch.translationJobUid}`;
+        // Filter by file URI (content ID) to show all translations across all jobs
+        const smartlingFile = `https://dashboard.smartling.com/app/projects/${translationBatch.projectId}/strings/?urlsFilter.urls=${content.id}&limit=200&offset=0`;
         window.open(smartlingFile, '_blank', 'noreferrer,noopener');
       },
     });

plugins/smartling/package.json

@@ -109,14 +109,18 @@ export class SmartlingApi {
          }
          async updateLocalJob(jobId: string, content: any[]) {
            const latestDraft = await appState.getLatestDraft(jobId);
-           const allContent = [...content];
            const processedSymbols = new Set<string>(); // Avoid duplicates
 
-           // Get existing entries to check for duplicate symbols
+           // Get existing entries to check for duplicate content and symbols
            const existingEntryIds = new Set((latestDraft.data.entries || []).map((entry: any) => entry.content?.id));
 
+           // Filter out content that already exists in the job
+           const newContent = content.filter(c => !existingEntryIds.has(c.id));
+           
+           const allContent = [...newContent];
+           
            // Extract and include symbol content for updates too
-           for (const contentItem of content) {
+           for (const contentItem of newContent) {
              try {
                // Fetch the full content to analyze for symbols
                const fullContent = await fetch(
@@ -159,7 +163,7 @@ export class SmartlingApi {
              published: 'draft',
            };
 
-           const symbolCount = allContent.length - content.length;
+           const symbolCount = allContent.length - newContent.length;
            if (symbolCount > 0) {
            } else {
            }