fix[plugin][smartling]: ENG-10907 smartling fixes (#4208)

## Description
Fixes in this PR:
1. Add a "View job in smartling" button on the job menu
2. Fix the webhook not updating when re-loading the plugin
3. Fix the content api query issue
4. Fix the nested symbol data not going for translation

**Loom**
https://www.loom.com/share/f8b2a75c4e234bbdbae3008f8a345ed0



<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Adds a Smartling job view action, updates webhooks when URLs change,
fixes content picker query, includes nested symbols in translation jobs,
and improves localized value extraction in utils.
> 
> - **Smartling Plugin**:
> - **Job UI**: Add "View job in smartling" action that opens the job in
Smartling using `projectId:translationJobUid`.
> - **Webhooks**: Update model webhook whenever the URL differs from the
template.
> - **Content Picker**: Query updated to `published or draft` for job
selection.
>   - **Translation Jobs**:
> - Update existing jobs to include recursively discovered nested
`Symbol` references and avoid duplicates.
> - New jobs also recursively include nested symbols during creation.
> - **Utils** (`packages/utils/src/translation-helpers.ts`):
> - When encountering `@builder.io/core:LocalizedValue`, store string
values directly and recurse into arrays/objects to collect nested
localized values.
> - **Dependencies**:
> - Bump `@builder.io/utils` to `1.1.26` and
`@builder.io/plugin-smartling` to `0.0.23-11`.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
35ce193. 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.25",
+  "version": "1.1.26",
   "description": "Utils for working with Builder.io content",
   "main": "./dist/index.js",
   "scripts": {

packages/utils/package.json

@@ -59,10 +59,27 @@ function recordValue({
     });
   } else if (typeof value === 'object' && value !== null) {
     if (value['@type'] === localizedType) {
-      results[path] = {
-        value: value?.[sourceLocaleId] || value?.Default,
-        instructions,
-      };
+      const extractedValue = value?.[sourceLocaleId] || value?.Default;
+
+      // If the extracted value is a string, store it directly
+      if (typeof extractedValue === 'string') {
+        results[path] = {
+          value: extractedValue,
+          instructions,
+        };
+      } else if (
+        Array.isArray(extractedValue) ||
+        (typeof extractedValue === 'object' && extractedValue !== null)
+      ) {
+        // If the extracted value is an array or object, recurse into it to find more localized values
+        recordValue({
+          results,
+          value: extractedValue,
+          path,
+          instructions,
+          sourceLocaleId,
+        });
+      }
     } else {
       Object.entries(value).forEach(([key, v]) => {
         recordValue({

packages/utils/src/translation-helpers.ts

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

plugins/smartling/package-lock.json

@@ -213,12 +213,12 @@ const initializeSmartlingPlugin = async () => {
       pkg.name
     );
 
-    // Check if webhook URL needs updating
+    // Check if webhook URL needs updating - update whenever the URL is different
     const currentWebhookUrl = existingModel.webhooks?.[0]?.url;
     const newWebhookUrl = updatedTemplate.webhooks[0].url;
 
-    if (currentWebhookUrl && !currentWebhookUrl.includes('preferredVersion=v2') && newWebhookUrl.includes('preferredVersion=v2')) {
-      // Update the existing model with v2 webhook
+    if (currentWebhookUrl !== newWebhookUrl) {
+      // Update the existing model with new webhook configuration
       existingModel.webhooks = updatedTemplate.webhooks;
     }
   }
@@ -649,6 +649,61 @@ const initializeSmartlingPlugin = async () => {
       },
     });
 
+    registerContentAction({
+      label: 'View job in smartling',
+      showIf(content, model) {
+        const translationModel = getTranslationModel();
+        if (!translationModel) return false;
+        if (!model?.name || model.name !== translationModel.name) return false;
+        
+        // translationBatch is set by backend after job is published
+        if (!content.meta || !content.data) return false;
+        
+        const meta = fastClone(content.meta);
+        const data = content.data;
+        
+        // Get project ID from jobDetails
+        const projectId = data?.get?.('jobDetails')?.get?.('project') || data?.get?.('jobDetails')?.project;
+        
+        // Get job UID from translationBatch (only available after job is authorized in Smartling)
+        const translationJobUid = meta?.translationBatch?.translationJobUid;
+        
+        return (
+          projectId &&
+          translationJobUid
+        );
+      },
+      async onClick(translationJob) {
+        if (!translationJob) {
+          appState.snackBar.show('Job information not available');
+          return;
+        }
+        
+        if (!translationJob.meta || !translationJob.data) {
+          appState.snackBar.show('Job information not available');
+          return;
+        }
+        
+        const meta = fastClone(translationJob.meta);
+        const data = translationJob.data;
+        
+        // Get project ID from jobDetails
+        const projectId = data?.get?.('jobDetails')?.get?.('project') || data?.get?.('jobDetails')?.project;
+        
+        // Get job UID from translationBatch
+        const translationJobUid = meta?.translationBatch?.translationJobUid;
+        
+        if (!projectId || !translationJobUid) {
+          appState.snackBar.show('Job information not available');
+          return;
+        }
+        
+        // Construct Smartling job URL with format: projectId:translationJobUid
+        const smartlingJobUrl = `https://dashboard.smartling.com/app/projects/${projectId}/account-jobs/${projectId}:${translationJobUid}`;
+        window.open(smartlingJobUrl, '_blank', 'noreferrer,noopener');
+      },
+    });
+
     registerContentAction({
       label: 'View translation job',
       showIf(content, model) {
@@ -873,13 +928,7 @@ function pickTranslationJob() {
         '@type': '@builder.io/core:Query',
         property: 'query.published',
         operator: 'is',
-        value: 'draft',
-      },
-      {
-        '@type': '@builder.io/core:Query',
-        property: 'query.published',
-        operator: 'is', 
-        value: 'published',
+        value: 'published or draft',
       },
     ],
   });