Skip to content

Staging → Main: groups rename, security fixes, proxy migration - #38

Merged
holkexyz merged 2 commits into
mainfrom
staging
Apr 8, 2026
Merged

Staging → Main: groups rename, security fixes, proxy migration#38
holkexyz merged 2 commits into
mainfrom
staging

Conversation

@holkexyz

@holkexyz holkexyz commented Apr 8, 2026

Copy link
Copy Markdown
Member

Summary

  • Rename organizations → groups throughout the codebase (routes, components, lib, API)
  • Security & a11y fixes from multiple CodeRabbit review rounds (safeRedirect, CSRF, session state, type guards, sanitization)
  • Membership records — use createRecord with TID rkeys instead of derived keys
  • SEO/GEO infrastructure — sitemap, robots.ts, manifest, OG images, llms.txt, structured data
  • Landing page refactor — move to /welcome with proxy-based redirect, SSR, focus traps
  • Next.js 16 migration — rename middleware.tsproxy.ts to fix deprecation warning

Test plan

  • Verify / redirects to /welcome when no session cookie present
  • Verify group creation, editing, and membership flows work end-to-end
  • Verify sign-in / sign-out cycle works correctly
  • Verify mobile layout (sidebar, bottom sheet, sign-out button)
  • Verify SEO routes: /sitemap.xml, /robots.txt, /manifest.webmanifest
  • Spot-check no console deprecation warnings from Next.js

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Refactor
    • Internal request handling mechanism has been reorganized to improve code structure and maintainability. All functionality and user experience remain unchanged.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
certified-app Ready Ready Preview, Comment Apr 8, 2026 7:24am

Request Review

@coderabbitai

coderabbitai Bot commented Apr 8, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4dba8128-af80-42ce-925b-3b882e5b3972

📥 Commits

Reviewing files that changed from the base of the PR and between 292a399 and abd89d2.

📒 Files selected for processing (1)
  • src/proxy.ts

📝 Walkthrough

Walkthrough

The exported function in src/proxy.ts has been renamed from middleware to proxy. The function signature and control flow remain identical, only the export name has changed.

Changes

Cohort / File(s) Summary
Function Export Rename
src/proxy.ts
Renamed exported function from middleware(request: NextRequest) to proxy(request: NextRequest), maintaining identical signature and implementation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 From middleware to proxy we rename,
A hop, a skip, a simpler frame!
No logic bent, just clearer still,
One little change shows clever will! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title mentions 'proxy migration' which relates to the file renaming (middleware.ts → proxy.ts) seen in the changeset, but this is a minor change compared to other substantial changes like groups rename and security fixes.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch staging

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.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@holkexyz
holkexyz merged commit 3551f17 into main Apr 8, 2026
3 checks passed
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