Skip to content

chore: standardize Openfort/Shield env var names#318

Open
joalavedra wants to merge 1 commit into
mainfrom
chore/standardize-env-var-names
Open

chore: standardize Openfort/Shield env var names#318
joalavedra wants to merge 1 commit into
mainfrom
chore/standardize-env-var-names

Conversation

@joalavedra

Copy link
Copy Markdown
Contributor

Standardizes the Openfort and Shield environment-variable names used across the example apps and the Playwright CI workflow. Publishable keys keep their framework prefix (NEXT_PUBLIC_ / VITE_); secret keys and the encryption share are server-side and carry no public prefix.

Renames

Old New
NEXT_PUBLIC_SHIELD_API_KEY NEXT_PUBLIC_SHIELD_PUBLISHABLE_KEY
NEXT_PUBLIC_OPENFORT_PUBLIC_KEY NEXT_PUBLIC_OPENFORT_PUBLISHABLE_KEY
NEXTAUTH_SHIELD_SECRET_KEY SHIELD_SECRET_KEY
NEXTAUTH_SHIELD_ENCRYPTION_SHARE SHIELD_ENCRYPTION_SHARE
NEXTAUTH_OPENFORT_SECRET_KEY OPENFORT_SECRET_KEY
VITE_APP_SHIELD_PUBLISHABLE_KEY VITE_SHIELD_PUBLISHABLE_KEY
VITE_APP_OPENFORT_PUBLISHABLE_KEY VITE_OPENFORT_PUBLISHABLE_KEY

19 files changed (example .env* files, READMEs, source under examples/, and .github/workflows/playwright.yml). The workflow's ${{ vars.* }} / ${{ secrets.* }} references are renamed to match; the corresponding GitHub Actions repo variables/secrets must be renamed for CI to keep working.

Left unchanged

  • ApiKeyType enum values PUBLIC_KEY_SHIELD / SECRET_KEY_SHIELD (values pk_shield / sk_shield) in the generated OpenAPI client (packages/internal/openapi-clients/) are SDK key-type constants, not env vars.
  • VITE_SHIELD_PUBLISHABLE_KEY in examples/apps/solana-sample/* (already correct) and VITE_SHIELD_URL (out of scope).
  • No genuine NEXTAUTH_SECRET / NEXTAUTH_URL vars are present in tracked text files.

🤖 Generated with Claude Code

@vercel

vercel Bot commented Jun 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
firebase-auth-embedded-wallet Error Error Jun 26, 2026 6:50pm
openfort-auth-non-custodial Ready Ready Preview, Comment Jun 26, 2026 6:50pm
openfort-js-sdk-solana Ready Ready Preview, Comment Jun 26, 2026 6:50pm
openfort-wagmi-next Ready Ready Preview, Comment Jun 26, 2026 6:50pm

Request Review

@pkg-pr-new

pkg-pr-new Bot commented Jun 26, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@openfort/openfort-js@318

commit: eaab52a

Rename client-exposed publishable/key vars to the standard base names:
- NEXT_PUBLIC_SHIELD_API_KEY      -> NEXT_PUBLIC_SHIELD_PUBLISHABLE_KEY
- NEXT_PUBLIC_OPENFORT_PUBLIC_KEY -> NEXT_PUBLIC_OPENFORT_PUBLISHABLE_KEY
- VITE_APP_SHIELD_PUBLISHABLE_KEY -> VITE_SHIELD_PUBLISHABLE_KEY
- VITE_APP_OPENFORT_PUBLISHABLE_KEY -> VITE_OPENFORT_PUBLISHABLE_KEY

The NEXTAUTH_-prefixed backend vars (NEXTAUTH_SHIELD_SECRET_KEY,
NEXTAUTH_SHIELD_ENCRYPTION_SHARE, NEXTAUTH_OPENFORT_SECRET_KEY) are intentional
backend-env names and are left unchanged. PUBLIC_KEY_SHIELD/SECRET_KEY_SHIELD
are SDK enum identifiers, not env vars, and are left unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.

1 participant