ljcoder2015 / obsidian-sheet-plus

obsidian spreadsheets solutions
Apache License 2.0
143 stars 6 forks source link

sheet plus overwriting the 'process' object #34

Closed scooper4711 closed 3 months ago

scooper4711 commented 3 months ago

Describe the bug I originally opened https://github.com/Vinzent03/obsidian-git/issues/761 because it was failing to execute process.cwd(). As it turns out, the process object looks one way without the sheets plus plugin loaded, and looks completely different with the plugin installed.

To Reproduce Steps to reproduce the behavior:

  1. prior to installing sheets plus, in the inspector, execute process.constructor.name.
  2. notice that the value returned is process.
  3. install and activate sheets plus.
  4. re-execute that code.
  5. notice that the signature of process has changed, and the result returned by process.constructor.name is now Object.

Expected behavior Process is a node variable, and should keep its class and associated methods.

Device (please complete the following information):

Obsidian (please complete the following information):

scooper4711 commented 3 months ago

Looks like you are both importing different process definitions.

scooper4711 commented 3 months ago

I've gone back and tested this, and the process object exists prior to importing your plugin. Completely clean vault, the process object looks like:

process {version: 'v20.14.0', versions: {…}, arch: 'arm64', platform: 'darwin', release: {…}, …}

I install sheet plus, and the process object now looks like:

{env: {…}}
ljcoder2015 commented 3 months ago

fix 1.3.5