For engineering leaders

The full instrument.

The Technical view is for people who will defend a number in a 1:1. Every score traces to stored evidence. We never rank on volume.

An AI reviewer that knows your systems. Reviews carry your business context, what this repo is, the ticket the change served, and the surrounding code — not just the diff.
Inline, on GitHub or Bitbucket Cloud. Every pull request is reviewed, typically within minutes. Findings land on the exact lines, are re-checked on every push, and close themselves when the fix is in. Direct commits to a branch are scored too.
Skill inventory and trajectory per person. Strengths, gaps, and whether someone is climbing — from merged work, never from commit counts.
Delivery alignment. Whether the change actually did what the ticket asked, linked automatically even when nobody tagged it.
Monthly architecture reviews on Team and Scale. A whole-repo read: layering, debt hotspots, dependency rot, security architecture.
Training built from your team's own PRs, runnable in a sandboxed editor — so feedback ends in practice, not a document.

PR #482 — Migrate payment retries to outbox patternScored

alice@ · payments-core · merged 2h ago

Correctness

Error handling

Testing

Maintainability

Scope discipline

+ if err := tx.Commit(ctx); err != nil { + return fmt.Errorf("outbox enqueue: %w", err)

Why 61 on testing: the retry path has no test covering the double-delivery case introduced at line 141 — exercise generated in Training.

Lead the agent

Your team already directs AI. We teach them to do it well.

We do not detect who used a tool, and we do not rank people on "AI skill." Those numbers are guesses, and a guess your team can dispute is a number they will ignore. What we do instead: NRD Sight already knows what good looks like in this repo. We hand the developer the prompt that would have produced that — and the prompt that fixes what didn't.

Agent brief — standing instructions for this repo: purpose, conventions, business priorities, never-do list. Paste into Cursor, Claude, or Grok project rules.
Start this work — ticket to spec: problem, acceptance, files to read, tests to add, out of scope.
Fix this — a scoped prompt next to every review finding and security issue. Confirm it still exists. Stay in scope. Add the test. Explain the PR.
Practice this — write the spec you would hand an agent for your last weak PR. Get it graded. Copy the rewrite.
Close the loop from the agent — on Team and up, the NRD Sight MCP server's get_pr_review lets a coding agent push, wait for the review of that exact commit, fix the critical and important findings, push again, and stop after three rounds with a summary for the human. It works with Claude Code, Codex, Cursor and VS Code, and agent instructions ship for each (a Claude Code skill, an AGENTS.md block, a Cursor rule); the same server and a read-only REST API expose the rest of your Sight data.

The professional bar is the same in every prompt: state the problem first, name the repo by URL, stay scoped, prefer existing helpers, add tests, invent nothing, put no secrets in the paste.

Copy prompt · review finding

I need you to address a code-review finding from a
principal-engineer pass on a merged pull request.

Rules:
- Stay scoped to this finding.
- Confirm the issue still exists.
- Add or update tests where it is behavioral.
- Open a PR that explains the fix in plain language.

**Issue:** Retry without idempotency key can double-capture
**Repository:** https://github.com/acme/payments
**Do this:** Reuse the request's idempotency key on every retry.

Why this prompt: stay scoped, confirm the issue still exists, add tests, and explain the change in the PR.

Merge gate · optional

Hold risky merges, on your terms.

Set it per organization, team, repository, or developer. Start in advisory and read the shadow report of what it would have blocked; move to acknowledge or strict when the numbers say it's worth it.

Native to your code host. A required GitHub check, “NRD Sight review”, or a Bitbucket Cloud build status with PR tasks (enforced on Bitbucket Premium; a warning otherwise).
Handled in the thread. Push the fix, or reply /nrd wont-fix, /nrd false-positive, or /nrd ack. Maintainers can /nrd override with a reason, and every override is recorded.
Fewer false alarms over time. A category your team keeps dismissing as a false positive stops blocking automatically.

How the merge gate works →

BLOCKINGNRD Sight review · strict

1 finding must be fixed or answered before this pull request can merge

web-infra · PR #517 · infra/site.tf:41 · public bucket

/nrd wont-fix this bucket serves the public marketing site's images, by design

Recorded: who answered, and why. The finding closes and the gate re-evaluates on the spot.

RefreshRequest override

Every pull request graded — then the prompt that would have scored higher.

The full instrument.

Your team already directs AI. We teach them to do it well.

The fastest way to make engineers hate measurement is to measure motion.

Hold risky merges, on your terms.

Paste a public GitHub pull request. No repo connection required.