markedmondson

Claude Code Review Assessment — Feb 2026

We reviewed 4 PRs where Claude's automated reviews were flagged as weak or wrong. Here's what we found and what we changed.

What Went Wrong

1. Suggestions that would break code (PR #42573)

• Suggested JS optional chaining on the left side of an assignment (?.value = 'none') — that's a syntax error
• Suggestion blocks omitted existing attributes from elements, so clicking "Apply suggestion" would silently drop data-behavior

markedmondson

module FeaturePolicy
  extend ActiveSupport::Concern

  class_methods do
    def policy_access_klass(klass)
      self.policy_access_klass = klass
    end

    included do
      class_attribute :policy_access_klass,

markedmondson

module SessionHelper
  extend ActiveSupport::Concern

  class ::SessionsBypassController < ActionController::Base
    def show
      session[:user_id] = params[:user_id]
      session[:team_id] = params[:team_id]
      session[:tenant] = Apartment::Tenant.current

markedmondson

#!/bin/bash

# Requires sudo apt-get install libfaketime
# Run with bash ./scraps/random_tests.sh

# You may or may not need these...
# export DYLD_FORCE_FLAT_NAMESPACE=1
# export DYLD_INSERT_LIBRARIES=/path/to/libfaketime.1.dylib

for i in {1..50}

markedmondson

module InkHelper
  def ink_container(*styles, &block)
    styles = ["container"].concat(Array(styles))

    content_tag(:table, class: styles.join(" ")) do
      ink_block do
        content_tag(:tr) do
          content_tag(:td) do
            capture { yield }
          end

markedmondson

How to install PhantomJS on Ubuntu

Version: 1.9.8

Platform: x86_64

First, install or update to the latest system software.

sudo apt-get update
sudo apt-get install build-essential chrpath libssl-dev libxft-dev

markedmondson

state_counts = [
	OpenStruct.new(name: "BC", country: "Canada", count: 5),
	OpenStruct.new(name: "British Columbia", country: "CA", count: 65),
	OpenStruct.new(name: "Ontario", country: "CA", count: 15),
	OpenStruct.new(name: "QC", country: nil, count: 25),
	OpenStruct.new(name: "Oregon", country: "USA", count: 35),
	OpenStruct.new(name: "WA", country: "United States", count: 45),
	OpenStruct.new(name: "CA", country: nil, count: 55),
	OpenStruct.new(name: "WA", country: "Australia", count: 65),
	OpenStruct.new(name: "WA", country: nil, count: 100)

markedmondson

state_counts = [
	OpenStruct.new(name: "BC", country: "Canada", count: 5),
	OpenStruct.new(name: "British Columbia", country: "CA", count: 65),
	OpenStruct.new(name: "Ontario", country: "CA", count: 15),
	OpenStruct.new(name: "QC", country: nil, count: 25),
	OpenStruct.new(name: "Oregon", country: "USA", count: 35),
	OpenStruct.new(name: "WA", country: "United States", count: 45),
	OpenStruct.new(name: "CA", country: nil, count: 55),
	OpenStruct.new(name: "WA", country: "Australia", count: 65)
]

markedmondson

state_counts = [
	OpenStruct.new(name: "BC", country: "Canada", count: 5),
	OpenStruct.new(name: "Ontario", country: "CA", count: 15),
	OpenStruct.new(name: "QC", country: nil, count: 25),
	OpenStruct.new(name: "Oregon", country: "USA", count: 35),
	OpenStruct.new(name: "WA", country: "United States", count: 45),
	OpenStruct.new(name: "CA", country: nil, count: 55),
	OpenStruct.new(name: "WA", country: "Australia", count: 65)
]

markedmondson

canada    = OpenStruct.new(name: "Canada", iso: "CA")
usa       = OpenStruct.new(name: "United States", iso: "US")
australia = OpenStruct.new(name: "Australia", iso: "AU")

all_countries = [
	canada,
	usa,
	australia
]