feat: COREPACK_NPM_REGISTRY defaults to the regitry of npm

Author: zanminkian

README.md

@@ -341,7 +341,9 @@ same major line. Should you need to upgrade to a new major, use an explicit
   (useful for commands like `yarn init`).
 
 - `COREPACK_NPM_REGISTRY` sets the registry base url used when retrieving
-  package managers from npm. Default value is `https://registry.npmjs.org`
+  package managers from npm. Default value is the result of command
+  `npm config get registry`, which is typically `https://registry.npmjs.org`
+  unless it has been customized.
 
 - `COREPACK_NPM_TOKEN` sets a Bearer token authorization header when connecting
   to a npm type registry.

package.json

@@ -32,6 +32,7 @@
     "esbuild": "^0.25.0",
     "eslint": "^9.22.0",
     "proxy-from-env": "^1.1.0",
+    "registry-url": "^7.1.0",
     "semver": "^7.6.3",
     "supports-color": "^10.0.0",
     "tar": "^7.4.0",

sources/corepackUtils.ts

@@ -224,6 +224,7 @@ export async function installVersion(installTarget: string, locator: Locator, {s
   let signatures: Array<{keyid: string, sig: string}>;
   let integrity: string;
   let binPath: string | null = null;
+  const registryUrl = npmRegistryUtils.getRegistryUrl();
   if (locatorIsASupportedPackageManager) {
     url = spec.url.replace(`{}`, version);
     if (process.env.COREPACK_NPM_REGISTRY) {
@@ -234,17 +235,17 @@ export async function installVersion(installTarget: string, locator: Locator, {s
           binPath = registry.bin;
         }
       }
-      url = url.replace(
-        npmRegistryUtils.DEFAULT_NPM_REGISTRY_URL,
-        () => process.env.COREPACK_NPM_REGISTRY!,
-      );
     }
+    url = url.replace(
+      npmRegistryUtils.DEFAULT_NPM_REGISTRY_URL,
+      () => registryUrl,
+    );
   } else {
     url = decodeURIComponent(version);
-    if (process.env.COREPACK_NPM_REGISTRY && url.startsWith(npmRegistryUtils.DEFAULT_NPM_REGISTRY_URL)) {
+    if (url.startsWith(npmRegistryUtils.DEFAULT_NPM_REGISTRY_URL)) {
       url = url.replace(
         npmRegistryUtils.DEFAULT_NPM_REGISTRY_URL,
-        () => process.env.COREPACK_NPM_REGISTRY!,
+        () => registryUrl,
       );
     }
   }

sources/httpUtils.ts

@@ -4,7 +4,7 @@ import {once}                     from 'events';
 import {stderr, stdin}            from 'process';
 import {Readable}                 from 'stream';
 
-import {DEFAULT_NPM_REGISTRY_URL} from './npmRegistryUtils';
+import {getRegistryUrl}           from './npmRegistryUtils';
 
 async function fetch(input: string | URL, init?: RequestInit) {
   if (process.env.COREPACK_ENABLE_NETWORK === `0`)
@@ -29,7 +29,7 @@ async function fetch(input: string | URL, init?: RequestInit) {
     input.username = input.password = ``;
   }
 
-  if (input.origin === (process.env.COREPACK_NPM_REGISTRY || DEFAULT_NPM_REGISTRY_URL) && process.env.COREPACK_NPM_TOKEN) {
+  if (process.env.COREPACK_NPM_TOKEN && input.origin === getRegistryUrl()) {
     headers =  {
       ...headers,
       authorization: `Bearer ${process.env.COREPACK_NPM_TOKEN}`,

sources/npmRegistryUtils.ts

@@ -1,5 +1,6 @@
 import {UsageError}               from 'clipanion';
 import {createVerify}             from 'crypto';
+import registryUrl, {defaultUrl}  from 'registry-url';
 
 import defaultConfig              from '../config.json';
 
@@ -11,10 +12,20 @@ import * as httpUtils             from './httpUtils';
 export const DEFAULT_HEADERS: Record<string, string> = {
   [`Accept`]: `application/vnd.npm.install-v1+json; q=1.0, application/json; q=0.8`,
 };
-export const DEFAULT_NPM_REGISTRY_URL = `https://registry.npmjs.org`;
+
+const normalize = (url: string) => url.endsWith(`/`) ? url.slice(0, -1) : url;
+
+export const DEFAULT_NPM_REGISTRY_URL = normalize(defaultUrl);
+
+export function getRegistryUrl() {
+  if (process.env.COREPACK_NPM_REGISTRY) {
+    return process.env.COREPACK_NPM_REGISTRY;
+  }
+  return normalize(registryUrl());
+}
 
 export async function fetchAsJson(packageName: string, version?: string) {
-  const npmRegistryUrl = process.env.COREPACK_NPM_REGISTRY || DEFAULT_NPM_REGISTRY_URL;
+  const npmRegistryUrl = getRegistryUrl();
 
   if (process.env.COREPACK_ENABLE_NETWORK === `0`)
     throw new UsageError(`Network access disabled by the environment; can't reach npm repository ${npmRegistryUrl}`);

yarn.lock

@@ -1315,6 +1315,7 @@ __metadata:
     esbuild: "npm:^0.25.0"
     eslint: "npm:^9.22.0"
     proxy-from-env: "npm:^1.1.0"
+    registry-url: "npm:^7.1.0"
     semver: "npm:^7.6.3"
     supports-color: "npm:^10.0.0"
     tar: "npm:^7.4.0"
@@ -2003,6 +2004,13 @@ __metadata:
   languageName: node
   linkType: hard
 
+"find-up-simple@npm:^1.0.1":
+  version: 1.0.1
+  resolution: "find-up-simple@npm:1.0.1"
+  checksum: 10c0/ad34de157b7db925d50ff78302fefb28e309f3bc947c93ffca0f9b0bccf9cf1a2dc57d805d5c94ec9fc60f4838f5dbdfd2a48ecd77c23015fa44c6dd5f60bc40
+  languageName: node
+  linkType: hard
+
 "find-up@npm:^5.0.0":
   version: 5.0.0
   resolution: "find-up@npm:5.0.0"
@@ -2377,6 +2385,13 @@ __metadata:
   languageName: node
   linkType: hard
 
+"ini@npm:^5.0.0":
+  version: 5.0.0
+  resolution: "ini@npm:5.0.0"
+  checksum: 10c0/657491ce766cbb4b335ab221ee8f72b9654d9f0e35c32fe5ff2eb7ab8c5ce72237ff6456555b50cde88e6507a719a70e28e327b450782b4fc20c90326ec8c1a8
+  languageName: node
+  linkType: hard
+
 "ini@npm:~1.3.0":
   version: 1.3.8
   resolution: "ini@npm:1.3.8"
@@ -3443,6 +3458,16 @@ __metadata:
   languageName: node
   linkType: hard
 
+"registry-url@npm:^7.1.0":
+  version: 7.1.0
+  resolution: "registry-url@npm:7.1.0"
+  dependencies:
+    find-up-simple: "npm:^1.0.1"
+    ini: "npm:^5.0.0"
+  checksum: 10c0/b9ccddc4d079cc3e6cc9b137aef1a71aaab02d04785890b8f86d1b63eb158e0198e17a444b3bba2986eadc0beff18fae1be058665b4197deec9422650100fb24
+  languageName: node
+  linkType: hard
+
 "resolve-from@npm:^4.0.0":
   version: 4.0.0
   resolution: "resolve-from@npm:4.0.0"