ion-js

ion-js copied to clipboard

Add a BigInt build target

3

Now that we've migrated to JSBI, we should make it possible for users to build a version of `ion-js` that uses `BigInt`s instead of `JSBI` values.

zslayton

Ion-test-driver complained that behavior changed for the commit 12cd9a011ad7aaa588764cc678d42d49216fc65a created by `desaikd`. Refer to the [workflow](https://github.com/amzn/ion-js/actions/runs/2000741133) for more details.

github-actions[bot]

Allow users to provide type hints to Value.from()

1

Currently, `Value.from` converts all JS arrays into `ion.dom.List`s and all `Uint8Array`s into `ion.dom.Blob`s. Users hoping to create `ion.dom.SExpression` or `ion.dom.Clob` have to instantiate them manually, which is much more tedious....

zslayton

LazyDom

3

Rather than materializing the complete tree eagerly, allow users to create a `LazyValue` that only parses the input buffer as much as is necessary to satisfy the method call at...

zslayton

`equals` is intended to provide intuitive value-based comparisons. Right now, any comparison between a DOM `Integer` and any other numeric type (besides `JSBI`) is false. This behavior is appropriate for...

zslayton

v4.2.0 added support for structs with repeated field names, e.g. ```js { foo: 1, foo: true, foo: bar } ``` To preserve backwards compatibility, existing struct accessor methods (`get`, `fields`)...

zslayton

https://github.com/amzn/ion-js/blob/master/src/dom/Sequence.ts#L8 says `Sequence`s extend Array, and since `dom.List` and `dom.SExpression` extends `Sequence`, I would expect them to support `map` for iterating the elements. However, this (https://runkit.com/embed/n9x2izs3st66) throws in 4.1.0.

fnmdx111

Ambiguous load/loadAll semantics when dealing with broken input

3

Docs on `load` (similarly `loadAll`) says > If no value is found, load will return null. However, if I feed a piece of broken input into `load`, I get exception...

fnmdx111

Currently, the (not yet released) `.equals()` method for `dom.Value` subclasses is strict by default. It checks for Ion equivalence between the two values being compared. JS values cannot be compared...

zslayton

It seems like read support for LST appends isn't implemented in ion-js. When trying to read through LST appends, the `binaryReader` throws an error stating ` Can't step in to...

desaikd