@fable-org/fable-library-js 2.5.0 → 2.5.1

package/Array.js

@@ -1,7 +1,7 @@
 import { disposeSafe, getEnumerator, copyToArray, defaultOf, Exception } from "./Util.js";
 import { Helpers_allocateArrayFromCons } from "./Native.js";
 import { setItem as setItem_1, item as item_2 } from "./Array.js";
-import { value as value_2, map as map_1, defaultArg, some } from "./Option.js";
+import { value as value_2, defaultArg, some } from "./Option.js";
 import { min as min_1, max as max_1 } from "./Double.js";
 import { SR_notEnoughElements, SR_inputMustBeNonNegative, SR_inputSequenceEmpty, SR_Arg_ArgumentOutOfRangeException, SR_indexOutOfBounds } from "./Global.js";
 import { Operators_IsNull } from "./FSharp.Core.js";
@@ -183,8 +183,9 @@ export function where(predicate, array) {
     return array.filter(predicate);
 }
 export function indexOf(array, item_1, start, count, eq) {
+    let option_1 = undefined;
     const start_1 = defaultArg(start, 0) | 0;
-    const end$0027 = defaultArg(map_1((c) => ((start_1 + c) | 0), count), array.length) | 0;
+    const end$0027 = defaultArg((option_1 = count, (option_1 != null) ? (start_1 + value_2(option_1)) : undefined), array.length) | 0;
     const loop = (i_mut) => {
         loop: while (true) {
             const i = i_mut;

package/CHANGELOG.md

@@ -1,5 +1,5 @@
 ---
-last_commit_released: 6568f35ece227fc30ce84111c8d2e975b09bac00
+last_commit_released: 619d7afddbaf25108e819abaaa9d8a6503c457d6
 updaters:
   - package.json:
       file: package.json
@@ -15,6 +15,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## 2.5.1 - 2026-07-24
+
+### 🐞 Bug Fixes
+
+* *(js/ts,python)* Inline Option/ValueOption combinators to avoid closures and allocations (#4836) ([61cac646](https://github.com/fable-compiler/Fable/commit/61cac646e6385d82247082dbb90a98b1646560de))
+
+<strong><small>[View changes on Github](https://github.com/fable-compiler/Fable/compare/6568f35ece227fc30ce84111c8d2e975b09bac00..619d7afddbaf25108e819abaaa9d8a6503c457d6)</small></strong>
+
 ## 2.5.0 - 2026-07-15
 
 ### 🚀 Features

package/List.js

@@ -253,12 +253,13 @@ export function FSharpList__get_Item_Z524259A4(xs, index) {
             const i = i_mut, xs_1 = xs_1_mut;
             const matchValue = xs_1.tail;
             if (matchValue != null) {
+                const t = value_1(matchValue);
                 if (i === index) {
                     return xs_1.head;
                 }
                 else {
                     i_mut = (i + 1);
-                    xs_1_mut = value_1(matchValue);
+                    xs_1_mut = t;
                     continue loop;
                 }
             }

package/Map.js

@@ -702,11 +702,15 @@ export function MapTreeModule_foldSectionOpt(comparer, lo, hi, f, m, x) {
                         return x_3;
                     }
                 }
-                else if ((comparer.Compare(lo, MapTreeLeaf$2__get_Key(m2)) <= 0) && (comparer.Compare(MapTreeLeaf$2__get_Key(m2), hi) <= 0)) {
-                    return f_1(MapTreeLeaf$2__get_Key(m2), MapTreeLeaf$2__get_Value(m2), x_1);
-                }
                 else {
-                    return x_1;
+                    const cLoKey_1 = comparer.Compare(lo, MapTreeLeaf$2__get_Key(m2)) | 0;
+                    const cKeyHi_1 = comparer.Compare(MapTreeLeaf$2__get_Key(m2), hi) | 0;
+                    if ((cLoKey_1 <= 0) && (cKeyHi_1 <= 0)) {
+                        return f_1(MapTreeLeaf$2__get_Key(m2), MapTreeLeaf$2__get_Value(m2), x_1);
+                    }
+                    else {
+                        return x_1;
+                    }
                 }
             }
             else {

package/MutableMap.js

@@ -272,7 +272,7 @@ export function Dictionary__set_Item_5BDDA1(this$, k, v) {
             setItem(getItemFromDict(this$.hashMap, matchValue[1]), matchValue[2], [k, v]);
         }
         else {
-            const value = void (getItemFromDict(this$.hashMap, matchValue[1]).push([k, v]));
+            void (getItemFromDict(this$.hashMap, matchValue[1]).push([k, v]));
         }
     }
     else {
@@ -286,7 +286,7 @@ export function Dictionary__Add_5BDDA1(this$, k, v) {
             throw ArgumentException_$ctor_Z721C83C5(format("An item with the same key has already been added. Key: {0}", k));
         }
         else {
-            const value = void (getItemFromDict(this$.hashMap, matchValue[1]).push([k, v]));
+            void (getItemFromDict(this$.hashMap, matchValue[1]).push([k, v]));
         }
     }
     else {

package/MutableSet.js

@@ -178,7 +178,7 @@ export function HashSet__Add_2B595(this$, k) {
             return false;
         }
         else {
-            const value = void (getItemFromDict(this$.hashMap, matchValue[1]).push(k));
+            void (getItemFromDict(this$.hashMap, matchValue[1]).push(k));
             return true;
         }
     }

package/Option.js

@@ -70,30 +70,13 @@ export function toArray(opt) {
 export function defaultArg(opt, defaultValue) {
     return (opt != null) ? value(opt) : defaultValue;
 }
-export function defaultArgWith(opt, defThunk) {
-    return (opt != null) ? value(opt) : defThunk();
-}
 export function orElse(opt, ifNone) {
     return opt == null ? ifNone : opt;
 }
-export function orElseWith(opt, ifNoneThunk) {
-    return opt == null ? ifNoneThunk() : opt;
-}
-export function filter(predicate, opt) {
-    return (opt != null) ? (predicate(value(opt)) ? opt : undefined) : opt;
-}
+// Only used by Replacements.Util.fs's curry/uncurry helpers, not by `Option.map` itself.
 export function map(mapping, opt) {
     return (opt != null) ? some(mapping(value(opt))) : undefined;
 }
-export function map2(mapping, opt1, opt2) {
-    return (opt1 != null && opt2 != null) ? mapping(value(opt1), value(opt2)) : undefined;
-}
-export function map3(mapping, opt1, opt2, opt3) {
-    return (opt1 != null && opt2 != null && opt3 != null) ? mapping(value(opt1), value(opt2), value(opt3)) : undefined;
-}
-export function bind(binder, opt) {
-    return opt != null ? binder(value(opt)) : undefined;
-}
 export function tryOp(op, arg) {
     try {
         return some(op(arg));

package/Seq.js

@@ -277,7 +277,7 @@ export function Enumerator_unfold(f, state) {
     return Enumerator_FromFunctions$1_$ctor_58C54629(() => {
         if (curr != null) {
             const x = value_1(curr)[0];
-            const st = value_1(curr)[1];
+            value_1(curr)[1];
             return x;
         }
         else {
@@ -286,7 +286,7 @@ export function Enumerator_unfold(f, state) {
     }, () => {
         curr = f(acc);
         if (curr != null) {
-            const x_1 = value_1(curr)[0];
+            value_1(curr)[0];
             const st_1 = value_1(curr)[1];
             acc = st_1;
             return true;

package/package.json

@@ -3,7 +3,7 @@
     "private": false,
     "type": "module",
     "name": "@fable-org/fable-library-js",
-    "version": "2.5.0",
+    "version": "2.5.1",
     "description": "Core library used by F# projects compiled with fable.io",
     "author": "Fable Contributors",
     "license": "MIT",