Greptile Summary

This PR fixes hunk session ... --repo <path> selectors failing when the path is a subdirectory or sits behind a symlink. Sessions register under their VCS toplevel (canonicalized via realpathSync), so a --repo path pointing anywhere inside the tree never matched the registered root.

• Introduces resolveRepoSelectorRoot, which calls detectVcs to walk up from the given path to the VCS root and then canonicalizes through symlinks with realpathSync.native, mirroring how the session's own repoRoot is stored.
• Both resolveExplicitSessionSelector (used by session get/review) and resolveReloadSelector now use this helper, replacing the old bare resolve() calls.
• Three new tests cover subdirectory resolution, symlink canonicalization, and the reload code path; the CHANGELOG entry is properly added.

Confidence Score: 5/5

Safe to merge — the change is narrowly scoped to path canonicalization before session lookup and cannot affect session data or VCS operations.

The new helper is small, well-commented, and has a clean fallback. detectVcs is synchronous and only returns a repoRoot when the directory is confirmed to exist on disk, so the subsequent realpathSync.native call is safe. Three targeted tests cover the exact scenarios described in the bug report. No observable behavior change for callers using already-canonical repo roots.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["--repo <path> provided"] --> B["resolve(repoPath)\n(absolute path, no symlink resolution)"]
    B --> C["detectVcs(resolved)\n(walk up looking for .git / .jj / .sl)"]
    C -->|"VCS root found"| D["realpathSync.native(repoRoot)\n(canonicalize through symlinks)"]
    C -->|"No VCS root"| E["return resolved\n(original absolute path, no canonicalization)"]
    D --> F["selector: { repoRoot: canonicalRoot }"]
    E --> F
    F --> G["Match against registered session\n(session repoRoot is also realpath-canonical)"]

_{Reviews (1): Last reviewed commit: "fix(session): resolve --repo selector to..." | Re-trigger Greptile}