Skip to content

fix(deploy): switch to lucky-api domain and stabilize smoke check - #127

Merged
LucasSantana-Dev merged 1 commit into
mainfrom
fix/lucky-api-domain
Mar 10, 2026
Merged

LucasSantana-Dev merged 1 commit into
mainfrom
fix/lucky-api-domain

Conversation

@LucasSantana-Dev

@LucasSantana-Dev LucasSantana-Dev commented Mar 10, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • switch hosted frontend API fallback from api.lucassantana.tech to lucky-api.lucassantana.tech
  • fix deploy workflow auth smoke script syntax and add fallback from /api/health/auth-config to /api/health
  • document lucky-api domain default in README and changelog

Infra updates applied

  • Cloudflare DNS route: lucky-api.lucassantana.tech -> nexus tunnel
  • Tunnel ingress now includes lucky-api.lucassantana.tech
  • Homelab backend env WEBAPP_REDIRECT_URI updated to https://lucky-api.lucassantana.tech/api/auth/callback
  • GitHub secret DEPLOY_WEBHOOK_URL updated to https://lucky-api.lucassantana.tech/webhook/deploy

Validation

  • npm run type:check --workspace=packages/frontend
  • workflow YAML parse via python3 + yaml.safe_load

Summary by CodeRabbit

Release Notes

  • Bug Fixes

    • Improved deployment reliability: health checks now automatically fall back to a generic endpoint if the primary endpoint is unavailable.
  • Chores

    • Updated API endpoint configuration to use lucky-api.lucassantana.tech for lucassantana.tech deployments.

@vercel

vercel Bot commented Mar 10, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
lucky Ready Ready Preview, Comment Mar 10, 2026 2:04am

@netlify

netlify Bot commented Mar 10, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for regal-bunny-0c8efe ready!

Name Link
🔨 Latest commit 1dfd050
🔍 Latest deploy log https://app.netlify.com/projects/regal-bunny-0c8efe/deploys/69af7bf3578fdb0008239241
😎 Deploy Preview https://deploy-preview-127--regal-bunny-0c8efe.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Mar 10, 2026 •

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The PR updates API endpoint configuration from api.lucassantana.tech to lucky-api.lucassantana.tech, adds a fallback health check mechanism in the deploy workflow (switching from auth-config to generic health endpoint on 404), and adjusts validation logic to narrow JSON checks to the auth-config path only.

Changes

Cohort / File(s) Summary
Deployment & Health Checks
.github/workflows/deploy.yml
Introduces fallback mechanism to switch from /api/health/auth-config to /api/health endpoint on 404 response; narrows Node-based JSON validation to auth-config path only; updates health check logging and error messaging.
Documentation
CHANGELOG.md, README.md
Updates API base URL from api.lucassantana.tech to lucky-api.lucassantana.tech in deployment examples and changelog entries.
Frontend API Configuration
packages/frontend/src/services/api.ts
Updates API base URL resolution for lucassantana.tech hosts from api.lucassantana.tech to lucky-api.lucassantana.tech.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested labels

size/m

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: switching to lucky-api domain and stabilizing the deploy smoke check with fallback mechanisms.
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 docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/lucky-api-domain

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

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

@sonarqubecloud

Copy link
Copy Markdown

@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

Size Change: +12 B (0%)

Total Size: 291 kB

Filename Size Change
packages/frontend/dist/assets/AutoMessages-BGYtjSoJ.js 1.56 kB +1.56 kB (new file) 🆕
packages/frontend/dist/assets/AutoMessages-CqOCNKTm.js 0 B -1.56 kB (removed) 🏆
packages/frontend/dist/assets/AutoMod-FJBRLEVI.js 2.73 kB +2.73 kB (new file) 🆕
packages/frontend/dist/assets/AutoMod-uJN0qDmh.js 0 B -2.73 kB (removed) 🏆
packages/frontend/dist/assets/badge-B4cZUzaj.js 453 B +453 B (new file) 🆕
packages/frontend/dist/assets/badge-D46fMEdz.js 0 B -453 B (removed) 🏆
packages/frontend/dist/assets/Card-BrIFAMmT.js 379 B +379 B (new file) 🆕
packages/frontend/dist/assets/Card-D7zYyaUG.js 0 B -380 B (removed) 🏆
packages/frontend/dist/assets/CommandsConfig-BoDMkgFt.js 0 B -1.43 kB (removed) 🏆
packages/frontend/dist/assets/CommandsConfig-LUbwGfjY.js 1.43 kB +1.43 kB (new file) 🆕
packages/frontend/dist/assets/Config-B6SeqPtf.js 1.61 kB +1.61 kB (new file) 🆕
packages/frontend/dist/assets/Config-hQmg1bhs.js 0 B -1.61 kB (removed) 🏆
packages/frontend/dist/assets/CustomCommands-BPsSHkLh.js 0 B -2.1 kB (removed) 🏆
packages/frontend/dist/assets/CustomCommands-qKtmfnQF.js 2.1 kB +2.1 kB (new file) 🆕
packages/frontend/dist/assets/DashboardOverview-DDdvkSC1.js 0 B -2.93 kB (removed) 🏆
packages/frontend/dist/assets/DashboardOverview-DYXUKauZ.js 2.93 kB +2.93 kB (new file) 🆕
packages/frontend/dist/assets/Features-C3Yv8GkR.js 0 B -2.39 kB (removed) 🏆
packages/frontend/dist/assets/Features-DYlv53jk.js 2.39 kB +2.39 kB (new file) 🆕
packages/frontend/dist/assets/index-BODCqMii.js 0 B -67 kB (removed) 🏆
packages/frontend/dist/assets/index-D9yyAzoM.js 67 kB +67 kB (new file) 🆕
packages/frontend/dist/assets/input-jQCPYx24.js 429 B +429 B (new file) 🆕
packages/frontend/dist/assets/input-slCE9CUf.js 0 B -425 B (removed) 🏆
packages/frontend/dist/assets/label-Bjoyr5_B.js 0 B -441 B (removed) 🏆
packages/frontend/dist/assets/label-CbBuRbzV.js 441 B +441 B (new file) 🆕
packages/frontend/dist/assets/LastFm-Cg7eFcVi.js 1.52 kB +1.52 kB (new file) 🆕
packages/frontend/dist/assets/LastFm-CQZKvZkv.js 0 B -1.52 kB (removed) 🏆
packages/frontend/dist/assets/Login-BsPSfPB3.js 0 B -2.23 kB (removed) 🏆
packages/frontend/dist/assets/Login-C8ApiiBt.js 2.22 kB +2.22 kB (new file) 🆕
packages/frontend/dist/assets/Lyrics-B8dOl9x3.js 1.3 kB +1.3 kB (new file) 🆕
packages/frontend/dist/assets/Lyrics-BbSaHn9N.js 0 B -1.3 kB (removed) 🏆
packages/frontend/dist/assets/Moderation-CGwW3Sq8.js 0 B -3.76 kB (removed) 🏆
packages/frontend/dist/assets/Moderation-IV3RA0ph.js 3.75 kB +3.75 kB (new file) 🆕
packages/frontend/dist/assets/ModerationConfig-CmkdWicp.js 0 B -1.93 kB (removed) 🏆
packages/frontend/dist/assets/ModerationConfig-DGmc9jMg.js 1.93 kB +1.93 kB (new file) 🆕
packages/frontend/dist/assets/Music-CmKLLdI2.js 0 B -6.08 kB (removed) 🏆
packages/frontend/dist/assets/Music-Cn10Yn1W.js 6.08 kB +6.08 kB (new file) 🆕
packages/frontend/dist/assets/MusicConfig-CFVdLd0b.js 0 B -1.62 kB (removed) 🏆
packages/frontend/dist/assets/MusicConfig-IddHclvI.js 1.62 kB +1.62 kB (new file) 🆕
packages/frontend/dist/assets/select-BErbxV_4.js 0 B -1.19 kB (removed) 🏆
packages/frontend/dist/assets/select-CvrCF0jT.js 1.19 kB +1.19 kB (new file) 🆕
packages/frontend/dist/assets/ServerLogs-dnMNBxPb.js 2.82 kB +2.82 kB (new file) 🆕
packages/frontend/dist/assets/ServerLogs-DU9R9nbG.js 0 B -2.82 kB (removed) 🏆
packages/frontend/dist/assets/ServerSettings-BP2wlnNi.js 0 B -1.89 kB (removed) 🏆
packages/frontend/dist/assets/ServerSettings-Bx22dsba.js 1.9 kB +1.9 kB (new file) 🆕
packages/frontend/dist/assets/ServersPage-AmZDWDFs.js 2.4 kB +2.4 kB (new file) 🆕
packages/frontend/dist/assets/ServersPage-Dd9z2FLD.js 0 B -2.4 kB (removed) 🏆
packages/frontend/dist/assets/Skeleton-CXKoTiGM.js 227 B +227 B (new file) 🆕
packages/frontend/dist/assets/Skeleton-Dr1DLTq3.js 0 B -228 B (removed) 🏆
packages/frontend/dist/assets/switch-B7_jZ8VB.js 0 B -516 B (removed) 🏆
packages/frontend/dist/assets/switch-Dj-GgJeq.js 515 B +515 B (new file) 🆕
packages/frontend/dist/assets/TrackHistory-gdm0ralT.js 0 B -1.84 kB (removed) 🏆
packages/frontend/dist/assets/TrackHistory-sZ6gthg8.js 1.84 kB +1.84 kB (new file) 🆕
packages/frontend/dist/assets/TwitchNotifications-B3uB9o6z.js 1.57 kB +1.57 kB (new file) 🆕
packages/frontend/dist/assets/TwitchNotifications-BkL7smhu.js 0 B -1.57 kB (removed) 🏆
packages/frontend/dist/assets/useGuildSelection-CJsIjWbs.js 295 B +295 B (new file) 🆕
packages/frontend/dist/assets/useGuildSelection-CxGHb-El.js 0 B -293 B (removed) 🏆
ℹ️ View Unchanged
Filename Size
packages/frontend/dist/assets/index-DLqvhx7-.css 13 kB
packages/frontend/dist/assets/usePageMetadata-Dudc5JrX.js 287 B
packages/frontend/dist/assets/vendor-forms-C1Gy4syj.js 21.1 kB
packages/frontend/dist/assets/vendor-radix-BTf9AecN.js 33.5 kB
packages/frontend/dist/assets/vendor-react-Bu1Oeml_.js 17.3 kB
packages/frontend/dist/assets/vendor-state-Szs8I3iE.js 25 kB
packages/frontend/dist/assets/vendor-ui-DSWwm_-g.js 64.2 kB
packages/frontend/dist/assets/zod-CLxBCFgw.js 1.58 kB

compressed-size-action

This branch was successfully deployed

1 active deployment
Preview — 1dfd0508 Deployed Mar 10, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant