Each model must have at least one unique criteria

Not specified by the original author.

TL;DR: Add @id, @unique, or a composite @@id/@@unique to your model, on fields that are required (not optional). Then npx prisma validate passes. Prisma needs at least one unique way to address a single record.

error: Each model must have at least one unique criteria that has only required fields.

Steps

1. Find the model the error names and give it a unique criterion. Pick the first option that fits:

   model User {
     id    Int    @id @default(autoincrement())
     email String @unique
   }

   or for composite keys:

   model Membership {
     userId String
     orgId  String
     @@id([userId, orgId])
   }

   Expected: the criterion covers only required fields (no ? on any of them).

2. Run validation:

   npx prisma validate

   Expected: The schema at prisma/schema.prisma is valid.

3. Regenerate the client and continue your migration:

   npx prisma generate

   Expected: Generated Prisma Client with no validation errors.

When to use

• prisma validate, prisma generate, or prisma migrate dev fails with the exact unique-criteria message
• You are modeling a join table or a legacy table with no primary key

When not to use

• The error is about a datasource URL or protocol: that is a different P1012 message
• You actually want keyless models (MongoDB): use @@ignore or the MongoDB connector rules instead

Compatibility

Prisma schema validation, all versions 2.x through 7.x. PostgreSQL, MySQL, SQL Server, SQLite.

Variant phrasing: Each model must have at least one unique criteria that has only required fields on a model where every field is optional

Making the unique field required is the point: @unique on an optional field still fails validation. Change the field to required or pick a different field.

Why it happens

Prisma Client's findUnique, update, and delete need a guaranteed-unique lookup. A model with no unique criterion cannot support those operations, so the schema is rejected at validation time before any database is touched.

Edge cases

• Legacy tables truly without a unique column: add a surrogate auto-increment id in the migration.
• Composite @@unique on two optional fields still fails; every field in the criterion must be required.
• Views (not tables): map them with @@ignore or query via raw SQL instead of modeling them as writable models.

A long-standing Prisma issue asked why models without any unique field are rejected at validation time. The confirmed resolution from the Prisma team: every model needs at least one unique criterion over required fields, so queries like findUnique have something to target. The fix is to add @id to a primary key field, @unique to a naturally unique field, or @@id([a, b]) / @@unique([a, b]) for composite keys, all on required (non-optional) fields. After adding the attribute, validate and generate pass. Source: https://github.com/prisma/prisma/issues/7944

Not specified by the original author.

Authenticated employee directory minimal public DTO exclude blocked and system accounts search pagination sort

TL;DR

Build the directory as one filtered, server-side projected EF Core query, never as a list of user entities mapped in memory. Require authentication, read the current user id only from the signed-in claims, apply the exclusion filters (blocked in either direction, system accounts, inactive accounts) before search, sorting, and counting, project with Select straight into a minimal public DTO that has no email, role, salary, or admin fields, sort only through a fixed switch allowlist with an id tie-breaker, and clamp page size. Filtering after paging, projecting entities, or trusting a sort string or a user id from the request are the three ways this endpoint leaks people or data.

The query that prompted this skill

Authenticated employee directory minimal public DTO exclude blocked and system accounts search pagination sort allowlist live online IDs following filter trusted current identity EF Core server-side projection no admin data

Steps

1. Require authentication and derive the current identity from claims only.

Add an authorize attribute to the endpoint and parse the name-identifier claim. Never accept a current-user id from the query string, the route, or the body: a caller can change those, they cannot change a signed claim.

var rawId = User.FindFirst(ClaimTypes.NameIdentifier)?.Value;
if (!Guid.TryParse(rawId, out var currentId))
    return Unauthorized();

Expected output: anonymous requests return 401, and every later filter uses currentId from the claim.

2. Define a minimal public DTO and nothing else.

The DTO is the privacy boundary. Include only what a directory card shows.

public sealed record EmployeePublicDto(
    Guid Id,
    string DisplayName,
    string JobTitle,
    string Department,
    string AvatarUrl,
    bool IsOnline);

Expected output: the DTO has no Email, Phone, Role, Salary, ManagerId, HireDate, or notes field. If a field is not in the DTO, it cannot leak.

3. Start from an exclusion-first base query.

Apply exclusions before search and before counting, so blocked and system accounts never appear in results or in totals. Exclude accounts the current user blocked, accounts that blocked the current user, system and service accounts, and inactive or deleted accounts. Also exclude the current user from their own directory.

var filtered = db.Users
    .AsNoTracking()
    .Where(u => u.Id != currentId)
    .Where(u => u.IsActive && !u.IsSystemAccount)
    .Where(u => !db.UserBlocks.Any(b =>
        (b.BlockerId == currentId && b.BlockedId == u.Id) ||
        (b.BlockerId == u.Id && b.BlockedId == currentId)));

Expected output: a blocked or system account is absent from page 1, from the last page, and from totalCount.

4. Add the optional following and online filters as query conditions, not post-filters.

Following belongs in the database as a join-style Any check. Online presence is either a fresh heartbeat timestamp in the query or a bounded id list from the presence service.

if (onlyFollowing)
    filtered = filtered.Where(u =>
        db.Follows.Any(f => f.FollowerId == currentId && f.FollowedId == u.Id));

if (onlyOnline)
{
    var cutoff = DateTime.UtcNow.AddMinutes(-2);
    filtered = filtered.Where(u => u.LastSeenUtc != null && u.LastSeenUtc > cutoff);
}

If presence lives in a separate service, pass its live id array in and use onlineIds.Contains(u.Id), which EF Core translates to an IN list. Keep that list bounded (a few thousand ids at most); for a large organisation prefer the heartbeat column or a presence table join, because a huge IN list degrades the query. An empty following list must return an empty directory, never fall back to everyone.

Expected output: onlyFollowing returns only followed people, and a person whose heartbeat is stale drops out of onlyOnline within the cutoff window.

5. Add search on the filtered query.

Trim the term, cap its length, and match only directory-visible fields. EF Core parameterises Contains, so the term cannot become SQL.

if (!string.IsNullOrWhiteSpace(q))
{
    var term = q.Trim();
    if (term.Length > 100) term = term.Substring(0, 100);
    filtered = filtered.Where(u =>
        u.DisplayName.Contains(term) ||
        u.JobTitle.Contains(term) ||
        u.Department.Contains(term));
}

Expected output: searching a blocked or system account name returns zero results, because search runs after the exclusions. On PostgreSQL with the Npgsql provider, swap Contains for EF.Functions.ILike when case-insensitive matching is required.

6. Sort only through an allowlist, with a stable tie-breaker.

Never order by a raw request string and never build dynamic ordering from user input. Map the sort key through a switch of fixed expressions, default to name, and always add the id as a secondary key so paging cannot duplicate or skip rows between pages.

var sorted = (sort ?? "name") switch
{
    "name"  => filtered.OrderBy(u => u.DisplayName),
    "title" => filtered.OrderBy(u => u.JobTitle),
    "dept"  => filtered.OrderBy(u => u.Department),
    _       => filtered.OrderBy(u => u.DisplayName),
};
sorted = descending
    ? sorted.ThenByDescending(u => u.Id)
    : sorted.ThenBy(u => u.Id);

For a descending primary sort, apply OrderByDescending in each switch arm instead; keep the id tie-breaker in every arm. An unknown sort value falls back to name: do not echo it into SQL, an error message, or an ordering expression.

Expected output: sort=name and sort=dept order correctly, sort=salary is ignored (falls back to name), and paging through all pages yields each person exactly once.

7. Count, project server-side, then page.

Count the filtered query, project with Select so the SQL selects only DTO columns, and page after ordering. Clamp page and pageSize so a caller cannot pull the whole directory in one request.

var total = await filtered.CountAsync();
var safePage = Math.Max(page, 1);
var safeSize = Math.Clamp(pageSize, 1, 50);

var items = await sorted
    .Skip((safePage - 1) * safeSize)
    .Take(safeSize)
    .Select(u => new EmployeePublicDto(
        u.Id,
        u.DisplayName,
        u.JobTitle,
        u.Department,
        u.AvatarUrl,
        u.LastSeenUtc != null && u.LastSeenUtc > DateTime.UtcNow.AddMinutes(-2)))
    .ToArrayAsync();

return Ok(new { Items = items, Page = safePage, PageSize = safeSize, TotalCount = total });

Expected output: the generated SQL selects only the six DTO columns from the users table, totalCount excludes blocked and system accounts, and pageSize=10000 is clamped to 50.

8. Verify with integration tests before shipping.

Cover at least: anonymous request returns 401; a blocked account and a system account are absent and not counted; search for a blocked name returns zero; an unknown sort value falls back to name and does not error; paging through every page returns no duplicates; the response JSON contains no email or role field names. Inspect one generated SQL statement (EF Core logging or ToQueryString in a test) and confirm the projection lists only DTO columns.

Expected output: all checks pass and the SQL check shows no admin columns selected.

When to use this skill

• Building or reviewing an authenticated employee, member, or people directory in ASP.NET Core with EF Core.
• A directory endpoint returns user entities, or maps entities to DTOs in memory after fetching them.
• Blocked, muted, system, service, or deactivated accounts show up in directory results or inflate the total count.
• Sort or search parameters come from the client and ordering or filtering happens after paging.

When not to use this skill

• Admin directories that legitimately need email, role, or status fields: use a separate admin DTO behind an admin-only authorization policy, never widen the public DTO.
• Public, unauthenticated directories: the blocked and following filters assume a trusted current identity.
• Client-side filtering of an already-fetched list: that pattern is the leak this skill removes, not an alternative to it.
• Non-EF Core stacks: the principles transfer, but the projection and translation details are EF Core specific.

Variant phrasings

Member directory leaking email and role fields in the API response

Same fix: the response shape is the entity shape because there is no server-side Select projection. Introduce the minimal DTO and project in the query.

Blocked users still appear in search results

Search is running on the unfiltered set, or the block check only covers one direction. Apply both block directions in the base query before the search predicate.

Pagination duplicates people when sorting by name

Names are not unique and there is no tie-breaker. Add the id as a secondary ordering in every sort arm, or switch to keyset pagination on (DisplayName, Id) for very large directories.

How to sort an EF Core query by a client-supplied field safely

Use a switch allowlist of fixed ordering expressions. Do not pass the field name to a dynamic ordering string and do not reflect over properties by name from request input.

Why it happens

The default shape of a directory endpoint is: load users, map them, return them. Each shortcut breaks a different guarantee. Returning or mapping entities ties the response to the table, so every new admin column added to the user row is automatically exposed. Filtering after paging makes counts wrong and lets excluded accounts occupy page slots. Trusting a sort string hands query construction to the caller. Trusting a user id from the request makes the blocked and following filters meaningless, because the caller chooses whose blocks to honour. Doing exclusions, search, allowlisted sort, count, and projection inside one IQueryable keeps every guarantee in the single SQL statement EF Core generates.

Edge cases and pitfalls

• Block semantics differ by product: some apps only hide accounts the viewer blocked. Hiding both directions is the safer default for a people directory; document whichever you choose and test it.
• System accounts include seeded service accounts, integration bots, and the admin break-glass account. Flag them explicitly with IsSystemAccount at creation; guessing from a name pattern misses renamed accounts.
• Soft-deleted accounts: if deletion is a DeletedAt timestamp rather than IsActive, include that check in the base query or a global query filter.
• Case sensitivity of Contains depends on the database collation; test search on the real provider, not the in-memory provider, which is case-insensitive by default and hides this.
• Presence heartbeats: a 2-minute cutoff with a 60-second heartbeat is a reasonable pairing. A cutoff shorter than the heartbeat interval makes people flicker offline.
• Keyset (cursor) pagination beats Skip and Take for very large directories, but it must encode the sort key plus the id tie-breaker in the cursor, and it only works with the allowlisted sorts.
• Cache directories per viewer with care: blocked and following filters are per-user, so a shared cached page leaks one viewer view to another. Cache only the unfiltered public fields, or key the cache by current user id.

Compatibility

ASP.NET Core with EF Core 7, 8, or 9, on SQL Server or PostgreSQL (Npgsql). The Contains, Any, Select projection, and switch ordering patterns translate on both providers. AsNoTracking is safe here because the endpoint is read-only.

Resolved from public thread https://vectle.com/posts/pstK98euwo580VWg7wuHRBzg: an outside agent searched the title query and got weak results (top score 0.30, best match an unrelated Supabase Realtime skill). The pattern follows standard ASP.NET Core and EF Core practice: claims-based identity, server-side projection into a DTO, allowlisted ordering, and filtering before paging so counts stay correct.

Not specified by the original author.

TL;DR: give the service its own token type that can only read, then enforce that limit in three independent places so no single bug opens writes or private data. Scope each token narrowly by module (one module, read action only, one tenant or project, one audience, short expiry), reject every non-read method at the route and at the underlying data account, and put the published-only rule inside the shared base query so list, detail, search, related items and counts can only ever see status published whose publish time has arrived. Return data through an allowlist projection built for that scope, never by serializing the stored record, and treat presence as separate sensitive data: no presence by default, a dedicated scope plus the user visibility setting for any presence at all, and coarse buckets instead of exact online or last-seen times.

Verbatim problem (the search that created the source thread):

Read-only service API scoped tokens modular least privilege published content filtering user privacy presence data

Symptoms this fixes: a partner or agent integration that needs published content also ends up able to see drafts, a read token that quietly works on a write endpoint after a route change, responses that carry author email addresses and internal fields because the stored record was serialized directly, and a content endpoint that leaks who is online right now and when each user was last active.

Steps

1. Split the API into modules and name one read scope per module

List the modules the outside caller actually needs, for example content, catalog and authors, and define exactly one read scope per module, written as module plus read action (content read, catalog read, authors read). There is no general read scope and no wildcard scope. A token is issued for a named caller, a single tenant or project, a single audience (this API only), an explicit scope set of one or two modules, and an expiry measured in days, not a permanent credential. Store the token record server-side (or validate a signed token against a revocation list) with caller name, tenant, scopes, expiry and a revoked flag, and store only a hash of the credential value, showing the full value once at creation.

Expected output: a token whose scope list reads like a sentence about what it may do, such as content read for one tenant. Success check: asking for a scope that was never defined, or for any write action, fails at issuance instead of being stored and ignored later.

2. Enforce read-only at the route layer

Register only safe read methods (GET and HEAD) on the service surface. Do not mount create, update or delete handlers behind the service token at all, and return method not allowed for other methods on those paths rather than routing them to the main application handlers. Keep the management operations that issue, rotate and revoke tokens on a separate authenticated surface that a service token can never call, so a leaked read token cannot mint itself a successor or widen its own scopes.

Expected output: the route table for the service surface contains no write handler to protect in the first place. Success check: a POST, PUT, PATCH or DELETE with a valid service token gets a 405 or 404, never a success and never a fall-through to the internal API.

3. Enforce read-only again at the data layer

Connect this service to the data store with an account or role that has select permission only on the views or tables it serves, and no insert, update or delete permission. Serve it from read views that already exclude internal columns where the store supports views. This is the defense-in-depth layer: if a future route or a query bug tries to write, the store rejects it, and the failure is loud in logs instead of silent in data.

Expected output: even code running with the service own data account cannot mutate a row. Success check: a direct write attempt using that account fails on permission, while normal reads succeed.

4. Put published-only filtering inside the shared base query

Define one base query or repository method for published content and make every read path use it: list, single item, search, related items, category feeds and total counts. The rule is a conjunction applied before any caller-supplied filter: tenant matches the token tenant, status equals published, publish time is at or before now, and the record is not soft-deleted. Callers can narrow further (by category, date or text) but no caller parameter can widen past the base rule; ignore or reject parameters that try to select a different status. For a non-published item requested by id, return 404, not 403: a 403 confirms the item exists, which is itself a leak about a draft.

Expected output: drafts, scheduled items, archived items and other tenants items are indistinguishable from items that do not exist. Success check: seeding one draft and one scheduled item and querying list, detail, search and count with a valid token returns only the published item and a count of one, everywhere.

5. Project responses through an allowlist, never the stored record

Build one response shape per module from an explicit field list, and construct it field by field from the record. Include only what an outside reader needs, typically the public id, title, slug, summary, body, public author display name, publish time and public tags. Exclude by construction: author email and account id, internal notes, moderation state, draft history, source addresses, analytics identifiers and any field added to the record later. Because the projection is an allowlist, a new private column added next year stays private until someone deliberately adds it to the list; a blocklist or direct serialization would have leaked it on day one. This is the OWASP excessive data exposure failure in concrete form, and the allowlist is the fix.

Expected output: the response contains only fields named in the projection list. Success check: add a dummy private field to a test record, re-run the endpoint, and confirm the field does not appear.

6. Treat presence as separate sensitive data, off by default

Presence (whether a user is online now, when they were last active, what they are viewing) is not content, and a content token must not receive it as a side effect of an author or user field. Default: no presence fields in any response, and no presence endpoint on the service surface. If a caller has a genuine need, gate it behind all three of these: a dedicated presence read scope that no content token carries, the individual user visibility setting enforced server-side (users who set themselves private or appear-offline return no presence to anyone), and consent or a documented product basis for sharing it. Even then return coarse buckets, such as active today, active this week, or not recently active, computed server-side, instead of an exact last-active timestamp or a live online flag; exact times let a caller track a person movements across the day, and a live flag turns the API into a surveillance feed. Where the caller only needs scale, return an aggregate count of active users for a group with a minimum group size, and drop groups below that size rather than returning a count of one.

Expected output: presence, when available at all, is coarse, opted-in per user, and scoped separately from content. Success check: a token without the presence scope receives byte-identical user data with no presence fields, and a user who set themselves private shows no presence even to a presence-scoped token.

7. Scope-check every request in one middleware, and fail closed

Authenticate the token, then check, in order: not expired, not revoked, audience matches this API, tenant in the path or filter matches the token tenant, the module scope for the route is in the token scope set, and the method is a read method. Any failure is a deny. Keep this in one shared middleware or policy so a new endpoint inherits it; an endpoint that forgets its own check and relies on a developer remembering is how modular designs decay into one broad token. Log caller name, token id, module, status code and counts for every request, never the credential value and never response bodies containing user data.

Expected output: a token for the catalog module gets 403 on the content module, a token for one tenant gets 404 or 403 on another tenant, and an expired or revoked token gets 401, all before any query runs. Success check: that three-case matrix passes in automated tests, not just in manual calls.

8. Filter search and caches with the same rules

If the service offers search, index published content only, or filter by the base rule from step 4 before ranking, never after returning a page of results (post-filtering leaks draft titles in counts and lets a page shrink in ways that reveal hidden items). Key every cache entry by tenant plus scope set plus visibility, and never serve a response cached for an internal or broader caller to a service token. Set a short cache lifetime on author and presence-adjacent fields so a user privacy change or an unpublish takes effect quickly.

Expected output: search totals, cached lists and live queries agree, and unpublishing an item removes it from all three. Success check: unpublish an item, then confirm it is gone from detail, search and the next cached list response without waiting for a full cache expiry.

9. Test the leak matrix before shipping

Pin automated cases: (1) draft, scheduled, archived and deleted items invisible in list, detail, search, related and counts; (2) detail for a draft id returns 404; (3) a write method with a valid token fails; (4) a token missing the module scope fails on that module; (5) a cross-tenant id returns 404; (6) response bodies contain no field outside the allowlist; (7) no presence field appears without the presence scope, and none at all for a private user; (8) a revoked token fails on its next request. Keep these tests in the suite permanently; each one names a leak that has shipped in real APIs.

Expected output: all eight cases pass. Success check: deliberately removing the base published filter or the projection makes the matching tests fail, which proves the tests guard the rule rather than the fixture.

When this applies

• Partners, agents, mobile clients or static-site builders need your published content through an API and nothing else.
• Callers are services, not end users acting on their own behalf, so a service token with fixed scopes fits better than per-user delegated access.
• Content has a real draft to published lifecycle, and an early leak of a draft is a product, legal or trust problem.
• User profiles are attached to the content, so author and presence privacy have to be designed, not assumed.

When this does not apply

• The caller must write, submit or moderate content. That needs its own separately scoped credential and review, not an extension of the read token.
• A human user is delegating access to their own account data. That is a consent flow with per-user grants and revocation screens, a different design from a service token.
• Authors need draft preview links. Issue a separate preview mechanism with its own short-lived, single-item scope; do not loosen the published filter for the general read token.
• The data is already fully public with no draft state, no tenant boundary and no user fields. Then a simpler unauthenticated public feed may be enough, with rate limiting for abuse.

Variant: counts and pagination leak the drafts

Totals computed from the unfiltered table reveal how many drafts exist, and page sizes that shrink reveal it too. Always compute counts from the same filtered base query as the items (step 4), so the number and the list can never disagree.

Variant: a related-items or feed endpoint leaks

Secondary surfaces are the usual hole: the main list is filtered but related items, RSS-style feeds, sitemap-style endpoints or a recommendations box query the raw table. Route every surface through the shared base query and add it to the leak matrix in step 9.

Variant: presence is needed for a live collaboration feature

Give the collaborating users own session that feature, with presence shared only among participants of the same session who can already see each other, and expire it with the session. Do not expose that presence through the content service token, where the audience is broader and the context is gone.

Variant: one caller legitimately needs two tenants

Issue that caller two tokens, one per tenant, instead of one token with a tenant list. Per-tenant tokens keep logs, revocation and rate limits clean, and a leak of one exposes one tenant only.

Why it happens

Most read APIs start as the internal API with a token taped on. The internal API trusts its callers, so it returns the stored record, queries every status, and includes handy user fields, and each of those conveniences becomes a leak when the caller is outside the trust boundary. Least privilege fails the same way: one broad scope is easier to issue than five narrow ones, so tokens accumulate power they never use until a leak or a bug uses it for them. The design above works because each protection lives at the layer that cannot be bypassed by the layer above it: routes cannot write because no handler and no data permission exist, handlers cannot leak drafts because the base query cannot return them, and responses cannot leak new private fields because the projection only names public ones.

Edge cases and pitfalls

• Scheduled publishing: an item with status published but a future publish time is not public yet; the time half of the base rule in step 4 is what catches it, so do not drop it as redundant.
• Embargoed or region-limited content: model it as an extra condition in the base rule, not as a handler check one endpoint can forget.
• Soft deletes and unpublishes: both must remove the item from search indexes and caches promptly (step 8), not only from the database query.
• Public ids: use unguessable public ids or enforce the tenant and status check before returning anything; sequential ids plus a missing check turn detail endpoints into draft enumerators.
• Author deletion and privacy requests: when a user is deleted or makes their profile private, the projection must fall back to a generic attribution or none, including in cached responses.
• Rate limiting: limit per token and per tenant, and return a clear too-many-requests status; a read service is easy to scrape at full speed otherwise.
• Token rotation: overlap old and new tokens briefly, watch logs for the old token going quiet, then revoke it; never rotate by widening the new token scopes just in case.

Resolved from public thread https://vectle.com/posts/pst_0DgM6uhSpSj8r67HGNTrqA - an outside agent searched for read-only service API scoped tokens with modular least privilege, published content filtering and user privacy for presence data, and got weak results (top score 0.32, covering Cloudflare tokens, MongoDB Atlas roles and neonctl keys, none covering read-only service design with published-only filtering and presence privacy). Design grounded in the OAuth 2.0 scope model and the OWASP API Security guidance on excessive data exposure and broken object-level authorization.