Samsung/ONE-vscode

Extension not activated on local installation

dayo09 opened this issue · 6 comments

dayo09 commented

What?

The current main branch runs on debugging session very well, but it doesn't work on installation.

dayo09 commented

Searching the problematic PR

git reset --hard 0.4.0 && \
rm one-vscode-0.4.0.vsix && \
vsce package && \
code --uninstall-extension Samsung.one-vscode && \
code --install-extension one-vscode-0.4.0.vsix

git reset --hard a001610 && \
rm one-vscode-0.4.0.vsix && \
vsce package && \
code --uninstall-extension Samsung.one-vscode && \
code --install-extension one-vscode-0.4.0.vsix
version status
ed89c8e Add result variable to JobCommand (#1573) fail
90ea0b1 [MPQEditor] Implement createFromDefaultExplorer (#1527) fail
b96f6a8 [MPQEditor] Register MPQEditor (#1528) fail
8d8fed5 [MPQEditor] Implement message processing (#1526) ok
cc00550 [MPQEditor] Implement webviewPanel init. (#1525)
edef879 [MPQEditor] Implement providing model nodes info. (#1524) ok
1ce4be5 [MPQEditor] Implement 'createMPQJson' (#1521)
a0132fa [MPQEditor] Implement 'updateDocumentBy' (#1520)
0ed653a [MPQEditor] Implement 'createDefaultMPQ' (#1519)
08972a3 [MPQEditor] Implement 'findMPQName' (#1518) ok
a001610 [docs] Add docs for visual tensor view (#1504) ok
0.4.0 ok
dayo09 commented

b96f6a8 [MPQEditor] Register MPQEditor (#1528) looks problematic.

Dev host

  console.log("MPQEditorProvider.register");
  MPQEditorProvider.register(context);
  console.log("MPQEditorProvider.register done");
[2023. 9. 13. 오전 11:29:49][extension.ts][info] MPQEditorProvider.register
[2023. 9. 13. 오전 11:29:49][extension.ts][info] MPQEditorProvider.register done
  • But it works well on development host.

Local host

Not showing anything in OUTPUT

It doesn't prints out activation log.
It seems the whole activation function didn't run.

Code version?

I upgraded my vscode from 1.81.1 -> 1.82.1 but it remains the same.

dayo09 commented

MPQEditor module

/cc @stamalakhov

With MPQEditor related modules commented out in extensions.ts, the extension works as normally at the latest main branch.

NOTE that the symptom looks as if it 'activate' function doesn't work at the beginning, because the first line Logger.info(tag, "one-vscode activate OK"); doesn't prints out. ONE-vscode extension only shows its viewer frames, which can be done only by loadingpackage.json, not extension.ts.

With MPQEditor related modules commented out in extensions.ts, the extension works as normally at the latest main branch.

@dayo09
Thank you. I'll try to fix it.

@dayo09
The problem code is

glob.sync(...)

at MPQEditor.ts. With this line commented ONE-vscode extension can be successfully loaded and used. I'm investigating how it can be fixed.

@dayo09
As a quick fix please see #1638, seems like it's more safe as readdirSync is used a lot in the extension.