feat: handle workspace protocol with any semver range specifier. fixes …

…pnpm#7578
EWhite613 · Feb 10, 2024 · 94b8bf7 · 94b8bf7
1 parent 1a3449e
commit 94b8bf7
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/pkg-manifest/exportable-manifest/src/index.ts b/pkg-manifest/exportable-manifest/src/index.ts
@@ -59,12 +59,13 @@ async function makePublishDependencies (
 }
 
 async function makePublishDependency (depName: string, depSpec: string, dir: string, modulesDir?: string) {
- if (!depSpec.startsWith('workspace:')) {
+ if (!depSpec.includes('workspace:')) {
  return depSpec
  }
 
- // Dependencies with bare "*", "^" and "~" versions
- const versionAliasSpecParts = /^workspace:(.*?)@?([\^~*])$/.exec(depSpec)
+ // Dependencies with bare "*", "^", "~",">=",">","<=",< versions
+ const workspaceSemverRegex = /workspace:(.*?)@?([\^~*]|>=|>|<=|<)/
+ const versionAliasSpecParts = workspaceSemverRegex.exec(depSpec)
  if (versionAliasSpecParts != null) {
  modulesDir = modulesDir ?? path.join(dir, 'node_modules')
  const { manifest } = await tryReadProjectManifest(path.join(modulesDir, depName))
@@ -80,10 +81,10 @@ async function makePublishDependency (depName: string, depSpec: string, dir: str
  if (depName !== manifest.name) {
  return `npm:${manifest.name!}@${semverRangeToken}${manifest.version}`
  }
- return `${semverRangeToken}${manifest.version}`
+ return depSpec.replace(workspaceSemverRegex, `${semverRangeToken}${manifest.version}`)
  }
- if (depSpec.startsWith('workspace:./') || depSpec.startsWith('workspace:../')) {
- const { manifest } = await tryReadProjectManifest(path.join(dir, depSpec.slice(10)))
+ if (depSpec.includes('workspace:./') || depSpec.includes('workspace:../')) {
+ const { manifest } = await tryReadProjectManifest(path.join(dir, depSpec.replace('workspace:', '')))
  if (!manifest?.name || !manifest?.version) {
  throw new PnpmError(
  'CANNOT_RESOLVE_WORKSPACE_PROTOCOL',
@@ -94,7 +95,7 @@ async function makePublishDependency (depName: string, depSpec: string, dir: str
  if (manifest.name === depName) return `${manifest.version}`
  return `npm:${manifest.name}@${manifest.version}`
  }
- depSpec = depSpec.slice(10)
+ depSpec = depSpec.replace('workspace:', '')
  if (depSpec.includes('@')) {
  return `npm:${depSpec}`
  }