ProfessionShoppingList/Core.lua:1736: attempt to index global 'ProfessionsFrame' (a nil value) [string "@ProfessionShoppingList/Core.lua"]:1736: in function CreateTradeskillAssets'
[string "@ProfessionShoppingList/Core.lua"]:3224: in function ?' [string "@ProfessionShoppingList/Core.lua"]:20: in function <ProfessionShoppingList/Core.lua:18> [string "=[C]"]: in function OpenTradeSkill'
[string "@Molinari/hack.lua"]:61: in function <Molinari/hack.lua:54>

Locals:
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index global 'ProfessionsFrame' (a nil value)"
app =

Tell the author of CanIMogIt to check if ProfessionsFrame exists before referencing it (or better yet wait for Blizzard_Professions addon to load), that's on them not Molinari.

While the quote states CanIMogIt it is the exact same error for this addon.

I don't know about CIMI, but Caerdon was also running into the same issue as PSL for the same reason:
We let the code relating to the profession windows run when the game tells us the event TRADE_SKILL_SHOW is fired, which happens when a profession window is opened. For some reason, having Molinari enabled makes that event fire even when the profession window hasn't been loaded yet, which then causes the error you've reported.

I've made the same change to my code as Caerdon has, by letting that code run when the Blizzard_Professions addon has been loaded as suggested by Molinari's author, which should hopefully resolve this issue.

Could you download the latest version of PSL from GitHub and confirm that's the case? When I get home from work I'll download Molinari and test it myself, as well. :)

Alright, no need for testing on your part - I've verified the error occuring before 932d147 and that it no longer occurs after. It'll be fixed with the next release, thanks for letting me know! :D