fix: replace hardcoded localhost with production domain in invitation…

[earzalien]

… link

---

Summary by cubic

Use the CLIENT_URL environment variable to build trip invitation links instead of the hardcoded http://localhost:3000, fixing broken invites in production. If CLIENT_URL is missing, the API now throws a clear error to prevent generating invalid links.

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

[cubic-dev-ai]

1 issue found across 1 file

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="server/src/modules/invitation/invitationActions.ts">

<violation number="1" location="server/src/modules/invitation/invitationActions.ts:105">
P2: Avoid hardcoding the frontend domain in code; build invitation links from configuration so deployments across environments don’t require code changes.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="server/src/modules/invitation/invitationActions.ts">

<violation number="1" location="server/src/modules/invitation/invitationActions.ts:105">
P2: Guard against missing CLIENT_URL; otherwise invitations can return `undefined/...` URLs when the env var is not configured.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}