Run action in its own Ruby environment

This action has its own Ruby dependencies that may or may not be
included in the consumer Gemfile. To avoid colliding dependencies, we
should run within the same Ruby setup as the consumer application, but
run our own `bundle install`, running anything that redirects to the
consumer application using `Bundler.with_unbundled_env` to opt out of
our Gemfile.

Author: neilvcarvalho

.github/workflows/integration.yml

@@ -54,7 +54,7 @@ jobs:
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           dry-run: "true"
-          working-directory: /tmp/test-app
+          rails-root: /tmp/test-app
 
       - name: Assert all outdated packages appear in dry-run output
         run: |

action.yml

@@ -32,29 +32,27 @@ inputs:
     description: "Git author email for commits."
     required: false
     default: "github-actions[bot]@users.noreply.github.com"
-  working-directory:
+  rails-root:
     description: "Directory containing the Rails app to run against. Useful when the app lives in a subdirectory or for integration testing."
     required: false
     default: "."
 
 runs:
   using: "composite"
   steps:
-    - name: Set up Ruby
-      uses: ruby/setup-ruby@v1
-      with:
-        bundler-cache: true
-        ruby-version: ${{ inputs.ruby-version }}
-
     - name: Run importmap-update
       shell: bash
-      working-directory: ${{ inputs.working-directory }}
+      working-directory: ${{ github.action_path }}
       env:
+        BUNDLE_GEMFILE:           ${{ github.action_path }}/Gemfile
+        RAILS_ROOT:               ${{ inputs.rails-root }}
         GITHUB_TOKEN:             ${{ inputs.github-token }}
         GITHUB_REPOSITORY:        ${{ github.repository }}
         INPUT_CONFIG_FILE:        ${{ inputs.config-file }}
         IMPORTMAP_BASE_BRANCH:    ${{ inputs.base-branch }}
         IMPORTMAP_DRY_RUN:        ${{ inputs.dry-run }}
         IMPORTMAP_AUTHOR_NAME:    ${{ inputs.author-name }}
         IMPORTMAP_AUTHOR_EMAIL:   ${{ inputs.author-email }}
-      run: ${{ github.action_path }}/exe/importmap-update 2>&1 | tee "${IMPORTMAP_RUN_LOG:-/dev/null}"
+      run: |
+        bundle install
+        bundle exec exe/importmap-update 2>&1 | tee "${IMPORTMAP_RUN_LOG:-/dev/null}"

exe/importmap-update

@@ -162,7 +162,7 @@ when :run
     exit 2
   end
 
-  runner = Importmap::Update::Commands::ShellRunner.new
+  runner = Importmap::Update::Commands::ShellRunner.new(cwd: ENV.fetch("RAILS_ROOT", "."))
   gh = Importmap::Update::GitHubClient.new(repo: repo, token: token)
   git = Importmap::Update::GitClient.new(
     runner: runner,

lib/commands.rb

@@ -47,8 +47,10 @@ def initialize(cwd: nil, env: nil)
         def run(*argv)
           opts = {}
           opts[:chdir] = @cwd if @cwd
-          stdout, stderr, status = Open3.capture3(@env, *argv, **opts)
-          Result.new(stdout: stdout, stderr: stderr, exit_code: status.exitstatus)
+          Bundler.with_unbundled_env do
+            stdout, stderr, status = Open3.capture3(@env, *argv, opts)
+            Result.new(stdout: stdout, stderr: stderr, exit_code: status.exitstatus)
+          end
         end
 
         # Raises on non-zero exit. Use when you have no recovery strategy