Cleanup IPC[SOCKET_CLI_FIX_PACKAGE_LOCK_FILE] use

Author: jdalton

src/shadow/arborist/lib/arborist/diff.ts

@@ -3,8 +3,7 @@ import constants from '../../../../constants'
 import type { Diff } from './types'
 import type { SafeNode } from '../node'
 
-const { LOOP_SENTINEL, NPM_REGISTRY_URL, SOCKET_CLI_FIX_PACKAGE_LOCK_FILE } =
-  constants
+const { LOOP_SENTINEL, NPM_REGISTRY_URL } = constants
 
 function getUrlOrigin(input: string): string {
   try {
@@ -28,11 +27,9 @@ export function getPackagesToQueryFromDiff(
   diff_: Diff | null,
   options?: GetPackagesToQueryFromDiffOptions
 ): PackageDetail[] {
-  const {
-    // Lazily access constants.IPC.
-    includeUnchanged = constants.IPC[SOCKET_CLI_FIX_PACKAGE_LOCK_FILE],
-    includeUnknownOrigin = false
-  } = <GetPackagesToQueryFromDiffOptions>{
+  const { includeUnchanged = false, includeUnknownOrigin = false } = <
+    GetPackagesToQueryFromDiffOptions
+  >{
     __proto__: null,
     ...options
   }

src/shadow/arborist/lib/arborist/reify.ts

@@ -368,7 +368,7 @@ export async function reify(
   // We are assuming `this[_diffTrees]()` has been called by `super.reify(...)`:
   // https://github.com/npm/cli/blob/v11.0.0/workspaces/arborist/lib/arborist/reify.js#L141
   let needInfoOn = getPackagesToQueryFromDiff(this.diff, {
-    includeUnchanged: IPC[SOCKET_CLI_FIX_PACKAGE_LOCK_FILE]
+    includeUnchanged: !!IPC[SOCKET_CLI_FIX_PACKAGE_LOCK_FILE]
   })
   if (!needInfoOn.length) {
     // Nothing to check, hmmm already installed or all private?