Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Subject: Bug Report: "Maximum call stack size exceeded" when connecting Obsidian Vault #66

Open
Romuloadv opened this issue Feb 2, 2025 · 2 comments

Comments

@Romuloadv
Copy link

Image

Description of the issue:
I am trying to connect my Obsidian vault to Smart Connect, but I receive the following error:

vbnet
Copiar
Editar
deep_merge_no_overwrite error (opts): Maximum call stack size exceeded
deep_merge_no_overwrite error (adapter): Maximum call stack size exceeded
Steps to reproduce:

Installed Smart Connect
Configured it to connect to my Obsidian vault located at: C:\Second brain
Started the connection process and received the error
System information:

OS: Windows 10/11
Smart Connect version: (Specify the exact version)
Obsidian Vault Encryption: Enabled via Obsidian Sync (paid plan)
Troubleshooting attempts:
✅ Reinstalled Smart Connect
✅ Deleted cache and reconfigured from scratch
✅ Ran Smart Connect as Administrator
✅ Attempted to disable Obsidian Sync temporarily

Additional context:

My Obsidian vault is encrypted using Obsidian Sync (end-to-end encryption).
The vault files are stored locally and are accessible, but Smart Connect fails when trying to merge configurations.
It seems like a recursion issue with the deep_merge_no_overwrite function.
Question:
Is there a specific configuration needed to make Smart Connect work with Obsidian Sync encrypted vaults?
Any workaround or patch available to fix this error?

Thank you for your support!

@Romuloadv
Copy link
Author

See more:

start Objectpaths: settings: "C:\Users\advro\AppData\Roaming\smart-connect\user_data\settings.json"user_data: "C:\Users\advro\AppData\Roaming\smart-connect\user_data"[[Prototype]]: Object__dirname: "C:\Users\advro\AppData\Local\Programs\smart-connect\resources\app.asar"[[Prototype]]: Objectconstructor: ƒ Object()hasOwnProperty: ƒ hasOwnProperty()isPrototypeOf: ƒ isPrototypeOf()propertyIsEnumerable: ƒ propertyIsEnumerable()toLocaleString: ƒ toLocaleString()toString: ƒ toString()valueOf: ƒ valueOf()defineGetter: ƒ defineGetter()defineSetter: ƒ defineSetter()lookupGetter: ƒ lookupGetter()lookupSetter: ƒ lookupSetter()proto: (...)get proto: ƒ proto()set proto: ƒ proto()
_sc.mjs:81742 Loading settings...
_sc.mjs:81790 settings Proxy(Object)[[Handler]]: ObjectdeleteProperty: ƒ deleteProperty(target2, property)get: ƒ get(target2, property)set: ƒ set(target2, property, value)[[Prototype]]: Object[[Target]]: Object[[IsRevoked]]: false
_sc.mjs:81814 init_env Proxy(Object)[[Handler]]: Object[[Target]]: Object[[IsRevoked]]: false
2_sc.mjs:60114 deep_merge_no_overwrite error (adapter): Maximum call stack size exceeded
deep_merge_no_overwrite @ _sc.mjs:60114
Show 1 more frame
Show less
_sc.mjs:81655 update-main-status Object v1.3.14

@brianpetro
Copy link
Owner

Hi @Romuloadv and thanks for the screenshot.

There isn't any special requirement for the e2e since Smart Connect works with files locally (the e2e only applies to Obsidian server communication).

Regarding the warning in the screenshot, does Smart Connect appear to be working otherwise? Are any other errors occurring when you use the Smart Connect GPTs?

🌴

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants