[rush] Fix an issue where usage of custom fields in dependenciesMeta caused rush install to think that shrinkwrap file is outdated

common/changes/@microsoft/rush/fix-dependencies-meta-custom-fields_2024-11-06-16-08.json

@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "@microsoft/rush",
+      "comment": "Fix an issue where usage of custom fields in dependenciesMeta caused rush install to think that shrinkwrap file is outdated",
+      "type": "none"
+    }
+  ],
+  "packageName": "@microsoft/rush"
+}

common/reviews/api/rush-lib.api.md

@@ -1005,11 +1005,13 @@ export class PackageJsonDependency {
 
 // @public (undocumented)
 export class PackageJsonDependencyMeta {
-    constructor(name: string, injected: boolean, onChange: () => void);
+    constructor(name: string, sourceData: object, onChange: () => void);
     // (undocumented)
     get injected(): boolean;
     // (undocumented)
     readonly name: string;
+    // (undocumented)
+    get sourceData(): object;
 }
 
 // @public (undocumented)

libraries/rush-lib/src/api/PackageJsonEditor.ts

@@ -50,19 +50,23 @@ export class PackageJsonDependency {
  * @public
  */
 export class PackageJsonDependencyMeta {
-  private _injected: boolean;
+  private _sourceData: object;
   private _onChange: () => void;
 
   public readonly name: string;
 
-  public constructor(name: string, injected: boolean, onChange: () => void) {
+  public constructor(name: string, sourceData: object, onChange: () => void) {
     this.name = name;
-    this._injected = injected;
+    this._sourceData = sourceData;
     this._onChange = onChange;
   }
 
+  public get sourceData(): object {
+    return this._sourceData;
+  }
+
   public get injected(): boolean {
-    return this._injected;
+    return (this._sourceData as { injected?: boolean }).injected ?? false;
   }
 }
 
@@ -107,7 +111,7 @@ export class PackageJsonEditor {
     const devDependencies: { [key: string]: string } = data.devDependencies || {};
     const resolutions: { [key: string]: string } = data.resolutions || {};
 
-    const dependenciesMeta: { [key: string]: { [key: string]: boolean } } = data.dependenciesMeta || {};
+    const dependenciesMeta: { [key: string]: object } = data.dependenciesMeta || {};
 
     const _onChange: () => void = this._onChange.bind(this);
 
@@ -183,7 +187,7 @@ export class PackageJsonEditor {
       Object.keys(dependenciesMeta || {}).forEach((packageName: string) => {
         this._dependenciesMeta.set(
           packageName,
-          new PackageJsonDependencyMeta(packageName, dependenciesMeta[packageName].injected, _onChange)
+          new PackageJsonDependencyMeta(packageName, dependenciesMeta[packageName], _onChange)
         );
       });
 

libraries/rush-lib/src/logic/installManager/WorkspaceInstallManager.ts

@@ -325,9 +325,7 @@ export class WorkspaceInstallManager extends BaseInstallManager {
       if (dependencyMetaList.length !== 0) {
         const dependenciesMeta: IDependenciesMetaTable = {};
         for (const dependencyMeta of dependencyMetaList) {
-          dependenciesMeta[dependencyMeta.name] = {
-            injected: dependencyMeta.injected
-          };
+          dependenciesMeta[dependencyMeta.name] = dependencyMeta.sourceData;
         }
 
         // get the relative path from common temp folder to package folder, to align with the value in pnpm-lock.yaml

libraries/rush-lib/src/logic/pnpm/PnpmShrinkwrapFile.ts

@@ -32,15 +32,14 @@ import { PnpmOptionsConfiguration } from './PnpmOptionsConfiguration';
 import type { IPnpmfile, IPnpmfileContext } from './IPnpmfile';
 import type { Subspace } from '../../api/Subspace';
 import { CustomTipId, type CustomTipsConfiguration } from '../../api/CustomTipsConfiguration';
+import { objectsAreDeepEqual } from '../../utilities/objectUtilities';
 
 const yamlModule: typeof import('js-yaml') = Import.lazy('js-yaml', require);
 
 export interface IPeerDependenciesMetaYaml {
   optional?: boolean;
 }
-export interface IDependenciesMetaYaml {
-  injected?: boolean;
-}
+export type IDependenciesMetaYaml = object;
 
 export type IPnpmV7VersionSpecifier = string;
 export interface IPnpmV8VersionSpecifier {
@@ -1040,8 +1039,8 @@ export class PnpmShrinkwrapFile extends BaseShrinkwrapFile {
         }
       }
 
-      for (const { name, injected } of dependencyMetaList) {
-        if (importer.dependenciesMeta?.[name]?.injected === injected) {
+      for (const { name, sourceData } of dependencyMetaList) {
+        if (objectsAreDeepEqual(importer.dependenciesMeta?.[name], sourceData)) {
           importerDependenciesMeta.delete(name);
         }
       }