ga4-export-fixer 0.4.3-dev.0 → 0.4.3-dev.1

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.0",
+  "version": "0.4.3-dev.1",
   "description": "",
   "main": "index.js",
   "files": [

package/tables/ga4EventsEnhanced.js

@@ -327,8 +327,6 @@ const createEnhancedEventsTable = (dataformPublish, config) => {
 
     const mergedConfig = utils.mergeSQLConfigurations(defaultConfig, sqlConfig);
 
-    const tableDescription = documentation.getTableDescription(mergedConfig);
-
     // Static defaults from defaultConfig.js (via mergedConfig, without user overrides)
     const staticDefaults = mergedConfig.dataformTableConfig || {};
 
@@ -345,19 +343,26 @@ 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),
-    };
-
-    // Merge: static defaults → dynamic fields → user overrides
+    // Merge: static defaults → dynamic fields (except description) → user overrides
     const dataformTableConfig = utils.mergeDataformTableConfigurations(
-        { ...staticDefaults, ...dynamicFields },
+        {
+            ...staticDefaults,
+            name: `${constants.DEFAULT_EVENTS_TABLE_NAME}_${dataset.replace('analytics_', '')}`,
+            schema: dataset,
+            columns: documentation.getColumnDescriptions(mergedConfig),
+        },
         userDataformTableConfig
     );
 
+    // 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));