diff --git a/lib/irb/completion.rb b/lib/irb/completion.rb index 597dcdaa81..812ebb09e7 100644 --- a/lib/irb/completion.rb +++ b/lib/irb/completion.rb @@ -64,15 +64,18 @@ module IRB end def self.retrieve_files_to_require_from_load_path - @@files_from_load_path ||= retrieve_gem_and_system_load_path.flat_map { |path| + @@files_from_load_path ||= retrieve_gem_and_system_load_path.map { |path| begin Dir.glob("**/*.{rb,#{RbConfig::CONFIG['DLEXT']}}", base: path) rescue Errno::ENOENT [] end - }.uniq.map { |path| - path.sub(/\.(rb|#{RbConfig::CONFIG['DLEXT']})\z/, '') - } + }.inject([]) { |result, names| + shortest, *rest = names.map{ |n| n.sub(/\.(rb|#{RbConfig::CONFIG['DLEXT']})\z/, '') }.sort + result.unshift(shortest) if shortest + result.concat(rest) + result + }.uniq end def self.retrieve_files_to_require_relative_from_current_dir diff --git a/test/irb/test_completion.rb b/test/irb/test_completion.rb index c534301bc6..c3462f4196 100644 --- a/test/irb/test_completion.rb +++ b/test/irb/test_completion.rb @@ -68,6 +68,11 @@ module TestIRB end end + def test_complete_require_library_name_first + candidates = IRB::InputCompletor::CompletionProc.("'cgi", "require ", "") + assert_equal candidates.first, "'cgi" + end + def test_complete_require_relative candidates = Dir.chdir(__dir__ + "/../..") do IRB::InputCompletor::CompletionProc.("'lib/irb", "require_relative ", "")