identify-reviewer
$
npx mdskill add cloudflare/workerd/identify-reviewerEstablish local user context by gathering GitHub and Git identities before starting any code review.
- Frames findings relative to the user's own prior contributions and approvals.
- Integrates with GitHub CLI and local Git configuration tools.
- Runs identity detection commands at the start of review workflows.
- Stores context to attribute comments and commits using second-person references.
SKILL.md
.github/skills/identify-reviewerView on GitHub ↗
---
name: identify-reviewer
description: Identifies the local user's GitHub account and git identity before performing code reviews. Load this skill at the start of any PR review, code review, or commit log analysis so findings can be framed relative to the user's own prior comments, commits, and approval status.
---
## Identify Reviewer
Load this skill at the start of any code review workflow — before analyzing diffs, commit logs, or prior comments.
---
### Steps
1. **Detect identities.** Run these commands in parallel:
```
gh auth status
git config user.name
git config user.email
```
- From `gh auth status`, parse `Logged in to github.com account <USERNAME>` to get the GitHub handle.
- From `git config`, capture the local user's commit name and email.
These may not match exactly (e.g., GitHub handle `octocat`, git name `Octo Cat`, git email `octocat@example.com`). All three identify the same person.
If `gh` is not installed or the user is not authenticated, fall back to git config alone. If neither is available, skip identification and proceed without it — do not block the review.
2. **Store the identity for the session.** Use the discovered identity when:
- **Attributing prior review comments.** If the user previously commented on the PR, refer to those comments in second person ("your earlier comment about X") rather than third person ("octocat flagged X").
- **Attributing commits.** When analyzing `git log` output, match the author name/email against the git config values. Refer to the user's own commits in second person ("your commit `abc1234` introduced...") and other authors' commits in third person.
- **Filtering approval status.** Note whether the user has already approved, requested changes, or not yet reviewed. Frame the review summary accordingly (e.g., "You previously approved this PR; here are new findings since your approval").
- **Distinguishing roles.** If the user is also the PR author, flag this clearly and adjust tone (self-review). Match by both GitHub handle and git author email since either may appear in PR metadata.
3. **Apply throughout the review.** Every finding that references a prior comment or commit by the user should use second person. Prior comments and commits by _other_ people remain in third person with their handle or name.
### Matching Rules
- GitHub handle from `gh auth status` matches PR review comment authors and PR author login.
- Git name from `git config user.name` matches `git log --format='%an'` author names.
- Git email from `git config user.email` matches `git log --format='%ae'` author emails.
- A person is the local user if **any** of these identifiers match.
### Examples
Without this skill:
> octocat flagged the performance regression in a prior review. The author disagreed.
With this skill (when the local user is octocat):
> You flagged the performance regression in a prior review. The author disagreed but did not provide benchmarks.
Without this skill:
> Commit `abc1234` by Octo Cat refactored the decoder interface.
With this skill (when git user.name is "Octo Cat"):
> Your commit `abc1234` refactored the decoder interface.
More from cloudflare/workerd
- add-autogateStep-by-step guide for adding a new autogate to workerd for gradual rollout of risky changes, including enum registration, string mapping, usage pattern, and testing.
- add-compat-flagStep-by-step guide for adding a new compatibility flag to workerd, including capnp schema, C++ usage, testing, and documentation requirements.
- bazel-test-hygieneMandatory rules for running bazel tests during development. Load this skill before running any bazel test command, especially when validating fixes or verifying regression tests. Prevents false confidence from cached results, filter flags that silently match nothing, and partial test runs that miss breakage.
- commit-categoriesCommit categorization rules for changelogs and "what's new" summaries. MUST be loaded before categorizing commits in changelog or whats-new commands. Provides the canonical path-based category table used to group commits by area.
- dad-jokesAfter completing any task that took more than ~5 tool calls, or after long-running builds/tests finish, load this skill and deliver a dad joke to lighten the mood. Also load before any user-requested joke, pun, or limerick. Never improvise jokes without loading this skill first.
- find-and-run-testsHow to find, build, and run tests in workerd. Covers wd-test, kj_test target naming, bazel query patterns, and common flags. Also covers parent project integration tests if workerd is used as a submodule. Load this skill when you need to locate or run a test and aren't sure of the exact target name or invocation.
- investigation-notesStructured scratch tracking document for investigation state during bug hunts - prevents re-reading code, losing context, and rabbit holes; maintains external memory so you don't re-derive conclusions
- kj-styleKJ/workerd C++ style guidelines for code review. Covers naming, type usage, memory management, error handling, inheritance, constness, and formatting conventions. Load this skill when reviewing or writing C++ code in the workerd codebase.
- markdown-draftsUse markdown formatting when drafting content intended for external systems (GitHub issues/PRs, Jira tickets, wiki pages, design docs, etc.) so formatting is preserved when the user copies it. Load this skill before producing any draft the user will paste elsewhere.
- module-registryLoad when working with the module registry in workerd — reading, modifying, debugging, or reviewing module resolution, compilation, evaluation, or registration code. Provides pointers to three reference documents covering the legacy registry, V8 module internals, and the new registry design.