search-engine-core code-review.instructions.md

A checklist for reviewing C++ server changes before submission, with fixes for common database, request-handling, initialization, linking, and memory problems.

In plain words
What is it for?
Use it before submitting changes to check request-abort handling, MongoDB usage, logging, lazy initialization, libraries, collection names, memory, and thread safety.
Why use it?
It catches recurring implementation mistakes before they reach review or production.

Instructions file for GitHub Copilot

Install

Getting it into your agent

One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.

agentmods
npx agentmods add instructions/hatefsystems/search-engine-core/code-review
Clone the repo
git clone --depth 1 https://github.com/hatefsystems/search-engine-core

Made for: GitHub Copilot.

Per session 250 This file is loaded in full into every session.
When invoked 250 The same file — it is already loaded in full.
Security scan A 0 findings. Scan, not verified.
Origin original No closer match found in the catalogue.
Token cost

What it costs to keep this loaded

Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.

ModelPer sessionOnce invoked
Fable 5 $0.00250 $0.00250
Opus 5 $0.00125 $0.00125
Sonnet 5 $0.00050 $0.00050
Haiku 4.5 $0.00025 $0.00025

Measured yesterday against content hash 42abb6bd20e7, method: parsed. Prices are Anthropic first-party input rates as of 2026-08-30, from the pricing page.

Security

Grade A, and why

search-engine-core code-review.instructions.md scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured yesterday.

A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.

Nothing flagged

None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.

.github/instructions/code-review.instructions.md · 29 lines

What it actually says

Code Review Checklist

Before submitting:

  • Every onData paired with onAborted
  • Controllers use lazy init (no service init in constructor)
  • Use LOG_DEBUG() not std::cout; respect LOG_LEVEL
  • MongoDB: basic builder + .extract() for complex updates
  • MongoDB instance initialized where needed
  • Error handling and logging in place
  • CMakeLists.txt updated if adding libs
  • No memory leaks; thread-safe where required

Common Issues

Symptom Cause Fix
Empty reply / POST crash Missing onAborted Pair with onData
MongoDB crash Instance not initialized MongoDBInstance::getInstance() first
Static init crash Services in controller ctor Lazy init with getters
Document loses fields Stream builder / no extract Basic builder + .extract()
Undefined reference Missing lib Add to target_link_libraries
Collection not found Inconsistent collection names Use same name in code and imports
Changes

What this file has done since we first saw it

Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.

  1. yesterday First seen · 29 lines · 250 tokens per session scan A 42abb6bd20e7

Subscribe to this mod's changes

search-engine-core code-review.instructions.md is an instructions file published in the GitHub repository hatefsystems/search-engine-core (5 stars, last pushed 27d ago), licensed Apache-2.0. It adds 250 tokens to every session, about $0.0013 per session on Opus 5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-31.

Related

Other instructions, from other repositories

Xapiand AGENTS.md

Instructions for Kronuz/Xapiand, covering agents.md — working in the xapiand codebase, first, the big split: own code vs. vendored, where things live, build & toggles and verifying changes end-to-end (the docs are a test suite).

Kronuz/Xapiand · 2,843 tokens

electron CLAUDE.md

Instructions for electron/electron, covering electron development guide, running nodemodules binaries, project overview, directory structure and build tools setup.

electron/electron · 2,653 tokens

electron copilot-instructions.md

Instructions for electron/electron, covering copilot instructions for electron, build system, linting, running a single test and example: npm run test -- -g "ipc".

electron/electron · 1,178 tokens

compiler-explorer AGENTS.md

Instructions for compiler-explorer/compiler-explorer, covering agents.md, build & test commands, important workflow requirements, style guidelines and architecture guidelines.

compiler-explorer/compiler-explorer · 2,422 tokens

weaviate CLAUDE.md

Instructions for weaviate/weaviate, covering claude.md, no bug is ever out of scope, build & run, build the binary (cgoenabled=0, static linking) and build debug binary (with delve support).

weaviate/weaviate · 2,591 tokens

compiler-explorer copilot-instructions.md

Instructions for compiler-explorer/compiler-explorer: For each answer to the user, evaluate your level of confidence in the correctness of the answer on a scale from 1 to 10, where 1 is very uncertain and 10 is absolutely certain. If your confidence level is below 8, state so and suggest ways to verify the answer.

compiler-explorer/compiler-explorer · 60 tokens