II.
SkillArea overview
Reference · liveskill-area:code-review-practice
Code Review Practice overview
Reviewing pull requests and diffs for correctness, security, performance, style, and tests — producing structured, actionable review comments.
Attributes
displayName
Code Review Practice
description
Reviewing pull requests and diffs for correctness, security,
performance, style, and tests — producing structured, actionable
review comments.
domains
expertiseLevels
- intermediate
- expert
- authoritative
Outgoing edges
applies_to4
- domain:software-engineering·DomainSoftware Engineering
- specialization:collaboration·SpecializationCollaboration Tools & Practices
- specialization:extreme-programming·SpecializationExtreme Programming
- specialization:cleanroom·SpecializationCleanroom Software Engineering
prerequisite_for_learning1
- skill-area:code-quality·SkillAreaCode Quality
Incoming edges
addresses1
- skill:code-review·SkillCode Review
lib_requires_skill_area76
- lib-agent:web-development--code-quality·LibraryAgentcode-quality
- lib-agent:web-development--code-reviewer·LibraryAgentcode-reviewer
- lib-process:shared--bug-report·LibraryProcesscradle/bug-report
- lib-process:shared--bugfix·LibraryProcesscradle/bugfix
- lib-process:shared--cleanup-runs·LibraryProcesscleanup-runs
- lib-process:shared--contribute·LibraryProcesscradle/contribute
- lib-process:shared--documentation-contribute-answer·LibraryProcesscradle/documentation-contribute-answer
- lib-process:shared--documentation-question·LibraryProcesscradle/documentation-question
- lib-process:shared--feature-harness-integration-contribute·LibraryProcesscradle/feature-harness-integration-contribute
- lib-process:shared--feature-implementation-contribute·LibraryProcesscradle/feature-implementation-contribute
- lib-process:shared--feature-request·LibraryProcesscradle/feature-request
- lib-process:shared--library-contribution·LibraryProcesscradle/library-contribution
- lib-process:shared--project-install·LibraryProcesscradle/project-install
- lib-process:shared--user-install·LibraryProcesscradle/user-install
- lib-process:shared--cc10x-router·LibraryProcessmethodologies/cc10x/cc10x-router
- lib-process:shared--ccpm-tracking·LibraryProcessmethodologies/ccpm/ccpm-tracking
- lib-process:shared--gastown-molecule·LibraryProcessmethodologies/gastown/gastown-molecule
- lib-process:shared--progress·LibraryProcessgsd/progress
- lib-process:shared--process-hardening-patterns·LibraryProcessmethodologies/process-hardening-patterns
- lib-process:shared--rpikit-implement·LibraryProcessmethodologies/rpikit/rpikit-implement
- lib-process:shared--root-cause-diagnosis·LibraryProcessShared root-cause diagnosis task for Phase 0 of all bugfix processes.
- lib-process:shared--finishing-a-development-branch·LibraryProcessmethodologies/superpowers/finishing-a-development-branch
- lib-process:shared--receiving-code-review·LibraryProcessmethodologies/superpowers/receiving-code-review
- lib-process:shared--process-mining·LibraryProcessprocesses/shared/analysis/process-mining
- lib-process:shared--build-failure-triage·LibraryProcessprocesses/shared/ci/build-failure-triage
- lib-process:shared--build-fixer·LibraryProcessprocesses/shared/ci/build-fixer
- lib-process:shared--ci-health-trends·LibraryProcessprocesses/shared/ci/ci-health-trends
- lib-process:shared--conflict-resolution·LibraryProcessprocesses/shared/ci/conflict-resolution
- lib-process:shared--idempotency-and-safe-abort·LibraryProcessprocesses/shared/ci/idempotency-and-safe-abort
- lib-process:shared--handoff-conventions·LibraryProcessprocesses/shared/communication/handoff-conventions
- lib-process:shared--single-channel-communication·LibraryProcessprocesses/shared/communication/single-channel-communication
- lib-process:shared--source-quote-discipline·LibraryProcessprocesses/shared/communication/source-quote-discipline
- lib-process:shared--completeness-gate·LibraryProcessComposable process component that verifies all identified issues have been
- lib-process:shared--cost-aggregation·LibraryProcessComposable process component that aggregates cost-proxy metrics
- lib-process:shared--cycle-aware-verification·LibraryProcessComposable cycle-aware verification component that validates fixes survive
- lib-process:shared--db-agent-explore·LibraryProcessGeneric "point an analyst agent at a local SQLite DB and ask a
- lib-process:shared--deterministic-quality-gate·LibraryProcessComposable deterministic quality gate component that enforces hard shell-based
- lib-process:shared--forbidden-markers-scanner·LibraryProcessComposable pre-deploy gate that scans built JS chunks for substring
- lib-process:shared--fork-contribution-sync·LibraryProcessComposable fork contribution maintenance component providing infrastructure
- lib-process:shared--index·LibraryProcessRe-exports from all shared composable process components.
- lib-process:shared--local-db-build·LibraryProcessGeneric "build a local SQLite database from a source manifest"
- lib-process:shared--feedback-loop-optimizer·LibraryProcessprocesses/shared/local-dev/feedback-loop-optimizer
- lib-process:shared--install-quality-gates·LibraryProcessprocesses/shared/local-dev/install-quality-gates
- lib-process:shared--local-dev-workflow·LibraryProcessprocesses/shared/local-dev-workflow
- lib-process:shared--monorepo-package-scaffold·LibraryProcessComposable monorepo package scaffolding component that creates a new package
- lib-process:shared--n-strikes-escalation·LibraryProcessRun a check task. On failure, run a fixer task and re-check.
- lib-process:shared--playwright-visual-smoke·LibraryProcessComposable process component that performs visual regression smoke tests
- lib-process:shared--prior-attempts-scanner·LibraryProcessComposable process component that scans .a5c/runs/ for prior runs
- lib-process:shared--semantic-release-setup·LibraryProcessprocesses/shared/release/semantic-release-setup
- lib-process:shared--scheduled-report·LibraryProcessprocesses/shared/reporting/scheduled-report
- lib-process:shared--runtime-call-tracer·LibraryProcess* @description Composable runtime call-path tracer that maps execution flows across a codebase
- lib-process:shared--scene-context-builder·LibraryProcessAssemble a "scene context" string for a subagent working on one
- lib-process:shared--source-discovery·LibraryProcessGeneric "discover authoritative open data sources for a domain
- lib-process:shared--tdd-triplet·LibraryProcessComposable TDD triplet component providing a factory for the three canonical
- lib-process:shared--ts-check·LibraryProcessComposable TypeScript compilation check component providing a hard shell gate
- lib-process:collaboration--six-dimension-review·LibraryProcessspecializations/collaboration/code-review/six-dimension-review
- lib-process:collaboration--validator·LibraryProcessspecializations/collaboration/code-review/validator
- lib-process:collaboration--branch-policies·LibraryProcessspecializations/collaboration/github/branch-policies
- lib-process:collaboration--conflict-resolver·LibraryProcessspecializations/collaboration/github/conflict-resolver
- lib-process:collaboration--draft-pr-policy·LibraryProcessspecializations/collaboration/github/draft-pr-policy
- lib-process:collaboration--issue-linking·LibraryProcessspecializations/collaboration/github/issue-linking
- lib-process:collaboration--issue-only-no-direct-commits·LibraryProcessspecializations/collaboration/github/issue-only-no-direct-commits
- lib-process:collaboration--label-taxonomy·LibraryProcessspecializations/collaboration/github/label-taxonomy
- lib-process:collaboration--pr-comment-response·LibraryProcessspecializations/collaboration/github/pr-comment-response
- lib-process:collaboration--pr-lifecycle-comment-response·LibraryProcessspecializations/collaboration/github/pr-lifecycle-comment-response
- lib-process:collaboration--pr-lifecycle-dependency-bump·LibraryProcessspecializations/collaboration/github/pr-lifecycle-dependency-bump
- lib-process:collaboration--pr-lifecycle-docs·LibraryProcessspecializations/collaboration/github/pr-lifecycle-docs
- lib-process:collaboration--pr-lifecycle-feature·LibraryProcessspecializations/collaboration/github/pr-lifecycle-feature
- lib-process:collaboration--pr-lifecycle-hotfix·LibraryProcessspecializations/collaboration/github/pr-lifecycle-hotfix
- lib-process:collaboration--pr-lifecycle-router·LibraryProcessspecializations/collaboration/github/pr-lifecycle-router
- lib-process:collaboration--pr-lifecycle-security·LibraryProcessspecializations/collaboration/github/pr-lifecycle-security
- lib-process:collaboration--pr-policies·LibraryProcessspecializations/collaboration/github/pr-policies
- lib-process:collaboration--producer·LibraryProcessspecializations/collaboration/github/producer
- lib-skill:web-development--eslint·LibrarySkilleslint
- lib-skill:web-development--git·LibrarySkillgit
- lib-skill:web-development--prettier·LibrarySkillprettier
requires_expertise5
- responsibility:code-review-coverage·ResponsibilityCode review coverage
- role:code-reviewer·RoleCode Reviewer
- role:engineering-manager·RoleEngineering Manager
- role:engineering-manager-convergent·RoleEngineering Manager
- role:tech-lead·RoleTech Lead
requires_skill_area8
- workflow:code-complexity-review·WorkflowCode Complexity Review
- workflow:user-story-refinement-session·WorkflowUser Story Refinement Session
- workflow:typo-and-lint-sweep·WorkflowTypo and Lint Sweep
- workflow:codebase-health-refactoring-sprint·WorkflowCodebase Health Refactoring Sprint
- workflow:open-source-contribution·WorkflowOpen Source Contribution
- workflow:cad-design-review·WorkflowCAD Design Review
- workflow:mentorship-pairing-cycle·WorkflowMentorship Pairing Cycle
- workflow:first-production-deploy-milestone·WorkflowFirst Production Deploy Milestone