Fix getNpmPackageAccess to respect publishConfig.registry

Fixes #763

Author: sindresorhus

source/ui.js

@@ -228,7 +228,7 @@ const ui = async ({packageManager, ...options}, {package_, rootDirectory}) => {
 		&& !options.tag
 	);
 
-	const alreadyPublicScoped = packageManager.id === 'yarn-berry' && options.runPublish && await util.getNpmPackageAccess(package_.name) === 'public';
+	const alreadyPublicScoped = packageManager.id === 'yarn-berry' && options.runPublish && await util.getNpmPackageAccess(package_) === 'public';
 
 	// Note that inquirer question.when is a bit confusing. Only `false` will cause the question to be skipped.
 	// Any other value like `true` and `undefined` means ask the question.

source/util.js

@@ -155,7 +155,13 @@ export const validateEngineVersionSatisfies = (engine, version) => {
 	}
 };
 
-export async function getNpmPackageAccess(name) {
-	const {stdout} = await execa('npm', ['access', 'get', 'status', name, '--json']);
-	return JSON.parse(stdout)[name]; // Note: returns "private" for non-existent packages
+export async function getNpmPackageAccess(package_) {
+	const arguments_ = ['access', 'get', 'status', package_.name, '--json'];
+
+	if (package_.publishConfig?.registry) {
+		arguments_.push('--registry', package_.publishConfig.registry);
+	}
+
+	const {stdout} = await execa('npm', arguments_);
+	return JSON.parse(stdout)[package_.name]; // Note: returns "private" for non-existent packages
 }

test/util/get-npm-package-access.js

@@ -0,0 +1,45 @@
+import test from 'ava';
+import {_createFixture} from '../_helpers/stub-execa.js';
+
+/** @type {ReturnType<typeof _createFixture<import('../../source/util.js')>>} */
+const createFixture = _createFixture('../../source/util.js', import.meta.url);
+
+test('main', createFixture, [{
+	command: 'npm access get status @my/pkg --json',
+	stdout: '{"@my/pkg": "public"}',
+}], async ({t, testedModule: {getNpmPackageAccess}}) => {
+	t.is(
+		await getNpmPackageAccess({name: '@my/pkg'}),
+		'public',
+	);
+});
+
+test('with publishConfig.registry', createFixture, [{
+	command: 'npm access get status @my/pkg --json --registry https://registry.npmjs.org/',
+	stdout: '{"@my/pkg": "public"}',
+}], async ({t, testedModule: {getNpmPackageAccess}}) => {
+	t.is(
+		await getNpmPackageAccess({
+			name: '@my/pkg',
+			publishConfig: {
+				registry: 'https://registry.npmjs.org/',
+			},
+		}),
+		'public',
+	);
+});
+
+test('with external registry', createFixture, [{
+	command: 'npm access get status @my/pkg --json --registry http://my-internal-registry.local',
+	stdout: '{"@my/pkg": "private"}',
+}], async ({t, testedModule: {getNpmPackageAccess}}) => {
+	t.is(
+		await getNpmPackageAccess({
+			name: '@my/pkg',
+			publishConfig: {
+				registry: 'http://my-internal-registry.local',
+			},
+		}),
+		'private',
+	);
+});