fix: resolve the checkout root correctly inside a git worktree

[josemarluedke]

Edit URLs are wrong for anyone running Docfy from a linked git worktree.

getRepoEditUrl computes path.relative(getRepoInfo(root).root, root). git-repo-info's own typings document the trap:

root: The root directory for the Git repo or submodule. If in a worktree, this is the directory containing the original copy, not the worktree.

So the relative path picks up the worktree's location and the URL points at a path that doesn't exist on the remote:

git-repo-info .root          : /Users/me/code/docfy            ← main checkout
git rev-parse --show-toplevel: /Users/me/code/docfy/.wt/branch ← actual worktree
→ relative = .wt/branch/docs   instead of   docs

This already caused damage

Four committed .gjs templates shipped with .claude/worktrees/<branch>/ baked into their @editUrl values — every "Edit this page" link on those demo pages 404'd until #214 fixed them by hand.

It is also the real reason repo-info.test.ts and generating-edit-url.test.ts fail locally. I spent most of the unified 11 work describing those as environmental noise and repeating that in PR descriptions. They were this bug, and treating them as noise is what let the bad templates through.

The fix

A worktree's metadata directory holds a gitdir file pointing at that worktree's .git file, whose parent is the top level we want.

One subtlety worth flagging for review: detecting a worktree requires comparing worktreeGitDir against commonGitDir, not checking worktreeGitDir for truthiness. It is always populated — outside a worktree it simply equals commonGitDir. My first pass got this wrong and would have taken the correction path always.

Regression test

CI runs in a normal checkout, so this fix could regress invisibly there. The new test builds a real repository and worktree in a temp directory and asserts both resolve identically.

I verified it actually catches the bug rather than just passing:

	with fix	without fix
edit url from the worktree	…/edit/main/docs/{filepath}	…/edit/**linked**/docs/{filepath} ❌
edit url from the main checkout	…/edit/main/docs/{filepath}	…/edit/main/docs/{filepath}

The second row matters as much as the first — it shows the fix doesn't alter normal-checkout behaviour.

The test uses fs.realpathSync on the temp dir because os.tmpdir() is a symlink on macOS and git records the resolved path; comparing against the unresolved one would fail there but pass on Linux.

Result

@docfy/core now passes 59/59 across 12 files in a worktree, where 6 tests in 3 files used to fail. And building test-app-vite from a worktree no longer rewrites the committed templates — removing the manual path-stripping step #215 needed three separate times.

Also verified: pnpm -r run compile clean, @docfy/ember-vite 60/60, @docfy/plugin-with-prose 2/2, both app builds succeed, lint and prettier clean.

🤖 Generated with Claude Code