refactor: simplify and make errors more explicit for Utils#frame method

Author: gaeljw

CHANGELOG.md

@@ -13,6 +13,8 @@ See also the [CHANGELOG](https://github.com/cucumber/cucumber-jvm/blob/master/CH
 
 ### Changed
 
+- [Internal] Fix `getLocation` wherever it was wrongly defined. ([#404](https://github.com/cucumber/cucumber-jvm-scala/issues/404))
+
 ### Deprecated
 
 ### Removed
@@ -24,6 +26,7 @@ See also the [CHANGELOG](https://github.com/cucumber/cucumber-jvm/blob/master/CH
 ### Changed
 
 - [Internal] Implement `HookDefinition.getHookType` and unimplement `Location.getLocation`. No impact expected for users.
+([#401](https://github.com/cucumber/cucumber-jvm-scala/issues/401))
 
 ## [8.29.0] (2025-07-15)
 

cucumber-scala/src/main/scala/io/cucumber/scala/Utils.scala

@@ -9,9 +9,15 @@ private[scala] object Utils {
     val frames = Thread.currentThread().getStackTrace
     val currentClass = self.getClass.getName
     // Note: the -1 check is here for Scala < 2.13 and objects
-    frames.reverse
-      .find(f => f.getClassName == currentClass && f.getLineNumber != -1)
-      .get
+    frames.findLast(f =>
+      f.getClassName == currentClass && f.getLineNumber != -1
+    ) match {
+      case Some(stackFrame) => stackFrame
+      case None             =>
+        throw new IllegalStateException(
+          s"Not able to find stack frame for $currentClass"
+        )
+    }
   }
 
 }