Tag: technical leadership

  • The New Code Review: How Humans Should Review Work From AI Coding Agents

    The New Code Review: How Humans Should Review Work From AI Coding Agents

    AI Can Write the Diff. Humans Still Own the Decision.

    AI coding agents are changing what code review is for. In a traditional review, a teammate usually explains the problem, writes the code, and opens a pull request with human intent behind every major choice. With an AI coding agent, implementation can arrive faster, broader, and sometimes more confidently than the underlying reasoning deserves.

    That does not make review less important. It makes review more judgment-heavy. The reviewer’s job is no longer just to spot syntax mistakes, suggest cleaner names, or ask for one more test. It is to decide whether the change should exist, whether it solves the right problem, whether it fits the system, and whether the team can safely maintain it later.

    Recent industry research points in the same direction: AI adoption in software work is rising, but trust, accuracy, and human verification remain central concerns. Stack Overflow’s 2025 Developer Survey found that more developers distrusted the accuracy of AI tools than trusted it, while DORA’s 2025 research reported broad workplace use of AI among technology professionals alongside ongoing questions about effective, reliable adoption. In practice, strong teams treat AI-generated code as a fast draft from a capable but non-accountable contributor. Useful? Often. Final? Not until a human has reviewed it.

    Why AI-Written Code Needs a Different Review Mindset

    AI coding agents are good at producing plausible code. That is both their strength and their risk. A human junior developer may ask clarifying questions, hesitate around unfamiliar systems, or leave obvious gaps. An AI agent may produce a complete-looking implementation even when the task is underspecified, the repository patterns are unclear, or the business rule is ambiguous.

    Reviewers should assume three things until proven otherwise: the agent may have optimized for local correctness instead of system fit, it may have filled in missing requirements without saying so, and it may have changed more than the task required. This is not a reason to reject AI assistance. It is a reason to review from the outside in.

    • Do not start by admiring the diff. Start by restating the user need or engineering goal.
    • Do not assume a passing test means the behavior is right. Ask whether the test proves the intended outcome.
    • Do not treat confident code as explained code. Require traceable reasoning for important changes.
    • Do not reward large, sweeping changes if a smaller change would have solved the problem.
    • Do not let the AI agent’s speed pressure the team into lowering review standards.

    Before Reading the Diff, Check the Assignment

    The most useful review often happens before the reviewer opens the changed files. If the task is vague, the code review will become a guessing game. For AI-generated work, reviewers should first inspect the prompt, ticket, acceptance criteria, or issue description that guided the agent.

    Ask whether the agent was given a clear target. What behavior should change? What should stay the same? Which files, APIs, roles, devices, permissions, or data boundaries matter? What constraints were stated? What constraints were assumed? If the task asks for “improve checkout validation,” the reviewer needs to know whether that means better error messages, stricter server-side rules, accessibility improvements, fraud prevention, or all of the above.

    • What exact problem is this change supposed to solve?
    • Who benefits from the change: user, admin, developer, support team, or business stakeholder?
    • What are the acceptance criteria, and are they measurable?
    • What areas of the system were intentionally out of scope?
    • Was the AI agent allowed to add dependencies, change database schemas, alter public APIs, or refactor unrelated code?
    • Is there a human-readable summary of what the agent changed and why?

    A Layered Review Workflow for AI Coding Agents

    A practical human-in-the-loop review works best in layers. Instead of reading every line from top to bottom immediately, move from purpose to risk to implementation detail. This helps reviewers avoid getting distracted by polished code that may not solve the right problem.

    1. Product Intent: Does This Solve the Right Problem?

    Start with the outcome. If the change is user-facing, verify that it matches the intended workflow, language, permission model, and failure states. If it is internal, verify that it improves the developer or operational experience without creating hidden obligations.

    AI agents can accidentally implement a nearby idea instead of the actual requirement. For example, an agent asked to “add admin filtering” might build a new search interface when the real need was a simple status dropdown on an existing table. The code may work, but the product judgment is wrong.

    • Does the change match the original request, not merely a related interpretation?
    • Are edge cases defined from the user’s point of view?
    • Could the new behavior surprise existing users?
    • Are copy, labels, errors, and empty states clear and appropriate?
    • Does the change respect role permissions and business rules?

    2. Architecture Fit: Does It Belong Here?

    Next, check whether the implementation fits the existing system. AI agents often infer patterns from nearby files, but they may miss deeper conventions: service boundaries, domain ownership, performance assumptions, release constraints, or framework-specific best practices.

    A good reviewer asks whether the change makes the codebase easier or harder to reason about six months from now. A solution that adds a new abstraction, helper, dependency, or background job should justify the extra moving parts.

    • Does the change follow existing project patterns?
    • Is the logic located in the right layer, such as UI, API, domain service, or data access?
    • Does it duplicate behavior that already exists elsewhere?
    • Does it introduce a new abstraction before the codebase needs one?
    • Would another developer know where to look when this feature breaks?

    3. Data, Security, and Privacy Risk: What Could Go Wrong?

    AI-generated code deserves careful review anywhere it touches authentication, authorization, payments, personally identifiable information, customer data, logs, file uploads, external APIs, or database writes. These are areas where a small plausible mistake can become a serious incident.

    Reviewers should pay special attention to silent trust changes. Did the code move validation from the server to the client? Did it expose extra fields in an API response? Did it log sensitive input? Did it make an admin-only operation reachable from a lower-privilege path? These problems may not stand out in a diff unless the reviewer is looking for them.

    • Are authorization checks still enforced on the server?
    • Are inputs validated and outputs encoded in the right places?
    • Does the change expose new data through responses, logs, analytics, or error messages?
    • Are secrets, tokens, and credentials handled safely?
    • Do database migrations preserve existing data and support rollback?
    • Does any new dependency increase supply-chain risk?

    4. Test Evidence: What Has Been Proven?

    For AI-generated work, reviewers should not ask only “Are there tests?” A better question is “What claim do these tests prove?” AI agents can create tests that mirror their own assumptions, assert implementation details, or cover the happy path while missing the real failure mode.

    Useful tests connect back to acceptance criteria. If the task is about permissions, tests should cover allowed and denied users. If the task is about data transformation, tests should include messy inputs. If the task is about a user interface, tests or review evidence should cover keyboard navigation, screen states, and error handling where appropriate.

    • Do the tests fail without the production change?
    • Do they cover the bug, feature, or risk described in the task?
    • Are negative cases included, not only happy paths?
    • Are edge cases represented with realistic data?
    • Is there evidence from local runs, CI, screenshots, logs, or manual verification when automated coverage is not enough?

    5. Readability and Maintainability: Can Humans Own This Code?

    AI agents can generate code that is syntactically correct but oddly shaped. The reviewer should make sure future humans can understand, debug, and extend it. Cleverness is not a virtue if it makes the team dependent on another AI pass to understand the implementation.

    Look for unnecessary generalization, inconsistent naming, overly defensive branches, and comments that describe what the code does without explaining why. Also watch for large formatting churn that hides the meaningful change.

    • Is the simplest reasonable solution used?
    • Are names consistent with the domain language of the project?
    • Can the code be understood without reading the original prompt?
    • Are comments used to explain non-obvious decisions rather than restating the code?
    • Does the diff avoid unrelated cleanup, formatting churn, and opportunistic refactors?

    6. Operational Impact: What Happens After Merge?

    Some changes are correct in isolation but risky in production. Reviewers should consider deployment, monitoring, performance, support, and rollback. AI agents may not know which parts of the system are fragile, expensive, rate-limited, or heavily used unless the prompt and repository context made that clear.

    • Could this increase latency, memory use, API calls, database load, or background job volume?
    • Does the change need feature flags, staged rollout, or migration sequencing?
    • Are errors observable through logs, metrics, or alerts?
    • Can the change be rolled back safely?
    • Will support, documentation, or customer-facing guidance need updates?

    When to Ask the AI Agent for a Self-Review

    A useful habit is to ask the AI coding agent to review its own work before the human review begins. This is not a substitute for human judgment. It is a way to surface assumptions, summarize changes, and generate a checklist of likely risk areas.

    Sources and Fact Check References

    • Stack Overflow Developer Survey 2025 – Stack Overflow’s 2025 Developer Survey found that more developers distrusted the accuracy of AI tools than trusted it.
    • DORA 2025 Research – DORA’s 2025 research reported broad workplace use of AI among technology professionals and examined reliable adoption of AI in software delivery.