Skip to content

chore: Remove arcjet#141

Merged
arjunkomath merged 1 commit intodevelopfrom
chore/remove-arcjet
Feb 20, 2026
Merged

chore: Remove arcjet#141
arjunkomath merged 1 commit intodevelopfrom
chore/remove-arcjet

Conversation

@arjunkomath
Copy link
Copy Markdown
Member

@arjunkomath arjunkomath commented Feb 20, 2026

Migrate to Vercel firewall

Summary by CodeRabbit

Release Notes

  • Chores
    • Removed external service dependency from project packages and updated configuration accordingly
    • Removed rate limiting protection that was previously enforced across multiple API endpoints including billing, email, pages, domains, and team management operations
    • Removed anti-abuse validation checks from user signup and form submission flows
    • Updated environment configuration files and removed related documentation references

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 20, 2026

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

Project Deployment Actions Updated (UTC)
changes-page Ready Ready Preview Feb 20, 2026 8:56am
changes-page-docs Ready Ready Preview Feb 20, 2026 8:56am
user-changes-page Ready Ready Preview Feb 20, 2026 8:56am

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 20, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

This pull request removes all Arcjet integration from the codebase, including the dependency from both apps/page and apps/web, environment variable entries, documentation references, and all protection/rate-limiting logic from multiple API endpoints.

Changes

Cohort / File(s) Summary
Configuration & Documentation
README.md, apps/page/.env.example, apps/web/.env.example
Removed Arcjet references from documentation and example environment files.
Package Dependencies
apps/page/package.json, apps/web/package.json
Removed @arcjet/next@1.0.0-alpha.20 dependency from both applications.
Page App API Routes
apps/page/pages/api/auth/request-magic-link.ts, apps/page/pages/api/notifications/subscribe-email.ts, apps/page/pages/api/roadmap/submit-triage.ts
Eliminated Arcjet-based anti-abuse, email protection, and rate-limiting checks from signup, subscription, and triage submission endpoints.
Web App Billing APIs
apps/web/pages/api/billing/create-billing-portal.ts, apps/web/pages/api/billing/redirect-to-checkout.ts
Removed rate-limiter invocations from billing portal and checkout redirect endpoints.
Web App Page Management APIs
apps/web/pages/api/pages/new.ts, apps/web/pages/api/pages/settings/add-domain.ts, apps/web/pages/api/pages/settings/check-domain.ts, apps/web/pages/api/pages/validate-url.ts
Removed rate-limiting checks from page creation, domain management, and URL validation endpoints.
Web App Content & Collaboration APIs
apps/web/pages/api/posts/index.ts, apps/web/pages/api/emails/subscribers/export-csv.ts, apps/web/pages/api/teams/invite/index.ts, apps/web/pages/api/teams/invite/accept/index.ts, apps/web/pages/api/teams/member/[id]/index.ts
Removed rate-limiting protection from post creation, subscriber export, team invitation, and member management endpoints.
Rate Limiting Utility
apps/web/utils/rate-limit.ts
Deleted the IP-based sliding-window rate limiter utility that implemented Arcjet protection with HTTP 429 responses.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Poem

🐰 Arcjet's armor, once so strong,

Now cast aside—we move along!

No more walls to slow our pace,

Just clean code in its rightful place.

Simpler paths for buns to race! 🌟

🚥 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 and concisely describes the main change: removing Arcjet integration across multiple files and packages in the codebase.
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 chore/remove-arcjet

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.

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.

1 participant