Fix ArrayIterator and ObjectIterator bounds checking

Co-authored-by: ziflex <1607148+ziflex@users.noreply.github.com>

Author: Copilot

pkg/runtime/array_iter.go

@@ -19,7 +19,13 @@ func (iter *ArrayIterator) HasNext(_ context.Context) (bool, error) {
 }
 
 func (iter *ArrayIterator) Next(_ context.Context) (value Value, key Value, err error) {
+	if iter.pos >= iter.length {
+		return None, None, Error(ErrInvalidOperation, "no more elements")
+	}
+
+	value = iter.values.data[iter.pos]
+	key = NewInt(iter.pos)
 	iter.pos++
 
-	return iter.values.data[iter.pos-1], NewInt(iter.pos - 1), nil
+	return
 }

pkg/runtime/helpers.go

@@ -361,7 +361,12 @@ func ToFloat(ctx context.Context, input Value) (Float, error) {
 
 		res := ZeroFloat
 
-		for hasNext, err := iterator.HasNext(ctx); hasNext && err == nil; {
+		for {
+			hasNext, err := iterator.HasNext(ctx)
+			if !hasNext || err != nil {
+				break
+			}
+
 			val, _, err := iterator.Next(ctx)
 
 			if err != nil {
@@ -429,17 +434,22 @@ func ToInt(ctx context.Context, input Value) (Int, error) {
 
 		res := ZeroInt
 
-		for hasNext, err := iterator.HasNext(ctx); hasNext && err == nil; {
+		for {
+			hasNext, err := iterator.HasNext(ctx)
+			if !hasNext || err != nil {
+				break
+			}
+
 			item, _, err := iterator.Next(ctx)
 
 			if err != nil {
-				return ZeroInt, err
+				continue
 			}
 
 			i, err := ToInt(ctx, item)
 
 			if err != nil {
-				return ZeroInt, err
+				continue
 			}
 
 			res += i

pkg/runtime/object_iter.go

@@ -28,10 +28,13 @@ func (iter *ObjectIterator) HasNext(_ context.Context) (bool, error) {
 }
 
 func (iter *ObjectIterator) Next(_ context.Context) (Value, Value, error) {
-	iter.pos++
+	if iter.pos >= len(iter.keys) {
+		return None, None, Error(ErrInvalidOperation, "no more elements")
+	}
 
-	value := iter.data[iter.keys[iter.pos-1]]
-	key := String(iter.keys[iter.pos-1])
+	key := iter.keys[iter.pos]
+	value := iter.data[key]
+	iter.pos++
 
-	return value, key, nil
+	return value, String(key), nil
 }