fix: add NEXT_PUBLIC_ELECTRIC_URL to macOS desktop build

[saddlepaddle]

Summary

• The macOS build in build-desktop.yml was missing NEXT_PUBLIC_ELECTRIC_URL in the compile env vars (Linux already had it)
• Without this, macOS builds fall back to the hardcoded default in t3-env, which works but is inconsistent with Linux

Test plan

• Verified Linux build already has the env var
• Confirmed t3-env default prevents build breakage even without the secret

---

Summary by cubic

Add NEXT_PUBLIC_ELECTRIC_URL to the macOS desktop build so it uses the secret like Linux instead of the t3-env default. This ensures consistent Electric configuration across platforms.

^{Written for commit 43979d2. Summary will update on new commits.}

Summary by CodeRabbit

• Chores
  • Updated desktop application build workflow configuration to include necessary environment setup for macOS and Linux compilation phases.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 055cc6e and 43979d2.

📒 Files selected for processing (1)

• .github/workflows/build-desktop.yml

---

📝 Walkthrough

Walkthrough

A GitHub Actions workflow configuration adds the NEXT_PUBLIC_ELECTRIC_URL environment variable to the build-desktop workflow's macOS and Linux build steps, sourced from secrets, to enable its usage during the Electron app compilation phase.

Changes

Cohort / File(s)	Summary
GitHub Actions Workflow .github/workflows/build-desktop.yml	Added NEXT_PUBLIC_ELECTRIC_URL environment variable sourced from secrets to macOS and Linux build steps for the compile:app phase.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• ci: add NEXT_PUBLIC_ELECTRIC_URL to desktop build env #1460: Adds the identical NEXT_PUBLIC_ELECTRIC_URL secret to the same build-desktop GitHub Actions workflow environment for the Electron compilation step.

Poem

A secret to share with the build so bright, 🐰✨
NEXT_PUBLIC_ELECTRIC_URL takes flight,
From macOS to Linux, the workflow now gleams,
With sparkling electrons fulfilling our dreams! ⚡

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description provides a clear summary and test plan but omits required template sections like Related Issues, Type of Change, Testing details, and Additional Notes.	Fill out the PR description template completely by adding the missing sections: Related Issues, Type of Change checkbox, detailed Testing section, and any Additional Notes.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix: add NEXT_PUBLIC_ELECTRIC_URL to macOS desktop build' directly and specifically describes the main change in the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch saddlepaddle/fix-desktop-build

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cubic-dev-ai]

No issues found across 1 file

[github-actions]

🚀 Preview Deployment

🔗 Preview Links

Service	Status	Link
Database (Neon)	✅	View Branch
Electric (Fly.io)	✅	View App
API (Vercel)	✅	Open Preview
Web (Vercel)	✅	Open Preview
Marketing (Vercel)	✅	Open Preview
Admin (Vercel)	✅	Open Preview
Docs (Vercel)	✅	Open Preview

---

Preview updates automatically with new commits