BeforeMerge

AI-native code review knowledge base. Structured rules that catch what linters miss.

Product

  • Features
  • Explore
  • Pricing
  • Docs
  • GitHub

Company

  • About
  • Blog
  • Contributors
  • Contributing

Legal

  • Privacy Policy
  • Terms of Service
  • MIT License

© 2026 BeforeMerge. Built by Peter Krzyzek

BeforeMerge
Features
Explore
PricingBlogContributorsInstall Guide
2Sign In
FeaturesExplore
RulesSkillsKnowledgePrompts
PricingBlogContributorsInstall Guide
Sign In

Explore

Browse 158 rules, 25 knowledge articles, and 25 prompt templates across security, performance, architecture, and quality.

158 rules

Close RLS Policy Gaps for All Operations

HIGH

Tables with RLS enabled but missing policies for certain operations silently deny access. Ensure every table has policies for SELECT, INSERT, UPDATE, and DELETE. [CWE-862 · A01:2021]

supabaserls
beforemerge-supabase-review

Enable RLS on Every Table with Complete Policies

CRITICAL

Every table must have Row Level Security enabled with at least one policy per operation. Tables without RLS are accessible to any authenticated user. [CWE-862 · A01:2021]

supabaserls
beforemerge-supabase-review

Use Migration Files Instead of MCP or Dashboard SQL

HIGH

Schema changes via Supabase MCP, SQL editor, or dashboard don't create migration files. This causes schema drift between environments.

supabaseDatabase
beforemerge-supabase-review

Use getUser() Instead of getSession() for Auth Checks

CRITICAL

getSession() reads from cookies and can be spoofed. getUser() verifies the token with the Supabase Auth server, making it tamper-proof. [CWE-287 · A07:2021]

sessionsupabase
beforemerge-supabase-review

Never Expose Service Role Key in Client-Side Code

CRITICAL

Using NEXT_PUBLIC_ prefix on SUPABASE_SERVICE_ROLE_KEY or DATABASE_URL embeds secrets into client-side JavaScript bundles, bypassing all RLS. [CWE-798 · A07:2021]

nextjssupabase
beforemerge-supabase-review

Validate Input at Runtime with Zod Instead of Type Assertions

HIGH

Using 'as' type assertions on external input (params, form data, request bodies) provides zero runtime safety. Use Zod for runtime validation. [CWE-20 · A03:2021]

supabaseValidation
beforemerge-supabase-review

Always Check Error Before Using Data from Supabase Queries

HIGH

Destructuring { data } without checking { error } from Supabase queries ignores failures silently. When error is non-null, data is always null. [CWE-252]

supabasequality
beforemerge-supabase-review

Distinguish Not-Found from Other Supabase Errors

MEDIUM

Treating all Supabase errors the same (if error, throw) hides whether a record is missing or the query itself failed. Check error codes for proper handling.

postgrestsupabase
beforemerge-supabase-review

Avoid select('*') — Request Only the Columns You Need

HIGH

Using .select('*') fetches all columns including large text/json fields, wastes bandwidth, leaks data shape, and prevents index-only scans.

Queriessupabase
beforemerge-supabase-review

Avoid N+1 Queries with Supabase Relational Selects

HIGH

Fetching parent records then looping to fetch children creates N+1 HTTP requests. Use Supabase nested .select('*, children(*)') to resolve in a single query. [CWE-400]

Queriessupabase
beforemerge-supabase-review

Add Indexes for Filtered and Ordered Columns

HIGH

Filtering or ordering on unindexed columns causes full table scans. RLS policy columns like user_id and org_id especially need indexes. [CWE-405]

Queriessupabase
beforemerge-supabase-review

Use Cursor-Based Pagination Instead of Offset Pagination

HIGH

Using .range() offset pagination for large datasets forces PostgreSQL to scan all skipped rows. Use cursor-based pagination with .gt()/.lt() for constant-time page fetches.

cursorQueries
beforemerge-supabase-review

Use Connection Pooling (Supavisor) for Serverless Deployments

HIGH

Each serverless invocation opening a direct database connection exhausts PostgreSQL's connection limit. Use Supavisor pooler URLs for all serverless environments.

supabasesupavisor
beforemerge-supabase-review

Use Batch Operations Instead of Single-Row Loops

HIGH

Inserting or updating rows one at a time in a loop creates N HTTP requests. Use .insert([...]) or .upsert([...]) to batch into a single request.

supabasebulk
beforemerge-supabase-review

Generate Database Types from Schema

MEDIUM

Hand-writing TypeScript interfaces for database tables leads to drift between code and schema. Use supabase gen types to generate types automatically.

supabasetypes
beforemerge-supabase-review

Follow Canonical Migration File Structure

MEDIUM

Migration files without proper structure (table, indexes, RLS, policies, comments) are harder to review and prone to missing critical steps like RLS.

supabaseDatabase
beforemerge-supabase-review

Use the Correct Supabase Client for Each Context

HIGH

Using the wrong Supabase client for the context breaks RLS, leaks auth state, or causes hydration errors. Match client type to Next.js rendering context.

middlewarenextjs
beforemerge-supabase-review

Prevent Prototype Pollution from Untrusted Input

CRITICAL

Spreading or Object.assign-ing untrusted user input into objects can pollute Object.prototype and lead to security bypasses. [CWE-1321]

ReactPrototype Pollution
beforemerge-react-review

Use Cryptographic Randomness for Tokens and IDs

CRITICAL

Math.random() is not cryptographically secure. Use crypto.randomUUID() or crypto.getRandomValues() for tokens, IDs, and security-sensitive values. [CWE-338]

Reactrandomness
beforemerge-react-review

Never Use eval() or new Function() with User Input

CRITICAL

Using eval(), new Function(), or innerHTML with user-controlled strings enables arbitrary code execution in the browser. [CWE-95 · A03:2021]

code-injectionReact
beforemerge-react-review

Sanitize Content Before dangerouslySetInnerHTML

CRITICAL

Using dangerouslySetInnerHTML with unsanitized user input enables XSS attacks. Always sanitize with DOMPurify or a trusted library. [CWE-79 · A03:2021]

SanitizationXSS
beforemerge-react-review

Always Clean Up useEffect Side Effects

MEDIUM

Missing cleanup in useEffect for subscriptions, timers, event listeners, and AbortControllers causes memory leaks, stale callbacks, and state updates on unmounted components.

useeffectReact
beforemerge-react-review

Never Use Array Index as Key for Dynamic Lists

MEDIUM

Using array index as key in lists that can be reordered, filtered, or inserted into causes React to mismap state to the wrong items, creating subtle and hard-to-debug UI bugs.

key-propreconciliation
beforemerge-react-review

Add Error Boundaries Around Unreliable UI Sections

MEDIUM

Without error boundaries, a single component crash unmounts the entire React tree. Wrap unreliable sections so failures are isolated and recoverable.

reliabilityReact
beforemerge-react-review

Automate these checks on every PR

BeforeMerge scans your pull requests against these rules automatically. Get actionable feedback before code ships to production.

Join WaitlistLearn More
Sort:
Previous1234567Next