ga4-export-fixer 0.4.3-dev.3 → 0.4.3-dev.5

package/documentation.js

@@ -282,13 +282,15 @@ const getTableDescription = (config) => {
     sections.push(`${constants.TABLE_DESCRIPTION_SUFFIX}\n${constants.TABLE_DESCRIPTION_DOCUMENTATION_LINK}`);
 
     // 8. Config JSON dump
-    const configJson = JSON.stringify(
-        Object.fromEntries(
-            Object.entries(config).filter(([key]) => !key.startsWith('default') && key !== 'dataformTableConfig')
-        ),
-        null,
-        2
+    const configForDump = Object.fromEntries(
+        Object.entries(config).filter(([key]) => !key.startsWith('default'))
     );
+    // Strip description and columns from dataformTableConfig to avoid circular reference and bloat
+    if (configForDump.dataformTableConfig) {
+        const { description, columns, ...rest } = configForDump.dataformTableConfig;
+        configForDump.dataformTableConfig = rest;
+    }
+    const configJson = JSON.stringify(configForDump, null, 2);
     sections.push(`The last full table refresh was done using this configuration:\n${configJson}`);
 
     return sections.join('\n\n');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ga4-export-fixer",
-  "version": "0.4.3-dev.3",
+  "version": "0.4.3-dev.5",
   "description": "",
   "main": "index.js",
   "files": [

package/tables/ga4EventsEnhanced.js

@@ -321,8 +321,6 @@ ${excludedEventsSQL}`,
 const createEnhancedEventsTable = (dataformPublish, config) => {
     const mergedConfig = utils.mergeSQLConfigurations(defaultConfig, config);
 
-    const tableDescription = documentation.getTableDescription(mergedConfig);
-
     // Compute dynamic fields from merged SQL config
     const getDatasetName = (sourceTable) => {
         if (utils.isDataformTableReferenceObject(sourceTable)) {
@@ -336,21 +334,28 @@ const createEnhancedEventsTable = (dataformPublish, config) => {
 
     const dataset = getDatasetName(mergedConfig.sourceTable);
 
-    const dynamicFields = {
-        name: `${constants.DEFAULT_EVENTS_TABLE_NAME}_${dataset.replace('analytics_', '')}`,
-        schema: dataset,
-        description: tableDescription,
-        columns: documentation.getColumnDescriptions(mergedConfig),
-    };
-
     // Build dataformTableConfig: static defaults (from defaultConfig.js) → dynamic fields → user overrides.
     // Deep-clone defaults to prevent Dataform's publish() from mutating nested objects (e.g. bigquery)
     // across multiple createTable calls in the same process.
     const dataformTableConfig = utils.mergeDataformTableConfigurations(
-        { ...JSON.parse(JSON.stringify(defaultConfig.dataformTableConfig || {})), ...dynamicFields },
+        {
+            ...JSON.parse(JSON.stringify(defaultConfig.dataformTableConfig || {})),
+            name: `${constants.DEFAULT_EVENTS_TABLE_NAME}_${dataset.replace('analytics_', '')}`,
+            schema: dataset,
+            columns: documentation.getColumnDescriptions(mergedConfig),
+        },
         config.dataformTableConfig
     );
 
+    // Pass dataformTableConfig to getTableDescription via a new object to avoid mutating mergedConfig
+    // (Dataform's sandboxed runtime may freeze objects returned by mergeSQLConfigurations)
+    const tableDescription = documentation.getTableDescription({ ...mergedConfig, dataformTableConfig });
+
+    // Set description (user override from the merge wins if provided)
+    if (!dataformTableConfig.description) {
+        dataformTableConfig.description = tableDescription;
+    }
+
     // create the table using Dataform publish()
     return dataformPublish(dataformTableConfig.name, dataformTableConfig).preOps(ctx => {
         return preOperations.setPreOperations(utils.setDataformContext(ctx, mergedConfig));