Computed behavior changed between 0.8.1 and 0.9.0

[Peter Siska]

Given this example:

const one = atom<string | undefined>(undefined);
const two = computed(one, (value: string | undefined) => !!value);

and this code:

console.log('one', one.get());
console.log('two', two.get());

in 0.8.1, the output / values were

undefined
false

in 0.9.0, the output / values are

undefined
undefined

Is this expected?

Example projects are here:

• 0.8.1: https://stackblitz.com/edit/vitejs-vite-jjawn6?file=src%2Fmain.ts
• 0.9.1: https://stackblitz.com/edit/vitejs-vite-m18s4j?file=src%2Fmain.ts

[Andrey Sitnik]

@TrySound could it be related with your changes?

[Bogdan Chadkin]

Yes, notify always forced recomputing before. Now check fails when undefined is compared to out of range index. Need to add array length check additionally.

Good catch @peschee, thanks

Fixing

[Bogdan Chadkin]

#202

[Andrey Sitnik]

The fix was released in 0.9.1

[Peter Siska]

Awesome, thanks for a quick fix!