Skip to content

fix(owner): use scoped registry for user lookup - #9786

Merged
martinrrm merged 1 commit into
latestfrom
fix/15426-owner-scoped-registry
Jul 27, 2026
Merged

fix(owner): use scoped registry for user lookup#9786
martinrrm merged 1 commit into
latestfrom
fix/15426-owner-scoped-registry

Conversation

@martinrrm

Copy link
Copy Markdown
Contributor

Summary

Ensure npm owner add and npm owner rm resolve users from the same registry used for the target package.

The user lookup now receives the package spec, allowing npm-registry-fetch to honor scoped registry configuration instead of falling back to the global registry.

Testing

Added regression coverage for split global/scoped registry configurations, including:

  • Preventing substitution of the user added as an owner.
  • Preventing removal of an unintended existing owner.

Resolve owner identities through the same scoped registry used for package reads and ownership mutations. Add regression coverage for cross-registry substitution in owner add and remove.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 1e1231bc-474e-4a27-8b78-242366f48e19
@martinrrm
martinrrm requested review from a team as code owners July 20, 2026 17:38
@martinrrm
martinrrm merged commit ef6cfea into latest Jul 27, 2026
28 checks passed
@martinrrm
martinrrm deleted the fix/15426-owner-scoped-registry branch July 27, 2026 23:08
@github-actions

Copy link
Copy Markdown
Contributor

🎉 Backport to release/v11 created: #9810

martinrrm added a commit that referenced this pull request Jul 28, 2026
Backport of #9786 to `release/v11`.

Co-authored-by: martin <martin.ruiz.mares@gmail.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 1e1231bc-474e-4a27-8b78-242366f48e19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants