Fix Hermes get original function name

.gitignore

@@ -1,3 +1,4 @@
 .DS_Store
 target
 Cargo.lock
+.idea

src/hermes.rs

@@ -108,10 +108,13 @@ impl SourceMapHermes {
 
         // Find the closest mapping, just like here:
         // https://github.com/facebook/metro/blob/63b523eb20e7bdf62018aeaf195bb5a3a1a67f36/packages/metro-symbolicate/src/SourceMetadataMapConsumer.js#L204-L231
-        let (_mapping_idx, mapping) =
-            greatest_lower_bound(&function_map.mappings, &token.get_src(), |o| {
-                (o.line, o.column)
-            })?;
+        // Mappings use 1-based index for lines, and 0-based index for cols, as seen here:
+        // https://github.com/facebook/metro/blob/f2d80cebe66d3c64742f67259f41da26e83a0d8d/packages/metro/src/Server/symbolicate.js#L58-L60
+        let (_mapping_idx, mapping) = greatest_lower_bound(
+            &function_map.mappings,
+            &(token.get_src_line() + 1, token.get_src_col()),
+            |o| (o.line, o.column),
+        )?;
         function_map
             .names
             .get(mapping.name_index as usize)

src/ram_bundle.rs

@@ -416,7 +416,7 @@ impl<'a> SplitRamBundleModuleIter<'a> {
     }
 }
 
-impl<'a> Iterator for SplitRamBundleModuleIter<'a> {
+impl Iterator for SplitRamBundleModuleIter<'_> {
     type Item = Result<(String, SourceView, SourceMap)>;
 
     fn next(&mut self) -> Option<Self::Item> {
@@ -458,7 +458,7 @@ pub fn is_ram_bundle_slice(slice: &[u8]) -> bool {
     slice
         .pread_with::<RamBundleHeader>(0, scroll::LE)
         .ok()
-        .map_or(false, |x| x.is_valid_magic())
+        .is_some_and(|x| x.is_valid_magic())
 }
 
 /// Returns "true" if the given path points to the startup file of a file RAM bundle

tests/test_hermes.rs

@@ -19,6 +19,12 @@ fn test_react_native_hermes() {
         ("input.js", 2, 0, None)
     );
     assert_eq!(sm.get_original_function_name(11857), Some("<global>"));
+
+    assert_eq!(
+        sm.lookup_token(0, 11947).unwrap().to_tuple(),
+        ("module.js", 1, 4, None)
+    );
+    assert_eq!(sm.get_original_function_name(11947), Some("foo"));
 }
 
 #[test]