fix(core): add additional component metadata to component ID generation (angular#50203)

alan-agius4 · alxhub · commit 52c74d3b4a8d · 2023-05-09T14:46:42.000-07:00
This commit add `exportAs`, `signals`, `inputs` and `outputs` into account when generating a component ID. See: angular#50158 (comment) PR Close angular#50203
diff --git a/packages/core/src/render3/definition.ts b/packages/core/src/render3/definition.ts
@@ -679,7 +679,7 @@ function getComponentId(componentDef: ComponentDef<unknown>): string {
   // https://github.com/angular/components/blob/d9f82c8f95309e77a6d82fd574c65871e91354c2/src/material/core/option/option.ts#L248
   // https://github.com/angular/components/blob/285f46dc2b4c5b127d356cb7c4714b221f03ce50/src/material/legacy-core/option/option.ts#L32
 
-  const hashSelectors = [
+  const hashSelectors = JSON.stringify([
     componentDef.selectors,
     componentDef.ngContentSelectors,
     componentDef.hostVars,
@@ -689,12 +689,16 @@ function getComponentId(componentDef: ComponentDef<unknown>): string {
     componentDef.decls,
     componentDef.encapsulation,
     componentDef.standalone,
+    componentDef.signals,
+    componentDef.exportAs,
+    componentDef.inputs,
+    componentDef.outputs,
     // We cannot use 'componentDef.type.name' as the name of the symbol will change and will not
     // match in the server and browser bundles.
     Object.getOwnPropertyNames(componentDef.type.prototype),
     !!componentDef.contentQueries,
     !!componentDef.viewQuery,
-  ].join('|');
+  ]);
 
   for (const char of hashSelectors) {
     hash = Math.imul(31, hash) + char.charCodeAt(0) << 0;
