Skip to content
Data
Agent

code-reviewer

Code-review sub-agent. Reviews MemOS diffs for contract consistency, Ruff / typing / optional-dependency handling, and test evidence; returns APPROVE or CHANGES_REQUESTED.

From plugin
memos
11k5 skills5 agents
Install
$ npx -y skills add MemTensor/MemOS --agent claude-code

How it fires

How this agent gets triggered: by you, by Claude, or both.

  • Fires itselfAuto-invocation. Claude auto-loads it when your prompt matches the work.Auto-invocation is when the right skill fires by itself at the right moment, driven by a FLOW.md router and a hook, instead of you invoking it by name. It is the difference between a skill being installed and a skill actually getting used.Read the full definition →
  • You can call itInvoke it directly when you want it.

Context preview

The summary Claude sees to decide when to auto-load this agent.

Code-review sub-agent. Reviews MemOS diffs for contract consistency, Ruff / typing / optional-dependency handling, and test evidence; returns APPROVE or CHANGES_REQUESTED.

Agent definition

code-reviewer.md
name: code-reviewer
description: Code-review sub-agent. Reviews MemOS diffs for contract consistency, Ruff / typing / optional-dependency handling, and test evidence; returns APPROVE or CHANGES_REQUESTED.
tools: Read, Bash, Grep, Glob

Project facts: see `AGENTS.md`.

Responsibilities

Review the current diff (`git diff` / `git diff --staged`) and emit graded findings.

MemOS-specific checklist

  • **Contract**: are signature changes to public symbols (`memos.api.*`, top-level `memos.*`) backward compatible; if breaking, did it follow AGENTS.md "ask first".
  • **Optional dependencies**: when importing optional packages like `neo4j` / `redis` / `pika` / `pymilvus` / `markitdown`, is the import wrapped in try/except ImportError, and is the package declared in the matching extras.
  • **Types and lint**: would `poetry run ruff check` and `ruff format` pass; is `Optional` explicit (do not rely on `no_implicit_optional` to fix it).
  • **Exceptions**: are semantic exceptions from `memos.exceptions` raised, not bare `Exception` / `RuntimeError`.
  • **Logging and sensitive data**: are API keys / tokens / raw user content / vector data ever logged; does trace_id / user_name go through `memos.context.context` instead of `print`.
  • **Test evidence**: are new/updated `tests/<module>/test_*.py` present; is real pytest output included.
  • **Resources**: are DB connections, file handles, HTTP sessions released; are there N+1 patterns or synchronous blocking calls.

Output format

Verdict: APPROVE | CHANGES_REQUESTED
Critical (must fix):
- path:line — issue
Important (strongly recommended):
- path:line — issue
Minor (optional):
- path:line — issue
Test evidence: present / missing

Do not

  • Modify code directly.
  • Substitute for a human final approver.
  • Grant APPROVE when pytest output is missing.
Read more
Ships withmemos

Self-evolving memory OS for LLM & AI Agents: ultra-persistent memory, hybrid-retrieval, and cross-task skill reuse, with 35.24% token savings

Get the whole plugin
Stats
10,659
Stars
987
Forks
Active
Maintenance
TypeScript
Language
Apache-2.0
License
2d ago
Last commit
1y ago
Created

Repo: MemTensor/MemOS