more migrations

Author: zth

packages/@rescript/runtime/Stdlib_Array.res

@@ -99,7 +99,10 @@ let compare = (a, b, cmp) => {
     : compareFromIndex(a, b, 0, cmp, lenA)
 }
 
-@deprecated("Use `copyWithin` instead")
+@deprecated({
+  reason: "Use `copyWithin` instead",
+  migrate: Array.copyWithin(~start=0),
+})
 @send
 external copyAllWithin: (array<'a>, ~target: int) => array<'a> = "copyWithin"
 

packages/@rescript/runtime/Stdlib_Array.resi

@@ -101,7 +101,10 @@ arr->Array.copyAllWithin(~target=2) == [100, 101, 100, 101, 102]
 arr == [100, 101, 100, 101, 102]
 ```
 */
-@deprecated("Use `copyWithin` instead")
+@deprecated({
+  reason: "Use `copyWithin` instead",
+  migrate: Array.copyWithin(~start=0),
+})
 @send external copyAllWithin: (array<'a>, ~target: int) => array<'a> = "copyWithin"
 
 /**

packages/@rescript/runtime/Stdlib_DataView.res

@@ -3,7 +3,23 @@ type t
 
 @new
 external fromBuffer: (Stdlib_ArrayBuffer.t, ~byteOffset: int=?, ~length: int=?) => t = "DataView"
-@new external fromBufferToEnd: (Stdlib_ArrayBuffer.t, ~byteOffset: int) => t = "DataView"
+@deprecated({
+  reason: "Use `fromBuffer` instead",
+  migrate: DataView.fromBuffer(
+    %insert.unlabelledArgument(0),
+    ~byteOffset=%insert.labelledArgument("byteOffset"),
+  ),
+})
+@new
+external fromBufferToEnd: (Stdlib_ArrayBuffer.t, ~byteOffset: int) => t = "DataView"
+@deprecated({
+  reason: "Use `fromBuffer` instead",
+  migrate: DataView.fromBuffer(
+    %insert.unlabelledArgument(0),
+    ~byteOffset=%insert.labelledArgument("byteOffset"),
+    ~length=%insert.labelledArgument("length"),
+  ),
+})
 @new
 external fromBufferWithRange: (Stdlib_ArrayBuffer.t, ~byteOffset: int, ~length: int) => t =
   "DataView"

packages/@rescript/runtime/Stdlib_DataView.resi

@@ -61,7 +61,11 @@ DataView.fromBuffer(ArrayBuffer.make(16), ~byteOffset=4, ~length=8)
 DataView.fromBufferToEnd(buffer, ~byteOffset=4)
 ```
 */
-@deprecated("Use `fromBuffer` instead") @new
+@deprecated({
+  reason: "Use `fromBuffer` instead",
+  migrate: DataView.fromBuffer(~byteOffset=%insert.labelledArgument("byteOffset")),
+})
+@new
 external fromBufferToEnd: (Stdlib_ArrayBuffer.t, ~byteOffset: int) => t = "DataView"
 
 /**
@@ -77,7 +81,11 @@ See [`DataView`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Referen
 DataView.fromBufferWithRange(ArrayBuffer.make(16), ~byteOffset=2, ~length=8)
 ```
 */
-@deprecated("Use `fromBuffer` instead") @new
+@deprecated({
+  reason: "Use `fromBuffer` instead",
+  migrate: DataView.fromBuffer(),
+})
+@new
 external fromBufferWithRange: (Stdlib_ArrayBuffer.t, ~byteOffset: int, ~length: int) => t =
   "DataView"
 

packages/@rescript/runtime/Stdlib_TypedArray.res

@@ -14,13 +14,21 @@ type t<'a>
 @get external length: t<'a> => int = "length"
 
 @send external copyAllWithin: (t<'a>, ~target: int) => array<'a> = "copyWithin"
-@deprecated("Use `copyWithin` instead") @send
+@deprecated({
+  reason: "Use `copyWithin` instead",
+  migrate: TypedArray.copyWithin(),
+})
+@send
 external copyWithinToEnd: (t<'a>, ~target: int, ~start: int) => array<'a> = "copyWithin"
 @send
 external copyWithin: (t<'a>, ~target: int, ~start: int, ~end: int=?) => array<'a> = "copyWithin"
 
 @send external fillAll: (t<'a>, 'a) => t<'a> = "fill"
-@deprecated("Use `fill` instead") @send
+@deprecated({
+  reason: "Use `fill` instead",
+  migrate: TypedArray.fill(),
+})
+@send
 external fillToEnd: (t<'a>, 'a, ~start: int) => t<'a> = "fill"
 @send external fill: (t<'a>, 'a, ~start: int, ~end: int=?) => t<'a> = "fill"
 
@@ -43,12 +51,20 @@ external fillToEnd: (t<'a>, 'a, ~start: int) => t<'a> = "fill"
 @send external lastIndexOfFrom: (t<'a>, 'a, int) => int = "lastIndexOf"
 
 @send external slice: (t<'a>, ~start: int, ~end: int=?) => t<'a> = "slice"
-@deprecated("Use `slice` instead") @send
+@deprecated({
+  reason: "Use `slice` instead",
+  migrate: TypedArray.slice(),
+})
+@send
 external sliceToEnd: (t<'a>, ~start: int) => t<'a> = "slice"
 @send external copy: t<'a> => t<'a> = "slice"
 
 @send external subarray: (t<'a>, ~start: int, ~end: int=?) => t<'a> = "subarray"
-@deprecated("Use `subarray` instead") @send
+@deprecated({
+  reason: "Use `subarray` instead",
+  migrate: TypedArray.subarray(),
+})
+@send
 external subarrayToEnd: (t<'a>, ~start: int) => t<'a> = "subarray"
 
 @send external toString: t<'a> => string = "toString"

tests/tools_tests/src/expected/StdlibMigration_DataView.res.expected

@@ -0,0 +1,3 @@
+let a = DataView.fromBuffer(ArrayBuffer.make(8), ~byteOffset=2)
+let b = DataView.fromBuffer(ArrayBuffer.make(8), ~byteOffset=2, ~length=4)
+

tests/tools_tests/src/expected/StdlibMigration_StdlibArray.res.expected

@@ -1,3 +1,6 @@
+let ba1 = [1, 2, 3, 4, 5]->Array.copyWithin(~target=2, ~start=0)
+let ba2 = Array.copyWithin([1, 2, 3, 4, 5], ~target=2, ~start=0)
+
 let b1 = [1, 2, 3, 4]->Array.copyWithin(~target=0, ~start=2)
 let b2 = Array.copyWithin([1, 2, 3, 4], ~target=0, ~start=2)
 

tests/tools_tests/src/expected/StdlibMigration_TypedArray_Operations.res.expected

@@ -0,0 +1,7 @@
+let ta = Uint8Array.fromArray([1, 2, 3, 4])
+
+let a1 = ta->TypedArray.copyWithin(~target=1, ~start=2)
+let a2 = ta->TypedArray.fill(9, ~start=1)
+let a3 = ta->TypedArray.slice(~start=1)
+let a4 = ta->TypedArray.subarray(~start=1)
+

tests/tools_tests/src/migrate/StdlibMigration_DataView.res

@@ -0,0 +1,2 @@
+let a = DataView.fromBufferToEnd(ArrayBuffer.make(8), ~byteOffset=2)
+let b = DataView.fromBufferWithRange(ArrayBuffer.make(8), ~byteOffset=2, ~length=4)

tests/tools_tests/src/migrate/StdlibMigration_StdlibArray.res

@@ -1,3 +1,6 @@
+let ba1 = [1, 2, 3, 4, 5]->Array.copyAllWithin(~target=2)
+let ba2 = Array.copyAllWithin([1, 2, 3, 4, 5], ~target=2)
+
 let b1 = [1, 2, 3, 4]->Array.copyWithinToEnd(~target=0, ~start=2)
 let b2 = Array.copyWithinToEnd([1, 2, 3, 4], ~target=0, ~start=2)