Skip to content

[820] General: Add frontend .env validation script for contract deployment#832

Merged
Junirezz merged 3 commits into
Junirezz:mainfrom
Max-Owolabi:fix/820-general-add-frontend-env-validation-script-for-contract-deployment
Jun 26, 2026
Merged

[820] General: Add frontend .env validation script for contract deployment#832
Junirezz merged 3 commits into
Junirezz:mainfrom
Max-Owolabi:fix/820-general-add-frontend-env-validation-script-for-contract-deployment

Conversation

@Max-Owolabi

Copy link
Copy Markdown
Contributor

Summary

  • Add scripts/validate-frontend-env.ts to validate VITE_VAULT_CONTRACT_ID, VITE_SOROBAN_RPC_URL, and VITE_STELLAR_NETWORK_PASSPHRASE (format, RPC/passphrase alignment, optional RPC reachability, and deployment.json cross-check).
  • Run validation in CI after testnet deploy (rust-wasm.yml) and add a dedicated unit-test job in frontend.yml.
  • Document the validation step in README Getting Started, ENV_SETUP_README.md, and frontend/.env.example.

Test plan

  • npm run test:validate-frontend-env
  • Manual: npm run validate:frontend-env -- --deployment-json deployment.json --strict --check-rpc with env vars set
  • CI: frontend env validation job passes on PR
  • CI: testnet deploy workflow validates env after smoke test (on merge to main)

Closes #820

Introduce validate-frontend-env.ts to verify VITE contract/RPC/passphrase
settings against deployment.json, wire it into CI after testnet deploy, and
document the check in setup guides.

Co-authored-by: Cursor <cursoragent@cursor.com>
@drips-wave

drips-wave Bot commented Jun 24, 2026

Copy link
Copy Markdown

@Max-Owolabi Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

Max-Owolabi and others added 2 commits June 24, 2026 21:16
Resolve duplicate VaultDashboard imports, type errors, CSS syntax,
and missing test dependencies so monorepo CI passes on this PR.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Junirezz Junirezz merged commit 749d2bb into Junirezz:main Jun 26, 2026
4 of 8 checks passed
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.

General: Add frontend .env validation script for contract deployment

2 participants