bundle install does not run the gems hooks

[ankitsny]

I am using rusky gem in my project for git hooks. and rusky has post-install script to create the git hooks.

but when I do bundle install then hooks are not getting created
but when I do gem install rusky it runs the postscript and creates the git-hooks.

so, for now, I am running rusky install explicitly, which is a little annoying.
do let me know how to invoke postscript of gems or any script after bundle install

[deivid-rodriguez]

Hi @anks333! Please provide steps so we can reproduce this issue in our environment. Thanks!

[ankitsny]

# Gemfile

ruby '>= 2.5.0', engine: 'jruby', engine_version: '9.2.6.0'

# Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
gem 'rails', '~> 5.2.3'
# Use jdbcpostgresql as the database for Active Record
gem 'activerecord-jdbcpostgresql-adapter', '~> 52.2'
# Use Puma as the app server
gem 'puma', '~> 3.11'
gem 'figaro'
# Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
gem 'jbuilder', '~> 2.5'
# Use Redis adapter to run Action Cable in production
# gem 'redis', '~> 4.0'
# Use ActiveModel has_secure_password
# gem 'bcrypt', '~> 3.1.7'

# Use ActiveStorage variant
# gem 'mini_magick', '~> 4.8'

# Use Capistrano for deployment
# gem 'capistrano-rails', group: :development

# Use Rack CORS for handling Cross-Origin Resource Sharing (CORS), making cross-origin AJAX possible
# gem 'rack-cors'
gem 'redis'
gem 'statsd-ruby'
gem 'slim'

group :development do
  gem 'listen', '>= 3.0.5', '< 3.2'
  gem 'rusky'
end

# Windows does not include zoneinfo files, so bundle the tzinfo-data gem
gem 'tzinfo-data', platforms: [:mingw, :mswin, :x64_mingw, :jruby]

[ankitsny]

this is my gem file when I run bundle install
it installs all the gems, but it does not run the post-install scripts of the gems.

[ankitsny]

if I do gem install rusky, it installs the gem and also executes the post-install scripts of the rusky gem.

in the context of node.js, modules have postinstall scripts, so when I run npm install it installs the module and runs the postinstall scripts.

[deivid-rodriguez]

I think this is actually a rubygems issue. Post install hooks only fire for gems that are already installed when ruby boots and requires rubygems:

$ # no rusky gem installed on my system

$ gem install rusky
Fetching rusky-0.2.5.gem
Successfully installed rusky-0.2.5
Parsing documentation for rusky-0.2.5
Installing ri documentation for rusky-0.2.5
Done installing documentation for rusky after 0 seconds
1 gem installed

$ # no post_install hooks were fired

$ gem install rusky
rusky > can't find .git directory, so skipping rusky process
Successfully installed rusky-0.2.5
Parsing documentation for rusky-0.2.5
Done installing documentation for rusky after 0 seconds
1 gem installed

$ # post_install hooks were now fired because rusky was already installed

[ankitsny]

if so, then if I run bundle install more than once then it should invoke the hooks

[deivid-rodriguez]

Yep, if bundle install actually installs something new, yeah.

[rreinhardt9]

From my understanding, it appears that the Rusky gem is using rubygems plugins in a way that is not intended 🤔 here is what makes me think that.

Plugins are "Extensions that use the RubyGems plugin API" and so are ways that you can extend and modify the behavior of Rubygems in general and not just for one specific gem. One way that you can share these plugins would be to gemify it and share it with others. Once the gem is installed, rubygems would find the plugin (and any hooks, commands, etc. it adds) during future installs.

Rusky attempts to use the plugin API to add it's own post install behavior https://github.com/ohbarye/rusky/blob/3edc4ddae4172a5bf5407226d2fe1f71434d64d7/lib/rubygems_plugin.rb#L6-L8 but from what I understand, this would not have the expected behavior. As we were seeing in the example above, installing the gem the first time will add this plugin behavior for subsequent installs, but on the initial install would not run (because during that install, the plugin was not installed or active yet).

If this understanding is correct, then the rubygems plugin API is working as expected and the observed issue is due to the way the Rusky gem is attempting to use it to run it's own post install behavior in a way the plugin API is not intended to be used.

Does this understanding seem correct to others that might have more context on the rubygems plugin API?

[simi]

You're right @rreinhardt9.

[deivid-rodriguez]

I understand that the rusky gem might be misusing this feature, but even in the case that the feature was being correctly used, we could still maybe do better?

Take yard for example, which provides a rubygems plugin to install the YARD documentation for a gem right after it is installed. Should yard be able to install YARD documentation for itself? It'd probably be nice? Making it work would be a matter of adding the plugins of the gem that was just installed to the list of loaded plugins, to give after_install hooks the chance to run for the newly installed gem.

So, instead of:

$ gem install yard
Fetching yard-0.9.25.gem
Successfully installed yard-0.9.25
1 gem installed

$ gem install yard:0.9.24
Fetching yard-0.9.24.gem
Successfully installed yard-0.9.24
Building YARD (yri) index for yard-0.9.24...
Installing YARD documentation for yard-0.9.24...
Done installing documentation for yard after 9 seconds
1 gem installed

we would get

$ gem install yard
Fetching yard-0.9.25.gem
Successfully installed yard-0.9.25
Building YARD (yri) index for yard-0.9.25...
Installing YARD documentation for yard-0.9.25...
Done installing documentation for yard after 9 seconds
1 gem installed

$ gem install yard:0.9.24
Fetching yard-0.9.24.gem
Successfully installed yard-0.9.24
Building YARD (yri) index for yard-0.9.24...
Installing YARD documentation for yard-0.9.24...
Done installing documentation for yard after 9 seconds
1 gem installed

By the way, we should review the https://guides.rubygems.org/plugins/ guide, since rubygems no longer uses Gem.find_files to load plugins since #3108.

[simi]

I think you can create "extension" to run custom script during gem install. Rakefile extension would be enough here.

[deivid-rodriguez]

Yes, you can.

In my opinion though, that's unrelated to whether we should consider this as an issue or not. I was personally surprised by this behaviour when I first found it, and only made sense after seeing the implementation. I believe we should either make this case work as I suggested, or improve our documentation about it.