Skip to content

fix: ci pnpm version mismatch and undefined headers in getBody#81

Closed
karol-broda wants to merge 6 commits into
better-auth:mainfrom
karol-broda:ci/pnpm-version-mismatch
Closed

fix: ci pnpm version mismatch and undefined headers in getBody#81
karol-broda wants to merge 6 commits into
better-auth:mainfrom
karol-broda:ci/pnpm-version-mismatch

Conversation

@karol-broda

@karol-broda karol-broda commented Dec 26, 2025

Copy link
Copy Markdown
Contributor

hey! noticed ci was failing because pnpm/action-setup was set to version 9 but package.json has packageManager set to pnpm 10.15.1.

removed the hardcoded version from the action so it reads from package.json directly.

also fixed a couple of bugs in utils.ts:

  • headers variable was undefined in getBody
  • getHeaders was using Object.entries on headers which breaks array-style headers like [["X-header", "1"]]

@karol-broda karol-broda changed the title ci: fix pnpm version mismatch fix: ci pnpm version mismatch and undefined headers in getBody Dec 26, 2025
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@bytaesu

bytaesu commented Jun 9, 2026

Copy link
Copy Markdown
Member

Hi @karol-broda, thanks for working on this! I'm closing this PR since I resolved this issue as well while migrating the release process to OIDC 😁

@bytaesu bytaesu closed this Jun 9, 2026
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.

4 participants