Fix peer dependencies for versions with range operators (#1378).

getPeerDependencies and getPeerDependenciesFromRegistry expect exact versions, not version ranges.

Author: raineorshine

src/lib/upgradePackageDefinitions.ts

@@ -46,8 +46,10 @@ export async function upgradePackageDefinitions(
     return !options.jsonUpgraded || !options.minimal || !satisfies(latestVersions[dep], currentDependencies[dep])
   })
 
+  const filteredLatestDependencies = pickBy(latestVersions, (spec, dep) => filteredUpgradedDependencies[dep])
+
   if (options.peer && !isEmpty(filteredUpgradedDependencies)) {
-    const upgradedPeerDependencies = await getPeerDependenciesFromRegistry(filteredUpgradedDependencies, options)
+    const upgradedPeerDependencies = await getPeerDependenciesFromRegistry(filteredLatestDependencies, options)
     const peerDependencies = { ...options.peerDependencies, ...upgradedPeerDependencies }
     if (!isEqual(options.peerDependencies, peerDependencies)) {
       const [newUpgradedDependencies, newLatestVersions, newPeerDependencies] = await upgradePackageDefinitions(

src/package-managers/npm.ts

@@ -653,11 +653,7 @@ export const greatest: GetVersion = async (
  * @returns Promised {packageName: version} collection
  */
 export const getPeerDependencies = async (packageName: string, version: Version): Promise<Index<Version>> => {
-  // if version number uses >, omit the version and find latest
-  // otherwise, it will error out in the shell
-  // https://github.com/raineorshine/npm-check-updates/issues/1181
-  const atVersion = !version.startsWith('>') ? `@${version}` : ''
-  const args = ['view', `${packageName}${atVersion}`, 'peerDependencies']
+  const args = ['view', `${packageName}@${version}`, 'peerDependencies']
   const result = await spawnNpm(args, {}, { rejectOnError: false })
   return result ? parseJson(result, { command: [...args, '--json'].join(' ') }) : {}
 }