ga4-export-fixer 0.4.3-dev.2 → 0.4.3-dev.4

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.2",
+  "version": "0.4.3-dev.4",
   "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,22 +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
-    // Uses defaultConfig.dataformTableConfig directly for defaults and config.dataformTableConfig
-    // directly for user overrides, bypassing mergeSQLConfigurations for this merge to ensure
-    // defaults are always applied and mergeDataformTableConfigurations handles tags correctly.
+    // 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(
-        { ...(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
     );
 
+    // Include the final dataformTableConfig in mergedConfig for the description's config dump
+    mergedConfig.dataformTableConfig = dataformTableConfig;
+    const tableDescription = documentation.getTableDescription(mergedConfig);
+
+    // 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));