newo 3.6.1 → 3.6.2

package/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [3.6.2] - 2026-04-23
+
+### Fixed
+
+- **`attributes.yaml` YAML escaping**: The attribute serializer previously ran `.replace(/\\"/g, '"')` as a "prettify" post-processing step, which stripped legitimate YAML escape characters from double-quoted scalars. Values containing double quotes (e.g. `["+37410333310"]`) were written as invalid YAML like `value: "["+37410333310"]"` — rejected by `yaml.load` with `bad indentation of a mapping entry`. This made `newo push --format newo_v2` unable to parse customer/project attribute files at all, and long compiled values (e.g. AMI) broke on the first embedded quote. The serializer now uses the existing `patchYamlToPyyaml` post-processor (already used on the V2 path) to convert JSON-like double-quoted values to single-quoted form (`value: '["+37410333310"]'`) and handle long-line wrapping in pyyaml style. Fix applied in both `src/sync/attributes.ts` and `src/domain/strategies/sync/AttributeSyncStrategy.ts`. Added 11 round-trip regression tests.
+
 ## [3.6.1] - 2026-04-14
 
 ### Documentation
@@ -994,7 +1000,8 @@ Another Item: $Price [Modifiers: modifier3]
 - GitHub Actions CI/CD integration
 - Robust authentication with token refresh
 
-[Unreleased]: https://github.com/sabbah13/newo-cli/compare/v3.3.0...HEAD
+[Unreleased]: https://github.com/sabbah13/newo-cli/compare/v3.6.2...HEAD
+[3.6.2]: https://github.com/sabbah13/newo-cli/compare/v3.6.1...v3.6.2
 [3.3.0]: https://github.com/sabbah13/newo-cli/compare/v3.2.0...v3.3.0
 [3.2.0]: https://github.com/sabbah13/newo-cli/compare/v3.1.0...v3.2.0
 [3.1.0]: https://github.com/sabbah13/newo-cli/compare/v3.0.0...v3.1.0

package/dist/domain/strategies/sync/AttributeSyncStrategy.js

@@ -14,6 +14,7 @@ import yaml from 'js-yaml';
 import path from 'path';
 import { getCustomerAttributes, getProjectAttributes, updateCustomerAttribute, updateProjectAttribute, listProjects } from '../../../api.js';
 import { writeFileSafe, customerAttributesPath, customerAttributesMapPath } from '../../../fsutil.js';
+import { patchYamlToPyyaml } from '../../../format/yaml-patch.js';
 import { sha256, saveHashes, loadHashes } from '../../../hash.js';
 /**
  * AttributeSyncStrategy - Handles attribute synchronization
@@ -171,18 +172,20 @@ export class AttributeSyncStrategy {
             ...attr,
             value_type: `__ENUM_PLACEHOLDER_${attr.value_type}__`
         }));
+        // Emit YAML without folding/wrapping; patchYamlToPyyaml handles long-line
+        // wrapping and converts JSON-like double-quoted values to single-quoted
+        // (so strings containing `"` stay valid YAML on reload).
         let yamlContent = yaml.dump({ attributes: attributesWithPlaceholders }, {
             indent: 2,
             quotingType: '"',
             forceQuotes: false,
-            lineWidth: 80,
+            lineWidth: -1,
             noRefs: true,
             sortKeys: false,
-            flowLevel: -1
+            flowLevel: -1,
         });
-        // Replace placeholders with enum syntax
         yamlContent = yamlContent.replace(/__ENUM_PLACEHOLDER_(\w+)__/g, '!enum "AttributeValueTypes.$1"');
-        yamlContent = yamlContent.replace(/\\"/g, '"');
+        yamlContent = patchYamlToPyyaml(yamlContent);
         return yamlContent;
     }
     /**

package/dist/sync/attributes.js

@@ -6,6 +6,7 @@ import { writeFileSafe, customerAttributesPath, customerAttributesMapPath, custo
 import path from 'path';
 import fs from 'fs-extra';
 import yaml from 'js-yaml';
+import { patchYamlToPyyaml } from '../format/yaml-patch.js';
 /**
  * Save customer attributes to YAML format and return content for hashing
  */
@@ -55,23 +56,22 @@ export async function saveCustomerAttributes(client, customer, verbose = false)
         const attributesYaml = {
             attributes: cleanAttributes
         };
-        // Configure YAML output to match reference format exactly
+        // Emit YAML without folding/wrapping; patchYamlToPyyaml handles long-line
+        // wrapping and converts JSON-like double-quoted values to single-quoted
+        // (so strings containing `"` stay valid YAML on reload).
         let yamlContent = yaml.dump(attributesYaml, {
             indent: 2,
             quotingType: '"',
             forceQuotes: false,
-            lineWidth: 80, // Wrap long lines to match reference format
+            lineWidth: -1,
             noRefs: true,
             sortKeys: false,
-            flowLevel: -1, // Never use flow syntax
-            styles: {
-                '!!str': 'folded' // Use folded style for better line wrapping of long strings
-            }
+            flowLevel: -1,
         });
-        // Post-process to fix enum format and improve JSON string formatting
+        // Post-process to fix enum format
         yamlContent = yamlContent.replace(/__ENUM_PLACEHOLDER_(\w+)__/g, '!enum "AttributeValueTypes.$1"');
-        // Fix JSON string formatting to match reference (remove escape characters)
-        yamlContent = yamlContent.replace(/\\"/g, '"');
+        // Convert JSON-like double-quoted values to single-quoted and wrap long lines
+        yamlContent = patchYamlToPyyaml(yamlContent);
         // Save all files: attributes.yaml, ID mapping, and backup for diff tracking
         await writeFileSafe(customerAttributesPath(customer.idn), yamlContent);
         await writeFileSafe(customerAttributesMapPath(customer.idn), JSON.stringify(idMapping, null, 2));
@@ -139,19 +139,19 @@ export async function saveProjectAttributes(client, customer, projectId, project
         const attributesYaml = {
             attributes: cleanAttributes
         };
-        // Configure YAML output
+        // Emit YAML without folding/wrapping; patchYamlToPyyaml handles long-line
+        // wrapping and converts JSON-like double-quoted values to single-quoted.
         let yamlContent = yaml.dump(attributesYaml, {
             indent: 2,
             quotingType: '"',
             forceQuotes: false,
-            lineWidth: 80,
+            lineWidth: -1,
             noRefs: true,
             sortKeys: false,
-            flowLevel: -1
+            flowLevel: -1,
         });
-        // Post-process to fix enum format
         yamlContent = yamlContent.replace(/__ENUM_PLACEHOLDER_(\w+)__/g, '!enum "AttributeValueTypes.$1"');
-        yamlContent = yamlContent.replace(/\\"/g, '"');
+        yamlContent = patchYamlToPyyaml(yamlContent);
         // Save to project directory
         const customerDir = path.join(process.cwd(), 'newo_customers', customer.idn);
         const projectDir = path.join(customerDir, 'projects', projectIdn);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "newo",
-  "version": "3.6.1",
+  "version": "3.6.2",
   "description": "NEWO CLI: Professional command-line tool with modular architecture for NEWO AI Agent development. Features account migration, integration management, webhook automation, AKB knowledge base, project attributes, sandbox testing, IDN-based file management, real-time progress tracking, intelligent sync operations, and comprehensive multi-customer support.",
   "type": "module",
   "bin": {

package/src/domain/strategies/sync/AttributeSyncStrategy.ts

@@ -38,6 +38,7 @@ import {
   customerAttributesPath,
   customerAttributesMapPath
 } from '../../../fsutil.js';
+import { patchYamlToPyyaml } from '../../../format/yaml-patch.js';
 import { sha256, saveHashes, loadHashes } from '../../../hash.js';
 
 /**
@@ -251,19 +252,21 @@ export class AttributeSyncStrategy implements ISyncStrategy<CustomerAttributesRe
       value_type: `__ENUM_PLACEHOLDER_${attr.value_type}__`
     }));
 
+    // Emit YAML without folding/wrapping; patchYamlToPyyaml handles long-line
+    // wrapping and converts JSON-like double-quoted values to single-quoted
+    // (so strings containing `"` stay valid YAML on reload).
     let yamlContent = yaml.dump({ attributes: attributesWithPlaceholders }, {
       indent: 2,
       quotingType: '"',
       forceQuotes: false,
-      lineWidth: 80,
+      lineWidth: -1,
       noRefs: true,
       sortKeys: false,
-      flowLevel: -1
+      flowLevel: -1,
     });
 
-    // Replace placeholders with enum syntax
     yamlContent = yamlContent.replace(/__ENUM_PLACEHOLDER_(\w+)__/g, '!enum "AttributeValueTypes.$1"');
-    yamlContent = yamlContent.replace(/\\"/g, '"');
+    yamlContent = patchYamlToPyyaml(yamlContent);
 
     return yamlContent;
   }

package/src/sync/attributes.ts

@@ -11,6 +11,7 @@ import {
 import path from 'path';
 import fs from 'fs-extra';
 import yaml from 'js-yaml';
+import { patchYamlToPyyaml } from '../format/yaml-patch.js';
 import type { AxiosInstance } from 'axios';
 import type { CustomerConfig } from '../types.js';
 
@@ -72,25 +73,24 @@ export async function saveCustomerAttributes(
       attributes: cleanAttributes
     };
 
-    // Configure YAML output to match reference format exactly
+    // Emit YAML without folding/wrapping; patchYamlToPyyaml handles long-line
+    // wrapping and converts JSON-like double-quoted values to single-quoted
+    // (so strings containing `"` stay valid YAML on reload).
     let yamlContent = yaml.dump(attributesYaml, {
       indent: 2,
       quotingType: '"',
       forceQuotes: false,
-      lineWidth: 80, // Wrap long lines to match reference format
+      lineWidth: -1,
       noRefs: true,
       sortKeys: false,
-      flowLevel: -1, // Never use flow syntax
-      styles: {
-        '!!str': 'folded' // Use folded style for better line wrapping of long strings
-      }
+      flowLevel: -1,
     });
 
-    // Post-process to fix enum format and improve JSON string formatting
+    // Post-process to fix enum format
     yamlContent = yamlContent.replace(/__ENUM_PLACEHOLDER_(\w+)__/g, '!enum "AttributeValueTypes.$1"');
 
-    // Fix JSON string formatting to match reference (remove escape characters)
-    yamlContent = yamlContent.replace(/\\"/g, '"');
+    // Convert JSON-like double-quoted values to single-quoted and wrap long lines
+    yamlContent = patchYamlToPyyaml(yamlContent);
 
     // Save all files: attributes.yaml, ID mapping, and backup for diff tracking
     await writeFileSafe(customerAttributesPath(customer.idn), yamlContent);
@@ -172,20 +172,20 @@ export async function saveProjectAttributes(
       attributes: cleanAttributes
     };
 
-    // Configure YAML output
+    // Emit YAML without folding/wrapping; patchYamlToPyyaml handles long-line
+    // wrapping and converts JSON-like double-quoted values to single-quoted.
     let yamlContent = yaml.dump(attributesYaml, {
       indent: 2,
       quotingType: '"',
       forceQuotes: false,
-      lineWidth: 80,
+      lineWidth: -1,
       noRefs: true,
       sortKeys: false,
-      flowLevel: -1
+      flowLevel: -1,
     });
 
-    // Post-process to fix enum format
     yamlContent = yamlContent.replace(/__ENUM_PLACEHOLDER_(\w+)__/g, '!enum "AttributeValueTypes.$1"');
-    yamlContent = yamlContent.replace(/\\"/g, '"');
+    yamlContent = patchYamlToPyyaml(yamlContent);
 
     // Save to project directory
     const customerDir = path.join(process.cwd(), 'newo_customers', customer.idn);