Ignored Workspaces displaying about vscord

[xhayper]

i will figure it out soon

[nick22985]

Doing more testing this is related to ignored workspaces when user is remote ssh into a vscode instance

#36 This prob be fixed apart of this issue

[xhayper]

cd4fabb might hopefully fix this, i will push the update out soon

[xhayper]

nevermind, can you please give me your settings so i can test it

[nick22985]

Hey @xhayper,
Here are my settings. There also looks to be a regression. Now it only shows idle and not the ignoreWorkspacesText. When going to a new file it then shows the repo. This only happens when you are ssh'ed inot a separate machine coding also

{
	"rpc.buttonInactiveLabel": "Check Out My Github",
	"rpc.buttonInactiveUrl": "https://github.com/nick22985",
	"rpc.appName": "Visual Studio Code",
	"rpc.ignoreOrganizations": ["IgnoredOrg"],
	"rpc.ignoreWorkspaces": ["Repo1", "Repo2", "Repo3", "Repo4", "Repo5", "Repo6"],
	"rpc.ignoreWorkspacesText": "[Redacted]",
	"rpc.buttonEnabled": true,
	"rpc.enabled": true,
	"rpc.checkIdle": true,
	"rpc.resetElapsedTimeAfterIdle": true
}

[xhayper]

ah shoot, i dont know how ssh coding work, this might be s trouble for me to debug

[nick22985]

@xhayper I did some digging into this. What it appears to be is that dataClass.workspaceFolder returns undefined

vscord/src/activity.ts

Line 105 in b3c4ae0

if (dataClass.workspaceFolder && "uri" in dataClass.workspaceFolder) {

which checks the const uri = window?.activeTextEditor?.document.uri; which is undefined

vscord/src/data.ts

Line 91 in b3c4ae0

const uri = window?.activeTextEditor?.document.uri;

Causing the check for the isExcluded to fall over and not run

vscord/src/activity.ts

Line 112 in b3c4ae0

if (isWorkspaceExcluded && dataClass.workspaceFolder && dataClass.workspaceFolder.name) {

[xhayper]

this is really REALLY a problem, because that's the only way we know how to find what folder we are on

[xhayper]

currently trying to think of a way we can work around this

[nick22985]

@xhayper
Maybe this is actually an issue with the vscode npm plugin

My one thought which would be kinda quick and dirty would be to regex whatever value is the string for the workspace folder. But I assume this will cause more issues for other things so prob better finding a better soloution to this.

My next idea would be trying to use workspace.workspaceFolders as this gets back the folders. The issue here being it contanins a array of worspace folders. Unsure how you can have multiple workspace folders in vscode so I cant rlly play around with this and see if the array is in some sort of order

https://gyazo.com/77aabefb8508751f752bbfb609912dc8/draw

[xhayper]

the problem with this is we need to use the current folder, so we have to figure out how the index affect the array before we use .workspaceFolders

[nick22985]

Yea that why I was saying idk how to get multiple workspaces to see if there is some sort of index or order to it

[nick22985]

So been thinking about this and poking around the vscode API docs. We can figure out if something is a remote by looking at the remoteName in the env. If it is not a remote it returns undefined. Thinking that we could encoperate a check around this to get the current file and folders a different way. Unsure atm just writing this here for now. When I get some time will see if I can figure out a good way of doing this

[xhayper]

that sounds like a great idea, unfortunately, i am working on a rewrite of config system and it's might for me right now

i will work on it tomorrow after the config system, you can help me by writing the description of each config :P

[nick22985]

Happy to help in anyway, not sure what u want me to do wtih the docs if you wanna talk about it more. Explain it here or u can add me on discord if u would like nick22985#1228

[xhayper]

should be fixed in dd7e1b6