Revision 0.34.15 (#1148)

* Use Math.Trunc to avoid 32-bit coersion for Integer

* Version

* ChangeLog

changelog/0.34.0.md

@@ -1,4 +1,6 @@
 ### 0.34.0
+- [Revision 0.34.15](https://github.com/sinclairzx81/typebox/pull/1148)
+  - [1147](https://github.com/sinclairzx81/typebox/issues/1147) Fix incorrect truncation for integers that exceed 32-bit values in Value.Convert
 - [Revision 0.34.14](https://github.com/sinclairzx81/typebox/pull/1140)
   - [1139](https://github.com/sinclairzx81/typebox/issues/1139) Update TypeCompiler Check for Promise (use instanceof Promise over Thenable check)
 - [Revision 0.34.13](https://github.com/sinclairzx81/typebox/pull/1124)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sinclair/typebox",
-  "version": "0.34.14",
+  "version": "0.34.15",
   "description": "Json Schema Type Builder with Static Type Resolution for TypeScript",
   "keywords": [
     "typescript",

src/value/convert/convert.ts

@@ -125,7 +125,7 @@ function TryConvertNumber(value: unknown) {
   return IsStringNumeric(value) ? parseFloat(value) : IsValueTrue(value) ? 1 : IsValueFalse(value) ? 0 : value
 }
 function TryConvertInteger(value: unknown) {
-  return IsStringNumeric(value) ? parseInt(value) : IsNumber(value) ? value | 0 : IsValueTrue(value) ? 1 : IsValueFalse(value) ? 0 : value
+  return IsStringNumeric(value) ? parseInt(value) : IsNumber(value) ? Math.trunc(value) : IsValueTrue(value) ? 1 : IsValueFalse(value) ? 0 : value
 }
 function TryConvertNull(value: unknown) {
   return IsString(value) && value.toLowerCase() === 'null' ? null : value

test/runtime/value/convert/integer.ts

@@ -76,4 +76,27 @@ describe('value/convert/Integer', () => {
     const result = Value.Convert(Type.Integer(), value)
     Assert.IsEqual(result, [])
   })
+  // ----------------------------------------------------------
+  // https://github.com/sinclairzx81/typebox/issues/1147
+  // ----------------------------------------------------------
+  it('Should convert large Integer 1', () => {
+    const N = 1738213389080
+    const R = Value.Convert(Type.Integer(), N)
+    Assert.IsEqual(R, N)
+  })
+  it('Should convert large Integer 2', () => {
+    const N = 1738213389080.5555
+    const R = Value.Convert(Type.Integer(), N)
+    Assert.IsEqual(R, 1738213389080)
+  })
+  it('Should convert large Integer 3', () => {
+    const N = '1738213389080'
+    const R = Value.Convert(Type.Integer(), N)
+    Assert.IsEqual(R, 1738213389080)
+  })
+  it('Should convert large Integer 3', () => {
+    const N = '1738213389080.555'
+    const R = Value.Convert(Type.Integer(), N)
+    Assert.IsEqual(R, 1738213389080)
+  })
 })