@kontent-ai/migration-toolkit 2.4.0 → 2.5.0

package/lib/import/importers/assets-importer.ts

@@ -1,19 +1,10 @@
-import { AssetModels, LanguageModels, ManagementClient, SharedContracts } from '@kontent-ai/management-sdk';
+import type { AssetModels, LanguageModels, ManagementClient, SharedContracts } from '@kontent-ai/management-sdk';
 import chalk from 'chalk';
 import mime from 'mime';
-import {
-    geSizeInBytes,
-    isNotUndefined,
-    Logger,
-    LogSpinnerData,
-    MigrationAsset,
-    MigrationAssetDescription,
-    MigrationReference,
-    processItemsAsync,
-    runMapiRequestAsync
-} from '../../core/index.js';
+import type { Logger, LogSpinnerData, MigrationAsset, MigrationAssetDescription, MigrationReference } from '../../core/index.js';
+import { geSizeInBytes, isNotUndefined, processItemsAsync, runMapiRequestAsync } from '../../core/index.js';
 import { shouldReplaceBinaryFile, shouldUpdateAsset } from '../comparers/asset-comparer.js';
-import { AssetToEdit, EditedAsset, ImportContext, ImportedAsset, ImportResult } from '../import.models.js';
+import type { AssetToEdit, EditedAsset, ImportContext, ImportedAsset, ImportResult } from '../import.models.js';
 
 export function assetsImporter(data: {
     readonly logger: Logger;

package/lib/import/importers/content-items-importer.ts

@@ -1,7 +1,8 @@
-import { ContentItemModels, ManagementClient } from '@kontent-ai/management-sdk';
+import type { ContentItemModels, ManagementClient } from '@kontent-ai/management-sdk';
 import chalk from 'chalk';
-import { LogSpinnerData, Logger, MigrationItem, findRequired, processItemsAsync, runMapiRequestAsync } from '../../core/index.js';
-import { ImportContext, ImportedItem } from '../import.models.js';
+import type { LogSpinnerData, Logger, MigrationItem } from '../../core/index.js';
+import { findRequired, processItemsAsync, runMapiRequestAsync } from '../../core/index.js';
+import type { ImportContext, ImportedItem } from '../import.models.js';
 
 export function contentItemsImporter(data: {
     readonly logger: Logger;

package/lib/import/importers/language-variant-importer.ts

@@ -1,22 +1,18 @@
-import { ElementContracts, LanguageVariantModels, ManagementClient, WorkflowModels } from '@kontent-ai/management-sdk';
+import type { ElementContracts, LanguageVariantModels, ManagementClient, WorkflowModels } from '@kontent-ai/management-sdk';
 import chalk from 'chalk';
 import { match } from 'ts-pattern';
-import {
-    findRequired,
-    isNotUndefined,
+import type {
     LanguageVariantSchedulesStateValues,
     LanguageVariantStateInTargetEnvironmentByCodename,
     Logger,
     LogSpinnerData,
     MigrationElement,
     MigrationItem,
-    MigrationItemVersion,
-    processItemsAsync,
-    runMapiRequestAsync,
-    workflowHelper
+    MigrationItemVersion
 } from '../../core/index.js';
+import { findRequired, isNotUndefined, processItemsAsync, runMapiRequestAsync, workflowHelper } from '../../core/index.js';
 import { importTransforms } from '../../translation/index.js';
-import { ImportContext, ImportedItem, ImportedLanguageVariant } from '../import.models.js';
+import type { ImportContext, ImportedItem, ImportedLanguageVariant } from '../import.models.js';
 import { throwErrorForMigrationItem } from '../utils/import.utils.js';
 import { workflowImporter as workflowImporterInit } from './workflow-importer.js';
 

package/lib/import/importers/workflow-importer.ts

@@ -1,15 +1,8 @@
-import { LanguageVariantModels, ManagementClient, SharedModels, WorkflowModels } from '@kontent-ai/management-sdk';
+import type { LanguageVariantModels, ManagementClient, WorkflowModels } from '@kontent-ai/management-sdk';
+import { SharedModels } from '@kontent-ai/management-sdk';
 import { match } from 'ts-pattern';
-import {
-    Logger,
-    LogSpinnerData,
-    MigrationItem,
-    MigrationItemVersion,
-    runMapiRequestAsync,
-    ShortestPathResult,
-    workflowHelper as workflowHelperInit,
-    WorkflowStep
-} from '../../core/index.js';
+import type { Logger, LogSpinnerData, MigrationItem, MigrationItemVersion, ShortestPathResult, WorkflowStep } from '../../core/index.js';
+import { runMapiRequestAsync, workflowHelper as workflowHelperInit } from '../../core/index.js';
 
 export function workflowImporter(config: {
     readonly logger: Logger;

package/lib/import/index.ts

@@ -1,2 +1,2 @@
-export * from './import.models.js';
 export * from './import-manager.js';
+export * from './import.models.js';

package/lib/import/utils/import.utils.ts

@@ -1,5 +1,5 @@
 import chalk from 'chalk';
-import { MigrationItem } from 'lib/core/index.js';
+import type { MigrationItem } from '../../core/index.js';
 
 export function throwErrorForMigrationItem(migrationItem: MigrationItem, message: string): never {
     throw Error(

package/lib/index.ts

@@ -1,9 +1,8 @@
 // Public API
 export * from './core/index.js';
 export * from './export/index.js';
-export * from './import/index.js';
 export * from './file/index.js';
-export * from './zip/index.js';
+export * from './import/index.js';
 export * from './toolkit/index.js';
-export * from './file/index.js';
 export * from './translation/index.js';
+export * from './zip/index.js';

package/lib/metadata.ts

@@ -1,6 +1,6 @@
 
 export const libMetadata = {
 	name: '@kontent-ai/migration-toolkit',
-    timestamp: 'Thu, 02 Jan 2025 12:45:27 GMT',
-    version: '2.4.0'
+    timestamp: 'Tue, 11 Mar 2025 11:12:11 GMT',
+    version: '2.5.0'
 };

package/lib/node/cli/actions/export-action.ts

@@ -1,6 +1,6 @@
 import { confirmExportAsync, defaultZipFilename, getDefaultLogger } from '../../../core/index.js';
 import { exportAsync, storeAsync } from '../../../toolkit/index.js';
-import { CliArgumentsFetcher } from '../cli.models.js';
+import type { CliArgumentsFetcher } from '../cli.models.js';
 
 export async function exportActionAsync(cliFetcher: CliArgumentsFetcher): Promise<void> {
     const logger = getDefaultLogger();

package/lib/node/cli/actions/import-action.ts

@@ -1,6 +1,6 @@
 import { confirmImportAsync, defaultZipFilename, getDefaultLogger } from '../../../core/index.js';
 import { extractAsync, importAsync } from '../../../toolkit/index.js';
-import { CliArgumentsFetcher } from '../cli.models.js';
+import type { CliArgumentsFetcher } from '../cli.models.js';
 
 export async function importActionAsync(argsFetcher: CliArgumentsFetcher): Promise<void> {
     const log = getDefaultLogger();

package/lib/node/cli/actions/migrate-action.ts

@@ -1,7 +1,7 @@
-import { migrateAsync } from '../../../toolkit/index.js';
 import { confirmMigrateAsync, getDefaultLogger } from '../../../core/index.js';
-import { SourceExportItem } from '../../../export/index.js';
-import { CliArgumentsFetcher } from '../cli.models.js';
+import type { SourceExportItem } from '../../../export/index.js';
+import { migrateAsync } from '../../../toolkit/index.js';
+import type { CliArgumentsFetcher } from '../cli.models.js';
 
 export async function migrateActionAsync(argsFetcher: CliArgumentsFetcher): Promise<void> {
     const log = getDefaultLogger();

package/lib/node/cli/app.ts

@@ -1,13 +1,13 @@
 #!/usr/bin/env node
 import chalk from 'chalk';
 
-import { handleError, exitProgram } from '../../core/index.js';
+import { match } from 'ts-pattern';
+import { exitProgram, handleError } from '../../core/index.js';
 import { exportActionAsync } from './actions/export-action.js';
 import { importActionAsync } from './actions/import-action.js';
 import { migrateActionAsync } from './actions/migrate-action.js';
 import { argumentsFetcherAsync } from './args/args-fetcher.js';
 import { cliArgs } from './commands.js';
-import { match } from 'ts-pattern';
 
 // Need to register --help commands
 cliArgs.registerCommands();

package/lib/node/cli/args/args-fetcher.ts

@@ -1,9 +1,10 @@
-import yargs from 'yargs';
-import { hideBin } from 'yargs/helpers';
 import chalk from 'chalk';
-import { CliAction, exitProgram, LiteralUnion } from '../../../core/index.js';
-import { CliArgumentsFetcher } from '../cli.models.js';
 import { match } from 'ts-pattern';
+import yargs from 'yargs';
+import { hideBin } from 'yargs/helpers';
+import type { CliAction, LiteralUnion } from '../../../core/index.js';
+import { exitProgram } from '../../../core/index.js';
+import type { CliArgumentsFetcher } from '../cli.models.js';
 
 type ArgvResult = {
     [x: string]: unknown;

package/lib/node/cli/args/args-setter.ts

@@ -1,6 +1,6 @@
 import yargs from 'yargs';
 import { hideBin } from 'yargs/helpers';
-import { CliArgumentsSetter, Command, CommandOption } from '../cli.models.js';
+import type { CliArgumentsSetter, Command, CommandOption } from '../cli.models.js';
 
 export function argumentsSetter(): CliArgumentsSetter {
     const argv = yargs(hideBin(process.argv));

package/lib/node/cli/cli.models.ts

@@ -1,4 +1,4 @@
-import { CliAction } from '../../core/index.js';
+import type { CliAction } from '../../core/index.js';
 
 export interface Command {
     readonly name: string;

package/lib/node/cli/commands.ts

@@ -4,9 +4,7 @@ export const cliArgs = argumentsSetter()
     .withCommand({
         name: 'export',
         description: 'Exports content (items & assets) from Kontent.ai environment',
-        examples: [
-            `kontent-ai-migration-toolkit export --sourceEnvironmentId=x --sourceApiKey=x --language=default --items=itemA,itemB`
-        ],
+        examples: [`kontent-ai-migration-toolkit export --sourceEnvironmentId=x --sourceApiKey=x --language=default --items=itemA,itemB`],
         options: [
             {
                 name: `sourceApiKey`,
@@ -95,8 +93,7 @@ export const cliArgs = argumentsSetter()
 
     .withCommand({
         name: 'migrate',
-        description:
-            'Migrates content (items & assets) from one Kontent.ai environment into another Kontent.ai environment',
+        description: 'Migrates content (items & assets) from one Kontent.ai environment into another Kontent.ai environment',
         examples: [
             `kontent-ai-migration-toolkit migrate --targetEnvironmentId=x --targetApiKey=x --sourceEnvironmentId=x --sourceApiKey=x --language=default --items=itemA,itemB`
         ],

package/lib/toolkit/elements-builder.ts

@@ -1,4 +1,4 @@
-import { MigrationElementModels } from '../core/index.js';
+import type { MigrationElementModels } from '../core/index.js';
 
 export const elementsBuilder = {
     textElement: (data: Omit<MigrationElementModels.TextElement, 'type'>): MigrationElementModels.TextElement => {
@@ -19,17 +19,13 @@ export const elementsBuilder = {
             type: 'custom'
         };
     },
-    dateTimeElement: (
-        data: Omit<MigrationElementModels.DateTimeElement, 'type'>
-    ): MigrationElementModels.DateTimeElement => {
+    dateTimeElement: (data: Omit<MigrationElementModels.DateTimeElement, 'type'>): MigrationElementModels.DateTimeElement => {
         return {
             ...data,
             type: 'date_time'
         };
     },
-    linkedItemsElement: (
-        data: Omit<MigrationElementModels.LinkedItemsElement, 'type'>
-    ): MigrationElementModels.LinkedItemsElement => {
+    linkedItemsElement: (data: Omit<MigrationElementModels.LinkedItemsElement, 'type'>): MigrationElementModels.LinkedItemsElement => {
         return {
             ...data,
             type: 'modular_content'
@@ -49,33 +45,25 @@ export const elementsBuilder = {
             type: 'number'
         };
     },
-    richTextElement: (
-        data: Omit<MigrationElementModels.RichTextElement, 'type'>
-    ): MigrationElementModels.RichTextElement => {
+    richTextElement: (data: Omit<MigrationElementModels.RichTextElement, 'type'>): MigrationElementModels.RichTextElement => {
         return {
             ...data,
             type: 'rich_text'
         };
     },
-    subpagesElement: (
-        data: Omit<MigrationElementModels.SubpagesElement, 'type'>
-    ): MigrationElementModels.SubpagesElement => {
+    subpagesElement: (data: Omit<MigrationElementModels.SubpagesElement, 'type'>): MigrationElementModels.SubpagesElement => {
         return {
             ...data,
             type: 'subpages'
         };
     },
-    taxonomyElement: (
-        data: Omit<MigrationElementModels.TaxonomyElement, 'type'>
-    ): MigrationElementModels.TaxonomyElement => {
+    taxonomyElement: (data: Omit<MigrationElementModels.TaxonomyElement, 'type'>): MigrationElementModels.TaxonomyElement => {
         return {
             ...data,
             type: 'taxonomy'
         };
     },
-    urlSlugElement: (
-        data: Omit<MigrationElementModels.UrlSlugElement, 'type'>
-    ): MigrationElementModels.UrlSlugElement => {
+    urlSlugElement: (data: Omit<MigrationElementModels.UrlSlugElement, 'type'>): MigrationElementModels.UrlSlugElement => {
         return {
             ...data,
             type: 'url_slug'

package/lib/toolkit/export.ts

@@ -1,5 +1,7 @@
-import { MigrationData, executeWithTrackingAsync } from '../core/index.js';
-import { ExportConfig, exportManager } from '../export/index.js';
+import type { MigrationData } from '../core/index.js';
+import { executeWithTrackingAsync } from '../core/index.js';
+import type { ExportConfig } from '../export/index.js';
+import { exportManager } from '../export/index.js';
 import { libMetadata } from '../metadata.js';
 
 export async function exportAsync(config: ExportConfig): Promise<MigrationData> {

package/lib/toolkit/file.ts

@@ -1,5 +1,6 @@
 import { Buffer as BufferProxy } from 'buffer';
-import { defaultZipFilename, executeWithTrackingAsync, getDefaultLogger, Logger, MigrationData } from '../core/index.js';
+import type { Logger, MigrationData } from '../core/index.js';
+import { defaultZipFilename, executeWithTrackingAsync, getDefaultLogger } from '../core/index.js';
 import { fileManager } from '../file/index.js';
 import { libMetadata } from '../metadata.js';
 import { zipManager } from '../zip/index.js';

package/lib/toolkit/import.ts

@@ -1,7 +1,8 @@
 import chalk from 'chalk';
 import { writeFile } from 'fs/promises';
 import { executeWithTrackingAsync } from '../core/index.js';
-import { ImportConfig, ImportResult, importManager as _importManager } from '../import/index.js';
+import type { ImportConfig, ImportResult } from '../import/index.js';
+import { importManager as _importManager } from '../import/index.js';
 import { libMetadata } from '../metadata.js';
 
 export async function importAsync(config: ImportConfig): Promise<ImportResult> {

package/lib/toolkit/index.ts

@@ -1,5 +1,5 @@
+export * from './elements-builder.js';
 export * from './export.js';
+export * from './file.js';
 export * from './import.js';
-export * from './elements-builder.js';
 export * from './migrate.js';
-export * from './file.js';

package/lib/toolkit/migrate.ts

@@ -1,14 +1,8 @@
-import { IRetryStrategyOptions } from '@kontent-ai/core-sdk';
-import {
-    ExternalIdGenerator,
-    Logger,
-    ManagementClientConfig,
-    MigrationData,
-    executeWithTrackingAsync,
-    getDefaultLogger
-} from '../core/index.js';
-import { SourceExportItem } from '../export/index.js';
-import { ImportResult } from '../import/index.js';
+import type { IRetryStrategyOptions } from '@kontent-ai/core-sdk';
+import type { ExternalIdGenerator, Logger, ManagementClientConfig, MigrationData } from '../core/index.js';
+import { executeWithTrackingAsync, getDefaultLogger } from '../core/index.js';
+import type { SourceExportItem } from '../export/index.js';
+import type { ImportResult } from '../import/index.js';
 import { libMetadata } from '../metadata.js';
 import { exportAsync } from './export.js';
 import { importAsync } from './import.js';

package/lib/translation/extraction/items-extraction.processor.ts

@@ -1,14 +1,14 @@
-import { ElementModels } from '@kontent-ai/management-sdk';
-import { GetFlattenedElementByIds } from 'lib/export/export.models.js';
+import type { ElementModels } from '@kontent-ai/management-sdk';
 import { match, P } from 'ts-pattern';
-import {
+import type {
     MigrationElementModels,
     MigrationElements,
-    parseAsMigrationReferencesArray,
     ReferencedDataInLanguageVariants,
     ReferencedDataInMigrationItems
 } from '../../core/index.js';
-import { GetFlattenedElementByCodenames } from '../../import/index.js';
+import { isArray, isString, parseAsMigrationReferencesArray } from '../../core/index.js';
+import type { GetFlattenedElementByIds } from '../../export/export.models.js';
+import type { GetFlattenedElementByCodenames } from '../../import/index.js';
 import { richTextProcessor } from '../index.js';
 
 export interface ExtractItemById {
@@ -43,24 +43,24 @@ export function itemsExtractionProcessor() {
 
                     match(typeElement.type)
                         .with('rich_text', () => {
-                            const rteValue = itemElement.value?.toString();
+                            const rteValue = itemElement.value && isString(itemElement.value) ? itemElement.value : '';
 
                             // extract referenced items
                             richTextProcessor()
-                                .processDataIds(rteValue ?? '')
+                                .processDataIds(rteValue)
                                 .ids.forEach((id) => extractedIds.itemIds.add(id));
 
                             richTextProcessor()
-                                .processLinkItemIds(rteValue ?? '')
+                                .processLinkItemIds(rteValue)
                                 .ids.forEach((id) => extractedIds.itemIds.add(id));
 
                             // extract referenced assets
                             richTextProcessor()
-                                .processAssetIds(rteValue ?? '')
+                                .processAssetIds(rteValue)
                                 .ids.forEach((id) => extractedIds.assetIds.add(id));
 
                             richTextProcessor()
-                                .processLinkAssetIds(rteValue ?? '')
+                                .processLinkAssetIds(rteValue)
                                 .ids.forEach((id) => extractedIds.assetIds.add(id));
 
                             // recursively extract data from components as well because they may reference additional assets & content items
@@ -78,12 +78,12 @@ export function itemsExtractionProcessor() {
                             extractedComponents.assetIds.forEach((id) => extractedIds.assetIds.add(id));
                         })
                         .with(P.union('modular_content', 'subpages'), () => {
-                            if (itemElement.value && Array.isArray(itemElement.value)) {
+                            if (itemElement.value && isArray(itemElement.value)) {
                                 itemElement.value.forEach((value) => (value.id ? extractedIds.itemIds.add(value.id) : {}));
                             }
                         })
                         .with('asset', () => {
-                            if (itemElement.value && Array.isArray(itemElement.value)) {
+                            if (itemElement.value && isArray(itemElement.value)) {
                                 itemElement.value.forEach((value) => (value.id ? extractedIds.assetIds.add(value.id) : {}));
                             }
                         })

package/lib/translation/helpers/rich-text.processor.ts

@@ -1,4 +1,4 @@
-import { AssetStateInTargetEnvironmentByCodename, ItemStateInTargetEnvironmentByCodename } from 'lib/core/index.js';
+import type { AssetStateInTargetEnvironmentByCodename, ItemStateInTargetEnvironmentByCodename } from '../../core/index.js';
 
 export type CodenameReplaceFunc<T> = (codename: string) => T;
 export type IdReplaceFunc = (id: string) => { codename: string };
@@ -32,10 +32,10 @@ const attributes = {
 
 const rteRegexes = {
     elements: {
-        objectTagRegex: new RegExp(`<object(.+?)</object>`, 'g'),
-        imgTagRegex: new RegExp(`<img(.+?)</img>`, 'g'),
-        figureTagRegex: new RegExp(`<figure(.+?)</figure>`, 'g'),
-        linkTagRegex: new RegExp(`<a([\\s\\S]*)</a>`, 'gm')
+        objectTagRegex: new RegExp(`<object(.+?)</object>`, 'gs'),
+        imgTagRegex: new RegExp(`<img(.+?)</img>`, 'gs'),
+        figureTagRegex: new RegExp(`<figure(.+?)</figure>`, 'gs'),
+        linkTagRegex: new RegExp(`<a(.+?)</a>`, 'gs')
     },
     ids: {
         dataItemIdAttrRegex: new RegExp(`${attributes.ids.itemIdAttributeName}=\\"(.+?)\\"`),
@@ -53,14 +53,7 @@ export function richTextProcessor() {
     const processDataIds = (richTextHtml: string, replaceFunc?: IdReplaceFunc): ProcessIdsResult => {
         const itemIds = new Set<string>();
 
-        if (!richTextHtml) {
-            return {
-                html: richTextHtml,
-                ids: itemIds
-            };
-        }
-
-        richTextHtml = richTextHtml.replaceAll(rteRegexes.elements.objectTagRegex, (objectTag) => {
+        const processedRte = richTextHtml.replaceAll(rteRegexes.elements.objectTagRegex, (objectTag) => {
             // skip processing for components
             if (objectTag.includes(attributes.componentIdentifierAttribute)) {
                 return objectTag;
@@ -85,7 +78,7 @@ export function richTextProcessor() {
         });
 
         return {
-            html: richTextHtml,
+            html: processedRte,
             ids: itemIds
         };
     };
@@ -93,14 +86,7 @@ export function richTextProcessor() {
     const processAssetIds = (richTextHtml: string, replaceFunc?: IdReplaceFunc): ProcessIdsResult => {
         const assetIds = new Set<string>();
 
-        if (!richTextHtml) {
-            return {
-                html: richTextHtml,
-                ids: assetIds
-            };
-        }
-
-        richTextHtml = richTextHtml.replaceAll(rteRegexes.elements.figureTagRegex, (figureTag) => {
+        const processedRte = richTextHtml.replaceAll(rteRegexes.elements.figureTagRegex, (figureTag) => {
             const assetIdMatch = figureTag.match(rteRegexes.ids.dataAssetIdAttrRegex);
             if (assetIdMatch && (assetIdMatch?.length ?? 0) >= 2) {
                 const assetId = assetIdMatch[1];
@@ -121,7 +107,7 @@ export function richTextProcessor() {
         });
 
         return {
-            html: richTextHtml,
+            html: processedRte,
             ids: assetIds
         };
     };
@@ -129,14 +115,7 @@ export function richTextProcessor() {
     const processLinkAssetIds = (richTextHtml: string, replaceFunc?: IdReplaceFunc): ProcessIdsResult => {
         const assetIds = new Set<string>();
 
-        if (!richTextHtml) {
-            return {
-                html: richTextHtml,
-                ids: assetIds
-            };
-        }
-
-        richTextHtml = richTextHtml.replaceAll(rteRegexes.elements.linkTagRegex, (linkTag) => {
+        const processedRte = richTextHtml.replaceAll(rteRegexes.elements.linkTagRegex, (linkTag) => {
             const assetIdMatch = linkTag.match(rteRegexes.ids.dataAssetIdAttrRegex);
             if (assetIdMatch && (assetIdMatch?.length ?? 0) >= 2) {
                 const assetId = assetIdMatch[1];
@@ -157,7 +136,7 @@ export function richTextProcessor() {
         });
 
         return {
-            html: richTextHtml,
+            html: processedRte,
             ids: assetIds
         };
     };
@@ -165,14 +144,7 @@ export function richTextProcessor() {
     const processLinkItemIds = (richTextHtml: string, replaceFunc?: IdReplaceFunc): ProcessIdsResult => {
         const linkItemIds = new Set<string>();
 
-        if (!richTextHtml) {
-            return {
-                html: richTextHtml,
-                ids: linkItemIds
-            };
-        }
-
-        richTextHtml = richTextHtml.replaceAll(rteRegexes.elements.linkTagRegex, (linkTag) => {
+        const processedRte = richTextHtml.replaceAll(rteRegexes.elements.linkTagRegex, (linkTag) => {
             const itemIdMatch = linkTag.match(rteRegexes.ids.dataItemIdAttrRegex);
             if (itemIdMatch && (itemIdMatch?.length ?? 0) >= 2) {
                 const itemId = itemIdMatch[1];
@@ -192,7 +164,7 @@ export function richTextProcessor() {
         });
 
         return {
-            html: richTextHtml,
+            html: processedRte,
             ids: linkItemIds
         };
     };
@@ -203,14 +175,7 @@ export function richTextProcessor() {
     ): ProcessCodenamesResult => {
         const itemCodenames = new Set<string>();
 
-        if (!richTextHtml) {
-            return {
-                codenames: itemCodenames,
-                html: richTextHtml
-            };
-        }
-
-        richTextHtml = richTextHtml.replaceAll(rteRegexes.elements.objectTagRegex, (objectTag) => {
+        const processedRte = richTextHtml.replaceAll(rteRegexes.elements.objectTagRegex, (objectTag) => {
             const codenameMatch = objectTag.match(rteRegexes.codenames.rteItemCodenameRegex);
             if (codenameMatch && (codenameMatch?.length ?? 0) >= 2) {
                 const codename = codenameMatch[1];
@@ -237,7 +202,7 @@ export function richTextProcessor() {
 
         return {
             codenames: itemCodenames,
-            html: richTextHtml
+            html: processedRte
         };
     };
 
@@ -247,14 +212,7 @@ export function richTextProcessor() {
     ): ProcessCodenamesResult => {
         const itemCodenames = new Set<string>();
 
-        if (!richTextHtml) {
-            return {
-                codenames: itemCodenames,
-                html: richTextHtml
-            };
-        }
-
-        richTextHtml = richTextHtml.replaceAll(rteRegexes.elements.linkTagRegex, (linkTag) => {
+        const processedRte = richTextHtml.replaceAll(rteRegexes.elements.linkTagRegex, (linkTag) => {
             const codenameMatch = linkTag.match(rteRegexes.codenames.rteLinkItemCodenameRegex);
             if (codenameMatch && (codenameMatch?.length ?? 0) >= 2) {
                 const codename = codenameMatch[1];
@@ -279,7 +237,7 @@ export function richTextProcessor() {
 
         return {
             codenames: itemCodenames,
-            html: richTextHtml
+            html: processedRte
         };
     };
 
@@ -289,14 +247,7 @@ export function richTextProcessor() {
     ): ProcessCodenamesResult => {
         const assetCodenames = new Set<string>();
 
-        if (!richTextHtml) {
-            return {
-                codenames: assetCodenames,
-                html: richTextHtml
-            };
-        }
-
-        richTextHtml = richTextHtml.replaceAll(rteRegexes.elements.figureTagRegex, (figureTag) => {
+        const processedRte = richTextHtml.replaceAll(rteRegexes.elements.figureTagRegex, (figureTag) => {
             const codenameMatch = figureTag.match(rteRegexes.codenames.rteAssetCodenameRegex);
             if (codenameMatch && (codenameMatch?.length ?? 0) >= 2) {
                 const codename = codenameMatch[1];
@@ -321,7 +272,7 @@ export function richTextProcessor() {
 
         return {
             codenames: assetCodenames,
-            html: richTextHtml
+            html: processedRte
         };
     };
 
@@ -331,14 +282,7 @@ export function richTextProcessor() {
     ): ProcessCodenamesResult => {
         const assetCodenames = new Set<string>();
 
-        if (!richTextHtml) {
-            return {
-                codenames: assetCodenames,
-                html: richTextHtml
-            };
-        }
-
-        richTextHtml = richTextHtml.replaceAll(rteRegexes.elements.linkTagRegex, (linkTag) => {
+        const processedRte = richTextHtml.replaceAll(rteRegexes.elements.linkTagRegex, (linkTag) => {
             const codenameMatch = linkTag.match(rteRegexes.codenames.rteAssetCodenameRegex);
             if (codenameMatch && (codenameMatch?.length ?? 0) >= 2) {
                 const codename = codenameMatch[1];
@@ -363,7 +307,7 @@ export function richTextProcessor() {
 
         return {
             codenames: assetCodenames,
-            html: richTextHtml
+            html: processedRte
         };
     };
 

package/lib/translation/index.ts

@@ -1,4 +1,4 @@
-export * from './helpers/rich-text.processor.js';
 export * from './extraction/items-extraction.processor.js';
+export * from './helpers/rich-text.processor.js';
 export * from './transforms/export-transforms.js';
 export * from './transforms/import-transforms.js';