feat: do not serialize the idMap of the normalized ast

flowr-analysis · Sep 1, 2023 · f0642cf · f0642cf
1 parent e537d58
commit f0642cf
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/src/cli/repl/server/connection.ts b/src/cli/repl/server/connection.ts
@@ -101,9 +101,15 @@ export class FlowRServerConnection {
 
 		void slicer.allRemainingSteps(false).then(results => {
 			sendMessage(this.socket, {
-				type: 'response-file-analysis',
-				id:   message.id,
-				results
+				type:    'response-file-analysis',
+				id:      message.id,
+				results: {
+					...results,
+					normalize: {
+						...results.normalize,
+						idMap: undefined
+					}
+				}
 			})
 		})
 	}

diff --git a/src/cli/repl/server/messages/analysis.ts b/src/cli/repl/server/messages/analysis.ts
@@ -41,6 +41,9 @@ export const requestAnalysisMessage: RequestMessageDefinition<FileAnalysisReques
  * Answer for a successful {@link FileAnalysisRequestMessage}.
  * It contains the results of the analysis in JSON format.
  *
+ * The `idMap` of the normalization step (see {@link NormalizedAst}) is not serialized as it would essentially
+ * repeat the complete normalized AST.
+ *
  * @note the serialization of maps and sets is controlled by the {@link jsonReplacer} as part of {@link sendMessage}.
  */
 export interface FileAnalysisResponseMessage extends FlowrBaseMessage {