fix: remove duplicate var (#8117)

TrevorBurnham · web-flow · commit d586f3b6da5c · 2025-02-18T11:55:48.000-08:00
Following up on #8115. Since `cleanFetchSpec` no longer differs from `fetchSpec`, it's no longer needed. cc @wraithgar
diff --git a/workspaces/arborist/lib/consistent-resolve.js b/workspaces/arborist/lib/consistent-resolve.js
@@ -20,11 +20,10 @@ const consistentResolve = (resolved, fromPath, toPath, relPaths = false) => {
       raw,
     } = npa(resolved, fromPath)
     if (type === 'file' || type === 'directory') {
-      const cleanFetchSpec = fetchSpec
       if (relPaths && toPath) {
-        return `file:${relpath(toPath, cleanFetchSpec)}`
+        return `file:${relpath(toPath, fetchSpec)}`
       }
-      return `file:${cleanFetchSpec}`
+      return `file:${fetchSpec}`
     }
     if (hosted) {
       return `git+${hosted.auth ? hosted.https(hostedOpt) : hosted.sshurl(hostedOpt)}`

Original file line number	Diff line number	Diff line change
`@@ -20,11 +20,10 @@ const consistentResolve = (resolved, fromPath, toPath, relPaths = false) => {`
`20`	`20`	`raw,`
`21`	`21`	`} = npa(resolved, fromPath)`
`22`	`22`	`if (type === 'file' \|\| type === 'directory') {`
`23`		`- const cleanFetchSpec = fetchSpec`
`24`	`23`	`if (relPaths && toPath) {`
`25`		- return `file:${relpath(toPath, cleanFetchSpec)}`
	`24`	+ return `file:${relpath(toPath, fetchSpec)}`
`26`	`25`	`}`
`27`		- return `file:${cleanFetchSpec}`
	`26`	+ return `file:${fetchSpec}`
`28`	`27`	`}`
`29`	`28`	`if (hosted) {`
`30`	`29`	return `git+${hosted.auth ? hosted.https(hostedOpt) : hosted.sshurl(hostedOpt)}`