Skip to content

Instantly share code, notes, and snippets.

Show Gist options
  • Save GlenCrawford/73039dd6604931cf2e35dade1bba4fa5 to your computer and use it in GitHub Desktop.
Save GlenCrawford/73039dd6604931cf2e35dade1bba4fa5 to your computer and use it in GitHub Desktop.
reinteractive Blog Post: Detecting unused routes with Rails 7.1

When looking at upcoming major releases of projects like Ruby on Rails, it can be easy to get distracted by the big, flashy features, such as Hotwire and encrypted attributes in Rails 7.0. However, sometimes I find myself getting more excited by smaller, more modest additions in upcoming releases. This is the case with the recent merge of a PR to Rails that will form part of Rails 7.1: Add routes --unused option to detect extraneous routes by Gannon McGibbon.

We all know the rails routes command which lists all the routes in your app, detailing their name, HTTP method, path, controller and action. But in a large app with lots of routes that might have seen a rewrite or two or a few years of churn, how do you know which routes are still valid (where "valid" means the controller and action that the route points to exists)?

Introducing the new --unused option (full command: rails routes --unused).

From an implementation point-of-view, essentially all it does is iterate over Rails.application.routes and select the ones that are unused, where "unused" is defined by this pleasingly succinct method:

def unused?
  controller_class_missing? || (action_missing? && template_missing?)
end

The output will be in the same familiar format as the main rails routes command which you have seen a million times before. Also note that we can filter the resulting routes to those that match a specific pattern with the -g (grep) option.

The motivation discussed in the pull request seems to be to improve the performance of your application's boot time, routing time and memory use, since routes can be expensive to draw in terms of both time and memory. While all this is certainly true, preventing the drawing of unused routes is only going to provide a noticeable performance improvement in large apps. For me, the real benefits are in preventing routes from being left behind during refactors that remove their companion controller and/or action, and forcing developers to be more specific about their routing code, in particular when using high-level declarations like resources.

When adding a new controller to your app, it is very easy to simply add, for example, resources :posts to your routes file even when you might really only plan on initially implementing the index and show actions, possibly thinking that you might eventually use all of the seven routes that such a line will draw. While not being specific about the routes you are actually using is handy in that it allows you to come back and implement the remaining actions without having to edit your routes file at a later date, it's possible that you're not going to, meaning that (in this scenario) you just left behind five unused routes in your app. By periodically checking for unused routes and narrowing them down to only the ones that are pointing to implemented controller actions (by specifying the only or except options) you prevent these unused routes from accumulating and causing performance issues, dead code and confusion over time.

After all, an unused route isn't really all that different from a method that is no longer called (or never was). Once Rails 7.1 is out, use rails routes --unused to keep on top of these and prevent them from accumulating over time. I will be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment