Consider .class a source extension during search

headius · headius · commit 8a7e30fe4aff · 2025-09-14T10:01:54.000+05:30
The logic here was ported from CRuby, which only considers .rb to be a source extension. This causes our logic to think a filename like "foo.class" is not a source file and try to load it as either "foo.class.rb" or "foo.class.class". This is the cause of jruby#8758 and prevents explicitly loading precompiled Ruby sources using the filename with .class extension. The fix here modifies the source extension check to also consider other configured source extensions, so that when .class searching is enabled, we will properly treat `require "foo.class"` as a source file. Fixes jruby#8758
diff --git a/core/src/main/java/org/jruby/runtime/load/LibrarySearcher.java b/core/src/main/java/org/jruby/runtime/load/LibrarySearcher.java
@@ -250,7 +250,12 @@ public static SuffixType getSuffixTypeForLoad(final String[] fileHolder) {
     }
 
     public static boolean isSourceExt(String file) {
-        return file.endsWith(".rb");
+        for (Suffix suffix : Suffix.SOURCES) {
+            if (file.endsWith(suffix.extension)) {
+                return true;
+            }
+        }
+        return false;
     }
 
     public static boolean isLibraryExt(String file) {

Original file line number	Diff line number	Diff line change
`@@ -250,7 +250,12 @@ public static SuffixType getSuffixTypeForLoad(final String[] fileHolder) {`
`250`	`250`	`}`
`251`	`251`
`252`	`252`	`public static boolean isSourceExt(String file) {`
`253`		`- return file.endsWith(".rb");`
	`253`	`+ for (Suffix suffix : Suffix.SOURCES) {`
	`254`	`+ if (file.endsWith(suffix.extension)) {`
	`255`	`+ return true;`
	`256`	`+ }`
	`257`	`+ }`
	`258`	`+ return false;`
`254`	`259`	`}`
`255`	`260`
`256`	`261`	`public static boolean isLibraryExt(String file) {`