Skip to content

test oidc auth by forcing empty credentials - #6717

Merged
gonzaloriestra merged 2 commits into
mainfrom
test-oidc-fixes
Dec 15, 2025
Merged

gonzaloriestra merged 2 commits into
mainfrom
test-oidc-fixes

Conversation

@alexanderMontague

@alexanderMontague alexanderMontague commented Dec 12, 2025 •

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

  • Part of enabling OIDC authentication when publishing new versions
  • The manual/nightly releases were working, but our regular patch/minor releases were not
  • This is because setting NPM_TOKEN or NODE_AUTH_TOKEN causes changeset/action to create an .npmrc, which then in turn tells npm to use the regular token flow
  • NPM_TOKEN was correctly being set to an empty string, but NODE_AUTH_TOKEN was not, so I took the same approach
  • I created a github action to test the OIDC auth flow, and it seems to have succeeded

WHAT is this pull request doing?

  • also set NODE_AUTH_TOKEN to an empty string to force provenance/OIDC authentication instead of the legacy token auth flow

How to test your changes?

  • retrigger our patch release

@alexanderMontague
alexanderMontague marked this pull request as ready for review December 12, 2025 17:18
@alexanderMontague
alexanderMontague requested a review from a team as a code owner December 12, 2025 17:18
@github-actions

Copy link
Copy Markdown
Contributor

Coverage report

St.❔
Category Percentage Covered / Total
🟡 Statements
79.47% (+0.24% 🔼)
14142/17795
🟡 Branches
73.53% (+0.42% 🔼)
6936/9433
🟡 Functions
79.63% (+0.26% 🔼)
3631/4560
🟡 Lines
79.84% (+0.26% 🔼)
13368/16744
Show new covered files 🐣
St.❔
File Statements Branches Functions Lines
🟢
... / admin-as-app.ts
100% 100% 100% 100%
🟢
... / metafield_definitions.ts
100% 100% 100% 100%
🟢
... / metaobject_definitions.ts
100% 100% 100% 100%
🟢
... / bulk-operation-run-mutation.ts
100% 100% 100% 100%
🟢
... / bulk-operation-run-query.ts
100% 100% 100% 100%
🟢
... / get-bulk-operation-by-id.ts
100% 100% 100% 100%
🟢
... / list-bulk-operations.ts
100% 100% 100% 100%
🟢
... / staged-uploads-create.ts
100% 100% 100% 100%
🔴
... / execute.ts
0% 0% 0% 0%
🔴
... / status.ts
0% 0% 0% 0%
🔴
... / pull.ts
0% 100% 0% 0%
🟢
... / execute-operation.ts
92.86% 80% 100% 92.31%
🔴
... / pull.ts
0% 0% 0% 0%
🟢
... / bulk-operation-status.ts
96.61% 92.11% 100% 100%
🟢
... / download-bulk-operation-results.ts
100% 100% 100% 100%
🟢
... / execute-bulk-operation.ts
92.98% 86.11% 100% 92.73%
🟢
... / format-bulk-operation-status.ts
100% 100% 100% 100%
🟢
... / run-mutation.ts
100% 100% 100% 100%
🟢
... / run-query.ts
100% 100% 100% 100%
🟡
... / stage-file.ts
73.53% 62.5% 85.71% 72.73%
🟢
... / watch-bulk-operation.ts
100% 100% 100% 100%
🟢
... / declarative-definitions.ts
98.54% 93.18% 100% 98.51%
🟢
... / common.ts
96.15% 93.33% 100% 95.45%
🟢
... / execute-command-helpers.ts
100% 100% 100% 100%
🔴
... / promiseWithResolvers.ts
33.33% 50% 50% 33.33%
Show files with reduced coverage 🔻
St.❔
File Statements Branches Functions Lines
🔴
... / execute.ts
0%
0% (-100% 🔻)
0% 0%
🟢
... / extension-instance.ts
84.8% (+0.23% 🔼)
77.6% (-0.91% 🔻)
92.06% (+0.13% 🔼)
85.11% (+0.24% 🔼)
🟡
... / specification.ts
69.09%
75.61% (+2.44% 🔼)
76.47% (-1.31% 🔻)
68.75%
🟢
... / ui_extension.ts
85.38% (-9.44% 🔻)
72.34% (-8.91% 🔻)
84% (-16% 🔻)
88% (-8.46% 🔻)
🟢
... / developer-platform-client.ts
84.62% (-1.5% 🔻)
71.43% (+0.84% 🔼)
81.82% (+1.82% 🔼)
93.75% (+0.42% 🔼)
🟢
... / api.ts
87.07% (-0.43% 🔻)
76.71% (-0.1% 🔻)
100%
86.49% (-0.43% 🔻)
🟢
... / SingleTask.tsx
84.21% (-15.79% 🔻)
50% (-50% 🔻)
80% (-20% 🔻)
84.21% (-15.79% 🔻)
🔴
... / environment.ts
35% (-5% 🔻)
41.18%
40% (-10% 🔻)
36.84% (-5.26% 🔻)
🔴
... / ui.tsx
50.82% (-0.79% 🔻)
42.86% (-5.53% 🔻)
54.55% (+1.42% 🔼)
50% (-0.82% 🔻)
🟢
... / console.ts
81.82% (+15.15% 🔼)
75% (-25% 🔻)
100% (+33.33% 🔼)
81.82% (+15.15% 🔼)
🔴
... / dev.ts
14.29% (+0.95% 🔼)
3.13% (+0.18% 🔼)
50% (-7.14% 🔻)
14.29% (+0.95% 🔼)
🟢
... / init.ts
88% (-0.89% 🔻)
71.43% (+4.76% 🔼)
86.67% (+4.85% 🔼)
88% (-0.89% 🔻)
🟢
... / storefront-renderer.ts
90.2% (-0.54% 🔻)
78.95%
81.82% (-1.52% 🔻)
90.2% (-0.54% 🔻)
🟡
... / theme-polling.ts
67.12% (-0.93% 🔻)
68.75% 78.57%
66.67% (-0.98% 🔻)

Test suite run success

3532 tests passing in 1413 suites.

Report generated by 🧪jest coverage report action from f0a718e

@gonzaloriestra
gonzaloriestra added this pull request to the merge queue Dec 15, 2025
Merged via the queue into main with commit ef50351 Dec 15, 2025
24 checks passed
@gonzaloriestra
gonzaloriestra deleted the test-oidc-fixes branch December 15, 2025 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants