Recursion limit exceeded about obsidian-export HOT 8 CLOSED

shouldn't each note be processed only once?

The current implementation is somewhat naive in that regard.

This function is responsible for parsing an Obsidian Markdown file:

If this encounters the ![[embed]] syntax, it calls embed_file:

embed_file, in turns, calls the parse_obsidian_note function from above again if it encounters another Markdown file:

This is what allows a loop to be created.

However, I'm already passing through a context struct to hold information about the note that is being processed:

Technically, I would be able to keep track of all files that we've processed up to the root note, which would allow implementing the exact behavior you mention (skip a note if it's already been processed). I may play around with that and implement this (possibly as an optional mode) somewhere over the next couple of weeks. Stay tuned 😄

from obsidian-export.

Hey there @cristianvasquez!

To make sure I understand correctly:

1. You have a note Transcluding.md
2. This note includes itself (it contains ![[Transcluding]])
3. You run into the Recursion limit exceeded error when trying to export a vault with this note.

Is that correct? If so, this is by design. A note which includes itself would otherwise lead to infinite recursion (which would eventually cause the program to abort with a panic).

I'm not sure off-hand how Obsidian itself handles this (I think it embeds about 7 iframes and then just stops?) but I chose to make this an explicit error to make a note like this very obvious.

Would you prefer to see a different behavior here?

from obsidian-export.

I don't know what should be expected. But this behavior forced me to change my notes, otherwise, I cannot export.

This happens if there is any 'closed circuit' in notes that expand.

By the way, why it expands the notes?

from obsidian-export.

Perhaps some sort of .gitignore but for the export would allow ignoring,

Or a fail silently option?

from obsidian-export.

This happens if there is any 'closed circuit' in notes that expand.

By this, do you mean note A which transcludes note B, where note B in turn transcludes note A again?

By the way, why it expands the notes?

I'm afraid I don't understand this question, could you elaborate?

Perhaps some sort of .gitignore but for the export would allow ignoring,

This is already supported, see https://github.com/zoni/obsidian-export#ignoring-files 😄

Or a fail silently option?

I think I'd like to understand the use-case slightly better before adding functionality, but I wouldn't be opposed to implementing something like that.

from obsidian-export.

Oh!... the ignore option was what I looked for :)

Regarding the question, I still don't get why a Recursion limit happened :) shouldn't each note be processed only once?

from obsidian-export.

Hey @cristianvasquez, happy new year. 😃

I've just published v0.5.0 which includes a new option, --no-recursive-embeds, to skip processing files more than once within the same note. I believe this will give you the behavior you were looking for.

from obsidian-export.

happy new year to you @zoni ! Thanks for the new option :)

from obsidian-export.