Session watcher not respecting reticulate delayed load, and other issues about vscode-r

[mkoohafkan]

This seems to be caused by the setting r.session.watchGlobalEnvironment. Setting this to false results in import("re", delay_load = TRUE) working as expected. It also resolved the continuous error throw issue in my last paragraph.

It seems that the session watcher is not working quite right with reticulate connections to Python modules and objects.

[mkoohafkan]

The issue is encountered in the VSCode-R session watcher function, and specifically this line triggers the evaluation. This code gets evaluated because is_promise() and is_active() both return FALSE for a delayed-load module (and thus the "else" part of the if-else statement is evaluated).

Perhaps the most straightforward solution is to add a condition in inspect_env()for checking if the object is a python.builtin.module and/or python.builtin.object object, and if so to avoid inspecting it.

[ElianHugh]

Thanks for the detailed writeup @mkoohafkan.

Your solution definitely works, and I'm happy to write a PR soon to remedy this. It would be nice to have non-delayed modules still be evaluated, but I haven't found a way of inspecting that from within R.