Skills

Skill 26 of 33

Convex Reviewer

Convex code reviewer — security, auth, validators, performance, and pattern checks for code in a convex/ directory.

1 minute · 222 words · 3 sections

Install

$ npx skills add convex/agent-skills --skill convex-reviewer

$ npx skills add convex/agent-skills

The first command installs just this skill, by the name in its SKILL.md; the second installs the whole repository.

Convex Code Reviewer

Structured review of Convex code for security, authorization, validators, performance, and schema design. Applies a Convex-specific checklist and flags anti-patterns with severity (Critical / Important / Suggestion).

Workflow

First pass — Security: verify all public functions check ctx.auth.getUserIdentity(), verify resource ownership before reads/writes, confirm no client-provided user IDs are trusted, confirm scheduled functions target internal.* not api.*.
Second pass — Performance: confirm no .filter() on DB queries (withIndex required), verify all foreign-key fields have indexes, confirm no Date.now() in query handlers, confirm .collect() is not used on unbounded queries.
Third pass — Code quality: confirm args and returns validators on every public function, no any types, promises are awaited, arrays in documents are bounded (<8192 elements).
Report findings grouped by severity; explain why each issue matters and suggest a fix.

Rules

Flag missing auth checks as Critical — any unauthenticated public mutation is a data-loss risk.
Flag .filter() on DB queries as Important — it is a full table scan.
Flag Date.now() in query handlers as Important — it breaks reactivity.
Flag missing args or returns validators as Important.
Flag scheduling to api.* (not internal.*) as Important.
Always explain why a change is needed, not just what to change.

About this skill

Trigger

Convex code reviewer — security, auth, validators, performance, and pattern checks for code in a convex/ directory. Use to review or audit Convex functions before shipping.

The verbatim description from this skill’s front matter — the string an agent matches on to decide whether to load it.

Colophon for this issue

Colophon

How this issue was made

Source

github.com/convex/agent-skills, branch main, last pushed 29 October 2026.

Licence

Apache-2.0 — the text of every skill is reproduced unmodified, frontmatter included, under the upstream licence.

Discovery

33 skills found by walking the repository tree for SKILL.md, not by matching a directory convention. One layout observed: skills/*/SKILL.md.

Issue colours

Resolved from a curated brand profile — hue 80°, chroma 0.160. Two accent tones are generated per issue and each is proven against its own ground before it ships: a single accent that passes AA on both light and dark paper is arithmetically impossible.

Heading repairs

1 repair applied to this skill so the document has one h1 and no skipped levels:

Shifted “3 headings” from h1 to h2 so the skill title is the only h1.

Spec compliance

Every skill satisfies the Agent Skills specification’s frontmatter rules.

Images

Images inside a skill come from the upstream repository. Where the author gave no alternative text we mark the image decorative rather than inventing a description — a plausible caption we made up is worse than none for the reader who depends on it.

Signal

Install counts come from skills.sh. They measure downloads, not quality, and an unranked repository is not an unread one.

Agent surfaces

The whole issue is available as one markdown document at /convex/agent-skills.md, and each skill at its own .md URL.

Publication

Set by Skills Docs from the source repository. Body text is Literata at the reader’s chosen size and measure; code is Geist Mono. Nothing on this page was written by us except this paragraph.

Convex Code Reviewer

First pass — Security: verify all public functions check ctx.auth.getUserIdentity(), verify resource ownership before reads/writes, confirm no client-provided user IDs are trusted, confirm scheduled functions target internal.* not api.*.

Second pass — Performance: confirm no .filter() on DB queries (withIndex required), verify all foreign-key fields have indexes, confirm no Date.now() in query handlers, confirm .collect() is not used on unbounded queries.

Third pass — Code quality: confirm args and returns validators on every public function, no any types, promises are awaited, arrays in documents are bounded (<8192 elements).

Report findings grouped by severity; explain why each issue matters and suggest a fix.

Flag missing auth checks as Critical — any unauthenticated public mutation is a data-loss risk.

Flag .filter() on DB queries as Important — it is a full table scan.

Flag Date.now() in query handlers as Important — it breaks reactivity.

Flag missing args or returns validators as Important.

Flag scheduling to api.* (not internal.*) as Important.

Always explain why a change is needed, not just what to change.