8000 GitHub - yippee-fun/strict_ivars: Make Ruby raise a NameError if you read an undefined instance variable.
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

yippee-fun/strict_ivars

Repository files navigation

Strict Ivars

If you reference an undefined method, constant or local varaible, Ruby will helpfully raise a NameError. But reference an undefined instance variable and Ruby just returns nil. This can lead to all kinds of bugs — many of which can lay dormant for years before surprising you with an unexpected outage, data breach or data loss event.

Strict Ivars solves this by making Ruby raise a NameError any time you read an undefined instance varaible. It’s enabled with two lines of code in your boot process, then it just works in the background and you’ll never have to think about it again. Strict Ivars has no known false-positives or false-negatives.

It’s especially good when used with Literal and Phlex, though it also works with regular Ruby objects and even ERB templates, which are actually pretty common spots for undefined instance variable reads to hide since that’s the main way of passing data to ERB.

When combined with Literal, you can essentially remove all unexpected nils. Literal validates your inputs and Strict Ivars ensures you’re reading the right instance variables.

Note

JRuby and TruffleRuby are not currently supported.

Setup

Strict Ivars should really be used in apps not libraries. Though you could definitely use it in your library’s test suite to help catch issues in the library code.

Install the gem by adding it to your Gemfile and running bundle install. You’ll probably want to set it to require: false here because you should require it manually at precisely the right moment.

gem "strict_ivars", require: false

Now the gem is installed, you should require and initialize the gem as early as possible in your boot process. Ideally, this should be right after Bootsnap is set up. In Rails, this will be in your boot.rb file.

require "strict_ivars"

You can pass an array of globs to StrictIvars.init as include: and exclude:

StrictIvars.init(include: ["#{Dir.pwd}/**/*"], exclude: ["#{Dir.pwd}/vendor/**/*"])

This example include everything in the current directory apart from the ./vendor folder (which is where GitHub Actions installs gems).

If you’re setting this up in Rails, your boot.rb file should look something like this.

ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../Gemfile", __dir__)

require "bundler/setup" # Set up gems listed in the Gemfile.
require "bootsnap/setup" # Speed up boot time by caching expensive operations.

require "strict_ivars"

StrictIvars.init(include: ["#{Dir.pwd}/**/*"], exclude: ["#{Dir.pwd}/vendor/**/*"])

If you’re using Bootsnap, you should clear your bootsnap cache by deleting the folder tmp/cache/bootsnap.

How does it work?

When Strict Ivars detects that you are loading code from paths its configured to handle, it quickly looks for instance variable reads and guards them with a defined? check.

For example, it will replace this:

def example
  foo if @bar
end

...with something like this:

def example
  foo if (defined?(@bar) ? @bar : raise)
end

The replacement happens on load, so you never see this in your source code. It’s also always wrapped in parentheses and takes up a single line, so it won’t mess up the line numbers in exceptions.

Writes:

Strict Ivars doesn’t apply to writes, since these are considered the authoritative source of the instance variable definitions.

@foo = 1

Or-writes:

Or-writes are considered an authoritative definition, not a read.

@foo ||= 1

And-writes:

And-writes are considered an authoritative definition, not a read.

@foo &&= 1

Common mistakes

Implicitly depending on undefined instance variables

def description
  return @description if @description.present?
  @description = get_description
end

This example is relying on Ruby’s behaviour of returning nil for undefiend instance variables, which is completely unnecessary. Instead of using present?, we could use defined? here.

def description
  return @description if defined?(@description)
  @description = get_description
end

Alternatively, as long as get_description doesn’t return nil and expect us to memoize it, we could use an “or-write” ||=

def description
  @description ||= get_description
end

Rendering instance variables that are only set somtimes

It’s common to render an instance variable in an ERB view that you only set on some controllers.

<div data-favourites="<%= @user_favourites %>"></div>

The best solution to this to always set it on all controllers, but set it to nil in the cases where you don’t have anything to render. This will prevent you from making a typo in your views.

Alternatively, you could update the view to be explicit about the fact this ivar may not be set.

<div data-favourites="<%= (@user_favourites ||= nil) %>"></div>

Better yet, add a defined? check:

<% if defined?(@user_favourites) %>
  <div data-favourites="<%= @user_favourites %>"></div>
<% end %>

Performance

Boot performance

Using Strict Ivars will impact startup performance since it needs to process each Ruby file you require. However, if you are using Bootsnap, the processed RubyVM::InstructionSequences will be cached and you probably won’t notice the incremental cache misses day-to-day.

Runtime performance

In my benchmarks on Ruby 3.4 with YJIT, it’s difficult to tell if there is any performance difference with or without the defined? guards at runtime. Sometimes it’s about 1% faster with the guards than without. Sometimes the other way around.

On my laptop, a method that returns an instance varible takes about 15ns and a method that checks if an instance varible is defined and then returns it takes about 15ns. All this is to say, I don’t think there will be any measurable runtime performance impact, at least not in Ruby 3.4.

Dynamic evals

There is a small additional cost to dynamically evaluating code via eval, class_eval, module_eval, instance_eval and binding.eval. Dynamic evaluation usually only happens at boot time but it can happen at runtime depending on how you use it.

Stability

Strict Ivars has 100% line and branch coverage and there are no known false-positives, false-negatives or bugs.

Uninstall

Becuase Strict Ivars only ever makes your code safer, you can always back out without anything breaking.

To uninstall Strict Ivars, first remove the require and initialization code from wherever you added it and then remove the gem from your Gemfile. If you are using Bootsnap, there’s a good chance it cached some pre-processed code with the instance variable read guards in it. To clear this, you’ll need to delete your bootsnap cache, which should be in tmp/cache/bootsnap.

About

Make Ruby raise a NameError if you read an undefined instance variable.

Topics

Resources

License

Code of conduct

Stars

Watchers

Forks

Packages

No packages published

Contributors 3

  •  
  •  
  •  

Languages

0