experimental-vm-modules bug introduced in v0.134.0

[jakedetels]

experimental-vm-modules bug introduced in v0.134.0

[lgrammel]

Thanks for the bug report. I suspect it is because I switched bundling to tsup. Can you share some code that can help me reproduce the issue?

[lgrammel]

@jakedetels I did some digging and it might be related to jest specifically (see https://jestjs.io/docs/ecmascript-modules ). Are you using jest by any chance? Does this also happen outside of tests?

[lgrammel]

@jakedetels can you try https://github.com/lgrammel/modelfusion/releases/tag/v0.135.1 ?

It should fix the issue. If you still run into it, please re-open.

[jakedetels]

@lgrammel, thanks for the quick attention on this. I tested with 0.135.1, and confirm that it fixed the previously reported issue and my tests are now passing again. However, the patch release introduced a bug/side effect related to the schema of the event objects that are emitted (which I leverage for logging purposes). In 133, event.input was an array of messages (see first screenshot). Now in 0.135.1, to access the same array that was at event.input earlier, code must be revised as event.input.input (see second screenshot). How do you advise proceeding? Is the new event object schema locked in, or will you restore to the pre-v0.135.1 version?

0.133.0

0.135.1

[lgrammel]

@lgrammel, thanks for the quick attention on this. I tested with 0.135.1, and confirm that it fixed the previously reported issue and my tests are now passing again. However, the patch release introduced a bug/side effect related to the schema of the event objects that are emitted (which I leverage for logging purposes). In 133, event.input was an array of messages (see first screenshot). Now in 0.135.1, to access the same array that was at event.input earlier, code must be revised as event.input.input (see second screenshot). How do you advise proceeding? Is the new event object schema locked in, or will you restore to the pre-v0.135.1 version?

0.133.0

0.135.1

@jakedetels this is expected (a result of introducing prompt functions). In general, the event structure is an area that will likely change a bit more going forward. Are you using the events / input information or is this caused by snapshot tests?

[jakedetels]

@lgrammel, I've been using the event object within a logging service I'm using. (Issue is unrelated to snapshot testing.) No worries if the event schema will change a bit further, but it would be nice to lock things in at some point soon, since the event object is part of MF's public API for function observers.

[lgrammel]

@jakedetels totally - it's good to know that you are using the event, so I'll keep that in mind when there are updates.