jgru/consult-org-roam

Question: purpose of consult-org-roam-file-find

Closed this issue · 3 comments

Thanks for this package! All of the provided functionality looks very useful, but I'm a little confused about the purpose of consult-org-roam-file-find. It seems to do the same thing as the normal org-roam-node-find with consult-org-roam-mode turned on; both of them search through the org-roam notes with consult completing. Just want to understand if I'm missing anything.

jgru commented

Hi @alberfgu,

Thanks for this package! All of the provided functionality looks very
useful,

Thank you, good to hear.

but I'm a little confused about the purpose of
consult-org-roam-file-find. It seems to do the same thing as the
normal org-roam-node-find with consult-org-roam-mode turned on;
both of them search through the org-roam notes with consult
completing. Just want to understand if I'm missing anything.

consult-org-roam-file-find searches, previews and opens files within the org-roam-directory.

org-roam-node-find with consult-org-roam-mode searches and previews single org-roam-nodes. Since there might be several nodes within one (potentially) large org-file this provides a more precise search and resembles org-roam-node-find's functionality.

If you have any further questions, do not hesitate to pose them.

HTH.

Best regards,
jgru

Thanks for the clarification, that's what I suspected. Since I generally don't use nodes for headings (i.e. one per file) they seem functionally equivalent to me. As a minor suggestion, the UI for consult-org-roam-file-find is a little clunkier because it keeps the full file path instead of the relative path to the org-roam-directory:

image

vs. org-roam-node-find:
image

Anyways, thanks again for the package, it's working really well!

jgru commented

Hi @albertfgu,

happy to hear that you like the package! Displaying the path is needed to use the function consult--file-preview. I opened a corresponding issue #16 to enhance this function. We would need to use an alist mapping filenames to full paths I suppose.