Using Gitar to Verify Pull Requests Beyond the Diff

Sonar’s Killian Carlsen-Phelan and Taylor Luttrell-Williams walk through ticket-transfer, pricing and CI-failure examples in a sample application.

TLTaylor Luttrell-WilliamsSpeaker · SonarKCKillian Carlsen-PhelanSpeaker · Sonar
Channel
AI Engineer
Published
Oct 29, 2026
Video
38:07
Read
6 min
Slides
6
Original
English
Open on YouTube
Scroll to read and the video shrinks to the bottom-right corner. Captions follow your reading language.
Reading languageAvailable in 2 languages
You're reading the original English.
SUMMARY

In 30 seconds

In a hands-on Sonar workshop, Taylor Luttrell-Williams introduces Gitar and Killian Carlsen-Phelan demonstrates three pull requests against a sample ticketing API. The presenters show a finding outside the immediate diff, project-specific pricing rules and a CI diagnosis with optional automated fixes. Their statements about product capability reflect a vendor demonstration, not an independent assessment.

  1. 1The review bottleneckLuttrell-Williams asks how teams can verify changes as agents increase the volume and speed of code production.
  2. 2A sample ticketing APICarlsen-Phelan explains the Box Office application, its order and check-in logic, and its existing test suite.
  3. 3A bug outside the diffThe demonstrated review flags that cancelling an order can invalidate a ticket already transferred to another person.
  4. 4Business rules as contextA project rule asks the reviewer to check that customer-facing ticket prices include mandatory fees.
  5. 5A failing CI checkThe third pull request breaks the rule preventing a ticket from being scanned twice; the demo enables optional automated remediation.
  6. 6Wider contextThe presenters describe cross-repository and cross-PR analysis, alongside issue-tracker and learned-review context.

Takeaways

  1. A change can break behavior in existing code even when the new diff and its tests appear sound.
  2. Project-specific review instructions can express a business rule that ordinary code correctness alone does not capture.
  3. A failed CI test can expose a missing invariant, such as preventing a ticket from being scanned twice.
  4. Automatic fixes and merging depend on explicitly selected settings and passing the configured checks.
  5. The presenters describe additional cross-repository, cross-PR and issue-tracker context capabilities, but the workshop does not independently establish their reliability.
FULL TEXT

Read the edited text by chapter

Transcript only
01

Setup and the verification problem

The Sonar presenters introduce the workshop and explain why they consider agent-generated changes challenging to review. They describe the tool’s claimed review and CI capabilities.

Taylor Luttrell-Williams introduces herself and Killian Carlsen-Phelan as developer-content engineers at Sonar. Their workshop demonstrates Gitar, a code-change verification agent, using a sample ticketing project and three pull requests. The claims about the tool’s capabilities come from its vendor’s presentation.

The workshop uses a 14-day trial and a dedicated GitHub repository. The presenters say the exercises run in a browser and require a GitHub account and the Gitar app. They plan to start with an out-of-the-box review, add project-specific instructions, and then demonstrate CI failure analysis.

Luttrell-Williams frames the problem as a verification bottleneck. Coding agents can generate changes quickly, but teams still need to determine whether the resulting code is secure, maintainable, reliable and correct for the intended business behavior. She says fixed-size review teams can struggle as the queue of changes grows.

She argues that agents may lack project-specific context and can miss existing architecture or coding conventions. Her concern is that faster production can produce more incidents if teams do not verify the result. These are the presenter’s claims; the workshop does not provide an independent incident-rate analysis.

Luttrell-Williams describes Gitar as a change-verification agent that reviews pull requests, diagnoses CI failures and can suggest or apply fixes. She says users can choose their level of automation and review involvement. She also claims that the product is tuned to reduce false positives and can provide analytics.

According to the presentation, context can include the pull-request diff and discussion, CI logs, and project-specific instructions. A review is triggered when a pull request is opened; findings and suggested changes are posted back to the request. Automated fixing and merging are optional settings, not inherent guarantees of correctness.

02

Preparing the sample application

Carlsen-Phelan forks a workshop repository and introduces its Python ticketing API. The exercise opens three pull requests.

Carlsen-Phelan takes over the hands-on demonstration. He says participants should install the Gitar GitHub app and fork the workshop repository into the organization where the app is installed. He emphasizes unchecking “copy the main branch only,” because the exercises use several branches.

He then enables GitHub Actions in the fork and checks that the repository appears in the Gitar configuration. These settings are part of the demonstration: Actions and later merge permissions determine which automated checks and actions are available in that repository.

The sample application, Box Office, is a small Python ticketing API. Carlsen-Phelan describes orders.py for purchasing and cancelling tickets and checkin.py for admitting attendees by QR code. The repository also contains tests, which run whenever a pull request is opened.

He opens pull requests for the three exercise branches against his fork, allowing reviews to run while he explains the scenarios. He cautions participants to choose their own fork as the base repository rather than accidentally opening the requests against the upstream project.

03

A transfer bug outside the diff

The first change adds ticket transfers. Gitar reports that an existing cancellation path can invalidate a transferred ticket, and the presenter requests a fix.

The first pull request adds the ability to transfer a ticket to a friend. It includes basic validation for missing tickets, invalid email addresses and voided tickets, as well as tests. At the point shown, the tests and CI workflow are green.

Carlsen-Phelan contrasts static analysis with a contextual review. He says static analysis can trace certain security issues, while the demonstrated agent also considers descriptions, intended behavior and existing code. He presents the approaches as complementary; this example does not prove that either approach catches every defect.

Gitar posts a finding: a buyer can transfer a ticket and then cancel the original order, invalidating the recipient’s ticket. The missing case is the reverse of an existing test that prevents transferring a ticket already voided. The relevant cancellation logic is in the existing orders.py file, outside the immediate change being reviewed.

Ticket transfer review finding
SLIDE 1 / 6Ticket transfer review findingThe GitHub review describes how an order cancellation can invalidate a ticket after it was transferred.

The presenter explains why a reviewer looking only at the new diff could miss the interaction. Understanding the risk also requires knowing how the existing cancellation path works. He uses the example to illustrate why broader context matters as changes and codebases grow.

To request remediation, he shows the “apply fix” control or a Gitar fix comment. He says the agent adds a commit and rechecks the change. Later in the demonstration he shows the proposed correction in orders.py, where an additional check has been added; the workshop does not establish that every automatic fix will be correct.

04

Reviewing a business pricing rule

The second pull request adds a service fee. Project instructions ask the reviewer to check whether customer-facing prices include that fee.

The second pull request adds a $3.50 service fee per ticket. A $50 ticket would therefore cost $53.50 at checkout. The fictional business has an additional rule: every price shown to a customer must already include mandatory fees, so a technically functioning calculation can still violate the product requirement.

Project review instructions and pricing rule
SLIDE 2 / 6Project review instructions and pricing ruleThe pull request contains instructions for all-in ticket prices and a rule triggered by changes to pricing files.

Carlsen-Phelan shows two project configuration locations, .gitar/review and .gitar/rules. Review instructions can supply project context, while a rule can define a condition and an action. In the example, changes to pricing files trigger a check of customer-facing prices against the all-in rule.

He describes another possible rule: when a pull-request description contains a Slack link, an integration could use the conversation as review context. That is a product capability he describes, not an action performed by the reader of this article. Teams would need to configure whatever integrations and permissions they intend to use.

Customer-facing price check
SLIDE 3 / 6Customer-facing price checkThe review table contrasts displayed ticket prices with the amount charged at checkout.

The demonstrated review lists pricing fields and indicates which prices include the fee. It flags that the event-facing price is not all-in. Carlsen-Phelan notes that this is a business-rule mismatch rather than necessarily a syntax or type error. He leaves the second exercise unmerged.

05

CI diagnosis and optional automation

The third pull request fails a test against scanning the same ticket twice. The presenter enables auto-apply and auto-merge with required checks.

The third exercise demonstrates optional automation around a pull request that changes ticket scanning. The presenter enables GitHub auto-merge and configures a branch rule requiring status checks to pass. The workshop repository’s instructions contain the setup details for participants who want to reproduce the exercise.

The pull request has a failing test: a ticket that has already been scanned can be accepted again after the refactor. The review identifies the failed CI check and describes the missing protection. Carlsen-Phelan uses it to illustrate how an agent can inspect failure information instead of relying only on the diff.

Failed check on the ticket-scanning change
SLIDE 4 / 6Failed check on the ticket-scanning changeThe pull request is blocked by a failed test while showing the check-in code change.

He posts commands to enable auto-apply and auto-merge for this demonstration. He explains that auto-apply permits additional fix commits, and auto-merge is intended to act once the configured pipeline passes and the agent has no remaining findings. These actions depend on repository settings and the checks actually configured.

Enabling optional automated actions
SLIDE 5 / 6Enabling optional automated actionsThe demonstration comments enable auto-apply and auto-merge for the example pull request.

The presenter says Gitar will continue iterating and that automation can also be configured selectively by repository in its dashboard. During the recording, the tool is still working when he moves on; his explanation of the intended final merge should not be mistaken for an on-screen observation that the merge has already completed.

06

Cross-repository context and questions

The presenters describe functional validation, learned conventions and analysis across repositories and pull requests. They answer a question about connecting context.

Luttrell-Williams returns to describe advanced capabilities beyond the main exercise. She says functional validation can compare a change with an associated Jira ticket and that integrations can bring in context from tools including Jira, Linear and Slack, when configured.

She describes a knowledge feature intended to learn conventions and use reviewer feedback, including feedback on false positives. She says reviewer seniority affects weighting and that the interface shows where review rules came from. These are product descriptions from the presenters, not independently tested results.

The presenters distinguish cross-repository analysis, which looks for a change that can break code in another repository, from cross-PR analysis, which examines in-flight requests that may pass individually but interact badly together. They show a dashboard with findings and information about why a finding matters.

Cross-repository analysis dashboard
SLIDE 6 / 6Cross-repository analysis dashboardThe dashboard displays connected repositories and review findings.

In the closing questions, the presenters say Gitar can use MCP and other integrations for context. They distinguish project context from knowledge used to steer future review suggestions, and one presenter says he would need to follow up on the exact details of knowledge ingestion.

More from this channel

9 slides21:32EN
AI Engineer· Oct 29, 2026Shy Ruparel on AI agent reliability and recovery after failuresShy Ruparel · TemporalTemporal’s survey finds a gap between frequent agent use and dependable deployment. Ruparel demonstrates a synthetic payout workflow that resumes after workers stop.AgentsObservability & reliabilityAgent engineering+2
7 slides14:50EN
AI Engineer· Oct 29, 2026Cole Gannaway on code review, cross-repository risk and AI software factoriesCole Gannaway · QodoA one-line rename can pass local checks while affecting distant clients. Qodo’s Cole Gannaway explains how review context, shared rules and independent agents can expose those risks.Code review & verificationCoding agentsDeveloper workflows & testing+2
5 slides11:47EN
AI Engineer· Oct 29, 2026Revolut’s David Abitbol on self-hosting AI coding models at scaleDavid Abitbol · RevolutAbitbol shares engineering delivery trends, GPU-serving costs and latency tuning, then describes controls for operating the tools in a regulated bank.Local inferenceAI cost & performanceDeveloper productivity+2
5 slides16:34EN
AI Engineer· Oct 29, 2026Tariq Shaukat on Building Confidence in AI-Generated CodeTariq Shaukat · SonarSonar’s CEO draws on manufacturing quality practices to propose a way to govern and verify software produced with coding agents.Agent engineeringCode review & verificationSoftware engineering+2
4 slides21:47EN
AI Engineer· Oct 28, 2026Patrick Smyth on Open-Source Supply Chain Security in the AI EraPatrick Smyth · ChainguardChainguard’s Patrick Smyth discusses faster exploitation, malware in dependencies, zero-days and the expanding attack surface around developer workflows.CybersecurityOpen-source ecosystemsAI security+2
5 slides14:09EN
AI Engineer· Oct 28, 2026Taylor Luttrell-Williams on Finding Logic Vulnerabilities with SonarQube Hunter AgentTaylor Luttrell-Williams · SonarSonar’s Taylor Luttrell-Williams explains how a security agent maps a codebase, investigates logic flaws and presents validated findings alongside conventional scans.CybersecurityAI securityCode review & verification+2