[ENG-1450] Update wikilink

[trangdoan982]

https://www.loom.com/share/72dcd8692629433f984358bb4e86b2b4

---

[linear]

ENG-1450 Translate all wikilinks into relative path

[supabase]

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.

---

Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

[maparent]

Ok good work in general, one definite small change, and a bigger one MAYBE.
We now have another layer of nesting:
import/<vaultname>/discourseGraphs/<node>.md.
the discourseGraphs folder was not part of the structure before.
I think that was good for users, but I do realize it creates a risk of collision, and it makes path negotiation more complex. Maybe to bring to the group?

[graphite-app]

Variable shadowing bug: linkText is redeclared, overwriting the original markdown link display text parameter from line 691. This causes markdown links to lose their original display text and use the file path instead.

// Should be:
const linkPath = getRelativeLinkPath(resolvedFile.path);
if (linkText) {
  return `[${linkText}](${encodePathForMarkdownLink(linkPath)})`;
}
return `[${linkPath}](${encodePathForMarkdownLink(linkPath)})`;

The pattern at lines 694-700 and 710-714 correctly preserves the original linkText parameter while computing a new linkPath variable.

Suggested change

	const linkText = getRelativeLinkPath(resolvedFile.path);
	if (linkText) {
	return `[${linkText}](${encodePathForMarkdownLink(linkText)})`;
	}
	return `[[${linkText}]]`;
	const linkPath = getRelativeLinkPath(resolvedFile.path);
	if (linkPath) {
	return `[${linkText}](${encodePathForMarkdownLink(linkPath)})`;
	}
	return `[[${linkPath}]]`;

Spotted by Graphite Agent



Is this helpful? React 👍 or 👎 to let us know.

[devin-ai-integration]

Devin Review found 1 new potential issue.

View 12 additional findings in Devin Review.

[maparent]

Re earlier point:

We now have another layer of nesting: import/<vaultname>/discourseGraphs/<node>.md. the discourseGraphs folder was not part of the structure before. I think that was good for users, but I do realize it creates a risk of collision, and it makes path negotiation more complex. Maybe to bring to the group?

Was brought to group, new approach was accepted.