Add analysis info to header for yaml

.gitignore

@@ -357,3 +357,6 @@ MigrationBackup/
 
 # Ionide (cross platform F# VS Code tools) working folder
 .ionide/
+
+# Ignore .idea IDE files
+.idea/

src/PAModel/CanvasDocument.cs

@@ -11,28 +11,29 @@
 using Microsoft.PowerPlatform.Formulas.Tools.Utility;
 using System;
 using System.Collections.Generic;
+using System.Diagnostics;
 using System.IO;
 using System.Linq;
 
 namespace Microsoft.PowerPlatform.Formulas.Tools
 {
     /// <summary>
-    /// Represents a PowerApps document.  This can be save/loaded from a MsApp or Source representation. 
-    /// This is a full in-memory representation of the msapp file. 
+    /// Represents a PowerApps document.  This can be save/loaded from a MsApp or Source representation.
+    /// This is a full in-memory representation of the msapp file.
     /// </summary>
     public class CanvasDocument
     {
         /// <summary>
-        /// Current source format version. 
+        /// Current source format version.
         /// </summary>
         public static Version CurrentSourceVersion => SourceSerializer.CurrentSourceVersion;
 
         // Rules for CanvasDocument
-        // - Save/Load must faithfully roundtrip an msapp exactly. 
-        // - this is an in-memory representation - so it must parse/shard everything on load. 
-        // - Save should not mutate any state. 
+        // - Save/Load must faithfully roundtrip an msapp exactly.
+        // - this is an in-memory representation - so it must parse/shard everything on load.
+        // - Save should not mutate any state.
 
-        // Track all unknown "files". Ensures round-tripping isn't lossy.         
+        // Track all unknown "files". Ensures round-tripping isn't lossy.
         // Only contains files of FileKind.Unknown
         internal Dictionary<FilePath, FileEntry> _unknownFiles = new Dictionary<FilePath, FileEntry>();
 
@@ -43,7 +44,7 @@ public class CanvasDocument
         internal EditorStateStore _editorStateStore;
         internal TemplateStore _templateStore;
 
-        // Various data sources        
+        // Various data sources
         // This is references\dataSources.json
         // Also includes entries for DataSources made from a DataComponent
         // Key is parent entity name (datasource name for non cds data sources)
@@ -77,15 +78,15 @@ public class CanvasDocument
         internal IDictionary<string, LocalDatabaseReferenceJson> _dataSourceReferences;
 
         // Extracted from _properties.LibraryDependencies
-        // Must preserve server ordering. 
+        // Must preserve server ordering.
         internal ComponentDependencyInfo[] _libraryReferences;
 
         internal FileEntry _logoFile;
 
         // Save for roundtripping.
         internal Entropy _entropy = new Entropy();
 
-        // Checksum from existing msapp. 
+        // Checksum from existing msapp.
         internal ChecksumJson _checksum;
 
         // Track all asset files, key is file name
@@ -103,7 +104,7 @@ public class CanvasDocument
         #region Save/Load
 
         /// <summary>
-        /// Load an .msapp file for a Canvas Document. 
+        /// Load an .msapp file for a Canvas Document.
         /// </summary>
         /// <param name="fullPathToMsApp">path to an .msapp file</param>
         /// <returns>A tuple of the document and errors and warnings. If there are errors, the document is null.  </returns>
@@ -152,6 +153,7 @@ public ErrorContainer SaveToMsApp(string fullPathToMsApp)
             Utilities.EnsurePathRooted(fullPathToMsApp);
 
             var errors = new ErrorContainer();
+            Debugger.Launch();
             Wrapper(() => MsAppSerializer.SaveAsMsApp(this, fullPathToMsApp, errors), errors);
             return errors;
         }
@@ -219,7 +221,7 @@ public static (CanvasDocument, ErrorContainer) MakeFromSources(string appName, s
 
         #endregion
 
-        // Wrapper to ensure consistent invariants between loading a document, exception handling, and returning errors. 
+        // Wrapper to ensure consistent invariants between loading a document, exception handling, and returning errors.
         private static CanvasDocument Wrapper(Func<CanvasDocument> worker, ErrorContainer errors)
         {
             CanvasDocument document = null;
@@ -322,7 +324,7 @@ internal CanvasDocument(CanvasDocument other)
             _localAssetInfoJson = other._localAssetInfoJson.JsonClone();
         }
 
-        // iOrder is used to preserve ordering value for round-tripping. 
+        // iOrder is used to preserve ordering value for round-tripping.
         internal void AddDataSourceForLoad(DataSourceEntry ds, int? order = null)
         {
             // Key is parent entity name
@@ -379,7 +381,7 @@ internal void ApplyAfterMsAppLoadTransforms(ErrorContainer errors)
             var componentInstanceTransform = new ComponentInstanceTransform(errors);
             var componentDefTransform = new ComponentDefinitionTransform(errors, _templateStore, componentInstanceTransform);
 
-            // Transform component definitions and populate template set of component instances that need updates 
+            // Transform component definitions and populate template set of component instances that need updates
             foreach (var ctrl in _components)
             {
                 AddComponentDefaults(ctrl.Value, templateDefaults);
@@ -436,7 +438,7 @@ internal void ApplyBeforeMsAppWriteTransforms(ErrorContainer errors)
             var componentInstanceTransform = new ComponentInstanceTransform(errors);
             var componentDefTransform = new ComponentDefinitionTransform(errors, _templateStore, componentInstanceTransform);
 
-            // Transform component definitions and populate template set of component instances that need updates 
+            // Transform component definitions and populate template set of component instances that need updates
             foreach (var ctrl in _components)
             {
                 componentDefTransform.BeforeWrite(ctrl.Value);
@@ -469,10 +471,10 @@ private void AddComponentDefaults(BlockNode topParent, Dictionary<string, Contro
         }
 
 
-        // Called after loading. This will check internal fields and fill in consistency data. 
+        // Called after loading. This will check internal fields and fill in consistency data.
         internal void OnLoadComplete(ErrorContainer errors)
         {
-            // Do integrity checks. 
+            // Do integrity checks.
             if (_header == null)
             {
                 errors.FormatNotSupported("Missing header file");
@@ -505,7 +507,7 @@ internal void OnLoadComplete(ErrorContainer errors)
             }
         }
 
-        // Get ComponentIds for components we've imported. 
+        // Get ComponentIds for components we've imported.
         internal HashSet<string> GetImportedComponents()
         {
             var set = new HashSet<string>();
@@ -684,10 +686,10 @@ private void RestoreAssetFilePaths(ErrorContainer errors)
             }
         }
 
-        // Helper for traversing and ensuring unique control names. 
+        // Helper for traversing and ensuring unique control names.
         internal class UniqueControlNameVistor
         {
-            // Control names are case sensitive. 
+            // Control names are case sensitive.
             private readonly Dictionary<string, SourceLocation?> _names = new Dictionary<string, SourceLocation?>(StringComparer.Ordinal);
             private readonly ErrorContainer _errors;
 
@@ -698,7 +700,7 @@ public UniqueControlNameVistor(ErrorContainer errors)
 
             public void Visit(BlockNode node)
             {
-                // Ignore test templates here. 
+                // Ignore test templates here.
                 // Test templates have control-like syntax, but allowed to repeat names:
                 //    Step4 As TestStep:
                 if (AppTestTransform.IsTestSuite(node.Name.Kind.TypeName))