Skip to content
This repository was archived by the owner on Oct 28, 2026. It is now read-only.

Prepare tokenless npm publishing - #314

Merged
Vladimir Morozov (vmoroz) merged 1 commit into
mainfrom
PR/publish-final-notice
Oct 27, 2026
Merged

Vladimir Morozov (vmoroz) merged 1 commit into
mainfrom
PR/publish-final-notice

Conversation

@vmoroz

@vmoroz Vladimir Morozov (vmoroz) commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

Prepare tokenless npm publishing

Why

Prepare the existing Publish workflow to release the final react-native-xaml
notice version using npm trusted publishing instead of a stored npm token.

What

  • Tokenless authentication. Use Node 24, require npm 11.5.1 or later, and
    grant OIDC permission only to the publishing job. Remove the token reference
    without deleting the stored secret.
  • Approved restores. Install locked npm dependencies and restore native
    and Codegen NuGet dependencies through PackageFeedProxy, with no implicit
    Codegen restore afterward.
  • Preserved behavior. Retain the existing triggers, checks, runner choices,
    pinned actions and package version. npm-side trust must be configured before
    any future publishing run.

Testing

Workflow schemas, npm-version checks, proxy restores, TypeScript/C# builds
and both example suites passed. Live OIDC and publishing were not exercised.

Changelog

Should this change be included in the release notes: no. Prepare tokenless
authentication and approved dependency restores for the publishing workflow.

Microsoft Reviewers: Open in CodeFlow

Use supported Node and npm with job-scoped OIDC instead of a stored token.
Restore publishing prerequisites through the approved package feeds.
Preserve the prepared package version and existing publication triggers.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@vmoroz
Vladimir Morozov (vmoroz) requested a review from a team as a code owner October 27, 2026 23:11
@vmoroz
Vladimir Morozov (vmoroz) merged commit a648015 into main Oct 27, 2026
11 checks passed
@vmoroz
Vladimir Morozov (vmoroz) deleted the PR/publish-final-notice branch October 27, 2026 23:37
Vladimir Morozov (vmoroz) added a commit that referenced this pull request Oct 28, 2026
# Avoid token-style npm config for OIDC publishing

## Why

The tokenless setup from #314 fails before dependency installation.
setup-node's
`registry-url` input generates an npmrc referencing `NODE_AUTH_TOKEN`,
which is
intentionally unset for OIDC; Yarn classic rejects that unresolved
reference.

## What

- **Tokenless installation.** Remove the redundant setup-node registry
input
  so Yarn does not require token authentication configuration.
- **Unchanged destinations.** Keep the existing explicit proxy-install
and
  public-publish registry flags, OIDC permissions, and package version.

## Testing

Reproduced the exact Yarn error; tokenless regression, frozen install,
build,
both example suites and workflow schema checks passed. Publication was
not retried.

## Changelog

Should this change be included in the release notes: no. Avoid
token-style npm
configuration in the OIDC publishing workflow.

###### Microsoft Reviewers: [Open in
CodeFlow](https://microsoft.github.io/open-pr/?codeflow=https://github.com/microsoft/react-native-xaml/pull/315)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants