Fork me on GitHub
#cursive
<
2022-04-22
>
cfleming11:04:47

https://twitter.com/CursiveIDE/status/1517473008872755200 (hopefully 🎉, but I’ll wait for feedback before officially proclaiming it)

🎉 4
👍 2
imre12:04:31

Good man Colin, this is super later for you there

cfleming12:04:01

Yep, I’ll be off to bed shortly 🙂

imre12:04:17

let me give it a quick try

imre12:04:29

(you don't have to wait for it lol)

cfleming12:04:54

Haha, I’ll be here for a little bit anyway

imre12:04:20

Was it something stupid that broke it?

cfleming12:04:03

It’s an extremely weird one. I can’t figure out how it ever worked at all. And I’ve trawled through the history of all the related files, and nothing changed in them recently. So I’m still not sure that I’ve fixed it, although the thing I fixed was definitely broken.

imre12:04:48

Wow. Was it resolution-related or settings-management related?

imre12:04:16

I installed this from disk, loaded up a project that showed this problem before and had my symbols resolved properly, so the first try succeeded

cfleming12:04:54

It was a confusing combination of the two. When the settings for the resolution customisations are loaded, there’s a map which wasn’t being initialised, which goes from the short name (e.g. defmutation) to a set of vars which have that short name (e.g Pathom’s defmutation, Fulcro’s defmutation, and any others you might have configured). That short name cache was correctly initialised for all the built in extensions, but not for the customisations, which is why they didn’t work. However, that code has been like that forever, so I don’t know how the customisations (at least the ones relating to indexing, e.g. resolve as def/defn) ever worked at all.

cfleming12:04:49

Re-customising did set that cache correctly, but that would have been lost on every restart. So this should have lost everyone’s customisations on every restart, which clearly wasn’t happening.

cfleming12:04:57

Great news!

cfleming12:04:59

So, there’s something I still don’t understand, which is frustrating.

imre12:04:18

wow that does sound weird

imre12:04:30

I'll keep trying and let you know

imre12:04:41

if I see any more issues

cfleming12:04:40

There are definitely some IntelliJ indexing bugs in there too, probably in 2022.1 since I haven’t seen them previously. The IDE would occasionally just stop resolving anything at all, and e.g. not find any namespaces to navigate to. That’s definitely a platform issue, but I’ll just have to hope they can fix that, I don’t have enough for a coherent bug report yet.

cfleming12:04:45

Great, thanks.

aratare13:04:47

upgrading now

aratare13:04:19

@U0567Q30W Can confirm it's no longer throwing errors and all ops (navigate, quick doc, etc.) are looking fine.

cfleming13:04:36

That is definitely 🎉

🎉 1
aratare13:04:27

Just wanna say, really appreciate you staying up late trying to get this fixed 🙂

cfleming13:04:07

No problem! I am very glad to hopefully have it fixed.

AJ Jaro15:04:30

Yes, thank you! I'll pull this in and check it out but I certainly appreciate the efforts here as well

cfleming22:04:05

No problem, I’m sorry this has been painful for so long.