hotfix for select(index)

[maksimbykov]

Fix #356

[maksimbykov]

@TarekRaafat please review this when you'll have time, thanks❤️

[AndrewElans]

Any news on this bug?

[AndrewElans]

I guess the reason why this TypeError: Cannot set properties of undefined (setting 'event') is fired is because select(index) is called when function init (ctx) is not yet complete.

I have fixed in my case by placing select(index) in the results event which seems to run after init is finished.

Before I tried to call select(index) on window.load and other events but in vain. Also in init event the same error was generated. I guess that even if the init event is fired, the init promise may still be not returned.

[TarekRaafat]

Hello @maksimbykov and @AndrewElans,

I apologize for my very delayed reply, and thanks to you and @AndrewElans for following up.

The bug you identified is valid, and I can see how calling select(index) before ctx.feedback is populated causes the TypeError. Good catch on tracing it back to #356.

The current fix of calling start(this, ' ') with a space as query works as a workaround, but I believe we need a more solid approach that does not depend on triggering a search to populate the context. I will address this properly as part of the upcoming v11 release, where the initialization and lifecycle flow is undergoing a broader overhaul.

I will be publishing the v11 release plan soon, so stay connected for updates.

Thanks for your valuable contribution and for your patience, much appreciated!

Have a nice day, cheers! :)