Sync URI::Source with RubyLSP

[amomchilov]

Motivation

Fixes #2471

1. Fix incorrect shim for URI::Source#fragment
2. Remove untyped call to alias_method
3. Sync this file with its copy in RubyLSP (see also: Sync URI::Source with Tapioca ruby-lsp#3891)
   • Adding a missing delete_prefix("/") in the path parsing code
   • Add a missing / in the URI::Source#to_s output
4. Prevent definition (and registration) of this URI class if RubyLSP has already done it.
   • This fixes the constant redefinition issue.

[amomchilov]

• Sync URI::Source with RubyLSP #2475 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[vinistock]

The test failures are legit. In our source URIs, the gem version after the name is optional. When it's missing, it indicates whatever version is currently activated in a Bundle context.

# Explicit version
source://gem_name/1.2.3/lib/bar.rb#1

# Implicit
source://gem_name//lib/bar.rb#1

[Morriar]

Do we still need this since host and fragment are defined in the RBI?

[amomchilov]

Yes, but for a different reason now. See the comment on lines 29-30

I renamed this class from Tapioca::SourceURI to URI::Source (same as what RubyLSP would define), so these are redefinitions of those methods now

[Morriar]

Is this this error? 🤔 https://sorbet.run/#%23%20typed%3A%20true%0A%0Amodule%20URI%0A%20%20class%20Source%20%3C%20%3A%3AURI%3A%3AFile%0A%20%20%20%20def%20bar%3B%20end%0A%20%20%20%20%0A%20%20%20%20def%20foo%3B%20end%0A%20%20end%0Aend%0A%0Amodule%20URI%0A%20%20class%20Source%20%3C%20%3A%3AURI%3A%3AFile%0A%20%20%20%20alias_method%28%3Afoo%2C%20%3Abar%29%0A%20%20end%0Aend

[amomchilov]

Yep

I approved the wrong PR