fs.realpathSync(vaultPath) recomputed once per wikilink
Maintainers usually reply within 1 day
A pull request for this has already been merged.
- #452 by @Kuldeep2822k — merged
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 82/100
- Issue type
- Refactor
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- nodejs, typescript
- Domain
- backend
Research direction
Start in src/storage/wikilink.ts: resolveWikilinkTarget (around line 207) calls realpathSync(vaultPath) on every call, and resolveWikilinkRoadmap (around line 382) loops over sections and links. Read both functions to see how the vault root reaches the target resolver. Done means the vault root is canonicalized once per roadmap resolution, the resolved root is passed in as a parameter, and existing wikilink tests still pass.
Written by the indexing model from the issue text.
Description
Location: src/storage/wikilink.ts:207 (from resolveWikilinkRoadmap :382)
resolveWikilinkTarget calls realpathSync(vaultPath) on every invocation; resolveWikilinkRoadmap loops all sections x links, so the vault root is canonicalized N times for an N-link roadmap.
Fix: Hoist realpathSync(vaultPath) into resolveWikilinkRoadmap and pass the resolved root in.
From the 2026-10-10 codebase audit sweep (CODEBASE_AUDIT.md).
- Dominant language
- TypeScript
- Stars
- 1
- Forks
- 0
- Avg merge
- 21h 20m
- Merged PRs (30d)
- 176
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from Kuldeep2822k/cli
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Kuldeep2822k/cli#550 ·
Maintainers usually reply within 1 day
-
Fix: lapses string-concatenates on each review instead of throwingPossibly taken @Kuldeep2822k claimed this 2 days ago. Openbug core priority: medium
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Kuldeep2822k/cli#547 ·
Maintainers usually reply within 1 day
-
Fix: non-interactive session start prints success but creates no sessionPossibly taken @Kuldeep2822k claimed this 1 day ago. Openbug cli priority: medium
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Kuldeep2822k/cli#546 ·
Maintainers usually reply within 1 day
-
Fix: clearActiveSession skips the containment guardPossibly taken @Kuldeep2822k claimed this 1 day ago. Openbug priority: medium security
Difficulty 2/5 1-3 hours Newbie friendliness 83/100
Kuldeep2822k/cli#544 ·
Maintainers usually reply within 1 day
-
Fix: loader coerces blank numeric frontmatter to 0 instead of documented defaultsPossibly taken @Kuldeep2822k claimed this 1 day ago. Openbug priority: medium storage
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
Kuldeep2822k/cli#543 ·
Maintainers usually reply within 1 day
All issues in Kuldeep2822k/cli
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
scaleway/ultraviolet#7089 ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
tencent-connect/dsh-qqbot#81 ·
Maintainers usually reply within 1 day
-
[Bug]: Pen taps on sidebar rows start a drag and lose their clickPossibly taken @makhnatkin claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
clangd/vscode-clangd#1009 ·