feat(contact): add contact page, migrate help/demo forms to useMutation

[waleedlatif1]

Summary

• Add /contact page with rate-limited form that mirrors the in-product help ticket flow (forwards to help@, sends confirmation email via renderHelpConfirmationEmail)
• Link /contact from the landing footer; typed landing_contact_submitted PostHog event
• Migrate help modal and demo-request modal from raw fetch + useState to useMutation
• Remove stale z-index MutationObserver + setInterval(100ms) hack from help modal — emcn --z-popover (300) already renders above --z-modal (200)
• Fix help modal's image preview URL lifecycle (no longer revokes still-visible URLs on every upload)
• Guard all three forms against double-submit while a request is in flight
• Strip ~14 unnecessary useCallback wrappers from the two modals

Type of Change

• New feature

Testing

Tested manually. Typecheck and biome clean.

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 21, 2026 6:57pm

[cursor]

PR Summary

Medium Risk
Adds a new public POST /api/contact endpoint that sends emails and applies IP-based rate limiting; misconfiguration or validation mistakes could impact deliverability or spam exposure. Remaining changes are mostly client-side refactors to submission state management.

Overview
Adds a new landing /contact page with a validated contact form and direct-contact mailto links, and exposes a new POST /api/contact route that rate-limits by IP, forwards submissions to help@…, and attempts to send a confirmation email.

Refactors the landing demo-request modal and the in-product help modal to use useMutation (including double-submit guards and consistent error handling), extracts a shared LandingField component with improved accessibility wiring, and updates help image preview lifecycle/z-index handling. Also adds a footer link to /contact and introduces a typed PostHog event landing_contact_submitted.

^{Reviewed by Cursor Bugbot for commit 2e8ff3c. Configure here.}

[greptile-apps]

Greptile Summary

This PR adds a /contact landing page with a rate-limited form (mirroring the help-ticket flow), links it from the footer, migrates the help and demo-request modals from raw fetch + useState to useMutation, removes a large MutationObserver/setInterval z-index hack, fixes image-preview URL lifecycle in the help modal, and extracts LandingField into a shared component with proper accessibility (aria-describedby injection via cloneElement). The implementation is clean; previous reviewer concerns about schema exposure, confirmation-email type mapping, and ARIA wiring are all addressed in this iteration.

Confidence Score: 5/5

Safe to merge — all previously raised concerns are resolved and no new P0/P1 issues found.

All three P1/P2 findings from the previous review round (schema exposure on 400, confirmation email type mapping, aria-describedby wiring) are addressed. The useMutation migration is correct across all three forms, helpMutation.reset() is now called on modal open, image URL cleanup is correctly ordered via imagesRef, and the rate limiter is appropriately configured for a public endpoint. No remaining blocking issues.

No files require special attention.

Important Files Changed

Filename	Overview
apps/sim/app/api/contact/route.ts	New public POST endpoint with token-bucket rate limiting, Zod validation, topic-mapped confirmation email, and structured error handling; previously raised issues (schema leakage, wrong email type) are resolved.
apps/sim/app/(landing)/components/contact/contact-form.tsx	New contact form using useMutation with client-side Zod validation, double-submit guard, and accessible LandingField wrappers; logic is sound.
apps/sim/app/(landing)/components/forms/landing-field.tsx	New shared form-field wrapper that injects aria-describedby and aria-invalid via cloneElement when an error is present; cleanly addresses the previous ARIA concern.
apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/help-modal/help-modal.tsx	Migrated to useMutation, removed the MutationObserver/setInterval z-index hack, fixed image-URL revoke lifecycle (imagesRef pattern), helpMutation.reset() correctly called on modal open.
apps/sim/app/(landing)/components/demo-request/demo-request-modal.tsx	Migrated from raw fetch/useState to useMutation, unnecessary useCallback wrappers removed, shared LandingField imported, demoMutation.reset() called in resetForm.
apps/sim/app/(landing)/components/contact/consts.ts	New config file with Zod schema, topic enum, topic-to-help-type mapper, and label helper; properly separated from rendering code per project conventions.
apps/sim/app/(landing)/contact/page.tsx	New contact page with metadata, direct-contact cards, and ContactForm; footer conditionally rendered for hosted env.
apps/sim/app/(landing)/components/footer/footer.tsx	Adds Contact link to RESOURCES_LINKS; trivial change.
apps/sim/lib/posthog/events.ts	Adds landing_contact_submitted event type with topic property; consistent with existing event map pattern.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User fills /contact form] --> B{Client-side Zod validation}
    B -- invalid --> C[Show field errors]
    B -- valid --> D[contactMutation.mutate]
    D --> E[POST /api/contact]
    E --> F{Rate limit check\n10 tokens, 5/min refill}
    F -- exceeded --> G[429 Too Many Requests]
    F -- allowed --> H{Server Zod validation}
    H -- invalid --> I[400 Invalid request data]
    H -- valid --> J[sendEmail to help@domain\nsubject: CONTACT:TOPIC subject]
    J -- failure --> K[500 Failed to send]
    J -- success --> L[sendEmail confirmation\nto submitter]
    L --> M[201 success response]
    M --> N[onSuccess: PostHog event\nlanding_contact_submitted]
    N --> O[Show success screen]
    L -. fire & forget\nerrors are logged .-> M

Loading

_{Reviews (2): Last reviewed commit: "improvement(landing): extract shared Lan..." | Re-trigger Greptile}

[waleedlatif1]

@greptile

[waleedlatif1]

@cursor review

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit 2e8ff3c. Configure here.}