atom-ide-purescript icon indicating copy to clipboard operation
atom-ide-purescript copied to clipboard

Completion suggests outdated entries

Open rintcius opened this issue 7 years ago • 6 comments

After moving some code around, the IDE keeps suggesting entries from the old location instead of the new one.

Also full build doesn't resolve it. However, after restarting Atom the new locations are picked up.

rintcius avatar Sep 02 '17 06:09 rintcius

Did you try "Restart IDE server" (or psc-ide or whatever it is referenced as)? That should sort it without requiring a restart.

Have to ask @kRITZCREEK what needs to happen for the old module to be discarded.

nwolverson avatar Sep 02 '17 15:09 nwolverson

No didn't try to restart psc-ide - thanks for the tip!

Not !00% sure if I remember correctly, but I think that apart from not discarding the old module it also doesn't pick up the new one.

rintcius avatar Sep 09 '17 15:09 rintcius

The reason the old module doesn't disappear is because the externs file remains in the output directory. I don't see a good way for us to detect which files in the output directory are still valid or which are leftovers. Happy to take suggestions :D

I think the filewatcher doesn't pick up new files, but that problem should be fixed once editor-mode lands.

kritzcreek avatar Sep 09 '17 15:09 kritzcreek

I think I did clean the output dir. Could it be not discarding the outdated modules for the same reason as to why it doesn't pick up new ones?

Editor-mode - sounds intriguing :) What spot do I need to watch to know when it lands?

rintcius avatar Sep 09 '17 15:09 rintcius

Editor-mode - sounds intriguing :) What spot do I need to watch to know when it lands? 0.12.0

Could it be not discarding the outdated modules for the same reason as to why it doesn't pick up new ones?

Yeah, it doesn't because file deletion events are even less reliable than modification events. We'll have to make do with reset/reload/restart for now.

kritzcreek avatar Sep 09 '17 16:09 kritzcreek

editor-mode has now landed (maybe there's something else to do, didn't really test yet). No ideas about the general moving code question

nwolverson avatar Jan 01 '18 22:01 nwolverson