`rails_rbi:jobs` raise unknown prefix error

[boiiiboi]

Describe the bug:

After running rails_rbi:jobs, following error occured.

-- Generate sigs for ActiveJob ActiveStorage::BaseJob --
rails aborted!
unknown prefix

Steps to reproduce:

1. Run rails_rbi:jobs

Expected behavior:

Expect to generate sigs successfully.

Versions:

• Ruby: ruby 3.2.0 (2022-12-25 revision a528908271) [arm64-darwin22]
• Rails: Rails 7.0.4.1
• Sorbet: Sorbet typechecker 0.5.10626 git 1204e682b8e83a919d8ec8a0753022f241f393ee debug_symbols=true clean=1
• Sorbet-Rails: 0.7.34

[supersam654]

I'm running into this too while upgrading to Ruby 3.2.

• Ruby: ruby 3.2.1 (2023-02-08 revision 31819e82c8) [x86_64-linux]
• Rails: Rails 7.0.4.1
• Sorbet: Sorbet typechecker 0.5.10595 git cb7684fdb328e6e6d5f3f2687a3073f680f97df7 debug_symbols=true clean=0
• Sorbet-Rails: 0.7.34

After poking around a bit, it looks like the "unknown prefix" error is coming from Parlour:

https://github.com/AaronC81/parlour/blob/baddfc5d9dbfcde15aebe88b2bff89b200ecb137/lib/parlour/rbi_generator/parameter.rb#L75-L82

Parlour appears to be struggling with "forward everything arguments" (https://bugs.ruby-lang.org/issues/16253) which ActiveJob uses for perform_now and perform_later. Notably, this syntax was introduced in Ruby 2.7.

https://github.com/rails/rails/blob/23e0345fe900dfd7edd6e8e5a7a6bd54b2a7d2ed/activejob/lib/active_job/enqueuing.rb#L28

https://github.com/rails/rails/blob/23e0345fe900dfd7edd6e8e5a7a6bd54b2a7d2ed/activejob/lib/active_job/execution.rb#L17

The method Parlour is generating RBI for is re-constructed in sorbet-rails around:

https://github.com/chanzuckerberg/sorbet-rails/blob/447ecbe4a2586ce8aa9055290d571fe34a55e27f/lib/sorbet-rails/sorbet_utils.rb#L23-L69

That method is generating invalid Parlour Parameters:

# run with `rails runner repro.rb`

class Foo
  def m(...); end
end

method_def = Foo.instance_method(:m)
params = SorbetRails::SorbetUtils.parameters_from_method_def(method_def)
params.each { |param| pp param }

Prints:

Warning: unable to parse the source of m
#<Parlour::RbiGenerator::Parameter:0x00007fc43a24d758
 @default=nil,
 @kind=:double_splat,
 @name="**",
 @type="T.untyped">
#<Parlour::RbiGenerator::Parameter:0x00007fc43a24d1b8
 @default=nil,
 @kind=:double_splat,
 @name="****",
 @type="T.untyped">
#<Parlour::RbiGenerator::Parameter:0x00007fc43a24cf38
 @default=nil,
 @kind=:block,
 @name="&&",
 @type="T.untyped">

This fix (which I'm hoping to have a PR for today if I don't get terribly sidetracked) appears to be updating parameters_from_method_def so it generates Parlour Parameters with valid names.

[simoleone]

This also happens when alias and alias_method are used because recent versions of ruby seem to use forward everything on the alias.